Grades are a hygiene check, not a security guarantee.

MCP, MCP server

Comfy Mcp

Comfy Mcp, an MCP server listed in the PlugVet directory.

Compatibility: Not stated.

Grade C

One Medium finding. Read the findings before installing.

Graded by PlugVetRubric v2

Not a guarantee. A hygiene check of the source at the commit shown, not a security audit.

Rescanned at commit 6da93118533f.

PERMISSION_SHELL_UNDECLARED

Why this grade

  • PERMISSION_SHELL_UNDECLARED Medium

    Runs shell commands that the manifest or description does not mention.

    src/comfy_mcp/argv.py, line 67 ``subprocess.Popen`` raises a bare ``ValueError: embedded null byte`` on one —

Any High, or two or more Medium findings, give D. One Medium gives C. Low findings or a security Note give B. Each code names a finding behind the letter.

Grades are a hygiene check of the source at the commit shown, on the date checked, not a security audit or a guarantee. The free directory grades are automated. Before release, a PlugVet engineer reviews every paid report and any grade we hold back, including all F grades, against the source code and the evidence we quote.

What was checked

Plain notes from the hygiene checks.

  • Permissions

    Concern

    requests shell access. No manifest field names shell or command execution.

    PERMISSION_SHELL_UNDECLARED

    src/comfy_mcp/argv.py, line 67

    ``subprocess.Popen`` raises a bare ``ValueError: embedded null byte`` on one —

    src/comfy_mcp/argv.py, line 345

    but makes ``subprocess.Popen`` raise a bare ``ValueError``, which would surface

    src/comfy_mcp/argv.py, line 383

    NUL is a legal MCP string that makes ``subprocess.Popen`` raise a bare

    Show more

    src/comfy_mcp/argv.py, line 459

    # `subprocess.Popen` — an internal error that no caller converts and that

    src/comfy_mcp/argv.py, line 481

    ``subprocess.Popen`` raises a bare ``ValueError: embedded null byte``, which

    AGENTS.md, line 155

    `--json` path (`subprocess.run`); `calls` records `cmd`/`env`/`timeout`/`encoding` per

    AGENTS.md, line 173

    (`subprocess.Popen` + a bounded `communicate`, off-loaded to a thread pool for async

    SECURITY.md, line 54

    - **Shell injection — structurally absent.** Every spawn is an argv list: `_run_comfy_raw` (`subprocess.Popen`), `_run_comfy_streaming` and `_start_login` (`asyncio.create_subproce

  • Install and update

    Pass

    No install-time script found, and dependencies are pinned or absent.

  • Network

    Pass

    No network call found.

  • Secrets

    Concern

    reads a credential or environment file.

    SECRETS_PATH_DOTENV

    .gitignore, line 28

    .env

  • Agent instructions

    Pass

    No instruction to ignore earlier instructions or to hide a step.

  • Obfuscation

    Concern

    contains an encoded blob in a file that should be readable.

    OBFUSCATION_ENCODED_BLOB

    README.md, line 30

    <a href="https://modelcontextprotocol.io"><img src="https://img.shields.io/badge/MCP-Compatible-green?style=for-the-badge&logo=data:image/svg+xml;base64,PHN2ZyB4bWxucz0iaHR0cDovL3d

  • Licence

    Pass

    A licence file is present (LICENSE).

  • Maintainer activity

    Pass

    The latest commit is within the last 365 days.

Get the full report on this tool, £49

Note

Hygiene check of commit 6da93118533f. requests shell access. No manifest field names shell or command execution. reads a credential or environment file. contains an encoded blob in a file that should be readable.

Date and source

Date checked:

View source

github.com

Commit graded: 6da93118533ff52b3a2527f0ce80f1ae8d73b6b5

This link goes to the original host. PlugVet does not host or check what you download from there, and installing it is your own decision and risk.

Read how this letter is chosen