Content
61%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 content is highly actionable with executable code throughout, but it is over-inlined — duplicating bundle scripts and omitting validation checkpoints in pipeline workflows — which hurts conciseness, workflow clarity, and progressive disclosure.
Suggestions
Replace the full inline implementations of capabilities that already exist in scripts/ (e.g. parse_vcf.py, calculate_tmb.py, nmf_metagenes.py, ddr_network.py) with brief summaries plus links to those files, keeping only a short Quick Start inline.
Add explicit validation checkpoints to the Typical Workflows — e.g. after Mutect2 filtering and SnpEff annotation, verify the output VCF is non-empty and tabix-indexed before parsing, with a fix-and-retry loop on failure.
Add a "Bundle scripts" or "References" section that lists and links each script in scripts/ so the file structure is discoverable from SKILL.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Prose is tight and avoids explaining basics Claude already knows, but ~530 lines fully inline seven capabilities plus three end-to-end workflows — content that exists as parallel bundle scripts is duplicated inline rather than referenced, which could be tightened. | 3 / 5 |
Actionability | Every capability is backed by concrete, copy-paste-ready Python with real function signatures, library calls, and output examples covering the common cases across VCF parsing, Mutect2, SVs, CNVkit, NMF, DDR networks, and TMB. | 5 / 5 |
Workflow Clarity | The three "Typical Workflows" give clear numbered sequences, but they drive subprocess-based pipeline/batch operations with no validation or verification checkpoints and no error-feedback loop, so per the rubric workflow clarity is capped at 3. | 3 / 5 |
Progressive Disclosure | The body is monolithic: bundle scripts exist in scripts/ (calculate_tmb.py, ddr_network.py, nmf_metagenes.py, parse_vcf.py) that parallel the inline implementations, but the body never references or links to them, so content that belongs in separate files is inlined with no navigation. | 2 / 5 |
Total | 13 / 20 Passed |