Content
78%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 genuinely lean, well-organized overview that routes each API action to a specific rule file with concrete paths and commands in the body. Its one material defect is that every referenced file is missing from the bundle, which breaks progressive disclosure and undercuts actionability for the core tasks.
Suggestions
Ship the referenced bundle files (use-connect-query.md, api-invalidate-cache.md, api-toast-errors.md, protobuf-no-edit.md, and the rules/ directory) or remove/inline the dangling references so each row in the Quick Reference table resolves.
Add one short executable hook example (e.g. a useConnectQuery call) so the most common action — fetching data — is actionable even before following a reference.
Resolve the ambiguity between the Quick Reference table's per-file pointers and the closing 'See rules/ directory' line by using one consistent navigation scheme.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~35-line body is lean and table-driven: every line is a rule, path, or command ("Regenerate protos: `task proto:generate`"). No concepts Claude already knows are explained, and nothing is padded — every token earns its place. | 5 / 5 |
Actionability | Gives concrete specifics — exact paths ("/src/react-query/", "/src/protogen/"), a runnable command ("task proto:generate"), and a named function ("formatToastErrorMessage" in onError). Not 5 because the core actions ("Fetch data | use-connect-query.md") defer entirely to files that are not present, leaving no executable example of actual hook usage; not 3 because what is inline is specific and directly usable. | 4 / 5 |
Workflow Clarity | The Quick Reference table clearly sequences the core flow ("After mutation | api-invalidate-cache.md") with well-defined action-to-rule mapping, appropriate for a pattern-reference skill with no destructive or batch operations. Not 5 because the steps each action delegates to live in files missing from the bundle, so the full sequence cannot be followed; not 3 because the routing itself is unambiguous. | 4 / 5 |
Progressive Disclosure | The body is structured as a proper overview with clearly signaled one-level references ("Fetch data | use-connect-query.md", "See `rules/` directory"), but none of the referenced files — use-connect-query.md, api-invalidate-cache.md, api-toast-errors.md, protobuf-no-edit.md, or the rules/ directory — exist in the bundle, so navigation is broken. Not 4 or 5 because the disclosure chain leads nowhere; not 2 because the overview itself is well organized rather than an inlined monolith. | 3 / 5 |
Total | 16 / 20 Passed |