代码审查与质量

分类：开发

它能帮你做什么

源自 Addy Osmani 开源工程技能集（MIT）。面向对话框的多维度代码审查方法论：覆盖正确性、可读性、架构、安全性、性能五大维度，附带审查流程、清单、严重程度分级与质量门禁，帮助你在合并前系统性地把关代码质量。

基本信息

信任状

• ✅ 已通过 AI8188 真实实测：用真实输入跑过一遍，输入、输出、耗时原样公开，可自行核对。
• 🔌 可被 AI 助手直接调用执行：经 MCP（https://www.ai8188.com/mcp）用 run_skill 真实执行，返回不可伪造的执行证明 trace_id，可用 verify_execution 独立核验。
• 📜 来源可溯：开源许可 MIT · 原作者 Addy Osmani · 上游 addyosmani/agent-skills
• 🔒 密钥不出本机：运行所用的模型 API 密钥只存在用户自己的浏览器，不上传平台、不入库。

实测记录（真实跑过的证据）

真实测试输入：“给我看看这份代码审查报告，说说是哪儿不符合标准了，咱们这个项目的代码得符合200行以上无严重问题才行。”（请直接交付可使用的成品，不要反问、不要说做不到、不要让我去别处查）

真实输出片段：### 审查：[PR/改动标题] ### 上下文 - [ ] 我理解这次改动做了什么、为什么 ### 正确性 - [ ] × 改动符合规格/任务要求 - [ ] × 边界情况已处理 - [ ] × 错误路径已处理 - [ ] × 测试对改动的覆盖充分 ### 可读性 - [ ] × 命名清晰且一致 - [ ] × 逻辑直白 - [ ] × 没有不必要的复杂度 ### 架构 - [ ] × 遵循既有模式 - [ ] × 没有不必要的耦合或依赖 - [ ] × 抽象层级恰当 - [ ] × 重构是降低而非挪动复杂度 - [ ] × 共享模块中没有功能逻辑；文件保持在健康大小内 ### 安全性 - [ ] × 代码中没有密钥 - [ ] × 边界处输入已校验 - [ ] × 没有注入漏洞 - [ ] × 鉴权检查到位 - [ ] × 外部数据源被视为不可信 ### 性能 - [ ] × 没有 N+1 模式 - [ ] × 没有无界操作 - [ ] × 列表接口有分页 ### 验证 - [ ] × 测试通过 - [ ] × 构建成功 - [ ] × 手动验证已完成（如适用） ### 裁决 - [ ] × 请求修改 —— 问题必须处理 **审查发现：** - 代码中存在多处命名不规范、逻辑复杂、架构不符合要求等问题。 - 安全性方面存在潜在风险，如未对用户输入进行校验。 -

判定：✅ 通过 · 引擎 deepseek · Fri Sep 25

说明：以上为 AI8188 用真实输入在平台通道真跑该技能的记录，输入、输出、产出原样公开，可自行核对。

最近一次实测引擎：deepseek（Thu Sep 24）

实测记录（公开可查）：https://www.ai8188.com/api/reports?type=skill&id=684

运行

本技能可直接在 AI8188 网页内运行（无需安装）：https://www.ai8188.com/run/skill/684

页面：https://www.ai8188.com/s/gh-code-review-001　全量实测报告：https://www.ai8188.com/reports

（本页是给搜索引擎与 AI 助手抓取的静态版本；在浏览器打开会进入完整交互界面。技能提示词正文不在本页公开。）