create-quiver 0.15.0 → 0.15.2

package/specs/quiver-v33-approval-ux-and-planner-progress/pr.md

@@ -0,0 +1,109 @@
+## Title
+
+Quiver v33: Approval UX and planner progress
+
+## Summary
+
+- Adds shared approval-candidate guidance for planner phases.
+- Adds guided approval selection for `ai approve` in TTY when `--version` is omitted.
+- Preserves explicit `--version` behavior for CI/no-TTY/scripted use.
+- Hardens incomplete `ai revise --input` usage.
+- Aligns progress/loaders for provider-backed planner/reviewer commands.
+- Synchronizes approval guidance across `ai approvals`, `flow`, `ai status`, `ai resume`, and `spec create --interactive`.
+
+## Scope
+
+- Approval candidate model and formatting.
+- Interactive approval selector.
+- Technical-plan plan-review state visibility and blocking.
+- Revise input validation.
+- Planner/reviewer/repair-plan progress UX.
+- Workflow-surface integration.
+- Docs, tests, smokes, and release readiness.
+
+## Files
+
+- `.github/workflows/ci.yml`
+- `README.md`
+- `README_FOR_AI.md`
+- `docs/CLI_UX_GUIDE.md`
+- `docs/reference/commands.md`
+- `docs/COMMANDS.md.template`
+- `docs/WORKFLOW.md.template`
+- `src/create-quiver/commands/ai.js`
+- `src/create-quiver/commands/flow.js`
+- `src/create-quiver/commands/spec.js`
+- `src/create-quiver/index.js`
+- `src/create-quiver/lib/git.js`
+- `src/create-quiver/lib/approvals.js`
+- `src/create-quiver/lib/ai/approval-candidates.js`
+- `src/create-quiver/lib/ai/plan-review.js`
+- `src/create-quiver/lib/paths.js`
+- `src/create-quiver/lib/ai/run-state.js`
+- `tests/lib/git.test.js`
+- `tests/lib/paths.test.js`
+- `tests/**`
+- `specs/quiver-v33-approval-ux-and-planner-progress/**`
+
+## How to Test (DETAILED - REQUIRED)
+
+### Required Environment
+
+- Node.js and npm.
+- No provider credentials are required for dry-run/print-prompt tests.
+- TTY-like behavior should be tested through injected prompt helpers.
+
+### Worktree Access
+
+- One dedicated branch/worktree for `quiver-v33-approval-ux-and-planner-progress`.
+- One commit per slice.
+
+### Use Cases
+
+- Run `ai approve --phase acceptance` in a prompt-capable test harness without `--version`.
+- Run `ai approve --phase technical-plan` for plan-review states: missing, stale, unapproved, approve, approve-with-risk, revise.
+- Run `ai approve --phase acceptance` in no-TTY/CI without `--version` and verify it fails actionably.
+- Run `ai approve --phase acceptance --version 1` and verify existing scripted behavior remains.
+- Run `ai revise --phase acceptance --input` and `ai revise --phase technical-plan --input` with missing, invalid, and accidental extra input.
+- Run provider-backed planner commands in human mode and verify progress does not leak into JSON/dry-run/print-prompt.
+
+### Technical Verification
+
+- `node --test tests/lib/approvals.test.js tests/commands/ai-run-state.test.js tests/commands/ai-review-plan.test.js`
+- `node --test tests/commands/ai-plan.test.js tests/commands/ai-prepare-context-planner.test.js`
+- `node --test tests/commands/flow.test.js tests/commands/spec-create.test.js`
+- `node --test tests/commands/ai-agent.test.js tests/lib/cli-selectors.test.js tests/lib/cli-ux.test.js`
+- `node --test tests/lib/git.test.js tests/lib/paths.test.js`
+- `node --test tests/**/*.test.js`
+- `npm run smoke:create-quiver`
+- `npm run smoke:doctor-fixtures`
+- `npm run smoke:guided-workflow`
+- `npm run package:quiver`
+- `node scripts/ci/smoke-cross-platform.js`
+- `git diff --check`
+- `node bin/create-quiver.js spec validate specs/quiver-v33-approval-ux-and-planner-progress`
+
+## Evidence
+
+- `git diff --check` passed.
+- `node bin/create-quiver.js spec validate specs/quiver-v33-approval-ux-and-planner-progress` passed.
+- `node --test tests/lib/git.test.js tests/lib/paths.test.js` passed.
+- `node --test tests/**/*.test.js` passed: 496/496.
+- `npm run smoke:create-quiver` passed.
+- `npm run smoke:doctor-fixtures` passed.
+- `npm run smoke:guided-workflow` passed.
+- `npm run package:quiver` passed.
+- `node scripts/ci/smoke-cross-platform.js` passed after adding `npm ci` to the GitHub Actions smoke matrix.
+- Detailed slice evidence is in `specs/quiver-v33-approval-ux-and-planner-progress/EVIDENCE_REPORT.md`.
+
+## Rollback
+
+Revert slice commits in reverse order. Preserve explicit `--version` approval behavior and existing non-interactive planner workflows during rollback.
+
+## Risks / Notes
+
+- Approval gates are critical workflow safety controls; selector UX must not approve stale or blocked drafts.
+- Candidate summaries must remain redacted and truncated.
+- Provider progress must not contaminate machine-readable output.
+- GitHub Actions cross-platform smoke requires dependencies installed before invoking the source CLI.
+- This PR does not publish npm.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-00-approval-ux-foundation/CLOSURE_BRIEF.md

