fix(security): npm security fixes (2026-08-03) - #1436
fix(security): npm security fixes (2026-08-03)#1436github-actions[bot] wants to merge 2 commits into
Conversation
| "tmp@<0.2.6": "^0.2.6", | ||
| "js-cookie@<3.0.7": "^3.0.7", | ||
| "react-router@<7.18.0": "^7.18.0", | ||
| "react-router@<8.3.0": "^8.3.0", |
There was a problem hiding this comment.
This looks like a potential regression: the override now forces react-router to ^8.3.0, but react-router-dom@7.17.0 (used in the lockfile for libs/jitsu-react) declares a hard dependency on react-router@7.17.0. The resolved lockfile now pairs react-router-dom@7.17.0 with react-router@8.3.0, and react-router@8.3.0 also requires React >=19.2.7 while this repo is on React 18.3.1. Could we pin to a non-breaking 7.x security patch range instead?
There was a problem hiding this comment.
Reviewed the dependency override updates in package.json, webapps/console/package.json, and the lockfile refresh. I found one compatibility risk worth addressing (left inline) around the react-router override; the rest of the version bumps look consistent with the security update intent.
| "tmp@<0.2.6": "^0.2.6", | ||
| "js-cookie@<3.0.7": "^3.0.7", | ||
| "react-router@<7.18.0": "^7.18.0", | ||
| "react-router@<8.3.0": "^8.3.0", |
There was a problem hiding this comment.
This looks like a potential runtime regression: this override now forces react-router to ^8.3.0, but the lockfile still resolves react-router-dom@7.17.0, and that package gets paired with react-router@8.3.0. react-router@8.3.0 declares peers react >=19.2.7, while this repo is on React 18.3.1. Could we keep the fix on a safe 7.x line instead so react-router-dom@7.x stays compatible?
Batch npm dependency security updates for Dependabot alerts (minimum severity: moderate).
Included fixes:
Skipped (already satisfied / would downgrade):
Risks:
Verification: