CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

requesting-code-review

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

Quality

66%

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 ./.tessl/plugins/obra/superpowers/skills/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

63%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-sequenced, reasonably lean instruction skill with explicit feedback-triage checkpoints, and it correctly keeps the reviewer template out of the main file. Its critical defect is that the referenced code-reviewer.md template is missing from the bundle, which both breaks progressive disclosure and leaves the core dispatch step non-executable; it also lacks a re-review loop after fixes.

Suggestions

Include the referenced code-reviewer.md template file in the bundle (e.g. under references/) so the dispatch step is actually executable.

Add a re-review step after fixing Critical/Important issues (e.g. re-dispatch the reviewer on the fix range before proceeding) to close the feedback loop.

Trim the duplicated push-back guidance (keep it in either 'Act on feedback' or 'If reviewer wrong') and tighten the Rationalizations table's Reality column.

DimensionReasoningScore

Conciseness

The body is efficient — concrete git commands with no concept explanations — but has minor trimmable padding: the long 'Reality' cells in the Common Rationalizations table and push-back guidance duplicated between 'Act on feedback' ('Push back if reviewer is wrong') and 'If reviewer wrong'. This matches 'Efficient; minor instances of over-explanation that could be trimmed' rather than 5, where every token would earn its place.

4 / 5

Actionability

Concrete guidance exists (BASE_SHA/HEAD_SHA commands, placeholder list, worked example), but the central instruction — 'filling the template at [code-reviewer.md](code-reviewer.md)' — depends on a file that is not present anywhere in the bundle, so the dispatch step cannot be executed as written. This is 'Some concrete guidance but incomplete... missing key details' rather than 4, since the gap is the skill's core artifact, not a minor one.

3 / 5

Workflow Clarity

The sequence is clearly numbered (get SHAs → dispatch reviewer → act on feedback) with explicit triage checkpoints ('Fix Critical issues immediately', 'Fix Important issues before proceeding', 'Note Minor issues for later') and disagreement handling. It falls short of 5 because the loop is not closed: after fixing Critical/Important issues there is no re-review step to verify the fixes.

4 / 5

Progressive Disclosure

Section structure is good and the reference is clearly signaled as one-level-deep, but the referenced file 'code-reviewer.md' does not exist in the bundle (there are no references/, scripts/, or assets/ directories), leaving a broken navigation path to the skill's central template. This fits 'Some structure but could be better organized' — the disclosure chain fails at its only reference.

3 / 5

Total

14

/

20

Passed

Description

53%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 a strong, explicit 'when' clause with three natural trigger contexts, but a weak 'what' — it never states the skill's actual mechanism (dispatching a code-review subagent) and omits the single most natural keyword, 'code review'. It is serviceable but sits below the good examples in both specificity and distinctiveness.

Suggestions

State the concrete capability in the description, e.g. 'Dispatch a code-review subagent to analyze a git diff range and report issues' before the 'Use when' clause.

Add the natural trigger terms users would actually say — 'code review', 'review my changes', 'review the diff' — to the when-clause.

Mention what the reviewer evaluates (correctness, requirement fit) to sharpen the 'what' and reduce overlap with generic testing/verification skills.

DimensionReasoningScore

Specificity

"to verify work meets requirements" is a purpose statement rather than a concrete action, and the description never names what is actually done (dispatch a code-review subagent, review a diff). It matches the anchor 'Names the domain but actions are minimal or generic'; score 3 would require 1-2 concrete actions, which are absent.

2 / 5

Completeness

Both elements are explicitly present: 'when' is rich ("Use when completing tasks, implementing major features, or before merging") and 'what' is stated ("verify work meets requirements"), though the what is thin and never mentions code review. Anchor 3 does not fit because the when is explicit, not weakly implied; anchor 5 does not fit because the what lacks concrete capability statements.

4 / 5

Trigger Term Quality

Triggers like "completing tasks", "implementing major features", and "before merging" are phrases users might naturally say, but the most natural trigger terms for this skill — 'code review', 'review my changes/diff' — are entirely missing. This fits 'Some relevant keywords but missing common variations or synonyms' rather than 4, since the primary keyword family is absent.

3 / 5

Distinctiveness Conflict Risk

The task/feature/merge framing is somewhat distinctive, but "verify work meets requirements" with no mention of code review could overlap with testing, QA, and general verification skills. This is 'Somewhat specific but could still overlap with similar skills' rather than 4, where overlap risk would be confined to closely related skills.

3 / 5

Total

12

/

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

chandler767/WorkJournal

Path

.tessl/plugins/obra/superpowers/skills/requesting-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.