Content
71%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 strong, information-dense orchestration skill: exact tool parameters, explicit sequencing, evidence tiers, and a completeness checklist make it highly actionable. Its weaknesses are the absence of any progressive disclosure (the full API reference is inlined in a ~280-line monolith with no bundle files), minor token padding (basic genetics, chatty asides), one incoherent self-referential fallback bullet, and no runnable Python example despite the compute-first directive.
Suggestions
Move the per-tool parameter reference (Phases 0-8 tool blocks) into a references/ file (e.g., references/tool-reference.md), keeping SKILL.md as the workflow overview with clearly signaled one-level-deep pointers per phase.
Fix the self-referential fallback bullet 'Disease lookup: try `Orphanet_search_diseases` if `Orphanet_search_diseases` fails' — presumably it should name a different tool such as `Orphadata_search_by_name`.
Trim the basic inheritance-mode explanations and the deep-intronic digression to one-line pointers, and add one small runnable Python example (e.g., a pandas merge of Orphanet genes with GenCC classifications) to back the 'COMPUTE, DON'T DESCRIBE' directive.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-redundant, database-specific guidance ('The parameter is `name` (NOT `query`)', 'Check `len(studies) > 0` rather than `total_count`'), but includes trimmable over-explanation: basic inheritance-mode genetics Claude already knows ('Autosomal recessive: need TWO hits (homozygous or compound heterozygous)') and a chatty multi-clause aside on deep intronic variants ("but 'usually' is doing real work"). This matches 'efficient; minor instances of over-explanation that could be trimmed' rather than 5, whose every-token-earns-its-place bar the padded asides miss, and rather than 3, since the over-explanation is minor relative to the parameter-level detail Claude cannot know. | 4 / 5 |
Actionability | Every tool has exact required parameters and types ('Orphanet_get_disease: `orpha_code` (string REQUIRED, e.g., "558")') and the example workflows give concrete calls ('Orphanet_search_diseases(name="Marfan syndrome") -> ORPHAcode 558'), but the workflows use `->` shorthand rather than copy-paste code and no runnable Python analysis example appears despite the 'COMPUTE, DON'T DESCRIBE' directive. This is 'mostly executable guidance with minor gaps' — not 5 (not fully copy-paste ready), not 3 (the tool calls are concrete and specific, not pseudocode). | 4 / 5 |
Workflow Clarity | Phases 0-9 are explicitly sequenced with an ordering rule ('phenotype -> disease -> gene -> variant, not the reverse'), evidence-grading tiers, a completeness checklist, and a fallback/error-recovery section. It falls short of 5 because one fallback bullet is self-referential and incoherent ('Disease lookup: try `Orphanet_search_diseases` if `Orphanet_search_diseases` fails') and per-phase exit criteria are sometimes implicit; it is above 3 because the checklist, tiering, and fallbacks provide real validation checkpoints. The destructive/batch cap does not apply — this is read-only research orchestration. | 4 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/ directories), so the entire ~280-line per-tool API reference is inlined in SKILL.md and loads on every invocation. Section headers are clear (one section per phase), which keeps it above 2 ('minimal structure'), but bulk parameter reference that clearly belongs in a separate reference file is inline with no one-level-deep pointers, matching 'some structure... content that should be separate is inline' rather than 4. | 3 / 5 |
Total | 15 / 20 Passed |