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 well-structured, example-rich command reference with a clear retrieval cascade and result-selection guidance. Its main defects are the broken code fences in the patent-search section, an orphaned references/citation.bib bundle file with no navigation to it, and some duplicated content (Overview bullets vs. section headers, repeated API-key guidance).
Suggestions
Fix the code fences in section 9 (Patent Sequence Search) — the `bash uv run scripts/ncbi_fetch.py patent-search ...` examples lost their triple-backtick blocks and render as broken inline text; restore them to fenced bash blocks like the other nine sections.
Link or remove the orphaned bundle file references/citation.bib — the Reference section should cite it (e.g. for the E-utilities documentation) so every bundle file is discoverable, or the file should be dropped.
Trim duplication: the 10-item Overview bullet list repeats the per-section descriptions, and NCBI_API_KEY guidance appears in both Prerequisites and Core Rules — consolidate each to one location to reduce token cost.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and example-driven — each subcommand is a one-line description plus a bash snippet, with no explanations of biology or libraries Claude already knows. Minor trimmable redundancy exists: the Overview's 10-item bullet list repeats the per-section descriptions, and NCBI_API_KEY guidance appears in both Prerequisites ("Raises the NCBI rate limit from 3 to 10 requests/second") and Core Rules ("the query speed limits are automatically increased significantly"). Not 5: those duplications could be cut; not 3: there is no padded or unnecessary explanation beyond these minor instances. | 4 / 5 |
Actionability | Nearly every subcommand has a copy-paste-ready command with realistic arguments (e.g. `uv run scripts/ncbi_fetch.py gene-protein WRR4B --organism "Arabidopsis thaliana"`), and output handling is specified ("All subcommands write structured JSON output. Use `--output FILE`..."). The gap: the Patent Sequence Search examples lost their code fences ("`bash uv run scripts/ncbi_fetch.py patent-search --patent-number US10123456...`"), rendering section 9's commands as broken inline text. Not 5: one section's examples are malformed; not 3: guidance is genuinely executable throughout. | 4 / 5 |
Workflow Clarity | The "Standard Sequence Retrieval Cascade" gives an explicit 7-step priority order, and result interpretation is addressed ("When multiple results are returned, select by: Closest match to expected length... Source priority (RefSeq > GenPept > patent)"), plus an error-recovery hint ("the tool will report `is_genomic: true` so you can fall back to a homology-based approach"). Not 5: there is no explicit validate-then-proceed checkpoint (e.g. verifying fetched length against expectation is only implied by --target-length); not 3: the sequence is explicit with most checkpoints present, and operations are read-only so no destructive-batch cap applies. | 4 / 5 |
Progressive Disclosure | scripts/ncbi_fetch.py is properly referenced and exists, and section structure is clear, but references/citation.bib is present in the bundle yet never mentioned or linked anywhere in the body — an orphaned, unnavigable reference. Additionally, the full 10-subcommand command reference (~150 lines) is inlined in SKILL.md, which is heavier than an overview pointing to detailed materials. Not 4: an unlisted bundle file is more than a minor organization gap; not 2: the structure and headings are good and paths that are referenced resolve correctly. | 3 / 5 |
Total | 15 / 20 Passed |