メニュー
Use when implementation is done and tests pass, and the user must choose how to integrate or retain completed branch work.
デフォルトでは、最初にソースを確認する Prompt が選択されています。直接コマンドに切り替えるか、ローカルコピーをダウンロードすることもできます。
インストールを決める前に、SKILL.md と SkillsMP に表示されている付属ファイルをお読みください。
Codex または Claude でインストール この Prompt をコピーして Codex、Claude、または他のアシスタントに貼り付けると、Skill ページを確認してインストールできます。
直接コマンドでは確認用 Prompt が省略されます。実行前にソースを確認してください。
npx skills add https://github.com/evanfang0054/agent-harness --skill finishing-a-development-branchコマンドは1行のまま表示されます。コピー前に横へスクロールして全体を確認してください。
ローカルで確認しますか？SkillsMP が現在取得できるファイルをダウンロードできます。
You MUST use this before any creative work — features, components, behavior changes. Explores intent, requirements, and design before implementation.
Use when receiving code review feedback. Verify before implementing; technical correctness over blind agreement.
Use when executing implementation plans with independent tasks in the current session
SKILL.md を表示中
| name | finishing-a-development-branch |
| description | Use when implementation is done and tests pass, and the user must choose how to integrate or retain completed branch work. |
Verify the completed result, offer only safe next actions, and preserve work unless the user explicitly requests its destruction.
Announce at start: "I'm using the finishing-a-development-branch skill to complete this work."
Run the relevant project test suite before offering choices. If it fails, STOP: report failures and fix them before merge or PR.
Before any cd to the main repository or base branch, capture the current repository data:
GIT_DIR=$(git rev-parse --git-dir)
GIT_COMMON=$(git rev-parse --git-common-dir)
WORKTREE_PATH=$(git rev-parse --show-toplevel)
Determine the base branch without changing directories:
git merge-base HEAD main 2>/dev/null || git merge-base HEAD master 2>/dev/null
Resolve the current project's common root from the captured common Git directory. git rev-parse --git-common-dir may return either an absolute or a relative .git path, so handle both before any directory change:
if [[ "$GIT_COMMON" = /* ]]; then
PROJECT_ROOT=$(dirname "$GIT_COMMON")
else
PROJECT_ROOT=$(cd "$(dirname )" && )
Only the captured WORKTREE_PATH is eligible for removal. Use this guard exactly; its slash-delimited patterns allow descendants of .worktrees/ and worktrees/ only, not prefix lookalikes such as .worktrees-evil:
# BEGIN owned-worktree-removal-guard
remove_captured_owned_worktree() {
case "$WORKTREE_PATH" in
"$PROJECT_ROOT"/.worktrees/*|"$PROJECT_ROOT"/worktrees/*)
git worktree remove "$WORKTREE_PATH"
;;
*)
printf 'Preserving non-project-owned worktree: %s\n' "$WORKTREE_PATH"
;;
esac
}
# END owned-worktree-removal-guard
Never place git worktree remove outside this guard. It preserves host-owned, parent-owned, and unmatched workspace paths.
For a normal branch or named worktree, present exactly:
Implementation complete. What would you like to do?
1. Merge back to <base-branch> locally
2. Push and create a Pull Request
3. Keep the branch as-is (I'll handle it later)
Which option?
For detached HEAD, present exactly:
Implementation complete. What would you like to do?
1. Push as a new branch and create a Pull Request
2. Keep as-is
Which option?
The default menu never includes discard.
Only after the user selects merge:
cd <base-branch>
git pull
git merge <feature-branch>
<test command>
If merged-result verification fails: STOP. Keep the branch and worktree for investigation. Do not delete anything.
After successful verification, delete the feature branch only if appropriate. Remove the captured worktree only through remove_captured_owned_worktree.
Push the branch and create a pull request using the available forge workflow. gh pr create is acceptable when available, but it is not the only valid forge workflow. Keep the branch, worktree, and artifacts.
Report the retained branch or detached state. Keep the worktree and artifacts.
PR and Keep preserve all branch, worktree, and SDD artifacts. Finishing options do not run SDD cleanup; the SDD final-review-clean transition owns current-plan cleanup.
Discuss discard only after the user explicitly asks to discard the work. Show the exact branch, commits, and eligible worktree path, then require this exact confirmation:
Type `discard` to confirm.
Only after that exact response may destructive commands run. For an eligible captured project-owned worktree, call remove_captured_owned_worktree; otherwise its fallback preserves the worktree.
| Excuse | Reality |
|---|---|
| "Tests passed earlier this session" | Run the suite on the tree you are about to integrate. A green run only proves the tree it ran on. |
| "They obviously want it merged" | Integration is your human partner's decision. Present the menu and wait. |
| "They seem done with this feature — I'll offer to discard it" | The menu is complete as written. Discard happens only when your human partner asks for it in so many words. |
| "'Yeah, get rid of it' counts as confirmation" | Only the typed word discard authorizes deletion. |
| "The PR is up, so the worktree is clutter now" | PR feedback gets fixed in that worktree. It stays until the work lands. |
| "This other worktree looks stale — I'll clean it too" | Clean up only worktrees under .worktrees/ or worktrees/. Everything else belongs to the host. |
| "The merged-result failure is probably flaky" | A failing merged result stops everything. Branch and worktree stay put while you investigate. |
| "The base branch is obviously main" | Confirm the fork point or ask. Merging into the wrong base is expensive to undo. |
| "The push was rejected — force-push will fix it" | A rejected push means the remote moved. Investigate; force-push only on your human partner's explicit request. |
Never:
Called by:
Pairs with: