Skip to content

ref(node): Vendor koa instrumentation#20956

Open
nicohrubec wants to merge 11 commits into
developfrom
nh/vendor-koa-instrumentation
Open

ref(node): Vendor koa instrumentation#20956
nicohrubec wants to merge 11 commits into
developfrom
nh/vendor-koa-instrumentation

Conversation

@nicohrubec
Copy link
Copy Markdown
Member

@nicohrubec nicohrubec commented May 18, 2026

Vendors @opentelemetry/instrumentation-koa into the SDK with no logic changes. Types from @types/koa, @types/koa-compose, and @types/koa__router are inlined to avoid requiring these packages as dependencies.

Closes #20153

nicohrubec and others added 2 commits May 18, 2026 07:31
Closes #20153

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 18, 2026

size-limit report 📦

Path Size % Change Change
@sentry/browser 26.92 kB - -
@sentry/browser - with treeshaking flags 25.35 kB - -
@sentry/browser (incl. Tracing) 44.91 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 47.16 kB - -
@sentry/browser (incl. Tracing, Profiling) 49.91 kB - -
@sentry/browser (incl. Tracing, Replay) 84.52 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 74.02 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 89.23 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 101.84 kB - -
@sentry/browser (incl. Feedback) 44.1 kB - -
@sentry/browser (incl. sendFeedback) 31.73 kB - -
@sentry/browser (incl. FeedbackAsync) 36.84 kB - -
@sentry/browser (incl. Metrics) 28.01 kB - -
@sentry/browser (incl. Logs) 28.15 kB - -
@sentry/browser (incl. Metrics & Logs) 28.84 kB - -
@sentry/react 28.66 kB - -
@sentry/react (incl. Tracing) 47.16 kB - -
@sentry/vue 31.85 kB - -
@sentry/vue (incl. Tracing) 46.78 kB - -
@sentry/svelte 26.94 kB - -
CDN Bundle 29.34 kB - -
CDN Bundle (incl. Tracing) 47.47 kB - -
CDN Bundle (incl. Logs, Metrics) 30.71 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 48.59 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 70 kB - -
CDN Bundle (incl. Tracing, Replay) 84.91 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 85.97 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 90.77 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 91.85 kB - -
CDN Bundle - uncompressed 86.46 kB - -
CDN Bundle (incl. Tracing) - uncompressed 142.93 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 90.66 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 146.4 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 215.31 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 261.63 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 265.08 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 275.33 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 278.77 kB - -
@sentry/nextjs (client) 49.66 kB - -
@sentry/sveltekit (client) 45.4 kB - -
@sentry/node-core 62.2 kB - -
@sentry/node 166.93 kB -0.16% -263 B 🔽
@sentry/node - without tracing 74.64 kB - -
@sentry/aws-serverless 109.62 kB - -
@sentry/cloudflare (withSentry) - minified 171.49 kB - -
@sentry/cloudflare (withSentry) 429.51 kB - -

View base workflow run

nicohrubec and others added 2 commits May 18, 2026 10:11
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@nicohrubec nicohrubec marked this pull request as ready for review May 18, 2026 08:34
@nicohrubec nicohrubec requested a review from a team as a code owner May 18, 2026 08:34
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@nicohrubec nicohrubec marked this pull request as draft May 18, 2026 08:52
@mydea
Copy link
Copy Markdown
Member

mydea commented May 18, 2026

let's generally avoid adding type dev dependencies, let's just inline them (this applies to all such instrumentation)

Comment thread packages/node/package.json Outdated
Comment on lines +95 to +96
"@types/koa": "3.0.2",
"@types/koa__router": "12.0.5",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

let's avoid this!

Copy link
Copy Markdown
Member Author

@nicohrubec nicohrubec May 18, 2026

Choose a reason for hiding this comment

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

jep that was my hunch (that's why I put it back in draft), thanks for confirming

nicohrubec and others added 6 commits May 18, 2026 15:22
…outer

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@nicohrubec nicohrubec marked this pull request as ready for review May 18, 2026 14:00
@nicohrubec nicohrubec requested a review from a team May 18, 2026 14:31
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.

Vendor @opentelemetry/instrumentation-koa

2 participants