Quality Review【免费下载链接】oh-my-claudecodeTeams-first Multi-agent orchestration for Claude Code项目地址: https://gitcode.com/GitHub_Trending/oh/oh-my-claudecodeSummaryOverall: [EXCELLENT / GOOD / NEEDS WORK / POOR]Logic: [pass / warn / fail]Error Handling: [pass / warn / fail]Design: [pass / warn / fail]Maintainability: [pass / warn / fail]Critical Issuesfile.ts:42- [CRITICAL] - [description and fix suggestion]Design Issuesfile.ts:156- [anti-pattern name] - [description and improvement]Positive Observations[Things done well to reinforce]Recommendations[Priority 1 fix] - [Impact: High/Medium/Low]四个维度Logic / Error Handling / Design / Maintainability的 pass/warn/fail 三态判定恰好映射第一节的职责清单Positive Observations 与 Recommendations 则分别落实正向反馈与按影响排序的修复建议两条成功标准。 **注意**该模板是通用质量评审版本。当切换模式时见第七节输出模板会替换为对应的 ## Style Review 等专用模板。 --- ## 六、五类失败模式与规避Failure Modes To Avoid Prompt 用黑名单方式列出评审者最易犯的五类错误并给出对照范例 | 失败模式 | 危害 | 规避手段 | |---|---|---| | 不读代码就评审 | 基于文件名或 diff 摘要形成偏见 | 始终读取完整代码上下文 | | 把风格当质量 | 命名/格式问题被误报为质量问题 | 风格检查须显式用 modelhaiku 触发 | | 只见树木不见森林 | 罗列 20 个小味道却漏掉核心算法错误 | 先查逻辑再查模式 | | 模糊批评 | 这个函数太复杂了没有落点 | 必须给出 file:line 复杂度数据 拆分建议 | | 零正向反馈 | 只列问题打击团队士气 | 主动记录做得好的部分 | 其中模糊批评给出了教科书级范例——**好例**paginator.ts:42 存在 off-by-onefor (let i 0; i items.length; i) 会访问未定义的 items[items.length]应将 改为 **坏例**代码可以重构一下提升可维护性无文件引用、无具体问题、无修复建议。这一对比直接内化为 Final_Checklist 的验收项每条问题是否都带 file:line、严重度与修复建议 --- ## 七、三种可切换的评审模式 ### 7.1 Style_Review_Mode样式审查modelhaiku 触发 当以轻量级模型 haiku 调用时Quality Reviewer 顺带接管原 style-reviewer 的职责覆盖**格式一致性、命名约定、语言惯用法、lint 规则合规、import 组织**。协议要点 1. 先读项目配置文件.eslintrc、.prettierrc、tsconfig.json、pyproject.toml 等理解约定 2. 检查格式缩进、行长、空白、大括号风格 3. 检查命名变量 camelCase/snake_case、常量 UPPER_SNAKE、类 PascalCase、文件命名遵循项目约定 4. 检查语言惯用法JS 用 const/let 而非 var、Python 用列表推导、Go 用 defer 清理资源 5. 检查 import 组织按约定、无未使用 import、若项目如此则按字母序 6. 标注哪些问题可自动修复prettier、eslint --fix、gofmt。 约束**引用项目约定而非个人偏好**只聚焦 CRITICAL混用 tab/空格、命名极度不一致与 MAJOR大小写约定错误、非惯用模式不为 TRIVIAL 问题吹毛求疵。输出使用独立模板 markdown ## Style Review ### Summary **Overall**: [PASS / MINOR ISSUES / MAJOR ISSUES] ### Issues Found - file.ts:42 - [MAJOR] Wrong naming convention: MyFunc should be myFunc (project uses camelCase) ### Auto-Fix Available - Run prettier --write src/ to fix formatting issues7.2 Performance_Review_Mode性能审查显式请求触发当请求涉及性能分析、热点识别或优化时启用职责包括算法复杂度问题O(n²) 循环、多余重渲染、N1 查询、内存泄漏与 GC 压力、延迟敏感路径与 I/O 瓶颈、性能剖析插桩点建议、数据结构与算法选型评估、缓存机会与失效正确性评估。评级收敛为三档CRITICAL生产影响/ HIGH可测量的性能退化/ LOW次要。7.3 Quality_Strategy_Mode质量策略发布就绪评估触发当请求涉及发布就绪、质量门禁或风险评估时启用职责包括对照风险面评估测试覆盖充分性单元/集成/e2e、识别变更路径缺失的回归测试、评估发布就绪度阻塞缺陷、已知回归、未测试路径、标记发布前必须通过的质量门禁、评估新功能的监控与告警覆盖并将变更按证据分级为SAFE / MONITOR / HOLD三档。三种模式与默认质量评审共同构成该 Agent 的能力矩阵——这也是 合并后 code-reviewer 的 description 将其概括为 severity-rated feedback, logic defect detection, SOLID principle checks, style, performance, and quality strategy 的直接来源。八、实战验证benchmark 如何评测这个 Prompt该 prompt 存放在benchmarks/code-reviewer/prompts/下并非偶然——run-benchmark.ts 的头部注释明确说明其用途对比合并后的新 code-reviewer 与旧的 quality-reviewer prompt 的评审质量。8.1 运行方式npx tsx benchmarks/code-reviewer/run-benchmark.ts [options]【免费下载链接】oh-my-claudecodeTeams-first Multi-agent orchestration for Claude Code项目地址: https://gitcode.com/GitHub_Trending/oh/oh-my-claudecode创作声明：本文部分内容由AI辅助生成（AIGC），仅供参考
