Skip to content

ref(node): Stop tracing models.retrieve / models.get in Anthropic AI - #23206

Merged
nicohrubec merged 2 commits into
feat/anthropic-drop-count-tokensfrom
feat/anthropic-drop-models
Aug 10, 2026
Merged

ref(node): Stop tracing models.retrieve / models.get in Anthropic AI#23206
nicohrubec merged 2 commits into
feat/anthropic-drop-count-tokensfrom
feat/anthropic-drop-models

Conversation

@nicohrubec

@nicohrubec nicohrubec commented Aug 10, 2026

Copy link
Copy Markdown
Member

models.retrieve and models.get (the same underlying call) only read model metadata and run no inference, so the Anthropic integration no longer emits spans for them.

Fixes #23147
Fixes #23146

@nicohrubec
nicohrubec force-pushed the feat/anthropic-drop-models branch 3 times, most recently from c4b9704 to 1e7972a Compare August 10, 2026 08:40
`models.retrieve` and `models.get` (the same underlying SDK call) are
model-metadata reads with no model inference, so we no longer emit spans
for them. Removes both methods from the monkey-patch registry and the
orchestrion config (including the now-unused `models` channel and
subscriber), drops the `models`-specific request/response handling, and
removes `'models'` from the non-inference operation set now that no
anthropic method maps to it.

Fixes #23147
Fixes #23146

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nicohrubec
nicohrubec force-pushed the feat/anthropic-drop-models branch from 1e7972a to db2be60 Compare August 10, 2026 08:42
@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

⚠️ Warning: Base artifact is not the latest one, because the latest workflow run is not done yet. This may lead to incorrect results. Try to re-run all tests to get up to date results.

Path Size % Change Change
@sentry/browser 30.35 kB - -
@sentry/browser - with treeshaking flags 28.52 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 26.85 kB - -
@sentry/browser (incl. Tracing) 48.63 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 48.64 kB - -
@sentry/browser (incl. Tracing, Profiling) 53.48 kB - -
@sentry/browser (incl. Tracing, Replay) 88.06 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 77.47 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 92.78 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 105.47 kB - -
@sentry/browser (incl. Feedback) 47.69 kB - -
@sentry/browser (incl. sendFeedback) 35.18 kB - -
@sentry/browser (incl. FeedbackAsync) 40.33 kB - -
@sentry/browser (incl. Metrics) 31.43 kB - -
@sentry/browser (incl. Logs) 31.66 kB - -
@sentry/browser (incl. Metrics & Logs) 32.34 kB - -
@sentry/react 32.14 kB - -
@sentry/react (incl. Tracing) 50.83 kB - -
@sentry/vue 35.45 kB - -
@sentry/vue (incl. Tracing) 50.6 kB - -
@sentry/svelte 30.38 kB - -
CDN Bundle 32.32 kB - -
CDN Bundle (incl. Tracing) 49.6 kB - -
CDN Bundle (incl. Logs, Metrics) 33.84 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 50.9 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 74.38 kB - -
CDN Bundle (incl. Tracing, Replay) 87.13 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 88.41 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 92.94 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 94.19 kB - -
CDN Bundle - uncompressed 95.83 kB - -
CDN Bundle (incl. Tracing) - uncompressed 148.84 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 100.42 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 152.82 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 229.34 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 268.09 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 272.06 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 281.79 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 285.74 kB - -
@sentry/nextjs (client) 53.41 kB - -
@sentry/sveltekit (client) 49.04 kB - -
@sentry/core/server 65.7 kB +0.02% +7 B 🔺
@sentry/core/browser 52.02 kB +0.01% +4 B 🔺
@sentry/node 119.07 kB +0.19% +222 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 0 B added added
@sentry/node - without tracing 83.19 kB +0.39% +318 B 🔺
@sentry/aws-serverless 92.47 kB +0.33% +298 B 🔺
@sentry/cloudflare (withSentry) - minified 214.91 kB - -
@sentry/cloudflare (withSentry) 530.62 kB - -

View base workflow run

@nicohrubec
nicohrubec marked this pull request as ready for review August 10, 2026 09:32
@nicohrubec
nicohrubec requested review from a team as code owners August 10, 2026 09:32
@nicohrubec
nicohrubec requested review from JPeer264, andreiborza and mydea and removed request for a team August 10, 2026 09:32

@andreiborza andreiborza left a comment

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.

clean

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nicohrubec
nicohrubec merged commit 1439ed3 into develop Aug 10, 2026
273 checks passed
@nicohrubec
nicohrubec deleted the feat/anthropic-drop-models branch August 10, 2026 10:06
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.

Stop tracing models.retrieve in Anthropic AI integration Stop tracing models.get in Anthropic AI integration

2 participants