Content
77%Reviews 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 with clear sequenced workflows, validation checkpoints, and feedback loops, but it is verbose with repeated guidance and keeps detailed reference material inline rather than splitting it into separate progressive-disclosure files.
Suggestions
Consolidate the repeated SI-gate, API-fallback, and no-credentials-in-chat rules into single canonical sections and reference them instead of restating.
Move the status-code catalog, TSV headers, and triage/domain classification tables into a separate reference file (e.g. references/status_codes.md) linked from the overview.
Fix or remove the dangling src/wizard.py reference, since no src/ directory exists in the bundle.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~615-line body avoids explaining basic concepts Claude knows, but restates the same rules repeatedly (SI gate, API-fallback, no-credentials-in-chat appear in several sections) and could be tightened substantially, so it is mostly efficient but not lean. | 2 / 3 |
Actionability | It provides copy-paste-ready commands with real flags (e.g. 'node scripts/batch_download.mjs --dois ... --no-si --out', 'python3 scripts/configure_credentials.py set elsevier --stdin'), concrete status codes, and exact TSV headers, giving fully executable guidance. | 3 / 3 |
Workflow Clarity | Routing is shown as a decision tree, Preconditions are a numbered checklist, the WoS route is a sequenced 10-step list, and failure handling has explicit feedback loops (auto-verify -> fail -> user handoff -> retry once -> do_not_auto_retry) with verification checkpoints (%PDF, manifest, extract_pdf_text.py). | 3 / 3 |
Progressive Disclosure | Bundle scripts are referenced by real one-level paths and the body is well-sectioned, but all reference material (status-code catalog, triage tables, TSV formats, domain lists) is inline in one monolithic SKILL.md with no separate reference docs, and src/wizard.py is referenced but absent. | 2 / 3 |
Total | 10 / 12 Passed |