Content
93%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.
An excellent instruction-style skill body: dense with project-specific, directly actionable rules and named helper APIs, lean with no over-explanation, and well-organized with sensible delegation to related skills. The only notable gap is the absence of any verification checkpoint for confirming that access controls were applied correctly.
Suggestions
Add a brief closing verification step, e.g., 'Before finishing, confirm every read/write path for the changed table passes through the owner filter or accessFilter helper.'
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly every sentence is a project-specific rule (Zod schemas, Drizzle parameter binding, `ownableColumns()`, `ssrfSafeFetch`, `needsApproval`) rather than explanation of concepts Claude already knows; the only background clause, "React escapes rendered text", is three words serving as rationale for the rule that follows. No padded or repetitive sections exist, matching the lean-and-efficient anchor rather than the minor-trimming-needed level of 4. | 5 / 5 |
Actionability | Guidance is concrete and executable throughout: named APIs with signatures (`accessFilter(table, sharesTable)`, `resolveAccess(type, id)`, `assertAccess(type, id, role)`, `ssrfSafeFetch` from `@agent-native/core/extensions/url-safety`), an import path, specific anti-patterns to avoid, and a decision table mapping data patterns to access models. Per the instruction-skill scoring note, the absence of code blocks is not penalized since the guidance is specific and directly actionable, placing it at the top anchor rather than the minor-gaps level of 4. | 5 / 5 |
Workflow Clarity | The body is organized as a clear conditional decision procedure (validate inputs -> pick an access model from the table -> apply sharing helpers -> handle untrusted content -> protect credentials/approvals) with unambiguous branch guidance. It stops short of 5 because no validation/verification checkpoints tell Claude how to confirm a change is secure (e.g., re-checking that every read/write path passes the access filter), though as design-time guidance it involves no destructive or batch operations that would trigger the workflow-clarity cap. | 4 / 5 |
Progressive Disclosure | The single-file body is well-sectioned and appropriately sized for inline rules, and it correctly delegates deeper material to sibling skills ("Use the shared credential flow in the `secrets` skill"; a Related skills section listing `actions`, `secrets`, `sharing`) instead of inlining it. There are no bundle files to mis-organize, no buried references, and no monolithic wall of text, matching the well-organized single-file pattern the rubric treats as full marks. | 5 / 5 |
Total | 19 / 20 Passed |