Content
90%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 lean, highly actionable reference skill with executable code, version-aware guidance, and a properly split-out search-directives reference. Workflow sequencing and validation are solid but reference-oriented rather than offering explicit error-recovery loops; progressive disclosure is good though the main file is long.
Suggestions
Add an explicit validate→fix→retry loop for batch/large mutations (e.g., wrap createMovies in a batching pattern with a re-run on failure) to lift workflow_clarity above 4.
Consider moving the full per-directive syntax blocks (@relationship, @cypher, @authorization variants) into a references file the way @fulltext/@vector are handled, keeping SKILL.md as a tighter overview.
Add a one-line trigger note near the top of the body tying natural phrases ('GraphQL API for Neo4j', '.graphql typeDefs', 'Apollo Federation') to the When-to-Use list to broaden trigger-term coverage.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense, code-first content with pointed inline comments ('columnName must exactly match the RETURN alias — mismatch returns null silently') and no padding explaining what GraphQL, Neo4j, or libraries are; version-sensitive material is isolated in 'Version Matrix' and 'v6 → v7 Breaking Changes' sections rather than scattered inline. | 5 / 5 |
Actionability | Copy-paste ready executable code throughout — install commands, a complete minimal Apollo+Neo4j server, full directive syntax, security config, OGM CRUD, and subscriptions — plus a 'Common Errors' table mapping each failure to a concrete fix. | 5 / 5 |
Workflow Clarity | Sequenced steps ('Step 1 — Install', 'Step 2 — Minimal Server Setup', Security Steps 1–3) with validation (assertIndexesAndConstraints wrapped in try/catch, a startup Checklist) and an error→fix table, but it is more reference-manual than linear workflow and lacks an explicit validate→fix→retry feedback loop for batch mutations. | 4 / 5 |
Progressive Disclosure | Good structure with a clearly signaled one-level-deep reference ('Full syntax, generated query shapes, provider config: references/search-directives.md') that exists in the bundle, and clean section headers aid navigation, though the ~485-line body inlines substantial directive reference material that could be split further. | 4 / 5 |
Total | 18 / 20 Passed |