Content
42%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 body contains the raw material of a good skill — a parameter table, input format specifications, a real executable script, and a smoke-check command — but it is buried in roughly 200 lines of duplicated template boilerplate, self-referential filler, contradictory dependency declarations, and non-executable usage examples. A user or Claude could eventually operate the skill, but only after navigating substantial noise and resolving the conflicting dependency and usage information.
Suggestions
Cut the template sections (Risk Assessment, Security Checklist, Evaluation Criteria, Lifecycle Status, Output Requirements, Response Template, Inputs to Collect, Output Contract, Validation and Safety Rules) and the 'See `## X` above' filler — keep the parameter table, input formats, error handling, and usage examples, targeting well under 100 lines.
Resolve the contradictions: one dependency block (Python 3.8+ vs 3.10+, pandas/plotly 'unspecified' vs pinned versions vs 'No additional Python packages required'), and make 'Basic Usage' an actual executable command (e.g. `python scripts/main.py -i input.tsv -o krona_chart.html`) instead of an empty commented block; remove the nonsensical clinical-narrative `--input` example.
Link the real reference explicitly (e.g. '**Runtime checklist**: See [references/runtime_checklist.md](references/runtime_checklist.md)') and move the remaining governance content into it, so SKILL.md is a lean overview pointing one level deep.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 271-line body is heavily padded with template governance sections ('Risk Assessment', 'Security Checklist', 'Evaluation Criteria', 'Lifecycle Status', 'Output Requirements', 'Response Template', 'Inputs to Collect', 'Output Contract', 'Validation and Safety Rules') that restate discipline Claude already has, plus circular filler like 'See `## Usage` above for related details' and 'See `## Prerequisites` above' (including a self-referential pointer inside the Usage section itself) and contradictory dependency blocks (Python '3.10+' vs '3.8+', pandas 'unspecified' vs '>= 1.3.0', 'Prerequisites: No additional Python packages required' vs a plotly/pandas install command). Borderline toward anchor 1 ('heavily padded'), held at 2 because a core of genuinely useful, non-redundant material (parameter table, input format specs) survives amid the boilerplate. | 2 / 5 |
Actionability | There is concrete guidance — a full parameter table with defaults, a `py_compile` smoke check, `--help` invocation, input format examples for Kraken2/Bracken and custom TSV — but the executable examples are undermined: 'Basic Usage' is an empty ```text block containing only a comment, the 'Audit-Ready' example passes a clinical narrative string ('--input "Audit validation sample with explicit symptoms, history, assessment, and next-step plan."') to a script that parses TSV, and the example path 'cd "20260318/scientific-skills/Data Analytics/metagenomic-krona-chart"' is a hardcoded placeholder-style path. This matches anchor 3 ('some concrete guidance but incomplete ... missing key details') — better than 2 because real commands and format specs exist, short of 4 because the copy-paste usage examples are not executable as written. | 3 / 5 |
Workflow Clarity | A rough sequence exists ('Example run plan: 1. Confirm the user input... 4. Review the generated output'), a non-destructive smoke check is included (`python -m py_compile scripts/main.py`), and error/fallback handling is described ('If `scripts/main.py` fails, report the failure point... provide a manual fallback'). But the steps are generic process governance rather than concrete chart-generation instructions, they are duplicated across three overlapping sections (Workflow, Example run plan, Response Template), and there is no output validation checkpoint (e.g., verify the HTML file was created and renders). This matches anchor 3 ('sequence present but checkpoints missing or implicit'), not 4, which requires concrete per-step commands with most checkpoints present. | 3 / 5 |
Progressive Disclosure | The bundle does have appropriate structure — `scripts/main.py` is a real entry point and `references/runtime_checklist.md` is a genuine one-level-deep reference — but the body never names or links the actual reference file, saying only 'Reference material available in `references/`' and 'Reference guidance: `references/` contains supporting rules, prompts, or checklists'. Meanwhile large amounts of content that belong in reference files (the entire governance/template apparatus, the security and evaluation checklists) are inlined in SKILL.md. This fits anchor 3 ('references present but not clearly signaled; content that should be separate is inline'); not 2 because real structure and a working entry point exist, not 4 because the single reference is left undiscoverable by name. | 3 / 5 |
Total | 11 / 20 Passed |