Content
92%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 exemplary operational skill: fully executable commands, a consent-gated merge loop with layered validation and stale-head guards, and a clean one-level-deep reference structure. The only notable improvement is trimming minor duplication in the reference index and alternate commands.
Suggestions
Drop the 'Reference index' section (its seven links all appear inline at their point of use) or keep only the index and remove the repeated inline citations.
Remove the alternate manual bootstrap one-liner ('Or: gh api user -q .login && cd site && bun install && make check-netlify') from Phase 0; the check-tools.sh entry plus the tool-bootstrap.md reference already cover it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body assumes Claude's competence throughout — it never explains what Dependabot, Netlify, or Lighthouse are — and every phase is operational. It falls just short of 5 because the 'Reference index' section duplicates links already given inline and Phase 0 includes an alternate one-liner command that could be trimmed. | 4 / 5 |
Actionability | Guidance is fully executable: concrete commands with exact flags and jq filters (`gh pr list --author 'app/dependabot' ...`), `make ci` / `make site-netlify-validate`, a real PR example for `merge-next.sh` with env vars, and a copy-ready verdict template. All referenced scripts exist in the bundle. | 5 / 5 |
Workflow Clarity | The multi-phase process is clearly sequenced with explicit validation checkpoints (Phase 0 'Stop on failure', stale-head guard on headRefOid, Layer A checks, Layer B required in Full mode, poll loop with max ~5 min), feedback loops for error recovery (debug-netlify-pr.sh, merge-failures.md), and consent gating on the destructive merge step. | 5 / 5 |
Progressive Disclosure | SKILL.md acts as a lean overview with well-signaled, one-level-deep references that all exist in references/ (7 files) and scripts/ (3 files), with details appropriately split out (risk tiers, package notes, build debugging, merge failures) and an index for navigation. | 5 / 5 |
Total | 19 / 20 Passed |