Skip to content

feat: Review Readiness Dashboard + gstack-slug helper (v0.5.1)#130

Merged
garrytan merged 4 commits intomainfrom
garrytan/skill-usage-tracker
Mar 17, 2026
Merged

feat: Review Readiness Dashboard + gstack-slug helper (v0.5.1)#130
garrytan merged 4 commits intomainfrom
garrytan/skill-usage-tracker

Conversation

@garrytan
Copy link
Owner

Summary

  • Every /plan-ceo-review, /plan-eng-review, and /plan-design-review now logs its result to ~/.gstack/projects/{slug}/{branch}-reviews.jsonl
  • At the end of each review, a Review Readiness Dashboard displays run counts, timestamps, and a CLEARED TO SHIP (3/3) or N/3 CLEAR verdict
  • /ship pre-flight reads the dashboard and prompts when reviews are missing (informational, not blocking)
  • New bin/gstack-slug helper replaces 14 inline copies of the SLUG sed pipeline across all templates — DRY, tested, sanitizes / in branch names
  • New {{REVIEW_DASHBOARD}} resolver in gen-skill-docs.ts (shared across 4 templates)

Pre-Landing Review

No issues found.

TODOS

  • Added: Smart review relevance detection (P3) — auto-skip Design Review when no UI files changed
  • Added: /merge skill — review-gated PR merge (P2) — code review tracking belongs at merge time, not plan time

Test plan

  • 181 tests pass (40 gen-skill-docs + 141 skill-validation), 0 failures
  • gstack-slug helper: 5 unit tests (exists, executable, SLUG/BRANCH output, no slashes, eval-compatible)
  • REVIEW_DASHBOARD resolver: 5 content tests (appears in 4 consumer files, key elements present)
  • Generated files fresh (--dry-run passes)
  • Manual: JSONL written and read correctly end-to-end

🤖 Generated with Claude Code

garrytan and others added 4 commits March 17, 2026 08:28
Extract the opaque SLUG sed pipeline into a shared 5-line shell script.
Replace 8 inline copies across templates with eval $(gstack-slug).
Sanitizes branch names (/ → -) to prevent subdirectory creation.
…ranch

Each review skill logs its result to JSONL. A shared {{REVIEW_DASHBOARD}}
placeholder displays run counts, timestamps, and a CLEARED TO SHIP verdict.
/ship pre-flight reads the dashboard and prompts when reviews are missing.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@garrytan garrytan merged commit 73b00b4 into main Mar 17, 2026
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