Skip to content

feat(planning): replace Outside view with Success & grain gates#98

Merged
michaelbarton merged 1 commit into
masterfrom
update-plan-review-gates
Jun 28, 2026
Merged

feat(planning): replace Outside view with Success & grain gates#98
michaelbarton merged 1 commit into
masterfrom
update-plan-review-gates

Conversation

@michaelbarton

Copy link
Copy Markdown
Owner

Summary

  • Drop the Outside view gate from plan-review.sh and planning.mdc
  • Merge Exit criteria with a success narrative into a single Success & exit criteria gate
  • Add What is the grain of the data as an explicit standalone gate (previously buried as a sub-bullet inside Invariants)

Both files are updated in lock-step so hook gate names match planning rule section headings.

Test plan

  • Edit a file under .claude/plans/ and confirm PostToolUse hook fires
  • Audit output lists 7 lines; "Outside view" absent; "Success & exit criteria" and "What is the grain of the data" present

🤖 Generated with Claude Code

Drop the Outside view gate; merge Exit criteria with a success
narrative into "Success & exit criteria"; add "What is the grain of
the data" as an explicit standalone gate so it cannot be buried inside
Invariants.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@michaelbarton
michaelbarton merged commit 77d5c04 into master Jun 28, 2026
2 checks passed
@michaelbarton
michaelbarton deleted the update-plan-review-gates branch June 28, 2026 23:15
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