@@ -0,0 +1,30 @@
+# CLOSURE_BRIEF - slice-00 Approval UX foundation
+
+## Summary
+
+Created the v33 planning package for approval UX, planner progress, shared approval decision guidance, and revise input guardrails.
+
+## Validation Against Acceptance Criteria
+
+- [x] Spec folder created.
+- [x] Every slice has `slice.json`, `EXECUTION_BRIEF.md`, and `CLOSURE_BRIEF.md`.
+- [x] Product code was not modified.
+- [x] JSON validation captured.
+- [x] Spec validation captured.
+- [x] `git diff --check` captured.
+
+## Relevant Changes
+
+- New folder: `specs/quiver-v33-approval-ux-and-planner-progress/`.
+
+## Pending
+
+- None.
+
+## Remaining Risks
+
+- The repository still lacks `docs/INDEX.md`; this spec used the documented Quiver sources available in the repo.
+
+## Future Recommendations
+
+Keep future approval-gate changes covered by focused selector, no-TTY, review-state, and workflow-surface tests.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-00-approval-ux-foundation/EXECUTION_BRIEF.md

@@ -0,0 +1,56 @@
+# EXECUTION_BRIEF - slice-00 Approval UX foundation
+
+## Context
+
+The user approved a revised production-reviewed plan for approval UX, planner progress, revise input guardrails, and shared approval decision guidance.
+
+## Objective
+
+Create the v33 planning package under `specs/quiver-v33-approval-ux-and-planner-progress/` without implementing product code.
+
+## Scope
+
+- `specs/quiver-v33-approval-ux-and-planner-progress/SPEC.md`
+- `specs/quiver-v33-approval-ux-and-planner-progress/STATUS.md`
+- `specs/quiver-v33-approval-ux-and-planner-progress/EVIDENCE_REPORT.md`
+- `specs/quiver-v33-approval-ux-and-planner-progress/EXECUTION_PLAN.md`
+- `specs/quiver-v33-approval-ux-and-planner-progress/pr.md`
+- all slice folders and handoff briefs
+
+## Acceptance Criteria
+
+- Spec package exists.
+- Every slice has `slice.json`, `EXECUTION_BRIEF.md`, and `CLOSURE_BRIEF.md`.
+- `slice-00` records the approved acceptance criteria and revised technical plan.
+- JSON files parse.
+- Spec validation passes.
+- No product code is modified.
+
+## Technical Plan Summary
+
+Use recent Quiver specs as the local contract and keep slice-00 documentation-only.
+
+## Suggested Steps
+
+1. Create the spec folder.
+2. Create top-level spec artifacts.
+3. Create slice folders and JSON contracts.
+4. Create execution and closure briefs for every slice.
+5. Validate JSON and spec structure.
+
+## Restrictions
+
+- Do not edit `src/`.
+- Do not edit tests.
+- Do not claim implementation is complete.
+
+## Risks
+
+- Missing `docs/INDEX.md` remains a documentation debt outside this slice.
+
+## Completion Checklist
+
+- [x] Spec artifacts created.
+- [x] Slice contracts created.
+- [x] Handoff briefs created.
+- [x] Validation commands run.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-00-approval-ux-foundation/slice.json

