Content
67%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 body is a well-structured, actionable instruction skill with concrete validation commands and a clear phased procedure. Its main weaknesses are repeated boundary statements across three sections and a few dangling or unreferenced bundle paths.
Suggestions
Merge the overlapping do-not statements in Preconditions, Constraints, and Execution Boundaries into a single authoritative boundary list to recover tokens.
Reconcile the reference list with the actual bundle: either add the missing `../../references/...` files or note they live in a shared cross-skill tree, and reference `references/task-profile.json` from the Inputs or Procedure section.
Number the validation checkpoints inside the Procedure (e.g., an explicit "Validate before writing" step) so the fail-fast loop is as explicit as the good_overall workflow example.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and avoids padding of concepts Claude already knows, but Preconditions, Constraints, and Execution Boundaries repeatedly restate the same "do not mutate code / create Linear objects / author strategy" boundaries and could be consolidated. | 3 / 5 |
Actionability | Concrete paths (`.harness/reframes/**`), an explicit return-field list, and copy-paste validation commands (`check_bluf_structure.py ... --json`, `./bin/ask skills audit ... --json`) give mostly executable guidance with only minor gaps. | 4 / 5 |
Workflow Clarity | A 12-step procedure with explicit fail-fast validation and a Blocked feedback loop sequences the work clearly; the destructive/batch operations have validation present, though a couple of checkpoints are implicit rather than numbered. | 4 / 5 |
Progressive Disclosure | Section structure is good with well-signaled one-level references and a deferred-context index, but several `../../references/...` paths are not present in the bundle and the bundled `references/task-profile.json` is never referenced from the body. | 4 / 5 |
Total | 15 / 20 Passed |