fix(pty): prevent crash loop when closing app with running processes

[rabanspiegel]

Summary

• Guard IPC sends to destroyed WebContents with safeSendToOwner() helper
• Add WebContents destruction listener to kill PTYs when window closes
• Add before-quit handler to clean up all PTYs on app shutdown
• Guard pty:started broadcast with per-window isDestroyed() check

Test plan

• Start app with npm run dev
• Create a task with a running agent (e.g., Claude)
• Close the app (Cmd+Q) while the agent is active
• Verify clean exit with no crash dialog
• Verify pty:ownerDestroyed logs appear in terminal output

Fixes #676

---

Note

Improves PTY lifecycle safety to avoid crashes during window/app shutdown.

• Adds safeSendToOwner() to guard IPC sends when WebContents is destroyed
• Attaches per-PTY onData/onExit listeners that use safe sending and ignore stale onExit events
• Registers WebContents destroyed handler to maybeMarkProviderFinish(), killPty(), and clear PTY ownership/listener state
• Guards pty:started broadcast by checking webContents.isDestroyed() per window
• On app before-quit, iterates all PTYs to maybeMarkProviderFinish(), killPty(), and clears owners/listeners
• Minor: import app from electron to support shutdown handling

^{Written by Cursor Bugbot for commit 39e7aed. This will update automatically on new commits. Configure here.}

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Review	Updated (UTC)
docs	Ready	Preview, Comment	Jan 21, 2026 10:30pm

[cursor]

Bugbot Autofix prepared a fix for the bug found in the latest run.

• ✅ Fixed: Stale destruction listener kills PTY when owner changes
  • Added ownership verification in the destruction listener to prevent stale listeners from killing PTYs reused by other windows.

[cursor]

Bugbot Autofix prepared a fix for the bug found in the latest run.

• ✅ Fixed: Race condition: old PTY onExit corrupts new PTY state
  • Added stale check to onExit callback that verifies the PTY is still active (getPty(id) === proc) before modifying owners/listeners maps, preventing old PTY exit from corrupting new PTY state.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is ON. A Cloud Agent has been kicked off to fix the reported issue.}

[cursor]

Bugbot Autofix prepared a fix for the bug found in the latest run.

• ✅ Fixed: Missing telemetry cleanup in PTY shutdown handlers
  • Added maybeMarkProviderFinish calls before killPty in both the destroyed listener and before-quit handler to ensure telemetry cleanup matches the pty:kill pattern.