Content
65%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is highly actionable with executable, copy-paste-ready Shopify code across the full app lifecycle, but it is a monolithic ~1020-line document that overruns the SKILL.md overview role and lacks explicit validation checkpoints in its build/deploy workflow.
Suggestions
Add a numbered end-to-end workflow (scaffold -> develop -> build -> validate -> deploy) with explicit checkpoints, e.g. run `shopify app build` and confirm no errors before `shopify app deploy`.
Split the monolithic body into one-level-deep reference files (e.g. DEPLOYMENT.md, EXTENSIONS.md, TESTING.md, GDPR.md) and keep SKILL.md a concise overview with clearly signaled links.
Trim standard boilerplate Claude already knows (basic Polaris layout, generic vitest/Playwright scaffolding) to recover token budget for the Shopify-specific patterns only Claude would not recall.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient, executable code and config that earns its place, but at ~1020 lines it is far heavier than a SKILL.md overview should be and includes standard patterns Claude already knows (Polaris app layout, Remix settings page, vitest/Playwright boilerplate), fitting the 'mostly efficient but could be tightened' anchor rather than the lean level. | 2 / 3 |
Actionability | It provides fully executable GraphQL queries/mutations, shopify.app.toml configs, CLI commands, Prisma schema, and deployment YAML that are copy-paste ready, matching the 'fully executable code/commands; specific examples' anchor; it is not below because there is no vague or pseudocode-only guidance. | 3 / 3 |
Workflow Clarity | Checklists and some sequencing exist, but the core build-to-deploy flow has no explicit validation checkpoint (e.g. build/verify before `shopify app deploy`), and because deploying app versions and GDPR data deletion are risky operations the missing feedback loop caps this at 2 per the rubric notes. | 2 / 3 |
Progressive Disclosure | The content is well sectioned with clear headings, but it is monolithic: every topic is inline, no references/scripts/assets files exist, and material that should be split out (deployment, extensions, testing) lives in SKILL.md; the under-50-line simple-skill exception does not apply to a ~1020-line document, so it sits at 'some structure but content that should be separate is inline'. | 2 / 3 |
Total | 9 / 12 Passed |