Content
57%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.
A highly actionable, well-organized body of executable cancer-genomics code with useful workflows and troubleshooting, but it inlines complete implementations that already exist as bundled scripts without ever referencing them, and multi-tool pipelines lack explicit validation checkpoints. Pointing capability sections at the existing scripts and adding verify steps would address the weakest dimensions.
Suggestions
Reference the existing bundle scripts from each capability section (e.g. 'Full CLI implementation: scripts/parse_vcf.py') and inline only the minimal pattern, cutting the duplicated ~400 lines of implementation code from SKILL.md.
Add explicit validation checkpoints to the workflows (e.g. verify PASS variant count is non-zero after filtering, confirm all four CNVkit output files exist before segmentation) with a fix-and-retry loop like the Troubleshooting section.
Fill the placeholder gaps in Workflow 4 (define PATHWAY_GENES/DRIVER_GENE sourcing, GROUP column) and add the missing antitarget-coverage step to the CNVkit pipeline so examples run as written.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient and code-dense with little concept padding, but the Overview paragraph restates the description and roughly 400 lines of implementation (parse_vcf, NMF, DDR, TMB) duplicate code that already exists as bundle scripts, so it could be tightened substantially. Not 4 because the duplication is more than minor over-explanation. | 3 / 5 |
Actionability | Nearly all guidance is concrete, executable Python (cyvcf2 parsing, GATK/CNVkit subprocess wrappers, NMF, Fisher's exact cohort analysis) with real parameters and outputs. Minor gaps keep it below 5: Workflow 4 uses undefined placeholders (PATHWAY_GENES, DRIVER_GENE, clin.GROUP) and the CNVkit pipeline consumes an antitarget coverage file it never creates. | 4 / 5 |
Workflow Clarity | Multi-step workflows are clearly sequenced (call → filter → annotate → parse; coverage → reference → fix → segment), and subprocess calls fail fast via check=True, but these batch multi-tool pipelines have no explicit verify checkpoints (e.g. confirming non-empty outputs, sample counts, or PASS variant tallies before downstream analysis), matching the 'steps listed but validation implicit' anchor. | 3 / 5 |
Progressive Disclosure | Section structure is well organized (Overview, When to Use, Quick Start, seven capabilities, workflows, troubleshooting), but the four ready-made scripts in scripts/ (parse_vcf.py, nmf_metagenes.py, ddr_network.py, calculate_tmb.py) are never referenced from the body — their full implementations are inlined instead, so content that should live one level deep is in SKILL.md. Not 2 because the section organization is solid, not 4 because bundle navigation is entirely absent. | 3 / 5 |
Total | 13 / 20 Passed |