MCP, MCP server
Code Review Agent Mcp
AI code review agent that doesn't sugarcoat.
Compatibility: Not stated.
Grade C
One Medium finding. Read the report before installing.
Not a guarantee. A hygiene check of the source at the commit shown, not a security audit.
SECRETS_PATH_CREDENTIAL
Why this grade
- SECRETS_PATH_CREDENTIAL Medium
Mentions a credential store path (~/.ssh, id_rsa, .npmrc, .aws/credentials, credentials.json, .netrc).
src/code_review_agent/security.py, line 4
- Path traversal (review_file reading /etc/passwd, ~/.ssh/id_rsa, etc.)
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
src/code_review_agent/prompts/system_prompt.py, line 57
- eval(), exec(), os.system(), subprocess with shell=True
src/code_review_agent/tools/review.py, line 560
# subprocess.run so we can kill the process on timeout or task
src/code_review_agent/tools/review.py, line 561
# cancellation. subprocess.run's timeout kills the process on
Show more
src/code_review_agent/tools/review.py, line 565
proc = subprocess.Popen(
CHANGELOG.md, line 30
- **HIGH**: Blocking `subprocess.run` in async function - now runs in `asyncio.to_thread`.
CHANGELOG.md, line 75
- `review_commit` now spawns `git show` via `subprocess.Popen` and wraps `communicate()` in `asyncio.wait_for`. On `TimeoutError` or `CancelledError` the child process is explicitl
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
benchmarks/snippet_06_hardcoded_credentials.py, line 24
response = requests.post(
Secrets
Concern
reads a credential or environment file.
SECRETS_PATH_CREDENTIAL SECRETS_PATH_DOTENV
src/code_review_agent/security.py, line 4
- Path traversal (review_file reading /etc/passwd, ~/.ssh/id_rsa, etc.)
src/code_review_agent/security.py, line 47
"/.npmrc", # npm tokens
src/code_review_agent/security.py, line 51
"/.env", # Environment files with secrets
Show more
src/code_review_agent/security.py, line 52
"/.netrc", # Netrc with credentials
src/code_review_agent/security.py, line 99
4. Reject sensitive paths (.ssh, .aws, .env, /etc/passwd, etc.)
src/code_review_agent/security.py, line 135
# '.env.example' is commonly a checked-in template and remains reviewable;
src/code_review_agent/security.py, line 136
# other '.env.*' files are treated as potentially secret-bearing.
src/code_review_agent/security.py, line 139
safe_env_templates = {".env.example", ".env.sample", ".env.template"}
Agent instructions
Pass
No instruction to ignore earlier instructions or to hide a step.
Obfuscation
Pass
No encoded or minified payload found.
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 07fa8407e449. 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.
Date and source
Date checked:
github.com
Commit graded: 07fa8407e44901f3d0dd62f75ce06e0b48c8992b
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.