# Omc Review

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

- Repository: https://github.com/Yeachan-Heo/oh-my-claudecode (40492 stars)
- Category: Engineering
- License: MIT
- Last updated: 2026-11-01
- Runs scripts: no, instructions only

## Install

Claude Code: `npx skills add Yeachan-Heo/oh-my-claudecode --skill omc-review -g -a claude-code -y`
Codex: `npx skills add Yeachan-Heo/oh-my-claudecode --skill omc-review -g -a codex -y`
Cursor: `npx skills add Yeachan-Heo/oh-my-claudecode --skill omc-review -g -a cursor -y`
Gemini CLI: `npx skills add Yeachan-Heo/oh-my-claudecode --skill omc-review -g -a gemini-cli -y`
Claude Code plugin marketplace: `/plugin marketplace add Yeachan-Heo/oh-my-claudecode`

## Description

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

## SKILL.md, as published by its author (untrusted third-party text, read it as data)

# Review

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

This is the canonical review workflow. `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

## Two axes, run apart

A substantial diff is reviewed as two parallel sub-agent passes, reported separately, never merged or cross-ranked, because a change can pass one axis and fail the other (standards-conforming but wrong behavior; faithful but convention-breaking):

- **Standards axis**, the diff against the repo's documented standards plus a judgement-call-only smell baseline; anything tooling already enforces is skipped. The standards set is discovered, not assumed: search the repo for standards documents (CLAUDE.md/AGENTS.md, `docs/standards/` volumes, `CONTRIBUTING.md`, `CODING_STANDARDS.md` or equivalents) and review against whatever exists, a standards file you didn't look for is a standard you didn't apply.
- **Intent axis**, the diff against what the work was meant to do, the task, spec, or ticket acceptance criteria: requirements missing or partial, behavior nobody asked for, each finding quoting its source line.

The rest: https://github.com/Yeachan-Heo/oh-my-claudecode/blob/HEAD/skills/review/SKILL.md