Content
82%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 reference skill: it delivers concrete, executable project-specific rules with genuine validation checkpoints and defers bulk detail to canonical SSOT files. Weaknesses are mild — a dated incident anecdote, cross-reference noise, and reliance on external repo files that are not part of the skill bundle for critical details like the archived-agent list and blocker report format.
Suggestions
Move the 2026-06-17 incident anecdote into a brief 'canonical failure example' note without the raw date, or into a deprecated/old-patterns section, to avoid time-sensitive detail in the main body.
Ship the archived-agent names list and migration table (or at least the blocker report format) as files in a references/ directory inside the skill bundle, so the skill is self-contained instead of depending on unverifiable external repo paths.
Expand the TRUST 5 failure actions into a short recovery loop (e.g., 'if golangci-lint fails: fix, re-run, then re-run the full test suite') so each gate has an explicit fix-and-revalidate step.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, non-obvious project policy with no basic-concept padding — every section states rules Claude could not infer. Minor trimmable material keeps it below the lean anchor: the dated "2026-06-17 incident (29 SPECs wrongly flagged...)" anecdote and parenthetical cross-reference chatter like "(CLAUDE.local.md §WARN)" are padding-adjacent, and the time-sensitive date is not placed in a deprecated/old-patterns section. | 4 / 5 |
Actionability | Guidance is fully concrete and executable: exact commands ("go test ./...", "-count=1", "-race", "golangci-lint run"), exact file paths ("internal/config/envkeys.go", "internal/config/defaults.go"), a copy-paste "ToolSearch(query: \"select:AskUserQuestion,TaskCreate,...\")" pattern, the retained-agent whitelist, and a concrete per-spawn "Agent(subagent_type: \"general-purpose\", tools: ..., prompt: ...)" template covering the common delegation case. | 5 / 5 |
Workflow Clarity | The TRUST 5 gates table functions as an explicit validation checklist ("Every change must pass all five dimensions before completion" with per-pillar failure actions), and "After fixing any test, run the FULL suite" is a real feedback loop. However, the skill is a multi-topic reference rather than a sequenced process, and error-recovery loops (what to do when a gate fails beyond the one-word failure action) are only implicit, so the full sequence-with-recovery anchor is not met. | 4 / 5 |
Progressive Disclosure | The body is a well-sectioned overview that appropriately defers bulk detail (archived agent names, the migration table, the blocker report format) to one-level-deep referenced paths, with a Cross-References section for navigation. But the bundle ships no references/ files at all — the deferred content lives in external repo paths (".claude/rules/moai/...") that cannot be verified as part of the skill, leaving organization gaps against the clear-well-signaled-references anchor. | 4 / 5 |
Total | 17 / 20 Passed |