Content
81%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 strong, dense workflow skill: the 9-step loop embeds genuine validation and feedback checkpoints, and the rules/traps/techniques content is project-specific knowledge Claude could not reconstruct. The main gaps are the dated release history in the intro, missing tool invocation syntax, and the unresolved `templates/` reference.
Suggestions
Move the dated provenance (the 2026-09-30 pass, releases 1.0.71/1.0.72, the 5-of-8 A/B result) into `references/thornvigil.md`, which already tracks project state, keeping the body time-independent.
Add one-line invocation syntax for the key tools (e.g., how `keydesign.mjs` and `armcheck.mjs` are actually run — interpreter, arguments, working directory), or point to a single place in the references where the syntax is shown.
Resolve the `templates/` reference: either include the `templates/` directory (critic-prompt.md, ab-judge-prompt.md) in the bundle or state where those prompt templates live, since `references/review.md` depends on them.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and project-specific (rig limits, tool flags, traps) with almost no explanation of concepts Claude already knows, but the intro carries time-sensitive history — "Distilled from the Thornvigil knight-and-archer pass (2026-09-30 to 10-01, releases 1.0.71 and 1.0.72)" and "won 5 of 8 blind A/B comparisons" — which the guidelines penalize outside an old-patterns/deprecated section. This fits the 4 anchor (efficient, minor instances that could be trimmed) better than 3, since the padding is only a few lines. | 4 / 5 |
Actionability | Concrete named tools with specific numeric outputs are given ("run `keydesign.mjs`. It returns the hand target and flags: raise over 100°; flex outside 12–140°"; `armcheck.mjs`, `at.mjs`, `vidcompose.py` + `encode.swift`), and the rules table has measurable limits. However no actual invocation syntax (arguments, working directory, how to call the .mjs tools) appears in the body, so it is not copy-paste ready — the 4 anchor (mostly executable, minor gaps) rather than 5. | 4 / 5 |
Workflow Clarity | "The loop" is a clearly sequenced 9-step process with explicit validation checkpoints and feedback loops: "Fix numbers that break a rule before spending a capture", "When you add a test for a bug, temporarily restore the bug and prove the test fails", "verify every claim against full-resolution frames and live-rig numbers before acting on it", and the ship step gates on tests/build. This matches the 5 anchor (explicit validation steps, feedback loops) and exceeds the 4 anchor's "minor validation gaps". | 5 / 5 |
Progressive Disclosure | Good structure: an overview body with the at-a-glance rules table, and well-signaled one-level-deep references that all exist ("Read `references/thornvigil.md` for the project's files, commands and current state", "Full rules... are in `references/rules.md`"). But the body's "See `references/review.md` and `templates/`" points to a `templates/` directory that does not exist in the bundle (review.md further references `templates/critic-prompt.md` and `templates/ab-judge-prompt.md`), a dangling navigation path. This is the 4 anchor (good structure, minor organization gaps) rather than 5 (easy navigation with no dead ends). | 4 / 5 |
Total | 17 / 20 Passed |