fix(mcp): replace retired claude-sonnet-4-5 as the default Anthropic model by pcbeingused333 · Pull Request #1754 · getsentry/toolkit

pcbeingused333 · 2026-10-30T16:37:49Z

Anthropic retires claude-sonnet-4-5 on 2026-11-30 (model deprecations). After that date, embedded agents fail with a model-not-found error unless ANTHROPIC_MODEL is set. This PR switches the default to claude-sonnet-5-5, Anthropic's recommended replacement.

Changes

Default model: claude-sonnet-5-5 in the provider constant, its doc comment, the sentry mcp --anthropic-model usage text and the mcp.md skill reference.
@ai-sdk/anthropic ^3.0.33 → ^3.0.137 (current ai-v6 tag). This bump is required. 3.0.33 doesn't know claude-sonnet-5-5, so it handles Output.object with a hidden json tool and sends tool_choice: {type: "any"} on every step, including the toolChoice: "none" finalize step. It also caps max_tokens at 4096. Sonnet 5.5 rejects forced tool choice with a 400, so every Anthropic-backed embedded agent call would fail. 3.0.137 knows the model: it sends tool_choice as auto, or none on the finalize step, uses native output_config.format for structured output, sends no sampling params, sets max_tokens: 128000, and passes thinking blocks back unchanged in the tool loop.
Docs: fixed the stale Anthropic default in docs/operations/embedded-agents.md.

Verification

Captured the request bodies through a mocked fetch, using the same generateText + Output.object + prepareStep setup as callEmbeddedAgent, on both SDK versions (results above).
pnpm run tsc, pnpm run lint and the @sentry/mcp-core tests pass.

Behavior note

Sonnet 5.5 runs adaptive thinking by default. The Anthropic provider passes no provider options, so embedded agent steps may use more tokens and take longer than on 4.5. Setting thinking: { type: "between_tools" } would turn off up-front thinking if that matters. It's left out of this PR.

Drafted with Claude Code assistance.

via @BYK.

--

View Session in Sentry

…model Anthropic retires claude-sonnet-4-5 on 2026-11-30, after which the embedded agents fail with a model-not-found error unless ANTHROPIC_MODEL is set. Use claude-sonnet-5-5 as the default and update the CLI usage text. No sampling parameters are sent, so the new model needs no other change. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Signed-off-by: Alex Castillo <alex.castillog33@gmail.com>

vercel · 2026-10-30T16:37:56Z

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
sentry-local	Skipped		Nov 2, 2026 1:53pm UTC

BYK · 2026-10-30T18:07:14Z


 // Default configuration constants
-const DEFAULT_ANTHROPIC_MODEL = "claude-sonnet-4-5";
+const DEFAULT_ANTHROPIC_MODEL = "claude-sonnet-4-6";


Why not 5.5?

BYK · 2026-10-30T18:07:50Z

The tests in this file are garbage

Review feedback: use the newest Sonnet instead of 4-6, and keep only the default-model test (the ANTHROPIC_MODEL cases exercise the SDK, not this code). Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Signed-off-by: Alex Castillo <alex.castillog33@gmail.com>

It only pinned the constant. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Signed-off-by: Alex Castillo <alex.castillog33@gmail.com>

pcbeingused333 · 2026-10-30T20:21:55Z

Switched to claude-sonnet-5-5. Dropped the test file: pinning the constant adds nothing.

This comment was drafted with Claude Code.

…lt-model # Conflicts: # packages/mcp-server/src/cli/usage.ts

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 494708e. Configure here.}

@ai-sdk/anthropic 3.0.33 does not know claude-sonnet-5-5. It treats it as an unknown model, so structured output falls back to a hidden json tool with tool_choice {type: "any"} on every step and max_tokens 4096. Sonnet 5.5 rejects forced tool choice with a 400, so every Anthropic-backed embedded agent call would fail. 3.0.137 (latest ai-v6) knows the model: it sends native output_config.format, tool_choice auto, no sampling params, and a 128k max_tokens. Also fix the stale Anthropic default in embedded-agents.md. Co-Authored-By: BYK <126780+BYK@users.noreply.github.com>

vercel · 2026-11-02T14:21:43Z

@sentry-junior[bot] is attempting to deploy a commit to the Sentry Team on Vercel.

A member of the Team first needs to authorize it.

@BYK

