Content
35%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill is essentially a table of contents with no substantive content of its own. It defers all concrete guidance to a referenced playbook that isn't provided in the bundle, leaving the SKILL.md without any executable examples, specific patterns, or actionable instructions. The security domain demands concrete, precise guidance (e.g., JWT signing examples, RBAC middleware patterns, token rotation code), but none is provided at any level.
Suggestions
Add at least 2-3 concrete, executable code examples directly in SKILL.md covering the most common patterns (e.g., JWT token creation/validation, middleware auth guard, password hashing).
Include validation checkpoints in the workflow, such as 'Verify token expiry is set', 'Test that unauthorized requests return 401/403', or 'Confirm secrets are not in logs'.
Resolve the inconsistent references — 'resources/implementation-playbook.md' vs './sub-skills/implementation-playbook.md' — and ensure the referenced file actually exists in the bundle.
Replace the abstract instruction steps ('Define users, tenants, flows...') with specific, actionable guidance that Claude can directly apply, such as concrete data models, middleware patterns, or configuration templates.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The content is relatively brief but includes some unnecessary sections like 'Use this skill when' and 'Do not use this skill when' that add little value for Claude. The instructions section is lean but could be tighter. | 2 / 3 |
Actionability | The instructions are entirely abstract and high-level ('Define users, tenants, flows...', 'Choose auth strategy...', 'Design authorization model...'). There are no concrete code examples, specific commands, executable snippets, or copy-paste ready patterns. All substantive content is deferred to a referenced file. | 1 / 3 |
Workflow Clarity | The instructions list steps in a logical sequence (define requirements → choose strategy → design model → plan secrets → reference playbook), but there are no validation checkpoints, no error recovery steps, and no feedback loops for what are inherently security-critical operations. | 2 / 3 |
Progressive Disclosure | References to 'resources/implementation-playbook.md' and 'sub-skills/implementation-playbook.md' are present, but no bundle files were provided to verify they exist. The SKILL.md itself is too thin — it's essentially just a pointer with almost no standalone value. The two different paths to what appears to be the same resource is confusing. | 2 / 3 |
Total | 7 / 12 Passed |