Content
71%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 router is exceptionally lean and well-sequenced, with a genuinely good on-demand reference design, but it is written for a bundle layer (manifest + core + static fragments) that is not present, making the core loading protocol non-executable and leaving two bundle files orphaned. As shipped, the SKILL.md is a pointer system whose primary targets are missing.
Suggestions
Ship or inline the routing layer the body depends on: manifest.yaml, the `always_load`/core files (principles.md, workflow.md, output-contract.md), and the static/ source-format fragments referenced in steps 1–4 — or rewrite those steps against files that actually exist in this bundle.
Wire `scripts/validate_reader_math.py` into the workflow as an explicit verification step (e.g., in a validation section: `python scripts/validate_reader_math.py <reader_dir>` before responding), replacing the checklist currently delegated to the missing core/output-contract.md.
Add `references/article-anatomy.md` to the on-demand table in step 5 (or remove it from the bundle) so every bundled file is reachable from the router.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~65-line router is lean: every section is instruction ("Read [manifest.yaml](manifest.yaml)", "State the detected value in one short line", "Do **not** read every fragment in static/"), with no explanations of concepts Claude already knows and no padding. It assumes competence throughout, matching the anchor-5 example; there is no over-explanation to trim that would justify 4. | 5 / 5 |
Actionability | The guidance names concrete values (five source_format options with detection meanings) and a priority order, but the executable targets are missing: "Read [manifest.yaml](manifest.yaml)", the `always_load` files, `core/principles.md`/`core/workflow.md`/`core/output-contract.md`, the `static/` fragments, and `../nature-shared/core/terminology-ledger.md` do not exist in this bundle, so an agent following step 1 fails immediately. It is above 2 (there is specific, structured guidance, not high-level hints) but below 4 because key execution details are unresolvable as shipped; the bundled `scripts/validate_reader_math.py` is also never wired in. | 3 / 5 |
Workflow Clarity | Steps 1–5 are clearly sequenced with an explicit mid-process checkpoint ("State the detected value in one short line to the user before processing, so they can correct you cheaply") and a fallback rule for multi-format sources and constrained inputs ("still create a draft reader and label missing pages… Do not switch to summary mode"). Not 5 because the "pre-response verification checklist" is delegated to `core/output-contract.md`, which is absent from the bundle, leaving the final validation step implicit; the operation is not destructive/batch so no cap applies. | 4 / 5 |
Progressive Disclosure | The on-demand reference table is well designed — figure-extraction.md, output-spec.md, equation-handling.md, and grounding-rules.md all exist, are one level deep, and are clearly signaled by need. However, scoring against the actual bundle: the primary navigation layer the skill is built around (manifest.yaml, always_load/core files, static/ fragments) is absent, `references/article-anatomy.md` is orphaned (never referenced), and `scripts/` is undiscoverable from the body. This lands at 3 — good structure undermined by broken navigation paths — rather than 4/5, but above 2 since existing references are properly signaled, not buried. | 3 / 5 |
Total | 15 / 20 Passed |