Skip to content

fix: bump nanoid to ^5.1.16 and vitest to ^4.1.11 (dependency vulnerabilities) - #40

Open
devin-ai-integration[bot] wants to merge 3 commits into
mainfrom
devin/dep-vulns-major-batch/2026-09-28
Open

devin-ai-integration[bot] wants to merge 3 commits into
mainfrom
devin/dep-vulns-major-batch/2026-09-28

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Dependency vulnerability fixes (automated)

Generated by the Devin Dependency Security Vuln Fix automation run on 2026-10-19 (first opened 2026-09-28).
Change type: Major version bump(s), manifest/lockfile only.

Warning

We're not just looking for a review like a normal PR. Security can offer this fix as a recommendation, but doesn't have the tooling or domain knowledge to safely verify changes like this end-to-end for each repo. Please review, test, and own deployment before merging. For more information: https://launchdarkly.atlassian.net/wiki/spaces/SEC/pages/5360943572/Dependency+Vulnerability+Remediation+with+Devin.

Findings addressed

Package Ecosystem Bump type Current → Target Severity Age Source(s) Advisory
nanoid (packages/web-extension) npm Major ^4.0.0 → ^5.1.16 High 130d Dependabot, Wiz GHSA-28wg-ghj8-5hjv, GHSA-2v37-7h3g-55p8, GHSA-xwg4-73v4-xw9w, GHSA-mwcw-c2x4-8c55 / CVE-2024-55565 (Dependabot #93–#96)
vitest (11 workspace packages) npm Major ^3.2.6 → ^4.1.11 Medium 39d Dependabot, Wiz GHSA-82fw-gwwq-j7x9 (Dependabot #97–#107)

Notes for reviewers:

Deferred / excluded findings

Verification

Re-checked on 2026-10-19 after merging main:

  • Install: ✅ yarn install resolves nanoid 5.1.16 and vitest 4.1.11. The only failure is packages/rrvideo's Playwright browser postinstall, which can't reach the Playwright CDN from the automation sandbox. This happens on main too.
  • Build: ✅ yarn build:all passes 22/22. Locally, web-extension's manifest schema validation had to be skipped because the sandbox can't fetch the schema; that change isn't committed.
  • Tests: ✅ yarn test on vitest 4 matches main locally, with one exception. rrweb cross-origin-iframes.test.ts > should map input events correctly fails in the sandbox's headless Chrome, but run on its own it fails on main too, so it's an environment issue. The other local failures, in rrdom, rrweb and rrvideo, are the same as on main. CI runs the full suite on this head.
  • Lint: ✅ yarn lint has no new errors. The only errors are the same 3 rrweb-player .svelte parser errors as on main.

…cy vulnerabilities

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants