Content
80%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 textbook progressive-disclosure hub: the body is a lean, well-signaled router over a real, well-split bundle of reference files, and the description is equally strong. The only substantive weakness is that the body delegates all operational detail — including validation before destructive stack operations — to the references, leaving the body itself without any verification checkpoint.
Suggestions
Add a one-line safety gate to the body for destructive operations, e.g. "Before merge or rebase, confirm the stack state with `gh stack view --json`" — this would satisfy the validation expectation for destructive/batch workflows without compromising the lean router design.
Include a single inline quick-start command (e.g. the `gh stack init` / `add` / `submit --auto` core loop as a 3-4 line snippet) so the most common task is actionable without a reference hop, raising actionability toward copy-paste-ready guidance.
State the failure-first step explicitly in the routing list (e.g. "on any non-zero exit, check the exit-code table in usage.md before retrying") so the error-recovery loop is visible in the body, not only implied by the commands.md and troubleshooting.md pointers.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~15-line body spends exactly one line on identity ("`gh stack` is a GitHub CLI extension for stacked branches and pull requests") and every remaining token on routing — no padding, no explanation of concepts Claude already knows. It matches the anchor "Lean and efficient; assumes Claude's competence; every token earns its place" rather than the 4 anchor, which reserves room for trimmable over-explanation that is absent here. | 5 / 5 |
Actionability | The routing directives are concrete and unambiguous — named files with explicit trigger conditions ("before creating a stack or deciding layers", "on unexpected failures or for preconditions, side effects, and ordering guarantees") — so Claude knows exactly what to do next, and per the scoring note the absence of code in an instruction/routing skill is not penalized per se. It falls short of the 5 anchor ("copy-paste ready code or commands") because the body contains no inline command or quick-start example; any concrete action requires first opening a reference file. | 4 / 5 |
Workflow Clarity | The body presents a clear navigation sequence (read usage.md first, then the trigger-matched reference), but the skill manages destructive/batch operations (merge, rebase, sync across a whole stack) and the body itself contains no validation or verification checkpoint — the cap in the judging guidelines therefore limits this to 3. It is not a 2 because the sequence that is present is coherent and each reference's purpose is well defined, and the references the body routes to do carry the validation loops (exit-code table, `gh stack view --json` to confirm, abort paths). | 3 / 5 |
Progressive Disclosure | The body is a pure overview routing to four one-level-deep references, all of which exist in the bundle (references/usage.md, stack-design.md, commands.md, troubleshooting.md), each with a clearly signaled trigger; the reference files do not nest further (usage.md's pointer to stack-design.md is a sibling, not a chain). This matches the 5 anchor — clear overview, well-signaled one-level-deep references, easy navigation — with no organization gaps to justify a 4. | 5 / 5 |
Total | 17 / 20 Passed |