Skip to content

feat: plan files always show review status (v0.11.1.1)#345

Merged
garrytan merged 2 commits intomainfrom
garrytan/show-plan-status-at-end-of-plan-always
Mar 23, 2026
Merged

feat: plan files always show review status (v0.11.1.1)#345
garrytan merged 2 commits intomainfrom
garrytan/show-plan-status-at-end-of-plan-always

Conversation

@garrytan
Copy link
Owner

Summary

  • Every plan file created through a gstack skill now gets a ## GSTACK REVIEW REPORT section before exiting plan mode — even if no formal reviews have run yet
  • Added Plan Status Footer to generateCompletionStatus() in the preamble (~25 lines)
  • If a review skill already wrote a richer report, the preamble version skips
  • If no reviews exist, writes a "NO REVIEWS YET" placeholder with links to /autoplan and individual reviews

Test Coverage

All new code paths have test coverage.

CODE PATH COVERAGE
===========================
[+] scripts/gen-skill-docs.ts — generateCompletionStatus()
    ├── [★★★ TESTED] Plan Status Footer content — gen-skill-docs.test.ts
    └── [★★★ TESTED] Key elements (GSTACK REVIEW REPORT, gstack-review-read, ExitPlanMode, NO REVIEWS YET)
─────────────────────────────────
COVERAGE: 2/2 paths tested (100%)

Pre-Landing Review

No issues found. Template-only change to prompt generation — no runtime code paths.

Eval Results

No prompt-related files changed — evals skipped.

TODOS

No TODO items completed in this PR.

Test plan

  • gen-skill-docs tests pass (126 tests, 0 failures)
  • skill-validation tests pass (385 tests, 0 failures)
  • Spot-check: Plan Status Footer present in office-hours/SKILL.md
  • Spot-check: Plan Status Footer present in plan-eng-review/SKILL.md
  • Both Claude and Codex host SKILL.md files regenerated

🤖 Generated with Claude Code

garrytan and others added 2 commits March 22, 2026 18:34
Add Plan Status Footer to generateCompletionStatus() in the preamble.
When in plan mode before ExitPlanMode, Claude writes a GSTACK REVIEW
REPORT section to the plan file — either populated from review logs
or a "NO REVIEWS YET" placeholder. Skips if a review skill already
wrote a richer report.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@garrytan garrytan merged commit 264c1ca into main Mar 23, 2026
2 checks passed
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.

1 participant