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 high-quality rules skill: concrete method inventory with import paths, a clear decision workflow, explicit exceptions, and a real, clearly signaled reference file for the audit detail. Minor room to tighten the rationale paragraph and fold the action-fetch timeout details into a reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and mostly lean: a compact need→method table, terse numbered steps, short do/don't lists, and minimal boilerplate. It is not 5 because the 'Why' paragraph re-enumerates consequences ("rediscover auth, base paths, request-source headers, JSON parsing, error handling, optimistic updates, sync invalidation, and route quirks") — explanation that could be trimmed by a sentence — and the action-fetch-behavior paragraph carries detail that could partly live in a reference. | 4 / 5 |
Actionability | Fully concrete guidance throughout: named hooks with exact import paths (`useActionQuery` / `useActionMutation` from `@agent-native/core/client/hooks`), an explicit call signature `callAction(name, params, { method, signal, timeoutMs })`, file-placement rules (`packages/core/src/client/*`, `templates/<app>/app/hooks/*`), and paired good/bad code examples. A developer can act on this directly without invention. | 5 / 5 |
Workflow Clarity | The three-step decision sequence is clear and ordered: look for an existing client API (table), else add the narrowest helper with placement rules and 'Add focused tests for URL construction, headers, response parsing, error shape, and any sync invalidation', then teach the helper. It is not 5 because the validation guidance is a checklist mention rather than an explicit validate→fix→retry feedback loop, and the exceptions list is presented as cases rather than a decision checkpoint. | 4 / 5 |
Progressive Disclosure | Clear sectioned overview (Rule / Why / How / Exceptions / Don't / Related Skills / References) with a single, well-signaled one-level-deep reference — `references/legacy-client-fetch-audit-2026-06-03.md` — which exists in the bundle and is described in place ('known legacy cleanup targets found when this rule was added'). Content is appropriately split: the SKILL.md holds the rule, the audit detail lives in the reference, and related skills are named for navigation. | 5 / 5 |
Total | 18 / 20 Passed |