Content
88%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.
Excellent instructional content: concrete, executable, and grounded in this repo's measured history, with a genuine validation workflow for proving CI changes. The one liability is embedded time-sensitive facts (dates, plan quotas, timings) that will rot and are not isolated into a maintenance section, plus a rules table dense enough that a reference split could help.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and repo-specific with no padding and no explanation of concepts Claude already knows — every line is a decision input (scripts, functions, YAML patterns, measured timings). It misses the top anchor because time-sensitive specifics ('on 2026-09-29 the Fast tests gate waited a median 445 s', 'moved to Enterprise Cloud in October 2026... 500 jobs') are woven into normative sections rather than isolated in an old-patterns/deprecated section, and these could be trimmed or consolidated. | 4 / 5 |
Actionability | Fully executable guidance throughout: exact commands ('node --experimental-strip-types --test scripts/ci-change-scope.test.ts scripts/ci-build-workspaces.test.ts scripts/ci-test-lanes.test.ts', 'actionlint', 'oxlint --quiet'), copy-paste YAML ('group: <name>-${{ github.event.pull_request.number || github.ref }}' with 'cancel-in-progress: true'), and named implementation hooks ('requiresFullCoreFastTests' in scripts/ci-test-lanes.ts, 'splitLargePackages', 'resolveMaxWorkers', 'CHECK_NAMES'). Specific examples cover the common cases, including running each snippet 'against a matching input and an empty one'. | 5 / 5 |
Workflow Clarity | 'Proving a CI change' is an explicit validation checklist (classifier test cases for on/off/full-run states, real-history runs with GITHUB_OUTPUT, actionlint on the workflow) that even addresses the self-test gap ('a PR that edits ci.yml runs full CI itself, so its own checks cannot show the saving'). Failure-safety rules are explicit: 'When scope cannot be computed, run everything or fail. Never skip.' — clear sequence with checkpoints and error-handling policy. | 5 / 5 |
Progressive Disclosure | A single-file skill (no references/, scripts/, or assets/ bundle dirs) with clear section headers and a well-organized rules table; nothing is buried and there are no nested references. It misses 5 because at 127 lines the 10-row CI-job rules table carries long narrative 'why' columns that could plausibly live in a reference file, and the skill is well past the under-50-line simple-skill case. | 4 / 5 |
Total | 18 / 20 Passed |