MCP, MCP server
io.github.Alberto-Codes/docvet
Real-time docstring quality diagnostics for Python files
Compatibility: Not stated.
Grade C
One Medium finding. Read the findings before installing.
Not a guarantee. A hygiene check of the source at the commit shown, not a security audit.
Rescanned at commit e12870c2cd24, unchanged since the check below.
PERMISSION_SHELL_UNDECLARED
Why this grade
- PERMISSION_SHELL_UNDECLARED Medium
Runs shell commands that the manifest or description does not mention.
.claude/rules/pytest.md, line 71
subprocess.run(["git", "init"], cwd=tmp_path, check=True, capture_output=True)
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
.claude/rules/pytest.md, line 71
subprocess.run(["git", "init"], cwd=tmp_path, check=True, capture_output=True)
.claude/rules/pytest.md, line 180
mocker.patch("docvet.checks.freshness.subprocess.run") # Correct
.claude/rules/pytest.md, line 181
mocker.patch("subprocess.run") # Wrong - too broad
Show more
SECURITY.md, line 17
- Runs `git diff`, `git blame`, and `git log` via `subprocess.run` with explicit argument lists (no shell expansion)
_bmad-output/implementation-artifacts/23-1-cross-platform-ci-matrix-trust-gate.md, line 90
| Line ending parsing | LOW | Already safe - all parsing uses `.splitlines()`, all `subprocess.run` uses `text=True` |
_bmad-output/implementation-artifacts/23-1-cross-platform-ci-matrix-trust-gate.md, line 101
- All `subprocess.run` calls use `text=True` - enables universal newline mode
_bmad-output/implementation-artifacts/27-1-fix-hunk-to-symbol-false-positives.md, line 143
- **Mocking**: `check_freshness_diff()` tests mock `subprocess.run` to inject diff output and provide source code strings parsed via `ast.parse()`.
_bmad-output/implementation-artifacts/4-3-cli-wiring-for-freshness-diff-mode.md, line 113
result = subprocess.run(
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
_bmad/tea/testarch/knowledge/contract-testing.md, line 684
const response = await axios.get(`/users/${id}`, axiosConfig);
_bmad/tea/testarch/knowledge/fixture-architecture.md, line 42
const response = await request.fetch(url, {
_bmad/tea/testarch/knowledge/fixture-architecture.md, line 196
const response = await fetch(url, {
Show more
_bmad/tea/testarch/knowledge/pactjs-utils-consumer-helpers.md, line 52
const res = await fetch(`${mockServer.url}/movies/1`);
_bmad/tea/testarch/knowledge/pactjs-utils-consumer-helpers.md, line 122
const res = await fetch(`${mockServer.url}/movies`);
_bmad/tea/testarch/knowledge/pactjs-utils-overview.md, line 107
const res = await fetch(`${mockServer.url}/users/1`);
_bmad/tea/testarch/knowledge/pactjs-utils-request-filter.md, line 61
const response = await fetch('http://localhost:8080/auth/token', {
_bmad/tea/testarch/knowledge/pactjs-utils-request-filter.md, line 201
// ✅ Clean separation - async setup wraps token fetch (CommonJS-safe)
Secrets
Concern
reads a credential or environment file.
SECRETS_PATH_DOTENV
.gitignore, line 46
# Environment
.gitignore, line 47
.env
_bmad/bmm/workflows/3-solutioning/create-architecture/steps/step-06-structure.md, line 145
├── .env.local
Show more
_bmad/bmm/workflows/3-solutioning/create-architecture/steps/step-06-structure.md, line 146
├── .env.example
_bmad/bmm/workflows/3-solutioning/create-architecture/steps/step-06-structure.md, line 184
├── .env
_bmad/bmm/workflows/3-solutioning/create-architecture/steps/step-06-structure.md, line 185
├── .env.example
_bmad/bmm/workflows/document-project/documentation-requirements.csv, line 2
web,true,true,true,true,true,package.json;tsconfig.json;*.config.js;*.config.ts;vite.config.*;webpack.config.*;next.config.*;nuxt.config.*,src/;app/;pages/;components/;api/;lib/;st
_bmad/bmm/workflows/document-project/documentation-requirements.csv, line 3
mobile,true,true,true,true,true,package.json;pubspec.yaml;Podfile;build.gradle;app.json;capacitor.config.*;ionic.config.json,src/;app/;screens/;components/;services/;models/;assets
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. the file cap was reached before every file was read.
OBFUSCATION_FILE_OVER_READ_CAP OBFUSCATION_FILE_CAP_REACHED
uv.lock, line 1
version = 1
_bmad/tea/workflows/testarch/ci/steps-e/step-01-assess.md, 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 e12870c2cd24. requests shell access. No manifest field names shell or command execution. 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. the file cap was reached before every file was read.
Date and source
Date checked:
github.com
Commit graded: e12870c2cd2434929f9683cc99f0b0306e6e5508
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.