drydock-cli 3.0.89__tar.gz → 3.0.90__tar.gz

{drydock_cli-3.0.89/drydock_cli.egg-info → drydock_cli-3.0.90}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 3.0.89
+Version: 3.0.90
 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.89 → drydock_cli-3.0.90}/drydock/tools/__init__.py

@@ -911,6 +911,13 @@ def tool_bash(params: dict, config: dict) -> str:
     try:
         proc = subprocess.Popen(
             cmd, shell=True, stdout=subprocess.PIPE, stderr=subprocess.STDOUT,
+            # stdin=DEVNULL so a command that reads stdin gets immediate EOF
+            # (correct for a non-interactive tool) instead of inheriting the
+            # TUI's terminal — where it would steal the user's keystrokes or hang
+            # waiting for input Textual has captured. The agent still feeds input
+            # explicitly via a pipe/redirect (echo x | cmd, cmd < file), which
+            # overrides this.
+            stdin=subprocess.DEVNULL,
             text=True, cwd=config.get("cwd"), start_new_session=True,
         )
         config.setdefault("_abort", {})["proc"] = proc

{drydock_cli-3.0.89 → drydock_cli-3.0.90/drydock_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 3.0.89
+Version: 3.0.90
 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.89 → drydock_cli-3.0.90}/drydock_cli.egg-info/SOURCES.txt

@@ -53,6 +53,7 @@ tests/test_bash_background.py
 tests/test_bash_output_bounding.py
 tests/test_bash_process_group.py
 tests/test_bash_safety.py
+tests/test_bash_stdin.py
 tests/test_bash_timeout_network.py
 tests/test_cci.py
 tests/test_cli_agents.py

{drydock_cli-3.0.89 → drydock_cli-3.0.90}/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.89"
+version = "3.0.90"
 description = "Drydock — a local, provider-agnostic terminal coding agent for local LLMs"
 readme = "README.md"
 requires-python = ">=3.11"

drydock_cli-3.0.90/tests/test_bash_stdin.py

@@ -0,0 +1,40 @@
+"""tool_bash must NOT inherit the caller's stdin (in the TUI that's Textual's
+terminal): a command that reads stdin should get immediate EOF, not hang waiting
+for input or steal the user's keystrokes. The agent still feeds input explicitly
+via a pipe/redirect, which overrides the DEVNULL stdin."""
+from __future__ import annotations
+
+import time
+
+from drydock.tools import tool_bash
+
+
+def _run(cmd, timeout=6):
+    t0 = time.monotonic()
+    out = tool_bash({"command": cmd, "timeout": timeout}, {"cwd": "/tmp"})
+    return time.monotonic() - t0, out
+
+
+def test_stdin_reader_gets_eof_not_hang():
+    dt, out = _run("read x; echo \"got:[$x]\"")
+    assert dt < 2.0 and "got:[]" in out, "reading stdin should EOF instantly, not hang"
+
+
+def test_cat_no_input_eofs():
+    dt, out = _run("cat && echo DONE")
+    assert dt < 2.0 and "DONE" in out
+
+
+def test_python_input_raises_eoferror_cleanly():
+    _, out = _run('python3 -c "input()" 2>&1 | tail -1')
+    assert "EOFError" in out
+
+
+def test_piped_input_still_works():
+    _, out = _run("echo hello | cat")
+    assert "hello" in out
+
+
+def test_redirect_from_file_still_works():
+    _, out = _run("printf 'a\\nb\\n' > /tmp/ddin.txt; wc -l < /tmp/ddin.txt")
+    assert "2" in out