Content
36%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 instructional core — a role statement, a best-practices list, and an OpenAPI skeleton — is serviceable but thin, and it is buried under a legacy agent-definition YAML block that consumes most of the file without guiding Claude. The OpenAPI skeleton itself is corrupted ($ in place of /), so the one concrete artifact is not copy-paste usable, and there is no workflow or validation step for verifying generated specs. The skill needs the legacy block removed, the skeleton repaired, and an ordered procedure with a lint/validation checkpoint added.
Suggestions
Delete or move the legacy agent-definition YAML block out of SKILL.md; it is renderer/registry configuration, not guidance, and accounts for ~75% of the body.
Repair the corrupted skeleton (https://api.example.com, application/json, request/response examples) and add a concrete validation step such as `npx @redocly/cli lint openapi.yaml` or `spectral lint openapi.yaml` after spec creation.
Convert "Key responsibilities" into an ordered workflow (discover routes/controllers -> draft paths and schemas -> define security schemes -> validate -> write the spec file) with explicit checkpoints.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dominated by a ~115-line legacy agent-definition YAML blob (triggers, colors, hooks with echo commands, optimization.batch_size, memory limits) that is machine configuration, not instruction, plus a "Best practices" list of things Claude already knows ("Use descriptive summaries", "Follow OpenAPI 3.0 specification strictly"). It is above score 1 because the post-blob sections are not themselves tutorial padding about what OpenAPI is, but several large sections earn no tokens. | 2 / 5 |
Actionability | There is a concrete artifact — a full OpenAPI 3.0 skeleton under "## OpenAPI structure" — but it is not executable: every slash has been corrupted to "$" ("https:/$api.example.com", "application$json", "$endpoint", "request$response"), making the YAML invalid, and there is no validation command (e.g. a spectral/swagger CLI lint) or concrete step for analyzing an existing API. This matches anchor 3 (concrete guidance present but incomplete, pseudocode-like, missing key details) rather than 4, which requires mostly-executable code with only minor gaps. | 3 / 5 |
Workflow Clarity | "Key responsibilities" is a numbered list of goals (create specs, document endpoints, define schemas) rather than a sequenced procedure — there is no order of operations for producing documentation from an existing codebase and no validation checkpoint that the produced spec is valid, despite spec generation being exactly the kind of output-verification task that needs one. A rough ordering of concerns exists, so this sits above the incoherent anchor (1) but clearly below the validation-gaps-at-every-step anchor (3). | 2 / 5 |
Progressive Disclosure | The instructional body has reasonable section headers (responsibilities, best practices, structure, documentation elements) and no broken or nested references, but the 115-line legacy config blob is content inlined in SKILL.md that clearly belongs in a separate file (or in the trash), and there are no bundle files at all (references/, scripts/, assets/ do not exist) for the advanced detail a spec-authoring skill needs. This matches anchor 3: some structure, content that should be separate is inline. | 3 / 5 |
Total | 10 / 20 Passed |