SkillAudit report — punkpeye/fastmcp
Scanned 2026-04-23 by SkillAudit v0.2 (static checks + LLM-assisted prompt-injection red-team).
Commit: 72c1166 · Stars: 3055 · Days since last push: 4
LLM prompt-injection probe: no-tool-surface
Overall grade: D (60/100)
| Axis | Score | Grade | |
|---|---|---|---|
| security | 60/100 | D | ❌ |
| permissions | 100/100 | A | ✅ |
| credentials | 70/100 | C | ⚠️ |
| maintenance | 100/100 | A | ✅ |
| compatibility | 70/100 | C | ⚠️ |
| docs | 90/100 | A | ✅ |
Security findings
Production sources:
- HIGH
src/DiscoveryDocumentCache.ts:101— HTTP client call with user-controlled argument 'url' — no URL allowlist / validation found in file
const res = await fetch(url);
- WARN
src/FastMCP.ts:82— HTTP client call with user-controlled argument 'input' (validation markers present but not verified against this call-site)
const response = await fetch(input.url);
- WARN
src/FastMCP.ts:151— HTTP client call with user-controlled argument 'input' (validation markers present but not verified against this call-site)
const response = await fetch(input.url);
Test-site findings (lower weight): 74 total in test/ paths — first 3 shown
- HIGH
src/FastMCP.https.test.ts:93— Template-string URL with interpolation — no validation possible on composed string
const response = await fetch(\http://localhost:${port}/ping\);
- HIGH
src/FastMCP.oauth-proxy.test.ts:45— Template-string URL with interpolation — no validation possible on composed string
const dcrResponse = await fetch(
- HIGH
src/FastMCP.oauth-proxy.test.ts:62— Template-string URL with interpolation — no validation possible on composed string
const metadataResponse = await fetch(
Permissions
_No findings on this axis._
Credentials
Production sources:
- HIGH
src/examples/session-context.ts:247— console.* of process.env — entire env leaks to stdout/stderr and LLM context
console.log(\`
Maintenance
_No findings on this axis._
Compatibility
Production sources:
- WARN
(meta)— No engines (Node) or python_requires declared — cross-client compatibility unverified
Documentation
Production sources:
- WARN
(meta)— No SECURITY.md — no disclosure channel for vulnerabilities
missing
Methodology
SkillAudit v0.2 clones the repo at the provided ref (default: default branch, HEAD) into an ephemeral sandbox, runs six static checks over .js/.ts/.py sources, queries the GitHub API for maintenance signals, and runs an LLM-assisted prompt-injection red-team over the MCP tool surface. Each axis is scored against the rubric at
The prompt-injection axis extracts each server.tool(...) / @app.tool registration + the first ~60 lines of handler body, hands them to Claude Haiku 4.5 with a red-team system prompt, and asks for structured findings on untrusted-content flow into tool responses. One API call per scan, bounded at ~15K input tokens.
How to improve this grade
- Security — static: validate tool-input URLs against an allowlist before fetch/axios calls; use
execFilewith argv arrays instead ofexecwith template strings; never pass untrusted strings tosubprocesswithshell=True. - Security — prompt injection: never return fetched web-page / file / email content verbatim in a tool response. Wrap with a framing marker (e.g.,
<untrusted-content>...</untrusted-content>), summarize rather than inline, and never let untrusted content share a turn with credentials or other tool output. - Credentials findings: redact env-var reads before log lines and error messages; treat any string that ends up in a tool response as public.
- Maintenance: if the repo is inactive, document the maintenance model — "MCP tool, no breaking changes expected" is a legitimate signal.
- Docs: add a README install + usage section with a copy-pasteable command; add a SECURITY.md with a disclosure channel.
_Report generated by skillaudit.dev_
Want your repo audited?
First 100 audits go to waitlist signups in order. The engine runs against public GitHub URLs today.
Join the waitlist →