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.
A highly actionable, well-sequenced workflow: every step has executable commands, validation checkpoints, and failure-recovery paths, and the destructive operations (branch push, issue/PR creation, worktree cleanup) are guarded by explicit stop conditions. The main cost is token weight — the defensive source-sync scaffolding and the PR template that duplicates the generator script inflate the body without adding proportionate guidance.
Suggestions
Move the sync_latest_source/valid_source_checkout/recover_corrupt_source_checkout bash from step 3 into a bundled script (e.g. scripts/sync-sources.sh) and invoke it in one line, cutting ~40 lines of git plumbing Claude can rederive if needed.
Drop or compress the "PR Body Template" section: scripts/create-pr-body.mjs already generates that exact structure, so showing the JSON input shape plus one sentence pointing to the generated output format is sufficient.
Extract the Verified-Fix Issue Template into a references/ file referenced by path, so the main workflow stays scannable and the template is loaded only at delivery time.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean executable bash with little concept explanation, but the ~45-line sync_latest_source/valid_source_checkout defensive scaffolding in step 3 is git plumbing Claude could rederive, and the "PR Body Template" section duplicates the structure that scripts/create-pr-body.mjs already generates. Mostly efficient with some content that could be tightened — the 3 anchor. | 3 / 5 |
Actionability | Fully executable throughout: exact gh/git commands with fallbacks, copy-paste-ready templates for both targets, and the PR body generator is a real bundled script with its JSON input shape and CLI invocation shown. Placeholders like <short-slug> are conventional and unambiguous. | 5 / 5 |
Workflow Clarity | Thirteen sequenced steps with explicit validation checkpoints (failing regression test written before production changes, confirmation the failure is the bug not a fixture, git status inspected before commit, regression + adjacent tests + real-surface QA after) and error-recovery loops (corrupt-checkout quarantine, label-management fallback, gh-clone fallback). Stop conditions and a cleanup receipt close the loop — matches the 5 anchor. | 5 / 5 |
Progressive Disclosure | Clear sections and a real, well-signaled one-level-deep reference to scripts/create-pr-body.mjs (verified to exist with matching input shape). Minor gaps: the long source-sync bash block and the two full deliverable templates are inlined in SKILL.md and would sit better in bundled scripts/references, keeping the overview leaner. | 4 / 5 |
Total | 17 / 20 Passed |