Skip to content

ref(aws-serverless)!: Migrate import hook to makeOrchestrionLoader - #22862

Draft
andreiborza wants to merge 1 commit into
ab/js-3203-astro-orchestrion-loaderfrom
ab/js-3203-aws-serverless-orchestrion-loader
Draft

ref(aws-serverless)!: Migrate import hook to makeOrchestrionLoader#22862
andreiborza wants to merge 1 commit into
ab/js-3203-astro-orchestrion-loaderfrom
ab/js-3203-aws-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/aws-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.13 kB - -
@sentry/browser (incl. sendFeedback) 34.66 kB - -
@sentry/browser (incl. FeedbackAsync) 39.76 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.6 kB - -
@sentry/react (incl. Tracing) 49.33 kB - -
@sentry/vue 34.74 kB - -
@sentry/vue (incl. Tracing) 49.05 kB - -
@sentry/svelte 29.85 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.82 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.04 kB - -
CDN Bundle (incl. Tracing) - uncompressed 142.27 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 99.75 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 146.25 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 224.51 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 261.52 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 265.49 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 275.23 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 279.18 kB - -
@sentry/nextjs (client) 51.92 kB - -
@sentry/sveltekit (client) 47.51 kB - -
@sentry/core/server 79.6 kB - -
@sentry/core/browser 51.7 kB - -
@sentry/node 121.35 kB - -
@sentry/node/import (ESM hook with diagnostics-channel injection) 0 B removed removed
@sentry/node - without tracing 84.88 kB - -
@sentry/aws-serverless 93.16 kB +0.01% +1 B 🔺
@sentry/cloudflare (withSentry) - minified 198.21 kB - -
@sentry/cloudflare (withSentry) 487.43 kB - -

View base workflow run

@andreiborza
andreiborza force-pushed the ab/js-3203-aws-serverless-orchestrion-loader branch from e5cf212 to 7f0c0ad Compare July 30, 2026 13:02
@andreiborza
andreiborza force-pushed the ab/js-3203-aws-serverless-orchestrion-loader branch from 7f0c0ad to 655f519 Compare July 31, 2026 08:40
Switch the `@sentry/aws-serverless/import` loader to the orchestrion-based
`makeOrchestrionLoader` and drop the obsolete `./loader` export.
@andreiborza
andreiborza force-pushed the ab/js-3203-aws-serverless-orchestrion-loader branch from 655f519 to 6bcd1a6 Compare July 31, 2026 09:59
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