CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review

Perform Fallow's comprehensive pre-merge review. Use after implementation or when asked to review a branch, pull request, or diff.

Quality

87%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Low

Low-risk findings worth noting

SKILL.md
Quality
Evals
Security

Quality

Content

100%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 exemplary lean orchestration skill: a numbered, well-sequenced review workflow with explicit classification labels, a finding-filter checklist, a fix-and-rerun feedback loop, and detail properly deferred to clearly named project documents. Every line is actionable instruction with no padding or concept explanation.

DimensionReasoningScore

Conciseness

The ~30-line body is pure directive ("Re-read AGENTS.md, the active plan, the current branch, and the complete diff"; "trace the call site before you accept a bug claim") with zero concept explanations or padding — every token earns its place, matching anchor 5 ('Lean and efficient; assumes Claude's competence'). Not 4 because there are no instances of over-explanation left to trim.

5 / 5

Actionability

Concrete, executable guidance throughout for an instruction-only skill: exact file paths ('docs/development/quality-gates.md', 'docs/development/review-routing.md', 'CONTRIBUTING.md'), an explicit decision rule ("A defect or an unfinished part of the task under 'Possible follow-ups' or 'Known limits' is a BLOCK"), and fixed labels (APPROVE, CONCERN, BLOCK). Not 4 because the common conditional cases ("When runtime behavior changed", "When the change rejects input that was accepted before") are covered with specific instructions, leaving no gaps.

5 / 5

Workflow Clarity

Nine steps in coherent order (context, gates, reviewers, review areas, PR-body check, behavior comparison, finding filter, classify, fix) with an explicit feedback loop ("Fix every block, rerun the blocking reviewer, and update verification") and a verification mandate ("Review the actual source and live output. Do not infer correctness from a green compile") — matching anchor 5's 'explicit validation steps; feedback loops for error recovery'. Not 4 because checkpoints (classification, verification notes, rerun loop) are explicit rather than minor-gapped.

5 / 5

Progressive Disclosure

The skill is under 50 lines with no bundle files (no references/, scripts/, or assets/ exist) and is organized as a clean numbered workflow, so it qualifies for the simple-skill exception (5 with well-organized sections). Detail is correctly deferred one level deep to clearly signaled project docs ('docs/development/quality-gates.md', 'CONTRIBUTING.md'); nothing that belongs in a separate file is inlined. Not 4 because references are clearly signaled and organization has no gaps.

5 / 5

Total

20

/

20

Passed

Description

70%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 follows the strong what-plus-when pattern with an explicit, concrete 'Use when' clause naming branch, pull request, and diff triggers. Its main weakness is the what-side: 'comprehensive pre-merge review' is a single high-level action whose breadth is asserted ('comprehensive') rather than stated, and common trigger synonyms like 'PR' or 'code review' are missing.

Suggestions

Replace the vague qualifier 'comprehensive' with the review's actual components, e.g. 'Runs quality gates, routes reviewers, classifies findings as APPROVE/CONCERN/BLOCK, and fixes blocks' so the what enumerates concrete actions.

Add common trigger variations to the 'Use when' clause, such as 'PR', 'code review', or 'before merging', to broaden natural keyword coverage.

DimensionReasoningScore

Specificity

"Perform Fallow's comprehensive pre-merge review" names the domain (pre-merge review) with a single action, matching anchor 3 ('Names domain and 1-2 concrete actions, but not comprehensive') — 'comprehensive' asserts breadth without stating what the review covers (gates, reviewer routing, classification). Not 2 because the domain and action are more concrete than 'Processes PDF files'; not 4 because no several specific actions are enumerated.

3 / 5

Completeness

Both parts are explicit: what ("Perform Fallow's comprehensive pre-merge review") and when ("Use after implementation or when asked to review a branch, pull request, or diff") with concrete triggers. Not 5 because the what is a single high-level action where 'comprehensive' claims scope rather than stating capabilities, unlike anchor 5's enumerated actions; not 3 because the 'Use when' clause is fully explicit rather than missing or weakly implied.

4 / 5

Trigger Term Quality

"review a branch, pull request, or diff" plus "after implementation" and "pre-merge" are natural phrases a user would say, matching anchor 4 ('Good keyword coverage; a few natural terms missing'). Not 5 because common variations are absent ('PR', 'code review', 'changes', 'before merging'); not 3 because multiple natural trigger terms are present, not just domain jargon.

4 / 5

Distinctiveness Conflict Risk

"Fallow's ... pre-merge review" scopes the skill to a clear project niche with a distinct timing trigger, matching anchor 4 ('Mostly distinct; minor overlap risk with closely related skills'). Not 5 because "review a branch, pull request, or diff" would also fire a generic code-review skill; not 3 because the project-scoped framing is more specific than 'Works with document files'.

4 / 5

Total

15

/

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.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

fallow-rs/fallow

Path

.agents/skills/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.