Skip to content

Keep validation at entry points and clarify execution flow - #35

Merged
github-actions[bot] merged 1 commit into
mainfrom
refactor/validation-and-flow
Sep 12, 2026
Merged

github-actions[bot] merged 1 commit into
mainfrom
refactor/validation-and-flow

Conversation

@claudecafe

Copy link
Copy Markdown
Collaborator

CLI input validation was shared outside command handlers, and internal types were inferred from runtime schemas. Commands now own their option schemas and validate inputs directly, while authoring, rendering, and publishing use independent TypeScript types.

The refactor also makes execution flow explicit across capture logic, report generation, Worker routes, website interactions, and maintenance scripts: main operations come first, validation has purpose-named functions, and peer operations have explicit branches. Existing input errors, output formatting, report presentation, credentials, and release behavior are preserved.

  • Keep src/cli.ts as the only file directly under src/; organize implementation under domain directories.
  • Move CLI file/config boundaries under src/cli/ and separate format schemas from internal types.
  • Update imports and the architecture source map.

Validation: pnpm check passed (334 tests; 23 visual tests passed, 1 existing skip), pnpm build:worker passed, release JavaScript and infrastructure shell syntax checks passed. Browser smoke checks passed for website copy, animation completion, replay, and skip; local report preview returned HTTP 200.

Draft for owner review because this repository automatically merges non-draft PRs.

Co-Authored-By: ことね <kotone@claudecafe.dev>
@claudecafe
claudecafe marked this pull request as ready for review September 12, 2026 23:46
@github-actions
github-actions Bot enabled auto-merge (rebase) September 12, 2026 23:46
@github-actions
github-actions Bot merged commit bda5ed0 into main Sep 12, 2026
3 of 5 checks passed
@claudecafe
claudecafe deleted the refactor/validation-and-flow branch September 16, 2026 13:13
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.

2 participants