Content
75%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 highly actionable, well-structured skill body: every command is executable, parameters are tabulated, and troubleshooting covers realistic failure modes with recovery steps. The main costs are redundant sections (duplicated examples, repeated platform constraints) and a small path mismatch between the body and the actual bundle location.
Suggestions
Delete the 'Example Usage' section or fold its unique case (multi-category TestFilter) into the 'Scripts' section to remove verbatim command duplication.
State platform requirements once (the categories table already encodes them) instead of repeating them in the compatibility line and 'Prerequisites'.
Fix the script path references to match the bundle layout ('scripts/Run-IntegrationTests.ps1'), or move the iOS/sequential-run walkthroughs into a reference file to keep SKILL.md lean.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly dense and useful (tables, commands, troubleshooting), but there is real redundancy: the 'Example Usage' section repeats two commands verbatim from the 'Scripts' section, and platform requirements are stated three times (compatibility frontmatter, categories table, prerequisites). It is not a 4 because the duplication is structural rather than a couple of stray sentences. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready pwsh commands throughout, a complete parameter table with defaults, concrete test names (e.g., 'RunOniOS_MauiDebug'), and exact recovery commands for locked folders and simulator errors. Specific examples cover the common cases. | 5 / 5 |
Workflow Clarity | A clear 4-step workflow with concrete commands, an explicit provisioning verification block ('ls .dotnet/dotnet'), and a rich troubleshooting table with error-recovery feedback loops. Not a 5 because the main test-run workflow lacks an explicit validation checkpoint that build/install succeeded before running tests, relying on the script's internal behavior. | 4 / 5 |
Progressive Disclosure | Well-sectioned with headers, tables, and a single bundle script (scripts/Run-IntegrationTests.ps1) that is referenced by path from the body, so everything is one level deep. Not a 5 because the body references the script under '.github/skills/run-integration-tests/scripts/' rather than the bundle's actual 'scripts/' location, and the iOS/sequential-run sections could live in a reference file. | 4 / 5 |
Total | 16 / 20 Passed |