skill by

OM Code Review

Review a diff, branch, or PR against correctness, security, breaking-change, and quality standards — runs the validation gate, applies the built-in checklist plus any repo-local one, and produces severity-ranked findings with an approve/request-changes verdict. The review engine behind om-auto-review-pr and om-review-prs.

Add skill

Follow @openmercato

Permissions

Files

SKILL.md

om-code-review

Code Review

Review code changes against the repository's architecture, security, convention, and quality standards. Produce actionable, categorized findings and a clear merge verdict.

Contract

Input — exactly one unit of review:

a PR number (fetch the diff and metadata via the tracker operations get-pr-diff / get-pr),
a branch name (review its diff against the merge-base with $BASE_BRANCH),
an explicit commit range or diff,
nothing — default to the current branch's diff against the merge-base with , including uncommitted changes.

5 files · 50 KB15 KB

Install

It’s free, and every skill you add syncs into every AI tool on your computer, instantly.

Get the Skillet appor

npx skilletmd add @openmercato/om-code-review -y

Version history

Severity	Criteria	Action
blocker	Security vulnerability, data corruption or loss risk, cross-scope data leak, missing permission check, breaking contract change without a deprecation path, failing validation gate	MUST fix before merge
major	Correctness bug on a realistic path, missing regression test for a bug fix, weakened assertions, unbounded query on growing data, unresolved race on shared state, architecture violation	MUST fix before merge unless the maintainer explicitly accepts and documents the risk
minor	Convention violation, suboptimal pattern, missing best practice, readability problem	Should fix; does not block on its own
nit	Style suggestion, optional polish	Author's call

OM Code Review

Code Review

Contract

Review Workflow

Validation Gate (MANDATORY)

Rules

UI Performance Gate

Output Format

Severity and Verdict

Quick Rule Reference

Breaking Changes (blocker)

Security (blocker)

Data Integrity (blocker/major)

Migration Sanity Gate (blocker)

Testing (major)

Review Heuristics

Rules

Security boundaries