chore(deps): bump uuid from 8.3.2 to removed in the npm_and_yarn group across 1 directory - #3410
Conversation
Bumps the npm_and_yarn group with 1 update in the / directory: [uuid](https://github.com/uuidjs/uuid). Removes `uuid` --- updated-dependencies: - dependency-name: uuid dependency-version: dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com>
🔍 Dependency Analysis Summary
Actual changes:
Several supporting transitives also changed ( Overall risk: LOW, with one packaging issue that should be addressed before merging — see below. 📋 Detailed Changelog Review
|
| Requirement | Project value | OK? |
|---|---|---|
@sentry/webpack-plugin@5 needs Node ≥ 18 |
runtime is Node 24.x | ✅ |
@sentry/webpack-plugin@5 needs webpack ≥ 5 |
"webpack": "^5.105.0" |
✅ |
cypress@15.15.0 engine |
Node 24.x | ✅ |
Code usage check
webpack.config.js:1,117uses onlysentryWebpackPlugin({ authToken, org, project, telemetry }). These options are unchanged in v5 — no code changes required.cypress.config.tsandcypress/support/*use standard, stable APIs. Nocy.end()usage.uuidis not imported by any source underassets/,components/chat_widget/src/, orcypress/— it was purely transitive.
🐛 Lockfile inconsistency (please address before merging)
package-lock.json adds cypress: "15.15.0" to the root dependencies block in addition to optionalDependencies, while package.json only lists it under optionalDependencies:
package.json:87-89→optionalDependencies: { cypress: "^15.15.0" }package-lock.json:25→ adds"cypress": "15.15.0"todependencies(no caret, not in package.json)package-lock.json:79-81→ also keepsoptionalDependencies: { cypress: "^15.15.0" }
This looks like an npm install --save cypress@15.15.0 rather than a managed lockfile refresh. The risk: production installs that skip optionalDependencies may now still pull cypress (and its ~120 MB binary download script) because it appears under regular dependencies in the lock. Recommend regenerating the lockfile cleanly, e.g.:
rm package-lock.json node_modules/.package-lock.json
npm install --include=optional
…and confirming the resulting lock keeps cypress only under optionalDependencies.
🛠️ Recommendations
- Action required: Fix the
cypresslockfile duplication before merge (see above). After that, the upgrade is safe. - Testing focus:
- Production webpack build with sourcemap upload to Sentry — confirm
SENTRY_AUTH_TOKEN/SENTRY_ORG/SENTRY_PROJECTflow still works against thesentry-cli2.58.x binary on the CI runner. The branch inwebpack.config.js:113only fires onGITHUB_REF=refs/heads/main, so this won't exercise on the PR itself. npm run devandnpm run buildsucceed locally and in CI.- One cypress run (
npm run cypress:run) to confirm the@cypress/requestv4 transition is fine.
- Production webpack build with sourcemap upload to Sentry — confirm
- Follow-up tasks:
- License-review the
@sentry/cliswitch to FSL-1.1-MIT for the prebuilt CLI binaries if your org tracks third-party licenses.
- License-review the
- Merge recommendation: REVIEW_NEEDED — approve the dep bumps, but fix the lockfile entry for
cypressfirst.
Bumps the npm_and_yarn group with 1 update in the / directory: uuid.
Removes
uuidDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditionsYou can disable automated security fix PRs for this repo from the Security Alerts page.