Content
77%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 well-structured, highly actionable testing procedure with excellent workflow guardrails (hard-stop precondition, boundary guidance, resolver short-circuit assertion, anti-patterns table) and a clean one-level-deep bundle split. The weaknesses are repeated library defaults and URL citations across sections, and a graphql-armor cost-bust example that does not demonstrably exceed its stated limit.
Suggestions
State each library default and its source URL exactly once (e.g., in Step 2) and remove the repetitions in the Overview, Step 3, Step 4 comments, and the References list — keep References as the single URL index.
Make the graphql-armor over-cost example verifiably over-limit: compute the cost of the sample query under objectCost=2/scalarCost=1/depthCostFactor=1.5, or generate fields programmatically as the graphql-cost-analysis example does (Array.from({ length: MAX_COST + 1 }, ...)).
Consolidate the Overview's library list with Step 1's install commands so each library is introduced once with its repo/URL, cutting roughly 20 lines of duplicated citation.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient with no explanations of concepts Claude already knows, but library defaults are repeated (max-depth "n = 6" and "maxCost = 5000" in both Overview and Step 2; "objectCost = 2, scalarCost = 1, depthCostFactor = 1.5" in Step 3 and again in a Step 4 comment), and the same escape.tech plugin URLs are cited three to four times across Overview, Steps 2–4, and References. That is more than the minor trimming of the 4 anchor; it fits the 3 anchor ("mostly efficient but... could be tightened"). | 3 / 5 |
Actionability | Install commands, copy-paste TypeScript test code for all three libraries on both Apollo and Envelop, and a resolver-instrumentation verification pattern ("expect(resolverCallCount).toBe(0)") make the guidance mostly fully executable. However the graphql-armor over-cost example relies on an uncomputed "Stack fields so calculated cost > maxCost=200" over a ~15-field query that likely stays under 200, so that test as written could pass vacuously — a real gap for a skill whose purpose is over-limit construction. Fits the 4 anchor, not the 5 anchor. | 4 / 5 |
Workflow Clarity | Clear numbered sequence (Install → Identify limit → Craft queries → Write tests → Assert rejection before execution) with explicit validation checkpoints: the HALT hard-stop when no limit is configured, boundary guidance ("Use limit+1 to be unambiguous"), config-drift detection ("Tests that pass in NODE_ENV=test but fail in NODE_ENV=production... signal a configuration drift problem"), and a six-row anti-patterns table serving as a failure-mode checklist. Matches the 5 anchor. | 5 / 5 |
Progressive Disclosure | The two bundle files (references/introspection.md, references/persisted-queries.md) exist, are one level deep, and are clearly signaled from both the Overview and the References section; the SKILL.md body keeps only the testing procedure inline and externalizes the reference catalogs. Content is appropriately split with easy navigation, matching the 5 anchor. | 5 / 5 |
Total | 17 / 20 Passed |