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

fix(agent): restore Claude Opus 5 reasoning efforts - #3924

Merged
trunk-io[bot] merged 1 commit into
mainfrom
posthog-code/fix-opus-5-reasoning-effort
Jul 29, 2026
Merged

fix(agent): restore Claude Opus 5 reasoning efforts#3924
trunk-io[bot] merged 1 commit into
mainfrom
posthog-code/fix-opus-5-reasoning-effort

Conversation

@tatoalo

@tatoalo tatoalo commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Problem

Cloud tasks configured with Claude Opus 5 and an explicit reasoning effort fail while starting the agent server. The shared model-policy extraction in #3616 omitted Opus 5 from the Claude effort map.

Changes

  • Restore all supported reasoning efforts for claude-opus-5
  • Add regression coverage for the failing high setting and extended effort support

Restore the Opus 5 effort policy omitted during the shared model-policy refactor so cloud agent servers accept configured reasoning effort again.

Add regression coverage for the failing high effort and extended max effort settings.

Generated-By: PostHog Code
Task-Id: 751a37dc-0eb5-4201-9a10-e2c694aada61
@trunk-io

trunk-io Bot commented Jul 29, 2026

Copy link
Copy Markdown

😎 Merged successfully - details.

@github-actions

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 23f1e58.

@tatoalo tatoalo self-assigned this Jul 29, 2026
@tatoalo
tatoalo marked this pull request as ready for review July 29, 2026 10:43
@tatoalo
tatoalo requested a review from a team July 29, 2026 10:43
@tatoalo tatoalo added the Stamphog This will request an autostamp by stamphog on small changes label Jul 29, 2026
@greptile-apps

greptile-apps Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(agent): restore Claude Opus 5 reason..." | Re-trigger Greptile

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Tiny, well-tested fix restoring a missing model entry in a shared config map; two current-head approvals and no concerns.

  • Author wrote 0% of the modified lines and has 16 merged PRs in these paths (familiarity MODERATE).
  • jonathanlab, sampennington reviewed the current head.
Gate mechanics and policy version
Gate Result
prerequisites all clear
deny-list no deny categories matched
size 1L, 1F substantive, 3L/2F incl. docs/generated/snapshots — within ceiling
tier T1-agent / T1a-trivial (3L, 2F, single-area, fix)
stamphog 2.0.0b3 .stamphog/policy.yml @ 3b00ef5 · reviewed head 23f1e58

@tatoalo
tatoalo enabled auto-merge (squash) July 29, 2026 10:53
@trunk-io
trunk-io Bot merged commit afb8fd5 into main Jul 29, 2026
45 of 46 checks passed
@trunk-io
trunk-io Bot deleted the posthog-code/fix-opus-5-reasoning-effort branch July 29, 2026 11:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Stamphog This will request an autostamp by stamphog on small changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants