Skip to content

fix(security): pin production audit overrides for MCP SDK and transformers - #48

Open
Roarpeng wants to merge 1 commit into
mainfrom
cursor/security-audit-deps-1c71
Open

Roarpeng wants to merge 1 commit into
mainfrom
cursor/security-audit-deps-1c71

Conversation

@Roarpeng

Copy link
Copy Markdown
Owner

Why

Weekly Security Audit (cron: 0 3 * * 1) failed npm audit --omit=dev on @roarpeng/graphflow@1.18.5.

Direct production deps are unchanged (@modelcontextprotocol/sdk@1.30.0, @huggingface/transformers@3.8.1). Transitive advisories:

Package Severity Via Pinned to
hono moderate MCP SDK 4.13.7
@hono/node-server moderate MCP SDK 1.19.17
fast-uri high ajv 3.1.6
ip-address high express-rate-limit 10.7.0
body-parser low express 2.3.0
qs moderate express 6.16.0
sharp high transformers 0.35.4

npm run security:audit is green locally after the overrides.

Risk

sharp 0.35.4 is a minor jump past transformers’ ^0.34.x range. GraphFlow only loads transformers for text embeddings, not the image pipeline. If a later transformers release pins a patched sharp, the override can drop.

Tests

tests/security-audit-script.test.ts asserts the override pins stay present.

Open in Web Open in Cursor 

…ormers

Weekly Security Audit (npm audit --omit=dev) failed on hono,
@hono/node-server, fast-uri, ip-address, body-parser, qs (via
@modelcontextprotocol/sdk) and sharp (via @huggingface/transformers).
Pin patched versions with package.json overrides; do not bump those
direct dependencies' majors.
@Roarpeng
Roarpeng marked this pull request as ready for review September 14, 2026 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants