Content
96%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.
An exemplary orchestration skill body: executable commands with a dry-run/review/apply safety contract, explicit conflict and interruption recovery, a precise routing table, and tightly scoped conditional references. The only structural flaw is that three references escape the bundle to ../../docs/, where the files are not present in the shipped structure.
Suggestions
Move native-conformance.md, capacity-and-recovery.md, and upstream-references.md into references/ (or state explicitly that they ship with the product checkout) so every link in SKILL.md resolves within the skill bundle.
For the ../../docs/ links that are intentionally outside the skill, add the expected location relative to the portable core installation so a model that cannot find them at ../../docs/ has a fallback resolution path.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and imperative throughout — "Treat the installed product as code and the selected user vault as data", "Fail closed when selection is missing or ambiguous", "Raw source payloads are create-only" — with zero background explanation of Obsidian, Git, or vaults that Claude already knows. It matches anchor 5 (assumes Claude's competence, every token earns its place) rather than anchor 4, which expects over-explanation worth trimming; none is evident. | 5 / 5 |
Actionability | Concrete, executable commands cover the common cases: `python3 "$CORE" --help`, full init/adopt dry-run and `--apply` invocations with explicit flags, and `python3 "$CORE" checkpoint OPERATION_ID --vault ...`, plus an intent-to-skill routing table and a fail-closed vault resolution order. Placeholder values have explicit provenance ("pass the emitted `approved_plan_sha256` unchanged"), matching anchor 5; anchor 4 would require missing key details, and none are apparent. | 5 / 5 |
Workflow Clarity | The setup flow has an explicit validation sequence (dry-run by default, "Before `--apply`, show the selected path and changed-path preview, then pass the emitted `approved_plan_sha256`"), destructive paths are gated ("Do not use `--force` unless the user has reviewed the conflicts and explicitly approved replacement"), and error recovery is explicit ("On a conflict, re-read and rebuild. On interruption, use `transaction recover`"). This is anchor 5's clear sequence with validation checkpoints and feedback loops, not anchor 4, which would have minor validation gaps. | 5 / 5 |
Progressive Disclosure | The body is a well-signaled overview with one-level-deep references — "Read only the reference needed for the current request" with 14 conditionally-scoped links, all of which exist in references/ — which approaches anchor 5. However, three references point to `../../docs/native-conformance.md`, `../../docs/capacity-and-recovery.md`, and `../../docs/upstream-references.md`, which do not exist in the provided bundle structure, a minor navigation gap that anchor 4 ('minor organization gaps') captures better than anchor 5. | 4 / 5 |
Total | 19 / 20 Passed |