[Aikido] Fix 11 security issues in langsmith, form-data, uuid and 1 more - #39
[Aikido] Fix 11 security issues in langsmith, form-data, uuid and 1 more#39aikido-autofix[bot] wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 3 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 6d226b9. Configure here.
| "jest-environment-jsdom": "^29.6.2", | ||
| "langchain": "^0.3", | ||
| "lerna": "^8.1.8", | ||
| "langchain": "^1.2.28", |
There was a problem hiding this comment.
LangChain 1.x breaks ConversationChain import
High Severity
The LangChain bump to 1.x leaves a ConversationChain import from langchain/chains, which v1 moved to @langchain/classic/chains and no longer exports from the main package. Integration tests fail at import time, so the LangChain upgrade is not actually exercised.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 6d226b9. Configure here.
| "langchain": "^0.3", | ||
| "lerna": "^8.1.8", | ||
| "langchain": "^1.2.28", | ||
| "lerna": "^10.0.0", |
There was a problem hiding this comment.
Lerna 10 requires Node 22
High Severity
lerna was jumped from 8.x to 10.x, which is ESM-only and requires Node 22.13+. Repo engines and CI still use Node 18/20, so npx lerna publish from the documented release flow fails on the supported Node versions.
Reviewed by Cursor Bugbot for commit 6d226b9. Configure here.
| "@langchain/core": "^0.3.3", | ||
| "@opentelemetry/auto-instrumentations-node": "^0.50.0", | ||
| "@langchain/core": "^1.1.29", | ||
| "@opentelemetry/auto-instrumentations-node": "^0.76.0", |
There was a problem hiding this comment.
OpenTelemetry SDK versions are incompatible
Medium Severity
@opentelemetry/auto-instrumentations-node was raised to 0.76, which needs @opentelemetry/sdk-node 0.218+ (OTEL JS 2.x). The project still pins sdk-node@^0.53.0. The Vercel integration test constructs NodeSDK from 0.53 with 0.76 instrumentations, which is an unsupported mix.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 6d226b9. Configure here.


Problem
Upgrade langsmith, form-data, uuid, and brace-expansion to fix prototype pollution, CRLF injection, buffer overflow, and untrusted deserialization vulnerabilities.
Changes
✅ 11 CVEs resolved by this upgrade, including 2 critical 🚨 CVEs
This PR will resolve the following CVEs:
This issue affects form-data: < 2.5.4, 3.0.0 - 3.0.3, 4.0.0 - 4.0.3.
✅ The package upgrades for
langsmith(0.2.5 => 0.6.0) anduuid(10.0.0 => 11.1.1) do not affect this codebase.Both packages are transitive dependencies through
langchain@^0.3and are not directly imported or used in the codebase:langsmith: No direct imports found. The only reference (
LANGSMITH_HIDDEN_TAGinlangfuse-langchain/src/callback.ts:4) is a string constant unaffected by langsmith API changes.uuid: No direct imports found. The codebase uses Node.js's built-in
crypto.randomUUID()instead of the uuid package.The Node.js version requirement (>=18) is compatible with uuid 11.x's supported range (Node 16-20).
All breaking changes by upgrading uuid from version 10.0.0 to 14.0.2 (CHANGELOG)
cryptois now expected to be globally defined (requires node@20+)v3(),v5(), andv6()now throw aRangeErrorifoffset < 0oroffset + 16 > buf.length(previously allowed out-of-bounds writes with invalid offset)Release info Sub-libraries affected
Bump level
Libraries affected
Changelog notes
Note
Medium Risk
Major LangChain and peer-range changes can break integrators on 0.3.x; risk is mostly compatibility and CI/integration test behavior, not auth or data paths.
Overview
Bumps the LangChain stack from 0.3 to 1.2.x (
langchain,@langchain/core,@langchain/communityin integration tests) and refreshes related dev tooling (lerna10,@opentelemetry/auto-instrumentations-node0.76).langfuse-langchainnow declareslangchainpeer>=0.0.157 <2.0.0(was capped below 0.4), so consumers can install LangChain 1.x alongside the integration package.No application source changes in the diff—only
package.jsonversion pins, aligned with security-driven transitive upgrades (e.g. langsmith, form-data, uuid) described in the PR.Reviewed by Cursor Bugbot for commit 6d226b9. Bugbot is set up for automated code reviews on this repo. Configure here.