Content
88%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.
An exceptionally actionable body: every workflow carries exact commands, explicit validation checkpoints, and error-recovery guidance for repo-specific failure modes, with clean one-level references. The main structural improvement would be moving the long scenario playbooks into reference files to keep SKILL.md lean.
Suggestions
Move the stacked-PR recovery and PR-metadata redaction playbooks into dedicated reference files (e.g. references/stacked-pr-recovery.md), leaving a one-line trigger summary in SKILL.md so simple invocations don't load them.
Trim version-specific evidence (package versions, line counts) to the durable lesson, or consolidate it into a gotchas reference, to reduce time-sensitive detail in the main file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, repo-specific operational knowledge that assumes Claude's competence (no explanations of git or CI basics), with every section earning its place. Minor trims are possible — time-sensitive evidence details like "@cloudbase/manager-node 5.8.6 installed vs 5.8.8 locked" and "the same 46 lines" are version-specific detail not placed in a deprecated/old-patterns section. Not 5 for those minor instances; not 3 because there is no unnecessary concept explanation or padding. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready commands cover every inline workflow: "git merge-tree --write-tree <A's-branch> <B's-branch>", "gh api \"repos/<owner>/<repo>/actions/runs?branch=<branch>\" --jq ...", "GIT_SEQUENCE_EDITOR=true git rebase -i --autosquash <main-sha>", "node scripts/update-compat-baseline.mjs", plus concrete per-file conflict-resolution rules. Not 4 because there are no pseudocode steps or missing key details — even failure modes come with the exact command to judge them by. | 5 / 5 |
Workflow Clarity | Multi-step processes are clearly numbered with explicit validation gates ("git diff --stat <old-sha> HEAD — it must be empty", "git apply --check <patch> before git apply <patch>", "git status --porcelain | grep -v '^M '` must come back empty", "Close with node scripts/diff-compat-config.mjs (Has blocking diff: NO)") and feedback loops for destructive operations (backup patch before git reset --hard, --force-with-lease, reopen-to-retrigger workflows). Not 4 because validation is explicit and verifiable at every risky step, including error-recovery guidance for misleading tool output. | 5 / 5 |
Progressive Disclosure | Clear overview with well-signaled, verified, one-level-deep references ("references/source-commands.md" under "Source of truth", "references/command-catalog.md" under "Command mapping"), and both files exist. Not 5 because ~70 lines of scenario-specific playbooks (stacked-PR recovery, PR-metadata redaction) are inlined in SKILL.md rather than split into referenced files, so they are loaded even for a simple commit; not 3 because what is inline is clearly sectioned and the reference structure itself is clean. | 4 / 5 |
Total | 18 / 20 Passed |