Content
22%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill reads more like a table of contents or high-level checklist than an actionable skill document. It completely lacks concrete code examples, specific implementation patterns, or executable guidance for authentication/authorization—a domain where precise, secure code matters greatly. The content defers almost everything to an external playbook without providing enough substance in the main file to be independently useful.
Suggestions
Add at least one concrete, executable code example for a common auth pattern (e.g., JWT token validation middleware, session setup, or password hashing) directly in the SKILL.md.
Replace the abstract instruction bullets ('Define users...', 'Choose auth strategy...') with specific, actionable steps that include concrete decisions, commands, or configuration snippets.
Add explicit validation/verification checkpoints to the workflow, such as 'Test token expiration handling', 'Verify secrets are not in logs', or 'Run auth integration tests before deploying'.
Include a brief summary of key patterns from the implementation playbook (e.g., a JWT example vs. session example) so the SKILL.md is useful on its own without requiring the external file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Reasonably concise but the 'Use this skill when' and 'Do not use this skill when' sections add moderate overhead without providing actionable implementation guidance. The instructions section is lean but borders on too vague. | 2 / 3 |
Actionability | The instructions are entirely abstract ('Define users, tenants, flows...', 'Choose auth strategy...', 'Design authorization model...'). There are no concrete code examples, specific commands, configuration snippets, or executable guidance whatsoever. It describes rather than instructs. | 1 / 3 |
Workflow Clarity | The five instruction bullets loosely imply a sequence but lack any validation checkpoints, feedback loops, or concrete steps. For a security-critical domain involving credential storage and token lifecycle, the absence of verification steps is a significant gap. | 1 / 3 |
Progressive Disclosure | There is a reference to `resources/implementation-playbook.md` for detailed patterns, which is good one-level-deep disclosure. However, the SKILL.md itself provides almost no substantive content as an overview—it's essentially just a pointer with vague bullet points, making the split feel like the main content is entirely deferred rather than summarized. | 2 / 3 |
Total | 6 / 12 Passed |