fix(deps): upgrade vulnerable dependencies to resolve Dependabot security alerts#644
Conversation
Upgrade transitive Python dependencies via uv lock: - litellm: 1.82.0 -> 1.83.0 (fixes critical auth bypass + high privilege escalation) - aiohttp: 3.13.3 -> 3.13.5 (fixes multiple HTTP vulnerabilities) - Pygments: 2.19.2 -> 2.20.0 (fixes ReDoS vulnerability) Add npm override for lodash-es in app/package.json: - lodash-es: 4.17.21 -> 4.18.1 (fixes prototype pollution + code injection) Note: pypdf vulnerabilities (alerts #54-64) cannot be fixed because graphrag-sdk pins pypdf to <6.0.0 and the fix requires >=6.7.1. This needs an upstream graphrag-sdk release. Resolves Dependabot alerts: #70-84 (except #54-64) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
📝 WalkthroughWalkthroughThe Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
app/package.json (1)
1-12: Consider adding anenginesfield to enforce Node 20+.The project requires Node version 20+ for frontend development and builds. Adding an
enginesfield would help prevent developers from using incompatible Node versions.📦 Proposed addition of engines field
{ "name": "code-graph", "version": "0.3.0", "private": true, "type": "module", + "engines": { + "node": ">=20.0.0" + }, "scripts": {Based on learnings: "Ensure Node version is 20+ for frontend development and builds" applies to app/** files.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@app/package.json` around lines 1 - 12, The package.json is missing an engines declaration to enforce Node 20+ for the frontend dev/build scripts (e.g., "dev", "build", "build:dev"); add an "engines" field in package.json with "node": ">=20" (or ">=20.0.0") so tooling and package managers can warn or block older Node versions, and update any contributor docs if present to reflect this requirement.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@app/package.json`:
- Around line 1-12: The package.json is missing an engines declaration to
enforce Node 20+ for the frontend dev/build scripts (e.g., "dev", "build",
"build:dev"); add an "engines" field in package.json with "node": ">=20" (or
">=20.0.0") so tooling and package managers can warn or block older Node
versions, and update any contributor docs if present to reflect this
requirement.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: d35066ff-97f4-4b67-a111-cccabe6528b8
⛔ Files ignored due to path filters (2)
app/package-lock.jsonis excluded by!**/package-lock.jsonuv.lockis excluded by!**/*.lock
📒 Files selected for processing (1)
app/package.json
Summary
Upgrade vulnerable dependencies to resolve open Dependabot security alerts.
Changes
Python (uv.lock):
npm (app/package.json + app/package-lock.json):
overrides) — fixes prototype pollution (Bump react-dom and @types/react-dom #82) and code injection (Bump next from 14.1.0 to 14.2.3 #81)Not fixable in this PR
graphrag-sdkpinspypdf>=5.9.0,<6.0.0but the fix requires>=6.7.1. Needs an upstreamgraphrag-sdkrelease.Testing
uv sync --all-extras✅uv run ruff check .— same 70 pre-existing errors, no regressionsnpm --prefix ./app run build✅ (includes tsc type-check)Memory / Performance Impact
N/A — lock file updates only, no code changes.
Related Issues
Resolves Dependabot alerts #70–#84 (except #54–#64)
Summary by CodeRabbit