[PR #15240] (A better) Refactoring of how connection restarting is handled #30536

Open
opened 2026-01-31 09:41:25 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/15240

State: closed
Merged: Yes


A different take on #14548.

We didn't love that a connection could transition back in the state diagram, from Closed -> Start. That felt wrong. To remedy this, we're going to allow the ControlCore to...

ASK the app to restart its connection. This is a much more sensible approach, than leaving the ConnectionInfo in the core and having the core do the restart itself. That's mental.

Cleanup from #14060

Closes #14327

Obsoletes #14548

**Original Pull Request:** https://github.com/microsoft/terminal/pull/15240 **State:** closed **Merged:** Yes --- A different take on #14548. > We didn't love that a connection could transition back in the state diagram, from Closed -> Start. That felt wrong. To remedy this, we're going to allow the ControlCore to... ASK the app to restart its connection. This is a much more sensible approach, than leaving the ConnectionInfo in the core and having the core do the restart itself. That's mental. Cleanup from #14060 Closes #14327 Obsoletes #14548
claunia added the pull-request label 2026-01-31 09:41:25 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#30536