Skip to content

fix: remediate npm audit findings - #8624

Closed
TGambit65 wants to merge 1 commit into
NousResearch:mainfrom
TGambit65:fix/npm-audit-remediation
Closed

fix: remediate npm audit findings#8624
TGambit65 wants to merge 1 commit into
NousResearch:mainfrom
TGambit65:fix/npm-audit-remediation

Conversation

@TGambit65

Copy link
Copy Markdown

Summary

  • pin @appium/logger to 2.0.6 via an npm override to eliminate the remaining root browser-tool audit findings
  • refresh the root package-lock.json with the patched dependency graph
  • update the WhatsApp bridge lockfile to patched transitive versions

Verification

  • ran hermes doctor -> All checks passed
  • ran npm audit in the repo root -> 0 vulnerabilities
  • ran npm audit in scripts/whatsapp-bridge -> 0 vulnerabilities

Notes

  • the tinker-atropos submodule was initialized and installed locally to satisfy doctor, but that installation is environment state rather than a tracked repo change

- pin @appium/logger to 2.0.6 via npm override
- refresh root package-lock.json
- update whatsapp-bridge lockfile to patched dependency versions
@TGambit65

Copy link
Copy Markdown
Author

Heads up: the PR's GitHub Actions workflows are currently in an action_required state rather than actually running. That usually means a maintainer needs to approve workflow execution for this fork PR before Tests / Docker Build / Supply Chain Audit can start. Once they’re approved and running, I’ll inspect any actual failures.

@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the contribution, @TGambit65! These npm audit findings were resolved by a maintainer in PR #8745, which merged on 2026-04-13 (the same day this PR was opened).

Running npm audit on current main confirms 0 vulnerabilities — the fix landed via a different approach (no @appium/logger override pin required).

Evidence:

Closing as implemented on main. This is an automated hermes-sweeper review.

@teknium1 teknium1 closed this Apr 28, 2026
@alt-glitch alt-glitch added type/security Security vulnerability or hardening P3 Low — cosmetic, nice to have tool/browser Browser automation (CDP, Playwright) labels Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have tool/browser Browser automation (CDP, Playwright) type/security Security vulnerability or hardening

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants