Content
92%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 exemplary scaffold skill: dense with project-specific knowledge, correctly sequenced with validation and a checklist, and focused on the real footgun (four registration sites). The only soft spot is the thin permissions step and placeholder endpoint mapping, which keep actionability just below fully executable.
Suggestions
Add a minimal {Name}Permissions skeleton (one resource class + All collection) to Step 2 so the permissions step is executable without opening CatalogPermissions.
Show one concrete endpoint registration example (e.g. a MapGet with the version-set group) instead of only the commented `// group.MapCreate{Entity}Endpoint();` placeholder.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence: no basic concepts are explained, code comments carry only non-obvious rules ("MUST be last — applies tenant + soft-delete filters"), and every section adds project-specific knowledge (e.g. the module load-order table). | 5 / 5 |
Actionability | Mostly executable guidance: full copy-paste-ready code for the module class and DbContext, exact `dotnet sln` and `dotnet test` commands, and a concrete copy-don't-write rule for the .csproj files. Minor gaps remain — the permissions section is a one-line pointer with no example shape, and endpoint mapping is left as a commented placeholder (`// group.MapCreate{Entity}Endpoint();`). | 4 / 5 |
Workflow Clarity | Seven clearly sequenced steps with an explicit verification step (`dotnet build` expecting 0 warnings, Architecture.Tests "must pass"), a closing checklist, and failure-mode diagnosis for the riskiest operation ("Miss the Mediator marker → handlers silently undiscovered"). The destructive/batch validation cap does not apply since validation is present. | 5 / 5 |
Progressive Disclosure | No bundle files exist, and the ~130-line body is appropriately kept in a single file: well-organized sections, with detail delegated one level deep via clearly signaled pointers to `create-migration`, `.agents/rules/architecture.md`, and `.agents/rules/eventing.md`, plus "mirror the shape of `CatalogPermissions`" instead of inlining an example. | 5 / 5 |
Total | 19 / 20 Passed |