Content
67%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 dense, domain-expert body with genuinely valuable gotchas and strong executable script invocations, undermined by redundancy (the coding-denominator rule appears three times), a CRISPR primer of background knowledge, one broken reference (QUICK_START.md), and Python pattern examples whose functions are not in the bundle. Deduplicating and moving the tool/phase tables into the existing references would lift both conciseness and progressive disclosure.
Suggestions
State the coding-only denominator rule once (in Analysis Conventions) and reference it from the script table and CRITICAL section instead of repeating it three times; same for the 2-row Excel header guidance.
Fix or remove the 'QUICK_START.md' reference in Additional Resources — the file does not exist in the bundle.
Clarify the provenance of the Python pattern functions (answer_vaf_mutation_fraction, FilterCriteria, variant_analysis_pipeline, parse_vcf) — either note they are ToolUniverse tools or provide runnable equivalents in scripts/, since none are defined in the bundled scripts.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient domain-specific gotchas (ploidy defaults, 2-row VarSeq headers, coding denominators) that earn their tokens, but at ~500 lines the body is noticeably padded: the coding-only denominator rule is stated three times (PRIMARY SCRIPTS table, CRITICAL section, Analysis Conventions), multi-row Excel headers are explained twice, and the CRISPR section re-explains PAM/NGG basics Claude already knows. Fits 'mostly efficient but includes some unnecessary explanation or could be tightened'; not 2 because the bulk is genuinely non-obvious operational knowledge rather than generic filler. | 3 / 5 |
Actionability | The three primary scripts get exact copy-paste bash invocations with flags, workdir, and named output keys to grep ('SNP_COUNT_ALLELES', 'AVERAGE_PER_SAMPLE'), which is fully executable. However, the Python pattern examples call functions (answer_vaf_mutation_fraction, variant_analysis_pipeline, parse_vcf, FilterCriteria) that are not defined in any bundled script and whose provenance (ToolUniverse tools vs local code) is never made explicit — a minor gap. Matches 'Mostly executable guidance; concrete code or commands with minor gaps'; not 5 because those examples cannot be run as written. | 4 / 5 |
Workflow Clarity | Sequencing is clear and prioritized (RULE ZERO check-first → PRIMARY SCRIPTS → domain recipes → phase workflow) with some real checkpoints (the ~0.4 synonymous-fraction sanity check with a recompute loop, 'PLOIDY=<value>' emitted for confirmation, report-all-three-conventions sensitivity). Fits 'Clear sequence with most checkpoints present; minor validation gaps'; not 5 because there are three competing entry paths (phases, scripts-first, RULE ZERO) with no single explicit order among them, and the phase workflow itself lacks validation steps. | 4 / 5 |
Progressive Disclosure | Good structure: four real one-level-deep reference files (references/vcf_filtering.md, mutation_classification_guide.md, annotation_guide.md, sv_cnv_analysis.md — all verified present) clearly signaled inline and in a Reference Documentation section, plus six bundled scripts documented with a usage table. Minor gaps keep it from 5: 'QUICK_START.md' is referenced under Additional Resources but does not exist in the bundle, and a fair amount of reference-grade material (full tool tables, phase-by-phase detail) is inlined in an already-long SKILL.md. | 4 / 5 |
Total | 15 / 20 Passed |