Content
92%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 tightly written, expert-level migration procedure: lean prose, exact class and settings names, an explicitly ordered flow sequence with required gates, per-flow and final validation, a mandated 4-section report, and clean separation of policy from lookup tables in a verified one-level-deep reference. The main gap is the absence of any syntax examples for the dmr constructs (Controller/DTOs, CookieSpec, external_path), leaving those details to the linked external docs.
Suggestions
Add a minimal worked example of a dmr Controller with input/output TypedDict DTOs for the rebuilt user-details endpoint, since that is the one flow with no allauth equivalent and no inline example.
Show a one-line usage example of CookieSpec / NewCookie with the preserved flags (httponly, secure, samesite, path) and of external_path() for exposing allauth endpoints, so cookie preservation and OpenAPI wiring are copy-paste ready rather than doc lookups.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~220-line body is dense with non-obvious, decision-critical guidance (behavioral vs transport parity, security posture rules, drift accounting, pitfalls) and explains nothing Claude already knows about Django or DRF; the architectural framing that allauth keeps owning account flows is exactly the kind of context Claude lacks. Only one or two reassurance sentences ("This is normal, and it is the right place...") could be trimmed, which does not reach the 4 anchor's 'minor instances of over-explanation'. | 5 / 5 |
Actionability | Quotes: "Add `allauth`, `allauth.account`, `allauth.headless` to `INSTALLED_APPS`", "`XSessionTokenSyncAuth`", "`HeaderJWTSyncAuth`", "`@sensitive_post_parameters`", "Return `NO_STORE_HEADERS` from `dmr.security` in `@modify`". Guidance names exact classes, settings keys, decorators, and an ordered 8-flow sequence, but provides no syntax examples for the dmr constructs it invokes (`Controller` with typed DTOs, `CookieSpec`, `external_path()`), delegating those to the external docs — fitting 'mostly executable guidance; concrete code or commands with minor gaps' rather than the 5 anchor's copy-paste-ready examples. | 4 / 5 |
Workflow Clarity | Quotes: "Migrate one flow at a time, in this order, because later flows depend on being able to log in", "(required gate)", "then run the repository's checks", "Do not mark a flow done until linters pass, tests pass, the client is updated, and the report is updated", plus the Output Checklist. The sequence is explicit with rationale, two required gates act as checkpoints, validation is mandated per flow and at the finish gate, and a final checklist covers the complex process — matching the 5 anchor's explicit validation steps and checklists. | 5 / 5 |
Progressive Disclosure | The body keeps procedure and policy in SKILL.md while all lookup tables (endpoint, MFA, social, auth-class, settings, serializer maps) live in the single one-level-deep reference [references/dj-rest-auth-to-dmr-map.md], which exists in the bundle, is clearly signaled in the required-docs list, and is pointed to twice more by name ("See the settings table in the local map", "See the auth table in the local map"). The reference chains no further files, and the split is appropriate for navigation — matching the 5 anchor. | 5 / 5 |
Total | 19 / 20 Passed |