Content
43%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 body delivers genuinely concrete code examples, but it functions as a 500-line inlined reference manual rather than a lean SKILL.md overview: it re-teaches API basics Claude already knows and duplicates its own bundle files, while offering no design workflow. Navigation is further undermined by three referenced bundle files that do not exist.
Suggestions
Cut the SKILL.md body to a concise overview: remove explanations of basics (HTTP method semantics, REST noun/verb rules, GraphQL query/mutation roles) and replace the inline FastAPI/GraphQL/Resolver/DataLoader pattern code with links into the existing references/rest-best-practices.md and references/graphql-schema-design.md, which already cover the same material.
Fix the Resources section: remove or actually create the three nonexistent entries (references/api-versioning-strategies.md, assets/graphql-schema-template.graphql, scripts/openapi-generator.py — there is no scripts/ directory).
Add a short sequenced workflow that gives the skill a process spine, e.g. 1) draft resources/schema, 2) apply versioning and error-format decisions, 3) validate the design against assets/api-design-checklist.md, wiring the existing checklist asset into actual use.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body re-explains fundamentals Claude already knows — "Resources are nouns (users, orders, products), not verbs", "GET: Retrieve resources (idempotent, safe)", "Queries for reading data / Mutations for modifying data", "Clients request exactly what they need" — and then inlines ~400 lines of pattern code (FastAPI pagination endpoint, ~90-line GraphQL schema, resolvers, DataLoaders) that largely duplicates references/rest-best-practices.md and references/graphql-schema-design.md. This is noticeably verbose with several padded sections, fitting the level-2 anchor rather than the minor-trimming of level 3. | 2 / 5 |
Actionability | The patterns are conveyed through concrete, largely executable code: a complete FastAPI paginated list endpoint, ariadne resolver implementations with cursor pagination, and DataLoader classes with batch_load_fn logic. Minor gaps keep it below level 5: helpers like fetch_users, hash_password, decode_cursor and imports like Any and ValidationError are undefined, so examples are not fully copy-paste ready. | 4 / 5 |
Workflow Clarity | There is no sequenced design or review process at all — the body is a catalog of concepts and patterns with no ordering (no 'design the schema, then the resources, then errors, then validate against the checklist' flow) and no validation checkpoints, even though the skill's stated use cases ('reviewing API specifications', 'establishing API design standards') are process-shaped and a checklist asset exists but is never wired into a workflow. This sits below level 3, which requires at least a present step sequence. | 2 / 5 |
Progressive Disclosure | Section headers are clear and the Resources section signals bundle files with one-line descriptions, but content that should be separate is inlined: the REST/GraphQL pattern sections duplicate the 408-line rest-best-practices.md and 583-line graphql-schema-design.md. It cannot score 4 because navigation is partially broken — 3 of the 7 listed resources do not exist (references/api-versioning-strategies.md, assets/graphql-schema-template.graphql, scripts/openapi-generator.py, and there is no scripts/ directory at all). | 3 / 5 |
Total | 11 / 20 Passed |