Content
77%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 highly actionable and well-sequenced git workflow with strong validation and recovery loops, but it is verbose due to heavy repetition of the commit-splitting rule and is a single monolithic 1000+ line file that would benefit from splitting the three modes into reference files.
Suggestions
Consolidate the repeated 'multiple commits by default' enforcement into one authoritative section and reference it from later phases instead of restating the rule (and its ALL-CAPS framing) five times.
Split the three modes (COMMIT, REBASE, HISTORY_SEARCH) into separate reference files (e.g. references/commit-mode.md, references/rebase-mode.md, references/history-search.md) and keep SKILL.md as a mode-detection overview that links to them, one level deep.
Remove the leftover '<!-- PLACEHOLDER_PHASE2 -->' marker and ensure each phase section is finalized rather than left as a placeholder.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly actionable, but the 'split into multiple commits' rule is restated many times (CORE PRINCIPLE, MANDATORY SELF-CHECK, Phase 3.0/3.5, FINAL CHECK, Anti-Patterns) with heavy ALL-CAPS enforcement that could be consolidated, placing it at 'mostly efficient but could be tightened'. | 3 / 5 |
Actionability | It provides copy-paste-ready git commands, detection regexes, and concrete examples across all three modes (e.g. 'git log -S "searchString" --oneline', 'GIT_SEQUENCE_EDITOR=: git rebase -i --autosquash $MERGE_BASE'), covering the common cases fully. | 5 / 5 |
Workflow Clarity | Multi-step processes are clearly sequenced into numbered phases with explicit validation checkpoints and feedback loops for destructive operations (rebase --abort, reflog recovery, --force-with-lease warnings, bisect reset, a blocking FINAL CHECK checklist). | 5 / 5 |
Progressive Disclosure | The file is internally well-structured with mode-detection routing and phase headers, but at ~1013 lines everything is inlined in one monolithic SKILL.md with no bundle files and no references — the three modes clearly could be split into separate files, and a leftover '<!-- PLACEHOLDER_PHASE2 -->' marker signals incomplete organization. | 3 / 5 |
Total | 16 / 20 Passed |