Content
65%Weight 40%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A well-organized, highly actionable skill body with executable commands and complete output schemas, weakened by redundancy (API-key instructions repeated), a self-contradictory retmax rule, and an unreferenced bundle file. The core guidance is strong; tightening and reconciling the retmax statements would lift the lower dimensions.
Suggestions
Reconcile the retmax guidance: the script defaults to 0 (fetch all), so remove or correct the 'defaults to --retmax 200 / MUST set higher' text in Core Rules to avoid contradicting the script.
Consolidate the NCBI_API_KEY / credentials instructions into one place (e.g., Prerequisites) and reference it from Core Rules and the dedicated section rather than repeating the full paragraph.
Link references/citation.bib from the body (e.g., under a Citations section) so the existing bundle file is discoverable rather than orphaned.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient — command examples, argument lists, and output schemas are all directly useful with no padding of concepts Claude already knows — but it repeats the NCBI_API_KEY/credentials instructions in three places (Prerequisites, Core Rules, and a dedicated section) and carries a self-contradictory retmax statement. This matches 'mostly efficient but includes some unnecessary explanation or could be tightened'; not 3 due to the repetition and contradiction, not 1 because there is no concept-explaining fluff. | 2 / 3 |
Actionability | Every command is given as a fully executable `uv run scripts/clinvar_api.py ...` invocation with complete argument lists, concrete output JSON schemas (field names and types), and per-command examples that match the actual script. This matches the 'fully executable code/commands; copy-paste ready' anchor and is not 2 (no pseudocode or missing key details). | 3 / 3 |
Workflow Clarity | Workflows (Count-First, search→evidence, triangulation) are clearly sequenced and include a verification checkpoint ('verify total_count and fetched_count match'), but the 'Core Rules' retmax guidance ('defaults to --retmax 200', 'MUST set higher') contradicts the script's actual default (0 = fetch all), creating a real ambiguity for batch 'List all' operations, and the mandatory coordinate rule lacks a complete verify loop for the dbSNP fallback. This fits 'steps listed but validation gaps'; not 3 because of the contradiction, not 1 because sequences and some checkpoints are present. | 2 / 3 |
Progressive Disclosure | Content is organized into clear sections and the one script reference (scripts/clinvar_api.py) is real and one level deep, but the four command references are inlined as a large block that could live in a separate file, and references/citation.bib exists in the bundle yet is never linked or signaled from the body. This matches 'some structure; references present but not clearly signaled; content that should be separate is inline'; not 3 (unreferenced bundle file, large inline reference), not 1 (no multi-level nesting). | 2 / 3 |
Total | 9 / 12 Passed |