CtrlK
BlogDocsLog inGet started
Tessl Logo

code-review

Run a comprehensive code review

54

Quality

60%

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
SKILL.md
Quality
Evals
Security

Quality

Content

75%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 well-structured, actionable task card with concrete commands, a deterministic synthesis gate, and an explicit exit condition. It assumes Claude's competence and avoids concept padding, with only minor verbosity from repeated gating language and template placeholders.

DimensionReasoningScore

Conciseness

The body is dense and operational, assuming Claude's competence without explaining basic concepts, but the independence/approval gating language is restated several times (Execution, Final Synthesis, Exit condition), fitting 'Efficient; minor instances of over-explanation that could be trimmed'. It is not a 5 because of that repeated gating prose.

4 / 5

Actionability

It supplies concrete shell commands (git status/diff, omx state write) and structured task() prompt templates with a report skeleton, but the task() blocks use pseudo-syntax with [scope and artifacts] placeholders that must be filled, matching 'Mostly executable guidance; concrete code or commands with minor gaps'. It is not a 5 because the prompts are templates rather than copy-paste-ready complete examples.

4 / 5

Workflow Clarity

A clear numbered Execution sequence is followed by an explicit Architectural Status decision tree, approval criteria, and a defined Exit condition, giving 'Clear sequence with most checkpoints present'. It is not a 5 because the error-recovery feedback loop for a failed/unavailable lane is described as a blocking state rather than a validate-fix-retry loop.

4 / 5

Progressive Disclosure

Content is split into well-labeled sections (When to use, Inputs, Execution, Review taxonomy, State/HUD, Final Synthesis, Evidence/output, Exit) with a single clearly-signaled one-level reference to templates/AGENTS.md, matching 'Good structure; most content is appropriately placed; references mostly clear'. It is not a 5 because the body exceeds ~50 lines and the lone external reference is mentioned in prose rather than enumerated as a reference list.

4 / 5

Total

16

/

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 is concise and names a recognizable domain, but it is too terse: a single generic action verb, no trigger/Use-when clause, and no synonyms to aid discovery. It sits just below the midpoint of the rubric.

Suggestions

Add a 'Use when...' clause naming concrete triggers (e.g. 'Use when the user asks for a code review, merge-readiness check, or quality/security assessment of a diff or PR').

Replace the generic 'Run' with concrete review actions such as 'identify spec, security, quality, and maintainability findings with severity ratings'.

Include natural synonyms (PR review, merge readiness, quality assessment) to improve trigger-term coverage and reduce overlap with adjacent skills.

DimensionReasoningScore

Specificity

The description names the domain ("code review") but the only action verb is the generic "Run" with the vague modifier "comprehensive", matching the anchor 'Names the domain but actions are minimal or generic'. It is not a 3 because no concrete review actions (e.g. find security issues, rate severity) are listed.

2 / 5

Completeness

It gives a clear 'what' (run a comprehensive code review) but provides no 'when'/Use-when trigger guidance; per the judging guidelines a missing 'Use when...' clause caps completeness at 3. It is not a 4 because the 'when' is entirely absent rather than merely weak.

3 / 5

Trigger Term Quality

"code review" is a natural phrase users would say, but no synonyms or variations (PR review, merge readiness, quality assessment) are present, fitting 'Some relevant keywords but missing common variations or synonyms'. It is not a 4 because coverage is limited to a single keyword.

3 / 5

Distinctiveness Conflict Risk

"code review" is a recognizable niche but the terse, modifier-heavy phrasing could overlap with debugging, security-review, or general coding skills, matching 'Somewhat specific but could still overlap with similar skills'. It is not a 4 because no qualifier narrows it away from those adjacent skills.

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.

Validation16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository
Yeachan-Heo/oh-my-codex
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.