drydock-cli 3.0.102__tar.gz → 3.0.104__tar.gz

{drydock_cli-3.0.102/drydock_cli.egg-info → drydock_cli-3.0.104}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 3.0.102
+Version: 3.0.104
 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.102 → drydock_cli-3.0.104}/drydock/extract.py

@@ -62,7 +62,7 @@ def _extract_pdf(path: str | Path) -> str | None:
         pass
     # 2) pypdf (optional dependency: drydock-cli[pdf]).
     try:
-        import pypdf
+        import pypdf  # pyright: ignore[reportMissingImports]
 
         reader = pypdf.PdfReader(str(path))
         text = "\n\n".join((pg.extract_text() or "") for pg in reader.pages).strip()
@@ -113,7 +113,7 @@ def pdf_backend_available() -> bool:
     if which("pdftotext"):
         return True
     try:
-        import pypdf  # noqa: F401
+        import pypdf  # noqa: F401  # pyright: ignore[reportMissingImports]
 
         return True
     except ImportError:

{drydock_cli-3.0.102 → drydock_cli-3.0.104}/drydock/tools/__init__.py

@@ -1067,6 +1067,11 @@ def tool_bash(params: dict, config: dict) -> str:
             if cancel is not None and cancel.is_set():
                 kill_process_group(proc)
                 proc.wait()
+                # The drain thread blocks in read(8192) until the buffer fills or
+                # EOF; killing the process delivers EOF, so join lets it append the
+                # last buffered output BEFORE we read chunks (else partial races
+                # the drain and comes back empty for small-output commands).
+                reader.join(1.0)
                 partial = _sanitize_bash_output(
                     _collapse_repeated_lines("".join(list(chunks)))
                 ).rstrip()
@@ -1090,6 +1095,7 @@ def tool_bash(params: dict, config: dict) -> str:
             if time.monotonic() - start > timeout:
                 kill_process_group(proc)
                 proc.wait()
+                reader.join(1.0)  # flush buffered output before building partial
                 bigger = min(timeout * 4, 1800)
                 msg = (
                     f"Error: command timed out after {timeout}s. If it is "
@@ -1256,7 +1262,7 @@ def tool_consult(params: dict, config: dict) -> str:
     """Ask the configured second/advisor model (e.g. Gemini) for a second opinion."""
     from drydock import advisor
 
-    question = (params.get("question") or params.get("prompt") or "").strip()
+    question = _as_str_arg(params.get("question") or params.get("prompt")).strip()
     if not question:
         return "Error: Consult needs a `question`."
     return advisor.consult(question, config, context=params.get("context", "") or "")
@@ -1398,7 +1404,7 @@ def tool_gitcommit(params: dict, config: dict) -> str:
     try:
         return gittools.commit(
             _git_cwd(config),
-            params.get("message") or "",
+            _as_text(params.get("message")),
             add_all=params.get("add_all", True) is not False,
         )
     except gittools.GitError as e:
@@ -1578,7 +1584,7 @@ def tool_websearch(params: dict, config: dict) -> str:
     """Search the internet (DuckDuckGo). Read-only; clean message when offline."""
     from drydock import web
 
-    query = (params.get("query") or "").strip()
+    query = _as_str_arg(params.get("query")).strip()
     if not query:
         return "Error: `WebSearch` needs a `query`."
     try:
@@ -1596,7 +1602,7 @@ def tool_webfetch(params: dict, config: dict) -> str:
     """Fetch a URL and return readable text. Read-only; clean message offline."""
     from drydock import web
 
-    url = (params.get("url") or "").strip()
+    url = _as_str_arg(params.get("url")).strip()
     if not url:
         return "Error: `WebFetch` needs a `url`."
     try:
@@ -1616,7 +1622,7 @@ def tool_knowledge(params: dict, config: dict) -> str:
     If no index exists, says so cleanly rather than erroring."""
     from drydock import graphrag
 
-    query = (params.get("query") or "").strip()
+    query = _as_str_arg(params.get("query")).strip()
     if not query:
         return "Error: `Knowledge` needs a `query` describing what to look up."
     cwd = config.get("cwd") or os.getcwd()

{drydock_cli-3.0.102 → drydock_cli-3.0.104/drydock_cli.egg-info}/PKG-INFO

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

@@ -103,6 +103,7 @@ tests/test_subagent.py
 tests/test_system_prompt_help.py
 tests/test_todo.py
 tests/test_tool_arg_coercion.py
+tests/test_tool_arg_coercion_more.py
 tests/test_tool_arg_parsing.py
 tests/test_tools_undo.py
 tests/test_tui.py

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

drydock_cli-3.0.104/tests/test_tool_arg_coercion_more.py

@@ -0,0 +1,37 @@
+"""Wrong-type args crashed more tools with AttributeError ('.strip()' on a
+list/int): GitCommit message, Knowledge query, Consult question, WebFetch url,
+WebSearch query. All coerced now — no tool should raise on a wrong-type arg."""
+from __future__ import annotations
+
+import drydock.tools as T
+
+
+def _no_crash(fn, **kw):
+    out = fn(kw, {"cwd": "/tmp"})   # must return a string, not raise
+    assert isinstance(out, str)
+    return out
+
+
+def test_gitcommit_message_list():
+    _no_crash(T.tool_gitcommit, message=["line1", "line2"])
+
+
+def test_gitcommit_message_int():
+    _no_crash(T.tool_gitcommit, message=42)
+
+
+def test_knowledge_query_list():
+    _no_crash(T.tool_knowledge, query=["find x"])
+
+
+def test_consult_question_list():
+    _no_crash(T.tool_consult, question=["help me"])
+
+
+def test_webfetch_url_list():
+    _no_crash(T.tool_webfetch, url=["http://example.invalid"])
+
+
+def test_websearch_query_int():
+    out = _no_crash(T.tool_websearch, query=42)
+    assert "42" in out