feat(review): read ## Review section from CLAUDE.md as repo-specific context - #452
Open
evb87-tech wants to merge 1 commit into
Open
evb87-tech wants to merge 1 commit into
evb87-tech wants to merge 1 commit into
Conversation
…c context /review now checks CLAUDE.md for a `## Review` section before scope drift detection. This gives teams a clean place for repo-specific calibration — high-risk paths, auto-fix boundaries, escalation rules, external consumers — without forking the skill or cramming non-checklist content into checklist.md. Additive only. Skips silently if the section doesn't exist. The mandatory checklist.md remains the source of truth for the rubric and output format. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
8 tasks
Author
|
@garrytan any feedback ? |
time-attack
pushed a commit
to time-attack/gstack
that referenced
this pull request
Jul 14, 2026
time-attack
pushed a commit
to time-attack/gstack
that referenced
this pull request
Jul 22, 2026
garrytan#452 Three reviewed bug-fix overlays (MCP-packaging review, office-hours inline approval doc, host-neutral ## Review calibration), pinned set 22 -> 25 with executable regression fixtures. garrytan#1241 (compact AUQ preamble) held back: it drifts 43 baseline-render oracle hashes and needs a separate provenance call. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
feat(review): read
## Reviewsection from CLAUDE.md as repo-specific context/reviewalready reads several repo-local inputs: the mandatory checklist, the optional design checklist,DESIGN.mdfor design calibration, andCLAUDE.mdfor testing config.What it doesn't have is a clean place for the other half: scope calibration (where intent lives, ticketing conventions), risk calibration (high-risk paths, trust boundaries), escalation rules (who to involve for auth/billing changes), and auto-fix boundaries (what should never be AUTO-FIXed without explicit approval).
Today that context usually ends up crammed into
checklist.md(wrong shape), maintained in a fork ofreview/SKILL.md(drifts), or repeated manually in every review prompt (forgotten).This PR adds a single optional step before scope drift detection: if
CLAUDE.mdhas a## Reviewsection, read it as additive context. If not, skip silently.What changes
Step 1.25 is added to
/review, between branch check and scope drift detection:CLAUDE.mdfor a## Reviewsection.The mandatory
.claude/skills/review/checklist.mdremains the source of truth for the rubric, severity, output format, and fix-vs-ask behavior. The## Reviewsection is for everything else.Why CLAUDE.md
/reviewalready readsCLAUDE.mdfor testing guidance — this extends an existing pattern, not a new one.CLAUDE.mdis already project-owned instruction space — teams version it with their code..agentshosts.## Ship,## QA) without new machinery.Files changed
review/SKILL.md.tmpl,review/SKILL.mddocs/skills.mdexamples/CLAUDE.md.review.example