[PR #8607] Add Microsoft.Terminal.Remoting.dll #27249

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

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

State: closed
Merged: Yes


Adds a Microsoft.Terminal.Remoting.dll to our solution. This DLL will
be responsible for all the Monarch/Peasant work that's been described in
#7240 & #8135.

This PR does not implement the Monarch/Peasant architecture in any
significant way. The goal of this PR is to just to establish the project
layout, and the most basic connections. This should make reviewing the
actual meat of the implementation (in a later PR) easier. It will also
give us the opportunity to include some of the basic weird things we're
doing (with CoRegisterClass) in the Terminal now, and get them
selfhosted, before building on them too much.

This PR does have windows registering the Monarch class with COM. When
windows are created, they'll as the Monarch if they should create a new
window or not. In this PR, the Monarch will always reply "yes, please
make a new window".

Similar to other projects in our solution, we're adding 3 projects here:

  • Microsoft.Terminal.Remoting.lib: the actual implementation, as a
    static lib.
  • Microsoft.Terminal.Remoting.dll: The implementation linked as a DLL,
    for use in WindowsTerminal.exe.
  • Remoting.UnitTests.dll: A unit test dll that links with the static
    lib.

There are plenty of TODOs scattered about the code. Clearly, most of
this isn't implemented yet, but I do have more WIP branches. I'm using
projects/5 as my
notation for TODOs that are too small for an issue, but are part of the
whole Process Model 2.0 work.

References

  • #5000 - this is the process model megathread
  • #7240 - The process model 2.0 spec.
  • #8135 - the window management spec. (please review me, I have 0/3
    signoffs even after the discussion we had 😢)
  • #8171 - the Monarch/peasant sample. (please review me, I have 1/2)

PR Checklist

  • Closes nothing, this is just infrastructure
  • I work here
  • Tests added/passed
  • [n/a] Requires documentation to be updated
**Original Pull Request:** https://github.com/microsoft/terminal/pull/8607 **State:** closed **Merged:** Yes --- Adds a `Microsoft.Terminal.Remoting.dll` to our solution. This DLL will be responsible for all the Monarch/Peasant work that's been described in #7240 & #8135. This PR does _not_ implement the Monarch/Peasant architecture in any significant way. The goal of this PR is to just to establish the project layout, and the most basic connections. This should make reviewing the actual meat of the implementation (in a later PR) easier. It will also give us the opportunity to include some of the basic weird things we're doing (with `CoRegisterClass`) in the Terminal _now_, and get them selfhosted, before building on them too much. This PR does have windows registering the `Monarch` class with COM. When windows are created, they'll as the Monarch if they should create a new window or not. In this PR, the Monarch will always reply "yes, please make a new window". Similar to other projects in our solution, we're adding 3 projects here: * `Microsoft.Terminal.Remoting.lib`: the actual implementation, as a static lib. * `Microsoft.Terminal.Remoting.dll`: The implementation linked as a DLL, for use in `WindowsTerminal.exe`. * `Remoting.UnitTests.dll`: A unit test dll that links with the static lib. There are plenty of TODOs scattered about the code. Clearly, most of this isn't implemented yet, but I do have more WIP branches. I'm using [`projects/5`](https://github.com/microsoft/terminal/projects/5) as my notation for TODOs that are too small for an issue, but are part of the whole Process Model 2.0 work. ## References * #5000 - this is the process model megathread * #7240 - The process model 2.0 spec. * #8135 - the window management spec. (please review me, I have 0/3 signoffs even after the discussion we had 😢) * #8171 - the Monarch/peasant sample. (please review me, I have 1/2) ## PR Checklist * [x] Closes nothing, this is just infrastructure * [x] I work here * [x] Tests added/passed * [n/a] Requires documentation to be updated
claunia added the pull-request label 2026-01-31 09:20:52 +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#27249