CtrlK
BlogDocsLog inGet started
Tessl Logo

comprehensive-review-full-review

Use when working with comprehensive review full review

37

Quality

35%

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 ./skills/comprehensive-review-full-review/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

53%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 presents a well-sequenced, genuinely detailed multi-phase review orchestration with concrete per-agent prompts, but it is diluted by template boilerplate and an unexplained placeholder mechanism, and it lacks any validation or error-recovery checkpoints between phases. Its single external reference points to a file that is missing from the bundle. Tightening the filler and either shipping or removing the playbook reference would materially raise quality.

Suggestions

Remove or fix the dangling reference: either ship `resources/implementation-playbook.md` in the bundle or delete the "If detailed examples are required, open..." line, since the file does not exist.

Define the placeholder contract — explain how {phase1_architecture_context} and {all_previous_contexts} are captured from prior agent outputs and injected into later prompts, or replace them with explicit instructions (e.g., "append the Phase 1B architecture findings summary here").

Add validation checkpoints between phases: instruct Claude to check that each agent returned a non-empty, expected-shaped result before proceeding, and to stop and ask the user if a phase fails or findings conflict.

Cut the "[Extended thinking: ...]" meta-paragraph and the generic Use/Do-not-use/Limitations boilerplate, which restate information already in the description and phase structure.

DimensionReasoningScore

Conciseness

The core phase prompts are dense and instructional, but the body carries noticeable padding Claude does not need: the bracketed "[Extended thinking: ...]" meta-paragraph that narrates the workflow's own design, generic template bullets ("Clarify goals, constraints, and required inputs"), a boilerplate "Do not use this skill when" section, and a 10-item Success Criteria list that largely restates the phase outputs. Not 4 because several sections (Extended thinking block, Use/Do-not-use boilerplate) could be cut entirely with no loss of instruction.

3 / 5

Actionability

The guidance is mostly executable: each phase gives a named subagent type, a copy-pasteable review prompt, expected output, and dependency context (e.g., "Use Task tool with subagent_type=\"security-auditor\"" with a full OWASP/Snyk/GitLeaks prompt), and the flags (--security-focus, --framework [name]) are concrete. Not 5 because the interpolated placeholders ({phase1_architecture_context}, {phase2_security_context}, {all_previous_contexts}) are never defined — how to capture and pass prior-phase output is left implicit — and the subagent types are asserted without fallback if unavailable. Not 3 because the prompts are real, specific, and directly usable rather than high-level hints.

4 / 5

Workflow Clarity

The four phases are clearly sequenced with explicit inter-phase dependencies ("incorporating Phase 1 findings", per-step Context lines), but there are no validation checkpoints: no instruction on how to verify a phase's output before proceeding, how to handle a failed or empty agent result, or when to stop and escalate. The anchor-3 fit ("steps listed but validation gaps; checkpoints missing or implicit") is exact — not 4 because every verification step is absent, and not 2 because the sequence and data flow between phases are well defined.

3 / 5

Progressive Disclosure

The bundle contains no references/, scripts/, or assets/ directories, yet the body instructs "open `resources/implementation-playbook.md`" — a dangling reference to a file that does not exist, so the skill's only navigation path fails. The section structure itself is reasonable, but ~160 lines of detailed per-agent prompts are inlined monolithically and the sole external reference is broken and buried in a bullet list, matching anchor 2 ("references are buried"). Not 3 because a present-but-unclear reference is recoverable, whereas a nonexistent one leaves the reader with nowhere to go.

2 / 5

Total

12

/

20

Passed

Description

16%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 a placeholder-quality tautology that restates the skill name instead of describing its capabilities. It provides a circular trigger, no statement of what the skill does, and no terms that would reliably route a real user request to it. The frontmatter squanders the most important routing surface of the skill.

Suggestions

State the concrete capability first in third person, e.g., "Orchestrates a multi-phase code review using specialized subagents covering code quality, architecture, security, performance, testing, documentation, and CI/CD compliance, and consolidates findings into a prioritized P0–P3 report."

Add explicit trigger phrases users would naturally say: "Use when the user asks for a comprehensive or full code review, an end-to-end review of a change, or a multi-dimensional review covering security, performance, and test coverage."

Drop the duplicated garbled phrase "comprehensive review full review" entirely — it is neither a capability description nor a natural trigger term.

DimensionReasoningScore

Specificity

The description is entirely vague: "Use when working with comprehensive review full review" names no concrete action or capability whatsoever — it only restates the skill's garbled name as a trigger. Not above 2 because score 2 requires at least minimal named actions (e.g., "Processes PDF files"), and none are present.

1 / 5

Completeness

Only a "when" clause is present ("Use when working with...") and it is circular; the "what" — what the skill actually does (orchestrating multi-phase code review with subagents) — is entirely absent, matching anchor 2 ("only 'when' is present without 'what'"). Not 3 because the "when" itself is too vague to count as a clear trigger, and not 1 because an explicit 'Use when' clause does exist.

2 / 5

Trigger Term Quality

"comprehensive review" and "full review" are one or two generic keywords; the duplicated, malformed phrase "comprehensive review full review" reads as boilerplate rather than natural user language, and no synonyms or variations (e.g., "code review", "audit", "review my PR") appear. Not 3 because even the anchor-3 example names a concrete object users would say ("Works with PDF files"), whereas this phrase would rarely be spoken verbatim.

2 / 5

Distinctiveness Conflict Risk

The phrase is entirely generic and would collide with virtually any review-related skill (code review, PR review, security review, document review); nothing distinguishes this multi-phase agent-orchestration skill. Not 2 because even anchor 2 requires a recognizable domain scope, and the description cannot even communicate the code-review domain unambiguously.

1 / 5

Total

6

/

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

frontmatter_unknown_keys

Unknown frontmatter key(s) found; consider removing or moving to metadata

Warning

Total

15

/

16

Passed

Repository
sickn33/agentic-awesome-skills
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.