Content
75%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, largely actionable single-file skill: concrete CLI commands, a rules table, a complete CI workflow, and sensible Federation-specific guidance with a composition validation step. The main gaps are the unshown creation of per-subgraph base schemas, a signature-only custom-rule example, and the absence of failure-path guidance when breaking changes are detected.
Suggestions
Show how the per-subgraph base schemas are produced (e.g., a git show/cat-file step writing /tmp/<subgraph>-base), since the Federation loop currently diffs against files whose creation is never demonstrated.
Replace the custom-rule prose with a minimal executable JS module (module.exports = ({ changes }) => ...) so the --rule './rules/our-custom.js' example is copy-paste ready.
Add a short 'When the gate fails' note covering the remediation path (fix the breaking change vs. suppress via suppressRemovalOfDeprecatedField / a custom rule), and trim the repeated '(Per [gqi-diff][diff])' citation parentheticals.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely lean — commands and tables with minimal padding — but contains minor trimmable instances: repeated "(Per [gqi-diff][diff].)" citation parentheticals, a three-bullet 'When to use' section that restates the description, and orientation prose ("GraphQL Inspector is a schema-management CLI from The Guild...") that slightly over-explains. Fits 'efficient; minor instances of over-explanation that could be trimmed' rather than the every-token-earns-its-place anchor at 5. | 4 / 5 |
Actionability | Quotes: "npm install --global @graphql-inspector/cli", "graphql-inspector diff schema.old.graphql schema.new.graphql", a full copy-paste GitHub Actions YAML with base-schema extraction, and a rules table with flags — mostly executable and concrete. Not 5 because the Federation loop diffs against "/tmp/${subgraph#subgraphs/}-base" files whose creation is never shown, and the custom-rule section gives only the export signature ("exports a function ({ changes }) => filteredChanges") with no actual JS module example — minor gaps in otherwise copy-paste-ready guidance. | 4 / 5 |
Workflow Clarity | The flow (install → diff locally or against prod introspection → apply rules → CI gate → per-subgraph Federation diff) is clearly sequenced, with the exit-code gate stated explicitly ("exits non-zero when breaking changes exist... the gate is the exit code") and a composition validation step ("rover supergraph compose... to confirm the composition still produces a valid supergraph"). Not 5 because there is no guidance for the failure path — what to do when breaking changes are reported (remediate vs. suppress with which rule) is left implicit. This is a read-only diff/gate skill, so the destructive/batch validation cap at 3 does not apply. | 4 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/), and the single-file body is well organized with clear sections, tables, and a References list of one-level-deep external links ([overview], [diff]) plus named sibling skills. Fits 'good structure; most content appropriately placed; minor organization gaps' — the ~35-line CI YAML and Federation details could arguably live in a references/ file, and the under-50-lines simple-skill exception for a score of 5 does not apply to this ~190-line body. | 4 / 5 |
Total | 16 / 20 Passed |