Skip to content
This repository was archived by the owner on Mar 15, 2026. It is now read-only.

Improve review scope guidance with diff-focused methodology#14

Open
jachris wants to merge 1 commit intomainfrom
topic/improve-review-scope-guidance
Open

Improve review scope guidance with diff-focused methodology#14
jachris wants to merge 1 commit intomainfrom
topic/improve-review-scope-guidance

Conversation

@jachris
Copy link
Copy Markdown

@jachris jachris commented Jan 28, 2026

Summary

  • Expand the "Review Scope" section into a comprehensive diff-focused review guide
  • Add the "causation test" to help reviewers determine if findings are caused by the change
  • Add clear tables showing what to report vs what to avoid
  • Define confidence deductions for scope violations (pre-existing issues, pattern matching, etc.)

Test plan

  • Read through the updated skill documentation
  • Verify the guidance helps reviewers focus on changes introduced by the PR

🤖 Generated with Claude Code

Expand the "Review Scope" section into a comprehensive guide for
diff-focused code reviews. Add the "causation test" to help reviewers
determine if a finding is caused by the change being reviewed, clear
tables of what to report vs avoid, and confidence deductions for
scope violations.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@github-actions github-actions bot added the plugin:dev Automatic file formatting hook for edited files label Jan 28, 2026
@mavam mavam force-pushed the main branch 2 times, most recently from 75a5678 to 97e07be Compare January 28, 2026 19:13
@jachris jachris requested a review from mavam January 29, 2026 15:16
Copy link
Copy Markdown
Member

@mavam mavam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The table formatting didn't kick in; doesn't matter though, it's not auto-enforced.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

plugin:dev Automatic file formatting hook for edited files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants