Content
71%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.
A highly actionable security reference with excellent executable code examples, clear access-control sequencing, and CI-backed validation checkpoints. Its main weaknesses are a monolithic, over-long body with inline time-sensitive history that should be split into reference files or a deprecation section.
Suggestions
Move the historical cross-tenant-leak context (the 2026-04-29 narrative and detailed guard-script internals) into an 'Old patterns / deprecated' or separate reference file so time-sensitive information doesn't bloat the core rules.
Extract the long 'Same-Origin Workspace Apps' and per-guard opt-out detail into a references/ file, keeping SKILL.md as a concise overview with one-level-deep pointers, to improve progressive disclosure.
Tighten prose in the guards and credentials sections by leading with the rule and the minimal code snippet, then linking to details rather than narrating the bug class inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most content is genuinely framework-specific (guard scripts, ssrfSafeFetch, accessFilter) and earns its place, but inline time-sensitive history ('On 2026-04-29...', '2026-04 cross-tenant leak class') and lengthy guard-script prose are not isolated in a deprecation/old-patterns section, and several sections could be tightened. | 3 / 5 |
Actionability | Multiple copy-paste-ready TypeScript snippets with imports (defineAction+Zod, Drizzle/parameterized SQL, ssrfSafeFetch, resolveCredential, accessFilter, authorize, needsApproval) cover the common cases with fully executable guidance. | 5 / 5 |
Workflow Clarity | The custom-route section gives a clear numbered 1-2-3 sequence, CI guards and db-check-scoping provide validation checkpoints, and a closing Checklist verifies the rules; most of the skill is a rules catalog rather than a sequenced workflow, so it stops short of the explicit validate-fix-retry loops at 5. | 4 / 5 |
Progressive Disclosure | Good section headers organize the content and related skills are signaled at the end, but no bundle files exist and substantial content that could live in separate references (Same-Origin Workspace Apps deep-dive, detailed guard-script internals, historical credential-leak context) is inlined in a single ~340-line file. | 3 / 5 |
Total | 15 / 20 Passed |