Content
90%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 dense, project-specific contract with a copy-paste config example, exact JSON field paths, named code owners, and an exact test command — highly actionable with no wasted tokens. Workflow is organized topically with an explicit validation section rather than a stepwise sequence, and operator guidance is correctly externalized to a single well-signaled reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every sentence carries project-specific rules (parser ownership, exact error cases, lifecycle semantics, test contracts) with zero concept-explanation padding. Version references ("Caddy v2.11.7", "go-authcrunch v1.3.4") state the current contract with explicit recheck guidance rather than time-conditional instructions, so they earn their place. | 5 / 5 |
Actionability | The body provides a copy-paste Caddyfile block (its fragment status explicitly justified), the exact JSON field path `apps.security.config.state` with its literal shape, named owning files and functions (`caddyfile_state.go`, `NewStateConfigFromDirectives`, `App.Start`, `App.Cleanup`), and an exact runnable test command with flags. As an instruction/config skill its guidance is fully executable. | 5 / 5 |
Workflow Clarity | Clear topical progression (Configuration → Lifecycle and operation → Validation) with an explicit validation command and coverage-retention guidance, so the destructive-operation cap does not apply. However, the body is organized as a reference contract rather than a stepwise sequence with feedback loops; deployment/recovery sequencing is deferred to the referenced operations guide. | 4 / 5 |
Progressive Disclosure | Good structure: operator guidance is properly split into one real, clearly signaled one-level-deep reference (`references/operations.md`) with explicit read-when conditions, and sections are well organized. It falls short of anchor 5 because the body itself is a dense full contract — e.g. the ~14-line E2E coverage enumeration and detailed lifecycle semantics could arguably live in reference files, keeping SKILL.md closer to an overview. | 4 / 5 |
Total | 18 / 20 Passed |