py-review-skill

What this skill does

Use this skill to perform a structured Python code review.

It should catch correctness bugs, edge cases, regressions, test gaps, security issues, and maintainability problems.

When to use

• Reviewing a Python diff, patch, or pull request
• Checking a script before merging
• Auditing a change for risky behavior
• Summarizing review findings for a human

Review workflow

1. Read the change and identify the intent.
2. Check correctness first:
• logic errors
• missing branches
• off-by-one issues
• null/empty handling
• exception paths
3. Check safety:
• command execution
• file handling
• input validation
• secrets exposure
• network or subprocess risks
4. Check tests:
• existing coverage
• missing regression tests
• behavior under failure cases
5. Check maintainability:
• naming
• duplication
• clarity
• unnecessary complexity
6. Report only real issues. Do not invent problems.

Output format

Prefer a concise review with:

• Overall verdict
• High-priority issues first
• File/line references when available
• Suggested fixes

Example:

• High: foo.py:42 — off-by-one in loop causes last item to be skipped.
• Medium: bar.py:18 — no timeout on network call.

Notes

• Be specific.
• Be practical.
• If the change is fine, say so.
• If you cannot verify something, say what is missing.

Local helpers

• scripts/review.py — core review logic
• scripts/review_runner.py — wrapper/runner for review execution