@@ -0,0 +1,65 @@
+{
+  "slice_id": "slice-00-approval-ux-foundation",
+  "ticket": "QUIVER-33-00",
+  "type": "docs",
+  "title": "Approval UX foundation",
+  "objective": "Create the v33 spec package for approval UX, planner progress, and shared decision guidance.",
+  "description": "Publishes the approved spec, execution plan, PR body, status, evidence report, slice definitions, and handoffs without changing product code.",
+  "git": {
+    "branch_type": "feature",
+    "base_branch": "main",
+    "branch_slug": "v33-approval-ux-foundation",
+    "branch_name": "feature/QUIVER-33-00-v33-approval-ux-foundation"
+  },
+  "files": [
+    "specs/quiver-v33-approval-ux-and-planner-progress/**"
+  ],
+  "expected_read_paths": [
+    "README_FOR_AI.md",
+    "docs/CLI_UX_GUIDE.md",
+    "docs/reference/commands.md",
+    "docs/workflows/full-ai-spec-to-pr.md",
+    "specs/quiver-v29-planner-prepare-context-cli-ux/**",
+    "specs/quiver-v30-interactive-cli-ux-agent-selection/**",
+    "specs/quiver-v31-ai-model-catalog-agent-selection/**"
+  ],
+  "allowed_write_paths": [
+    "specs/quiver-v33-approval-ux-and-planner-progress/**"
+  ],
+  "depends_on": [],
+  "parallel_safe": "no",
+  "parallel_safe_reason": "slice-00 is the foundation barrier for the spec.",
+  "must": [
+    "Create SPEC.md, STATUS.md, EVIDENCE_REPORT.md, EXECUTION_PLAN.md, and pr.md.",
+    "Create slice.json, EXECUTION_BRIEF.md, and CLOSURE_BRIEF.md for every slice.",
+    "Capture approved acceptance criteria and production review fixes.",
+    "Avoid product code changes."
+  ],
+  "not_included": [
+    "Implementing approval selectors.",
+    "Changing command behavior.",
+    "Updating public docs outside this spec package.",
+    "Publishing a package."
+  ],
+  "acceptance": [
+    "Spec folder exists under specs/quiver-v33-approval-ux-and-planner-progress.",
+    "Every slice has slice.json, EXECUTION_BRIEF.md, and CLOSURE_BRIEF.md.",
+    "Every slice.json parses successfully.",
+    "Spec validation passes.",
+    "No product code is modified."
+  ],
+  "tests": [
+    "git diff --check",
+    "find specs/quiver-v33-approval-ux-and-planner-progress -name \"slice.json\" -print -exec node -e \"JSON.parse(require('fs').readFileSync(process.argv[1], 'utf8'))\" {} \\;",
+    "node bin/create-quiver.js spec validate specs/quiver-v33-approval-ux-and-planner-progress"
+  ],
+  "validation_hints": [
+    "Keep this slice documentation-only.",
+    "Do not claim implementation or package publication."
+  ],
+  "estimated_hours": 2,
+  "status": "completed",
+  "blocked_reason": null,
+  "actual_hours": 2,
+  "completed_at": "2026-05-28"
+}

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-01-approval-candidates-model/CLOSURE_BRIEF.md

@@ -0,0 +1,19 @@
+# CLOSURE_BRIEF - slice-01 Shared approval candidates model
+
+## Summary
+
+Completed shared approval-candidate construction for acceptance and technical-plan phases. Candidates include current/latest flags, approvability, next command, redacted/truncated previews, and technical-plan review decision data.
+
+## Validation
+
+- [x] Candidate model tests pass.
+- [x] Existing approval and plan-review tests pass.
+- [x] No provider execution added.
+
+## Pending
+
+- None.
+
+## Remaining Risks
+
+- None critical; downstream surfaces consume the model in slice-06.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-01-approval-candidates-model/EXECUTION_BRIEF.md

