create-quiver 0.15.1 → 0.15.2

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-04-revise-input-guardrails/slice.json

@@ -0,0 +1,73 @@
+{
+  "slice_id": "slice-04-revise-input-guardrails",
+  "ticket": "QUIVER-33-04",
+  "type": "fix",
+  "title": "Revise input guardrails",
+  "objective": "Handle missing, incomplete, nonexistent, and accidental extra `ai revise --input` usage before provider execution.",
+  "description": "Hardens `ai revise` for both acceptance and technical-plan phases so incomplete inputs fail actionably or guide the user in TTY without silently ignoring mistakes.",
+  "git": {
+    "branch_type": "fix",
+    "base_branch": "main",
+    "branch_slug": "v33-revise-input-guardrails",
+    "branch_name": "fix/QUIVER-33-04-v33-revise-input-guardrails"
+  },
+  "files": [
+    "src/create-quiver/index.js",
+    "src/create-quiver/commands/ai.js",
+    "tests/commands/ai-plan.test.js",
+    "tests/commands/cli-contract.test.js"
+  ],
+  "expected_read_paths": [
+    "src/create-quiver/index.js",
+    "src/create-quiver/commands/ai.js",
+    "tests/commands/ai-plan.test.js",
+    "tests/commands/cli-contract.test.js",
+    "SPEC.md"
+  ],
+  "allowed_write_paths": [
+    "src/create-quiver/index.js",
+    "src/create-quiver/commands/ai.js",
+    "tests/commands/ai-plan.test.js",
+    "tests/commands/cli-contract.test.js",
+    "specs/quiver-v33-approval-ux-and-planner-progress/**"
+  ],
+  "depends_on": [
+    "slice-01-approval-candidates-model"
+  ],
+  "parallel_safe": "yes",
+  "parallel_safe_reason": "Can run after slice-01, but coordinate command-file conflicts with slice-02 and slice-05.",
+  "must": [
+    "Detect `ai revise --phase acceptance --input` without a value.",
+    "Detect `ai revise --phase technical-plan --input` without a value.",
+    "Fail before provider execution for missing or nonexistent files.",
+    "Handle accidental trailing args such as `s` with a clear error or documented parse behavior.",
+    "In no-TTY/CI, show an explicit corrected command.",
+    "In TTY, show focused guidance or selector only if useful artifacts exist."
+  ],
+  "not_included": [
+    "Generating feedback files automatically.",
+    "Running providers when input is invalid.",
+    "Changing revise prompt semantics."
+  ],
+  "acceptance": [
+    "`ai revise --phase acceptance --input` fails before provider execution with a useful next command.",
+    "`ai revise --phase technical-plan --input` fails before provider execution with a useful next command.",
+    "Nonexistent feedback input fails before provider execution.",
+    "Accidental extra args are not silently accepted.",
+    "No-TTY/CI does not prompt.",
+    "Existing valid revise commands still work."
+  ],
+  "tests": [
+    "node --test tests/commands/ai-plan.test.js tests/commands/cli-contract.test.js",
+    "git diff --check"
+  ],
+  "validation_hints": [
+    "Cover parser-level missing value behavior and command-level missing file behavior separately.",
+    "Avoid provider mocks for invalid input paths because provider execution should not start."
+  ],
+  "estimated_hours": 6,
+  "status": "completed",
+  "blocked_reason": null,
+  "actual_hours": 6,
+  "completed_at": "2026-05-28"
+}

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-05-provider-progress-alignment/CLOSURE_BRIEF.md

@@ -0,0 +1,20 @@
+# CLOSURE_BRIEF - slice-05 Provider progress alignment
+
+## Summary
+
+Completed provider progress alignment for planner/reviewer flows. `ai plan`, `ai revise`, `ai review-plan`, and `ai repair-plan` now use real preparation checks and provider spinner behavior in human TTY mode.
+
+## Validation
+
+- [x] Planner progress tests pass.
+- [x] Review-plan progress tests pass.
+- [x] Repair-plan progress tests pass.
+- [x] Machine-output cleanliness tests pass.
+
+## Pending
+
+- None.
+
+## Remaining Risks
+
+- None critical; final smoke and package readiness passed.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-05-provider-progress-alignment/EXECUTION_BRIEF.md

@@ -0,0 +1,51 @@
+# EXECUTION_BRIEF - slice-05 Provider progress alignment
+
+## Context
+
+`ai prepare-context --with-planner` provides the UX baseline. Planner and reviewer commands must look alive during real provider execution without affecting machine output.
+
+## Objective
+
+Align progress/loaders for `ai plan`, `ai revise`, `ai review-plan`, and `ai repair-plan`, and audit existing long-running flows.
+
+## Scope
+
+- `src/create-quiver/commands/ai.js`
+- existing CLI UX helpers if needed
+- focused progress tests
+
+## Acceptance Criteria
+
+- Live provider-backed planner/reviewer commands show progress in human TTY.
+- Dry-run and print-prompt do not show loaders.
+- JSON/no-TTY/CI/no-color stay clean.
+- Provider failure stops the spinner.
+- Existing compliant flows are protected by regression tests or noted.
+
+## Technical Plan Summary
+
+Reuse `createCommandUx`, `shouldShowHumanProgress`, `writeProgressChecks`, and `runProviderWithProgress`. Avoid new abstractions unless real duplication appears.
+
+## Suggested Steps
+
+1. Audit current progress usage.
+2. Add missing progress stages.
+3. Add tests with injected spinner/UX helpers.
+4. Add no-color/no-TTY regression coverage where gaps exist.
+
+## Restrictions
+
+- Do not change provider prompts.
+- Do not add progress to JSON output.
+- Do not mark fake work as completed.
+
+## Risks
+
+- Progress output can be hard to test if implementation is coupled to terminal rendering.
+
+## Completion Checklist
+
+- [ ] Planner progress covered.
+- [ ] Review-plan progress covered.
+- [ ] Repair-plan progress covered.
+- [ ] Machine-output cleanliness covered.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-05-provider-progress-alignment/slice.json