…1804) Pull requests from forks always fail three `Test` jobs, `sentry`, `NPM package (sentry)` and `CLI asset catalog (macOS arm64)`, plus the aggregate `test` job. The cause isn't the PR's code: `vars.SENTRY_CLIENT_ID` is empty on fork `pull_request` runs, so `packages/cli/script/build.ts` and `bundle.ts` exit with `SENTRY_CLIENT_ID environment variable is required`. Evidence: - In the #1754 run [41461213614](https://github.com/getsentry/toolkit/actions/runs/41461213614), the step env shows `SENTRY_CLIENT_ID:` with no value, and all three jobs stop on that error. - The other recent fork runs that included the CLI, [41430282402](https://github.com/getsentry/toolkit/actions/runs/41430282402) and [41122572999](https://github.com/getsentry/toolkit/actions/runs/41122572999), fail the same jobs. Same-repo PR runs from the same day pass. ### Change - In `test.yml`, `SENTRY_CLIENT_ID` now falls back to `test-client-id` in the three CLI build jobs. Local E2E and the CLI unit tests already use that placeholder. When the repository variable is set, it still wins. - `test.yml` uploads and publishes nothing, so the placeholder never ships. Release builds go through `cli-build.yml`, which this PR doesn't touch. - `docs/operations/github-actions.md` gets a short note on the fallback. ### Verification - Locally, `SENTRY_CLIENT_ID= pnpm --filter sentry run bundle` reproduces the CI error. With `SENTRY_CLIENT_ID=test-client-id`, the bundle completes. - `pnpm run lint` and `pnpm run docs:check` pass. After this merges, #1754 needs `main` merged in (or a new push) to pick up the workflow change.  via @BYK.    -- [View Session in Sentry](https://sentry.sentry.io/explore/conversations/local%3Aweb%3Ac7729b8a4160db0aeba2f707/?project=4510944073809921)  --------- Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com> Co-authored-by: BYK <126780+BYK@users.noreply.github.com>

vercel Bot temporarily deployed to Preview October 30, 2026 16:37 Inactive

github-actions Bot added the risk: medium PR risk score: medium label Oct 30, 2026

BYK requested changes Oct 30, 2026

View reviewed changes

Comment thread packages/mcp-core/src/internal/agents/anthropic-provider.test.ts Outdated

BYK Oct 30, 2026

Copy link
Copy Markdown

Member

Choose a reason for hiding this comment

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

The tests in this file are garbage

pcbeingused333 and others added 2 commits October 30, 2026 16:19

test(mcp): drop the default-model test

2710f1e

It only pinned the constant. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Signed-off-by: Alex Castillo <alex.castillog33@gmail.com>

vercel Bot temporarily deployed to Preview October 30, 2026 20:22 Inactive

BYK approved these changes Oct 30, 2026

View reviewed changes

BYK enabled auto-merge (squash) October 30, 2026 20:53

Merge remote-tracking branch 'upstream/main' into fix/anthropic-defau…

494708e

…lt-model # Conflicts: # packages/mcp-server/src/cli/usage.ts

auto-merge was automatically disabled November 2, 2026 13:53
Head branch was pushed to by a user without write access

vercel Bot temporarily deployed to Preview November 2, 2026 13:53 Inactive

cursor Bot reviewed Nov 2, 2026

View reviewed changes

Comment thread packages/mcp-core/src/internal/agents/anthropic-provider.ts Resolved

BYK approved these changes Nov 2, 2026

View reviewed changes

BYK enabled auto-merge (squash) November 2, 2026 14:24

sentry-junior Bot mentioned this pull request Nov 2, 2026

ci(test): fall back to a placeholder SENTRY_CLIENT_ID for fork PRs #1804

Merged

sentry-junior Bot mentioned this pull request Nov 3, 2026

Document query syntax in search_sentry and drop the embedded search agents #1896

Open

Navigation Menu

Repository navigation

fix(mcp): replace retired claude-sonnet-4-5 as the default Anthropic model - #1754

fix(mcp): replace retired claude-sonnet-4-5 as the default Anthropic model#1754
pcbeingused333 wants to merge 5 commits into
getsentry:mainfrom
pcbeingused333:fix/anthropic-default-model

pcbeingused333 commented Oct 30, 2026 •

edited by sentry-junior Bot

Loading

Uh oh!

vercel Bot commented Oct 30, 2026 •

edited

Loading

Uh oh!

BYK Oct 30, 2026

Uh oh!

BYK Oct 30, 2026

Uh oh!

pcbeingused333 commented Oct 30, 2026

Uh oh!

cursor Bot left a comment

Uh oh!

Uh oh!

vercel Bot commented Nov 2, 2026

Uh oh!

This branch was previously deployed

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

pcbeingused333 commented Oct 30, 2026 • edited by sentry-junior Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

Verification

Behavior note

Uh oh!

vercel Bot commented Oct 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

BYK Oct 30, 2026

Choose a reason for hiding this comment

Uh oh!

BYK Oct 30, 2026

Choose a reason for hiding this comment

Uh oh!

pcbeingused333 commented Oct 30, 2026

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

vercel Bot commented Nov 2, 2026

Uh oh!

This branch was previously deployed

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

pcbeingused333 commented Oct 30, 2026 •

edited by sentry-junior Bot

Loading

vercel Bot commented Oct 30, 2026 •

edited

Loading