Content
93%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 lean, highly actionable rulebook for the F#/C# boundary: exact settings, signatures, and attributes, with executable examples correctly split into a one-level-deep reference file. The only gap is the absence of an explicit verification step (build/test run) in the code-writing workflow; the review workflow is fully specified with a checklist and output format.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Bullet-form rules with terse, load-bearing rationales for non-obvious facts ("C# cannot use `with` on an F# record", "F# does not write the nullness of a byref element into the metadata") and no tutorial padding or explanation of concepts Claude already knows. The few explanatory sentences (e.g. why `<Nullable>enable</Nullable>` matters, why `%A` is not AOT-safe) carry project-specific information that cannot be assumed, so this fits anchor 5 rather than 4. | 5 / 5 |
Actionability | Gives copy-paste-ready guidance: the exact .fsproj property block, exact member signatures ("static member TryParse(text, out result, out failure) : bool"), exact attributes ("[<Out; MaybeNullWhen(false)>]"), concrete surface types (IReadOnlyList<T>, ImmutableArray<T>), and an exact review command with output format ("Report each finding as `path:line: rule N: problem. fix.`"). Nothing is pseudocode and the common cases are covered, matching anchor 5; bulk executable examples are appropriately delegated to the verified references/patterns.md. | 5 / 5 |
Workflow Clarity | "Before you start" gives a clear 3-step sequence, and the Review checklist plus Review mode (exact `git diff` command, most-severe-first ordering, defined output format) validate the review path; however the write path has no explicit build/test verification step — validation is only implicit ("A unit test calls `Create` for each literal"). This fits anchor 4 (clear sequence, minor validation gaps) rather than 5, which requires an explicit verify-and-fix loop. | 4 / 5 |
Progressive Disclosure | The body states the rules and delegates all code to references/patterns.md, which exists in the bundle, is exactly one level deep, and is well signaled twice ("Read that file before you write new boundary code" and in Related). No bulk content is inlined that belongs in the reference, and the Related section navigates onward to repo docs — matching anchor 5. | 5 / 5 |
Total | 19 / 20 Passed |