@@ -0,0 +1,91 @@
+{
+  "slice_id": "slice-05-provider-progress-alignment",
+  "ticket": "QUIVER-33-05",
+  "type": "feature",
+  "title": "Provider progress alignment",
+  "objective": "Align progress messages and loaders for provider-backed planner and reviewer commands.",
+  "description": "Ensures `ai plan`, `ai revise`, `ai review-plan`, and `ai repair-plan` use the same production-safe progress contract as `ai prepare-context --with-planner`, while auditing existing long-running flows.",
+  "git": {
+    "branch_type": "feature",
+    "base_branch": "main",
+    "branch_slug": "v33-provider-progress-alignment",
+    "branch_name": "feature/QUIVER-33-05-v33-provider-progress-alignment"
+  },
+  "files": [
+    "src/create-quiver/commands/ai.js",
+    "src/create-quiver/lib/ai/executor.js",
+    "src/create-quiver/lib/ai/execution-plan.js",
+    "src/create-quiver/lib/cli/ux.js",
+    "tests/commands/ai-plan.test.js",
+    "tests/commands/ai-review-plan.test.js",
+    "tests/commands/ai-prepare-context-planner.test.js",
+    "tests/commands/ai-execute-slice.test.js",
+    "tests/commands/ai-execute-plan.test.js",
+    "tests/commands/ai-pr.test.js",
+    "tests/lib/cli-ux.test.js"
+  ],
+  "expected_read_paths": [
+    "src/create-quiver/commands/ai.js",
+    "src/create-quiver/lib/ai/executor.js",
+    "src/create-quiver/lib/ai/execution-plan.js",
+    "src/create-quiver/lib/cli/ux.js",
+    "tests/commands/ai-prepare-context-planner.test.js",
+    "tests/lib/cli-ux.test.js",
+    "SPEC.md"
+  ],
+  "allowed_write_paths": [
+    "src/create-quiver/commands/ai.js",
+    "src/create-quiver/lib/ai/executor.js",
+    "src/create-quiver/lib/ai/execution-plan.js",
+    "src/create-quiver/lib/cli/ux.js",
+    "tests/commands/ai-plan.test.js",
+    "tests/commands/ai-review-plan.test.js",
+    "tests/commands/ai-prepare-context-planner.test.js",
+    "tests/commands/ai-execute-slice.test.js",
+    "tests/commands/ai-execute-plan.test.js",
+    "tests/commands/ai-pr.test.js",
+    "tests/lib/cli-ux.test.js",
+    "specs/quiver-v33-approval-ux-and-planner-progress/**"
+  ],
+  "depends_on": [
+    "slice-01-approval-candidates-model"
+  ],
+  "parallel_safe": "yes",
+  "parallel_safe_reason": "Can run after slice-01, but command-file conflicts with approval slices should be coordinated.",
+  "must": [
+    "Show human progress for `ai plan --phase acceptance` live provider execution.",
+    "Show human progress for `ai plan --phase technical-plan` live provider execution.",
+    "Show human progress for `ai revise` live provider execution through the shared plan path.",
+    "Show human progress for `ai review-plan` live provider execution.",
+    "Show human progress for `ai repair-plan` live provider execution.",
+    "Do not show loaders for dry-run or print-prompt.",
+    "Do not contaminate JSON, no-color, CI, or no-TTY output.",
+    "Audit existing long-running flows and add regression tests only where behavior should remain unchanged."
+  ],
+  "not_included": [
+    "Changing provider prompts.",
+    "Changing provider adapters.",
+    "Adding new UX dependencies."
+  ],
+  "acceptance": [
+    "Planner commands show real staged progress in human mode.",
+    "Reviewer and repair-plan commands show real staged progress in human mode.",
+    "Dry-run and print-prompt remain provider-free and loader-free.",
+    "Provider failure stops spinner and surfaces actionable context.",
+    "Existing prepare-context, onboard, execute-slice, execute-plan, and pr progress behavior is covered by regression tests or explicitly documented as already compliant."
+  ],
+  "tests": [
+    "node --test tests/commands/ai-plan.test.js tests/commands/ai-review-plan.test.js tests/commands/ai-prepare-context-planner.test.js tests/lib/cli-ux.test.js",
+    "node --test tests/commands/ai-execute-slice.test.js tests/commands/ai-execute-plan.test.js tests/commands/ai-pr.test.js",
+    "git diff --check"
+  ],
+  "validation_hints": [
+    "Use injected spinner helpers in tests.",
+    "Avoid fake progress checks that claim completed work before it happens."
+  ],
+  "estimated_hours": 8,
+  "status": "completed",
+  "blocked_reason": null,
+  "actual_hours": 8,
+  "completed_at": "2026-05-28"
+}

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-06-workflow-surface-integration/CLOSURE_BRIEF.md

@@ -0,0 +1,20 @@
+# CLOSURE_BRIEF - slice-06 Workflow surface integration
+
+## Summary
+
+Completed workflow surface integration through shared approval-candidate helpers. `ai approvals`, `flow`, `ai status`, `ai resume`, and `spec create --interactive` now align on recommended approval versions and blocked review guidance.
+
+## Validation
+
+- [x] Approval status tests pass.
+- [x] Flow tests pass.
+- [x] Run-state tests pass.
+- [x] Spec create tests pass.
+
+## Pending
+
+- None.
+
+## Remaining Risks
+
+- None critical; drift-prone surfaces are covered by focused tests and full suite.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-06-workflow-surface-integration/EXECUTION_BRIEF.md

