Everything Claude Code 的 python-reviewer Agent 是面向生产环境的专业 Python 代码审查工具，专注于 PEP 8 规范、Pythonic 编程习惯、类型注解、常见安全漏洞和性能隐患的自动检测。它能自动分析代码变更、静态检查、类型验证及安全扫描，输出结构化审查报告，是所有 Python 项目必备的质量保障环节。相比直接问 Claude，专用 Agent 能实现无遗漏、系统化、可追溯的自动化代码审查，极大提升 AI 编程助手的专业性与安全性。

Everything Claude Code Python Reviewer Agent：PEP 8、Pythonic 惯用法、类型注解与安全代码审查

在 AI 编程助手（如 Claude Code、Codex、Cursor 等）辅助开发逐渐成为主流的今天，如何让 AI 参与代码审查既专业、又安全可靠，成为团队生产力提升的关键。Everything Claude Code 的 python-reviewer Agent，正是为此场景量身定制的生产级解决方案。本文将系统介绍其核心能力、使用方式、与其他 Agent 的协作模式，以及实际对话示例，帮助你在 Python 项目中最大化 AI 代码审查的价值。

1. python-reviewer Agent 能解决什么问题？

python-reviewer 是 Everything Claude Code（ECC）内置的 38 个专业 Agent 之一，专门用于Python 代码的自动化审查。它的设计目标是让每一次 Python 代码变更都能：

• 自动检测 PEP 8 合规性：如 import 顺序、命名、缩进、空格等基础规范。
• 识别非 Pythonic 写法：比如 C 风格循环、字符串拼接、魔法数字、可变默认参数等。
• 强制类型注解：所有公开函数都需类型标注，避免 Any 滥用，参数可空需明确 Optional。
• 安全漏洞扫描：SQL 注入、命令注入、路径穿越、eval/exec 滥用、硬编码密钥、弱加密等。
• 高风险代码习惯：如裸 except、异常吞噬、未用上下文管理器、深层嵌套、重复代码等。
• 性能与并发隐患：N+1 查询、同步/异步混用、共享状态无锁等。
• 主流框架专项检查：Django、FastAPI、Flask 的安全与最佳实践。

核心场景：

• 所有 Python 代码的新增、修改、重构后，自动或手动触发审查
• 团队协作 PR、代码合并前的质量与安全把关
• 个人开发时的即时自检，防止低级错误与安全隐患流入主干

2. python-reviewer 的具体能力与限制

具体能力

1. 自动分析代码变更
   • 通过 git diff -- '*.py' 精准定位本次提交或变更的 Python 文件，聚焦增量审查。
2. 静态分析与类型检查
   • 集成 ruff、mypy、pylint、black --check 等主流工具，自动运行并解析结果。
3. 结构化输出
   • 所有问题分为 CRITICAL（阻断）、HIGH（高优先级）、MEDIUM（警告），并严格按如下格式输出：

     [SEVERITY] Issue title
     File: path/to/file.py:42
     Issue: 描述
     Fix: 建议修改

4. 安全与最佳实践优先级
   • 安全问题（如注入、密钥、反序列化）永远最高优先，必须阻断。
   • 类型注解、Pythonic 惯用法、代码质量、并发风险等均有针对性检查。
5. 框架专项规则
   • Django 检查 N+1 查询、事务原子性、迁移遗漏等
   • FastAPI 检查 CORS、Pydantic 校验、响应模型、异步阻塞等
   • Flask 检查错误处理、CSRF 防护

不能做的事

• 不负责业务逻辑正确性：只关注代码规范、安全、类型、性能、惯用法。
• 不自动修复问题：仅输出结构化审查报告，修复需开发者或配合其他 Agent 完成。
• 不覆盖非 Python 文件：仅针对 .py 文件，其他语言需用对应 Reviewer Agent。
• 不输出超出格式的内容：所有输出严格结构化，便于后续自动处理或集成。

3. 触发方式：自动激活 vs 手动调用

• 自动激活：
  • 在 ECC 工作流中，每当 Python 代码有变更（如提交、保存、PR 创建），系统会自动触发 python-reviewer Agent，无需手动干预。
  • 结合 Hooks 机制，可实现如 Pre-commit/Pre-PR 自动拦截，防止不合规代码流入主干。
• 手动调用：
  • 在 AI 编程助手界面，开发者可直接选择“用 python-reviewer 审查当前变更”。
  • 适合本地开发、代码初稿自检，或在集成流程外临时复查。

最佳实践：
建议将 python-reviewer 设为所有 Python 项目的必经审查环节，通过自动化 Hook 保证每次提交都被覆盖，避免人工遗漏。

