Make extracted CLI modules the only source of truth

openspec/changes/agent-codex-dedupe-cli-main-context-output-scaffold-2026-04-22-14-29/proposal.md

@@ -0,0 +1,17 @@
+## Why
+
+- `src/cli/main.js` still mirrors constants and helper implementations that already exist in `src/context.js`, `src/output/index.js`, and `src/scaffold/index.js`.
+- Those mirrors already drifted: `MANAGED_GITIGNORE_PATHS`, `CLI_COMMAND_DESCRIPTIONS`, and `MAINTAINER_RELEASE_REPO` disagree across modules, so whichever copy a caller reaches changes behavior.
+- The current follow-up should stay mechanical: delete dead/duplicate helpers, make the extracted modules the only source of truth, and lock the drift cases with focused tests.
+
+## What Changes
+
+- Import shared constants/session helpers from `src/context.js` instead of redefining them in `src/cli/main.js`.
+- Import presentation helpers from `src/output/index.js` and scaffold/file-install helpers from `src/scaffold/index.js`, then delete the local mirrors in `src/cli/main.js`.
+- Remove dead or duplicated helpers in `src/cli/main.js`, including the duplicate `gitRefExists`, duplicate auto-finish failure log, unused command handlers, unused prompt helper, and redundant truthy-flag wrapper.
+- Add focused regression coverage for the shared-source ownership and the concrete drift cases (`--current` help text, `.vscode` gitignore exceptions, and maintainer release repo resolution).
+
+## Impact
+
+- Primary files: `src/cli/main.js`, `src/context.js`, `src/output/index.js`, `src/scaffold/index.js`, and `test/cli-args-dispatch.test.js`.
+- Main risk is behavior drift while deleting local copies, so verification stays focused on syntax plus targeted CLI regression suites before broader checks.

openspec/changes/agent-codex-dedupe-cli-main-context-output-scaffold-2026-04-22-14-29/specs/cli-modularization/spec.md

@@ -0,0 +1,21 @@
+## MODIFIED Requirements
+
+### Requirement: Module seams mirror operational responsibility
+The CLI SHALL keep shared helper ownership in the extracted `src/` modules instead of duplicating the same constants or helper implementations in `src/cli/main.js`.
+
+#### Scenario: Shared context/output/scaffold seams stay single-source
+- **WHEN** maintainers inspect `src/cli/main.js`
+- **THEN** shared constants and session helpers are imported from `src/context.js`
+- **AND** presentation helpers are imported from `src/output/index.js`
+- **AND** scaffold/file-install helpers are imported from `src/scaffold/index.js`
+- **AND** `src/cli/main.js` does not redefine those helpers locally.
+
+### Requirement: Refactor preserves targeted CLI behavior
+The modularization SHALL preserve the current command surface for targeted verified flows while deleting the local duplicate helpers.
+
+#### Scenario: Shared-source drift cases remain stable after cleanup
+- **WHEN** focused CLI regression suites are run after the helper cleanup
+- **THEN** setup and doctor help continue advertising `--current`
+- **AND** managed gitignore handling preserves the `.vscode` exceptions required for shared settings
+- **AND** release gating resolves the default maintainer repo to the package root instead of the `src/` directory
+- **AND** syntax/require-time failures do not occur from duplicate helper definitions.

openspec/changes/agent-codex-dedupe-cli-main-context-output-scaffold-2026-04-22-14-29/tasks.md

@@ -0,0 +1,40 @@
+## Definition of Done
+
+This change is complete only when all of the following are true:
+
+- Every checkbox below is checked.
+- The agent branch reaches `MERGED` state on `origin` and the PR URL + state are recorded in the completion handoff.
+- If any step blocks, add a `BLOCKED:` line under section 4 and stop.
+
+## Handoff
+
+- Handoff: change=`agent-codex-dedupe-cli-main-context-output-scaffold-2026-04-22-14-29`; branch=`agent/codex/dedupe-cli-main-context-output-scaffold-2026-04-22-14-29`; scope=`src/cli/main.js`, `src/context.js`, `src/output/index.js`, `src/scaffold/index.js`, `test/cli-args-dispatch.test.js`; action=`delete remaining shared-helper duplication from src/cli/main.js and keep context/output/scaffold as the only live source of truth`.
+
+## 1. Specification
+
+- [x] 1.1 Capture the mechanical cleanup scope and acceptance criteria for the remaining shared helper duplication.
+- [x] 1.2 Add a `cli-modularization` spec delta covering context/output/scaffold single-source ownership and the known drift cases.
+
+## 2. Implementation
+
+- [x] 2.1 Add focused regression coverage for shared-source ownership and the known drift cases before editing the cleanup targets.
+- [x] 2.2 Move duplicated constants/session helpers in `src/cli/main.js` to `src/context.js` imports and reconcile `MANAGED_GITIGNORE_PATHS`.
+- [x] 2.3 Move duplicated presentation helpers in `src/cli/main.js` to `src/output/index.js` imports.
+- [x] 2.4 Move duplicated scaffold/file-install helpers in `src/cli/main.js` to `src/scaffold/index.js` imports and make scaffold reuse `context`'s `toDestinationPath`.
+- [x] 2.5 Remove dead or duplicate helpers from `src/cli/main.js` (`installMany`, `initWorkspace`, `doctorAudit`, `syncDoctorLocalSupportFiles`, `promptYesNo`, `envFlagEnabled`, one `gitRefExists`, and the duplicate auto-finish failure log).
+
+## 3. Verification
+
+- [x] 3.1 Run `node --check src/cli/main.js src/context.js src/output/index.js src/scaffold/index.js`.
+- [x] 3.2 Run `node --test test/cli-args-dispatch.test.js`.
+- [x] 3.3 Run focused CLI regression suites that cover setup/doctor/install surfaces touched by the cleanup.
+- [x] 3.4 Run `openspec validate agent-codex-dedupe-cli-main-context-output-scaffold-2026-04-22-14-29 --type change --strict`.
+- [x] 3.5 Run `openspec validate --specs`.
+
+Verification note: `node --check src/cli/main.js src/context.js src/output/index.js src/scaffold/index.js`, `node --test test/cli-args-dispatch.test.js`, `node --test test/cli-args-dispatch.test.js test/setup.test.js test/doctor.test.js test/install.test.js test/metadata.test.js`, and `npm test` all passed after the cleanup. `openspec validate agent-codex-dedupe-cli-main-context-output-scaffold-2026-04-22-14-29 --type change --strict` exited `0`, and `openspec validate --specs` exited `0` with `No items found to validate`.
+
+## 4. Cleanup
+
+- [ ] 4.1 Run `gx branch finish --branch agent/codex/dedupe-cli-main-context-output-scaffold-2026-04-22-14-29 --base main --via-pr --wait-for-merge --cleanup`.
+- [ ] 4.2 Record PR URL and final merge state (`MERGED`) in the completion handoff.
+- [ ] 4.3 Confirm the sandbox worktree is removed and no local/remote refs remain for the branch.