Skip to content

Conversation

@max-sixty
Copy link
Owner

Summary

  • Move the "Removing worktree..." progress message to appear after pre-remove hooks run, not before
  • Fixes confusing output where a slow hook made it unclear whether the hook or removal was hanging
  • Aligns with wt commit pattern where pre-commit hooks run before "Committing..." message

Before:

◎ Removing current worktree...
◎ Running pre-remove project:docs:
   lsof -ti :10075 | xargs kill...
✓ Removed...

After:

◎ Running pre-remove project:docs:
   lsof -ti :10075 | xargs kill...
◎ Removing current worktree...
✓ Removed...

Test plan

  • All integration tests pass (767 tests)
  • Snapshot tests updated to reflect new output order
  • Pre-commit lints pass

🤖 Generated with Claude Code

Move the "Removing worktree..." progress message to appear after
pre-remove hooks run, not before. This fixes confusing output where
users saw "Removing..." then a slow hook ran, making it unclear
whether the hook or the removal was hanging.

Before:
  ◎ Removing current worktree...
  ◎ Running pre-remove project:docs:
     lsof -ti :10075 | xargs kill...
  ✓ Removed...

After:
  ◎ Running pre-remove project:docs:
     lsof -ti :10075 | xargs kill...
  ◎ Removing current worktree...
  ✓ Removed...

This aligns with the pattern in `wt commit` where pre-commit hooks
run before the "Committing..." message.

Co-Authored-By: Claude <[email protected]>
@max-sixty max-sixty merged commit a2c7878 into main Jan 10, 2026
18 of 19 checks passed
@max-sixty max-sixty deleted the pre-remove branch January 10, 2026 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants