Content
70%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 delivers an excellent, strongly gated workflow (mandatory search, validate-before-return, bounded retry loop) with concrete commands and complete Shopify-specific examples. Its weaknesses are redundancy in the workflow instructions, inlined general Liquid reference material, and several references to schemas and tools that are absent from the bundle.
Suggestions
Remove the duplicated "MANDATORY: Search/Validate" sections near the end that restate the workflow already defined in "Required Tool Calls (do not skip)".
Fix or remove dead references: schemas/section.json, schemas/theme_block.json, schemas/theme_settings.json, schemas/translations.json, and the search_docs_chunks tool are not present in the bundle.
Move the generic Liquid reference (delimiters, standard filters, forloop, global objects) into a references/ file, keeping only Shopify-specific gotchas and examples in SKILL.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most of the body is a dense, useful reference (Shopify-specific tags, gotchas, filter lists), but it is noticeably redundant: the two trailing "MANDATORY" sections restate the workflow commands and 3-retry loop already given in "Required Tool Calls", and it explains basic Liquid (assign/capture, common string/math filters, delimiters) that Claude already knows. This fits anchor 3 ('mostly efficient but includes some unnecessary explanation or could be tightened') better than anchor 2, since there is no concept-explaining padding of the bad-example kind. | 3 / 5 |
Actionability | Commands are copy-paste ready with exact flags ("scripts/search_docs.mjs \"<query>\"", validate.mjs with --model/--artifact-id/--revision), and code examples like the block example with {% doc %}/{% stylesheet %}/{% schema %} are complete and executable. It falls short of 5 due to minor gaps: placeholders require substitution (acceptable but explicit), and several referenced targets (schemas/section.json, schemas/theme_block.json, search_docs_chunks) do not exist in the bundle. | 4 / 5 |
Workflow Clarity | The workflow is an explicit numbered sequence — search before writing, write, validate before returning, "If validation fails: search for the error type, fix, re-validate (max 3 retries)", and "Return code only after validation passes" — with two validate modes (full-app vs. stateless) and an error-message-driven recovery loop. This matches the top anchor exactly: clear sequence with explicit validation, feedback loops, and retry limits. | 5 / 5 |
Progressive Disclosure | Headers and sections are well organized and the runtime-doc-lookup via scripts is a legitimate one-level disclosure mechanism, but the ~200-line inline Liquid reference (filters, tags, global objects) is content that belongs in a separate reference file, and references to schemas/section.json, schemas/theme_block.json, schemas/theme_settings.json, schemas/translations.json, and search_docs_chunks point at files/tools that do not exist in the bundle. This sits between anchor 2 (inline content that belongs in separate files) and anchor 3 (structured but should be split), landing at 3 because the organization itself is good. | 3 / 5 |
Total | 15 / 20 Passed |