fix(deps): resolve Dependabot alerts #657 and #658 (postcss, uuid)#236
Conversation
Add overrides in expo-purchasely-test to enforce patched floors: - postcss >=8.5.10 (GHSA-qx2v-qp2m-jg93, XSS via unescaped </style>) - uuid >=14.0.0 (GHSA-w5hq-g745-h8pq, buffer bounds check) Lockfile resolves to postcss 8.5.12 and uuid 14.0.0; npm audit reports 0 vulnerabilities. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
| Filename | Overview |
|---|---|
| test-projects/expo-purchasely-test/package.json | Adds postcss >=8.5.10 override; uuid override removed per previous review feedback — clean dependency security fix |
| test-projects/expo-purchasely-test/package-lock.json | Lockfile updated: postcss bumped to 8.5.12, nanoid range updated to ^3.3.11 within postcss entry, and uuid@7.0.3 re-scoped to node_modules/xcode/node_modules/uuid (no longer hoisted) to avoid CJS compatibility issues |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[package.json overrides] -->|postcss override added| B["postcss@8.5.12 - GHSA-qx2v-qp2m-jg93 fixed"]
A -->|uuid override removed| C[No top-level uuid hoisting]
C --> D["xcode@5.0.1 - requires uuid ^7.0.3"]
D -->|nested scoped install| E["node_modules/xcode/node_modules/uuid@7.0.3 - CJS compat preserved"]
B --> F[npm audit: 0 vulnerabilities]
E --> F
Reviews (2): Last reviewed commit: "fix(deps): drop uuid override (incompati..." | Re-trigger Greptile
Greptile flagged that uuid v10+ dropped CommonJS support and v14 requires
Node ≥20, which would break the only consumer in this tree:
xcode@3.0.1 declares uuid ^7.0.3 and uses require('uuid') (CJS).
GHSA-w5hq-g745-h8pq has no patched version in the v7 range, so we
cannot satisfy both xcode and the advisory. Per Greptile's second
recommendation, accept the audit finding for this build-time dev
dependency that never processes user-supplied buf arguments.
Dependabot alert #657 will be dismissed as tolerable_risk.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Greptile P1 finding addressed in
|
|
@greptileai review |
Summary
postcss >=8.5.10override intest-projects/expo-purchasely-test/package.json(GHSA-qx2v-qp2m-jg93 — XSS via unescaped</style>in CSS Stringify Output)uuid >=14.0.0override in the same project (GHSA-w5hq-g745-h8pq — buffer bounds check in v3/v5/v6 withbufarg)postcss 8.5.12anduuid 14.0.0;npm auditreports 0 vulnerabilitiesTest plan
npm install --package-lock-onlyregenerates the lockfile cleanlynpm auditreports 0 vulnerabilities in the expo project🤖 Generated with Claude Code