Content
53%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 content is actionable and reasonably well-structured, but it is bloated with inlined reference material and time-sensitive version pins, and its progressive disclosure is undermined by broken reference-file links. The deploy workflow also lacks an explicit validation checkpoint.
Suggestions
Create the missing reference/ files (cli-commands.md, graphql-admin-api.md, ui-extensions.md) and move the inlined CLI command lists, GraphQL endpoint details, and extension component references into them so SKILL.md stays a lean overview.
Add an explicit validation checkpoint to the Development Workflow before deploy (e.g. 'Run `shopify app dev`, verify the extension renders in the dev store, and confirm webhook deliveries before `shopify app deploy`').
Move the pinned '2025-01' API version into a single dated 'Current API version' note (or a deprecated/old-patterns section) instead of embedding it in endpoint URLs and prose so the skill does not silently go stale.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~450-line body is mostly efficient reference material, but prose intros ('Real-time event notifications from Shopify', 'Custom data storage for extending Shopify resources') and time-sensitive version pins ('2025-01', 'as of January 2025') sprinkled through endpoints add padding that will go stale and could be tightened. | 3 / 5 |
Actionability | Provides executable copy-paste code for the main cases (GraphQL product query/mutation, React checkout extension, CLI commands, shopify.app.toml), but secondary sections like webhook signature validation and the OAuth flow are described only as bullet hints without code, leaving minor gaps. | 4 / 5 |
Workflow Clarity | The 'Development Workflow' gives a clear init→configure→dev→generate→test→deploy sequence, but the 'test' step is a vague 'Install app on test store' with no explicit pass/fail validation checkpoint before the outward-facing deploy, so the destructive-deploy cap of 3 applies. | 3 / 5 |
Progressive Disclosure | The body signals reference files ('see [reference/cli-commands.md]', 'reference/graphql-admin-api.md', 'reference/ui-extensions.md') but the reference/ directory does not exist, so the links are broken; meanwhile large chunks of CLI/API/extension reference that should live in those files are inlined, leaving a bloated monolith with dangling references. | 2 / 5 |
Total | 12 / 20 Passed |