Content
87%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
An exceptionally concise, actionable, well-organized instruction skill whose only gap is the lack of a validation/feedback loop around the code-mutating fix step. Adding a verify-then-commit checkpoint would round it out.
Suggestions
Add an explicit validation checkpoint after applying fixes, e.g. 'After fixes, re-read each changed query to confirm the rule is satisfied before finalizing.'
For fix=true, include a brief error-recovery note (what to do if a fix breaks compilation or a queryFn signature) to form a validate→fix→retry loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and rule-driven with no padding or explanation of concepts Claude already knows; every line is a concrete, load-bearing rule. Not a 2 because there is no unnecessary explanation to tighten. | 3 / 3 |
Actionability | Rules are concrete and specific ('forward signal', 'explicit staleTime from a named exported constant', 'onSettled not onSuccess', 'save previous data in onMutate, roll back in onError'). As an instruction-only skill the absence of code is not penalized, and the guidance is fully actionable. Not a 2 because no key details are missing. | 3 / 3 |
Workflow Clarity | The three Steps are sequenced but applying fixes (fix=true default) is a code-mutating/batch operation with no validation checkpoints or error-recovery feedback loop, which the guidelines cap at 2. Not a 3 because there is no validate→fix→retry loop for the destructive fix step; not a 1 because the sequence is present and unambiguous. | 2 / 3 |
Progressive Disclosure | A single, well-organized SKILL.md with clearly signaled one-level-deep references (three tkdodo.blog URLs) and no bundle files to misroute. Not a 2 because content is appropriately compact and references are clearly labeled rather than buried or nested. | 3 / 3 |
Total | 11 / 12 Passed |