Repository navigation
fix: override transitive uuid 8.3.2 -> 11.1.1 (major, dev-only via nyc) - #173
Open
devin-ai-integration[bot] wants to merge 2 commits into
Open
devin-ai-integration[bot] wants to merge 2 commits into
devin-ai-integration[bot] wants to merge 2 commits into
Conversation
…cessinfo (GHSA-w5hq-g745-h8pq) Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Contributor
Author
|
I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".
|
…-batch/2026-09-28
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Dependency vulnerability fixes (automated)
Generated by the Devin Dependency Security Vuln Fix automation run on 2026-10-19.
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
How: the only
uuid@^8consumer isistanbul-lib-processinfo@2.0.3. It comes in through thenycdevDependency, and no script in this repo currently runsnyc. It pinsuuid ^8.3.2, so no fix exists inside that range. Added a Yarn resolution:yarn.locknow foldsuuid@^8.3.2into the existinguuid@11.1.1entry, which the extension's directuuiddependency already uses, so no new package versions come in.istanbul-lib-processinfoonly callsconst { v4: uuidv4 } = require('uuid'), and uuid 11's CommonJS build still exports that.2026-10-19 update: the branch is still current with
main(0 commits behind) and Dependabot alert #216 is still open, so the change is the same. Install, build, lint and tests were re-run today. The Wiz MCP server was unavailable on 2026-10-12 and 2026-10-19, so the Wiz attribution comes from the 2026-10-05 reconcile.Deferred / excluded findings
uuid11.1.1 → 14.0.0) is unrelated to this transitive copy. It isn't a security fix, because the directuuid11.1.1 is already patched.Verification
yarn install --frozen-lockfile(Yarn 1.22.22, Node 24.19.0)yarn build:setup && yarn compile(webpack)xvfb-run -a yarn testlocally (VS Code test host, 13 passing). The 2026-09-28 targeted check also passed:nyc --silent node a.jsexits 0 withistanbul-lib-processinforesolvinguuid@11.1.1.yarn lintandyarn prettier:check. The repo's pre-commit hooks can't be installed in the sandbox because the hook repos can't be cloned, so the eslint and prettier runs above stand in for them.