Content
65%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 highly actionable with comprehensive executable commands, but it is held back by redundancy between its two major technique sections, generic boilerplate, missing validation checkpoints on destructive workflows, and a Resources section whose referenced bundle files do not exist.
Suggestions
Merge the redundant 'Core Concepts' and 'Practical Workflows' sections and remove the generic 'Instructions' and 'Do not use this skill when' boilerplate to tighten conciseness.
Add explicit validation checkpoints to destructive workflows — e.g. insert a test/verify step between `git rebase -i main` and `git push --force-with-lease`, and verify recovered state after `git reset --hard` in the recovery workflow.
Create the referenced bundle files (references/git-rebase-guide.md, references/git-conflict-resolution.md, references/git-history-rewriting.md, assets/git-workflow-checklist.md, assets/git-aliases.md, scripts/git-clean-branches.sh) or remove the broken references so navigation is functional.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The bulk is concrete commands, but there is a generic boilerplate 'Instructions' section ('Clarify goals, constraints, and required inputs...'), a filler 'Do not use this skill when' section, and substantial redundancy between 'Core Concepts' and 'Practical Workflows' which re-cover the same five techniques. | 3 / 5 |
Actionability | Dense with copy-paste-ready, executable git commands for every technique (rebase, cherry-pick, bisect, worktree, reflog) and end-to-end workflows covering the common cases. | 5 / 5 |
Workflow Clarity | Workflows are sequenced but lack explicit validation checkpoints for destructive operations — e.g. Workflow 1 force-pushes after rebase with no test/verify step, and Workflow 5 recovers via reset --hard with no verification; per the rubric, missing validation on destructive/batch operations caps this dimension at 3. | 3 / 5 |
Progressive Disclosure | A '## Resources' section clearly signals one-level-deep references, but all six referenced files (references/*.md, assets/*.md, scripts/*.sh) do not exist — navigation is broken — and substantial detail that belongs in those separate files is instead inlined across the ~400-line monolithic body. | 3 / 5 |
Total | 14 / 20 Passed |