CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-against-solid-principles

Single-pass SOLID review — dispatches one reviewer over the complete ruleset, emitting the fixed candidate schema.

56

Quality

70%

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 ./examples/solid-review-ab/single-high-intelligence-agent/.claude/skills/review-against-solid-principles/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

72%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.

A tight, well-structured orchestration skill with concrete paths and an unambiguous single-reviewer procedure. Its main weakness is the absence of any validation/verification step for the batch review output, which caps workflow clarity, plus minor redundancy in repeating the output path.

Suggestions

Add a verification step after the review: confirm ./findings/findings.md exists, is non-empty, parses against the fixed candidate schema, and covers every target file under ../corpus/cases/.

Deduplicate the output path: state it once in the Fixed paths list and drop its repetition in steps 1 and 3 (fold step 3 into step 1's write instruction).

Include the fixed candidate schema inline (or a pointer to it) so the reviewer's expected output format is unambiguous.

DimensionReasoningScore

Conciseness

The body is lean, assumes Claude's competence, and explains nothing it doesn't need to, but the output path './findings/findings.md' is stated three times (Fixed paths list, step 1, and step 3) and step 3 restates what step 1 already covers. This is anchor 4 (efficient, minor instances that could be trimmed) rather than 5, where every token earns its place.

4 / 5

Actionability

The guidance is concrete: exact ruleset path, an explicit target glob ('every *.py under ../corpus/cases/'), a named subagent type ('principles-reviewer'), a specified output path, and an explicit single-reviewer constraint. As an instruction-only skill the absence of code is not penalized; it sits at anchor 4 (mostly executable guidance, minor gaps) because it never shows how to invoke the subagent or what the 'fixed candidate schema' looks like, so it is not copy-paste ready (5).

4 / 5

Workflow Clarity

The sequence is clearly listed (dispatch one reviewer, do not partition/fan out, write findings), but the workflow is a batch operation over every corpus file with no verification checkpoint — no step confirms findings were written, that they match the fixed schema, or that all targets were covered. Per the judging guidelines this caps workflow clarity at 3, which takes precedence over the simple-skill exception; it is above level 2 because the sequence itself is well-defined, not gappy.

3 / 5

Progressive Disclosure

The skill is under 50 lines, has no bundle files (references/, scripts/, assets/ do not exist), and needs no external references — the ruleset and corpus live outside the skill by experimental design. Per the judging guidelines, a short skill with no need for external references scores 5 with well-organized sections, which this has (title, one-line summary, Fixed paths, numbered Procedure).

5 / 5

Total

16

/

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 is concise, third-person, and states a clear 'what', but it lacks any 'when' trigger guidance and its natural trigger vocabulary is thin. It reads more like an internal experiment label ('Single-pass', 'fixed candidate schema') than a user-facing skill description.

Suggestions

Add an explicit 'Use when...' clause, e.g. 'Use when reviewing code for SOLID principle violations or when the user asks for a design-principles review.'

Include natural trigger terms and synonyms users would actually say: 'SOLID principles', 'design principles', 'single responsibility', 'code review'.

State concretely what the review produces (findings of principle violations per file) rather than the internal shorthand 'emitting the fixed candidate schema'.

DimensionReasoningScore

Specificity

The description names the domain ('Single-pass SOLID review') and 1-2 concrete actions ('dispatches one reviewer over the complete ruleset, emitting the fixed candidate schema'), but coverage is not comprehensive — it never says what the review examines or what the findings contain. It matches anchor 3 (domain plus 1-2 concrete actions); it is not level 4-5 because the action list stops at two, and not level 2 because those actions are genuinely concrete rather than generic.

3 / 5

Completeness

The 'what' is clear (single-pass SOLID review dispatching one reviewer that emits the fixed candidate schema), but there is no 'Use when...' clause or any equivalent trigger guidance, which caps completeness at 3 per the judging guidelines. It is above level 2 because 'what' is explicit rather than vague, and below level 4 because 'when' is entirely absent rather than just imprecise.

3 / 5

Trigger Term Quality

'SOLID review' is a relevant keyword a user might naturally say, but common variations are missing ('SOLID principles', 'design principles', 'code review'), and 'ruleset'/'candidate schema' are internal jargon rather than user-facing trigger terms. This fits anchor 3 (some relevant keywords, missing common variations); level 4 would require a fuller set of natural phrases like 'PDF files, forms, document extraction'.

3 / 5

Distinctiveness Conflict Risk

'Single-pass SOLID review' carves out a clear niche with minimal conflict risk against unrelated skills, though the generic word 'review' leaves minor overlap risk with general code-review skills. This is anchor 4 (mostly distinct, minor overlap with closely related skills) — not 5 because the description does not fully disambiguate from a plain code review, and not 3 because SOLID is a specific, well-bounded trigger.

4 / 5

Total

13

/

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

Jamie-BitFlight/claude_skills

Path

examples/solid-review-ab/single-high-intelligence-agent/.claude/skills/review-against-solid-principles/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.