Content
67%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 well-organized, highly prescriptive policy skill with executable code examples and specific API and path-level guidance. Its weaknesses are redundancy — an export table that restates the code example above it — and two structural ambiguities: a "Never" section containing no prohibitions, and raw-SQL guidance split between an avoid-list and a helper subsection.
Suggestions
Delete the export table under the Drizzle example — it restates what the code and its inline comments already show, saving ~10 lines of pure padding (conciseness).
Rename or rewrite the "### Never" section under Database so it either lists actual prohibitions or is folded into the surrounding guidance; readers scanning it expect a prohibition list (conciseness, workflow clarity).
Align the query-builder example with the schema snippet (define ownerEmail in the meals table or drop it from the .where() clause) so the example is fully copy-paste ready (actionability).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The export table ("`pgTable` Defines a PostgreSQL table", "`text` Defines a text column") duplicates what the preceding code example already demonstrates, and the "### Never" heading under Database introduces no prohibitions — just prose about docs wording. The rest is tight and prescriptive, but these padded spots keep it from 4. | 3 / 5 |
Actionability | Concrete and executable throughout: a complete Drizzle schema snippet, a query-builder example, named H3 APIs ("defineEventHandler, readBody, getMethod, setResponseHeader"), and precise paths ("packages/core/src/templates/"). Held back from 5 because the query example uses "meals.ownerEmail", which is not defined in the shown schema snippet. | 4 / 5 |
Workflow Clarity | This is a rule/policy skill, not a multi-step process, and its directives are unambiguous ("Never use fs, child_process, or path in server routes and plugins"); the single-action exception applies. Not 5 because the mislabeled Database "Never" section and the tension between "Avoid db.execute(...)" and the "Raw SQL helpers" subsection that documents getDbExec() create mild ambiguity about where raw SQL is permitted. | 4 / 5 |
Progressive Disclosure | A single-file skill with no bundle files, organized under clear section headers (Rule, Database, Hosting Agnostic, Related Skills) with useful cross-skill pointers. At ~110 lines it exceeds the under-50-line simple-skill threshold, and all content lives inline — appropriate here, but it earns 4 rather than 5. | 4 / 5 |
Total | 15 / 20 Passed |