CtrlK
BlogDocsLog inGet started
Tessl Logo

requesting-code-review

Use when completing tasks, implementing major features, or before merging to verify work meets requirements

52

Quality

58%

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 ./eval/local/skills/benchmarks/dependency/superpowers/requesting-code-review/SKILL.md

The canonical home for this skill is requesting-code-review in obra/superpowers

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 tight, actionable playbook: concrete git commands, a named dispatch procedure with placeholders, severity-based feedback triage, and clear red flags. Its main gaps are the missing code-reviewer.md template (breaking both full executability and reference navigation) and a partially redundant workflows section, plus no explicit re-review loop after fixes.

DimensionReasoningScore

Conciseness

The body is lean and operational — no explanations of concepts Claude already knows, and every section carries guidance (commands, triage rules, red flags). Minor trimming is possible: the 'Integration with Workflows' section restates triggers already listed in 'When to Request Review', and the narrative example is slightly padded, keeping it below the 'every token earns its place' anchor.

4 / 5

Actionability

Concrete, executable commands are provided (git rev-parse for both SHAs, a realistic git log/grep/awk pipeline in the example) plus a named subagent type, placeholder list, and severity-based triage rules. Not score 5 because the central dispatch step depends on the template in code-reviewer.md, which is not present in the bundle, so the reader cannot fully execute the key step from the skill alone.

4 / 5

Workflow Clarity

A clear 3-step sequence (get SHAs → dispatch reviewer → act on feedback) with explicit checkpoints (fix Critical immediately, Important before proceeding, Minor later) and a push-back path when the reviewer is wrong. Not score 5 because there is no re-review loop after fixes are applied — the 'fix → verify the fix' feedback loop is left implicit.

4 / 5

Progressive Disclosure

The ~100-line body is well organized into scannable sections, and the sole external reference ([code-reviewer.md](code-reviewer.md)) is clearly signaled, one level deep, and appropriately holds the bulky prompt template out of the overview. Not score 5 because that referenced file does not exist in the bundle (no references/, scripts/, or assets/ directories), leaving the primary detailed artifact unreachable despite good signaling.

4 / 5

Total

16

/

20

Passed

Description

41%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 has an explicit and reasonably specific trigger clause, but it entirely omits what the skill actually does (dispatch a code-reviewer subagent) and never uses the natural term 'code review'. This leaves it vague on capabilities and prone to firing on any task-completion or verification request.

Suggestions

State the 'what' explicitly, e.g. 'Dispatches a code-reviewer subagent with a crafted diff context (base/head SHAs) to catch issues before they cascade' — this would lift both specificity and completeness.

Add the natural trigger terms users actually say: 'code review', 'review my changes/diff', 'pull request', 'PR', 'before merging to main'.

Narrow the overly broad trigger 'completing tasks' to concrete checkpoints (e.g. 'after completing a task in subagent-driven development, after a major feature, before merging') to reduce conflict with generic verification and review skills.

DimensionReasoningScore

Specificity

The description names the domain only obliquely ("verify work meets requirements") and states no concrete actions — it never says it dispatches a code-reviewer subagent, reviews diffs, or acts on feedback. It sits at 'names the domain but actions are minimal or generic' rather than score 3, which requires 1-2 concrete actions.

2 / 5

Completeness

The 'when' is explicit and specific ("Use when completing tasks, implementing major features, or before merging"), but the 'what' is only a vague purpose clause ("verify work meets requirements") with no stated capability. This mirrors anchor 3's one-clear-one-weak profile; not score 4 because the 'what' — not the 'when' — is the weak half, and it is weaker than 'could be more specific'.

3 / 5

Trigger Term Quality

It includes some natural, relevant phrases ("completing tasks", "implementing major features", "before merging") but misses the most common user phrasings — 'code review', 'review my changes/diff', 'pull request'/'PR' never appear. Not score 4 because common variations and synonyms are absent, not just 'a few'.

3 / 5

Distinctiveness Conflict Risk

"Use when completing tasks" is an all-encompassing trigger that overlaps heavily with many similar skills (verification, testing, code-review, PR-review skills). The merge/feature language narrows it to software development, but within that domain it is very broad — closer to 'very broad; high overlap risk with many similar skills' than to anchor 3's 'somewhat specific'.

2 / 5

Total

10

/

20

Passed

Validation

93%

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

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

relative_links

Relative link issues: 2 missing

Warning

Total

15

/

16

Passed

Repository
rpamis/comet
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.