Content
81%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 strong, disciplined migration runbook: strict parity policy, gated 12-step workflow, and concrete repo-derived pitfalls. Main gaps are repetition of the drift rules across sections and the absence of a single illustrative code example, with some detailed material that belongs in the reference file inlined instead.
Suggestions
Consolidate the drift/approval rules stated in Migration Policy, Translation Rules, step 9, and the Output Checklist into one authoritative section to remove redundancy and tighten conciseness.
Add one short, complete DMR controller example (Controller with Body[...] DTO and a ResponseSpec extra response) so the translation rules and error strategy have a copy-paste-ready anchor.
Move the repo-derived 'Migration Pitfalls' list (and optionally the Output Checklist) into references/ninja-to-dmr-map.md or a dedicated pitfalls reference, keeping SKILL.md as a leaner overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is terse, bullet-driven, and assumes Django/Pydantic knowledge (no concept explanations), but drift policy is restated across Migration Policy, Translation Rules, step 9, and the Output Checklist, and step 9 mixes response-validation directives into an auth/throttling step. Anchor 5 ('every token earns its place') doesn't hold due to this repetition; it is well above anchor 3 since there is no padded explanation. | 4 / 5 |
Actionability | Guidance names exact constructs to find ("NinjaExtraAPI", "http_get/http_post/..."), exact mappings ("Body[...]", "Query[...]", "Path[...]"), and concrete pitfalls with fixes ("use ratelimit.core.ALL", "field-level strict=False", "extra='forbid'"). It falls short of anchor 5 because there is no illustrative code snippet (e.g., a minimal DMR controller) and CI validation says 'run the same commands CI uses' without an example invocation; for an instruction-only migration skill the specifics otherwise carry it above anchor 3. | 4 / 5 |
Workflow Clarity | A 12-step sequence with explicit gates and feedback loops: "Decide error strategy (required gate)", per-slice validation via repository-native CI (step 11), and a finish gate ("Do not mark slice done until: linters pass, tests pass... or failures are explicitly approved drift"), plus a required 3-section report and output checklist. This matches the anchor 5 example's validate-fix-retry structure with explicit checkpoints. | 5 / 5 |
Progressive Disclosure | Clear section headers and a well-signaled, real, one-level-deep reference ([references/ninja-to-dmr-map.md](references/ninja-to-dmr-map.md)) plus the DMR LLM docs URL. Not anchor 5 because the ~190-line body inlines substantial detail (repo-derived pitfalls, the full checklist, DTO translation rules) that could partly live in the reference file, leaving SKILL.md closer to an overview. | 4 / 5 |
Total | 17 / 20 Passed |