Skip to content

Document npm overrides for dependency security management#1324

Closed
Copilot wants to merge 5 commits intomainfrom
copilot/fix-dependabot-security-alerts
Closed

Document npm overrides for dependency security management#1324
Copilot wants to merge 5 commits intomainfrom
copilot/fix-dependabot-security-alerts

Conversation

Copy link
Copy Markdown

Copilot AI commented Feb 6, 2026

npm overrides in package.json force specific package versions throughout the dependency tree. Used to fix 67 security vulnerabilities (1 critical, 18 high) in transitive dependencies, resolving 87% including all critical/high severity issues.

Documentation added

  • docs/npm-overrides-faq.md - Quick-start guide with examples and Q&A
  • SECURITY.md - Complete override list with CVE links, audit procedures, reporting process
  • docs/adr/0008-npm-overrides-for-security.md - Architecture decision record with technical rationale
  • Package READMEs - Security sections for api, app, cms, shared with package-specific overrides

Example override

{
  "overrides": {
    "lodash": "^4.17.23",
    "fast-xml-parser": "^5.3.4",
    "glob": "^11.1.0"
  }
}

Forces secure versions everywhere, even when nested packages request vulnerable versions.

Why overrides vs alternatives

  • Update direct deps: Insufficient - parent packages hadn't updated
  • Wait for upstream: Unacceptable delay for critical vulnerabilities
  • Major version upgrades: Breaking changes require separate effort (NestJS v10→v11, vue-tsc v1→v3)
  • Overrides: Immediate fix, no code changes, easily reversible

Results

Severity Fixed
Critical 1/1 (100%)
High 18/18 (100%)
Moderate/Low 37/48 (77%)
Total 56/67 (87%)

Remaining 11 moderate vulnerabilities require major version upgrades.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Fix all Dependabot security alerts Fix dependabot security alerts (87% resolved) Feb 6, 2026
Copilot AI requested a review from MrDirkelz February 6, 2026 14:43
Copilot AI changed the title Fix dependabot security alerts (87% resolved) Document npm overrides for dependency security management Feb 9, 2026
Copilot AI and others added 4 commits February 10, 2026 09:20
…eted updates

Co-authored-by: MrDirkelz <170883679+MrDirkelz@users.noreply.github.com>
Co-authored-by: MrDirkelz <170883679+MrDirkelz@users.noreply.github.com>
Co-authored-by: MrDirkelz <170883679+MrDirkelz@users.noreply.github.com>
Co-authored-by: MrDirkelz <170883679+MrDirkelz@users.noreply.github.com>
@MrDirkelz MrDirkelz force-pushed the copilot/fix-dependabot-security-alerts branch from cc276aa to 325caff Compare February 10, 2026 07:20
@MrDirkelz MrDirkelz marked this pull request as ready for review February 10, 2026 07:21
… and Fastify components to latest versions, and address security vulnerabilities by updating markdown-it to version 14.1.1. Clean up unused imports in db.service.ts.
@ivanslabbert
Copy link
Copy Markdown
Contributor

Close and delete associated branch

@MrDirkelz
Copy link
Copy Markdown
Collaborator

This has been fixed in a seperate PR

@MrDirkelz MrDirkelz closed this Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants