CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

code-review

Run a comprehensive code review

Quality

64%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./plugins/oh-my-codex/skills/code-review/SKILL.md

The canonical home for this skill is code-review in Yeachan-Heo/oh-my-codex

SKILL.md
Quality
Evals
Security

Quality

Content

73%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

This is a strongly operational task card: it gives exact commands, verbatim subagent prompts, an explicit severity/status taxonomy, and a deterministic decision procedure with real feedback loops and failure handling. Its weaknesses are moderate redundancy in the quality-gates material and pseudo-templated agent invocations rather than fully executable calls. Progressive disclosure is reasonable for a self-contained card, though the gates and state contracts would benefit from extraction into reference files.

Suggestions

Consolidate the two quality-gates sections ('Custom code-review quality gates' and the closing 'Quality Gates' pointer) into one, and break the long gates paragraph into a short bulleted rule list (spawn, advisory vs blocking, re-review) to tighten token usage.

Move the detailed quality-gates configuration contract and State/HUD phase contract into a reference file (e.g. references/quality-gates.md), keeping a one-line pointer in the body to improve progressive disclosure.

Provide a concrete filled-in example of one task() invocation (with actual scope text and a real agent-call syntax) so the execution step is fully copy-paste ready.

DimensionReasoningScore

Conciseness

The card is dense and procedural with no padding about concepts Claude already knows, but it could be tightened: the quality-gates configuration is covered twice (the 'Custom code-review quality gates' section and again in the closing 'Quality Gates' section), and the single ~250-word gates paragraph mixes spawn rules, blocking semantics, and re-review policy in one block. This fits 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than anchor 4's 'minor instances'.

3 / 5

Actionability

Concrete, executable guidance dominates: verbatim git scope-recording commands, the exact `omx quality-gates list --stage code-review --json` and `omx state write` invocations with JSON payloads, full copy-ready subagent prompt texts, and a deterministic if/else synthesis ladder. It stops short of anchor 5 because the `task()` blocks are a pseudo-invocation containing '[scope and artifacts]' placeholders rather than fully copy-paste-ready calls.

4 / 5

Workflow Clarity

The sequence is explicit (record scope → parallel independent lanes → configured quality gates → synthesis → report → exit condition) with strong validation checkpoints: explicit approval criteria, blocking semantics for unavailable lanes and failed gates, and genuine feedback loops ('Re-review the corrected scope and re-run gates before clearing that blocker'). This matches 'clear sequence with explicit validation steps; feedback loops for error recovery; checklists', and the destructive/batch cap does not apply since the review is explicitly read-only.

5 / 5

Progressive Disclosure

No bundle files exist, and the single file is well-sectioned with clear one-level-deep references (templates/AGENTS.md for shared invariants, an external URL for gates configuration) and no nested reference chains. It matches 'good structure; most content is appropriately placed; minor organization gaps' — the detailed gates and State/HUD contract material could plausibly live in separate reference files — and falls short of anchor 5, which expects well-signaled reference files splitting the content.

4 / 5

Total

16

/

20

Passed

Description

36%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

The description is a bare domain label with a generic verb and no use-when triggers, despite the body containing rich, specific behavior (severity taxonomy, dual review lanes, quality gates, deterministic verdicts) that could anchor a strong description. It is serviceable for routing only in an environment with no competing review-like skills. Most of its failures come from omitting information the skill already possesses.

Suggestions

Add a 'Use when...' clause with concrete triggers, e.g. 'Use when the user asks to review a diff, PR, or commit, requests a merge-readiness or quality/security assessment, or asks whether changes are safe to merge.'

Replace 'comprehensive' with 2-3 concrete capabilities drawn from the body, e.g. 'Runs parallel code-reviewer and architecture review lanes, rates findings CRITICAL-HIGH-MEDIUM-LOW with file:line evidence, and returns a deterministic APPROVE / REQUEST CHANGES / COMMENT verdict.'

Include natural trigger variations users actually say ('review my code', 'review this PR', 'look over my changes', 'merge readiness check') to improve trigger-term coverage and distinctiveness.

DimensionReasoningScore

Specificity

The description only offers the generic action 'Run' plus the buzzword 'comprehensive'; it names the code-review domain but states no concrete capabilities (e.g., severity-rated findings, architecture review, merge verdict). It matches the 'names the domain but actions are minimal or generic' anchor and falls short of anchor 3, which requires 1-2 concrete stated actions.

2 / 5

Completeness

'Run a comprehensive code review' gives a vague 'what' and contains no 'when' clause or equivalent trigger guidance at all, which caps completeness at 3 per the judging guidelines. It sits between anchor 2 (vague 'what', no 'when') and anchor 3 (clear 'what', missing 'when'), but the 'what' is barely more than the domain name, keeping it at 3 rather than above.

3 / 5

Trigger Term Quality

'code review' is a genuinely natural phrase a user would say, but it is the only trigger term; common variations like 'review my diff', 'PR review', 'merge readiness', or 'security assessment' are absent. This fits 'some relevant keywords but missing common variations or synonyms' rather than anchor 4's 'good keyword coverage'.

3 / 5

Distinctiveness Conflict Risk

The code-review domain is somewhat specific, but with no distinguishing details (security findings, architecture lane, quality gates, merge verdict) it would readily overlap with other review or code-quality skills. This matches 'somewhat specific but could still overlap with similar skills', not anchor 4's 'mostly distinct'.

3 / 5

Total

11

/

20

Passed

Validation

100%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

Yeachan-Heo/oh-my-codex

Path

plugins/oh-my-codex/skills/code-review/SKILL.md

Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.