用途:该技能面向提交代码供审查的 Python 开发者,目标是以较高质量标准检查 Pythonic 模式、类型安全与可维护性,并给出具体改进建议。主要能力与规则:对现有文件的修改非常严格,新增复杂度需有充分理由,优先抽取到新的模块或类;对新代码较务实,若隔离且可用即可接受,但仍指出明显改进点,关注可测试性与可维护性。类型提示方面要求函数参数与返回值始终标注,使用 Python 3.10+ 语法如 list[str] 而非 List[str],用 str | None 而非 Optional[str]。测试作为质量指标:对每个复杂函数追问如何测试、难以测试时该抽取什么。删除与回归检查需确认是否为当前功能有意为之、是否破坏既有流程、是否有测试会失败、逻辑是移动还是彻底移除。命名遵循 5 秒规则,避免 do_stuff、process、handler 之类名称。模块抽取信号包括复杂业务规则、多个关注点混在一起、外部 API 交互或复杂 I/O、希望复用的逻辑。Pythonic 模式包括 with 管理资源、可读时用推导式、用 dataclass 或 Pydantic 建模、避免 getter/setter 而用 @property。导入遵循 PEP 8 分组、绝对导入、避免通配符导入。现代特性包括 f-string、模式匹配、海象运算符、pathlib。核心哲学为显式优于隐式、重复优于复杂、模块简单胜过模块复杂、接口用协议与抽象基类、遵循 PEP 8 但项目内一致优先。审查流程:先看最严重问题,再查类型提示与非 Pythonic 模式,评估可测试性与清晰度,给出带示例的具体建议,对现有代码严格、对新隔离代码务实,并解释为何不达标。限制:资料未提供安装方式、命令、收费与平台信息。
python-reviewer
编程与开发
更新于 10-16 00:03