drydock-cli 3.0.92__tar.gz → 3.0.93__tar.gz

{drydock_cli-3.0.92/drydock_cli.egg-info → drydock_cli-3.0.93}/PKG-INFO

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

@@ -942,7 +942,13 @@ def tool_bash(params: dict, config: dict) -> str:
             # 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,
+            # errors="replace" so BINARY / non-UTF8 output (cat a binary, a tool
+            # emitting raw bytes, gzip to stdout) decodes to replacement chars
+            # instead of raising UnicodeDecodeError inside the reader thread —
+            # which killed the thread and handed the agent "(no output)", losing
+            # even the text parts of mixed output.
+            text=True, encoding="utf-8", errors="replace",
+            cwd=config.get("cwd"), start_new_session=True,
         )
         config.setdefault("_abort", {})["proc"] = proc
         # Read output in a daemon thread with a HARD byte cap, so memory can't

{drydock_cli-3.0.92 → drydock_cli-3.0.93/drydock_cli.egg-info}/PKG-INFO

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

@@ -50,6 +50,7 @@ tests/test_advisor.py
 tests/test_approval.py
 tests/test_back_command.py
 tests/test_bash_background.py
+tests/test_bash_binary_output.py
 tests/test_bash_output_bounding.py
 tests/test_bash_process_group.py
 tests/test_bash_safety.py

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

drydock_cli-3.0.93/tests/test_bash_binary_output.py

@@ -0,0 +1,33 @@
+"""tool_bash must not choke on binary / non-UTF8 stdout. text=True with the
+default (strict) decode raised UnicodeDecodeError INSIDE the reader thread — the
+thread died and the agent got "(no output)", losing even the text parts of mixed
+output. errors="replace" keeps it non-crashing and preserves the text. Found by
+probing the binary-output I/O path directly."""
+from __future__ import annotations
+
+from drydock.tools import tool_bash
+
+
+def _run(cmd):
+    return tool_bash({"command": cmd, "timeout": 8}, {"cwd": "/tmp"})
+
+
+def test_invalid_utf8_does_not_crash_and_keeps_text():
+    out = _run(r"printf '\xff\xfe hello'")
+    assert "hello" in out and "(no output)" not in out
+
+
+def test_mixed_text_and_binary_preserves_text():
+    # the readable lines must survive even though raw bytes sit between them
+    out = _run(r"echo START; printf '\xc0\xc1\xff'; echo END")
+    assert "START" in out and "END" in out
+
+
+def test_binary_file_head_returns_something():
+    out = _run("head -c 80 /bin/ls")
+    assert out.strip() and "(no output)" not in out and "EXCEPTION" not in out
+
+
+def test_plain_text_unaffected():
+    out = _run("echo hello && echo world")
+    assert "hello" in out and "world" in out