Skip to content

ci: run the PR reviewer agent on opus at high effort - #381

Merged
thomaspinder merged 1 commit into
mainfrom
ci/pr-reviewer-opus
Oct 5, 2026
Merged

thomaspinder merged 1 commit into
mainfrom
ci/pr-reviewer-opus

Conversation

@thomaspinder

Copy link
Copy Markdown
Collaborator

Summary

The pr-reviewer agent now sets its own model, effort and blocked tools.

 ---
 name: pr-reviewer
 description: ...
+model: opus
+effort: high
+disallowedTools: Edit, Write, NotebookEdit
 ---
  • model: opus: with no model set, the first review (GPJax#808) ran on Sonnet 5.5.
  • effort: high: Opus 5.5 defaults to medium.
  • disallowedTools: the prompt already says "You do not edit files". This line enforces it. It is disallowedTools, not a tools: allowlist, so the action's comment tools stay available.

The workflow passes no --model, so the frontmatter decides. claude --agent pr-reviewer uses the same settings locally.

Evidence

  • Before: the GPJax#808 run log shows "model": "claude-sonnet-5-5".
    After: the session init event from claude -p --agent pr-reviewer --output-format stream-json --verbose:
    model = claude-opus-5-5
    Edit, Write, NotebookEdit -> absent
    Read                      -> present
    
  • The init event does not report effort. effort in frontmatter applies when the agent runs as the main session, per the Claude Code subagent docs.

Merge Danger

Door: two-way

Remove the three lines to revert.

Blast Radius: isolated

Only @claude-pr-review runs change. Each review uses more Claude subscription quota than on Sonnet.

🤖 Generated with Claude Code

Set model, effort and disallowedTools in the pr-reviewer frontmatter.
With no model set, the review on GPJax#808 ran on Sonnet 5.5.
disallowedTools enforces the "do not edit files" instruction.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@codecov

codecov Bot commented Oct 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@thomaspinder
thomaspinder merged commit 7299ae4 into main Oct 5, 2026
14 checks passed
@thomaspinder
thomaspinder deleted the ci/pr-reviewer-opus branch October 5, 2026 00:08
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