drydock-cli 3.0.108__tar.gz → 3.0.109__tar.gz

{drydock_cli-3.0.108/drydock_cli.egg-info → drydock_cli-3.0.109}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 3.0.108
+Version: 3.0.109
 Summary: Drydock — a local, provider-agnostic terminal coding agent for local LLMs
 Author: Frank Bobe III
 License-Expression: Apache-2.0

{drydock_cli-3.0.108 → drydock_cli-3.0.109}/drydock/tools/__init__.py

@@ -6,6 +6,7 @@ Each tool is a function (params, config) -> str.
 from __future__ import annotations
 
 import os
+import sys
 import re
 import difflib
 import glob as _glob
@@ -92,28 +93,40 @@ def _detect_bash() -> str | None:
     return None
 
 
-_IS_WINDOWS = os.name == "nt"
+# Windows even if a non-native Python (rare) reports a posix os.name — sys.platform
+# is the reliable signal on CPython.
+_IS_WINDOWS = os.name == "nt" or sys.platform.startswith("win")
 
 
 def _detect_shell() -> tuple[str, str]:
     """Pick the shell tool_bash runs commands in — returns (kind, path).
 
+    Override with env DRYDOCK_SHELL=powershell|cmd|bash to force one. Otherwise:
     POSIX → bash (so the model's bash-isms work), else /bin/sh.
     Windows → PowerShell (what users actually launch drydock in — pwsh, else
-    Windows PowerShell), else cmd.exe. WSL / Git-Bash is NOT required on Windows;
-    if you'd rather use them, install bash and it's picked up on POSIX-like paths.
+    Windows PowerShell), else cmd.exe. WSL / Git-Bash is NEVER required on Windows.
     """
     import shutil
-    if _IS_WINDOWS:
-        for name in ("pwsh", "powershell"):
-            p = shutil.which(name)
-            if p:
-                return ("powershell", p)
+
+    def _powershell():
+        p = shutil.which("pwsh") or shutil.which("powershell")
+        return ("powershell", p) if p else None
+
+    def _bash():
+        b = _detect_bash()
+        return ("bash", b) if b else ("sh", shutil.which("sh") or "/bin/sh")
+
+    forced = os.environ.get("DRYDOCK_SHELL", "").strip().lower()
+    if forced in ("powershell", "pwsh"):
+        return _powershell() or ("cmd", shutil.which("cmd") or "cmd.exe")
+    if forced == "cmd":
         return ("cmd", shutil.which("cmd") or "cmd.exe")
-    b = _detect_bash()
-    if b:
-        return ("bash", b)
-    return ("sh", shutil.which("sh") or "/bin/sh")
+    if forced in ("bash", "sh"):
+        return _bash()
+
+    if _IS_WINDOWS:
+        return _powershell() or ("cmd", shutil.which("cmd") or "cmd.exe")
+    return _bash()
 
 
 # Resolved once at import (in whatever environment drydock runs — Linux/macOS,

{drydock_cli-3.0.108 → drydock_cli-3.0.109/drydock_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 3.0.108
+Version: 3.0.109
 Summary: Drydock — a local, provider-agnostic terminal coding agent for local LLMs
 Author: Frank Bobe III
 License-Expression: Apache-2.0

{drydock_cli-3.0.108 → drydock_cli-3.0.109}/pyproject.toml

@@ -7,7 +7,7 @@ build-backend = "setuptools.build_meta"
 # PyPI distribution name is drydock-cli (the established install name, continued
 # from the retired v2 fork); the import package + CLI command stay `drydock`.
 name = "drydock-cli"
-version = "3.0.108"
+version = "3.0.109"
 description = "Drydock — a local, provider-agnostic terminal coding agent for local LLMs"
 readme = "README.md"
 requires-python = ">=3.11"

{drydock_cli-3.0.108 → drydock_cli-3.0.109}/tests/test_windows_shell.py

@@ -32,21 +32,16 @@ def test_powershell_invocation_argv(monkeypatch):
     """When the shell is PowerShell, Popen is called [pwsh -NoProfile
     -NonInteractive -Command <cmd>] with shell=False — not shell=True."""
     seen = {}
-    real = subprocess.Popen
 
-    class FakeProc:
-        stdout = None
-        pid = 1
-        def wait(self): return 0
-        def poll(self): return 0
     def spy(popen_cmd, *a, **kw):
-        seen["cmd"] = popen_cmd; seen["shell"] = kw.get("shell")
+        seen["cmd"] = popen_cmd
+        seen["shell"] = kw.get("shell")
         raise RuntimeError("stop-after-capture")  # don't actually run pwsh on Linux
 
     monkeypatch.setattr(T, "_SHELL_KIND", "powershell")
     monkeypatch.setattr(T, "_SHELL_PATH", r"C:\ps\pwsh.exe")
     monkeypatch.setattr(subprocess, "Popen", spy)
-    out = T.tool_bash({"command": "Get-ChildItem", "timeout": 5}, {"cwd": "."})
+    T.tool_bash({"command": "Get-ChildItem", "timeout": 5}, {"cwd": "."})
     assert seen["shell"] is False
     assert seen["cmd"][:4] == [r"C:\ps\pwsh.exe", "-NoProfile", "-NonInteractive", "-Command"]
     assert seen["cmd"][4] == "Get-ChildItem"
@@ -65,3 +60,20 @@ def test_prompt_note_windows_cmd():
 
 def test_prompt_note_empty_on_posix():
     assert tuning._shell_env_note() == ""
+
+
+def test_env_override_forces_powershell():
+    with mock.patch.dict("os.environ", {"DRYDOCK_SHELL": "powershell"}), \
+         mock.patch("shutil.which", side_effect=lambda n: r"C:\pwsh.exe" if n in ("pwsh", "powershell") else None):
+        assert T._detect_shell()[0] == "powershell"
+
+
+def test_env_override_forces_cmd():
+    with mock.patch.dict("os.environ", {"DRYDOCK_SHELL": "cmd"}), \
+         mock.patch("shutil.which", return_value=r"C:\cmd.exe"):
+        assert T._detect_shell()[0] == "cmd"
+
+
+def test_env_override_forces_bash():
+    with mock.patch.dict("os.environ", {"DRYDOCK_SHELL": "bash"}):
+        assert T._detect_shell()[0] in ("bash", "sh")