Content
42%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 body is dominated by generic packaging boilerplate at the expense of domain instruction, and its most important actionable content is wrong: the documented CLI (--variant/--vcf/--gene/--scores/--output), usage examples, and promised outputs (report.json, ACMG interpretation, confidence assessment) do not match the actual script, which only accepts --revel/--cadd/--polyphen/--demo and prints a composite score to stdout. The workflow and structure are present but templated, with broken internal cross-references.
Suggestions
Rewrite the Usage and Parameters sections to match the actual script interface (--revel, --cadd, --polyphen, --demo) or extend scripts/main.py to support the documented --variant/--vcf/--gene/--scores flags, and remove claims of outputs the script does not produce (ACMG interpretation, confidence assessment, report.json).
Cut the generic template sections (Security Checklist, Risk Assessment, Lifecycle Status, Evaluation Criteria, Response Template, Output Requirements, Implementation Details) that contain no domain-specific content, reducing the file to the workflow, real CLI, and score-integration details.
Remove or fix the broken "See `## X` above" cross-references in Dependencies, Example Usage, and Implementation Details, which point to sections that appear later in the file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~190-line body is heavily padded with generic template sections ("Security Checklist", "Risk Assessment", "Lifecycle Status", "Evaluation Criteria", "Response Template", "Output Requirements") that add no domain knowledge, plus filler like "See `## Prerequisites` above for related details". This matches "Noticeably verbose; several unnecessary explanations or padded sections"; not 1 because it never explains concepts Claude already knows and does contain real executable commands. | 2 / 5 |
Actionability | Some concrete guidance works ("python -m py_compile scripts/main.py", "python scripts/main.py --help", the run plan), but the primary Usage examples ("--variant \"chr17:43094692:G:A\" --gene \"BRCA1\"", "--vcf variants.vcf --output report.json") and the Parameters section document an interface that does not exist in scripts/main.py (actual flags are --revel, --cadd, --polyphen, --demo), so the key examples are not executable. Fits "Some concrete guidance but incomplete... missing key details"; not 2 because several genuinely executable verification commands are present. | 3 / 5 |
Workflow Clarity | The Workflow section lists a sequenced 5-step process with an early scope-validation checkpoint and a fallback path, but the steps are generic template language, and validation of actual execution results is only implicit ("Review the generated output"). The documented VCF batch processing has no output verification step, so the batch-operation cap of 3 applies. Not 4 because checkpoints concern request scope rather than domain execution validation. | 3 / 5 |
Progressive Disclosure | The bundle is appropriately shallow (SKILL.md plus a real, clearly-signaled "scripts/main.py" reference), but the internal navigation is broken: "See `## Prerequisites` above", "See `## Usage` above", and "See `## Workflow` above" appear in sections that precede the referenced sections, and substantial generic template content is inlined rather than split out. Fits "Some structure but could be better organized; references present but not clearly signaled"; not 4 because the misplaced cross-references actively confuse navigation. | 3 / 5 |
Total | 11 / 20 Passed |