Content
88%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.
An excellent rule catalog: every rule pairs a rationale with concrete detection patterns and false-positive carve-outs, and the invocation contract (path group, rule-id citation) is stated unambiguously up front. The main weaknesses are repetitive Why/Look-for duplication within several rules and a single-file layout with no progressive disclosure for what is a long reference document.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with DevTools-specific invariants Claude would not know, with no padding about general concepts — every token is rule content. It falls short of anchor 5 only because several rules' **Why** and **Look for** sections restate the same guidance nearly verbatim (e.g. `UI_DECLARATIVE_CHILD_WIDGET_COMPOSITION` repeats the `widget(ChildWidget, {...})` preference three times), which could be trimmed. | 4 / 5 |
Actionability | Each rule provides directly executable review guidance: grep-able patterns with real identifiers (`UI.Fragment.build`, `Platform.assertNever(x)` in `default:`, `import * as Buttons from '../../ui/components/buttons/buttons.js'`, `querySelector('[attr="' + val + '"]')`), explicit severity levels, and precise "Do not flag" carve-outs that prevent false positives. Per the rubric's instruction-skill note, this fully covers the common cases without needing code blocks. | 5 / 5 |
Workflow Clarity | It is a single-task skill (review changes in the path group) whose single action is unambiguous: the intro specifies matching `^front_end/(ui|panels)/.*\.(ts|css)$`, applying the 17 rules, and "cite the rule id in findings" for output format. No destructive or batch operations are involved, so no validation cap applies and the simple-skill exception holds. | 5 / 5 |
Progressive Disclosure | Structure is good — one section per rule with a consistent Severity/Why/Look-for/Do-not-flag schema — but the skill is a monolithic ~18KB single file with no bundle reference files, and its only links (`docs/ui_engineering.md`, `code-review`) point outside the bundle. This matches "good structure; minor organization gaps" rather than the appropriately-split, well-signaled one-level-deep references of anchor 5. | 4 / 5 |
Total | 18 / 20 Passed |