菜单
评估 Python 代码是否符合本仓库制定的质量标准，涵盖代码风格、类型注解、测试覆盖、 可读性和性能。在提交代码前或审查他人代码时使用此 skill。
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
Python 编程最佳实践，涵盖性能优化、可读性提升、数据结构选择、并发编程和常见陷阱规避。 在编写高质量 Python 代码时参考此 skill。
使用 pytest 编写高质量测试的通用规范。涵盖单元测试、集成测试（内部流程）、生产环境冒烟测试、 测试目录三层分离、fixture 使用、Mock 策略、参数化测试、异步测试和覆盖率目标。 在创建或修改测试文件时使用此 skill。
在简化或重构 Python 代码，且存在重复逻辑、深层嵌套、冗长分支或无意义中间变量，同时必须保持行为不变时使用此 skill。
Python 代码风格与质量检查规范，使用 ruff 进行格式化和 lint 检查，使用 basedpyright 进行类型检查。 在编写、修改或检查 Python 代码时使用此 skill。
基于 SOC 职业分类
| name | python-code-review |
| description | 评估 Python 代码是否符合本仓库制定的质量标准，涵盖代码风格、类型注解、测试覆盖、 可读性和性能。在提交代码前或审查他人代码时使用此 skill。 |
对照本仓库制定的以下标准评估代码质量：
类型注解
X | None 而非 Optional[X]list[X]、dict[K, V]）而非 typing.List、typing.Dictfrom __future__ import annotations（视项目配置）命名规范
_ 开头list、type、next 等）文档字符串
工具检查
ruff check . 无报错ruff format --check . 无格式问题basedpyright 无类型错误覆盖率与完整性
可读性
print 语句残留可维护性
conftest.py复杂度
重复代码
可读性
if-elif（≥4 个分支）已替换为字典映射或 match-case数据结构
set 而非 listCounterdeque 而非 list性能
lru_cache / cache）join 而非 +=常见陷阱
def f(x=[]):）None 的比较使用 is/is not在审查结束时，按以下格式给出结论：
## 代码审查结论
### ✅ 符合标准的方面
- 类型注解完整，使用了 Python 3.10+ 语法
- 测试遵循 AAA 模式，命名清晰
- ...
### ⚠️ 需要改进的方面
1. `process_data()` 函数缺少返回值类型注解
- 位置：`src/processor.py:42`
- 建议：添加 `-> list[Result]`
2. 测试覆盖率不足
- 位置：`tests/test_processor.py`
- 建议：补充异常路径测试（当输入为空列表时）
3. 魔法数字
- 位置：`src/config.py:15`：`if retries > 3:`
- 建议：提取为常量 `MAX_RETRIES = 3`
### 🚫 必须修复的问题（阻断合并）
- `basedpyright` 报告 2 个类型错误（见下方详情）
- `ruff` 报告 1 个 B006 错误（可变默认参数）
### 总体评分
- 代码风格：⭐⭐⭐⭐☆
- 测试质量：⭐⭐⭐☆☆
- 代码简洁性：⭐⭐⭐⭐⭐
- 最佳实践：⭐⭐⭐⭐☆
在提交或合并前运行以下命令确保基础质量：
# 格式化 + lint
ruff format .
ruff check . --fix
# 类型检查
basedpyright
# 运行相关测试（只跑本次修改涉及的模块）
pytest tests/test_affected_module.py -v
# 带覆盖率
pytest tests/test_affected_module.py --cov=src/affected_module --cov-report=term-missing