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 is highly actionable and mostly concise, with executable examples across the core TypeORM patterns, but it inlines a large amount of content that could be split into reference files and provides no validation checkpoints for its destructive database operations.
Suggestions
Add explicit validation/recovery steps around migrations and schema commands (e.g. run `migration:show` after `migration:run`, and warn/confirm before `schema:drop`).
Define the missing Profile entity and add it to the project structure, or remove references to it.
Move the full entity/repo/subscriber/transaction code into reference files (e.g. references/entities.md) and keep SKILL.md as an overview with one-level-deep links.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly lean, code-forward content that does not re-explain TypeORM basics, but the prerequisites are duplicated (frontmatter vs body) and the migration commands repeat across "Database Initialization" and "Common Commands" sections. | 4 / 5 |
Actionability | Provides complete, copy-paste-ready TypeScript for DataSource, entities, repository, subscriber, and transactions, but references a Profile entity that is never defined and absent from the project structure listing. | 4 / 5 |
Workflow Clarity | Scaffold/init/migration commands are sequenced, but destructive operations (schema:sync, schema:drop) and migrations lack explicit validation checkpoints or feedback loops, which caps workflow clarity at 3 per the destructive-operations guideline. | 3 / 5 |
Progressive Disclosure | Sections are clearly headed and well-organized, but at ~380 lines everything is inlined in SKILL.md with no reference files; entity/repo/subscriber examples that belong in separate files are kept inline. | 3 / 5 |
Total | 14 / 20 Passed |