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.
The body is an efficient, highly actionable reference: concrete per-option commands for both platforms, a clear tool-vs-script preference order, and useful repo-specific pitfalls (compilation prerequisite, integration-test exclusion). The main gaps are the conceptual-only runTests example and the absence of an explicit verification step for the compilation requirement.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean: terse bullet guidance, copy-paste command blocks per option, and no explanation of concepts Claude already knows (it never explains what unit tests or Mocha are). Not anchor 5 because of minor redundancy — the multiple-files case is demonstrated twice (bare paths and repeated --run) and both the bare-path and --run sections cover overlapping ground that could be trimmed. | 4 / 5 |
Actionability | Every shell option has a fully executable, copy-paste-ready example for both platforms (e.g., `./scripts/test.sh --run src/vs/editor/test/common/model.test.ts --grep "should split lines"`). Not anchor 5 because the runTests tool section provides only an "Example (conceptual)" in prose with no actual tool invocation, a pseudocode stand-in without explicit justification. | 4 / 5 |
Workflow Clarity | A clear decision sequence is present: prefer runTests, fall back to platform scripts when unavailable, use test-integration scripts for integration tests, and ensure compilation before running ("Ensure the `VS Code - Build` watch task is running... Test failures caused by stale output are a common pitfall"). Not anchor 5 because the compilation prerequisite is stated as a caution rather than an explicit verification step or command, and there is no error-recovery loop for stale-output failures. | 4 / 5 |
Progressive Disclosure | The body is well organized with clear headers (Preferred tool / Fallback scripts / per-option sections / Integration tests / Compilation requirement) and the one external reference ("See the `integration-tests` skill") is clearly signaled and one level deep. Not anchor 5 because the option reference is entirely inline with no split of detailed material, and the referenced integration-tests material lives outside this skill without further navigation structure. | 4 / 5 |
Total | 16 / 20 Passed |