Content
76%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 well-structured, highly actionable orchestration skill with concrete gh commands and clear phase sequencing. Its main weakness is the absence of post-action verification feedback loops for destructive/batch operations, which limits workflow clarity despite strong pre-condition gates.
Suggestions
Add post-action verification steps (e.g. re-check `gh pr view --json state` after merge, or confirm a comment was posted before closing) so batch/destructive actions have explicit validate→fix→retry loops.
Tighten the tone-coaching lines (e.g. 'Be genuinely helpful — imagine you're a senior maintainer...') into concise rules to reduce non-essential tokens.
Consider extracting the long per-handler specs (HANDLE_ISSUE_*, HANDLE_PR_*) into reference files linked from SKILL.md to improve progressive disclosure.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean tables, code blocks, and phased structure with little concept over-explanation, but tone coaching like 'Be genuinely helpful — imagine you're a senior maintainer who cares about the community' is mildly fluffy and could be trimmed. | 4 / 5 |
Actionability | Fully executable copy-paste-ready gh CLI commands appear throughout (issue/PR list, comment, close, merge, api diffs) alongside concrete report schemas, covering the common cases. | 5 / 5 |
Workflow Clarity | Phases 1–5 are clearly sequenced with pre-action gates (the six merge conditions, IF YES/NO and OUTCOME A/B/C branches), but this batch/destructive skill lacks post-action verification feedback loops (e.g. confirm a comment posted or a merge succeeded before proceeding), which caps the score per the rubric's batch-operation rule. | 3 / 5 |
Progressive Disclosure | Single self-contained file with no bundle references, organized into well-labeled sections (ARCHITECTURE, PHASE 1–5, ANTI-PATTERNS, QUICK START); the per-handler specs are long and could arguably live in separate reference files, a minor organization gap. | 4 / 5 |
Total | 16 / 20 Passed |