Content
65%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A well-organized, highly actionable style guide with concrete examples and a clear Draft/Edit/Polish/Format process. It loses points for some advisory prose, a light validation loop, and a shared-guide @-import whose target file is missing from the bundle.
Suggestions
Tighten the 'Start here' rhetorical questions and editorial asides (e.g. 'Nobody wants to be in docs longer than necessary') into direct imperatives to improve conciseness.
Add an explicit verify→fix→retry checkpoint in the writing process (e.g. after 'Verify examples actually work', state 'if an example errors, fix it and re-verify before moving on') to strengthen the workflow loop.
Ensure the @./../_shared/metabase-style-guide.md import resolves to a real bundled file, or inline the essential style rules so the skill is self-contained.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient bullet/table format with no padding about concepts Claude already knows, but editorial asides like "Nobody wants to be in docs longer than necessary" and the three rhetorical "Start here" questions could be tightened. Not 3 because some prose is advisory filler rather than direct guidance; not 1 because it is lean and un-padded overall. | 2 / 3 |
Actionability | Concrete do/don't pairs throughout — headings ("Set SAML before adding users" ✅ vs "Environment variables" ❌), link examples, a copy-paste command ("yarn prettier --write <file-path>"), and a Write-This/Not-This quick-reference table. Fully specific, actionable guidance matching anchor 3. | 3 / 3 |
Workflow Clarity | A clear Draft → Edit → Polish → Format sequence is present, with a light checkpoint ("Verify examples actually work") and a final prettier step. Not 3 because validation is implicit/light with no validate→fix→retry feedback loop; not 1 because the ordered stages are unambiguous. | 2 / 3 |
Progressive Disclosure | Uses a single well-signaled one-level @-import ("@./../_shared/metabase-style-guide.md") and the body is well-sectioned, but the referenced file is not present in the bundle (no references/scripts/assets dirs, file not found), so navigation to the detailed material fails. Not 3 because the referenced path is not a real, verifiable file; not 1 because the structure is clean and only one level deep rather than a nested chain. | 2 / 3 |
Total | 9 / 12 Passed |