drydock-cli 3.0.110__tar.gz → 3.0.112__tar.gz

{drydock_cli-3.0.110/drydock_cli.egg-info → drydock_cli-3.0.112}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 3.0.110
+Version: 3.0.112
 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.110 → drydock_cli-3.0.112}/drydock/cli.py

@@ -130,7 +130,8 @@ def run_oneshot(prompt: str, config: dict) -> None:
                 # under -p) shows WHAT each tool did, not just its name. Without
                 # this, a timed-out run is an opaque wall of "[Bash]" with no way
                 # to tell a genuine 100-command exploration from a tight loop.
-                print(f"  [{event.name}] {_summarize(event.inputs)}",
+                from drydock.tools import tool_display_name
+                print(f"  [{tool_display_name(event.name)}] {_summarize(event.inputs)}",
                       file=sys.stderr, flush=True)
             elif isinstance(event, ToolEnd):
                 # One-line outcome so failures/repeats are visible in the trace.

{drydock_cli-3.0.110 → drydock_cli-3.0.112}/drydock/graphrag.py

@@ -101,9 +101,20 @@ def _chunk_text(text: str, max_chars: int = 1200) -> list[str]:
     return chunks
 
 
+def _unquote(p: str) -> str:
+    """Strip surrounding quotes a user wrapped a path in — common on Windows:
+    `/graphrag build "C:\\Users\\me\\Documents"`. Without this the quote chars
+    become part of the path, isabs() is False, it's joined under cwd, and nothing
+    is found ("No text found. Nothing was indexed.")."""
+    p = p.strip()
+    if len(p) >= 2 and p[0] == p[-1] and p[0] in "\"'":
+        p = p[1:-1]
+    return p
+
+
 def _iter_text_files(paths: list[str]):
     for p in paths:
-        path = Path(p)
+        path = Path(_unquote(p))
         if path.is_file():
             yield path
         elif path.is_dir():
@@ -120,8 +131,9 @@ def _ingest_files(paths, cwd, chunks, entity_chunks, edges, skip_sources):
     """Chunk + extract + graph every text file under paths into the (mutable)
     accumulators, skipping any source already present. Returns files added."""
     added = 0
+    cleaned = [_unquote(p) for p in paths]
     for fp in _iter_text_files([str(Path(cwd) / p) if not os.path.isabs(p) else p
-                                for p in paths]):
+                                for p in cleaned]):
         rel = os.path.relpath(str(fp), cwd)
         if rel in skip_sources:
             continue

{drydock_cli-3.0.110 → drydock_cli-3.0.112}/drydock/tools/__init__.py

@@ -145,6 +145,20 @@ _SHELL_KIND, _SHELL_PATH = _detect_shell()
 _BASH_SHELL = _detect_bash()
 
 
+def shell_display_name() -> str:
+    """Human label for the shell the Bash tool actually runs commands in — so the
+    TUI can show 'PowerShell'/'cmd' on Windows instead of the schema name 'Bash'."""
+    return {"powershell": "PowerShell", "cmd": "cmd",
+            "bash": "Bash", "sh": "sh"}.get(_SHELL_KIND, "Bash")
+
+
+def tool_display_name(name: str) -> str:
+    """Map a tool's schema name to its display label. The `Bash` tool (the name
+    the model calls) is shown as the actual shell — so on Windows a user sees
+    'PowerShell', not 'Bash'."""
+    return shell_display_name() if name == "Bash" else name
+
+
 # ANSI escape sequences (CSI colour/cursor, OSC title, and lone two-char escapes).
 # Some tools emit these even to a pipe (--color=always, forced-colour test runners),
 # and they're pure display control — noise that wastes the model's tokens.

{drydock_cli-3.0.110 → drydock_cli-3.0.112}/drydock/tui/app.py

@@ -1214,7 +1214,8 @@ class DrydockApp(App):
             self._render_todo(m.inputs.get("tasks", ""))
             self._last_card = None  # tool_end is a no-op for the checklist
             return
-        card = ToolCard(m.name, summarize_inputs(m.inputs))
+        from drydock.tools import tool_display_name
+        card = ToolCard(tool_display_name(m.name), summarize_inputs(m.inputs))
         self._last_card = card
         self._mount(card)
 

{drydock_cli-3.0.110 → drydock_cli-3.0.112/drydock_cli.egg-info}/PKG-INFO

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

@@ -80,6 +80,7 @@ tests/test_gittools.py
 tests/test_glob_edit_edges.py
 tests/test_graphify_example.py
 tests/test_graphrag.py
+tests/test_graphrag_quoted_path.py
 tests/test_grep_and_read_robust.py
 tests/test_guards_and_tools.py
 tests/test_hallucinated_tools.py

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

drydock_cli-3.0.112/tests/test_graphrag_quoted_path.py

@@ -0,0 +1,43 @@
+"""A user wrapping the path in quotes — common on Windows, e.g.
+`/graphrag build "C:\\Users\\me\\Documents"` — must still index. Before the fix
+the quote chars became part of the path, isabs() was False, it was joined under
+cwd, and nothing was found ('No text found. Nothing was indexed.')."""
+from __future__ import annotations
+
+import os
+import tempfile
+
+from drydock import graphrag
+
+
+def _make_docs():
+    src = tempfile.mkdtemp()
+    open(os.path.join(src, "notes.txt"), "w").write("Apollo landed on the Moon in 1969.")
+    open(os.path.join(src, "readme.md"), "w").write("# Project\nDrydock is a coding agent.")
+    store = os.path.join(tempfile.mkdtemp(), "kb.json")  # store OUTSIDE the scanned dir
+    return src, store
+
+
+def test_double_quoted_path_indexes():
+    src, store = _make_docs()
+    stats = graphrag.build_index([f'"{src}"'], store)
+    assert stats["files"] == 2 and stats["chunks"] >= 2
+
+
+def test_single_quoted_path_indexes():
+    src, store = _make_docs()
+    stats = graphrag.build_index([f"'{src}'"], store)
+    assert stats["files"] == 2
+
+
+def test_unquoted_path_still_works():
+    src, store = _make_docs()
+    assert graphrag.build_index([src], store)["files"] == 2
+
+
+def test_unquote_helper():
+    assert graphrag._unquote('"C:\\Users\\x"') == "C:\\Users\\x"
+    assert graphrag._unquote("'/a/b'") == "/a/b"
+    assert graphrag._unquote("/plain/path") == "/plain/path"
+    assert graphrag._unquote('  "spaced"  ') == "spaced"
+    assert graphrag._unquote('"') == '"'  # lone quote unchanged

{drydock_cli-3.0.110 → drydock_cli-3.0.112}/tests/test_windows_shell.py

@@ -92,3 +92,11 @@ def test_is_windows_detection_signals():
     # real Linux / WSL (no Windows env)
     assert not _is_windows_env("posix", "linux", {})
     assert not _is_windows_env("posix", "linux", {"HOME": "/home/x"})
+
+
+def test_tool_display_name_maps_bash_to_shell():
+    from unittest import mock
+    for kind, expect in [("powershell", "PowerShell"), ("cmd", "cmd"), ("bash", "Bash"), ("sh", "sh")]:
+        with mock.patch.object(T, "_SHELL_KIND", kind):
+            assert T.tool_display_name("Bash") == expect
+            assert T.tool_display_name("Read") == "Read"  # other tools unchanged