io.github.DustinTrap/kvm-pilot v0.1.0b8
Host io.github.DustinTrap/kvm-pilot in a governed environment — SSO in front, every tool call attributed and audited.
Security controls
Deterministic, evidence-backed. Score earned from passing controls; a failed guard caps it.
Security controls
Deterministic, evidence-backed. Score earned from passing controls; a failed guard caps it.
Model–MCP Runtime Guardrails
- WarnTool Definition Integrityguard16 tool description(s) drifted after publish (no risk signals in diff)
Every consecutive version pair is diffed for new injection markers or destructive scope — a risky diff anywhere in history is a rug-pull (fail, durable); benign description drift warns.
healthcheckcapabilitiesclassify_screenwait_for_stateFix: Never add hidden directives or destructive scope to an already-published tool; document description changes in release notes.
- PassIndirect Prompt Injection (IPI) Defensesguardno injection markers
Tool/prompt/resource text is free of hidden instructions that could hijack the agent.
- PassStrict JSON Schema EnforcementadditionalProperties:false present
Tool inputs are constrained (additionalProperties:false), so unexpected arguments can't be smuggled in.
- PassUser-in-the-Loop / Approval Scopeguardno destructive tool scope
No over-broad or destructive tools (arbitrary shell, bulk-delete) that warrant human approval.
Application Security Checks
- WarnNo SSRF sinksguard1 occurrence
Fetching tool-supplied URLs can pivot into internal networks and metadata services.
src/kvm_pilot/detect.py:94Fix: Allow-list destinations; reject arbitrary/loopback/link-local URLs.
- WarnDependencies pinned (lockfile)no lockfile found
A lockfile makes installs reproducible and resistant to silent dependency swaps.
Fix: Commit a lockfile (package-lock.json / pnpm-lock.yaml / poetry.lock).
- WarnAdoption & popularitylimited adoption
A small, capped nudge from stars/downloads — widely-used servers get more eyes on bugs. It can never offset a real security failure.
Fix: Adoption grows naturally with real-world use; this never gates the score.
- PassNo command-injection sinksguardno sinks found
Untrusted tool input reaching a shell yields remote code execution.
- PassNo dynamic code executionguardno sinks found
eval()/exec()/Function() on tool-derived strings allows arbitrary code execution.
- PassNo path traversalguardno sinks found
Naive path checks let tools read/write outside intended directories (EscapeRoute-class).
- PassNo unsafe deserializationguardno sinks found
pickle/yaml.load/etc. on untrusted data can execute code.
- PassNo committed secretsguardno secrets found
Hardcoded keys/tokens in published source are live credentials an attacker can use.
- PassCredentials sourced from environmentreads credentials from environment
Reading secrets from env/secret stores avoids hardcoding them.
- PassNo known-vulnerable dependencies1 runtime deps, no known CVEs
Runtime dependencies (parsed from the lockfile) are scanned against OSV.dev for published CVEs. Advisory: flagged dependencies lower the score but don't hard-cap it, since transitive reachability is unproven.
- PassActively maintainedrecent commits
Unmaintained servers don't receive security fixes.
- PassRepository not archivedguardactive
Archived repositories will never be patched.
- PassDeclares a licenseApache-2.0
A clear license is required for legal enterprise use.
- PassHas a security policySECURITY.md present
A SECURITY.md gives a private path to report vulnerabilities.
- Not checkedNo install/post-install scriptsguardpypi metadata unavailable
install hooks run arbitrary code on every consumer at install time.
- Not checkedPackage name not typosquattingguardpypi metadata unavailable
Names mimicking popular packages are a common malware delivery vector.
- Not checkedPublished with provenancepypi metadata unavailable
Build provenance attests the artifact was built from the claimed source by CI.
- Not checkedEstablished maintainerpypi metadata unavailable
Brand-new / single anonymous maintainers raise takeover and malware risk.
- Not checkedSigned releasesnot evaluated
Signed releases let consumers verify artifacts weren't tampered with.
Transport & Trust Model
- WarnNetwork Exposurebinds 0.0.0.0
Binding 0.0.0.0 or exposing debug inspectors widens the attack surface.
Fix: Bind to 127.0.0.1 by default; never ship open debug endpoints.
- PassExecution Sandboxingcontainerized
A container/sandbox image limits blast radius; a server that runs natively has full host access.
- N/ATransport Encryption (TLS)guardno remote endpoints
Plaintext HTTP exposes traffic and bearer tokens to interception.
- N/AIAM / Authentication Scopingno remote endpoints
OAuth 2.1 / Protected Resource Metadata gates who can invoke tools.
- N/ALive Endpoint Reachablenot dynamically scanned
A dynamic scan connected to the declared remote endpoint and it responded — verified live, not a dead URL.
- N/AAuthentication Enforced (live)not dynamically scanned
If the server declares auth is required, it must actually reject anonymous clients. Serving tools to unauthenticated callers is a real exposure.
Tools (34)
AI-flagged — for review
Observations from an AI review of tool descriptions. These are advisory only and do not affect the score — they can be noisy and need human judgement.
- infoTool "healthcheck" description changed after publish
Benign definition drift: the description changed vs the prior version with no risk signals in the diff. Clients approve tools by name and don't re-review on update, so the change is invisible to existing grants.
- infoTool "capabilities" description changed after publish
Benign definition drift: the description changed vs the prior version with no risk signals in the diff. Clients approve tools by name and don't re-review on update, so the change is invisible to existing grants.
- infoTool "classify_screen" description changed after publish
Benign definition drift: the description changed vs the prior version with no risk signals in the diff. Clients approve tools by name and don't re-review on update, so the change is invisible to existing grants.