Content
81%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, highly actionable skill body with executable commands for every subcommand, explicit error-recovery loops, and appropriate offloading of setup detail to a one-level-deep reference. Its weaknesses are redundancy — the Mindset section restates three points that the Anti-Patterns section already covers — and a broken link to requirements.txt.
Suggestions
Delete the 'Mindset' section (or reduce it to one line): all three of its points are already covered more usefully by the 'Anti-Patterns' WHY/BAD/GOOD entries, removing ~10 lines of duplication.
Add a requirements.txt file at the skill root (or fix the 'Dependencies' reference to point at the actual location, e.g., inside scripts/) so the broken reference link resolves.
Move the inline PubTator MCP server JSON block into references/setup-and-troubleshooting.md alongside the other configuration detail, keeping only the one-line 'prefer the MCP' guidance in the body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | No basic-concept padding, but the same three points are repeated across sections: MCP-first appears in 'When Not to Use', 'Recommended MCP Server', 'Mindset #2', 'Workflow step 1', and an anti-pattern; rate-limit handling appears in 'Mindset #1', 'Workflow step 4', and its own anti-pattern; open-access caution appears in 'Mindset #3' and an anti-pattern. The 'Mindset' section duplicates the Anti-Patterns section almost entirely and could be cut. This is 'mostly efficient but includes some unnecessary explanation or could be tightened' — not 4, where only minor trimming would be needed. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready commands cover every common case: all five subcommand invocations with real flags ('search --keywords "CRISPR gene editing" --results 10', 'metadata --pmid "33303479" --format json', etc.), a concrete 429 recovery command ('sleep 30 && ./scripts/pubmed_search.py search ...'), and a ready-to-use MCP server JSON config. Not below 5: specific examples cover the common cases with no gaps. | 5 / 5 |
Workflow Clarity | The five-step sequence is clear and includes explicit validation and feedback loops for the risky batch operation: step 4 handles 429s with a wait-30s-retry-once loop and an explicit prohibition on tight retries, 'ALWAYS add a short delay between batch calls' guards bulk fetching, and step 5 is a user-confirmation gate before triage. Not 4: checkpoints are explicit rather than implicit — error recovery (429), rate-limit prevention, and the human gate are all stated. | 5 / 5 |
Progressive Disclosure | Good structure: setup detail is appropriately split into references/setup-and-troubleshooting.md (one level deep, clearly signaled twice, and the file exists), and the References section enumerates all bundle files. Minor gaps: the 'Dependencies: requirements.txt' link is broken (no requirements.txt exists in the bundle), and the inline MCP server JSON config arguably belongs in the references file. Not 5: the broken reference link is a real navigation gap; not 3: content splitting and signaling are otherwise well done. | 4 / 5 |
Total | 17 / 20 Passed |