Content
92%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-engineered workflow skill: exact commands, explicit validation and failure-recovery loops for the risky stash/worktree operations, and a properly offloaded conventions reference. The only improvement space is minor redundancy between steps 2 and 8 and the verbose step-9 report checklist.
Suggestions
State the original-checkout/cleanup-cwd rule once (step 2) and have step 8 reference it instead of restating 'return to the original checkout (or another safe cwd outside the target worktree)'.
Compress step 9's eight-item report list into a single line naming the report fields (mode, worktree path, branch, commit hash+message, remote, PR URL, worktree kept/removed, skipped verification).
Consider moving the repo-specific 'bun run build:pages' requirement alongside other repo conventions or noting its scope, so the generic workflow body stays repo-agnostic.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes Claude's competence (no git/worktree concept explanations), but the original-checkbook-restoration idea is stated twice (step 2's "Record the original checkout path up front" and step 8's "first return to the original checkout"), and step 9 enumerates eight report items where a shorter checklist would do — minor trimmable instances, matching the efficient-with-minor-trims anchor at 4 rather than the every-token-earns-its-place anchor at 5. | 4 / 5 |
Actionability | Commands are copy-paste ready throughout: "git stash push --include-untracked -m \"<message>\"", "git worktree add -b <branch> ../.worktrees/<repo>/<branch> HEAD", "git push -u origin <branch>", "gh pr create --base <base> --head <branch> --title <title> --body <body>", "bun run build:pages", plus explicit error paths (stash conflicts, gh unauthenticated, dirty-worktree cleanup refusal). | 5 / 5 |
Workflow Clarity | Nine clearly ordered steps with explicit validation checkpoints and feedback loops — "Apply the captured stash inside the new worktree and verify the diff landed there", "Confirm git status --short is clean after the commit", "Never remove a dirty worktree", and a retry-once recovery for the cwd-in-target-worktree failure — fully covering the destructive stash/worktree-removal operations. | 5 / 5 |
Progressive Disclosure | The single bundle file references/pr-conventions.md exists, is clearly signaled in step 7 ("must follow [PR and Commit Conventions](references/pr-conventions.md)"), and is exactly one level deep; nothing that belongs in a separate file is inlined in the ~80-line body. | 5 / 5 |
Total | 19 / 20 Passed |