Content
82%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.
The body is a lean, highly actionable procedure set with concrete tool calls, numbered workflows, and explicit safety boundaries. It is slightly long for a single SKILL.md and could offload the detailed Move-local-skills classification into a reference file, but guidance quality is strong.
Suggestions
Move the detailed environment-bound classification table (Move local skills step 2) into a references/ file, keeping SKILL.md as a concise overview with a one-level-deep pointer — this would lift progressive_disclosure toward 5.
Add an explicit validate-then-retry feedback loop for the zip upload (e.g. 'if the response does not list the expected URIs, re-run curl') to turn confirm/verify checkpoints into full error-recovery loops.
Tighten the opening rationale paragraph and the 'why not write/edit' justifications to a single clause each to recover token budget without losing the safety guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense and procedural, assuming Claude's competence without explaining basics; a few rationale passages (e.g. the opening 'OpenViking stores skills as data…' and some 'why not write/edit' justifications) could be trimmed without losing guidance. | 4 / 5 |
Actionability | Copy-paste ready commands with full argument signatures — `find(query="…", context_type="skill")`, `add_skill(data="…")`, and the complete zip-then-curl upload pipeline — cover the common cases concretely. | 5 / 5 |
Workflow Clarity | Multi-step procedures (Create, Install, Move local skills) are numbered with confirm/verify checkpoints and a classify-then-confirm-then-verify flow; validation is present so the destructive-cap is avoided, though explicit error-recovery feedback loops are thin. | 4 / 5 |
Progressive Disclosure | No bundle files exist; the body is self-contained and organized into eight clearheaded sections (Find, Use, Create, Install, Share, Delete, Move, Boundaries) with a short overview, but at ~180 lines it is larger than the simple-skill exception and carries inlined detail that a reference file could hold. | 4 / 5 |
Total | 17 / 20 Passed |