[PR #16601] General improvements in preparation for #16598 #30955

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

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

State: closed
Merged: Yes


This contains all the parts of #16598 that aren't specific to session
restore, but are required for the code in #16598:

  • Adds new GUID<>String functions that remove the {} brackets.
  • Adds SessionId to the ITerminalConnection interface.
  • Flush the ApplicationState before we terminate the process.
  • Not monitoring state.json for changes is important as it prevents
    disturbing the session state while session persistence is ongoing.
    That's because when ApplicationState flushes to disk, the FS
    monitor will be triggered and reload the ApplicationState again.
**Original Pull Request:** https://github.com/microsoft/terminal/pull/16601 **State:** closed **Merged:** Yes --- This contains all the parts of #16598 that aren't specific to session restore, but are required for the code in #16598: * Adds new GUID<>String functions that remove the `{}` brackets. * Adds `SessionId` to the `ITerminalConnection` interface. * Flush the `ApplicationState` before we terminate the process. * Not monitoring `state.json` for changes is important as it prevents disturbing the session state while session persistence is ongoing. That's because when `ApplicationState` flushes to disk, the FS monitor will be triggered and reload the `ApplicationState` again.
claunia added the pull-request label 2026-01-31 09:44:04 +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#30955