Skip to content

Conversation

@newhook
Copy link
Owner

@newhook newhook commented Jan 28, 2026

Summary

  • Fixed TUI's destroyWork function to call EnsureControlPlane() after scheduling the destroy worktree task
  • Without this fix, if the control plane wasn't already running, the scheduled destroy task would sit in the database forever
  • Consistent with how createWork and other TUI operations handle control plane spawning

Test plan

  • Verify build passes
  • Verify tests pass
  • Manual test: In TUI, destroy a work when control plane tab doesn't exist - should spawn control plane and process the destroy

🤖 Generated with Claude Code

newhook and others added 2 commits January 28, 2026 15:42
The TUI's destroyWork function schedules a destroy_worktree task but
wasn't ensuring the control plane was running to process it. If the
control plane wasn't already running, the scheduled task would sit
in the database forever.

This adds EnsureControlPlane() after scheduling the destroy task,
consistent with how createWork and other TUI operations handle it.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Same issue as destroyWork - TriggerPRFeedbackCheck schedules a task
but wasn't ensuring the control plane was running to process it.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@newhook newhook merged commit 74686a2 into main Jan 28, 2026
3 checks passed
@newhook newhook deleted the fix-tui-destroy-ensure-control-plane branch January 28, 2026 20:45
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