Legacy CLI cutover and removal

[SorraTheOrc]

Summary: Complete migration by enforcing framework-only command routing, removing legacy parser/dispatcher code, and validating full parity gates.

User Story

As a maintainer, I can rely on a single CLI framework implementation without dual paths, reducing regression risk and maintenance overhead.

Expected UX Change

• User experience: no command behavior surprises from mixed routing paths.
• Player experience: deterministic audio output remains stable due to parity-gated cutover.

Acceptance Criteria

• All user-facing commands are implemented in yargs and no longer depend on legacy parse/dispatch paths.
• A command coverage matrix confirms framework ownership of every command/subcommand.
• Full CLI test suite passes, including text, JSON, and integration parity checks.
• Legacy code paths (src/cli.ts parser/dispatcher and compatibility shims) are removed or reduced to non-behavioral compatibility exports only.
• Migration notes/changelog document the cutover and any intentional deltas.

Minimal Implementation

• Add a migration gate/checklist for command coverage in CI.
• Flip default routing to framework-only and remove command-level fallbacks.
• Delete legacy parsing/dispatch code after all feature migrations are complete.
• Finalize docs and rollback guidance for maintainers.

Dependencies

• blocked-by:TF-0MMJ892CE0Y4MFFL
• blocked-by:TF-0MMJ894WA1IHEKV9
• blocked-by:TF-0MMJ897F20HTCQHE
• blocked-by:TF-0MMJ899TT0RPVHEG
• blocked-by:TF-0MMJ89CGG1W2LXI9

Deliverables

• Removed legacy parser/dispatcher code.
• CI/parity gate artifacts and passing pipeline.
• Final migration notes and release/changelog update.

[SorraTheOrc]

\n\n**OpenCode**\n\nImplementation progress: switched canonical CLI entrypoint to framework ( now delegates to ), moved legacy parser/dispatcher into , simplified compatibility shim ( re-exports canonical entrypoint), added framework command ownership export in , and added cutover matrix/checklist docs at with README link. Validation: npm test passed (112 files, 2356 tests). Commit hash: not committed yet.

[SorraTheOrc]

\n\n**OpenCode**\n\nImplementation progress: switched canonical CLI entrypoint to framework (src/cli.ts now delegates to yargsMain), moved legacy parser/dispatcher into src/cli.legacy.ts, simplified compatibility shim (src/compat/cli.ts re-exports canonical entrypoint), added framework command ownership export in src/cli.yargs.ts, and added cutover matrix/checklist docs at docs/cli-cutover-checklist.md with README link. Validation: npm test passed (112 files, 2356 tests). Commit hash: not committed yet.

[SorraTheOrc]

\n\n**OpenCode**\n\nCommitted cutover changes in 734d2b7 with message: TF-0MMJOJPE914LR5F6: cut over canonical CLI to yargs. Files: src/cli.ts, src/cli.legacy.ts, src/cli.yargs.ts, src/cli.yargs.integration.test.ts, src/compat/cli.ts, docs/cli-cutover-checklist.md, README.md. Summary: canonical entrypoint now routes through yargs, legacy parser moved to dedicated legacy module, compatibility shim simplified, and cutover ownership matrix/checklist documented. Validation: npm test passed (112 files, 2356 tests).

[SorraTheOrc]

\n\n**OpenCode**\n\nCommitted and pushed b824de2: TF-0MMJOJPE914LR5F6: remove legacy CLI shim and add yargs types. Files updated include CLI core rename (), yargs delegation updates (), canonical entrypoint update (), removal of compatibility shim (), test import updates to canonical CLI, docs updates (, ), and TypeScript yargs typings (, ). Validation: npm run typecheck and npm test both pass.

[SorraTheOrc]

\n\n**OpenCode**\n\nCommitted and pushed b824de2: TF-0MMJOJPE914LR5F6: remove legacy CLI shim and add yargs types. Files updated include CLI core rename (src/cli.core.ts), yargs delegation updates (src/cli.yargs.ts), canonical entrypoint update (src/cli.ts), removal of compatibility shim (src/compat/cli.ts), test import updates to canonical CLI, docs updates (docs/cli-cutover-checklist.md, docs/cli-library-evaluation.md), and TypeScript yargs typings (package.json, package-lock.json). Validation: npm run typecheck and npm test both pass.

[SorraTheOrc]

\n\n**OpenCode**\n\nPR created: https://github.com//pull/238\nIncludes commits 734d2b7 and b824de2 for canonical yargs cutover, compatibility shim removal, docs updates, and yargs typing fixes. Validation on branch: npm run typecheck and npm test passed.

[SorraTheOrc]

\n\n**OpenCode**\n\nPlan: changelog\n- Added/reused feature children for completion of yargs migration:\n - null Native Core Command Execution\n - null Native Advanced Command Execution\n - null Legacy Runtime Detachment and Removal\n - null Cutover Gates and Release Handoff\n- Added dependency chain: null -> null, null -> (null,null), null -> null\n- Linked cleanup task TF-0MMJVDBHQ0YWQBY2 to migration-completion feature via dependency: TF-0MMJVDBHQ0YWQBY2 -> null\n- This decomposition supersedes the prior single-child structure and keeps existing items reusable/idempotent.

[SorraTheOrc]

\n\n**OpenCode**\n\nPlanning Complete. Approved feature decomposition for yargs migration completion:\n- Native Core Command Execution (TF-0MMKEK8ZY0HWS138)\n- Native Advanced Command Execution (TF-0MMKEK9D801SPYR7)\n- Legacy Runtime Detachment and Removal (TF-0MMKEK9QH0V0E0HI)\n- Cutover Gates and Release Handoff (TF-0MMKEKA3V1N2HN6C)\nDependency chain: TF-0MMKEK9D801SPYR7 -> TF-0MMKEK8ZY0HWS138; TF-0MMKEK9QH0V0E0HI -> (TF-0MMKEK8ZY0HWS138, TF-0MMKEK9D801SPYR7); TF-0MMKEKA3V1N2HN6C -> TF-0MMKEK9QH0V0E0HI.\nLinked cleanup dependency: TF-0MMJVDBHQ0YWQBY2 -> TF-0MMKEK9QH0V0E0HI.\nOpen questions retained: whether compile is in-scope for this cutover; tolerance for wording drift in human-facing help/errors when JSON and exit-code semantics remain stable.