ref(core): Consolidate getOperationName into one shared utility by nicohrubec · Pull Request #19971 · getsentry/sentry-javascript

nicohrubec · 2026-03-25T11:22:52Z

We have this function in both the shared utilities (used by google-genai and anthropic) and in openai with slightly different names for no apparent reason. We also had a separate helper that just prepends gen_ai to the operation name in both cases, which seems unnecessary. Doing some cleanup here

Closes#19978 (added automatically)

github-actions · 2026-03-25T11:23:26Z

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

New Features ✨

Deps

Bump mongodb-memory-server-global from 10.1.4 to 11.0.1 by dependabot in #19888
Bump stacktrace-parser from 0.1.10 to 0.1.11 by dependabot in #19887

Other

(elysia) Elysia SDK by logaretm in #19509
(nuxt) Conditionally use plugins based on Nitro version (v2/v3) by s1gr1d in #19955

Bug Fixes 🐛

Cloudflare

Send correct events in local development by JPeer264 in #19900
Forward ctx argument to Workflow.do user callback by Lms24 in #19891

Core

Preserve .withResponse() on Anthropic instrumentation by nicohrubec in #19935
Truncate content array format in Vercel by nicohrubec in #19911
Send internal_error as span status for Vercel error spans by nicohrubec in #19921
Do not overwrite user provided conversation id in Vercel by nicohrubec in #19903
Return same value from startSpan as callback returns by s1gr1d in #19300

Deps

Update lockfile to resolve h3@1.15.10 by chargome in #19933
Bump fast-xml-parser to 5.5.8 in @azure/core-xml chain by chargome in #19918
Bump next to 15.5.14 in nextjs-15 and nextjs-15-intl E2E test apps by chargome in #19917
Bump socket.io-parser to 4.2.6 to fix CVE-2026-33151 by chargome in #19880

Other

(craft) Add missing mainDocsUrl for @sentry/effect SDK by bc-sentry in #19860
(nestjs) Add node to nest metadata by chargome in #19875
(serverless) Add node to metadata by nicohrubec in #19878

Internal Changes 🔧

Core

Consolidate getOperationName into one shared utility by nicohrubec in #19971

Remove duplicate buildMethodPath utility from openai by nicohrubec in #19969

Deps Dev

Bump effect from 3.19.19 to 3.20.0 by dependabot in #19926
Bump qunit-dom from 3.2.1 to 3.5.0 by dependabot in #19546
Bump @react-router/node from 7.13.0 to 7.13.1 by dependabot in #19544

Nuxt

Extract core logic for storage/database to prepare for Nuxt v5 by s1gr1d in #19920
Extract handler patching to extra plugin for Nitro v2/v3 by s1gr1d in #19915

Other

(astro) Re-enable server island tracing e2e test in Astro 6 by Lms24 in #19872
(ci) Fix "Gatbsy" typo in issue package label workflow by chargome in #19905
(claude) Enable Claude Code Intelligence (LSP) by s1gr1d in #19930
(cloudflare) Enable multi-worker tests for CF integration tests by JPeer264 in #19938
(elysia) Drop @elysiajs/opentelemetry dependency by logaretm in #19947
(lint) Resolve oxlint warnings by isaacs in #19893
(node-integration-tests) Remove unnecessary file-type dependency by Lms24 in #19824
(remix) Replace glob with native recursive fs walk by roli-lpci in #19531
(sveltekit) Replace recast + @babel/parser with acorn by roli-lpci in #19533
Add external contributor to CHANGELOG.md by javascript-sdk-gitflow in #19925
Add external contributor to CHANGELOG.md by javascript-sdk-gitflow in #19909

_{🤖 This preview updates automatically when you update the PR.}

github-actions · 2026-03-25T11:54:35Z

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.69 kB	+0.2%	+49 B 🔺
@sentry/browser - with treeshaking flags	24.17 kB	+0.14%	+33 B 🔺
@sentry/browser (incl. Tracing)	42.67 kB	+0.13%	+54 B 🔺
@sentry/browser (incl. Tracing, Profiling)	47.33 kB	+0.12%	+55 B 🔺
@sentry/browser (incl. Tracing, Replay)	81.48 kB	+0.08%	+57 B 🔺
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags	71.06 kB	+0.1%	+69 B 🔺
@sentry/browser (incl. Tracing, Replay with Canvas)	86.17 kB	+0.06%	+50 B 🔺
@sentry/browser (incl. Tracing, Replay, Feedback)	98.41 kB	+0.04%	+36 B 🔺
@sentry/browser (incl. Feedback)	42.48 kB	+0.08%	+30 B 🔺
@sentry/browser (incl. sendFeedback)	30.35 kB	+0.15%	+43 B 🔺
@sentry/browser (incl. FeedbackAsync)	35.4 kB	+0.12%	+39 B 🔺
@sentry/browser (incl. Metrics)	26.96 kB	+0.15%	+38 B 🔺
@sentry/browser (incl. Logs)	27.1 kB	+0.12%	+32 B 🔺
@sentry/browser (incl. Metrics & Logs)	27.78 kB	+0.15%	+39 B 🔺
@sentry/react	27.45 kB	+0.22%	+58 B 🔺
@sentry/react (incl. Tracing)	45.01 kB	+0.14%	+60 B 🔺
@sentry/vue	30.13 kB	+0.16%	+46 B 🔺
@sentry/vue (incl. Tracing)	44.52 kB	+0.09%	+39 B 🔺
@sentry/svelte	25.7 kB	+0.16%	+40 B 🔺
CDN Bundle	28.35 kB	+0.27%	+75 B 🔺
CDN Bundle (incl. Tracing)	43.57 kB	+0.15%	+62 B 🔺
CDN Bundle (incl. Logs, Metrics)	29.22 kB	+0.27%	+77 B 🔺
CDN Bundle (incl. Tracing, Logs, Metrics)	44.43 kB	+0.17%	+75 B 🔺
CDN Bundle (incl. Replay, Logs, Metrics)	68.29 kB	+0.13%	+85 B 🔺
CDN Bundle (incl. Tracing, Replay)	80.41 kB	+0.1%	+73 B 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)	81.31 kB	+0.1%	+76 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback)	85.97 kB	+0.12%	+103 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)	86.86 kB	+0.1%	+86 B 🔺
CDN Bundle - uncompressed	82.7 kB	+0.1%	+77 B 🔺
CDN Bundle (incl. Tracing) - uncompressed	128.62 kB	+0.05%	+64 B 🔺
CDN Bundle (incl. Logs, Metrics) - uncompressed	85.57 kB	+0.1%	+77 B 🔺
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed	131.49 kB	+0.05%	+64 B 🔺
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed	209.22 kB	+0.05%	+102 B 🔺
CDN Bundle (incl. Tracing, Replay) - uncompressed	245.5 kB	+0.04%	+89 B 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed	248.35 kB	+0.04%	+89 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed	258.41 kB	+0.04%	+89 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed	261.26 kB	+0.04%	+89 B 🔺
@sentry/nextjs (client)	47.4 kB	+0.08%	+37 B 🔺
@sentry/sveltekit (client)	43.12 kB	+0.12%	+51 B 🔺
@sentry/node-core	56.42 kB	+0.13%	+73 B 🔺
@sentry/node	173.44 kB	+0.17%	+279 B 🔺
@sentry/node - without tracing	96.43 kB	+0.1%	+87 B 🔺
@sentry/aws-serverless	113.44 kB	+0.09%	+100 B 🔺

