Content
65%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is highly actionable with executable code for every capability, but it is a monolithic ~530-line document that ignores its own scripts/ bundle and lacks validation feedback loops in its multi-step workflows.
Suggestions
Replace inlined code with one-level-deep references to the existing scripts/ files (e.g. 'See scripts/parse_vcf.py') to enforce progressive disclosure and avoid duplication.
Add explicit validate-then-retry checkpoints to each Typical Workflow (e.g. re-run FilterMutectCalls if FilterRec records remain, or assert PASS-only counts before annotation).
Trim or move the duplicated Quick Start / Core Capabilities code now that scripts/ holds the canonical implementations.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~530-line body is mostly efficient executable code, but it duplicates logic that already exists in the scripts/ bundle and could be tightened by pointing to those files rather than inlining everything. | 2 / 3 |
Actionability | Provides complete, executable, copy-paste-ready code throughout — e.g. parse_vcf, run_mutect2, cnvkit_pipeline, extract_metagenes, calculate_tmb — with concrete function signatures and real library calls. | 3 / 3 |
Workflow Clarity | The 'Typical Workflows' are numbered and sequenced, but batch/bioinformatics operations lack explicit validation checkpoints or fix-and-retry feedback loops, so per the rubric workflow clarity is capped at 2. | 2 / 3 |
Progressive Disclosure | Sections are organized, but code that should live in the provided scripts/ bundle (parse_vcf.py, calculate_tmb.py, ddr_network.py, nmf_metagenes.py) is inlined instead and those bundle files are never referenced from the body. | 2 / 3 |
Total | 9 / 12 Passed |