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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants