skill-review-sendmsg（技能审核发送消息）

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

1. Check safety:

- command execution - file handling - input validation - secrets exposure - network or subprocess risks

1. Check tests:

- existing coverage - missing regression tests - behavior under failure cases

1. Check maintainability:

- naming - duplication - clarity - unnecessary complexity

1. 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