Skip to content

Conversation

@nicohrubec
Copy link
Member

As discussed moving the AI integrations from core/utils to core/tracing.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 12, 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,902 - 8,554 +4%
GET With Sentry 1,388 16% 1,430 -3%
GET With Sentry (error only) 6,143 69% 6,255 -2%
POST Baseline 1,206 - 1,222 -1%
POST With Sentry 543 45% 546 -1%
POST With Sentry (error only) 1,066 88% 1,073 -1%
MYSQL Baseline 3,327 - 3,353 -1%
MYSQL With Sentry 513 15% 498 +3%
MYSQL With Sentry (error only) 2,720 82% 2,768 -2%

View base workflow run

@nicohrubec nicohrubec marked this pull request as ready for review November 12, 2025 13:44
@nicohrubec nicohrubec changed the title ref(ai): Move ai integrations from utils to tracing ref(core): Move ai integrations from utils to tracing Nov 12, 2025
@nicohrubec nicohrubec self-assigned this Nov 12, 2025
Copy link
Member

@andreiborza andreiborza left a comment

Choose a reason for hiding this comment

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

Thanks!

@nicohrubec nicohrubec merged commit 4f71c94 into develop Nov 12, 2025
195 checks passed
@nicohrubec nicohrubec deleted the nh/move-ai-integrations-to-tracing branch November 12, 2025 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants