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.
A tightly written, well-sequenced skill with validation and feedback loops exactly where a destructive git operation needs them, and near-ideal token efficiency. The main gaps are the never-stated core rebase command and two referenced bundle files that are absent from the shipped skill directory.
Suggestions
Include the actual rebase invocation (e.g. `git rebase --onto <new_base> <old_base> <curr>` and the interactive/commit-by-commit variant step 8 assumes) so the mechanical core is executable, not just the surrounding checks.
Ship or verify the out-of-tree references: `../../assets/writing.md` and `<plugin root>/scripts/prose-lint.py` do not exist in the bundle, so steps 7 and 11 point at files a user may not have; bundle them or inline their essentials.
Give one worked classification example in step 4 (a commit, the delta fact, the class, the one-line justification) so the demanded justification format is concrete.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and lean: no explanation of git or rebase basics, every sentence instructs ("Never `-X theirs` or `-X ours`", "Lint and unit-test the affected area after each commit", "Read each of these on `new_base`, never from memory"). The two-sentence opener ("Git can replay commits. It cannot tell you whether the replayed solution still makes sense...") is the only framing text and earns its place by justifying the skill's existence — matching anchor 5's "every token earns its place". | 5 / 5 |
Actionability | Guidance is mostly concrete and executable: exact commands and flags (`git status --porcelain`, `prose-lint.py`, bans on `--no-verify`, `-X theirs`), a five-class decision table with per-class actions, and explicit risk-gate conditions. It misses anchor 5 because the core mechanical command of the whole skill — e.g. `git rebase --onto <new_base> <old_base> <curr>` — is never stated, and the classification step gives no concrete method or example for producing the one-line justifications it demands. | 4 / 5 |
Workflow Clarity | A 12-step (0–11) explicitly sequenced workflow with abundant validation checkpoints: a preflight dirty-tree check, a stop-and-ask risk gate at step 5, lint/unit tests after each commit, full verification at step 10, an evidence-based definition-of-done checklist, and a feedback loop ("One you cannot tick honestly sends you back to the step that produces it"). For a history-rewriting (destructive) operation this is exactly the anchor-5 pattern, well above the cap-3 risk the guidelines warn about. | 5 / 5 |
Progressive Disclosure | Structure is exemplary: a "Required references" table signaling each file with what it holds and exactly when to read it ("step 0, only if the tree is dirty", "when tempted to skip a step"), with detail correctly split out of the overview. It falls short of anchor 5 because two of the four referenced paths — `../../assets/writing.md` and `<plugin root>/scripts/prose-lint.py` — do not resolve in the provided bundle (only `references/stash.md` and `references/rationalizations.md` exist), leaving the navigation to those files unverifiable. | 4 / 5 |
Total | 18 / 20 Passed |