Content
50%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 skill is a lean, well-sectioned instruction-only skill with a sensible high-level workflow, but it stays abstract: no concrete examples, tooling, or deliverable formats, its purpose is restated three times, and its single progressive-disclosure reference points to a file that does not exist. It is serviceable as a prompt template but below the quality of the rubric's good examples.
Suggestions
Remove the redundant restatements: the opening paragraph and the Context section repeat the frontmatter description almost verbatim — collapse them into one sentence or drop them.
Add concrete grounding to Instructions, e.g. a minimal example of a route + scenario definition (JSON/YAML), a sample deterministic fixture, or named tooling options and the expected deliverable format (mock server config + README for switching scenarios).
Fix the broken reference: either create resources/implementation-playbook.md or remove the two pointers to it — currently the only progressive-disclosure path leads to a nonexistent file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is short and free of concept explanations, but the same purpose statement is repeated three times: the opening line ('You are an API mocking expert specializing in creating realistic mock services...') restates the frontmatter, and the Context section ('The user needs to create mock APIs for development, testing, or demonstration purposes. Focus on creating flexible, realistic mocks...') restates it again. This matches anchor 3 — mostly efficient but includes unnecessary repetition that could be tightened. Not a 2 because there is no explanation of known concepts or heavy padding; not a 4-5 because the triple restatement and generic Limitations boilerplate are tokens that earn no new information. | 3 / 5 |
Actionability | The Instructions bullets name concrete artifacts to produce — routes, scenarios, state transitions, deterministic fixtures with randomness toggles, run/scenario-switch documentation — which goes beyond high-level hints (ruling out 2). But nothing is executable or specific: no example route/scenario structure, no sample fixture, no candidate tooling (e.g. WireMock, Prism, MSW, nock), and no expected deliverable format, so key details are missing — matching anchor 3, and falling short of 4's 'concrete code or commands with minor gaps'. | 3 / 5 |
Workflow Clarity | The Instructions present a sensible ordered sequence (clarify contract and error/latency expectations → define routes, scenarios, state transitions → provide fixtures → document running and scenario switching), which exceeds anchor 2's 'rough sequence with many gaps'. However, validation checkpoints are only implicit — the sole guard is a generic 'Stop and ask for clarification if required inputs... are missing' in Limitations, with no step to verify the mock matches the contract or that scenarios behave as expected — matching anchor 3's 'sequence present but checkpoints missing or implicit'. | 3 / 5 |
Progressive Disclosure | The body is well-sectioned and the single reference is clearly signaled with its contents ('resources/implementation-playbook.md for code samples, checklists, and templates', one level deep) — but no resources/ directory or implementation-playbook.md exists anywhere in the bundle, so the reference is a dangling pointer that breaks the disclosure path. Structure and signaling alone would support 4, but a referenced path that fails on open means navigation is only partially functional, landing at anchor 3. Not a 2 because the body itself is well-organized and nothing is over-inlined. | 3 / 5 |
Total | 12 / 20 Passed |