Skip to content

Conversation

@andreiborza
Copy link
Member

@andreiborza andreiborza commented Jan 27, 2026

@github-actions
Copy link
Contributor

github-actions bot commented Jan 27, 2026

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 9,184 - 9,023 +2%
GET With Sentry 1,729 19% 1,693 +2%
GET With Sentry (error only) 6,193 67% 6,130 +1%
POST Baseline 1,222 - 1,201 +2%
POST With Sentry 620 51% 585 +6%
POST With Sentry (error only) 1,078 88% 1,056 +2%
MYSQL Baseline 3,399 - 3,337 +2%
MYSQL With Sentry 477 14% 474 +1%
MYSQL With Sentry (error only) 2,754 81% 2,703 +2%

View base workflow run

- npm
- yarn
- pnpm
- bun
Copy link
Member

Choose a reason for hiding this comment

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

Deno 😢

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll add.

Copy link
Member Author

Choose a reason for hiding this comment

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

Added.

Copy link
Member

Choose a reason for hiding this comment

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

Would it make sense to add a "Other"? You'll never know what will come tomorrow

@andreiborza andreiborza enabled auto-merge (squash) January 27, 2026 15:05
@andreiborza andreiborza disabled auto-merge January 27, 2026 16:41
Comment on lines 26 to 36
id: package
attributes:
label: Which SDK are you using?
label: Where are you installing the Sentry SDK from?
description:
If you're using the CDN bundles, please specify the exact bundle (e.g. `bundle.tracing.min.js`) in your SDK
setup.
options:
- '@sentry/browser'
- '@sentry/node'
- '@sentry/node - express'
- '@sentry/node - fastify'
- '@sentry/node - koa'
- '@sentry/node - hapi'
- '@sentry/node - connect'
- '@sentry/node-native'
- '@sentry/angular'
- '@sentry/astro'
- '@sentry/aws-serverless'
- '@sentry/bun'
- '@sentry/cloudflare'
- '@sentry/cloudflare - hono'
- '@sentry/deno'
- '@sentry/ember'
- '@sentry/gatsby'
- '@sentry/google-cloud-serverless'
- '@sentry/nestjs'
- '@sentry/nextjs'
- '@sentry/nuxt'
- '@sentry/react'
- '@sentry/react-router'
- '@sentry/remix'
- '@sentry/solid'
- '@sentry/solidstart'
- '@sentry/svelte'
- '@sentry/sveltekit'
- '@sentry/tanstackstart-react'
- '@sentry/vue'
- '@sentry/wasm'
- Sentry NPM package
- Sentry Browser Loader
- Sentry Browser CDN bundle
validations:
Copy link

Choose a reason for hiding this comment

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

Bug: Changing the issue template label from Which SDK are you using? will break the regex in the issue-package-label.yml workflow, causing automated labeling to fail.
Severity: HIGH

Suggested Fix

Update the regex in .github/workflows/issue-package-label.yml to match the new label text. Additionally, either update the workflow's mapping logic to handle the new generic dropdown options or revert the options to the specific package names the workflow expects.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: .github/ISSUE_TEMPLATE/bug.yml#L25-L36

Potential issue: The `.github/workflows/issue-package-label.yml` workflow uses a regex
to find the heading `### Which SDK are you using?` in new issues to automatically apply
labels. This pull request changes the corresponding label in the `bug.yml` issue
template to `Where are you installing the Sentry SDK from?`. This mismatch will cause
the regex to fail for all new issues, breaking the automated labeling and triage
process. Furthermore, the new dropdown options are too generic for the workflow's
mapping logic, which expects specific package names.

Did we get this right? 👍 / 👎 to inform future reviews.

Copy link
Member

Choose a reason for hiding this comment

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

@andreiborza that one is a good catch though

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.

chore: Add System Info and Package Manager fields to bug template

4 participants