Content
56%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.
Highly actionable content with concrete, executable API examples and clearly sequenced workflows, undermined by significant verbosity and duplication of the bundle's reference and script files. Refactoring SKILL.md into a lean overview with pointers would resolve most of the deficit.
Suggestions
Replace the nine per-endpoint code sections with a short endpoint table and defer examples to references/api_reference.md, keeping only one or two quick-start examples inline.
Delete the inlined rate-limiting, error-handling, and caching code (duplicated from scripts/query_clinpgx.py) and reference the script's functions instead.
Trim the "Key pharmacogenes", "Drug categories", and "Important Notes" list sections to the few entries that add non-obvious value, or move them to the reference file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~640-line body is noticeably verbose: nine full per-endpoint code walkthroughs, "Key pharmacogenes" and "Drug categories" list sections, and "Important Notes" prose duplicate material already present in references/api_reference.md and scripts/query_clinpgx.py — the rate-limiting, error-handling, and caching code is reproduced nearly verbatim from the script. | 2 / 5 |
Actionability | Guidance is mostly copy-paste-ready: concrete requests.get calls with real endpoints and parameters for every resource type, plus complete error-handling and caching helpers. Minor gaps remain — Workflow 4 step 3 calls an undefined calculate_phenotype_frequencies function and Workflow 2's filtering snippet assumes response keys that are never verified. | 4 / 5 |
Workflow Clarity | Five workflows present clearly numbered, concrete steps with per-step code, and rate-limit compliance plus retry/backoff guidance is provided. The workflows themselves don't checkpoint response validation (only the separate error-handling section covers it), leaving minor validation gaps; no destructive-op cap applies since these are read-only queries. | 4 / 5 |
Progressive Disclosure | The bundle structure is real and well-signaled (references/api_reference.md and scripts/query_clinpgx.py exist and each is introduced with when to consult it), but content that clearly belongs in those files — the full nine-endpoint API tour with example code — is inlined in SKILL.md rather than summarized with pointers. | 3 / 5 |
Total | 13 / 20 Passed |