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.
A tight, well-structured testing guide that assumes competence and provides actionable examples across the common test types. The main gap is template placeholders in the handler example that prevent fully copy-paste-ready code.
Suggestions
Replace '_db = /* in-memory or test DbContext */' with a concrete in-memory DbContext setup snippet so the handler example is copy-paste ready.
Add a brief 'verify' loop after the Run section (e.g., run the new test, fix failures, keep Architecture.Tests green) to make the workflow feedback loop explicit.
Briefly justify the {Entity}/{X} placeholder convention so the template flexibility is documented rather than inferred.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean throughout: it states the stack, lists conventions, gives code examples, and run commands without explaining concepts Claude already knows; every section earns its place. | 5 / 5 |
Actionability | Provides concrete, mostly executable code for handler/validator/entity/service-dep tests and real dotnet commands, but template placeholders like {Entity}, {X}DbContext, and '_db = /* in-memory or test DbContext */' require filling in. | 4 / 5 |
Workflow Clarity | Clear structure from conventions to per-type examples to architecture guardrails and a Run section with verification, but lacks an explicit write-test -> run -> fix-failures feedback loop. | 4 / 5 |
Progressive Disclosure | Well-organized sections keep the quick-reference inline while offloading detailed rules to clearly signaled one-level-deep references (.agents/rules/testing.md and integration-testing.md). | 5 / 5 |
Total | 18 / 20 Passed |