chore: resolve open dependabot security alerts - #1446
Conversation
- undici -> 7.29.0 via override on @angular/build (high, alerts #264,263,262,261,260) - @hono/node-server -> 2.1.0 via override on @modelcontextprotocol/sdk (high, alert #232) - uuid -> 11.1.1 via override on @cucumber/messages and jest-cucumber (high, alert #179) - ip-address -> 10.4.0 (moderate, alerts covering transitive resolution) - js-yaml -> 4.3.1 (eslint dependency, moderate) - postcss -> 8.5.25 (moderate) - @angular/core, @angular/common, @angular/compiler, @angular/compiler-cli, @angular/animations, @angular/forms, @angular/platform-browser, @angular/platform-browser-dynamic, @angular/router -> 21.2.19 (moderate) Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
📝 WalkthroughWalkthroughThis PR adds npm package overrides for transitive dependencies (undici, hono/node-server, uuid) in the root package.json, and updates Angular framework package versions from ^21.2.17/^21.0.4 to ^21.2.19 in packages/angular/package.json and packages/angular/projects/angular-sdk/package.json. ChangesDependency version management
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/angular/package.json`:
- Around line 24-34: Update `@angular/common`, `@angular/compiler`,
`@angular/compiler-cli`, and `@angular/core` in packages/angular/package.json to
^21.2.19, then refresh the lockfile so the Angular dependency graph is aligned.
Also verify the sibling entries in
packages/angular/projects/angular-sdk/package.json at lines 28-31; no direct
change is needed there unless the lockfile refresh requires it.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 81ae27f7-ef7f-46cc-9a36-4728ace7119a
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (3)
package.jsonpackages/angular/package.jsonpackages/angular/projects/angular-sdk/package.json
| "@angular/animations": "^21.2.19", | ||
| "@angular/build": "^21.2.19", | ||
| "@angular/cli": "^21.2.19", | ||
| "@angular/common": "^21.2.17", | ||
| "@angular/compiler": "^21.2.17", | ||
| "@angular/compiler-cli": "^21.2.17", | ||
| "@angular/core": "^21.2.17", | ||
| "@angular/forms": "^21.2.17", | ||
| "@angular/platform-browser": "^21.2.17", | ||
| "@angular/platform-browser-dynamic": "^21.2.17", | ||
| "@angular/router": "^21.2.17", | ||
| "@angular/forms": "^21.2.19", | ||
| "@angular/platform-browser": "^21.2.19", | ||
| "@angular/platform-browser-dynamic": "^21.2.19", | ||
| "@angular/router": "^21.2.19", |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
printf 'Relevant package.json version entries:\n'
for f in packages/angular/package.json packages/angular/projects/angular-sdk/package.json; do
echo "--- $f"
python3 - <<'PY' "$f"
import json, sys
p=sys.argv[1]
with open(p) as fh:
data=json.load(fh)
for sec in ('dependencies','devDependencies','peerDependencies','overrides','resolutions'):
deps=data.get(sec)
if deps:
print(f'[{sec}]')
for k,v in sorted(deps.items()):
if k.startswith('`@angular/`'):
print(f'{k}: {v}')
PY
done
printf '\nAll Angular package declaration sites:\n'
rg -n '"`@angular/`(animations|build|cli|common|compiler|compiler-cli|core|forms|platform-browser|platform-browser-dynamic|router|openfeature)"\s*:' -S .Repository: open-feature/js-sdk
Length of output: 5191
🏁 Script executed:
#!/bin/bash
set -euo pipefail
printf 'Angular package lock/installer files present:\n'
git ls-files | rg '(^|/)(package-lock\.json|pnpm-lock\.yaml|yarn\.lock|bun\.lockb|bun\.lock)$|packages/angular/(package-lock\.json|pnpm-lock\.yaml|yarn\.lock|bun\.lockb|bun\.lock)$' || true
printf '\nLockfile Angular version entries (top-level occurrences if present):\n'
for lock in package-lock.json pnpm-lock.yaml yarn.lock bun.lock; do
if [ -f "$lock" ]; then
echo "--- $lock"
rg -n '`@angular/`(common|compiler|compiler-cli|core|form|platform-browser|router|animations|build|cli)',max-count=80 "$lock" || true
fi
doneRepository: open-feature/js-sdk
Length of output: 313
Align Angular package versions to ^21.2.19. packages/angular/package.json still pins @angular/common, @angular/compiler, @angular/compiler-cli, and @angular/core to ^21.2.17, while sibling Angular packages in this package and the SDK dependency entries use ^21.2.19. Update those four ranges to ^21.2.19 and refresh the lockfile to keep the Angular graph aligned.
📍 Affects 2 files
packages/angular/package.json#L24-L34(this comment)packages/angular/projects/angular-sdk/package.json#L28-L31
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@packages/angular/package.json` around lines 24 - 34, Update `@angular/common`,
`@angular/compiler`, `@angular/compiler-cli`, and `@angular/core` in
packages/angular/package.json to ^21.2.19, then refresh the lockfile so the
Angular dependency graph is aligned. Also verify the sibling entries in
packages/angular/projects/angular-sdk/package.json at lines 28-31; no direct
change is needed there unless the lockfile refresh requires it.
Summary
overridesonly where a vulnerable transitive version was pinned by a parent with no direct upgrade path)Dependabot Alerts Resolved
undici^7.29.0scoped to@angular/buildundici^7.29.0scoped to@angular/buildundici^7.29.0scoped to@angular/buildundici^7.29.0scoped to@angular/buildundici^7.29.0scoped to@angular/build@hono/node-server^2.1.0scoped to@modelcontextprotocol/sdkuuid^11.1.1scoped to@cucumber/messagesandjest-cucumberip-address10.4.0js-yaml4.3.1(eslint copy)postcss8.5.25@angular/*@angular/core,@angular/common,@angular/compiler,@angular/compiler-cli,@angular/animations,@angular/forms,@angular/platform-browser,@angular/platform-browser-dynamic,@angular/routerto21.2.19Notes on the
@hono/node-serveroverride: version2.0.5-2.0.9carry a separate known memory-leak DoS advisory (GHSA-9mqv-5hh9-4cgg), so the override target was set to^2.1.0rather than the minimum patched version.Verification
npm cisucceeds cleanly from a fresh installnpm run buildpassesnpm run test:jestpasses (35/35 suites, 612/612 tests)npm run test:angularpasses (3/3 suites, 45/45 tests)npm auditshows only pre-existing findings unrelated to the 14 resolved alerts