yeachan-heo/oh-my-claudecode

omc-review

Evaluate finished work for defects, risk, and simplification before it ships

查看源码

使用方法

  1. 复制安装命令,在自己的智能体环境安装前先检查源码包。
  2. 从原始文档核对所需工具、凭据和支持的宿主。
  3. 说明输入材料与期望交付物,并对照来源检查输出。
npx skills add https://github.com/yeachan-heo/oh-my-claudecode --skill omc-review

开始前需了解

  • 安装不包含模型订阅、API 额度或访问文件的权限。
  • 仓库星数和安装量反映活跃情况,不等于独立质量或安全测试。
仓库原始内容

按源仓库内容呈现,保留标题、案例、代码、表格、链接以及原文引用的演示图片。

Review

Use this skill to evaluate work that already exists. Review never authors the change it is judging.

This is the canonical review workflow. merge-readiness routes here, and ai-slop-cleaner is an opt-in lane within it.

Goal

Find what is actually wrong, ranked by severity, with enough detail to act on.

Workflow

  1. Establish what changed and what it was meant to do.
  2. Read the change against that intent.
  3. Check correctness first, then risk, then simplification.
  4. Verify each candidate finding before reporting it.
  5. Report findings most-severe first.

What to check

  • Correctness — logic defects, edge cases, error paths, concurrency
  • Risk — security boundaries, destructive operations, data integrity
  • Reuse — existing utilities or patterns the change should have used
  • Simplification — code that could be deleted or collapsed
  • Coverage — behavior that ships untested

Rules

  • Separate lanes: the reviewer must not be the author's same active context.
  • Verify before reporting. A plausible-sounding finding that does not reproduce is noise.
  • State severity honestly; do not pad the list to look thorough.
  • "No findings" is a valid result when the work is sound.
  • Advisory by default — review informs, it does not gate. Hard gates (release,

security, destructive operations) stay separate and fail closed.

Output

  • Findings, most-severe first, each with file, line, and concrete failure scenario
  • What was checked and found clean
  • Anything that could not be assessed

按任务相关推荐

来自同一仓库

更多 Skills

全部 Skills