@@ -0,0 +1,51 @@
+# EXECUTION_BRIEF - slice-01 Shared approval candidates model
+
+## Context
+
+Approval guidance is currently split across approval persistence, plan-review metadata, `ai approve`, `ai approvals`, `flow`, and run-state formatting.
+
+## Objective
+
+Create a shared read-only candidate model that every later approval UX surface can consume.
+
+## Scope
+
+- `src/create-quiver/lib/approvals.js`
+- `src/create-quiver/lib/ai/plan-review.js`
+- focused tests
+
+## Acceptance Criteria
+
+- Candidate data includes phase, version, path, source, current/latest state, review recommendation, blocking state, risks, and next command.
+- Only the latest draft is marked recommended/approvable.
+- Technical-plan candidates include plan-review state.
+- Candidate summaries are bounded and safe.
+- Existing approval metadata remains backward compatible.
+
+## Technical Plan Summary
+
+Add pure helper functions around existing metadata readers. Keep formatting and interactivity out of this slice except for minimal candidate summary helpers if needed.
+
+## Suggested Steps
+
+1. Model candidate shape.
+2. Build acceptance candidates from approval metadata.
+3. Build technical-plan candidates with plan-review context.
+4. Add unit tests for missing, draft, stale, approved, approve-with-risk, and revise states.
+5. Export only stable helpers needed by later slices.
+
+## Restrictions
+
+- Do not add prompts.
+- Do not change approval persistence format unless tests prove compatibility.
+- Do not relax review blocking.
+
+## Risks
+
+- A poorly scoped helper can become another duplicated formatter instead of a source of truth.
+
+## Completion Checklist
+
+- [ ] Shared candidate helpers implemented.
+- [ ] Candidate tests added.
+- [ ] Existing approval tests still pass.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-01-approval-candidates-model/slice.json

