Content
81%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.
Highly actionable content with concrete, executable git commands and an explicit, well-validated workflow that matches the project's checks. The main weaknesses are generic conflict-resolution prose that pads the token budget and a monolithic structure where the conflict guidance could live in a reference file.
Suggestions
Trim the Conflict Resolution Guidance section to the non-obvious specifics (zdiff3 marker behavior, :1/:2/:3 diff comparison, regenerate-generated-files strategy, accept-both-then-lint for imports) and cut generic advice Claude already knows about understanding both sides' intent.
Move detailed conflict-resolution tactics into a references/ file (e.g., references/conflicts.md) and keep a short pointer in SKILL.md, tightening the main body toward an overview.
State the Borg UI check commands once with a brief condition (backend vs. frontend) instead of nesting them inside the workflow step to reduce inline command listing.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The workflow steps are tight and command-driven, but the Conflict Resolution section spends many lines on generic guidance Claude already knows ("State what each side is trying to achieve", "Avoid accidental deletions or silent behavior changes", "Open files and understand intent on both sides"). Mostly efficient with some unnecessary explanation, matching the 3 anchor; the zdiff3-specific notes are genuinely additive, keeping it above a 2. | 3 / 5 |
Actionability | Fully executable throughout: "git config rerere.enabled true", "git pull --ff-only origin $(git branch --show-current)", "git -c merge.conflictstyle=zdiff3 merge origin/main", "git diff :1:path/to/file :2:path/to/file", plus concrete project checks ("ruff check app tests", "npm run check:locales && npm run typecheck"). Copy-paste ready commands covering the common cases. | 5 / 5 |
Workflow Clarity | A clear 9-step sequence with explicit validation (step 8: "git diff --check" plus ruff/pytest/npm checks), feedback loops ("rerun tests after each logical batch"; resolve → add → commit → merge --continue), and a final summarize step. This matches the 5 anchor with validation checkpoints and error-recovery guidance, so the destructive-operation cap does not apply. | 5 / 5 |
Progressive Disclosure | No bundle files exist; the ~95-line body is well-organized with clear section headers (Workflow, Conflict Resolution Guidance, When To Ask The User) and no nested references. However, the ~40-line conflict-resolution guidance is reference-grade detail inlined in SKILL.md, so structure is good but not the clear overview-with-references pattern of the 5 anchor. | 4 / 5 |
Total | 17 / 20 Passed |