Content
56%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 presents a genuinely well-structured agentic workflow — explicit phases, transitions, failure recovery, exit criteria, and strong stack-specific guardrails — but it is dragged down by concept re-teaching (DRY/SOLID/KISS), the absence of any executable code or example command, and four primary references to a nonexistent resources/ directory that break navigation for the skill's own verification step. It sits solidly at the midpoint of the rubric. Fixing the reference paths and trimming the concept explanations would move it to good.
Suggestions
Resolve or relocate the resources/ references (execution-protocol.md, checklist.md, orm-reference.md, error-playbook.md) — either ship them in the bundle or point to their real location, since the VERIFY phase instructs the agent to follow them.
Delete the explanatory text for DRY/SOLID/KISS and keep only the directives (e.g. 'Business logic in Service, data access in Repository; one responsibility per class; use the framework's DI'), and state the router→service→repository pattern once instead of three times.
Add one small concrete anchor — an example stack/stack.yaml verify block or a minimal router→service→repository snippet — so the verification and architecture guidance has an executable reference point.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most of the body is imperative, project-specific direction that earns its tokens, but there is real padding: the Guardrails section explains concepts Claude already knows ('DRY (Don't Repeat Yourself)', 'SOLID... Single Responsibility: Classes and functions should have one responsibility', 'KISS: Keep it simple and clear'), and the router→service→repository architecture is stated three times (Structural Flow, the Architecture Pattern diagram, and Core Rule 1). This fits anchor 3 ('mostly efficient but includes some unnecessary explanation or could be tightened') — the padding is localized sections, not the extensive concept-teaching of anchor 2. | 3 / 5 |
Actionability | There is concrete, specific guidance — "JWT + Argon2id for auth", "Parameterized queries only (never string interpolation)", named code-intelligence tools (`find_file`, `search_for_pattern`, `get_symbols_overview`, `find_symbol`), and precise layer boundaries — but no executable code or example commands anywhere, and verification is generic ('run the project's discovered verification commands, usually lint/typecheck/tests'). The stack-agnostic stance is justified by the stack/ template mechanism, yet key details like an example verify invocation or one skeleton router/service/repository snippet are missing, matching anchor 3 ('some concrete guidance but incomplete') better than anchor 4's 'concrete code or commands'. | 3 / 5 |
Workflow Clarity | The workflow is clearly sequenced (Entry steps, PREPARE→ACQUIRE→ACT→VERIFY→FINALIZE scenes, Transitions, Exit criteria) with explicit validation (VERIFY step plus 'Checklist (run before handoff): resources/checklist.md') and a feedback loop ('If verification fails, fix root cause before handoff'). It falls short of anchor 5 because checkpoints name command categories rather than anything executable — actual commands are deferred to project discovery or stack/stack.yaml — a minor but real validation gap that anchor 4 ('clear sequence with most checkpoints present') captures exactly. | 4 / 5 |
Progressive Disclosure | Sections are well organized and the References list clearly signals each file's purpose, but scored against the actual bundle: four of the primary references (resources/execution-protocol.md, resources/checklist.md, resources/orm-reference.md, resources/error-playbook.md) point to a directory that does not exist in the bundle — the workflow's VERIFY step depends on a file a reader cannot resolve. Combined with inline content that could live in references (13 core rules plus stack-manifest details), this fits anchor 3 ('references present but' effectively broken for navigation) rather than anchor 4, whose 'references mostly clear' presumes they resolve. | 3 / 5 |
Total | 13 / 20 Passed |