Content
66%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.
The body excels at workflow clarity with mandatory validation and a retry feedback loop, and delegates heavy logic to real bundle scripts. Its main weakness is conciseness — the same workflow and privacy notice are repeated multiple times — which inflates the token cost without adding guidance.
Suggestions
Collapse the three workflow descriptions into one canonical numbered sequence; reference it from the MANDATORY sections instead of restating it.
Merge the two identical privacy notices into a single 'Privacy notice' block covering both scripts.
Trim or remove the opening role paragraph that restates the description and the 'wrap in triple backticks' boilerplate Claude already follows.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Noticeably verbose: the search→validate workflow is restated three times (the 'Required Tool Calls' list, the 'MANDATORY: Search' section, and the 'MANDATORY: Validate' section), the identical privacy notice is duplicated for each script, and the opening role paragraph restates the description ('helps Shopify developers write GraphQL queries...'). | 2 / 5 |
Actionability | Concrete executable commands with full flag inventories ('scripts/search_docs.mjs "<query>"', 'scripts/validate.mjs --code ... --model ... --client-name ...') and a worked example ('predictiveSearch query'), but every command carries placeholders (YOUR_MODEL_NAME, YOUR_CLIENT_NAME, etc.) that must be substituted, so it is not quite copy-paste ready. | 4 / 5 |
Workflow Clarity | Clear 5-step sequence (search → write → validate → fix/retry → return only on pass) with an explicit validation checkpoint (validate.mjs) and a detailed feedback loop (read error, search the named type, fix exactly, re-validate, max 3 retries) for error recovery. | 5 / 5 |
Progressive Disclosure | Good structure: heavy logic lives in referenced bundle scripts (scripts/search_docs.mjs, scripts/validate.mjs — both real files) and the docs corpus in the gz asset, keeping SKILL.md as an overview with clearly signaled one-level-deep script invocations; minor gaps are the in-body redundancy and no explicit pointer to the asset file. | 4 / 5 |
Total | 15 / 20 Passed |