Content
96%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 dense, executable recipe: numbered steps, exact code, failure-mode mapping, and a validation section, with no filler. Its only structural weakness is inlining serialization-rule detail that duplicates the canonical AGENTS.md reference instead of splitting it out.
Suggestions
Move the detailed Serialization/AOT rules that duplicate sdks/dotnet/AGENTS.md into a short reference file (e.g. references/serialization-rules.md), keeping only the task-critical non-negotiables inline and pointing to AGENTS.md as canonical.
Add an explicit feedback loop to the Validate section, e.g. "if dotnet build fails on PublicAPI analyzer errors, add the missing members to PublicAPI.Unshipped.txt and rebuild".
Consider moving the PublicAPI.Unshipped.txt line-format block to the same reference file, keeping one representative example inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | No basic-concept explanations; every section is operational recipe material and assumes C#/System.Text.Json competence. The only narrative passage (the NullOmissionTest rationale) justifies a rule that guards a real regression, so it earns its tokens. | 5 / 5 |
Actionability | Gives a numbered 6-step recipe with exact file paths, sealed-class conventions, copy-paste converter switch arms, exact PublicAPI.Unshipped.txt line formats, a round-trip test example, and dotnet build/test commands — covering event, message, and input-content cases. | 5 / 5 |
Workflow Clarity | Clear sequence with an explicit four-spot checklist mapping each miss to its failure mode (build fails vs silent deserialize failure), a mandatory round-trip test, and a Validate section with build and test commands serving as recovery feedback. | 5 / 5 |
Progressive Disclosure | Well-sectioned body with one clearly signaled external reference (sdks/dotnet/AGENTS.md as canonical), but the inline Serialization/AOT rules and PublicAPI block duplicate canonical AGENTS.md material rather than splitting detail into a reference file — a minor organization gap. | 4 / 5 |
Total | 19 / 20 Passed |