Content
71%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 dense, highly actionable pattern catalogue with lean code examples covering unit, property-based, mocking, and integration testing, plus concrete CLI commands. Its main weakness is structure: everything is inlined in one long SKILL.md with no reference files, and no sequencing guidance connects the multiple activation scenarios.
Suggestions
Split the FsCheck deep-dive (custom generators, property patterns) and the ASP.NET Core integration-testing section into one-level-deep reference files (e.g. references/property-testing.md, references/integration-testing.md), keeping only quick-start examples inline.
Remove the redundant opening paragraph ('Comprehensive testing patterns for F# applications...') that restates the frontmatter description, and drop helper-dependent snippets' reliance on undefined values like validItem/validRequest by defining them once in the Test Organization section.
Add a short ordered workflow for the 'setting up test infrastructure' activation case (create test project → add packages → add helpers → run) so the multiple When-to-Activate paths have explicit sequencing.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly every section is executable code or a dense reference table with no teaching of basics Claude already knows; the one prose explanation (Unquote's quotation-based failure messages) is genuinely non-obvious. Minor padding remains (the intro sentence duplicates the frontmatter), so anchor 4 fits rather than the fully-lean anchor 5. | 4 / 5 |
Actionability | Provides extensive concrete guidance across all major cases: xUnit Facts/Theories, FsCheck properties with custom generators, NSubstitute, WebApplicationFactory, and fully-executable `dotnet test` commands. Falls short of anchor 5 because snippets depend on undefined helpers ("validItem", "validRequest", "testData") and the WebApplicationFactory example lacks its open statements. | 4 / 5 |
Workflow Clarity | The "When to Activate" section, task-typed sections, and a complete "Running Tests" command reference make navigation unambiguous, and no destructive/batch operations exist to require validation checkpoints. Not anchor 5 because multiple activation paths (writing tests, setting up infrastructure, debugging flaky tests) are never sequenced relative to each other. | 4 / 5 |
Progressive Disclosure | Sections are well-organized, but the ~280-line body keeps everything inline with no bundle files at all — the FsCheck deep-dive and ASP.NET Core integration-testing material are natural candidates for one-level-deep reference files. This matches anchor 3 (content that should be separate is inline) rather than anchor 4 (content appropriately split with clear references); not anchor 2 because internal structure and navigation are clear. | 3 / 5 |
Total | 15 / 20 Passed |