Content
56%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 skill is structurally excellent — point-of-need, one-level-deep references that all resolve, and a validation-obsessed state machine covering rebase, publication, and recovery paths. Its weaknesses are executable specificity (no actual git commands anywhere) and token efficiency: the giant mermaid chart's abstract label language makes it dense and hard to act on directly.
Suggestions
Add a short 'Commands' section with the concrete git invocations for each node (e.g., `git rebase --onto <target> <upstream> <source>`, `git rebase --continue`, `git worktree add <path> <source>`, `git push --force-with-lease <remote> <result>:<ref>`), or inline them in the flowchart labels.
Compress the mermaid chart: collapse the many near-identical "Failure or unobservable → Recover" edges into a single default edge per stage, and extract the worktree-writer/handoff negotiation into a reference file, keeping the main flowchart to the core rebase lifecycle.
Replace abstract labels like "bind current result OID R" and "reobserve the named result ref" with concrete git terms (e.g., "record `git rev-parse` of the result ref", "re-run `git rev-parse` and confirm it matches R") so the workflow reads as instructions rather than a policy document.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | There is no padding with concepts Claude already knows (no rebase tutorials, no git basics), and the "Result-bound verification" section is lean. But the ~100-line mermaid state machine carries heavy abstraction cost — labels like "bind current result OID R", "reobserve the named result ref", "checkpoint only task-authorized changes after worker checkpoint" require multiple re-reads to extract instruction, so the chart is noticeably over-dense relative to what it communicates. This sits at 3 ("mostly efficient but... could be tightened") rather than 4, because a substantial fraction of the flowchart's token load (worktree-writer negotiation, handoff obligations, many near-duplicate failure edges) could be compressed without losing guidance. | 3 / 5 |
Actionability | The body contains zero concrete commands — no `git rebase <source> <target>`, `git rebase --continue/--abort`, `git worktree add`, or `git push --force-with-lease` appears anywhere; every operation is stated abstractly ("create start-only branch worktree", "start fresh replay in bound start worktree", "push immutable R with the authorized exact lease"). The decision logic is detailed, which lifts it above the 1 anchor ("only describes rather than instructs"), but the specific executable steps are missing, matching the 2 anchor ("high-level hints but missing the specific steps to execute"). | 2 / 5 |
Workflow Clarity | The workflow is exceptionally thorough as a sequence: a complete state machine with explicit validation checkpoints ("bind validation to the immutable result commit R", "verify checks left tracked source unchanged", "Exit zero; post-fetch destination equals R"), feedback loops (correction commit → "invalidate old evidence and rebind R" → rerun checks; push lease retry), and a dedicated Recover path for every failure edge. It falls short of the 5 anchor only on readability — the cryptic node/label vocabulary ("Satisfied{...ancestry-only; T ancestor of S?}", "BindWT", "FinishMode") makes an otherwise clear sequence hard to follow, leaving minor clarity gaps consistent with the 4 anchor. | 4 / 5 |
Progressive Disclosure | The body is an overview plus a point-of-need index: all five referenced files exist in ./references/ (active-rebase-recovery.md, conflict-and-ambiguity.md, history-shape.md, named-stash.md, publication.md), each linked exactly where the workflow needs it ("read [named stash](./references/named-stash.md); save and bind exact entry"), one level deep with no nested references inside them. This matches the 5 anchor ("clear overview with well-signaled one-level-deep references; content appropriately split") — detailed procedures live in the references while the skill body carries only the flow and the verification policy. | 5 / 5 |
Total | 14 / 20 Passed |