code-reviewer

代码审查器

自动化代码审查工具，可分析拉取请求的复杂度和风险，扫描源代码中的代码异味和 SOLID 违规，并将发现汇总为结构化审查报告，其中包含结论、评分和按优先级排序的待办事项。支持 TypeScript、JavaScript、Python、Go、Swift 和 Kotlin。

核心功能

• PR 风险分析 — 扫描 git 差异，查找硬编码密钥、SQL 注入、调试语句、lint 绕过、any 类型和 TODO/FIXME，并给出 1-10 的复杂度评分。
• 代码质量检查 — 根据固定阈值检测过长函数、过大的文件、上帝类、嵌套过深、参数过多、圈复杂度过高、缺少错误处理、未使用的导入和魔法数字。
• SOLID 与反模式检测 — 标记结构、逻辑、安全、性能、测试和异步方面的反模式，并提供修复建议。
• 审查报告 — 将 PR 和质量检查结果合并为审查结论（批准 / 请求修改 / 阻止）、0-100 的评分以及按优先级排序的待办事项，支持文本、markdown 或 JSON 格式。
• 审查顺序优先级排序 — 对文件排序，优先检查安全敏感代码。
• 提交规范检查 — 验证分支中的提交是否符合 Conventional Commits 格式。

适用场景

• 审查 PR 时，需要在手动检查逻辑前快速进行初筛。
• 分析代码质量，或审计代码库中的代码异味和 SOLID 违规。
• 生成结构化审查报告或审查清单。
• 在 CI/CD 中依据质量评分或审查结论设置合并门槛。

先行确认

生成审查结果前，请确认以下输入。如果有任何一项未知或含糊不清，请先询问，不要自行假设：

• 审查内容 — git 差异/分支（base 和 head）或源代码目录（通过 --base/--head 设置，或指定分析器扫描的路径）
• 语言 — TypeScript、JavaScript、Python、Go、Swift 或 Kotlin（通过 --language 选择 code_quality_checker.py 应用的规则）
• 输出格式与门槛 — 报告格式（text/markdown/JSON）和任何通过/失败评分阈值（用于设置产物格式和 CI 结论）

停止规则：只询问对输出影响最大的 2-3 项。如果用户说“直接起草”，则继续操作，并在产物开头列出假设。

工具

工具	用途	命令
pr_analyzer.py	根据 git 差异评估 PR 的复杂度和风险	python scripts/pr_analyzer.py . --base main --head feature-branch --json
code_quality_checker.py	检测源代码中的代码异味和 SOLID 违规	python scripts/code_quality_checker.py ./src --language typescript --json
review_report_generator.py	将 PR 和质量检查结果合并为审查报告	python scripts/review_report_generator.py . --format markdown --output review.md

参考资料

根据任务加载相应的参考资料——保持此文件精简，按需查阅详细信息：

• references/tool_reference.md — 各工具的完整用法、检测项列表、阈值/结论表、所有 CLI 标志、JSON 输出示例、支持的语言扩展名、故障排查表和成功标准。运行、配置或调试脚本时阅读。
• references/code_review_checklist.md — 涵盖预审、正确性、安全性、性能、可维护性、测试和特定语言检查的系统化清单。进行人工审查时阅读。
• references/coding_standards.md — TypeScript、JavaScript、Python、Go、Swift 和 Kotlin 的语言专属标准（类型、空值安全、异步、错误处理、类设计）。评估代码风格和惯用写法是否合规时阅读。
• references/common_antipatterns.md — 反模式目录（结构、逻辑、安全、性能、测试、异步），包含示例、检测线索和修复建议。解释或修复已标记的代码异味时阅读。

范围与限制

涵盖：

• 对 git 差异中的静态模式进行风险检测（密钥、SQL 注入、调试语句、绕过 lint 检查）
• 结构化代码质量分析：函数长度、类大小、圈复杂度、参数数量、SOLID 违规
• PR 元数据评估：按风险优先级对文件分类、验证提交消息、评估复杂度
• 生成综合审查报告，包含结论、评分，以及适用于文本、markdown 和 JSON 格式的优先级行动项

不涵盖：

• 运行时或动态分析——使用 senior-qa 执行测试，使用 qa-browser-automation 进行端到端测试
• 安全漏洞扫描（CVE 数据库、依赖项审计）——使用 senior-security 或 senior-secops 进行 SAST/DAST 和供应链分析
• 性能分析或基准测试——使用 senior-backend 或 senior-fullstack 执行性能优化工作流
• 架构层面的审查（系统设计、服务边界、API 契约验证）——使用 senior-architect 编写架构决策记录并进行设计审查

集成点

技能	集成方式	数据流
senior-security	将 PR Analyzer 的关键发现送入安全审查工作流，进行更深入的 SAST/DAST 分析	pr_analyzer.py --json 输出 risks.critical[] → 安全评估输入
senior-qa	根据审查报告结论控制测试执行；当结论为 block 时阻止测试套件运行	review_report_generator.py --json 输出 summary.verdict → QA 流水线门禁
senior-architect	将高复杂度 PR（评分 7+）升级至架构审查	pr_analyzer.py 输出 summary.complexity_score → 架构审查触发条件
senior-fullstack	将代码质量评分与 fullstack 质量分析器结合，评估端到端项目健康状况	code_quality_checker.py --json 输出 → 与 code_quality_analyzer.py 指标合并
tdd-guide	将审查发现与测试覆盖率交叉比对；标记质量检查器指出的未测试代码路径	质量检查器按文件列出的 smells[] → TDD 覆盖率缺口分析
senior-devops	将审查报告集成至 CI/CD 流水线，作为自动化质量门禁	review_report_generator.py --json 输出 summary.score → 流水线通过/失败阈值