Content
82%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, expert-level operational skill: concrete commands, explicit decision rules, and honest scoping ('Treat this as a focused samtools/bcftools run envelope, not as a substitute for full cohort... workflows'). Its main gaps are the absence of error-recovery feedback loops when QC or readiness checks fail, and unspecified prerequisites (sample-sheet format, external script availability).
Suggestions
Add an explicit feedback loop after preflight/QC: what to do when --require-resource-plan readiness fails, contamination is detected, or execution is blocked by missing tools (e.g., re-run readiness, report the packaged validation state, or fall back to the compact runner).
Specify or link the dna_samples.tsv sample-sheet format (required columns, singleton/cohort/trio representation) so the commands are fully executable without guesswork.
Clarify where the referenced scripts live (bundle path vs. project path) and whether they must be verified to exist before invoking, closing the actionability and navigation gaps.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and operational throughout — it never explains concepts Claude already knows (no definition of BQSR, joint genotyping, or Ti/Tv), and every section (Essential Inputs, Route, Decision Points, Outputs) carries actionable content. Not 4: there is no over-explanatory sentence to trim; each token earns its place. | 5 / 5 |
Actionability | Three complete, copy-paste-ready commands with concrete flags ('ngs_preflight.py --pipeline dna_germline_variants --emit-install-plan', 'run_dna_germline_variants.py ... --known-sites dbsnp.vcf.gz --emit-gvcf --joint-call --execute') plus concrete decision rules. Not 5: the referenced scripts live outside the skill bundle, the sample-sheet (dna_samples.tsv) format is never specified, and no worked example with real inputs/outputs is given, leaving minor execution gaps. | 4 / 5 |
Workflow Clarity | A clear sequence exists — confirm Essential Inputs, run the preflight, route to sarek or the local runners, then apply Decision Points and produce Outputs — with validation checkpoints (preflight --emit-install-plan, 'Check sample identity, sex concordance, contamination, coverage...', resource_readiness.md). Not 5: there is no explicit feedback loop for what to do when QC/contamination checks fail or execution is blocked; not 3: validation steps are present for this batch operation, so the batch-operation cap does not apply. | 4 / 5 |
Progressive Disclosure | Sections are clean and well-organized with nothing bulk-inlined that belongs in a separate file, and no bundle files exist so no nested references. Not 5: the body is ~76 lines (above the under-50 simple-skill exemption) and the referenced execution scripts (plugins/ngs-analysis/scripts/*.py) are neither part of a bundle nor presented as navigable links, a minor organization gap matching the 'most content appropriately placed' anchor at 4. | 4 / 5 |
Total | 17 / 20 Passed |