Content
87%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is an efficient, actionable index with excellent progressive disclosure of detail into reference files. The only gap is that the destructive database/migration workflow lacks an explicit inline validation feedback loop, capping workflow clarity at 2.
Suggestions
Add a short inline validate-to-fix-to-retry note for the migration/database path (e.g. run `uv run pytest tests/unit/server/ -n auto` after a migration, and re-run `make typecheck-python` before committing) rather than relying solely on references/database-patterns.md.
Surface the `make check-graphql-permissions` CI guard as an explicit local pre-commit checkpoint in the 'Everyday Commands' or 'Hard Rules' section so the feedback loop is runnable before push.
Consider a one-line 'Before you finish' checklist (format → typecheck → relevant tests) to make the validation sequence explicit for multi-step backend changes.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean — a command table, an annotated directory tree, a task-to-reference routing table, and tightly-scoped rule sections — with no general-concept filler; it assumes Claude's competence and every line earns its place. | 3 / 3 |
Actionability | Provides copy-paste-ready commands ('make dev-backend', 'uv run pytest ... -n auto', 'make graphql'), a concrete directory map, a routing table, and an actionable hard rule ('MUST be a @strawberry.mutation with permission_classes=[...]'), matching the fully-executable anchor. | 3 / 3 |
Workflow Clarity | Task routing and validation commands (check-graphql-permissions, typecheck-python) are present, but the risky migration/database path is delegated to an external reference without an inline validate-to-fix-to-retry feedback loop, leaving checkpoints implicit rather than explicit. | 2 / 3 |
Progressive Disclosure | SKILL.md is a clear overview that routes tasks to four verified, one-level-deep reference files (graphql-patterns, test-patterns, llm-trace-tests, database-patterns), with content appropriately split and well-signaled navigation. | 3 / 3 |
Total | 11 / 12 Passed |