Content
38%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 padded persona résumé: a long catalog of well-known .NET technologies and generic behavioral traits with boilerplate template sections ('Clarify goals, constraints, and required inputs'), plus a dangling reference to a nonexistent resources/implementation-playbook.md. Its one strong element is the concrete, executable Code Style Preferences example, which partially redeems actionability.
Suggestions
Fix the broken reference: either create references/implementation-playbook.md (using the standard references/ directory) and point detailed examples there, or remove the pointer entirely.
Cut the Capabilities, Behavioral Traits, Knowledge Base, and Example Interactions catalog drastically — Claude already knows these technologies — keeping only the project-specific preferences (e.g., Result pattern, preferred test stack, code style) that differ from Claude's defaults.
Replace the generic 'Instructions' boilerplate and vague 'Response Approach' steps with a concrete workflow with validation checkpoints (e.g., how to verify EF query performance, what to check before recommending an architecture).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body (~190 lines) is dominated by an encyclopedic catalog of technologies and practices Claude already knows ('Modern C# features (12/13): required members, primary constructors', 'IMemoryCache for in-process caching', 'xUnit test framework', 'Docker containerization') plus behavioral-trait padding ('Favors composition over inheritance', 'Applies SOLID principles pragmatically'). This matches anchor 2 ('Noticeably verbose; several unnecessary explanations or padded sections') — several sections, not just one or two, earn nothing; it stops short of anchor 1 only because the Code Style Preferences section conveys genuine project-specific preference. | 2 / 5 |
Actionability | Most of the body is abstract capability listing with no steps or commands ('Writes idiomatic, modern C# code', 'Consider edge cases including error handling and concurrency'), but the Code Style Preferences section provides one complete, executable C# example demonstrating concrete preferences (primary constructors, records, switch expressions, Result pattern). This straddles anchors 2 and 3 — it has some real concrete guidance, but it is incomplete and isolated, so 3 is the best fit. | 3 / 5 |
Workflow Clarity | The 'Response Approach' section lists a clear 8-step sequence (understand requirements → design architecture → implement → optimize → ensure testability → document → consider edge cases → review security), but the steps are generic and include no validation checkpoints or feedback loops ('Apply relevant best practices and validate outcomes' is not a checkable step). This matches anchor 3 ('Steps listed but validation gaps; sequence present but checkpoints missing or implicit'). | 3 / 5 |
Progressive Disclosure | The only reference in the body is 'If detailed examples are required, open resources/implementation-playbook.md' — a path that does not exist in the bundle (no resources/, references/, scripts/, or assets/ directories), and the standard references/ directory is not used. Meanwhile, the large capability catalog that belongs in a separate reference file is fully inlined. This matches anchor 2 ('Minimal structure; content that clearly belongs in separate files is inlined; or references are buried'), aggravated by the broken reference path. | 2 / 5 |
Total | 10 / 20 Passed |