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.
An exceptionally strong conventions body: dense, evidence-backed, and immediately executable, with validation loops woven throughout. Its one real weakness is structural — it is a monolith, with several deep-dive sections that belong in reference files, which also holds its token economy back from a top score.
Suggestions
Move the deepest subsections — 'What GitHub does with each conclusion', the failure-rate-metric/warehouse passage under 'Required-check gates', the checkout duration table, and the Depot consumers list under 'The hourly master lane' — into references/ files (e.g. references/check-run-semantics.md, references/ci-health-metrics.md), leaving the rule, a one-line rationale, and a well-signaled link in SKILL.md.
Split the gate-orchestration rules ('Four rules for the gate body' and the OR-on/never-AND rules) into a references/gates.md and keep only the `!cancelled()` condition table and the fail-closed verdict rules inline, so a routine concurrency or caching question does not load the entire gates treatment.
Tighten the restated draft/ready rules in 'Draft vs ready-for-review' — the 'draft → skip' validity conditions appear twice (paragraph three and the bolded rule) — and compress evidence narratives to one line plus the run link.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly every sentence carries repo-specific, non-derivable detail ("45k tracked files and 4.6 GiB of packed objects", "500 runs per 10s per repo", WF rule IDs, measured run links) and nothing explains concepts Claude already knows, placing it above the 3 anchor; but it is not the 5 anchor's "every token earns its place" — the ~17-line failure-rate-metric passage ("joins `github_workflow_jobs` to that table on `run_id`, then scopes the job side to a single `run_attempt`") and some restated draft/ready rules exceed what the authoring decision needs. | 4 / 5 |
Actionability | Copy-paste-ready YAML for the common cases (canonical concurrency block, per-SHA group expression, `create-github-app-token` step with `|| github.token` fork fallback, sparse-checkout recipe, `curl -fsSL --retry 5 --retry-all-errors`, the `set -f` argument-splitting fix) plus exact commands (`hogli ci:plan`, `bin/hogli lint:workflows`) and named canonical files to copy from ("`ci-paths-filter.yml` is the smallest complete example") — the 5 anchor, not the 4 anchor, since the placeholders (`@<sha> # v6`) are deliberate templates backed by pointers to real files. | 5 / 5 |
Workflow Clarity | A clear sequence with explicit validation checkpoints and feedback loops: check "things already tried" first, copy a canonical file, then validate with `bin/hogli lint:workflows` and `actionlint` before pushing, check conditions with `hogli ci:plan`, and "update its expectation and run the suite" (`hogli test:workflows`) when a condition changes — closing with a 13-item "New-workflow checklist". Error-recovery paths are stated (linter rejections take `# hogli-lint: allow-* -- <reason>`), matching the 5 anchor's checklists and feedback loops; no destructive/batch cap applies since this is authoring guidance. | 5 / 5 |
Progressive Disclosure | Section structure is clear and the external pointers that exist are one-level-deep and well signaled ("Read its [ordered-image reference](../gating-production-deploys/references/ordered-image-releases.md)", "See [`/depot-ci`](../depot-ci/references/posthog-check-run-semantics.md)"), but the skill ships no references/ files of its own and inlines ~550 lines, including multi-screen deep dives (gate conclusion semantics, the warehouse failure-metric rules, the checkout duration table, the Depot consumers list). That matches the 3 anchor's "content that should be separate is inline" — above the 2 anchor because structure and signaling are good, below the 4 anchor because the inline deep-dives are more than "minor organization gaps". | 3 / 5 |
Total | 17 / 20 Passed |