Skip to content

ref(google-cloud-serverless)!: Migrate import hook to makeOrchestrionLoader - #22863

Merged
andreiborza merged 1 commit into
ab/js-3203-aws-serverless-orchestrion-loaderfrom
ab/js-3203-google-cloud-serverless-orchestrion-loader
Jul 31, 2026
Merged

ref(google-cloud-serverless)!: Migrate import hook to makeOrchestrionLoader#22863
andreiborza merged 1 commit into
ab/js-3203-aws-serverless-orchestrion-loaderfrom
ab/js-3203-google-cloud-serverless-orchestrion-loader

Conversation

@andreiborza

@andreiborza andreiborza commented Jul 30, 2026

Copy link
Copy Markdown
Member

What

Part of the @opentelemetry/instrumentation removal stack (based on #22843).

  • Switch @sentry/google-cloud-serverless's ./import loader from the legacy makeOtelLoaders build util to makeOrchestrionLoader, so the generated import-hook.mjs imports @sentry/server-utils/orchestrion/import-hook directly.
  • Drop the obsolete ./loader export (the iitm --loader entry, unused on Node >= 20.19).

Why

Orchestrion is the only instrumentation path now, so the framework's --import hook should register it directly instead of the removed iitm loader. --loader no longer applies (minimum Node is 20.19.0) and orchestrion has no --loader equivalent, so the ./loader export is dead.

@linear-code

linear-code Bot commented Jul 30, 2026

Copy link
Copy Markdown

JS-3203

@github-actions

github-actions Bot commented Jul 30, 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 29.82 kB - -
@sentry/browser - with treeshaking flags 28.02 kB - -
@sentry/browser (incl. Tracing) 47.08 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 47.09 kB - -
@sentry/browser (incl. Tracing, Profiling) 51.82 kB - -
@sentry/browser (incl. Tracing, Replay) 86.39 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 75.83 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 91.11 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 103.76 kB - -
@sentry/browser (incl. Feedback) 47.14 kB - -
@sentry/browser (incl. sendFeedback) 34.66 kB - -
@sentry/browser (incl. FeedbackAsync) 39.77 kB - -
@sentry/browser (incl. Metrics) 30.89 kB - -
@sentry/browser (incl. Logs) 31.12 kB - -
@sentry/browser (incl. Metrics & Logs) 31.8 kB - -
@sentry/react 31.61 kB - -
@sentry/react (incl. Tracing) 49.33 kB - -
@sentry/vue 34.74 kB - -
@sentry/vue (incl. Tracing) 49.05 kB - -
@sentry/svelte 29.86 kB - -
CDN Bundle 31.87 kB - -
CDN Bundle (incl. Tracing) 47.45 kB - -
CDN Bundle (incl. Logs, Metrics) 33.42 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 48.83 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 72.78 kB - -
CDN Bundle (incl. Tracing, Replay) 85.08 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 86.38 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 90.85 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 92.16 kB - -
CDN Bundle - uncompressed 95.11 kB - -
CDN Bundle (incl. Tracing) - uncompressed 142.34 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 99.82 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 146.32 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 224.58 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 261.6 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 265.56 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 275.3 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 279.26 kB - -
@sentry/nextjs (client) 51.92 kB - -
@sentry/sveltekit (client) 47.51 kB - -
@sentry/core/server 79.63 kB - -
@sentry/core/browser 51.7 kB - -
@sentry/node 121.41 kB - -
@sentry/node/import (ESM hook with diagnostics-channel injection) 0 B removed removed
@sentry/node - without tracing 84.91 kB +0.01% +1 B 🔺
@sentry/aws-serverless 93.19 kB +0.01% +1 B 🔺
@sentry/cloudflare (withSentry) - minified 198.35 kB - -
@sentry/cloudflare (withSentry) 487.75 kB - -

View base workflow run

@andreiborza
andreiborza force-pushed the ab/js-3203-google-cloud-serverless-orchestrion-loader branch from 800bbbb to 2b3b94c Compare July 30, 2026 13:02
@andreiborza
andreiborza force-pushed the ab/js-3203-google-cloud-serverless-orchestrion-loader branch 2 times, most recently from 3b6d21e to eb20860 Compare July 31, 2026 09:59
…Loader

Switch the `@sentry/google-cloud-serverless/import` loader to the orchestrion-based
`makeOrchestrionLoader` and drop the obsolete `./loader` export.
@andreiborza
andreiborza force-pushed the ab/js-3203-google-cloud-serverless-orchestrion-loader branch from eb20860 to e95b672 Compare July 31, 2026 12:55
@andreiborza
andreiborza marked this pull request as ready for review July 31, 2026 12:57
@andreiborza
andreiborza requested a review from a team as a code owner July 31, 2026 12:57
@andreiborza
andreiborza requested review from JPeer264, isaacs, mydea and nicohrubec and removed request for a team, isaacs and mydea July 31, 2026 12:57
@andreiborza
andreiborza merged commit 6006257 into develop Jul 31, 2026
47 of 75 checks passed
@andreiborza
andreiborza deleted the ab/js-3203-google-cloud-serverless-orchestrion-loader branch July 31, 2026 13:41
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.

3 participants