Content
65%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.
The body delivers excellent executable code for the common cases and keeps the overview lean, but it is dragged down by a padded Core Principles section, a missing migration workflow with validation for schema changes, and — most seriously — a referenced reference.md that does not exist in the bundle, breaking the disclosure chain for all advanced detail.
Suggestions
Add a short migration workflow with a validation checkpoint, e.g. generate (scripts/new-migration.sh), review the generated SQL, run via DataSource migration API, then verify with migration:show or a query against the migrations table.
Ship the referenced reference.md (or fix the links) — the body delegates primary-key options, relation types, repository methods, query builder, transactions, and framework integration to a file absent from the bundle.
Cut the role-statement line and the Core Principles platform list ('Works in Node.js, Browser, Ionic, Cordova, React Native, NativeScript, Expo, and Electron'); neither changes what Claude should do.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient (tight tsconfig block, one representative entity, one DataSource config, bullet Quick Guidance), but the 'You are an expert...' role line and the Core Principles section — 'Works in Node.js, Browser, Ionic, Cordova, React Native, NativeScript, Expo, and Electron', 'First-class support for database migrations' — are brochure filler Claude does not need. This is a distinct padded section beyond anchor 4's 'minor instances of over-explanation', matching anchor 3. | 3 / 5 |
Actionability | Everything instructs with runnable artifacts: a complete tsconfig JSON, a fully imported executable entity example, a copy-paste DataSource configuration with env handling, concrete repository method names ('find, findOne, create, save, update, delete, and softDelete'), and two scaffold scripts with usage lines. Matches the score-5 anchor: copy-paste ready and covering the common cases. | 5 / 5 |
Workflow Clarity | Prescriptive rules are clear ('Always release() a QueryRunner in a finally block', 'Never use synchronize: true in production'), but the skill covers migrations and schema changes — database operations the rubric's feedback-loop note flags — without any generate → review → run → verify sequence or validation checkpoint. Per the cap guidance for database operations without validation, this cannot exceed 3; it is not 2 because the guidance that exists is coherent and specific. | 3 / 5 |
Progressive Disclosure | The split is well intentioned (lean overview delegating detail to reference.md and two scripts), but the central reference file is missing from the bundle: 'see [reference.md](reference.md)' is promised three times as the home for relation types, the query builder, transactions, and framework integration, yet no reference.md exists — only scripts/new-entity.sh and scripts/new-migration.sh. The main navigation target is broken, which exceeds anchor 4's 'minor organization gaps' and fits anchor 3's structure-with-breakdown. | 3 / 5 |
Total | 14 / 20 Passed |