review-sendmsg

SKILL.md

# 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