Content
66%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.
The workflow engineering is excellent — unambiguous sequencing, explicit validation checkpoints, bounded retries, and hard safety rules with wrong/right examples. However, the skill leans on five rule files that are missing from the bundle (making its core authoring and split procedures unresolvable), and the body carries notable padding from version-history rationale and rules restated across multiple steps.
Suggestions
Ship the referenced rule files (description-contract.md, split-mode.md, pre-push-review.md, registration-poll.md, description-examples.md) in the bundle, or inline their load-bearing content — as it stands, Steps 1–5 and Split Mode point at content that does not exist.
Move version-history prose (the v3.5 background-watch removal rationale, legacy flag aliases like --no-preview-spec and --no-feedback) into a single short 'Deprecated / legacy' section so the main flow reads as current behavior only.
State each 'record the outcome now' / 'never soften a failure into a skip' rule once (near the report template in Step 10) instead of repeating it across Steps 5.5, 6.4, and 6.5.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly operational rather than tutorial-like, but several sections are padded with design history and restated rules: the removed v3.5 background-watch rationale ("It was removed because every job it did is already done…"), the legacy-flag paragraph ("--no-preview-spec is the pre-rename spelling, still honoured so existing scripts and muscle memory keep working"), and the same 'record it now, never soften a skip into a decline' rule repeated across Steps 5.5, 6.4, and 10. This fits anchor 3 — mostly efficient but includes unnecessary explanation that could be tightened — rather than 2, since the padding is project-specific rationale, not explanation of concepts Claude already knows. | 3 / 5 |
Actionability | Mostly executable, copy-paste-ready guidance: `git push`, `gh pr create --draft --title … --body …`, `timeout 540 gh pr checks <pr-number> --watch`, exact `Skill("review-loop", "<pr-url> --no-ci")` invocations, a first-match-wins flag→invocation table, and a full report template. It stops short of anchor 5 because the skill's core task — writing the description (Steps 1–5) — is delegated wholesale to `rules/description-contract.md`, which is not present in the bundle, so the central authoring guidance is not actually executable from what ships here. | 4 / 5 |
Workflow Clarity | The multi-step process is clearly sequenced (Steps 0 → 10 with explicit sub-steps) and saturated with validation checkpoints and feedback loops: CI watch with exit-code classification (0 / 124 / 127 / other), bounded retries ("up to 4 attempts total … After the fourth … escalate — never watch again"), ci-auto-fix capped at 2 invocations with print-as-you-go counters, pre-push flagged-findings surfacing, and a mandatory report with slots for every outcome. This matches anchor 5's explicit validation steps, error-recovery loops, and checklists. | 5 / 5 |
Progressive Disclosure | The body is well-signaled — "Full procedure lives in [`rules/split-mode.md`](./rules/split-mode.md)", "**→ [`rules/registration-poll.md`](./rules/registration-poll.md)**" — but none of the referenced files exist in the bundle: there is no rules/, references/, scripts/, or assets/ directory, so the description contract, split-mode procedure, pre-push review rules, registration poll, and all four examples are unreachable. Scored against the actual bundle structure, this is anchor 2's minimal effective structure: the split-out content is not inlined and the references do not resolve, rather than anchor 3, where references are present but merely poorly signaled. | 2 / 5 |
Total | 14 / 20 Passed |