Skip to content

Issue #25 PR B+C: CLI list adapter, isolate file-spawn - #29

Merged
atherdon merged 2 commits into
mainfrom
feat/issue-25-pr-b-c
Sep 29, 2026
Merged

atherdon merged 2 commits into
mainfrom
feat/issue-25-pr-b-c

Conversation

@atherdon

Copy link
Copy Markdown
Contributor

Follow-up to #26 / issue #25.

PR B

  • Removed TemplateGenerator.listTemplateFiles()
  • CLI --list calls adapters/list-files + markup-generator.pathExists
  • Engine lists catalog ids via ids()
  • Tests for directory scan live in adapters/list-files.test.ts

PR C

  • src/render.ts no longer imports fs or the spawn adapter
  • File entries only work when generator injects fileRenderer (allowFileTemplates: true)
  • Spawn + temp files moved to src/adapters/file-renderer.ts
  • Deleted src/render/file-adapter.ts

rg node:fs src/generator.ts src/render.ts should be empty.
node:fs / spawnSync remain in src/adapters/file-renderer.ts (temp payload + child process) and src/cli.ts no longer uses node:fs.

@atherdon
atherdon merged commit 0696753 into main Sep 29, 2026
1 of 2 checks passed
@atherdon
atherdon deleted the feat/issue-25-pr-b-c branch October 12, 2026 05:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant