Skip to content

ref(server-utils): Remove unused AI client types - #23289

Open
nicohrubec wants to merge 1 commit into
developfrom
ref/remove-unused-ai-client-types
Open

ref(server-utils): Remove unused AI client types#23289
nicohrubec wants to merge 1 commit into
developfrom
ref/remove-unused-ai-client-types

Conversation

@nicohrubec

@nicohrubec nicohrubec commented Aug 11, 2026

Copy link
Copy Markdown
Member

Removes the unused AnthropicAiClient, OpenAiClient, GoogleGenAIClient, and WorkersAiClient interfaces (plus the now-orphaned GoogleGenAIChat), which aren't referenced by the generic instrument*Client functions and are already absent from the public API since the v11 move to @sentry/server-utils.

The `AnthropicAiClient`, `OpenAiClient`, `GoogleGenAIClient`, and
`WorkersAiClient` interfaces are unused — the `instrument*Client` functions
take a generic `<T extends object>` rather than these shapes, and they are
already absent from the public API since the v11 move to `@sentry/server-utils`.
Also removes `GoogleGenAIChat`, orphaned once `GoogleGenAIClient` is gone, and
drops the sole test reference (a redundant cast).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 30.29 kB - -
@sentry/browser - with treeshaking flags 28.47 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 26.8 kB - -
@sentry/browser (incl. Tracing) 48.54 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 48.56 kB - -
@sentry/browser (incl. Tracing, Profiling) 51.44 kB - -
@sentry/browser (incl. Tracing, Replay) 87.95 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 77.36 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 92.66 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 105.36 kB - -
@sentry/browser (incl. Feedback) 47.61 kB - -
@sentry/browser (incl. sendFeedback) 35.11 kB - -
@sentry/browser (incl. FeedbackAsync) 40.26 kB - -
@sentry/browser (incl. Metrics) 31.28 kB - -
@sentry/browser (incl. Logs) 31.53 kB - -
@sentry/browser (incl. Metrics & Logs) 32.2 kB - -
@sentry/react 32.09 kB - -
@sentry/react (incl. Tracing) 50.73 kB - -
@sentry/vue 35.34 kB - -
@sentry/vue (incl. Tracing) 50.5 kB - -
@sentry/svelte 30.31 kB - -
CDN Bundle 31.59 kB - -
CDN Bundle (incl. Tracing) 48.87 kB - -
CDN Bundle (incl. Logs, Metrics) 33.8 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 50.83 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 74.34 kB - -
CDN Bundle (incl. Tracing, Replay) 86.46 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 88.33 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 92.17 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 94.14 kB - -
CDN Bundle - uncompressed 93.77 kB - -
CDN Bundle (incl. Tracing) - uncompressed 146.66 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 100.17 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 152.46 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 229.1 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 265.92 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 271.7 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 279.61 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 285.38 kB - -
@sentry/nextjs (client) 53.29 kB - -
@sentry/sveltekit (client) 48.96 kB - -
@sentry/core/server 65.44 kB - -
@sentry/core/browser 51.8 kB - -
@sentry/node 117.98 kB - -
@sentry/node/import (ESM hook with diagnostics-channel injection) 0 B added added
@sentry/node - without tracing 82.09 kB - -
@sentry/aws-serverless 91.5 kB - -
@sentry/cloudflare (withSentry) - minified 214.1 kB - -
@sentry/cloudflare (withSentry) 528.64 kB - -

View base workflow run

@nicohrubec
nicohrubec marked this pull request as ready for review August 11, 2026 13:35
@nicohrubec
nicohrubec requested a review from a team as a code owner August 11, 2026 13:35
@nicohrubec
nicohrubec requested review from andreiborza, logaretm and stephanie-anderson and removed request for a team and stephanie-anderson August 11, 2026 13:35
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.

1 participant