@@ -0,0 +1,53 @@
+# EXECUTION_BRIEF - slice-06 Workflow surface integration
+
+## Context
+
+After approval selection and review-decision data exist, other workflow surfaces must consume the same source of truth to prevent contradictory next steps.
+
+## Objective
+
+Integrate shared approval candidates into `ai approvals`, `flow`, `ai status`, `ai resume`, and `spec create --interactive`.
+
+## Scope
+
+- `src/create-quiver/commands/ai.js`
+- `src/create-quiver/commands/flow.js`
+- `src/create-quiver/commands/spec.js`
+- `src/create-quiver/lib/ai/run-state.js`
+- focused tests
+
+## Acceptance Criteria
+
+- `ai approvals` and `ai approve` recommend the same current version.
+- `flow` points to safe next steps consistent with candidates.
+- `ai status` and `ai resume` do not contradict approval guidance.
+- `spec create --interactive` shows reviewed and approved technical-plan input clearly.
+- Read-only outputs remain prompt-free.
+
+## Technical Plan Summary
+
+Replace duplicated next-command decisions with the shared candidate model where approval phases are involved.
+
+## Suggested Steps
+
+1. Update approval status output.
+2. Update flow decision points.
+3. Update run-state next-command guidance.
+4. Update spec create interactive summary.
+5. Add drift-prevention tests.
+
+## Restrictions
+
+- Do not prompt from read-only commands.
+- Do not change spec generation semantics.
+
+## Risks
+
+- Existing snapshot-style output tests may need precise updates.
+
+## Completion Checklist
+
+- [ ] `ai approvals` integrated.
+- [ ] `flow` integrated.
+- [ ] `ai status/resume` integrated.
+- [ ] `spec create --interactive` integrated.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-06-workflow-surface-integration/slice.json

@@ -0,0 +1,87 @@
+{
+  "slice_id": "slice-06-workflow-surface-integration",
+  "ticket": "QUIVER-33-06",
+  "type": "feature",
+  "title": "Workflow surface integration",
+  "objective": "Use shared approval candidates across status, approval, flow, resume, and spec-create surfaces.",
+  "description": "Integrates the shared decision model into `ai approvals`, `flow`, `ai status`, `ai resume`, and `spec create --interactive` so every surface recommends the same safe next step.",
+  "git": {
+    "branch_type": "feature",
+    "base_branch": "main",
+    "branch_slug": "v33-workflow-surface-integration",
+    "branch_name": "feature/QUIVER-33-06-v33-workflow-surface-integration"
+  },
+  "files": [
+    "src/create-quiver/commands/ai.js",
+    "src/create-quiver/commands/flow.js",
+    "src/create-quiver/commands/spec.js",
+    "src/create-quiver/lib/ai/run-state.js",
+    "src/create-quiver/lib/approvals.js",
+    "tests/commands/ai-run-state.test.js",
+    "tests/commands/flow.test.js",
+    "tests/commands/spec-create.test.js"
+  ],
+  "expected_read_paths": [
+    "src/create-quiver/commands/ai.js",
+    "src/create-quiver/commands/flow.js",
+    "src/create-quiver/commands/spec.js",
+    "src/create-quiver/lib/ai/run-state.js",
+    "src/create-quiver/lib/approvals.js",
+    "tests/commands/ai-run-state.test.js",
+    "tests/commands/flow.test.js",
+    "tests/commands/spec-create.test.js",
+    "SPEC.md"
+  ],
+  "allowed_write_paths": [
+    "src/create-quiver/commands/ai.js",
+    "src/create-quiver/commands/flow.js",
+    "src/create-quiver/commands/spec.js",
+    "src/create-quiver/lib/ai/run-state.js",
+    "src/create-quiver/lib/approvals.js",
+    "tests/commands/ai-run-state.test.js",
+    "tests/commands/flow.test.js",
+    "tests/commands/spec-create.test.js",
+    "specs/quiver-v33-approval-ux-and-planner-progress/**"
+  ],
+  "depends_on": [
+    "slice-01-approval-candidates-model",
+    "slice-02-approve-interactive-selection",
+    "slice-03-technical-plan-review-decision-data"
+  ],
+  "parallel_safe": "no",
+  "parallel_safe_reason": "This slice touches multiple workflow surfaces and should land after core approval behavior is stable.",
+  "must": [
+    "Update `ai approvals` to use shared approval candidate data.",
+    "Update `flow` recommendations to avoid conflicting approval guidance.",
+    "Update `ai status` and `ai resume` next safe commands where approval selection applies.",
+    "Update `spec create --interactive` to show reviewed and approved technical-plan input using shared data.",
+    "Keep read-only/machine-readable surfaces free of prompts.",
+    "Avoid duplicating decision logic in each command."
+  ],
+  "not_included": [
+    "Changing execution-plan logic.",
+    "Changing spec artifact generation semantics.",
+    "Adding new lifecycle phases unless unavoidable."
+  ],
+  "acceptance": [
+    "`ai approvals` shows the same recommended approval version as `ai approve`.",
+    "`flow` recommends approval commands consistent with candidate state.",
+    "`ai status` and `ai resume` do not contradict `ai approvals`.",
+    "`spec create --interactive` identifies the reviewed and approved technical plan clearly.",
+    "JSON and read-only commands remain prompt-free.",
+    "Tests cover drift-prone states."
+  ],
+  "tests": [
+    "node --test tests/commands/ai-run-state.test.js tests/commands/flow.test.js tests/commands/spec-create.test.js tests/commands/ai-review-plan.test.js",
+    "git diff --check"
+  ],
+  "validation_hints": [
+    "Prefer one shared formatting/data helper over duplicated command-specific state checks.",
+    "Keep command output stable enough for existing tests unless the visible contract intentionally changes."
+  ],
+  "estimated_hours": 8,
+  "status": "completed",
+  "blocked_reason": null,
+  "actual_hours": 8,
+  "completed_at": "2026-05-28"
+}

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-07-docs-tests-release-readiness/CLOSURE_BRIEF.md

