Skip to content

feat: use opus 4.6 and restructure review skill#53

Merged
russellhaering merged 1 commit intomainfrom
review-skill-improvements
Feb 26, 2026
Merged

feat: use opus 4.6 and restructure review skill#53
russellhaering merged 1 commit intomainfrom
review-skill-improvements

Conversation

@russellhaering
Copy link
Copy Markdown
Contributor

Summary

  • Switch to claude-opus-4-6 for better prompt compliance (was defaulting to Sonnet)
  • Increase max-turns from 15 to 100
  • Restructure the review skill to be more resilient to structural shortcuts

What changed in the skill

  • Step checklist at the top with explicit deliverables per step — agent knows upfront what's expected
  • Docs staleness promoted to Agent 4 — runs in parallel with code review agents instead of being a post-processing sub-item that gets skipped
  • Required summary template with a mandatory Documentation section — forces the agent to report on docs staleness
  • File classification table updated to include docs/connector.mdx → docs-reviewer

Why

The previous run on baton-twingate#16 had the docs staleness skill on disk but didn't flag the missing docs update (PR adds provisioning for roles without updating the Capabilities table). Root causes:

  1. Sonnet was taking shortcuts and skipping the multi-step orchestration
  2. The docs check was buried as item 6 in a validation step — easy to skip

- Switch model to claude-opus-4-6 for better prompt compliance
- Increase max-turns to 100
- Restructure skill with explicit step checklist and deliverables
- Promote docs staleness check to Agent 4 (docs-reviewer) so it runs
  in parallel with code review agents and can't be skipped
- Add required summary comment template with Documentation section
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 26, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (2)
  • .github/workflows/pr-review.yaml is excluded by none and included by none
  • skills/pr-review.md is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch review-skill-improvements

Comment @coderabbitai help to get the list of available commands and usage tips.

@russellhaering russellhaering merged commit 186522d into main Feb 26, 2026
2 checks passed
@russellhaering russellhaering deleted the review-skill-improvements branch February 26, 2026 18:12
russellhaering added a commit that referenced this pull request Feb 26, 2026
## Summary

- Doc staleness findings (D1-D4) are now **critical** severity — stale
docs block merge
- Streamlined summary comment template: severity counts on one line,
removed verbose sections
- Inline comments kept to 2-3 sentences instead of paragraphs
- Summary references inline findings briefly instead of repeating them

Follow-up to #53 based on
[baton-twingate#16](ConductorOne/baton-twingate#16)
output being too verbose.
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.

2 participants