Content
86%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 a highly actionable pattern reference — exact parameter names, settings flags, commands, and copy-adaptable code — organized into a clean overview with a one-level-deep bundle of seven real reference files navigated via the Quick reference table. The main weaknesses are token-level redundancy between the Non-negotiable rules and the Review checklist, and a workflow whose verification step lacks an explicit fix-and-retry feedback loop.
Suggestions
Trim the duplication between 'Non-negotiable rules' and 'Review checklist': several checklist items (parsed_* names, Body[Model] | None, concrete_views subclassing, as_view ignoring throttling=/auth=, dmr.routing.path, dmr_rf/dmr_client) restate rules from the same document — keep only review-specific flags or compress the restated ones to one-line pointers.
Add a feedback loop to the Workflow's Verify step, e.g. 'If check, the type checker, or tests fail, fix the flagged rules and re-run before exporting the schema' — this would also satisfy the error-recovery pattern the strongest workflows follow.
Deduplicate the docs-loading bullets: 'index: .../llms.txt' and 'everything at once: .../llms.txt' point to the same URL — either differentiate them (e.g. per-page .md URLs vs. the full bundle) or merge them into one bullet.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes Claude's competence — no basic Django/OpenAPI concept explanations, every rule is library-specific with exact names like 'parsed_body', 'Body[UserModel | None]', 'validate_responses'. It falls short of anchor 5 only through minor redundancy: roughly eight Review-checklist items restate Non-negotiable rules from the same document (parsed_* names, concrete_views subclassing, as_view ignoring throttling=/auth=, dmr.routing.path, dmr_rf/dmr_client), and the docs-loading bullets list the same llms.txt URL twice under 'index' and 'everything at once'. | 4 / 5 |
Actionability | Fully concrete guidance throughout: executable code snippets ('class UserController(Controller[MsgspecSerializer]): def post(self, parsed_body: Body[UserModel]) -> UserModel', the APIError raise, the concrete_views path(...) routing), exact commands ('uv pip show django-modern-rest', 'python manage.py check', 'python manage.py dmr_export_schema <path>'), and precise names for parameters, settings, serializers, and fixtures. Not a 4 because there are no missing key details for the common cases the body covers; full correct/wrong examples are correctly delegated to the reference files. | 5 / 5 |
Workflow Clarity | The 4-step Workflow is clearly sequenced with a concrete verification checkpoint (step 4: 'run python manage.py check, the project's type checker, and tests... run python manage.py dmr_export_schema <path> and review the diff') and a review checklist for complex work. It sits at 4 rather than 5 because the anchor-5 example includes an explicit error-recovery feedback loop ('If errors: fix and re-validate'), which this workflow omits — it states what to verify but not what to do when verification fails. | 4 / 5 |
Progressive Disclosure | Clear overview with well-signaled, one-level-deep references: the Quick reference table maps each topic to a real bundle file (all seven references/*.md files exist, each linked with a rule count and topic description), the essential 'Non-negotiable rules' stay in SKILL.md while full rules with correct/wrong examples live one level deep, and reference files link only back to SKILL.md and to external docs rather than nesting further. Not a 4 because navigation is complete and nothing that belongs in a separate file is inlined. | 5 / 5 |
Total | 18 / 20 Passed |