fix: force system conpty on windows

refs #761

1 files changed, 37 insertions, 0 deletions

diff --git a/vendor/patches/portable-pty/0001-force-system-conpty.patch b/vendor/patches/portable-pty/0001-force-system-conpty.patch
new file mode 100644
index 00000000..6074df57
--- /dev/null
+++ b/vendor/patches/portable-pty/0001-force-system-conpty.patch
@@ -0,0 +1,37 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: herdr maintainers <maintainers@herdr.dev>
+Date: Tue, 23 Jun 2026 00:00:00 +0000
+Subject: [PATCH] force system ConPTY
+
+Herdr should use the ConPTY functions exported by kernel32.dll and must not
+load a bare conpty.dll from PATH. portable-pty's sideload probe is intentional
+for WezTerm's bundled OpenConsole.exe/conpty.dll pair, but Herdr does not ship
+that pair.
+
+Herdr issue: https://github.com/ogulcancelik/herdr/issues/761
+Vendored base: portable-pty 0.9.0
+---
+ vendor/portable-pty/src/win/psuedocon.rs | 9 +--------
+ 1 file changed, 1 insertion(+), 8 deletions(-)
+
+diff --git a/vendor/portable-pty/src/win/psuedocon.rs b/vendor/portable-pty/src/win/psuedocon.rs
+index 0000000..0000000 100644
+--- a/vendor/portable-pty/src/win/psuedocon.rs
++++ b/vendor/portable-pty/src/win/psuedocon.rs
+@@ -49,14 +49,7 @@ fn load_conpty() -> ConPtyFuncs {
+         "this system does not support conpty.  Windows 10 October 2018 or newer is required",
+     );
+-
+-    // We prefer to use a sideloaded conpty.dll and openconsole.exe host deployed
+-    // alongside the application.  We check for this after checking for kernel
+-    // support so that we don't try to proceed and do something crazy.
+-    if let Ok(sideloaded) = ConPtyFuncs::open(Path::new("conpty.dll")) {
+-        sideloaded
+-    } else {
+-        kernel
+-    }
++
++    kernel
+ }
+
+ lazy_static! {