Content
88%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 excellent operational skill: fully executable bash at every step, careful safety rails (backup, rollback, force-with-lease, verification gates) around a destructive workflow, and no conceptual padding. The remaining gains are trimming a few code-restating glosses and moving the conflict-resolution policy and worktree conventions into reference files.
Suggestions
Delete the step-5 parenthetical that restates the `rebase --onto` code block verbatim — the code block alone is unambiguous.
Move the step-5 conflict-resolution rules (rerere, --ours/--theirs by change class, rollback) into a one-level-deep reference file (e.g. references/conflicts.md) to reduce the always-loaded body while keeping the details one hop away.
Extract the worktree path convention (`$HOME/.worktrees/${path_key}/${subject}`) and remote/trunk resolution into a short reference file, since they are reusable boilerplate rather than stacking-specific guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes git competence, explaining only non-obvious rationale ("`--no-track` is mandatory here: a branch cut from a remote base must not inherit the base's upstream", "Never hard-code `origin`"), and there is no conceptual padding. Not 5: a few glosses restate what the adjacent code already shows, e.g. the step-5 parenthetical "(equivalent to `git rebase --onto <base_ref> <merge-base> <subject>`, run with `-C \"$work_dir\"`.)" duplicates the code block immediately above it. | 4 / 5 |
Actionability | Every step ships copy-paste-ready bash with variables defined (remote/trunk resolution, worktree selection, `git rebase --onto`, `git push --force-with-lease`, `gh pr edit --base`, `gh stack link`), and edge cases like multiple remotes, missing PRs, and merged PRs each get concrete handling. Fully executable guidance covering the common cases. | 5 / 5 |
Workflow Clarity | A 12-step sequence for a destructive operation (rebase + force-push) with explicit validation checkpoints: dirty-tree check before rewriting (step 3), a pre-rebase backup with rollback (`reset --hard "${subject}.bak"`), a conflict feedback loop with per-conflict-class resolution rules and `rebase --continue` (step 5), tracking-line verification that must pass before continuing ("must show `[${remote}/${subject}]`; fix it before continuing"), and a final report checklist (step 12). This matches the feedback-loops and explicit-validation anchor. | 5 / 5 |
Progressive Disclosure | Good structure: a numbered workflow with per-step code blocks, a Never section, and inline signposts to related skills (`pr-create`, `pr-style`, `using-git-worktrees`, git-rebase's conflict handler); the single-procedure content is mostly appropriately placed inline. Not 5: at ~185 lines with no bundle files, self-contained sections like the step-5 conflict-resolution policy or the worktree-path convention could be split into one-level-deep reference files to slim the always-loaded body. | 4 / 5 |
Total | 18 / 20 Passed |