View base workflow run

github-actions · 2026-03-25T11:55:50Z

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.

Scenario	Requests/s	% of Baseline	Prev. Requests/s	Change %
GET Baseline	8,511	-	9,314	-9%
GET With Sentry	1,543	18%	1,600	-4%
GET With Sentry (error only)	5,811	68%	5,883	-1%
POST Baseline	1,137	-	1,171	-3%
POST With Sentry	496	44%	553	-10%
POST With Sentry (error only)	989	87%	1,041	-5%
MYSQL Baseline	3,023	-	3,224	-6%
MYSQL With Sentry	292	10%	450	-35%
MYSQL With Sentry (error only)	2,462	81%	2,646	-7%

View base workflow run

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.}

JPeer264

looks cleaner and has less overhead. LGTM

nicohrubec added 5 commits March 25, 2026 13:11

.
6b1f244

consolidate
1f47c32

text completions are no longer a thing
8ebae83

update comment
a3b3d3e

.
1a368b1

nicohrubec force-pushed the nh/get-operation-name-consolidation branch from 4649131 to 1a368b1Compare March 25, 2026 13:05

nicohrubec marked this pull request as ready for review March 25, 2026 14:08

github-actionsBot mentioned this pull request Mar 25, 2026
ref(core): Consolidate getOperationName into one shared utility #19978
Closed

nicohrubec requested review from JPeer264, chargome and s1gr1d March 25, 2026 14:09

cursorBot reviewed Mar 25, 2026
View reviewed changes

Comment threadpackages/core/src/tracing/ai/utils.ts

JPeer264 approved these changes Mar 26, 2026
View reviewed changes

nicohrubec merged commit 4a43db2 into developMar 26, 2026
243 of 245 checks passed

nicohrubec deleted the nh/get-operation-name-consolidation branch March 26, 2026 09:40

sentry-release-botBot mentioned this pull request Mar 31, 2026
publish: getsentry/sentry-javascript@10.47.0 getsentry/publish#7651
Closed
46 tasks

Navigation Menu

Uh oh!

Uh oh!

ref(core): Consolidate getOperationName into one shared utility - #19971

ref(core): Consolidate getOperationName into one shared utility#19971
nicohrubec merged 5 commits into
developfrom
nh/get-operation-name-consolidation

nicohrubec commented Mar 25, 2026•
edited by github-actions Bot
Loading

Uh oh!

github-actionsBot commented Mar 25, 2026•
edited
Loading

New Features ✨

Deps

Other

Bug Fixes 🐛

Cloudflare

Core

Deps

Other

Internal Changes 🔧

Core

Deps Dev

Nuxt

Other

Uh oh!

github-actionsBot commented Mar 25, 2026•
edited
Loading

Uh oh!

github-actionsBot commented Mar 25, 2026•
edited
Loading

Uh oh!

cursorBot left a comment

Uh oh!

Uh oh!

JPeer264 left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Uh oh!

Conversation

nicohrubec commented Mar 25, 2026• edited by github-actions BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actionsBot commented Mar 25, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Semver Impact of This PR

New Features ✨

Deps

Other

Bug Fixes 🐛

Cloudflare

Core

Deps

Other

Internal Changes 🔧

Core

Deps Dev

Nuxt

Other

Uh oh!

github-actionsBot commented Mar 25, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

size-limit report 📦

Uh oh!

github-actionsBot commented Mar 25, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

node-overhead report 🧳

Uh oh!

cursorBot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

JPeer264 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

nicohrubec commented Mar 25, 2026•
edited by github-actions Bot
Loading

github-actionsBot commented Mar 25, 2026•
edited
Loading

github-actionsBot commented Mar 25, 2026•
edited
Loading

github-actionsBot commented Mar 25, 2026•
edited
Loading