Content
53%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 body is a model of conciseness and the description does its job, but the skill is broken as a bundle: all six referenced markdown files are missing and the eight substantive shell scripts that do exist are never mentioned. As written, an agent following it hits a dead end at the first instruction.
Suggestions
Fix the reference targets: either add `skills/controller.md`, `commands/start.md`, `commands/continue.md`, `commands/validate.md`, `commands/push.md`, and `guidelines.md` to the bundle, or correct the paths in SKILL.md to point at the files that actually exist.
Reference the real bundle assets from the body — e.g. note that `scripts/preflight.sh` is the hard gate (exit 2 = not in a stack, no fallback), that `scripts/stack-status.sh` reports stale/diverged/PR-less layers, and that `scripts/push-guard.sh` guards the push phase — so the existing scripts are discoverable.
Since this is a batch, hard-to-reverse workflow (rebasing and force-pushing every branch), state the validation loop explicitly in the body or in a present file: validate the trunk-adjacent branch (scripts/validate-branches.sh) before push, and how to recover via `/continue` when a conflict stops the rebase.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~15-line body is lean and assumes competence: a routing table, the gating condition ("Refuses to run unless `gh stack` is installed and `gh stack view --json` exits 0"), and a pointer to scope rules. It explains nothing Claude already knows and every sentence carries new, load-bearing information. Fits the 'lean and efficient; every token earns its place' anchor exactly. | 5 / 5 |
Actionability | The body's instructions are "Read `skills/controller.md` and follow it" plus a command table mapping to `commands/*.md` — but none of those files exist in the bundle (only `scripts/` with 8 shell scripts exists, none referenced from the body). The one executable check written out (`gh stack view --json` exits 0) is concrete, but the specific steps for every phase live in files that are missing, so executing this skill as written fails immediately. This matches 'minimal concrete guidance; high-level hints but missing the specific steps to execute' better than the level-3 anchor, since the missing files are the guidance itself rather than a detail gap. | 2 / 5 |
Workflow Clarity | The four phases (start, continue, validate, push) are sequenced in the table and a gating preflight is stated, which is a real sequence. But the actual checkpoints and recovery loops are delegated to files that don't exist, so as written the validation checkpoints are implicit rather than explicit — matching 'steps listed but validation gaps; checkpoints missing or implicit'. This is also a batch/destructive workflow (rebase + push of every branch), and the body itself contains no validate→fix→retry loop, which caps it at 3. | 3 / 5 |
Progressive Disclosure | Scored against the actual bundle: every referenced path is broken — `skills/controller.md`, `commands/start.md`, `commands/continue.md`, `commands/validate.md`, `commands/push.md`, and `guidelines.md` do not exist anywhere in the bundle, while the 8 real files in `scripts/` (preflight, stack-status, validate-branches, push-guard, snapshot, etc.) are never referenced from the body. The one-level routing idea is right and the table is well-signaled, but navigation leads nowhere and the existing bundle content is undiscoverable, which fits the level-2 anchor (references buried/unusable) rather than the level-3 'references present but not clearly signaled'. | 2 / 5 |
Total | 12 / 20 Passed |