Repository navigation
fix: resolve uuid to 11.1.1 in React-Native example - #23
Open
devin-ai-integration[bot] wants to merge 1 commit into
Open
devin-ai-integration[bot] wants to merge 1 commit into
devin-ai-integration[bot] wants to merge 1 commit into
Conversation
Contributor
Author
|
I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".
|
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
examples/React-Native, transitive)Implemented as yarn
resolutions(uuid@^7.0.3,uuid@^8.0.0,uuid@^8.3.2→11.1.1, matching the existing pattern in thatpackage.json) plus the regeneratedyarn.lock. No source changes. The parents that pin uuid^7/^8(xcode,@expo/rudder-sdk-node,@expo/bunyan,@expo/code-signing-certificates) only use the CommonJS named exportsrequire('uuid').v4/.v1, which uuid 11 still provides. The Expo 57 upgrade in #18 would be the longer-term way to drop these old parents.Deferred / excluded findings
image-size1.2.1 → 2.0.3: High, 25d (GHSA-w3rx-r6r6-pgpr, GHSA-5p2g-fcmc-qvqq). There's no 1.x fix.metro0.81.4 callsrequire("image-size")(...)as a function, but in 2.x that export is an object ({ imageSize, default, ... }), so asset bundling would break. Every Metro release up to 0.83 pinsimage-size ^1.0.2, and only 0.87+ drops it. The fix is an Expo/React Native upgrade (React Native example: Expo 57 / TS 7 / Babel 8 upgrade + Observability + funnel experiment + CI #18).node-forge1.4.0,braces3.0.3,sprintf-js1.1.3, Wizhttp-cache-semantics; Wiz end-of-life library findings forglob,rimraf,inflightand the@babel/plugin-proposal-*plugins.@babel/core(GHSA-4x5r-pxfx-6jf8).compression1.8.2 andnext16.3.8 are still in the 7-day hold.Verification
2026-10-19: branch is 0 commits behind
main(unchanged since the 2026-10-05 verification below), so no re-merge was needed. Wiz MCP was unavailable this run (Dependabot-only).yarn install(Yarn 3.4.1 via@yarnpkg/cli-dist, Node 24) inexamples/React-Nativenpx tsc --noEmit; ✅npx expo export --platform androidand--platform ios(with a placeholderMOBILE_KEYin an uncommitted.env). Runtime smoke test:xcodeparsed a real.pbxproj, rangenerateUuid()/addPbxGroup()and wrote it back;uuid.v4()/v1()and@expo/rudder-sdk-nodeload fine.npx expo prebuild --platform iosfails identically onmain(existingtar7 resolution), unrelated to this change.