Content
75%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-engineered audit skill: eight precisely-specified checks with discovery commands, explicit PASS/FAIL conditions, a validation-gated workflow, and a concrete output template. The recurring weakness is unresolved references — the unnamed '5 spec docs' and the fragile relative link to architecture.md — which slightly dent actionability, workflow clarity, and navigation.
Suggestions
List the exact paths of the '5 spec docs (DESIGN.md + 4 ADRs)' in the Source of truth / How to run sections so step 1 is executable without searching.
Replace the fragile relative link ../../../.cf-studio/config/rules/architecture.md with an explicit repo-relative path and state what to do if the file is absent.
Trim mechanism-level rationale (e.g. the RMT merge-semantics paragraph and incident anecdotes) into a short 'Why these rules' note or the external rules file to tighten the body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational — discovery commands, exact config values, FAIL conditions — with nearly every token earning its place; padding is limited to a few explain-heavy asides ("RMT only collapses duplicates on background merge (never guaranteed at query time)... e.g. the slack_active_days = 42 incident" and the long inline fix template in Check 7). This fits anchor 4 ('efficient; minor instances of over-explanation that could be trimmed') better than anchor 5, since some rationale could be moved out or shortened without loss. | 4 / 5 |
Actionability | Concrete, executable guidance dominates: `find src/ingestion/silver -name "*.sql" -not -path "*disabled*"`, grep patterns, exact expected config strings, a runnable audit script (`audit_rmt_read_dedup.py`), and a literal output template. It falls short of anchor 5 only because key references are unresolvable as written — "Read all 5 spec docs (DESIGN.md + 4 ADRs)" gives no paths, and the architecture.md link is a fragile relative path — leaving minor gaps the model must fill. | 4 / 5 |
Workflow Clarity | "How to run" gives a clear numbered sequence (read specs → run 8 checks → per-check PASS/FAIL with paths/lines → summarize → suggest minimal fixes) with explicit checkpoints ("Refuse to report PASS without having read each spec doc this turn"; the deterministic script plus "confirm each by reading" loop in Check 8) and an output-format checklist. It sits at anchor 4 rather than 5 because the first step's inputs (which 5 spec docs, located where) are ambiguous, a minor validation gap. | 4 / 5 |
Progressive Disclosure | No bundle files (references/, scripts/, assets/) exist, and the body correctly externalizes the rules to one clearly signaled one-level-deep source (architecture.md) while keeping the operational checks inline; section-per-check organization is good. Anchor 4 ('good structure; most content appropriately placed; references mostly clear; minor organization gaps') fits — the gap being the unspecified spec-doc paths and the deep relative link, which keep navigation from being fully clean. | 4 / 5 |
Total | 16 / 20 Passed |