-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(deps): update sentry-javascript monorepo to v7.111.0 #1898
Conversation
Important Auto Review SkippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
Removed dependencies detected. Learn more about Socket for GitHub ↗︎ 🚮 Removed packages: npm/@sentry/node@7.110.1, npm/@sentry/profiling-node@7.110.1 |
Quality Gate passedIssues Measures |
Here's the code health analysis summary for commits Analysis Summary
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code changes involve updating the version of Sentry used in both the package.json and the pnpm-lock.yaml files. It is a minor change and generally does not seem to have an issue at first glance. However, testing should be performed to ensure compatibility and that Sentry’s update did not introduce breaking changes.
@@ -26,8 +26,8 @@ | |||
}, | |||
"devDependencies": { | |||
"@eslint/js": "^9.0.0", | |||
"@sentry/node": "7.110.1", | |||
"@sentry/profiling-node": "7.110.1", | |||
"@sentry/node": "7.111.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're updating the version of @sentry/node package. Please ensure the new version introduces no breaking changes.
"@sentry/node": "7.110.1", | ||
"@sentry/profiling-node": "7.110.1", | ||
"@sentry/node": "7.111.0", | ||
"@sentry/profiling-node": "7.111.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're updating the version of @sentry/profiling-node. It would be prudent to verify that this new version does not introduce any breaking changes.
@@ -49,11 +49,11 @@ importers: | |||
specifier: ^9.0.0 | |||
version: 9.0.0 | |||
'@sentry/node': | |||
specifier: 7.110.1 | |||
version: 7.110.1 | |||
specifier: 7.111.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sentry/node's version updated; it's wise to ensure compatibility with current codebase and that no deprecated functions are in use.
'@sentry/profiling-node': | ||
specifier: 7.110.1 | ||
version: 7.110.1 | ||
specifier: 7.111.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The @sentry/profiling-node version has been updated. Make sure to check for any breaking changes introduced in this version.
@@ -1113,13 +1113,13 @@ packages: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The integrity of @sentry-internal/tracing@7.111.0 package has changed. Ensure the change has not introduced any breaking modifications.
@@ -1248,17 +1248,17 @@ packages: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Verify @sentry/node's new integrity value; it reflects the change and could potentially introduce breaking changes.
'@sentry/types': 7.110.1 | ||
'@sentry/utils': 7.110.1 | ||
'@sentry-internal/tracing': 7.111.0 | ||
'@sentry/core': 7.111.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The resolution integrity of @sentry/profiling-node@7.111.0 package has been updated. It is advisable to check for any breaking changes introduced by this version.
/@sentry/profiling-node@7.110.1: | ||
resolution: {integrity: sha512-htqOzb/IPyC27LSAhWy/9J1fqNZNPhgDSXxPyHNo4aWHPpimDIEClues04KGWNNXH2GRe5ttwavX2PHUQG+dwQ==} | ||
/@sentry/profiling-node@7.111.0: | ||
resolution: {integrity: sha512-xJCi2EBf0rQ68kSuQHdKpLDOTjQJUtFGHn0QlaVacchgZZaTiDfJKD5dQhuBJZaul5Nrjq8OOFzwRo2/G8uVIA==} | ||
engines: {node: '>=8.0.0'} | ||
hasBin: true | ||
requiresBuild: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated @sentry/types version. Check for compatibility and deprecated functions after update.
@@ -1267,15 +1267,15 @@ packages: | |||
node-abi: 3.56.0 | |||
dev: true | |||
|
|||
/@sentry/types@7.110.1: | |||
resolution: {integrity: sha512-sZxOpM5gfyxvJeWVvNpHnxERTnlqcozjqNcIv29SZ6wonlkekmxDyJ3uCuPv85VO54WLyA4uzskPKnNFHacI8A==} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Be aware of the new integrity value for '@sentry/utils'. If this package is critical to your application, comprehensive testing is recommended.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes are primarily updates to the version of '@sentry' dependencies across various files. It's good to stay up-to-date with the latest versions to get bug fixes and other improvements, but it's also important to be sure that these updates don't introduce any breaking changes. No function, methods or core coding changes are present in the pull request.
@@ -26,8 +26,8 @@ | |||
}, | |||
"devDependencies": { | |||
"@eslint/js": "^9.0.0", | |||
"@sentry/node": "7.110.1", | |||
"@sentry/profiling-node": "7.110.1", | |||
"@sentry/node": "7.111.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The '@sentry/node' package has been updated from version '7.110.1' to '7.111.0. Please ensure that this doesn't break any existing functionality, especially error reporting and event capturing.
"@sentry/node": "7.110.1", | ||
"@sentry/profiling-node": "7.110.1", | ||
"@sentry/node": "7.111.0", | ||
"@sentry/profiling-node": "7.111.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The '@sentry/profiling-node' package has been updated from version '7.110.1' to '7.111.0'. Check if this update includes new profiling features or changes that may impact current usage.
@@ -49,11 +49,11 @@ importers: | |||
specifier: ^9.0.0 | |||
version: 9.0.0 | |||
'@sentry/node': | |||
specifier: 7.110.1 | |||
version: 7.110.1 | |||
specifier: 7.111.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The '@sentry/node' package version has also been updated in the pnpm-lock.yaml file. It's good that package.json and pnpm-lock.yaml files are in sync.
specifier: 7.110.1 | ||
version: 7.110.1 | ||
specifier: 7.111.0 | ||
version: 7.111.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The '@sentry/profiling-node' package version update is also reflected in the pnpm-lock.yaml file. Consistency is maintained between package.json and pnpm-lock.yaml files.
/@sentry/node@7.110.1: | ||
resolution: {integrity: sha512-n6sNzZJ/ChfyCI1FxuGWgloeevC8j2vax3vXM4IZrSIm5hS1d9L2oCJ4HEPuxGUxCkQ1f4kXPcdmNaQsWH0JBw==} | ||
/@sentry/node@7.111.0: | ||
resolution: {integrity: sha512-bTLZNETT7W89HEk04rwsch02KSpu++Yec/BEyM3AxUNY+ZQ9ZLL/lrNZuCwbe7fURpKoZrvGAhxpPjgs5UcB9w==} | ||
engines: {node: '>=8'} | ||
dependencies: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dependencies of '@sentry/node' are also updated to the new version '7.111.0'. This is expected as all are part of the '@sentry' package.
'@sentry/utils': 7.110.1 | ||
'@sentry-internal/tracing': 7.111.0 | ||
'@sentry/core': 7.111.0 | ||
'@sentry/types': 7.111.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The '@sentry/profiling-node' integrity line has changed, signifying a new download was made due to the version change. Make sure this doesn't impact any current profiling workflow.
@@ -1267,15 +1267,15 @@ packages: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The '@sentry/types' package's integrity hash has been updated due to the new version '7.111.0'. Make sure this doesn't alter any type definitions that are currently being used.
@@ -1267,15 +1267,15 @@ packages: | |||
node-abi: 3.56.0 | |||
dev: true | |||
|
|||
/@sentry/types@7.110.1: | |||
resolution: {integrity: sha512-sZxOpM5gfyxvJeWVvNpHnxERTnlqcozjqNcIv29SZ6wonlkekmxDyJ3uCuPv85VO54WLyA4uzskPKnNFHacI8A==} | |||
/@sentry/types@7.111.0: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The '@sentry/utils' package's new version '7.111.0' has changed its integrity hash. Please check if utility functions and helpers still work in the same way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This review summarizes the changes made on the '@sentry' packages from version 7.110.1 to 7.111.0 in both the package.json and pnpm-lock.yaml files. From the diff, we can see that the new updates have been reflected correctly across all dependencies. However, please ensure that the updated packages are tested thoroughly for compatibility issues with the current system before these changes are merged into the main branch.
@@ -26,8 +26,8 @@ | |||
}, | |||
"devDependencies": { | |||
"@eslint/js": "^9.0.0", | |||
"@sentry/node": "7.110.1", | |||
"@sentry/profiling-node": "7.110.1", | |||
"@sentry/node": "7.111.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The '@sentry/node' package has been updated to version 7.111.0. Double check for any breaking changes in the new version and test thoroughly before merging.
"@sentry/node": "7.110.1", | ||
"@sentry/profiling-node": "7.110.1", | ||
"@sentry/node": "7.111.0", | ||
"@sentry/profiling-node": "7.111.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The '@sentry/profiling-node' package has been updated to version 7.111.0. This might affect profiling in the application. Validate the profiling after upgrading.
@@ -49,11 +49,11 @@ importers: | |||
specifier: ^9.0.0 | |||
version: 9.0.0 | |||
'@sentry/node': | |||
specifier: 7.110.1 | |||
version: 7.110.1 | |||
specifier: 7.111.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ensure to test the '@sentry/node' package thoroughly after updating to the new version 7.111.0 in the 'pnpm-lock.yaml' file.
'@sentry/profiling-node': | ||
specifier: 7.110.1 | ||
version: 7.110.1 | ||
specifier: 7.111.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The '@sentry/profiling-node' package version in the 'pnpm-lock.yaml' file has been updated to 7.111.0. Make sure this does not cause any unexpected issues with profiling.
@@ -107,7 +107,7 @@ importers: | |||
dependencies: | |||
'@sentry/node': | |||
specifier: ^7.102.0 | |||
version: 7.110.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The '@sentry/node' package version has been updated in multiple places within the 'pnpm-lock.yaml' file. Please confirm that all dependencies of this package are correctly updated.
@@ -1248,17 +1248,17 @@ packages: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sentry/node has been updated correctly in resolution, but make sure to test because this is a major dependency in this system.
'@sentry/types': 7.110.1 | ||
'@sentry/utils': 7.110.1 | ||
'@sentry-internal/tracing': 7.111.0 | ||
'@sentry/core': 7.111.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Version update spotted in '@sentry/profiling-node'. Test to make sure the new version doesn't break any functionalities.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes in this commit primarily update version numbers for various packages in the package.json and pnpm-lock.yaml files. It's important to ensure these updates don't introduce breaking changes or conflicts with other packages. If the testing passes for this commit and there are no noticeable issues in runtime, then these changes are generally fine. Overall, the changes appear routine and are unlikely to introduce major issues.
@@ -26,8 +26,8 @@ | |||
}, | |||
"devDependencies": { | |||
"@eslint/js": "^9.0.0", | |||
"@sentry/node": "7.110.1", | |||
"@sentry/profiling-node": "7.110.1", | |||
"@sentry/node": "7.111.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line updates @sentry/node library to a new version. While this might be necessary, it's always a good idea to verify what changes were introduced in this new version in relation to the project using it.
"@sentry/node": "7.110.1", | ||
"@sentry/profiling-node": "7.110.1", | ||
"@sentry/node": "7.111.0", | ||
"@sentry/profiling-node": "7.111.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once again a version upgrade for a library from @sentry. Same steps apply as in the previous comment: check for potential breaking changes introduced in version 7.111.0.
@@ -49,11 +49,11 @@ importers: | |||
specifier: ^9.0.0 | |||
version: 9.0.0 | |||
'@sentry/node': | |||
specifier: 7.110.1 | |||
version: 7.110.1 | |||
specifier: 7.111.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sentry node version is updated in the pnpm-lock.yaml as well, which is good for consistency, make sure the changes in the new version do not break existing functionality.
specifier: 7.110.1 | ||
version: 7.110.1 | ||
specifier: 7.111.0 | ||
version: 7.111.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The specified version of @sentry/profiling-node is updated here. Check the CHANGELOG or release notes for this version update.
/@sentry-internal/tracing@7.110.1: | ||
resolution: {integrity: sha512-4kTd6EM0OP1SVWl2yLn3KIwlCpld1lyhNDeR8G1aKLm1PN+kVsR6YB/jy9KPPp4Q3lN3W9EkTSES3qhP4jVffQ==} | ||
/@sentry-internal/tracing@7.111.0: | ||
resolution: {integrity: sha512-CgXly8rsdu4loWVKi2RqpInH3C2cVBuaYsx4ZP5IJpzSinsUAMyyr3Pc0PZzCyoVpBBXGBGj/4HhFsY3q6Z0Vg==} | ||
engines: {node: '>=8'} | ||
dependencies: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sentry/core version is updated. This is a core library and changes in this could potentially have significant impacts on your application, a thorough testing would be necessary ensure everything works as expected.
engines: {node: '>=8'} | ||
dependencies: | ||
'@sentry/core': 7.110.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sentry/types version is updated. Verify the changes in version 7.111.0 to be sure they don't break any existing code.
@@ -1248,17 +1248,17 @@ packages: | |||
- supports-color |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sentry/node dependency's integrity checksum has changed due to a new version. It is standard behavior when package content changes.
engines: {node: '>=8'} | ||
dependencies: | ||
'@sentry-internal/tracing': 7.110.1 | ||
'@sentry/core': 7.110.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sentry/profiling-node dependency's integrity checksum has changed due to a new version, which is expected.
@@ -1267,15 +1267,15 @@ packages: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sentry/types checksum has changed, ensure that it is reflected wherever the dependency has been used.
@@ -1267,15 +1267,15 @@ packages: | |||
node-abi: 3.56.0 | |||
dev: true | |||
|
|||
/@sentry/types@7.110.1: | |||
resolution: {integrity: sha512-sZxOpM5gfyxvJeWVvNpHnxERTnlqcozjqNcIv29SZ6wonlkekmxDyJ3uCuPv85VO54WLyA4uzskPKnNFHacI8A==} | |||
/@sentry/types@7.111.0: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sentry/utils checksum changed as the version has been updated. As with all package updates, please ensure this update works with the rest of the project code.
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found Additional details and impacted files@@ Coverage Diff @@
## main #1898 +/- ##
=======================================
Coverage 42.50% 42.50%
=======================================
Files 83 83
Lines 10329 10329
Branches 126 126
=======================================
Hits 4390 4390
Misses 5939 5939
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
This PR contains the following updates:
7.110.1
->7.111.0
7.110.1
->7.111.0
Release Notes
getsentry/sentry-javascript (@sentry/node)
v7.111.0
Compare Source
server.address
to browserhttp.client
spans (#11663)browserTracingIntegration
(#11647)Bundle size 📦
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR has been generated by Mend Renovate. View repository job log here.