4. 与其他 Agent 的协作模式

Everything Claude Code 强调Agent-Orchestration（多 Agent 编排），python-reviewer 常与以下 Agent 组合使用：

• tdd-guide：先用 tdd-guide 生成测试用例，再用 python-reviewer 检查实现代码的规范与安全。
• code-reviewer：code-reviewer 负责跨语言通用规范，python-reviewer 负责 Python 细节，两者并行提升审查覆盖率。
• security-reviewer：如发现高危安全问题，可自动转交 security-reviewer 深度分析全仓库安全隐患。
• build-error-resolver：如静态分析或类型检查报错，可自动调用 build-error-resolver 进行修复建议。

这种多 Agent 协同，是 ECC 体系区别于单一 AI 助手的最大优势。更多 Agent 编排模式详见 Everything Claude Code 完全指南。

5. 为什么要用专门的 python-reviewer Agent，而不是直接问 Claude？

• 专业性：python-reviewer 预设了顶级 Python 团队的审查标准和细致的安全清单，远超通用 AI 的泛泛建议。
• 自动化与一致性：每次变更都自动、结构化、无遗漏地执行审查，避免人工提问的主观性和遗漏。
• 可追溯性：输出严格结构化，便于集成到 CI/CD、PR 流程、质量门控和历史审计。
• 与其他 Agent 协同：可与类型检查、测试生成、全局安全扫描等 Agent 组成完整的 AI 代码保障链路。
• 安全保障：所有安全问题均强制阻断，不会因疏忽被忽略，符合生产级安全要求。更多安全实践可参考 AI 编程助手安全指南。

6. 实际使用示例

假设你在开发一个 FastAPI 项目，刚提交了如下代码变更：

# app/api.py
from fastapi import APIRouter, Request
import sqlite3

router = APIRouter()

@router.post("/user")
async def create_user(request: Request):
    data = await request.json()
    conn = sqlite3.connect("users.db")
    conn.execute(f"INSERT INTO users (name) VALUES ('{data['name']}')")
    conn.commit()
    conn.close()
    return {"status": "ok"}

python-reviewer Agent 自动输出：

[CRITICAL] SQL Injection risk in query
File: app/api.py:9
Issue: f-string used to interpolate user input into SQL query, leading to SQL injection vulnerability.
Fix: Use parameterized queries with placeholders: conn.execute("INSERT INTO users (name) VALUES (?)", (data['name'],))

[HIGH] Missing type annotations on public function
File: app/api.py:7
Issue: Public API function 'create_user' lacks type hints for parameters and return type.
Fix: Add type annotations, e.g., async def create_user(request: Request) -> dict:

[MEDIUM] Missing docstring on public function
File: app/api.py:7
Issue: Public API function 'create_user' lacks a docstring describing its behavior.
Fix: Add a docstring to explain the function's purpose and parameters.

审查结论：

• 有 CRITICAL 问题（SQL 注入），必须修复后才能合并。
• HIGH 问题（类型注解）需尽快补全。
• MEDIUM 问题（缺少 docstring）建议完善，但不阻断合并。

7. 操作步骤与最佳实践

1. 配置 ECC 插件与 Agent
   参考 安装配置指南 完成 ECC 的安装与 python-reviewer 激活。
2. 集成自动化 Hook
   推荐结合 Everything Claude Code Hooks 实战 配置 Pre-commit/Pre-PR 自动触发。
3. 日常开发流程
   • 代码变更 → 自动触发 python-reviewer → 审查报告输出
   • 按报告修复问题，必要时调用其他 Agent 协同处理
   • 审查通过（无 CRITICAL/HIGH）后方可提交/合并
4. 定期全仓库审查
   可定期手动触发 python-reviewer 对全项目做一次安全与规范巡检，防止历史遗留问题。

更多关于 Python 编码规范与最佳实践，可参考 Everything Claude Code Python Patterns Skill。

---

FAQ

Q: python-reviewer 和 code-reviewer 有什么区别？
A: python-reviewer 专注于 Python 语言的细粒度规范、安全、类型和惯用法，code-reviewer 更偏向通用的代码质量和可维护性，建议并行使用。

Q: python-reviewer 能自动修复问题吗？
A: 它只负责发现和结构化输出问题，不自动修改代码。可结合 build-error-resolver 或 AI 编辑器的自动修复能力。

Q: 如果我的项目用的是 Django/Flask/FastAPI，python-reviewer 能识别框架特有风险吗？
A: 可以。它内置了对主流 Python Web 框架的专项检查（如 Django 的 select_related、Flask 的 CSRF、防 N+1 查询等）。

---