Skip to content

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

Merged
thomaspinder merged 1 commit into
mainfrom
ci/pr-reviewer-opus
Oct 4, 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 <[email protected]>
@github-actions github-actions Bot added documentation Improvements or additions to documentation size/xs ci Continuous Integration labels Oct 4, 2026
@github-actions

github-actions Bot commented Oct 4, 2026

Copy link
Copy Markdown

📖 Docs preview: https://pr-814--endearing-crepe-c2d5fe.netlify.app

Smoke render — the expensive notebooks run with reduced budgets, so
figures are not publication fidelity. /render-mode.txt says smoke.

@thomaspinder
thomaspinder merged commit 091cdde into main Oct 4, 2026
22 checks passed
@thomaspinder
thomaspinder deleted the ci/pr-reviewer-opus branch October 4, 2026 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Continuous Integration documentation Improvements or additions to documentation size/xs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant