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, highly actionable instruction skill: exact output formats, closed vocabularies, a worked example, and a workflow with an unusually thorough validation checklist. The main weakness is token efficiency — the validation vocabulary tables are duplicated between SKILL.md and references/REFERENCE.md, and two overlapping rule tables could be consolidated.
Suggestions
Keep the Validation Rules vocabulary in only one place: either trim the inline Keys/Position/mapping tables in SKILL.md to a brief summary and rely on references/REFERENCE.md, or drop the duplicate from the reference file.
Consolidate the Position table and the attribute-type-to-value mapping table (they encode the same rules twice); the worked example cells already demonstrate composition.
Make the validation feedback loop explicit in step 6, e.g. "If any check fails, fix the document and re-run all checks before reporting" — currently the fix-and-retry loop is only implicit.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is prescriptive throughout (closed vocabularies, exact formats, anti-pattern examples) and explains nothing Claude already knows, but it carries the full Validation Rules tables inline ("Keys", the Position table, and the attribute-type mapping table) while stating "The same tables are available in [references/REFERENCE.md]" — roughly 40 lines duplicated across SKILL.md and the reference file, and the Position table overlaps the subsequent attribute-type mapping table. Not a 4 because the duplication is more than a minor trim; not a 2 because most of the text is genuinely load-bearing rules that earn their tokens. | 3 / 5 |
Actionability | Everything is executable and copy-paste ready for an instruction skill: exact heading format ("A ### heading with the entity name in UPPERCASE (UPPER_SNAKE_CASE ... never `### Room Type`"), an exact 5-column table spec, a closed Data Types list, a closed Validation Rules vocabulary with worked example cells ("Not Null, Values: Pending, Active, Completed, Cancelled"), concrete Mermaid syntax, and a filled example entity table. Not a 4 because the common cases are covered completely with explicit negative examples. | 5 / 5 |
Workflow Clarity | The 7-step workflow is clearly sequenced with an explicit clarification step (including fallback behavior when no one can answer), TodoWrite task breakdown, per-entity sub-steps, and a 12-item validation checklist in step 6 before reporting. Not a 4 because validation checkpoints are explicit and exhaustive; the only nit is that "fix and re-validate" after a failed check is implicit rather than stated. | 5 / 5 |
Progressive Disclosure | Two clearly-signaled, one-level-deep references ([references/REFERENCE.md] and [references/clarify-checklist.md] in workflow step 2) exist in the bundle and are correctly linked, and sections are well organized. Not a 5 because the Validation Rules content is inlined in SKILL.md while also living in REFERENCE.md — the same content occupies two places, a minor organization gap; not a 3 because navigation is easy and no content that clearly belongs elsewhere is buried. | 4 / 5 |
Total | 17 / 20 Passed |