Content
67%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: concrete bash commands for adapter and fallback paths, a clear six-step workflow with good decision rules, and disciplined depth escalation. Weaknesses are moderate redundancy (adapter chain and argument list each stated twice), placeholder-style command menus instead of ready-to-run invocations, and no error-handling/validation loop after command execution.
Suggestions
State the adapter-resolution chain once — keep the executable bash in Step 2 and reduce the Constants entry to the canonical name and fallback policy — and merge the 'Overrides' example list with the Step 1 argument spec.
Replace the seven-command menu in Step 3 with one concrete, substitution-ready invocation per mode (e.g. `python3 "$DEEPXIV_FETCHER" search "agent memory" --max 10`) so commands are copy-paste ready.
Add a lightweight validation step after execution: check the JSON output parsed correctly and define what to do when the resolved adapter fails (e.g. demote to raw `deepxiv` CLI and retry) instead of noting the gap.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient — no explanations of concepts Claude already knows, and the positioning table is compact — but the adapter-resolution chain is described twice (Constants section plus a 15-line bash block in Step 2) and the argument list is duplicated between the 'Overrides' block and Step 1. Not 4 because this redundancy is real tightening work; not 2 because there is no concept-explanation padding or verbose filler. | 3 / 5 |
Actionability | Provides concrete, executable bash for both the adapter path and the raw CLI fallback, with real flags (--days, --limit, --format json) and a copy-ready resolution script. Not 5 because Step 3 is a template menu of seven commands with unresolved placeholders (QUERY, ARXIV_ID, MAX_RESULTS) rather than one copy-paste-ready invocation per mode, and it acknowledges but does not handle a non-functional adapter ('not currently auto-demoted'). Not 3 because the commands given are fully executable, not pseudocode. | 4 / 5 |
Workflow Clarity | Six clearly sequenced steps with explicit decision rules (default to 'brief' for bare arXiv IDs, fallback to raw CLI when the adapter is unresolved, 'ask before writing' to the wiki) and a well-defined depth-escalation progression. Not 5 because validation is thin — the smoke test is optional, errors from the adapter are not handled, and there is no verify-output checkpoint after execution. Not 3 because the sequence is coherent with most checkpoints (fallback, defaults, escalation gating) present and the operations are read-only rather than destructive. | 4 / 5 |
Progressive Disclosure | Well-organized sections (Role & Positioning, Constants, Setup, Workflow, Key Rules) with clearly signaled one-level-deep references to shared-references/integration-contract.md including section pointers. Not 5 because the referenced contract file lives outside the skill directory and cannot be verified within this bundle, and the full raw-CLI fallback command list is inlined where a reference file would keep SKILL.md leaner; not 3 because navigation is easy and references are explicitly signaled, not buried. | 4 / 5 |
Total | 15 / 20 Passed |