Skip to content

Conversation

onurtemizkan
Copy link
Collaborator

@onurtemizkan onurtemizkan commented Sep 11, 2025

Unblocks OTEL upgrades, where shimmer is not a sub-dependency anymore.

@onurtemizkan onurtemizkan changed the title chore(node): Inline types imported from shimmer ref(node): Inline types imported from shimmer Sep 11, 2025
Copy link
Contributor

github-actions bot commented Sep 11, 2025

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,715 - 8,375 +4%
GET With Sentry 1,276 15% 1,223 +4%
GET With Sentry (error only) 5,971 69% 5,778 +3%
POST Baseline 1,195 - 1,148 +4%
POST With Sentry 466 39% 418 +11%
POST With Sentry (error only) 1,064 89% 1,000 +6%
MYSQL Baseline 3,285 - 3,217 +2%
MYSQL With Sentry 427 13% 359 +19%
MYSQL With Sentry (error only) 2,672 81% 2,503 +7%

View base workflow run

@onurtemizkan onurtemizkan marked this pull request as ready for review September 11, 2025 11:32
@AbhiPrasad AbhiPrasad merged commit 15e22cd into develop Sep 11, 2025
129 checks passed
@AbhiPrasad AbhiPrasad deleted the onur/inline-shimmer branch September 11, 2025 13:17
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