@@ -0,0 +1,21 @@
+# CLOSURE_BRIEF - slice-07 Docs, tests, and release readiness
+
+## Summary
+
+Completed docs, template alignment, full test validation, smokes, package readiness, evidence, status metadata, and PR body updates for v33.
+
+## Validation
+
+- [x] Docs are aligned.
+- [x] Full tests pass.
+- [x] Smokes pass.
+- [x] Evidence and PR body are updated.
+
+## Pending
+
+- None.
+
+## Remaining Risks
+
+- None critical.
+- This slice does not claim npm publication.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-07-docs-tests-release-readiness/EXECUTION_BRIEF.md

@@ -0,0 +1,55 @@
+# EXECUTION_BRIEF - slice-07 Docs, tests, and release readiness
+
+## Context
+
+This slice closes the spec after implementation slices land. It synchronizes documentation, evidence, and PR readiness.
+
+## Objective
+
+Update docs and evidence, run validation, and prepare the PR for review without claiming package publication.
+
+## Scope
+
+- `docs/CLI_UX_GUIDE.md`
+- `docs/reference/commands.md`
+- `README_FOR_AI.md`
+- generated docs templates if affected
+- changelog/roadmap if local release process requires it
+- spec evidence and closure docs
+
+## Acceptance Criteria
+
+- Public and AI-facing docs match implemented behavior.
+- Generated templates match command guidance.
+- Focused and full tests pass or blockers are explicit.
+- Smoke/package readiness evidence is recorded.
+- Spec status and PR body are updated.
+
+## Technical Plan Summary
+
+Close documentation after behavior is real. Run focused tests first, then full suite and smokes.
+
+## Suggested Steps
+
+1. Update docs and templates.
+2. Run focused command tests.
+3. Run full tests and smokes.
+4. Update evidence and status.
+5. Update PR body.
+
+## Restrictions
+
+- Do not claim npm publication.
+- Do not introduce new behavior beyond documentation/test closure.
+
+## Risks
+
+- Full smokes can be slow or environment-sensitive; record exact blockers if any fail.
+
+## Completion Checklist
+
+- [ ] Docs updated.
+- [ ] Focused tests run.
+- [ ] Full tests/smokes run or blockers recorded.
+- [ ] Evidence report updated.
+- [ ] PR body ready.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-07-docs-tests-release-readiness/slice.json

