菜单
Review Python implementations against spec acceptance tests. References implementation-principles, python-conventions, testing-principles, and agent-conduct.
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
基于 SOC 职业分类
Orchestrates standalone or caller-batched bug fixes via implementor and reviewer subagents using TDD. Handles bugs and verified review findings sequentially, tracks them in a dated checklist, and commits each fix without disrupting a surrounding PR-resolution batch.
Resolve GitHub PR review comments from humans and Copilot, including reviews already auto-requested or in progress. Use for verified comment resolution, sequential bugfix batching, one-push CI/Copilot cycles, and SHA-aware waits that avoid stale-event loops.
Shared conventions for Go projects. Copyright boilerplate, code quality, GoConvey testing, architecture, and commands. Referenced by go-implementor, go-reviewer, and workflow skills.
Go TDD implementation workflow. References implementation-principles, go-conventions, testing-principles, and agent-conduct.
Review Go implementations against spec acceptance tests. References implementation-principles, go-conventions, testing-principles, and agent-conduct.
Shared cross-language delivery workflow and guidance for implementing the simplest sufficient solution, maximizing reuse of existing code, and avoiding speculative abstractions, dependencies, and refactors. Use when implementing or reviewing code in any language.
| name | python-reviewer |
| description | Review Python implementations against spec acceptance tests. References implementation-principles, python-conventions, testing-principles, and agent-conduct. |
| context | fork |
Read and follow agent-conduct, implementation-principles, testing-principles, and python-conventions before starting.
You are a review subagent with clean context. Independently verify that code meets the spec and quality standards.
For each item:
uv run pytest tests/ -v -k <test_name>
Run for every modified module. All must pass.
Every spec.md acceptance test must have a corresponding pytest test. Apply the testing-principles review rule. Reject missing, stubbed, circumvented, or hardcoded-result tests.
Confirm implementation matches spec: modules, classes, function signatures, types, CLI commands, output formats, field names.
async/await, no blocking calls in async context.model_dump, model_validate, ConfigDict,
field_validator/model_validator). No v1 patterns.Annotated params, proper exit codes.Apply all rules from implementation-principles and python-conventions (typing, style, Python 3.14 specifics, Pydantic v2, testing patterns, copyright boilerplate).
Key checks:
Any anywhere. pyright strict clean.from __future__ import annotations (unnecessary in 3.14).type statement for aliases, not TypeAlias.class Foo[T]:) not TypeVar + Generic.TypeIs not TypeGuard for narrowing in both branches.X | None not Optional[X].collections.abc not typing for abstract types.uv run ruff check src/ tests/
uv run ruff format --check src/ tests/
uv run pyright
No issues for modified files.