[PR #4856] [MERGED] Enable Passthrough for VT Input Mode in ConPty #25983

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/4856
Author: @carlos-zamora
Created: 3/9/2020
Status: Merged
Merged: 3/10/2020
Merged by: @undefined

Base: masterHead: dev/cazamor/vtmm/dispatch-and-engine-changes


📝 Commits (2)

  • 0057046 Enable passthrough for VTInputMode in ConPty
  • 09c02a7 only return false if we're in ConsolePty mode

📊 Changes

12 files changed (+203 additions, -10 deletions)

View changed files

📝 src/host/VtInputThread.cpp (+6 -0)
📝 src/host/outputStream.cpp (+13 -0)
📝 src/host/outputStream.hpp (+2 -0)
📝 src/terminal/adapter/IInteractDispatch.hpp (+2 -0)
📝 src/terminal/adapter/InteractDispatch.cpp (+11 -0)
📝 src/terminal/adapter/InteractDispatch.hpp (+2 -0)
📝 src/terminal/adapter/adaptDispatch.cpp (+96 -8)
📝 src/terminal/adapter/conGetSet.hpp (+2 -0)
📝 src/terminal/adapter/ut_adapter/adapterTest.cpp (+5 -0)
📝 src/terminal/parser/InputStateMachineEngine.cpp (+54 -1)
📝 src/terminal/parser/InputStateMachineEngine.hpp (+3 -1)
📝 src/terminal/parser/ut_parser/InputEngineTest.cpp (+7 -0)

📄 Description

This commit enables passthrough mode for VT Input Mode in ConPty. This
will be used to pass VT Input from Mouse Mode directly to the app on the
other side.

References

#545 - VT Mouse Mode (Terminal)
#376 - VT Mouse Mode (ConPty)

Detailed Description of the Pull Request / Additional comments

ConHost

  • Set the callback for the InputEngine.
  • Retrieve IsInVirtualTerminalInputMode from the InputBuffer

Adapter (Dispatch)

Retrieve VTInputMode setting from ConHost

Parser

  • Add a callback to passthrough unknown input sequences directly to the
    input queue.
  • If we're in VTInputMode, use the callback

Validation Steps Performed

Tests should still pass.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/4856 **Author:** [@carlos-zamora](https://github.com/carlos-zamora) **Created:** 3/9/2020 **Status:** ✅ Merged **Merged:** 3/10/2020 **Merged by:** [@undefined](undefined) **Base:** `master` ← **Head:** `dev/cazamor/vtmm/dispatch-and-engine-changes` --- ### 📝 Commits (2) - [`0057046`](https://github.com/microsoft/terminal/commit/0057046fe8231760018754ad2605013a262d1d7c) Enable passthrough for VTInputMode in ConPty - [`09c02a7`](https://github.com/microsoft/terminal/commit/09c02a7e12f3e4fa7ee1ec889ef0b8e06ac4c5f9) only return false if we're in ConsolePty mode ### 📊 Changes **12 files changed** (+203 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `src/host/VtInputThread.cpp` (+6 -0) 📝 `src/host/outputStream.cpp` (+13 -0) 📝 `src/host/outputStream.hpp` (+2 -0) 📝 `src/terminal/adapter/IInteractDispatch.hpp` (+2 -0) 📝 `src/terminal/adapter/InteractDispatch.cpp` (+11 -0) 📝 `src/terminal/adapter/InteractDispatch.hpp` (+2 -0) 📝 `src/terminal/adapter/adaptDispatch.cpp` (+96 -8) 📝 `src/terminal/adapter/conGetSet.hpp` (+2 -0) 📝 `src/terminal/adapter/ut_adapter/adapterTest.cpp` (+5 -0) 📝 `src/terminal/parser/InputStateMachineEngine.cpp` (+54 -1) 📝 `src/terminal/parser/InputStateMachineEngine.hpp` (+3 -1) 📝 `src/terminal/parser/ut_parser/InputEngineTest.cpp` (+7 -0) </details> ### 📄 Description This commit enables passthrough mode for VT Input Mode in ConPty. This will be used to pass VT Input from Mouse Mode directly to the app on the other side. ## References #545 - VT Mouse Mode (Terminal) #376 - VT Mouse Mode (ConPty) ## Detailed Description of the Pull Request / Additional comments ### ConHost - Set the callback for the InputEngine. - Retrieve `IsInVirtualTerminalInputMode` from the InputBuffer ### Adapter (Dispatch) Retrieve `VTInputMode` setting from ConHost ### Parser - Add a callback to passthrough unknown input sequences directly to the input queue. - If we're in VTInputMode, use the callback ## Validation Steps Performed Tests should still pass. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:13:06 +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#25983