@@ -0,0 +1,97 @@
+{
+  "slice_id": "slice-07-docs-tests-release-readiness",
+  "ticket": "QUIVER-33-07",
+  "type": "docs",
+  "title": "Docs, tests, and release readiness",
+  "objective": "Close documentation, regression coverage, smoke validation, and release readiness for v33.",
+  "description": "Updates public and AI-facing docs after implementation, runs full tests and smokes, records evidence, and prepares the PR for review.",
+  "git": {
+    "branch_type": "feature",
+    "base_branch": "main",
+    "branch_slug": "v33-docs-tests-release-readiness",
+    "branch_name": "feature/QUIVER-33-07-v33-docs-tests-release-readiness"
+  },
+  "files": [
+    "README_FOR_AI.md",
+    "docs/CLI_UX_GUIDE.md",
+    "docs/reference/commands.md",
+    "docs/COMMANDS.md.template",
+    "docs/AI_ONBOARDING_PROMPT.md.template",
+    "CHANGELOG.md",
+    "ROADMAP.md",
+    "specs/quiver-v33-approval-ux-and-planner-progress/**"
+  ],
+  "expected_read_paths": [
+    "README_FOR_AI.md",
+    "README.md",
+    "docs/CLI_UX_GUIDE.md",
+    "docs/reference/commands.md",
+    "docs/COMMANDS.md.template",
+    "docs/AI_ONBOARDING_PROMPT.md.template",
+    "CHANGELOG.md",
+    "ROADMAP.md",
+    "SPEC.md",
+    "EVIDENCE_REPORT.md"
+  ],
+  "allowed_write_paths": [
+    "README_FOR_AI.md",
+    "README.md",
+    "docs/CLI_UX_GUIDE.md",
+    "docs/reference/commands.md",
+    "docs/COMMANDS.md.template",
+    "docs/AI_ONBOARDING_PROMPT.md.template",
+    "CHANGELOG.md",
+    "ROADMAP.md",
+    "specs/quiver-v33-approval-ux-and-planner-progress/**"
+  ],
+  "depends_on": [
+    "slice-02-approve-interactive-selection",
+    "slice-03-technical-plan-review-decision-data",
+    "slice-04-revise-input-guardrails",
+    "slice-05-provider-progress-alignment",
+    "slice-06-workflow-surface-integration"
+  ],
+  "parallel_safe": "no",
+  "parallel_safe_reason": "Final docs, tests, smoke evidence, and release readiness must close after all implementation slices.",
+  "must": [
+    "Update CLI UX and command reference docs for approval selectors, revise guardrails, and progress behavior.",
+    "Update README_FOR_AI source-of-truth notes.",
+    "Update generated templates if command guidance changes.",
+    "Run focused tests and full test suite.",
+    "Run smoke commands and package readiness checks where feasible.",
+    "Update STATUS.md, EVIDENCE_REPORT.md, CLOSURE_BRIEF.md files, and pr.md with final evidence.",
+    "Avoid claiming package publication until it actually happens."
+  ],
+  "not_included": [
+    "New product behavior beyond closing docs/tests.",
+    "Actual npm publication.",
+    "Post-merge cleanup."
+  ],
+  "acceptance": [
+    "Docs describe TTY approval selection and no-TTY explicit version behavior.",
+    "Docs describe planner progress behavior without implying prompts are mandatory.",
+    "Generated templates remain aligned.",
+    "Focused tests for every changed command pass.",
+    "Full test suite passes or documented blockers are explicit.",
+    "Smoke/package readiness evidence is recorded.",
+    "Spec status and PR body are updated."
+  ],
+  "tests": [
+    "node --test tests/**/*.test.js",
+    "npm run smoke:create-quiver",
+    "npm run smoke:doctor-fixtures",
+    "npm run smoke:guided-workflow",
+    "npm run package:quiver",
+    "git diff --check",
+    "node bin/create-quiver.js spec validate specs/quiver-v33-approval-ux-and-planner-progress"
+  ],
+  "validation_hints": [
+    "If package smoke is too slow or blocked, record the exact blocker and run the maximal focused test subset.",
+    "Do not claim npm release or latest package availability."
+  ],
+  "estimated_hours": 6,
+  "status": "completed",
+  "blocked_reason": null,
+  "actual_hours": 6,
+  "completed_at": "2026-05-28"
+}