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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 3.0.102
+Version: 3.0.103
 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.103}/drydock/tools/__init__.py

@@ -1256,7 +1256,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 +1398,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 +1578,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 +1596,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 +1616,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.103/drydock_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 3.0.102
+Version: 3.0.103
 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.103}/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.103}/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.103"
 description = "Drydock — a local, provider-agnostic terminal coding agent for local LLMs"
 readme = "README.md"
 requires-python = ">=3.11"

drydock_cli-3.0.103/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