Content
36%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The instructional core of this skill is a compact, sensible role prompt with a good output template, but it is buried under a ~125-line inlined YAML agent-config blob that wastes most of the body's tokens, and it never defines the analysis workflow or how to produce the scores its template demands. Removing the config blob and adding a short, sequenced analysis procedure would transform the file. The strongest immediate fix is deleting or externalizing the YAML block.
Suggestions
Delete the ~125-line embedded YAML config block (triggers, capabilities, constraints, hooks, optimization settings) from the body — it consumes the majority of the skill's tokens without instructing Claude on anything.
Add a short sequenced workflow (e.g., 1. glob allowed file types within allowed paths, 2. review each file against the criteria, 3. assign severity and score, 4. emit the report in the given template) so the output template is actually derivable.
Specify how to compute the report's quantitative fields ('Overall Quality Score: X/10', 'Technical Debt Estimate: X hours') so the guidance is executable rather than aspirational.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Roughly 125 of the ~180 body lines are an embedded YAML agent-config blob (triggers, capabilities, constraints, hooks, memory limits, emoji usage) that instructs Claude on nothing — heavy padding that crowds the context window. The trailing markdown section is leaner, but it also re-lists code smells (long methods, duplicate code, god objects) that Claude already knows, fitting anchor 2 (noticeably verbose, several unnecessary/padded sections). | 2 / 5 |
Actionability | There is some concrete guidance — code-smell thresholds (">50 lines", ">500 lines"), a five-criteria checklist, and a fully specified markdown output-report template — but the analysis process itself is never specified: how to select files, how to compute the 'Overall Quality Score: X/10', or how the 'Technical Debt Estimate: X hours' is derived. This matches anchor 3 (some concrete guidance but incomplete, missing key details). | 3 / 5 |
Workflow Clarity | No sequenced steps exist anywhere in the body — only lists of responsibilities and criteria; the flow from analysis to report is merely implied by document order. The skill also declares batch operation (max_file_operations: 100, batch_size: 20) with no validation or verification checkpoint, which caps workflow_clarity at 3; the absence of any real sequence pulls it to anchor 2 ('rough sequence present but many gaps; validation absent'). | 2 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent) and the body references no external files, so there is no nesting problem. The markdown section is reasonably organized with clear headers (Key responsibilities, Analysis criteria, Code smell detection, Review output format), but the 125-line YAML config blob is inlined content that clearly belongs in a separate config file, fitting anchor 3 (some structure but content that should be separate is inline). | 3 / 5 |
Total | 10 / 20 Passed |