fix(deps): update sentry-javascript monorepo to v7.112.0 #1910
Conversation
![renovate[bot]](https://avatars.githubusercontent.com/in/2740?s=60&v=4){kind=small}
|
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 (
|
![codecov[bot]](https://avatars.githubusercontent.com/in/254?s=60&v=4){kind=small}
There was a problem hiding this comment.
The changes mainly consist of package version updates, specifically updates to sentry packages from version 7.111.0 to 7.112.0 and the addition of a new dependency 'localforage'. Given the version updates are minor, I will assume they don't introduce any breaking changes. However, prior to merging these changes, running the test suite and manual verification should be done to ensure compatibility. Also, it would be necessary to understand the changes and benefits brought by the new 'localforage' dependency, ensuring its addition is critical and would not just add unnecessary complexity.
| @@ -26,8 +26,8 @@ | |||
| }, | |||
| "devDependencies": { | |||
| "@eslint/js": "^9.0.0", | |||
| "@sentry/node": "7.111.0", | |||
| "@sentry/profiling-node": "7.111.0", | |||
| "@sentry/node": "7.112.0", | |||
There was a problem hiding this comment.
Version numbers of @sentry/node and @sentry/profiling-node have been updated. Please ensure their compatibility with the current codebase by executing test suite and necessary manual testing.
| @@ -49,11 +49,11 @@ importers: | |||
| specifier: ^9.0.0 | |||
| version: 9.0.0 | |||
| '@sentry/node': | |||
| specifier: 7.111.0 | |||
| version: 7.111.0 | |||
| specifier: 7.112.0 | |||
There was a problem hiding this comment.
Version numbers in package-lock file have also been updated. This ensures that the same version is installed across all environments.
| @@ -1258,17 +1258,27 @@ packages: | |||
There was a problem hiding this comment.
The @sentry/integrations package has been updated to version 7.112.0. Please verify its changes and compatibility with the project.
| engines: {node: '>=8'} | ||
| dependencies: | ||
| '@sentry-internal/tracing': 7.111.0 | ||
| '@sentry/core': 7.111.0 |
There was a problem hiding this comment.
'localforage' has been included as a new dependency. Please provide justification for its inclusion and ensure it does not introduce unnecessary complexity.
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@sentry/node@7.111.0, npm/@sentry/profiling-node@7.111.0 |
|
Here's the code health analysis summary for commits Analysis Summary
|
There was a problem hiding this comment.
The updates involve many Sentry modules. Please make sure to test these changes extensively as Sentry is a crucial part of error-tracing and logging. Also, take note of a new dependency localforage and make sure that it is compatible with the rest of the application.
| engines: {node: '>=8'} | ||
| dependencies: | ||
| '@sentry-internal/tracing': 7.112.0 | ||
| '@sentry/core': 7.112.0 |
There was a problem hiding this comment.
Check if '@sentry/profiling-node' version '7.112.0' has any major changes that might affect the application. Consult the changelog/documentation of the library for any major changes.
| '@sentry/types': 7.111.0 | ||
| '@sentry/utils': 7.111.0 | ||
| '@sentry/core': 7.112.0 | ||
| '@sentry/types': 7.112.0 |
There was a problem hiding this comment.
Ensure '@sentry/node' has been properly updated to version '7.112.0' throughout your codebase.
| @@ -1277,15 +1287,15 @@ packages: | |||
There was a problem hiding this comment.
The '@sentry/types' version '7.112.0' may bring some new types or modifications to existing ones. If your project is type-checked, ensure it does not break any existing type checks.
| /lie@3.1.1: | ||
| resolution: {integrity: sha512-RiNhHysUjhrDQntfYSfY4MU24coXXdEOgw9WGcKHNeEwffDYbF//u87M1EWaMGzuFoSbqW0C9C6lEEhDOAswfw==} | ||
| dependencies: | ||
| immediate: 3.0.6 |
There was a problem hiding this comment.
You've added a new external dependency localforage. Confirm that this package is necessary and used within the application, and that it complies with the software's license restrictions.
| @@ -2487,6 +2497,9 @@ packages: | |||
| engines: {node: '>= 4'} | |||
| dev: true | |||
|
|
|||
| /immediate@3.0.6: | |||
| resolution: {integrity: sha512-XXOFtyqDjNDAQxVfYxuF7g9Il/IbWmmlQg2MYKOH8ExIT1qg6xc4zyS3HaEEATgs1btfzxq15ciUiY7gjSXRGQ==} | |||
|
|
|||
There was a problem hiding this comment.
There is a dependency added 'lie', it is recommended to verify its usage and its compatibility with other parts of application.
| engines: {node: '>=8'} | ||
| dependencies: | ||
| '@sentry/types': 7.111.0 | ||
| '@sentry/types': 7.112.0 |
There was a problem hiding this comment.
Addtion of dependency 'immediate' detected. Make sure this dependency is necessary and does not impose security threats.
| @@ -1239,12 +1239,12 @@ packages: | |||
| - supports-color | |||
| dev: false | |||
There was a problem hiding this comment.
'@sentry/core' has been updated to a new version. Make sure this version upgrade does not impact the functionalities where it is used.
| /@sentry/node@7.111.0: | ||
| resolution: {integrity: sha512-bTLZNETT7W89HEk04rwsch02KSpu++Yec/BEyM3AxUNY+ZQ9ZLL/lrNZuCwbe7fURpKoZrvGAhxpPjgs5UcB9w==} | ||
| /@sentry/integrations@7.112.0: | ||
| resolution: {integrity: sha512-brN6eZkXuz1e/OKhMGJsAZjc0cUU+5G+LQWet+gGXWVGM2v7uY7mKDHr5Yl/c8WxeJBurjJzJn7YmtmR9++ZKQ==} | ||
| engines: {node: '>=8'} | ||
| dependencies: |
There was a problem hiding this comment.
Notice the new dependency '@sentry/integrations'. Validate its requirement and usage within the application.
| @@ -1258,17 +1258,27 @@ packages: | |||
There was a problem hiding this comment.
A new version of '@sentry/node' has been introduced, ensuring there are no breaking changes impacting the application.
| @@ -26,8 +26,8 @@ | |||
| }, | |||
| "devDependencies": { | |||
| "@eslint/js": "^9.0.0", | |||
| "@sentry/node": "7.111.0", | |||
| "@sentry/profiling-node": "7.111.0", | |||
| "@sentry/node": "7.112.0", | |||
There was a problem hiding this comment.
Package '@sentry/node' version has been updated, verify if the new version is compatible with your application.
There was a problem hiding this comment.
This code review covers changes made for upgrading and adding dependencies in the project's package.json and pnpm-lock.yaml files, presumably to implement new features or bug fixes or to improve security. Attention should be paid to testing these updates thoroughly, following each package's documentation to ensure any relevant breaking changes are taken into account.
| @@ -26,8 +26,8 @@ | |||
| }, | |||
| "devDependencies": { | |||
| "@eslint/js": "^9.0.0", | |||
| "@sentry/node": "7.111.0", | |||
| "@sentry/profiling-node": "7.111.0", | |||
| "@sentry/node": "7.112.0", | |||
There was a problem hiding this comment.
Note that @sentry/node has been updated from version 7.111.0 to 7.112.0. Ensure the change is tested thoroughly and confirms to any breaking changes in the documentation of this package.
| "@sentry/node": "7.111.0", | ||
| "@sentry/profiling-node": "7.111.0", | ||
| "@sentry/node": "7.112.0", | ||
| "@sentry/profiling-node": "7.112.0", |
There was a problem hiding this comment.
@sentry/profiling-node is updated from version 7.111.0 to 7.112.0. It is advisable to refer to the package's changelog or release notes to make sure it will not introduce any breaking changes.
| @@ -1258,17 +1258,27 @@ packages: | |||
There was a problem hiding this comment.
A new package '@sentry/integrations@7.112.0' is introduced. Verifying its need and use-case in the project would be beneficial. This package also has an extra dependency on 'localforage: 1.10.0', make sure to confirm that this does not conflict with any existing dependencies.
| engines: {node: '>=8'} | ||
| dependencies: | ||
| '@sentry-internal/tracing': 7.111.0 | ||
| '@sentry/core': 7.111.0 | ||
| '@sentry/types': 7.111.0 |
There was a problem hiding this comment.
The 'localforage' package has been added as version 1.10.0. Confirm that its addition to the project does not result in compatibility issues with other packages.
| @@ -2487,6 +2497,9 @@ packages: | |||
| engines: {node: '>= 4'} | |||
| dev: true | |||
|
|
|||
| /immediate@3.0.6: | |||
| resolution: {integrity: sha512-XXOFtyqDjNDAQxVfYxuF7g9Il/IbWmmlQg2MYKOH8ExIT1qg6xc4zyS3HaEEATgs1btfzxq15ciUiY7gjSXRGQ==} | |||
There was a problem hiding this comment.
The 'lie' package has been added as version 3.1.1. If it was indirectly installed through another package, this doesn't require any intervention. However, if it was manually added, validate that the package is being used as per its documentation.
| @@ -2683,6 +2696,11 @@ packages: | |||
| type-check: 0.4.0 | |||
| dev: true | |||
|
|
|||
| /lie@3.1.1: | |||
| resolution: {integrity: sha512-RiNhHysUjhrDQntfYSfY4MU24coXXdEOgw9WGcKHNeEwffDYbF//u87M1EWaMGzuFoSbqW0C9C6lEEhDOAswfw==} | |||
| dependencies: | |||
There was a problem hiding this comment.
@sentry/core, @sentry/types, and @sentry/utils packages have been updated to version 7.112.0 from 7.111.0. Testing these updates thoroughly is recommended to avoid any unexpected behaviour.
There was a problem hiding this comment.
The git diff shows that several packages have been updated, particularly several Sentry packages, from version 7.111.0 to 7.112.0 across both package.json and pnpm-lock.yaml. This typically suggests improvements, bug fixes, etc. However, caution is needed because while the new version may bring new features and bug fixes, it may also introduce new bugs or compatibility issues. No other major changes have been made.
| @@ -26,8 +26,8 @@ | |||
| }, | |||
| "devDependencies": { | |||
| "@eslint/js": "^9.0.0", | |||
| "@sentry/node": "7.111.0", | |||
| "@sentry/profiling-node": "7.111.0", | |||
| "@sentry/node": "7.112.0", | |||
There was a problem hiding this comment.
The version for '@sentry/node' package has been upgraded from 7.111.0 to 7.112.0. Make sure to look into the changes in this version to confirm if there would be no breaking changes.
| "@sentry/node": "7.111.0", | ||
| "@sentry/profiling-node": "7.111.0", | ||
| "@sentry/node": "7.112.0", | ||
| "@sentry/profiling-node": "7.112.0", |
There was a problem hiding this comment.
The package '@sentry/profiling-node' has been upgraded to 7.112.0. Review the update documentation to understand what changes this new version brings.
| @@ -49,11 +49,11 @@ importers: | |||
| specifier: ^9.0.0 | |||
| version: 9.0.0 | |||
| '@sentry/node': | |||
| specifier: 7.111.0 | |||
| version: 7.111.0 | |||
| specifier: 7.112.0 | |||
There was a problem hiding this comment.
'@sentry/node' version has been updated in pnpm-lock.yaml as well. If any problems arise after this update, this would be one of the places to check for possible reasons.
| /@sentry-internal/tracing@7.111.0: | ||
| resolution: {integrity: sha512-CgXly8rsdu4loWVKi2RqpInH3C2cVBuaYsx4ZP5IJpzSinsUAMyyr3Pc0PZzCyoVpBBXGBGj/4HhFsY3q6Z0Vg==} | ||
| /@sentry-internal/tracing@7.112.0: | ||
| resolution: {integrity: sha512-PkA3NaSg4nTWp9pwVsV9x0EBiY0pEAnIboIpMuLGE5MJ/FL10NC5Fn1GPebcxNnOou62dM7P/z7Wtcm8czAn6A==} | ||
| engines: {node: '>=8'} | ||
| dependencies: |
There was a problem hiding this comment.
'@sentry/core' version has been updated. Make sure to check the changelog for this version to understand what modifications were made.
| @@ -1258,17 +1258,27 @@ packages: | |||
There was a problem hiding this comment.
A new package '@sentry/integrations' has been introduced in the version 7.112.0, and dependencies like '@sentry/core', '@sentry/types' and '@sentry/utils' have been updated. It would be prudent to check the documentation of this new package, understand its purpose and confirm that this package does not conflict with existing code.
| '@sentry/utils': 7.111.0 | ||
| '@sentry/core': 7.112.0 | ||
| '@sentry/types': 7.112.0 | ||
| '@sentry/utils': 7.112.0 |
There was a problem hiding this comment.
The '@sentry/node' package has new dependencies '@sentry-internal/tracing' and '@sentry/integrations' mentioned in the version 7.112.0. Be aware of these new dependencies and check if they introduce any breaking changes to the existing 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 #1910 +/- ##
==========================================
- Coverage 42.42% 42.41% -0.01%
==========================================
Files 83 83
Lines 10351 10351
Branches 161 110 -51
==========================================
- Hits 4391 4390 -1
- Misses 5960 5961 +1
*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.111.0->7.112.07.111.0->7.112.0Release Notes
getsentry/sentry-javascript (@sentry/node)
v7.112.0Compare Source
Important Changes
Instead of installing
@sentry/integrations, you can now import the pluggable integrations directly from your SDKpackage:
Note that only the functional integrations (e.g.
xxxIntegration()) are re-exported.Other Changes
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.