mcpField
O
omc-review

omc-review

github.com/yeachan-heo测试与质量40.4k249 次安装1 天前更新
代码仓库

关于 omc-review

omc-review 是用于评估现有代码改动的审查工作流技能,不负责编写被审查的变更。它按严重程度验证并报告问题,也说明检查范围和无法评估的部分。

主要功能

  1. 01先明确改动及其目标
  2. 02优先检查正确性与风险
  3. 03核实候选问题后再报告
  4. 04按严重程度排列发现
  5. 05重大改动分别进行规范与意图审查

适用场景

  1. 01审查已完成的代码变更
  2. 02检查安全边界与数据完整性风险
  3. 03确认变更是否符合需求和测试覆盖

安装

复制下面的命令,在终端中运行。

$npx skills add https://github.com/yeachan-heo/oh-my-claudecode --skill omc-review

安装后在 Claude Code 中输入/omc-review确认已加载。

详细介绍

审查

使用此技能评估已有工作。审查绝不编写其所评判的更改。

这是规范的审查工作流。ai-slop-cleaner 是其中的可选流程。

目标

找出实际存在的问题,按严重程度排序,并提供足够的细节以便采取行动。

工作流

  1. 确认更改了什么,以及预期要实现什么。
  2. 对照预期检查更改。
  3. 先检查正确性,再检查风险,最后检查简化空间。
  4. 报告前验证每个候选问题。
  5. 按严重程度从高到低报告问题。

检查内容

  • 正确性 — 逻辑缺陷、边界情况、错误路径、并发问题
  • 风险 — 安全边界、破坏性操作、数据完整性
  • 复用 — 更改本应使用的现有工具或模式
  • 简化 — 可以删除或合并的代码
  • 覆盖 — 发布时未经测试的行为

两条独立审查轴

对于实质性差异,应由两个并行的子 Agent 分别审查,并分别报告——绝不合并,也不交叉排序,因为一项更改可能通过一条审查轴,却在另一条审查轴上不合格(符合标准但行为错误;忠实实现但违反惯例):

  • 标准轴 — 根据仓库中记录的标准,以及仅依赖判断的代码异味基线,审查差异;工具已强制执行的内容不在审查范围内。应查找而非假定标准集:在仓库中搜索标准文档(CLAUDE.md/AGENTS.md、docs/standards/ 下的文档、CONTRIBUTING.md、CODING_STANDARDS.md 或同类文件),并根据实际存在的标准进行审查——未查找的标准文件,就是未应用的标准。
  • 意图轴 — 对照工作预期目标(任务、规格说明或工单验收标准)审查差异:检查缺失或未完整实现的要求、未经请求的行为,并在每个问题中引用其来源行。

每个子 Agent 的简报应控制在约 400 个词以内,并聚焦于一条审查轴。汇总时不得跨审查轴选出唯一胜者——避免重新排序正是分开审查的目的;应分别说明各审查轴的结论。小型差异只需进行一次审查。

规则

  • 分开审查:审查者不得与作者处于同一活跃上下文。
  • 报告前先验证。无法复现的貌似合理的问题只是噪音。
  • 如实说明严重程度;不要为了显得全面而凑数。
  • 工作没有问题时,可以报告“未发现问题”。
  • 默认提供建议——审查用于提供信息,而非阻止发布。发布、安全和破坏性操作等硬性门禁应单独处理,并在失败时阻止通过。

输出

  • 按严重程度从高到低列出问题,并为每项提供文件、行号和具体的失败场景
  • 说明已检查且确认无误的内容
  • 说明无法评估的内容

评论

支持 Markdown