pincushion-mcp@1.11.23
Visual feedback as agent work packets. Stakeholders drop pins on your live app; your AI coding agent reads each pin (selector, screenshot, DOM, thread, acceptance criteria) via MCP and ships the fix.
AI Security Review
scanned 1mo ago · by lpm-firewall-aiAgent config mutation is a consented CLI capability: it registers only this package's MCP command in a project-local editor file after an explicit editor choice, and package.json has no install lifecycle hook. That is a guarded first-party agent setup, so it is a warning rather than a publish block.
Static reason
High-risk behavior combination matched malicious policy.; previous stored version diff introduced dangerous source
Trigger
User runs npx pincushion-mcp setup or claim and then names cursor, vscode, or claude.
Impact
The selected editor gains a first-party Pincushion MCP server for that project. Other servers and malformed configs are left unchanged, and npm install does not perform this write.
Mechanism
After that choice, the package atomically writes or updates only the pincushion key in .cursor/mcp.json, .vscode/mcp.json, or .mcp.json so the editor launches npx pincushion-mcp --project-dir .
AI rationale
Agent config mutation is a consented CLI capability: it registers only this package's MCP command in a project-local editor file after an explicit editor choice, and package.json has no install lifecycle hook. That is a guarded first-party agent setup, so it is a warning rather than a publish block.
Evidence
server.jssetup.mjslib/setup-core.jsclaim.mjspackage.jsonscripts/sync-surfaces.mjs.cursor/mcp.json.vscode/mcp.json.mcp.json
Decision evidence
public snapshotAI called this Suspicious at 74.0% confidence as Dangerous Capability with high false-positive risk.
Evidence for warning
- The setup subcommand is reached only when the user runs pincushion-mcp setup, and the wizard writes an editor MCP file only after the user types an editor name.
- writeEditorMcpConfig updates only a project-local Cursor, VS Code, or Claude config and sets the pincushion entry to npx pincushion-mcp --project-dir .
- claim writes that same entry only after an explicit editor choice and only inside a directory that has a project marker.
- package.json exposes server.js as the bin and has no preinstall, install, or postinstall script.
- scripts/sync-surfaces.mjs is a maintainer catalog check that lists sibling IDE docs; it is not an npm install hook.
Evidence against
- Editor detection only suggests a client; skipping or an unknown choice leaves config unchanged.
- The writer refuses ~/.claude.json, unparseable files, and entries it did not generate.
- Network use is the product Supabase, pincushion.io, PostHog, and Sentry paths, with telemetry opt-out, not a hidden install payload.
Behavioral surface
ChildProcessCrypto
Source & flagged code
11 flaggedmobile-wall.mjsView file
17// injectable so both modes are testable without a simulator.
L18: import { execFileSync } from 'node:child_process';
L19: import {
High
mobile-wall.mjs#virtual:normalized:round1View file
121function publicUrlFrom(text) {
L122: const m = /(https?:\/\/[^\s"'<>]+)/.exec(String(text || ''));\n return m ? m[1].replace(/[),.;]+$/, '') : '';\n}\n\nasync function fetchCloudAnnotations(ctx, { baseUrl, key, proje...
L123: }
Critical
Same File Env Network Execution
A single source file combines environment access, network access, and code or shell execution with blocking evidence.
mobile-wall.mjs#virtual:normalized:round1View on unpkg · L12117// injectable so both modes are testable without a simulator.
L18: import { execFileSync } from "node:child_process";
L19: import {
...
L32: const DEFAULT_OUT = "crit-wall";
L33: const DEFAULT_FUNCTION_BASE = "https://dpsqzszdviltqvethxbr.supabase.co/functions/v1";
L34:
...
L78: function isSignedIn() {
L79: if (process.env.PINCUSHION_LICENSE_KEY) return true;
L80: try { return !!readFileSync(join(homedir(), ".pincushion", "license-key"), "utf8").trim(); } catch { return false; }
L81: }
...
L97: const firstLine = (e) => String(e.message || e).split("\n")[0];
L98: const execErrText = (e) => [e?.stderr, e?.stdout, e?.message].filter(Boolean).join("\n");
Findings
5 Critical7 High4 Medium6 Low
CriticalSame File Env Network Executionmobile-wall.mjs#virtual:normalized:round1
CriticalCredential Exfiltrationserver.js
CriticalDownload Executemobile-wall.mjs#virtual:normalized:round1
CriticalTrigger Reachable Dangerous Capabilityserver.js
CriticalPrevious Version Dangerous Deltascripts/capture-snapshot.mjs
HighChild Processmobile-wall.mjs
HighShell
HighCommand Output Exfiltrationmobile-wall.mjs#virtual:normalized:round1
HighCopied Package Dependency Bridgelib/posthog.js
HighTrigger Reachable Credential Exfiltrationserver.js
HighTrigger Reachable Command Output Exfiltrationmobile-wall.mjs#virtual:normalized:round1
Affected versions and remediation
This report applies to pincushion-mcp@1.11.23.
See all versions of pincushion-mcp for other recorded verdicts.
- Review the evidence and your use of pincushion-mcp@1.11.23 before allowing it. Restrict the permissions described in this report.
- Choose an independently verified alternative or release. This report does not establish that other versions are safe.
Evidence last updated: .
Think this verdict is wrong? Dispute it on GitHub or read how reports are produced.