Content
78%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 dense, highly actionable authoring workflow: concrete verb-call syntax for every operation, exact frontmatter contracts, a scope decision table, and a genuinely validated RED→GREEN→pressure-test loop with feedback and rollback. Its main weakness is redundancy — the graph/wiki-link rules, install semantics, and Stage-1 gate are each stated more than once, inflating token cost without adding information.
Suggestions
State the project-vs-global reference-linking rule once (in Stage 5, where bundle files are added) and cut the near-duplicate paragraph in Stage 3 down to a one-line pointer — the ~10 duplicated lines are the largest single token saving available.
Consolidate Stage 1's import guidance: the `import({ source, skill, add })` syntax and the "do not run `npx skills add` as the default" rule each appear multiple times; give one decision path (OK MCP query → skills.sh URL → CLI fallback) with the call syntax shown once.
Drop Reminders that restate earlier stages verbatim (the Stage-1-gate and install-fan-out restatements) and keep only the genuinely new cautions — the scripts trust-domain warning and the grounding-claims rule — plus add an explicit "confirm with the user / capture `history` before `delete({ skill })`, since global skills are unversioned" step to close the destructive-path validation gap.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient — dense, imperative, and free of basic-concept padding — but it duplicates itself: the project-vs-global reference-linking rule appears nearly verbatim in Stage 3 ("a backticked `references/<topic>.md` path is fine; use a `[[references/<topic>]]` wiki-link only when…") and again in Stage 5 ("a backticked `references/<name>.md` path joins the graph just like a `[[references/<name>]]` wiki-link… Global skills are different…"), "Keep references one level deep" is stated in both stages, and the Reminders section restates the Stage 1 gate and Stage 8 install-fan-out points. Fits anchor 3 (mostly efficient, some unnecessary explanation, could be tightened). Not 2: nearly all tokens carry non-obvious OpenKnowledge-specific semantics Claude cannot already know. Not 4: the duplication is whole passages, more than minor instances. | 3 / 5 |
Actionability | Every operation has copy-paste-ready verb syntax with realistic parameters — `write({ skill: { name: "<lowercase-hyphen-name>", description: "<triggers>", body: "<markdown>", scope: "project" } })`, `edit({ skill: { name, file: "references/tiers.md", find: "…", replace: "…" } })`, `install({ name: "<name>", add: ["codex", "opencode"] })`, `skills({ query: "<2-4 trigger words>" })`, `import({ source: "https://www.skills.sh/<owner>/<repo>/<skill>", add: [...] })` — plus exact limits ("name — lowercase letters, digits, hyphens; ≤64; equals the directory"; "description — ≤1024 chars, no XML tags, no `version` field"), a scope decision table, and named pressures (time, authority, sunk cost). Matches anchor 5: examples cover all common cases; placeholders are inherent user content, not pseudocode, so the executable-example rule is satisfied. Not 4: no meaningful execution gaps beyond the abbreviated location-id list. | 5 / 5 |
Workflow Clarity | Eight clearly sequenced stages with explicit checkpoints: the Stage 1 gate ("Surface the overlap and the installed-skill plus marketplace search outcome before drafting or writing anything. Never discover overlap after the skill is written"), the Stage 4 RED baseline, Stage 6 GREEN with a validate→fix→retry loop ("confirm the agent still follows the rule — then patch any loophole and re-test"), the Stage 7 near-miss sanity check, and Stage 8 rollback via `history({ skill })` → `restore_version`. Fits anchor 4 (clear sequence, most checkpoints). Not 5: the destructive `delete({ skill })` path — which "removes the source and every location" — carries no pre-delete validation step, and "global skills are unversioned in this build" makes that deletion unrecoverable. Not 3: the core authoring workflow has explicit validation and feedback loops, so the destructive-operation cap doesn't apply to the primary flow. | 4 / 5 |
Progressive Disclosure | Verified against the actual bundle: both body references exist as real one-level-deep files — `references/pressure-testing.md` (signaled from Stage 6) and `references/description-optimization.md` (signaled from Stage 7) — with no dangling paths and no `scripts/`/`assets/` claims. The body stays within its own ~500-line budget and appropriately keeps deep procedural detail (the pressure-test loop, description-optimization method) in the references while the stages carry the decisions. Matches anchor 5: clear overview, well-signaled one-level-deep references, appropriate split. Not 4: both references are signaled exactly at the point of need and the split is clean. | 5 / 5 |
Total | 17 / 20 Passed |