MCP, MCP server
MCP Code Review Server
Production-ready developer tools, code templates, and AI prompt packs.
Compatibility: Not stated.
Grade B
Minor issues only, no Medium or High. Worth a quick look at the findings below before installing.
Not a guarantee. A hygiene check of the source at the commit shown, not a security audit.
Rescanned at commit a973e416bcc6, unchanged since the check below.
NETWORK_NO_FIXED_HOST SECRETS_PATH_DOTENV PERMISSION_SHELL_DECLARED SECRETS_ENV_READ OBFUSCATION_FILE_OVER_READ_CAP
Why this grade
- NETWORK_NO_FIXED_HOST Low
Makes a network call whose host is not written as a fixed address.
products/nextjs-saas-starter/src/app/(auth)/forgot-password/page.tsx, line 35
const response = await fetch("/api/auth/forgot-password", { - SECRETS_PATH_DOTENV Low
Mentions a .env file.
products/fastapi-starter/app/config.py, line 8
env_file=".env", - PERMISSION_SHELL_DECLARED Note
Runs shell commands, and the manifest or description says so.
products/mcp-code-review/src/mcp_code_review/cli.py, line 78
result = subprocess.run(command, capture_output=True, text=True, check=False) - SECRETS_ENV_READ Note
Reads an environment variable whose name contains KEY, SECRET, TOKEN, PASSWORD or CREDENTIAL.
products/nextjs-saas-starter/src/app/api/stripe/webhook/route.ts, line 28
const webhookSecret = process.env.STRIPE_WEBHOOK_SECRET; - OBFUSCATION_FILE_OVER_READ_CAP Note
A file was over the read cap and was not fully read.
products/nextjs-saas-starter/package-lock.json, line 1
{
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
Note
requests shell access. The manifest names this permission.
PERMISSION_SHELL_DECLARED
products/mcp-code-review/src/mcp_code_review/cli.py, line 78
result = subprocess.run(command, capture_output=True, text=True, check=False)
products/mcp-code-review/src/mcp_code_review/reviewer.py, line 147
"Command injection risk", "Use subprocess.run with args list, not shell=True", "command_injection"),
products/mcp-code-review/trial/sample.py, line 8
os.system(user_input)
Show more
blog/team-code-review-trial-en.html, line 19
<div class="card"><ul><li><strong>High:</strong> a command-injection finding for <code>os.system(user_input)</code>.</li><li><strong>Medium:</strong> a team-convention rule, such a
blog/team-code-review-trial.html, line 29
<li><strong>High：</strong>对 <code>os.system(user_input)</code> 的命令注入风险提示。</li>
products/code-review-agent/SKILL.md, line 30
- **Command injection**: user input passed to exec(), subprocess, os.system, child_process.exec
products/mcp-code-review/README.zh.md, line 122
| 4 | Command injection risk | security | Use subprocess.run with args list |
Install and update
Pass
No install-time script found, and dependencies are pinned or absent.
Network
Concern
can reach a host that is not written as a fixed address.
NETWORK_NO_FIXED_HOST
products/nextjs-saas-starter/src/app/(auth)/forgot-password/page.tsx, line 35
const response = await fetch("/api/auth/forgot-password", {
products/nextjs-saas-starter/src/app/(auth)/register/page.tsx, line 51
const response = await fetch("/api/user", {
products/nextjs-saas-starter/src/app/(dashboard)/billing/page.tsx, line 70
const response = await fetch("/api/stripe/create-checkout", {
Show more
products/nextjs-saas-starter/src/app/(dashboard)/billing/page.tsx, line 108
const response = await fetch("/api/stripe/portal", {
products/nextjs-saas-starter/src/app/(dashboard)/settings/page.tsx, line 58
const response = await fetch("/api/user", {
products/nextjs-saas-starter/src/app/(dashboard)/settings/page.tsx, line 93
const response = await fetch("/api/user", {
scripts/distribution_report.py, line 21
def _get_json(url: str, opener=urllib.request.urlopen) -> dict:
scripts/distribution_report.py, line 37
opener=urllib.request.urlopen,
Secrets
Concern
reads a credential or environment file.
SECRETS_ENV_READ SECRETS_PATH_DOTENV
products/fastapi-starter/app/config.py, line 8
env_file=".env",
products/nextjs-saas-starter/src/app/api/stripe/webhook/route.ts, line 28
const webhookSecret = process.env.STRIPE_WEBHOOK_SECRET;
products/nextjs-saas-starter/src/lib/auth.ts, line 71
...(process.env.AUTH_GOOGLE_ID && process.env.AUTH_GOOGLE_SECRET
Show more
products/nextjs-saas-starter/src/lib/auth.ts, line 75
clientSecret: process.env.AUTH_GOOGLE_SECRET,
products/nextjs-saas-starter/src/lib/auth.ts, line 79
...(process.env.AUTH_GITHUB_ID && process.env.AUTH_GITHUB_SECRET
products/nextjs-saas-starter/src/lib/auth.ts, line 83
clientSecret: process.env.AUTH_GITHUB_SECRET,
products/nextjs-saas-starter/src/lib/email.ts, line 8
const apiKey = process.env.RESEND_API_KEY;
products/nextjs-saas-starter/src/lib/email.ts, line 23
return !!process.env.RESEND_API_KEY;
products/nextjs-saas-starter/src/lib/stripe.ts, line 12
if (!process.env.STRIPE_SECRET_KEY) {
products/python-cli-generator/src/core/config.py, line 4
Reads from environment variables and/or .env file.
products/python-cli-generator/src/core/config.py, line 14
Application settings loaded from environment / .env file.
products/python-cli-generator/src/core/config.py, line 17
or a `.env` file in the project root.
products/python-cli-generator/src/core/config.py, line 39
env_file=".env",
products/python-cli-zh/src/core/config.py, line 6
2. .env 文件
products/python-cli-zh/src/core/config.py, line 19
所有字段都有合理的默认值。通过环境变量或 .env 文件覆盖。
products/python-cli-zh/src/core/config.py, line 40
env_file=".env",
Agent instructions
Pass
No instruction to ignore earlier instructions or to hide a step.
Obfuscation
Note
a file was over the read cap and was not fully read.
OBFUSCATION_FILE_OVER_READ_CAP
products/nextjs-saas-starter/package-lock.json, line 1
{
Licence
Pass
A licence file is present (LICENSE).
Maintainer activity
Pass
The latest commit is within the last 365 days.
Note
Hygiene check of commit a973e416bcc6. requests shell access. The manifest names this permission. can reach a host that is not written as a fixed address. reads a credential or environment file. a file was over the read cap and was not fully read.
Date and source
Date checked:
github.com
Commit graded: a973e416bcc628865bb83dcfa2a479b9aaf04878
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.