CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

code-review

Run a comprehensive code review

Quality

65%

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 ./skills/code-review/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

70%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.

The body is a dense, highly operational orchestration card with concrete git/omx commands, literal subagent prompts, a deterministic decision ladder, explicit approval criteria, and real feedback loops — strong on actionability and workflow clarity. Its weaknesses are redundancy (the unavailable-lane and blocking-gate rules restated across four sections, plus a duplicated Quality Gates section) and a monolithic all-inline structure with no reference files for gate and state-contract details.

Suggestions

State the unavailable-lane and blocking-gate rules once (e.g., in the Final Synthesis section) and reference them elsewhere; they currently recur in Execution, Custom code-review quality gates, Approval criteria/Do-not-self-review, and Exit condition, and the trailing 'Quality Gates' section duplicates the earlier 'Custom code-review quality gates' section.

Move the quality-gate mechanics and the State/HUD phase contract into a reference file (e.g. references/quality-gates.md) and keep a one-line pointer in SKILL.md, turning the 124-line monolith into an overview with one-level-deep references.

DimensionReasoningScore

Conciseness

The body is dense and operational with no padding about concepts Claude already knows, but the same rules are restated repeatedly: the unavailable-lane/do-not-approve rule appears in Execution step 2, the Approval criteria, the 'Do not self-review as a fallback' paragraph, and the Exit condition, and the 'Quality Gates' section at the end duplicates the 'Custom code-review quality gates' section. This fits anchor 3 ('Mostly efficient but... could be tightened'); the cross-section redundancy exceeds the 'minor instances' of anchor 4, while the absence of concept-explanation padding rules out 2.

3 / 5

Actionability

Guidance is largely executable: concrete commands ('git diff --stat', 'omx quality-gates list --stage code-review --json', a literal 'omx state write' payload), fully written subagent prompt templates, a concrete config example, and a complete report format. It falls short of anchor 5 because the task() calls are an abstracted delegation syntax with '[scope and artifacts]' placeholders rather than copy-paste-ready invocations, and the report template requires replacing illustrative values — minor gaps characteristic of anchor 4.

4 / 5

Workflow Clarity

The sequence is explicit (record scope -> parallel independent lanes -> custom gates -> deterministic synthesis -> report -> exit) with explicit validation checkpoints ('Await results before final synthesis', evidence required from both lanes) and genuine feedback loops ('Re-review the corrected scope and re-run gates before clearing that blocker'). The if/else Architectural Status Contract and explicit approval criteria make the final recommendation deterministic, matching anchor 5 ('Clear sequence with explicit validation steps; feedback loops for error recovery').

5 / 5

Progressive Disclosure

The card is a well-headered but monolithic 124-line body with no bundle files at all: the quality-gate mechanics and State/HUD contract details that would fit separate reference files are inlined, the quality-gates topic is split across two sections, and the only external pointers are a relative 'templates/AGENTS.md' path and one clearly signaled GitHub link. This fits anchor 3 ('Some structure but could be better organized; content that should be separate is inline') — the duplication and full inlining go beyond the 'minor organization gaps' of anchor 4, and the under-50-line exception for a 5 does not apply.

3 / 5

Total

15

/

20

Passed

Description

45%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 names the domain and one generic action and relies on the buzzword 'comprehensive' instead of concrete capabilities, with no 'Use when...' trigger clause and no synonyms beyond 'code review'. It is serviceable but generic, leaving completeness, trigger coverage, and specificity underdeveloped relative to the rubric's good examples.

Suggestions

Add 2-3 concrete capabilities, e.g. 'Run a merge-readiness code review: delegate independent code-reviewer and architect lanes, return severity-rated findings with file:line evidence, and issue an APPROVE / REQUEST CHANGES / COMMENT verdict.'

Append an explicit trigger clause such as 'Use when the user asks for a code review, merge-readiness check, or a review of a diff, PR, or commit before merging.'

Include natural synonyms and variations (diff review, PR review, merge readiness, quality or security assessment) so the description matches the varied phrasings users actually say.

DimensionReasoningScore

Specificity

The description names the domain and a single generic action — 'Run a comprehensive code review' — where 'comprehensive' is buzzword filler rather than a concrete capability. It matches anchor 2 ('Names the domain but actions are minimal or generic'); it is not 3 because no concrete review actions (severity-rated findings, dual-lane delegation, merge verdict) are stated, and not 1 because a real action in a named domain is present.

2 / 5

Completeness

There is a clear 'what' ('Run a comprehensive code review') but no 'when' — no 'Use when...' clause or equivalent trigger guidance, which the guidelines cap at 3. It fits anchor 3 exactly ('Has a clear what but when is missing or only weakly implied') and cannot be 4 without an explicit trigger clause.

3 / 5

Trigger Term Quality

'code review' is a genuinely natural keyword users would say, but it is the only one — common variations like 'review my diff', 'review this PR', 'merge readiness', or 'quality/security assessment' are absent. This fits anchor 3 ('Some relevant keywords but missing common variations or synonyms'); it is above 2 because 'code review' is the exact natural phrase, and below 4 because coverage is a single term with many synonyms missing.

3 / 5

Distinctiveness Conflict Risk

The phrase 'code review' carves out a recognizable niche, but the description offers no differentiating detail (merge-readiness verdict, independent dual-lane review) and would overlap with sibling skills like security review, diff/PR review, or simplification passes. This fits anchor 3 ('Somewhat specific but could still overlap with similar skills'); it lacks the distinct trigger set required for 4.

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

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.