@@ -0,0 +1,74 @@
+{
+  "slice_id": "slice-01-approval-candidates-model",
+  "ticket": "QUIVER-33-01",
+  "type": "feature",
+  "title": "Shared approval candidates model",
+  "objective": "Create a single approval-candidate model for acceptance, technical-plan, and plan-review decision data.",
+  "description": "Adds reusable read-only candidate construction so approval commands and workflow surfaces share the same current draft, version, review state, recommendation, and next-command guidance.",
+  "git": {
+    "branch_type": "feature",
+    "base_branch": "main",
+    "branch_slug": "v33-approval-candidates-model",
+    "branch_name": "feature/QUIVER-33-01-v33-approval-candidates-model"
+  },
+  "files": [
+    "src/create-quiver/lib/approvals.js",
+    "src/create-quiver/lib/ai/plan-review.js",
+    "tests/lib/approvals.test.js",
+    "tests/commands/ai-review-plan.test.js"
+  ],
+  "expected_read_paths": [
+    "src/create-quiver/lib/approvals.js",
+    "src/create-quiver/lib/ai/plan-review.js",
+    "src/create-quiver/commands/ai.js",
+    "tests/lib/approvals.test.js",
+    "tests/commands/ai-review-plan.test.js",
+    "SPEC.md"
+  ],
+  "allowed_write_paths": [
+    "src/create-quiver/lib/approvals.js",
+    "src/create-quiver/lib/ai/plan-review.js",
+    "tests/lib/approvals.test.js",
+    "tests/commands/ai-review-plan.test.js",
+    "specs/quiver-v33-approval-ux-and-planner-progress/**"
+  ],
+  "depends_on": [
+    "slice-00-approval-ux-foundation"
+  ],
+  "parallel_safe": "no",
+  "parallel_safe_reason": "This model is a dependency for later command and workflow surfaces.",
+  "must": [
+    "Expose candidate data without changing existing persisted approval metadata unless required.",
+    "Include phase, version, path, created_at, source_file, latest/current flags, status, recommendation, blocking state, recommended action, and next command.",
+    "Represent older drafts as history but mark them non-current and not recommended.",
+    "Expose technical-plan plan-review state using the existing plan-review semantics.",
+    "Use redacted/truncated summary helpers for any preview text.",
+    "Keep JSON/no-TTY consumers clean."
+  ],
+  "not_included": [
+    "Interactive prompts.",
+    "Changing approval execution behavior.",
+    "Changing plan-review blocking rules."
+  ],
+  "acceptance": [
+    "Acceptance candidates include the latest draft and draft history.",
+    "Technical-plan candidates include plan-review state and recommendation.",
+    "Older drafts are visible as history but not marked approvable.",
+    "A revise recommendation is represented as blocking.",
+    "Approve-with-risk is represented as approvable with risks.",
+    "Candidate summaries do not print unbounded draft contents."
+  ],
+  "tests": [
+    "node --test tests/lib/approvals.test.js tests/commands/ai-review-plan.test.js",
+    "git diff --check"
+  ],
+  "validation_hints": [
+    "Prefer pure functions that can be reused by commands without I/O side effects.",
+    "Do not call providers in this slice."
+  ],
+  "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-02-approve-interactive-selection/CLOSURE_BRIEF.md

@@ -0,0 +1,20 @@
+# CLOSURE_BRIEF - slice-02 Interactive approval selection
+
+## Summary
+
+Completed TTY approval selection for acceptance and technical-plan when `--version` is omitted, while preserving explicit version approval for scripts and CI.
+
+## Validation
+
+- [x] Acceptance selector test passes.
+- [x] Technical-plan selector test passes.
+- [x] No-TTY/CI failure test passes.
+- [x] Explicit version regression test passes.
+
+## Pending
+
+- None.
+
+## Remaining Risks
+
+- None critical; slice-06 integrates shared guidance into the remaining surfaces.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-02-approve-interactive-selection/EXECUTION_BRIEF.md

@@ -0,0 +1,53 @@
+# EXECUTION_BRIEF - slice-02 Interactive approval selection
+
+## Context
+
+Users currently need to run `ai approvals`, inspect versions manually, then pass `--version <n>`. The requested UX should behave like `ai agent set`: guide users in TTY and stay explicit in automation.
+
+## Objective
+
+Add TTY approval selection for `ai approve` when `--version` is omitted.
+
+## Scope
+
+- `src/create-quiver/commands/ai.js`
+- selector helpers if needed
+- approval tests
+
+## Acceptance Criteria
+
+- TTY acceptance approval without `--version` shows candidates.
+- TTY technical-plan approval without `--version` shows review-aware candidates.
+- No-TTY/CI requires `--version`.
+- Explicit `--version` behavior remains stable.
+- Cancellation writes no files.
+- Blocked candidates cannot be approved.
+
+## Technical Plan Summary
+
+Resolve candidates from slice-01, use `selectOption` for prompt-capable environments, and pass the selected version into the existing approval path.
+
+## Suggested Steps
+
+1. Add approval selection resolver.
+2. Wire resolver into `runApprove`.
+3. Add no-TTY guardrail message.
+4. Add tests for acceptance and technical-plan.
+5. Verify explicit version path remains unchanged.
+
+## Restrictions
+
+- Do not add `--review` support.
+- Do not weaken technical-plan validation.
+- Do not prompt in CI/no-TTY/JSON.
+
+## Risks
+
+- Prompt availability detection must match existing `ai agent set` behavior closely enough to avoid CI hangs.
+
+## Completion Checklist
+
+- [ ] TTY selector implemented.
+- [ ] No-TTY guardrail tested.
+- [ ] Cancellation tested.
+- [ ] Explicit `--version` tested.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-02-approve-interactive-selection/slice.json

@@ -0,0 +1,79 @@
+{
+  "slice_id": "slice-02-approve-interactive-selection",
+  "ticket": "QUIVER-33-02",
+  "type": "feature",
+  "title": "Interactive approval selection",
+  "objective": "Let `ai approve --phase acceptance|technical-plan` guide TTY users when `--version` is omitted while preserving explicit non-interactive behavior.",
+  "description": "Adds a selector patterned after `ai agent set`, recommends the current latest draft, blocks unsafe approvals, and keeps CI/no-TTY requiring `--version <n>`.",
+  "git": {
+    "branch_type": "feature",
+    "base_branch": "main",
+    "branch_slug": "v33-approve-interactive-selection",
+    "branch_name": "feature/QUIVER-33-02-v33-approve-interactive-selection"
+  },
+  "files": [
+    "src/create-quiver/commands/ai.js",
+    "src/create-quiver/lib/cli/selectors.js",
+    "src/create-quiver/lib/approvals.js",
+    "tests/commands/ai-review-plan.test.js",
+    "tests/commands/ai-plan.test.js",
+    "tests/lib/cli-selectors.test.js"
+  ],
+  "expected_read_paths": [
+    "src/create-quiver/commands/ai.js",
+    "src/create-quiver/lib/approvals.js",
+    "src/create-quiver/lib/cli/selectors.js",
+    "tests/commands/ai-review-plan.test.js",
+    "tests/commands/ai-plan.test.js",
+    "SPEC.md"
+  ],
+  "allowed_write_paths": [
+    "src/create-quiver/commands/ai.js",
+    "src/create-quiver/lib/cli/selectors.js",
+    "src/create-quiver/lib/approvals.js",
+    "tests/commands/ai-review-plan.test.js",
+    "tests/commands/ai-plan.test.js",
+    "tests/lib/cli-selectors.test.js",
+    "specs/quiver-v33-approval-ux-and-planner-progress/**"
+  ],
+  "depends_on": [
+    "slice-01-approval-candidates-model"
+  ],
+  "parallel_safe": "no",
+  "parallel_safe_reason": "This slice changes approval command behavior and should land before downstream consumers rely on it.",
+  "must": [
+    "Open a selector only when `--version` is omitted and prompts are available.",
+    "Require explicit `--version <n>` in CI/no-TTY.",
+    "Keep explicit `--version <n>` behavior backward compatible.",
+    "Recommend the current/latest approvable draft.",
+    "Show concise useful decision information for each candidate.",
+    "Allow cancellation without writes.",
+    "Do not approve older or blocked drafts silently."
+  ],
+  "not_included": [
+    "Changing plan-review result derivation.",
+    "Changing spec generation.",
+    "Adding editor review to approve."
+  ],
+  "acceptance": [
+    "`ai approve --phase acceptance` in TTY without `--version` shows a selector.",
+    "`ai approve --phase technical-plan` in TTY without `--version` shows a selector with review context.",
+    "No-TTY/CI without `--version` fails actionably.",
+    "`--version 1` still approves through the existing path when valid.",
+    "Canceling the selector writes no approval.",
+    "Older draft versions cannot be approved through the selector."
+  ],
+  "tests": [
+    "node --test tests/commands/ai-plan.test.js tests/commands/ai-review-plan.test.js tests/lib/cli-selectors.test.js",
+    "git diff --check"
+  ],
+  "validation_hints": [
+    "Use injected selector helpers in tests; do not require a real terminal.",
+    "Test no-TTY and CI explicitly."
+  ],
+  "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-03-technical-plan-review-decision-data/CLOSURE_BRIEF.md

@@ -0,0 +1,19 @@
+# CLOSURE_BRIEF - slice-03 Technical-plan review decision data
+
+## Summary
+
+Completed technical-plan candidate enrichment so approval decisions reflect review status, approval recommendation, blocking state, required fixes, optional hardening, risks, and next command.
+
+## Validation
+
+- [x] Plan-review state tests pass.
+- [x] Revise blocking test passes.
+- [x] Approve-with-risk test passes.
+
+## Pending
+
+- None.
+
+## Remaining Risks
+
+- None critical; workflow surfaces are integrated in slice-06.

package/specs/quiver-v33-approval-ux-and-planner-progress/slices/slice-03-technical-plan-review-decision-data/EXECUTION_BRIEF.md

@@ -0,0 +1,50 @@
+# EXECUTION_BRIEF - slice-03 Technical-plan review decision data
+
+## Context
+
+Technical-plan approval already has safety checks, but users need visible decision data before approving.
+
+## Objective
+
+Surface plan-review state and recommendation consistently during technical-plan approval.
+
+## Scope
+
+- `src/create-quiver/lib/ai/plan-review.js`
+- approval candidate helpers
+- `runApprove` selector output if needed
+- focused plan-review tests
+
+## Acceptance Criteria
+
+- Missing, stale, unapproved, approve, approve-with-risk, and revise states are distinguishable.
+- Revise blocks approval.
+- Approve-with-risk remains approvable and visible.
+- Required fixes, optional hardening, risks, and next command are shown where relevant.
+- Existing technical-plan approval validation remains intact.
+
+## Technical Plan Summary
+
+Extend the shared candidate data with normalized review-decision fields and consume them in the approval selector.
+
+## Suggested Steps
+
+1. Normalize review decision state.
+2. Add candidate annotations for technical-plan.
+3. Update selector rendering or approval summaries.
+4. Add tests for all review states.
+
+## Restrictions
+
+- Do not modify provider prompts.
+- Do not change review-result schema unless needed for backward compatibility.
+
+## Risks
+
+- Displaying unapproved vs reviewed states incorrectly can lead users to approve the wrong artifact.
+
+## Completion Checklist
+
+- [ ] Review states normalized.
+- [ ] Blocking behavior preserved.
+- [ ] Tests cover all review states.