Content
81%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 well-engineered CI skill body: a numbered contract with mandatory verification, a complete machine-readable output schema, concrete thresholds and loop guards, and a clean one-level split into the workflow-example reference. The main defects are a dangling reference to a HEALS.md 'bootstrap pattern' that the reference file does not contain, and some repetition of the verify discipline across sections.
Suggestions
Fix the dangling reference in Prerequisites: either add a first-run HEALS.md bootstrap snippet (minimal file header + one example HEAL- entry) to references/workflow-example.md, or point Prerequisite 4 at the sibling self-healing skill where the file format actually lives.
Include a minimal HEAL- entry template (the HEALS.md markdown format, not just the YAML output schema) so the CI skill is self-contained; currently a CI agent must read ../self-healing/SKILL.md to know what it is committing.
Trim the verify-before-persist restatements — the Purpose bullets, CI Contract items 3/5, and the Verify-Before-Persist section each restate the same rule; consolidating to the CI Contract plus one cross-reference would reclaim tokens without losing the discipline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient and assumes Claude's competence — no explanations of what CI, GitHub Actions, or lockfiles are; sections are compact and tables carry weight ("`pnpm test` failed | Re-run `pnpm test` after the patch"). Minor over-explanation that could be trimmed: the verify-before-persist discipline is restated in the Purpose bullets, CI Contract items 3 and 5, the Verify-Before-Persist section, and Anti-Patterns 1 and 3, and "**Never fake `verified`.** Faking is the exact failure mode this skill exists to prevent…" is rhetorical emphasis. Not 5 because of that repetition; not 3 because the padding is minor and each section still adds distinct operational content. | 4 / 5 |
Actionability | Mostly executable guidance: copy-paste install commands ("gh extension install github/gh-aw"), a fully specified machine-readable YAML output schema with realistic field values, concrete recurrence thresholds ("`Recurrence-Count >= 3`", "at least 2 distinct PRs/tasks", "30-day window"), real GitHub event triggers ("`workflow_run` (completed, conclusion: failure)"), and a concrete loop guard ("`if: github.actor != 'github-actions[bot]'`"). Not 5 because Prerequisite 4 points to a bootstrap pattern that does not exist — "see `references/workflow-example.md` for the bootstrap pattern" for creating `.learnings/HEALS.md` on first run, but that file contains only workflow templates, patch conventions, and permission/anti-loop checklists, leaving the first-run case without concrete coverage. Not 3 because everything else is concrete and executable, not pseudocode. | 4 / 5 |
Workflow Clarity | The multi-step loop is clearly sequenced with explicit validation checkpoints and error-recovery feedback loops: the numbered CI Contract ("1. Read CI logs, PR diff… 2. Avoid direct code modifications… 3. Re-run the failing check after applying the proposed patch…"), the Verify-Before-Persist table mapping each failure class to its re-run, the explicit fallback when verification is infeasible ("ships as `pending-verify` with explicit notes on what would prove it"), and the Anti-Patterns checklist ("Re-trigger loops… Silent retry of flaky tests"). This matches anchor 5's requirement of sequence + validation + feedback loops + checklist, which matters here since the skill can auto-commit to PR branches. Not 4 because validation is not merely present — it is the organizing principle, with a defined failure path when it cannot run. | 5 / 5 |
Progressive Disclosure | Scored against the actual bundle (SKILL.md + `references/workflow-example.md`): the split is appropriate — contract, schema, and rules inline; authoring templates and permission/anti-loop checklists in the single one-level-deep reference, clearly signaled ("Authoring patterns and example `.github/workflows/*.lock.yml` files live in [`references/workflow-example.md`]… Keep example workflows out of `.github/workflows` until you've explicitly decided to enable CI automation"), and the reference links back without nesting. Not 5 because the body's Prerequisite 4 cites a "bootstrap pattern" in that reference which is not actually there, a navigation defect for a reader following the pointer. Not 3 because the structure and signaling are otherwise clean, not buried or monolithic. | 4 / 5 |
Total | 17 / 20 Passed |