feat(review): read ## Review section from CLAUDE.md as repo-specific context#452
Open
evb87-tech wants to merge 1 commit intogarrytan:mainfrom
Open
feat(review): read ## Review section from CLAUDE.md as repo-specific context#452evb87-tech wants to merge 1 commit intogarrytan:mainfrom
## Review section from CLAUDE.md as repo-specific context#452evb87-tech wants to merge 1 commit intogarrytan:mainfrom
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>
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