Imported from yash-gadodia/claude-init (
templates/skills/finish/SKILL.md). Install upstream withnpx skills add yash-gadodia/claude-init --skill finish. Copyright stays with the author.
Finish: Land Your Work
Verify, present options, execute, clean up.
When This Triggers
Automatically after all implementation tasks pass review and verification.
Process
1. Verify
Run the FULL test suite. If ANY test fails, STOP. Fix before proceeding.
If docs/definition-of-done.md exists, run through its checklist too — tests passing is necessary but not sufficient to land.
2. Determine Base Branch
Check what branch you split from (git log --oneline --graph -10). Usually main or master. Ask if unclear.
3. Present Options
Show exactly these four choices:
- Merge locally — merge feature branch into base, delete feature branch
- Push & create PR — push branch, create PR with summary + test plan
- Keep branch — preserve as-is for later
- Discard — permanently delete branch and all changes
4. Execute
Merge locally:
git checkout <base>andgit pullgit merge <feature-branch>- Run tests again — merges can introduce conflicts that break things
git branch -d <feature-branch>
Push & create PR:
git push -u origin <feature-branch>- Create PR via
gh pr createwith summary and test plan sections
Keep branch:
- Do nothing. Note branch name for the user.
Discard:
- Ask user to type "discard" to confirm
git checkout <base>git branch -D <feature-branch>- Remove worktree if applicable
5. Clean Up
- Remove git worktree if applicable
- Summarize what was done
Rules
- NEVER merge without passing tests
- NEVER delete work without typed "discard" confirmation
- Always present all 4 options — don't assume which one the user wants
- Re-run tests after merge (merge can introduce breakage)
- If on main/master (no feature branch), only offer PR/keep/discard
