Content
85%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 operational, expert-level body: concrete commands, exact check names, explicit confirmation checkpoints, and approval-gated risky actions make it immediately executable, with only minor redundancy across sections. Its main structural weakness is that its own referenced file (references/master-red-incident.md) is absent from the bundle and substantial sub-topics are inlined rather than split into reference files.
Suggestions
Ship references/master-red-incident.md in the skill bundle (or inline its sandbox-compatible first step) — the body designates it as the required first step for the unattended Master-red workflow, but no references/ directory exists.
Move the 'Backend tests on Depot CI' subsection and the merge-queue branch semantics (trunk-merge/pr-<n>/<uuid>, bisection branches) into a reference file, keeping a short summary and pointer in SKILL.md to reduce inline bulk.
De-duplicate the platform-outage guidance between 'Rule out a platform outage first' and the Base rate section, and consolidate the Depot retry guidance that currently appears in both step 3 and the Depot CI subsection.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-obvious, environment-specific knowledge (exit 78 semantics, trunk-merge branch behavior, DEPOT_TOKEN overriding the saved login) and never explains concepts Claude already knows, but at ~380 lines it has some redundancy: the platform-outage guidance appears in both 'Rule out a platform outage first' and again in the Base rate section, and Depot retry is covered in both step 3 and the Backend-tests-on-Depot subsection. Anchor 4 ('minor instances of over-explanation that could be trimmed') fits better than anchor 5, since not every token earns its place. | 4 / 5 |
Actionability | Guidance is fully executable: copy-paste commands with exact jq filters (the github-actions[bot] comment query, the check-runs API call for 'Backend CI on Depot / Test results'), concrete tool invocations (hogli ci:insights search "<error>", depot ci retry <run-id> --failed --workflow <workflow-id>), classification and reproduction tables mapping log signals to first actions, and a literal report template. This matches anchor 5 ('copy-paste ready... specific examples cover the common cases'). | 5 / 5 |
Workflow Clarity | The multi-step process is clearly sequenced (insights digest → find the failing run → read the CI report comment → classify → base rate → local repro → report) with explicit validation checkpoints: 'confirm from run data before reporting a failure as already fixed', 'Check the current queue run with gh before reporting it as active', and 'If it disagrees with the current logs, report the mismatch and use the logs for the cause'. Risky operations are gated behind explicit approval in Safety rules, so no destructive/batch cap applies. This matches anchor 5's explicit validation and error-recovery loops. | 5 / 5 |
Progressive Disclosure | Sections are clearly headed and cross-skill handoffs are named, but the skill's own key reference — references/master-red-incident.md, the designated first step for the unattended workflow — does not exist in the bundle (there is no references/ directory at all), and ../../../docs/internal/ci-things-already-tried.md points outside the skill tree. Long operational detail (Backend tests on Depot CI, merge-queue branch semantics) is inlined in the single SKILL.md rather than split out. Anchor 3 ('references present but... content that should be separate is inline') fits; anchor 4 requires only 'minor organization gaps', which a broken primary reference is not. | 3 / 5 |
Total | 17 / 20 Passed |