Skip to content

fix(app): terminal no longer hangs on exit or ctrl + D and closes the pane#9506

Merged
adamdotdevin merged 1 commit intoanomalyco:devfrom
ProdigyRahul:fix/terminal-exit-hang
Jan 20, 2026
Merged

fix(app): terminal no longer hangs on exit or ctrl + D and closes the pane#9506
adamdotdevin merged 1 commit intoanomalyco:devfrom
ProdigyRahul:fix/terminal-exit-hang

Conversation

@ProdigyRahul
Copy link
Copy Markdown
Contributor

What does this PR do?

fixes terminal hanging for 5-10 seconds when user types exit or Ctrl+D. Previously, the terminal would become unresponsive and eventually respawn a new terminal.

2026-01-20.02-33-17.mp4

How did you verify your code works?

  1. Opened terminal with Ctrl+
  2. Typed exit → terminal tab closes immediately and pane toggles closed (no hang)
  3. Tested Ctrl+D → same immediate close behavior
  4. With multiple terminals: closed one → others remain, pane stays open
  5. Closed last terminal → pane closes automatically
  6. Pressed Ctrl+ again → pane opens, new terminal auto-creates as expected

its very annoying to have this issue :)

@github-actions
Copy link
Copy Markdown
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions
Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

No duplicate PRs found

@ProdigyRahul
Copy link
Copy Markdown
Contributor Author

fixes #9507 #6004

}),
)

sdk.event.on("pty.exited", (event) => {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

listens for when a terminal process exits and cleans up ui :)

@adamdotdevin adamdotdevin merged commit 01b1294 into anomalyco:dev Jan 20, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants