Skip to content

fix(security): clear dependency advisories - #17

Merged
jason-allen-oneal merged 2 commits into
mainfrom
fix/dependency-security-20260830
Aug 30, 2026
Merged

fix(security): clear dependency advisories#17
jason-allen-oneal merged 2 commits into
mainfrom
fix/dependency-security-20260830

Conversation

@jason-allen-oneal

Copy link
Copy Markdown
Member

Summary

  • isolate patched Nodemailer 9.0.6 behind an application-only npm alias so NextAuth 4 does not impose its optional Nodemailer 7 peer range
  • pin patched transitive versions for Babel, Ajv, flatted, minimatch, and picomatch
  • remove the temporary production-audit exception

Verification

  • npm ci
  • npm ls --all --omit=optional
  • npm audit: 0 vulnerabilities
  • npm run audit:prod
  • npm run lint
  • npm run typecheck
  • npm test: 20/20
  • npm run test:integration against disposable MySQL 8.4: 1/1
  • npm run build
  • git diff --check

Supersedes the incompatible Nodemailer major PR #14 with a valid dependency graph.

@jason-allen-oneal
jason-allen-oneal enabled auto-merge (squash) August 30, 2026 22:43

@morrow-bluedot morrow-bluedot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Independent exact-head review at dd9c419: lock integrity, full npm audit, production audit, lint, typecheck, 20 tests, and hosted build all pass. The app-only Nodemailer 9 alias is coherent with the unused optional NextAuth peer, and flatted is current at 3.4.4.

@jason-allen-oneal
jason-allen-oneal merged commit bd4b4a4 into main Aug 30, 2026
1 check passed
@jason-allen-oneal
jason-allen-oneal deleted the fix/dependency-security-20260830 branch August 30, 2026 23:16
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.

2 participants