Content
100%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.
An exceptionally well-structured rule skill: terse, project-specific rules with exact signatures and attributes, an executable settings block, and a complete review workflow with checklist and reporting format. Bundle structure follows progressive disclosure correctly, externalizing code examples to a verified one-level-deep reference with clear read-timing guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is a dense rule catalog with no padding: it never explains what F#, C#, or interop are, and every non-obvious statement earns its place (e.g. "F# does not write the nullness of a byref element into the metadata, so C# reads the value as never null"). Project-specific context ("morphir-dotnet writes domain types in F# and keeps the CLI edge in C#") is exactly what Claude could not know otherwise. | 5 / 5 |
Actionability | Guidance is fully executable: a copy-paste `.fsproj` property block, exact member signatures ("static member TryParse(text, out result, out failure) : bool"), exact attribute patterns ("[<Out; MaybeNullWhen(false)>]"), exact commands ("git diff <base>...HEAD -- '*.fs' '*.fsi' '*.fsproj' '*.cs'"), and a ready-to-use subagent prompt with an exact output format ("path:line: rule N: problem. fix."). It is not a score above 5, and it exceeds anchor 4 because concrete examples cover the common cases via references/patterns.md, which the body states compile. | 5 / 5 |
Workflow Clarity | "Before you start" is a clear 3-step sequence with an explicit checkpoint ("Check the project settings (rule 1). Without them, the C# compiler cannot see F# nullness"), and the review workflow ships a full checklist, a severity ordering ("most severe first"), a reporting format, and the no-findings case ("If there is no finding, say so"). Not the level below (4) because a validation checklist and reporting loop are explicitly present, not merely implied. | 5 / 5 |
Progressive Disclosure | The split is clean: rules live in SKILL.md and code examples live in references/patterns.md (verified to exist, 228 lines, one level deep, linking only back to ../SKILL.md). The reference is well signaled twice with read timing ("Read that file before you write new boundary code"), and Related points to repo docs without nesting. Not anchor 4 because there are no organization gaps — navigation and placement are both explicit. | 5 / 5 |
Total | 20 / 20 Passed |