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 well-disciplined routing skill: lean body, real version-specific knowledge, explicit sequencing with an unknown-version fallback, and genuine one-level-deep references. The main gaps are the missing command for checking the CLI version, triple-stated routing information, and the unsurfaced shared verified-syntax.md reference.
Suggestions
Add the concrete command for the entry-point step, e.g. "check the installed CLI version with `tfpolicy version`", so the first action is executable rather than implied.
Consolidate the routing information — the "INVOKES" header line, the Routing table, and the Examples section all restate the same two-way mapping; keep the table plus one example line each and drop the rest.
Add a brief mention of references/verified-syntax.md (the shared source of truth) in the body so the router can surface it directly instead of it being reachable only two levels deep through the sub-skills.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumption-free about concepts Claude already knows, and the dense 0.2.x/0.3.x bullets are all novel version-specific behavior that earns its tokens. It is not a 5 because the routing is stated three times (the "UTILITY SKILL — INVOKES" line, the Routing table, and the Examples section all restate the same two-way mapping), which could be consolidated. | 4 / 5 |
Actionability | Concrete, executable elements abound: a routing table with real file links, example utterances mapped to sub-skills, copy-paste `npx skills add` commands, and very specific version-dependent rules ("every mock `resource {}` block ... must declare `attrs` or `prior_attrs`"). It is not a 5 because the first instruction — "check the installed `tfpolicy` CLI version" — gives no command for how to check it, leaving the entry-point action underspecified. | 4 / 5 |
Workflow Clarity | The sequence is explicit and correctly ordered ("Before giving authoring or testing instructions, check the installed `tfpolicy` CLI version"), each task routes unambiguously via the table, and the unknown-version branch has a defined fallback ("ask the user to check it first or provide guidance that clearly distinguishes the paths"). This is a single-purpose routing skill with no risky operations, so no validation checkpoints are required. | 5 / 5 |
Progressive Disclosure | The body is a true overview with well-signaled, verified one-level links (references/tfpolicy-author.md, references/tfpolicy-test.md, including anchor links), and the detailed material is appropriately split into those files. It is not a 5 because the third bundle file, verified-syntax.md (the declared "source of truth"), is not surfaced from SKILL.md at all — it is only reachable two levels deep through the sub-skills, leaving the router unaware of it. | 4 / 5 |
Total | 17 / 20 Passed |