Content
78%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 highly actionable and well-sequenced: a 7-phase pipeline with concrete tool calls, return-shape documentation, gotchas, fallback strategies, and three worked example workflows covering the main entry points. Its main weaknesses are the fully inlined API catalog in a monolithic single file (no progressive disclosure into reference files) and minor over-explanation of pharmacology concepts Claude already knows.
Suggestions
Split the per-tool parameter/return-shape documentation (Phases 0-5 tool entries) into a references/ file (e.g., references/tool-catalog.md), keeping SKILL.md as a concise workflow overview with per-phase pointers — this would move progressive_disclosure from inline monolith to a well-signaled one-level-deep structure.
Trim the 'Metabolizer Status Reasoning' active-vs-prodrug explanation and the PharmVar paragraph to the non-obvious operational facts (e.g., which tool requires which ID format, the PHARMVAR_API_KEY requirement), cutting restatements of standard pharmacology Claude already knows.
Add explicit verification checkpoints between phases (e.g., after Phase 2, confirm the retrieved guideline's cpiclevel is A/B before proceeding to dosing interpretation) to close the minor workflow_clarity validation gaps.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-obvious operational knowledge (PostgREST auto-normalization, clinpgxid vs pharmgkbid, warfarin's 0-row exception, limit=1000 for FDA queries) and largely avoids explaining general concepts. However, the 'Metabolizer Status Reasoning' active-vs-prodrug explanation and the lengthy PharmVar paragraph restate pharmacology Claude largely knows, and phrases like 'Get it right before interpreting any metabolizer phenotype' are padding — minor over-explanation that could be trimmed, matching the 4 anchor. | 4 / 5 |
Actionability | Every tool is documented with required parameters, return shapes, and concrete values, and the Example Workflows section gives literal copy-paste invocations covering the three common entry points: drug-first ('CPIC_get_recommendations(drug="codeine", limit=50)' → filter phenotypes), gene-first ('CPIC_search_gene_drug_pairs(gene_symbol="CYP2C19", cpiclevel="A")'), and variant-first ('PharmGKB_search_variants(query="rs1799853")'). Specific examples cover the common cases, matching the 5 anchor. | 5 / 5 |
Workflow Clarity | A clear 7-phase pipeline with an ASCII overview diagram, per-phase Gotchas, and an explicit Fallback Strategies section providing error-recovery loops (e.g., 'drug auto-resolve fails → CPIC_get_drug_info + manual guideline_id extraction'). This is a read-only retrieval skill so the destructive/batch validation cap does not apply, but explicit verification checkpoints between phases are absent — a clear sequence with most checkpoints present and minor validation gaps, matching the 4 anchor rather than 5. | 4 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are all absent), so the full tool-by-tool API catalog — per-tool parameter schemas, return shapes, and gotchas for ~30 tools across ~280 lines — is inlined in SKILL.md. Section structure itself is good, but content that clearly belongs in a separate reference file is inline with no external split at all, matching the 3 anchor ('some structure but... content that should be separate is inline') rather than the 4 anchor, which requires most content to be appropriately placed across files. | 3 / 5 |
Total | 16 / 20 Passed |