CtrlK
BlogDocsLog inGet started
Tessl Logo

requesting-code-review

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

50

Quality

55%

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

Quality

Content

71%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 well-structured and actionable, with concrete commands, a dispatch template, and clear feedback triage. Its main weakness is a broken reference to code-reviewer.md, which is cited twice but missing from the bundle, and some repetition between the "When to Request" and "Integration" sections.

Suggestions

Add the missing code-reviewer.md file to the bundle (or a references/ dir) so the cited template is reachable.

Merge the "Integration with Workflows" section into "When to Request Review" to remove the duplicated review-cadence guidance.

Add an explicit re-review checkpoint (e.g. "after fixing Critical/Important issues, re-dispatch the reviewer before proceeding") to close the feedback loop.

DimensionReasoningScore

Conciseness

The body is mostly efficient with concrete commands and tight sections, though the "Integration with Workflows" section repeats the "review after each task" guidance already in "When to Request Review" and the example dialogue could be trimmed.

4 / 5

Actionability

It provides executable bash (`git rev-parse HEAD~1`), a concrete dispatch template with named placeholders, and a clear Critical/Important/Minor triage; the main gap is that the referenced template file code-reviewer.md does not exist in the bundle.

4 / 5

Workflow Clarity

The get-SHAs → dispatch → act-on-feedback sequence is clear with prioritized checkpoints and an implicit fix-then-continue loop, only missing an explicit re-review checkpoint after fixes.

4 / 5

Progressive Disclosure

Sections are well organized and the code-reviewer.md reference is clearly signaled with a markdown link, but the referenced file is absent from the bundle, so navigation fails at the one place detailed content is deferred.

3 / 5

Total

15

/

20

Passed

Description

40%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 provides a clear trigger clause but omits the core capability entirely — it never reveals that the skill dispatches a code reviewer subagent. This makes it incomplete and overly broad, risking conflicts with related dev skills.

Suggestions

Lead with the concrete action, e.g. "Dispatch a code reviewer subagent to catch issues before they cascade.

Keep the "Use when..." trigger but pair it with an explicit what, such as "...before merging to verify work meets requirements."

Add a more distinctive trigger term like "code review" or "review a pull request" to reduce overlap with generic development skills.

DimensionReasoningScore

Specificity

The phrase "to verify work meets requirements" names a goal but no concrete actions; it never states that the skill dispatches a code reviewer subagent, leaving actions minimal and generic.

2 / 5

Completeness

Only the "when" is present via the "Use when..." clause; the "what" is entirely absent because the description never says what the skill does, matching the anchor where only one of what/when is present.

2 / 5

Trigger Term Quality

"Use when completing tasks, implementing major features, or before merging" gives good, natural keyword coverage a user would actually say, though a few synonyms (e.g., "code review", "PR") are missing.

4 / 5

Distinctiveness Conflict Risk

"Completing tasks, implementing major features, or before merging" is very broad and would overlap with testing, linting, and general development skills, creating high overlap risk.

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.

Validation15 / 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.