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

68%

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 ./.agents/harness/adapters/antigravity/skills/requesting-code-review/SKILL.md

The canonical home for this skill is requesting-code-review in phatblat/dotfiles

SKILL.md
Quality
Evals
Security

Quality

Content

81%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 an extremely lean, generated redirect stub: perfectly concise and unambiguous about its single action, with one clearly signaled one-level-deep reference. Its weakness is that all actionable substance lives in the delegated file, which is not part of this bundle, so the skill as shipped here provides no steps, examples, or overview of its own. Judged as written, it executes its delegation pattern cleanly.

DimensionReasoningScore

Conciseness

The body is two lines — a generator comment and a single directive — with zero padding, no over-explanation of concepts Claude already knows, and every token earning its place. This cleanly matches the anchor-5 example of lean, competence-assuming content.

5 / 5

Actionability

The single instruction "Load and follow the shared skill at `~/.agents/skills/requesting-code-review/SKILL.md`" is concrete and executable (exact path), but all substantive guidance — steps, examples, commands — is delegated to a file outside the bundle that does not exist in this environment. This matches anchor 3 (some concrete guidance, key details missing) rather than 4, where gaps would be minor.

3 / 5

Workflow Clarity

This is a simple single-action skill and that action (load and follow the shared skill at the given path) is completely unambiguous; per the rubric's simple-skill exception, workflow clarity can score 5 without a multi-step sequence. No destructive or batch operations are involved, so no validation cap applies.

5 / 5

Progressive Disclosure

There is exactly one clearly signaled, one-level-deep reference and nothing is inlined that belongs elsewhere, which is good structure (anchor 4). It falls short of anchor 5 because the body is a bare pointer with no overview content of its own, and both the reference and the commented `scripts/agent-harnesses.py` live outside the bundle with no references/ or scripts/ structure to navigate.

4 / 5

Total

17

/

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 a clear, explicit trigger structure but never says what the skill actually does, and its headline trigger ("completing tasks") is far too broad. It also omits the most natural keywords for this domain ("code review", "review", "PR"), weakening both discoverability and distinctiveness. Overall it reads as a weak-to-middling description that leans entirely on its "when" clause.

Suggestions

State what the skill does before the "Use when" clause — e.g., "Requests a code review of completed work and relays reviewer feedback" — so the description answers both 'what' and 'when'.

Add the natural trigger terms users actually say for this domain: "code review", "review my work", "PR", "pull request".

Narrow the overly broad "completing tasks" trigger (e.g., to "finishing a coding task") to reduce conflict with testing, verification, and QA skills.

DimensionReasoningScore

Specificity

The only action phrase is "verify work meets requirements", which is generic; the description never states concrete actions like requesting a review, analyzing a diff, or relaying reviewer feedback. This matches anchor 2 (domain loosely named, actions minimal or generic) rather than 3, since no concrete actions are actually listed.

2 / 5

Completeness

The "when" is explicit and multi-triggered ("Use when completing tasks, implementing major features, or before merging"), but the "what" is only the vague purpose clause "verify work meets requirements" and the skill's actual function is never stated. This sits between anchor 2 (only when, no what) and anchor 4 (both present, one weak), so 3 fits best.

3 / 5

Trigger Term Quality

"implementing major features" and "before merging" are natural phrases users would say, but the most common triggers — "review", "code review", "PR", "pull request" — are entirely absent. Some relevant keywords exist (anchor 3), but coverage is not good enough for 4 given the missing core synonyms.

3 / 5

Distinctiveness Conflict Risk

"Use when completing tasks" is so broad it would trigger alongside virtually any skill, creating high overlap risk with testing, verification, and QA skills. It is not a 1 because "implementing major features" and "before merging" anchor it to a development/merge context, giving it some niche.

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

referenced_paths_exist

Referenced path issues: 1 missing

Warning

Total

15

/

16

Passed

Repository

phatblat/dotfiles

Path

.agents/harness/adapters/antigravity/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.