Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion packages/node-native/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"build:tarball": "npm pack"
},
"dependencies": {
"@sentry-internal/node-native-stacktrace": "^0.3.0",
"@sentry-internal/node-native-stacktrace": "^0.4.0",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No integration or E2E test for new feature

Low Severity

This is a feat PR that adds support for V8 v14 (Node v25+) by bumping @sentry-internal/node-native-stacktrace to ^0.4.0, but the PR does not appear to include any integration or E2E test verifying that the new V8 v14 / Node v25+ support actually works. Per the review rules, feat PRs are expected to include at least one integration or E2E test. Adding a test that exercises the event loop block detection on a supported Node version would help catch regressions.

Fix in Cursor Fix in Web

Triggered by project rule: PR Review Guidelines for Cursor Bot

Reviewed by Cursor Bugbot for commit c1769d6. Configure here.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@sentry-internal/node-native-stacktrace is already tested in Node v25 and once this is merged and released it will also be tested in Electron v38+

"@sentry/core": "10.48.0",
"@sentry/node": "10.48.0"
},
Expand Down
18 changes: 9 additions & 9 deletions yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -7742,13 +7742,13 @@
detect-libc "^2.0.3"
node-abi "^3.73.0"

"@sentry-internal/node-native-stacktrace@^0.3.0":
version "0.3.0"
resolved "https://registry.yarnpkg.com/@sentry-internal/node-native-stacktrace/-/node-native-stacktrace-0.3.0.tgz#68c80dcf11ee070a3a54406b35d4571952caa793"
integrity sha512-ef0M2y2JDrC/H0AxMJJQInGTdZTlnwa6AAVWR4fMOpJRubkfdH2IZXE/nWU0Nj74oeJLQgdPtS6DeijLJtqq8Q==
"@sentry-internal/node-native-stacktrace@^0.4.0":
version "0.4.0"
resolved "https://registry.yarnpkg.com/@sentry-internal/node-native-stacktrace/-/node-native-stacktrace-0.4.0.tgz#8f6e7a21537373a5623714c14d3350e1bb4602f0"
integrity sha512-cuRBBqnsHOJJqLCii9GvwedzjetsihIarq7TxCjgG88JyF8TZWRMlUBu/OogWhYZVU8uHqAeSvpbzolnmdhdkw==
dependencies:
detect-libc "^2.0.4"
node-abi "^3.73.0"
node-abi "^3.89.0"

"@sentry-internal/rrdom@2.34.0":
version "2.34.0"
Expand Down Expand Up @@ -22993,10 +22993,10 @@ nock@^13.5.5:
json-stringify-safe "^5.0.1"
propagate "^2.0.0"

node-abi@^3.3.0, node-abi@^3.73.0:
version "3.75.0"
resolved "https://registry.yarnpkg.com/node-abi/-/node-abi-3.75.0.tgz#2f929a91a90a0d02b325c43731314802357ed764"
integrity sha512-OhYaY5sDsIka7H7AtijtI9jwGYLyl29eQn/W623DiN/MIv5sUqc4g7BIDThX+gb7di9f6xK02nkp8sdfFWZLTg==
node-abi@^3.3.0, node-abi@^3.73.0, node-abi@^3.89.0:
version "3.89.0"
resolved "https://registry.yarnpkg.com/node-abi/-/node-abi-3.89.0.tgz#eea98bf89d4534743bbbf2defa9f4f9bd3bdccfd"
integrity sha512-6u9UwL0HlAl21+agMN3YAMXcKByMqwGx+pq+P76vii5f7hTPtKDp08/H9py6DY+cfDw7kQNTGEj/rly3IgbNQA==
dependencies:
semver "^7.3.5"

Expand Down
Loading