Content
86%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 tight, well-organized standards skill that respects the context window completely and includes an explicit verification gate, cross-referencing related skills appropriately. Its main weakness is the absence of worked examples and concrete commands, which leaves actionability and workflow recovery slightly short of top marks.
Suggestions
Add a minimal worked example under Conventions — a sample component with its `ComponentNameProps` interface and a paired RTL test — so the patterns are copy-paste ready instead of purely descriptive.
Show how to resolve and run the lint/typecheck/test/build commands (e.g., a one-line example of reading `.opencastle/project.instructions.md` and invoking the script) rather than only naming the slots.
Add a short error-recovery line to the Verification section (fix failures and re-run until all four checks exit zero) to close the workflow feedback loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is ~20 lean lines of pure convention statements ("One component per file: `ComponentName.tsx`... co-located with `ComponentName.module.scss` and `ComponentName.test.tsx`", "`strict` stays enabled in `tsconfig.json`; no `as` casts") with zero explanation of concepts Claude already knows. Every token earns its place, matching the score-5 anchor; there is no padding to trim as score 4 would require. | 5 / 5 |
Actionability | Guidance is concrete and executable in the main — specific file naming, export naming (`ComponentNameProps`), the strict/no-cast rule, and dompurify for sanitization — but there are no example snippets (no sample component skeleton, props interface, or RTL test) and the verification commands are deferred ("Resolve the exact commands via the **codebase-tool** slot") rather than given. This sits between the score-4 anchor (mostly executable with minor gaps) and score 5 (copy-paste ready examples covering common cases); the missing worked example keeps it at 4. | 4 / 5 |
Workflow Clarity | There is a clear two-phase flow — apply Conventions, then "Lint, typecheck, test, and build must all exit zero" provides an explicit validation checkpoint — but no error-recovery loop (what to do when a check fails) and no explicit sequencing language. This matches the score-4 anchor (clear sequence, most checkpoints, minor validation gaps); score 5 would require explicit fix-and-retry feedback loops, and this is not a destructive/batch operation that would impose the cap at 3. | 4 / 5 |
Progressive Disclosure | The skill is under 50 lines, has no bundle files (references/, scripts/, assets/ are absent), and the body is organized into three well-labeled sections (Conventions, Verification, Security) with only clearly signaled one-level pointers to other skills ("see **api-patterns**") and a project file (`.opencastle/project.instructions.md`). Per the rubric's simple-skill exception, this earns the score-5 anchor: well-organized sections with no need for external references. | 5 / 5 |
Total | 18 / 20 Passed |