Content
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A highly actionable, well-sequenced workflow guide with strong validation checkpoints on destructive git operations. It loses points on conciseness (restating conventional-commit basics and a heavy examples block) and progressive disclosure (everything inline with no reference files for a 260-line skill).
Suggestions
Trim or relocate basic conventional-commit rules (lowercase/imperative/no-trailing-period/<72 chars) that Claude already knows, keeping only the house-style deviations that earn their tokens.
Move the six commit-message examples and the merge-strategy table into a references/ file (e.g. references/commit-examples.md) referenced one level deep from SKILL.md, slimming the main body into an overview.
Condense the 'Always Active Principles' and 'Agent Git Workflow' sections, which overlap on staging-by-name and force-push guidance, into a single sequenced workflow to remove redundancy.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient and actionable, but restates conventional-commit basics Claude already knows ('Lowercase, imperative mood, no trailing period, Under 72 characters') and carries a bulky six-example block that could be trimmed — fitting 'mostly efficient but could be tightened' rather than the lean level-3 anchor. | 2 / 3 |
Actionability | Provides fully executable, copy-paste-ready commands (e.g. 'gh repo view --json defaultBranchRef --jq …', 'git push --force-with-lease origin feat/my-branch') and concrete fallback scripts, matching the level-3 'fully executable code/commands' anchor. | 3 / 3 |
Workflow Clarity | Clear numbered sequences (Agent Git Workflow, History Rewriting Before PR) with explicit validation checkpoints on destructive operations — 'Verify byte-for-byte match with backup' and 'Confirm with the user before force-pushing' — satisfying the level-3 anchor with feedback loops. | 3 / 3 |
Progressive Disclosure | No bundle files exist and the ~260-line SKILL.md is monolithic; the extensive examples and merge-strategy tables are inline rather than split into one-level-deep reference files, matching 'content that should be separate is inline' rather than the level-3 split-overview anchor. | 2 / 3 |
Total | 10 / 12 Passed |