Use when working with comprehensive review full review
33
17%
Does it follow best practices?
Impact
Pending
No eval scenarios have been run
Passed
No known issues
Optimize this skill with Tessl
npx tessl skill review --optimize ./skills/comprehensive-review-full-review/SKILL.mdQuality
Discovery
0%Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.
This is an extremely poor skill description that fails on every dimension. It contains no concrete actions, no meaningful trigger terms, and is essentially incoherent with the redundant phrase 'comprehensive review full review.' It would be indistinguishable from any other review-related skill.
Suggestions
Specify what is being reviewed and what concrete actions the skill performs (e.g., 'Reviews pull requests by analyzing code changes, checking for bugs, and suggesting improvements').
Add natural trigger terms that users would actually say, including domain-specific keywords (e.g., 'code review, PR review, pull request, diff analysis, .patch files').
Include a clear 'Use when...' clause with explicit trigger conditions (e.g., 'Use when the user asks for a code review, wants feedback on a pull request, or needs help analyzing code changes').
| Dimension | Reasoning | Score |
|---|---|---|
Specificity | The description contains no concrete actions whatsoever. 'Comprehensive review full review' is vague and abstract with no indication of what is being reviewed or what actions are performed. | 1 / 3 |
Completeness | The 'what' is entirely missing—there is no description of what the skill does. While it starts with 'Use when,' the trigger condition is nonsensical and redundant ('comprehensive review full review'), providing no meaningful guidance on when to use it. | 1 / 3 |
Trigger Term Quality | The only keyword is 'review,' which is extremely generic and could apply to countless domains. There are no natural, specific terms a user would say to trigger this skill. | 1 / 3 |
Distinctiveness Conflict Risk | The term 'review' is extremely generic and would conflict with any skill involving code review, document review, PR review, performance review, etc. There is nothing distinctive about this description. | 1 / 3 |
Total | 4 / 12 Passed |
Implementation
35%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill is excessively verbose and reads more like a product requirements document than a concise, actionable skill for Claude. While the phased workflow structure is logical and the prioritization framework in the consolidated report is useful, the content suffers from significant bloat, undefined variable references, lack of executable examples, and missing validation checkpoints between phases. The skill would be dramatically improved by cutting 60-70% of the content and focusing on concrete, executable guidance.
Suggestions
Reduce content by at least 60% — remove explanatory text Claude already knows (e.g., what OWASP Top 10 is, what cyclomatic complexity means) and trim the verbose subagent prompt templates to essential parameters only.
Define or explain how context variables like {phase1_architecture_context} and {all_previous_contexts} are constructed and passed between phases, or provide concrete code/commands for doing so.
Add explicit validation checkpoints between phases (e.g., 'Verify Phase 1 outputs contain quality metrics before proceeding to Phase 2; if missing, re-run with expanded scope').
Split the detailed phase descriptions into separate referenced files (e.g., phases/security-review.md) and keep SKILL.md as a concise overview with navigation links.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Extremely verbose at ~150+ lines with extensive explanations of review concepts Claude already understands. The prompt templates for subagents are bloated with redundant descriptions. Phrases like 'Extended thinking: This workflow performs an exhaustive code review by orchestrating multiple specialized agents...' add no actionable value. Much of the content reads like a product specification rather than a concise skill. | 1 / 3 |
Actionability | Provides structured phases with specific tool invocations (Task tool with subagent types) and prompt templates, which gives some concrete guidance. However, there is no executable code, no real commands, and the prompts reference undefined variables like {phase1_architecture_context} and {all_previous_contexts} without explaining how to obtain or construct them. The subagent types referenced (e.g., 'code-reviewer', 'architect-review') are not validated as real tool configurations. | 2 / 3 |
Workflow Clarity | The four-phase sequential structure is clearly laid out with dependencies between phases noted. However, there are no explicit validation checkpoints or feedback loops — no step says 'if this fails, do X.' For a complex multi-phase orchestration involving security scanning and destructive review operations, the absence of error handling and verification steps between phases is a significant gap. | 2 / 3 |
Progressive Disclosure | There is one reference to an external file ('resources/implementation-playbook.md') which is good, but the vast majority of detailed content is inline in a monolithic document. The configuration options, four phases, report template, and success criteria could be split into separate referenced files. The document would benefit greatly from being an overview that points to detailed phase descriptions. | 2 / 3 |
Total | 7 / 12 Passed |
Validation
90%Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.
Validation — 10 / 11 Passed
Validation for skill structure
| Criteria | Description | Result |
|---|---|---|
frontmatter_unknown_keys | Unknown frontmatter key(s) found; consider removing or moving to metadata | Warning |
Total | 10 / 11 Passed | |
93c57b2
Table of Contents
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.