Skip to content

Restore Opus 4.8 / max effort for Claude PR review; drop debug output#6260

Merged
dkayiwa merged 1 commit into
masterfrom
claude-pr-review-restore-opus
Jul 7, 2026
Merged

Restore Opus 4.8 / max effort for Claude PR review; drop debug output#6260
dkayiwa merged 1 commit into
masterfrom
claude-pr-review-restore-opus

Conversation

@dkayiwa

@dkayiwa dkayiwa commented Jul 7, 2026

Copy link
Copy Markdown
Member

Posting is now validated end-to-end: with the tool allow-list from #6259, claude[bot] posted review #4644899156 on #6189 with two genuine findings, correctly stayed silent on a clean test-refactor (#6228), and did not re-comment on already-resolved threads.

This reverts the temporary debug settings added in #6259:

  • --model claude-opus-4-8 and --effort max (was Sonnet 5 / medium for cheap iteration)
  • removes show_full_output: true

The real fix — --allowedTools granting the tools the review needs — stays.

🤖 Generated with Claude Code

Posting is validated (claude[bot] posted review #4644899156 on #6189 with
two real findings), so revert the temporary debug settings: back to
--model claude-opus-4-8 / --effort max and drop show_full_output. The
--allowedTools fix stays — that's what unblocked posting.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dkayiwa
dkayiwa merged commit dc043fb into master Jul 7, 2026
8 of 9 checks passed
@dkayiwa
dkayiwa deleted the claude-pr-review-restore-opus branch July 7, 2026 12:10
@sonarqubecloud

sonarqubecloud Bot commented Jul 7, 2026

Copy link
Copy Markdown

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.40%. Comparing base (ded0794) to head (33a28bb).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6260      +/-   ##
============================================
+ Coverage     59.37%   59.40%   +0.02%     
  Complexity     9517     9517              
============================================
  Files           731      731              
  Lines         38222    38222              
  Branches       5578     5578              
============================================
+ Hits          22696    22705       +9     
+ Misses        13486    13483       -3     
+ Partials       2040     2034       -6     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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