Skip to content

feat(core): Support embedding APIs in google-genai#19797

Open
nicohrubec wants to merge 10 commits intodevelopfrom
nh/fix-google-genai-tests
Open

feat(core): Support embedding APIs in google-genai#19797
nicohrubec wants to merge 10 commits intodevelopfrom
nh/fix-google-genai-tests

Conversation

@nicohrubec
Copy link
Member

@nicohrubec nicohrubec commented Mar 13, 2026

Add instrumentation support for the Google GenAI embeddings API (models.embedContent).

Closes #19535

@nicohrubec nicohrubec changed the title feat(core): Support embedding APIs in google-genii feat(core): Support embedding APIs in google-genai Mar 13, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 13, 2026

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 25.64 kB - -
@sentry/browser - with treeshaking flags 24.14 kB - -
@sentry/browser (incl. Tracing) 42.62 kB - -
@sentry/browser (incl. Tracing, Profiling) 47.28 kB - -
@sentry/browser (incl. Tracing, Replay) 81.42 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 71 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 86.12 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 98.37 kB - -
@sentry/browser (incl. Feedback) 42.45 kB - -
@sentry/browser (incl. sendFeedback) 30.31 kB - -
@sentry/browser (incl. FeedbackAsync) 35.36 kB - -
@sentry/browser (incl. Metrics) 26.92 kB - -
@sentry/browser (incl. Logs) 27.07 kB - -
@sentry/browser (incl. Metrics & Logs) 27.74 kB - -
@sentry/react 27.39 kB - -
@sentry/react (incl. Tracing) 44.95 kB - -
@sentry/vue 30.08 kB - -
@sentry/vue (incl. Tracing) 44.48 kB - -
@sentry/svelte 25.66 kB - -
CDN Bundle 28.27 kB - -
CDN Bundle (incl. Tracing) 43.5 kB - -
CDN Bundle (incl. Logs, Metrics) 29.13 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 44.34 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 68.2 kB - -
CDN Bundle (incl. Tracing, Replay) 80.32 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 81.22 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 85.86 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 86.76 kB - -
CDN Bundle - uncompressed 82.56 kB - -
CDN Bundle (incl. Tracing) - uncompressed 128.5 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 85.43 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 131.37 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 209.06 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 245.35 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 248.21 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 258.26 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 261.11 kB - -
@sentry/nextjs (client) 47.37 kB - -
@sentry/sveltekit (client) 43.07 kB - -
@sentry/node-core 56.34 kB +7.8% +4.07 kB 🔺
@sentry/node 173.26 kB -1.07% -1.86 kB 🔽
@sentry/node - without tracing 96.35 kB -1.11% -1.07 kB 🔽
@sentry/aws-serverless 113.34 kB +0.11% +115 B 🔺

View base workflow run

@github-actions
Copy link
Contributor

github-actions bot commented Mar 13, 2026

node-overhead report 🧳

Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.
⚠️ 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.

Scenario Requests/s % of Baseline Prev. Requests/s Change %
GET Baseline 9,719 - 9,233 +5%
GET With Sentry 1,727 18% 1,704 +1%
GET With Sentry (error only) 6,262 64% 6,023 +4%
POST Baseline 1,219 - 1,204 +1%
POST With Sentry 595 49% 595 -
POST With Sentry (error only) 1,060 87% 1,053 +1%
MYSQL Baseline 3,315 - 3,186 +4%
MYSQL With Sentry 435 13% 471 -8%
MYSQL With Sentry (error only) 2,682 81% 2,567 +4%

View base workflow run

@nicohrubec nicohrubec requested a review from andreiborza March 13, 2026 13:29
@nicohrubec nicohrubec marked this pull request as ready for review March 13, 2026 13:30
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

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

Fix All in Cursor

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

* Extract model from parameters or chat context object
* For chat instances, the model is available on the chat object as 'model' (older versions) or 'modelVersion' (newer versions)
*/
export function extractModel(params: Record<string, unknown>, context?: unknown): string {
Copy link
Member Author

Choose a reason for hiding this comment

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

from here on these functions were just moved to utils to make the linter happy

@nicohrubec nicohrubec requested review from chargome and isaacs March 13, 2026 14:34
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.

Support Embeddings APIs in Google GenAI

1 participant