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