chore: resolve open dependabot security alerts#1432
Conversation
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe root ChangesDependency Configuration Updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
There was a problem hiding this comment.
Pull request overview
This PR updates dev/build/test tooling dependency versions to address Dependabot security alerts, primarily via targeted overrides at the monorepo root and a patch-level Angular devDependency bump in the Angular workspace.
Changes:
- Bumped Angular workspace devDependencies for core framework packages from
^21.0.4to^21.2.17. - Added root-level
overridesto force patched transitive versions (e.g.,esbuild,vite,undici,multer,piscina,uuid, plus scoped overrides for@angular/*and@istanbuljs/*cases). - Added explicit root devDependency
react-dom@^18.2.0and@babel/core@^7.29.7to support the override/resolution strategy.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| packages/angular/package.json | Bumps Angular framework devDependencies to patched versions to address security alerts. |
| package.json | Adds root overrides for vulnerable transitive deps and pins additional devDependencies needed for stable resolution. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
tslib).overridesonly where a vulnerable version was exact-pinned by an upstream package (@angular/build,@angular/compiler-cli,@nestjs/platform-express,@istanbuljs/load-nyc-config) with no fixed upstream release available.Dependabot Alerts Resolved
@angular/core^21.2.17@angular/common^21.2.17@angular/compiler^21.2.17@babel/core@babel/core@^7.29.7devDep + scoped overrides for@angular/buildand@angular/compiler-cliesbuildoverridesto^0.28.1form-datahonojs-yaml@istanbuljs/load-nyc-configto^4.2.0multeroverridesto^2.2.0piscinaoverridesto^5.2.0tarundicioverridesto^7.28.0(6.x copy also resolved)uuid@cucumber/messagesandjest-cucumberto^11.1.1viteoverridesto^7.3.5Notes
react-dom@^18.2.0as an explicit root devDependency. It was previously an auto-installed peer dependency; pinning it keeps the lockfile resolvable with plainnpm ci(as CI uses) after the full lockfile regeneration required to apply theoverrides.@angular/build@21.2.16,@nestjs/platform-express) have no upstream release that bumps them. Scoped overrides were used where the affected version is local to one consumer.npm run build,npm test(590 jest tests + 45 angular tests pass), andnpm ciall succeed.