Content
75%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.
The body is a tight, well-sequenced orchestration document: it externalizes stale-worktree detection to scripts, gates a destructive batch operation behind preview and confirmation, and defines clear per-branch rules with failure isolation. Main gaps are path/environment mismatches and reliance on an external sibling skill for the actual execution and verification steps.
Suggestions
Fix the script paths to resolve relative to the skill's own directory (e.g., scripts/cleanup-worktree.sh) instead of the hardcoded .cursor/skills/cleanup-github-worktree/ location, so the commands match the shipped bundle.
Remove the redundant day-mapping table in Step 1 (the mapping is already stated in the AskQuestion options) and drop the duplicate path table in 脚本说明.
Add an explicit post-batch verification step (e.g., re-run git worktree list to confirm the removed worktrees are gone) rather than relying solely on the delegated skill's output for the summary table.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and command-driven, but has minor trimmable redundancy: the Step 1 day-mapping table repeats the mapping already embedded in the AskQuestion options ("1 周(对应 7 天)"), and the 脚本说明 table re-lists script paths already given in Step 2. This fits 'Efficient; minor instances of over-explanation that could be trimmed'; not score 5 because of that duplication, not score 3 because there are no conceptual explanations of things Claude already knows. | 4 / 5 |
Actionability | Concrete executable commands with argument placeholders are given per OS ("powershell -File .../cleanup-worktree.ps1 -Days <天数>", "bash .../cleanup-worktree.sh -d <天数>"), AskQuestion prompts and options are spelled out, and batch rules (skip-closed, skip-uncommitted, continue-on-failure) are specific. Not score 5 because the paths assume a fixed install location (.cursor/skills/...) that does not match the actual bundle layout, and per-branch execution depends on an external skill's Steps 1-5 without restating any of its commands. | 4 / 5 |
Workflow Clarity | A destructive batch workflow with real checkpoints: a pre-flight master-branch check, a non-destructive preview step, an explicit AskQuestion confirmation gate before deletion, per-branch error isolation ("单个分支处理失败不中断整批"), and a per-branch summary report. This satisfies the batch/destructive validation cap. Not score 5 because post-execution verification is entirely delegated to the external close-github-worktree-done skill and the final table is a presentation of assumed results rather than a stated validation step. | 4 / 5 |
Progressive Disclosure | Structure is appropriate: orchestration lives in SKILL.md while the filtering logic is externalized to two real scripts (both present in ./scripts/ and clearly inventoried in a table), with the output contract (branch names, one per line) documented so the interface is unambiguous. Not score 5 because the referenced paths (.cursor/skills/cleanup-github-worktree/scripts/...) do not match the actual bundle location, and the script-internals section (脚本职责) partially duplicates logic detail that belongs in the scripts themselves. | 4 / 5 |
Total | 16 / 20 Passed |