Skip to content

chore(deps): refresh npm locks to clear audit advisories - #39304

Closed
EvilHumphrey wants to merge 2 commits into
NousResearch:mainfrom
EvilHumphrey:security/npm-locks-20260604
Closed

chore(deps): refresh npm locks to clear audit advisories#39304
EvilHumphrey wants to merge 2 commits into
NousResearch:mainfrom
EvilHumphrey:security/npm-locks-20260604

Conversation

@EvilHumphrey

Copy link
Copy Markdown
Contributor

Summary

Refresh npm lockfiles to clear current npm audit advisories across the root workspace, website, and WhatsApp bridge dependency trees.

Changed files are lockfile-only:

  • package-lock.json
  • website/package-lock.json
  • scripts/whatsapp-bridge/package-lock.json

Audit impact

Before this lockfile refresh:

  • root npm audit: 2 high vulnerabilities
  • website npm audit: 9 vulnerabilities (3 high, 6 moderate)
  • scripts/whatsapp-bridge npm audit: 4 moderate vulnerabilities

After this lockfile refresh:

  • root npm audit: 0 vulnerabilities
  • website npm audit: 0 vulnerabilities
  • scripts/whatsapp-bridge npm audit: 0 vulnerabilities

The refreshed transitive/package versions include fixes around react-router, react-router-dom, express / body-parser, qs, ws, protobufjs, path-to-regexp, picomatch, postcss, brace-expansion, and undici.

Verification

Sanity/privacy check:

  • commit diff is limited to the three package-lock files above
  • checked the diff body for personal paths, tokens, secrets, and credential-like strings; no matches found
  • commit metadata uses the GitHub noreply address associated with the fork account

Commands run locally:

hermes doctor
npm audit
(cd website && npm audit)
(cd scripts/whatsapp-bridge && npm audit)
npm -w web run build
npm -w apps/desktop run type-check

Results:

  • hermes doctor: passed
  • root npm audit: found 0 vulnerabilities
  • website npm audit: found 0 vulnerabilities
  • scripts/whatsapp-bridge npm audit: found 0 vulnerabilities
  • npm -w web run build: passed
  • npm -w apps/desktop run type-check: passed

Note: existing web lint debt is intentionally not included in this PR; this is a narrow lockfile-only audit cleanup.

@EvilHumphrey
EvilHumphrey requested a review from a team June 4, 2026 20:38
@alt-glitch alt-glitch added type/security Security vulnerability or hardening P3 Low — cosmetic, nice to have dependencies Pull requests that update a dependency file javascript labels Jun 4, 2026
@EvilHumphrey

Copy link
Copy Markdown
Contributor Author

Closing in favor of cleaner replacements off current main:

This PR had become conflicted against main and bundled the Nix-coupled root lock together with the standalone locks, forcing the whole change to wait on the Nix step. Splitting it for reviewability.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file P3 Low — cosmetic, nice to have type/security Security vulnerability or hardening

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants