drydock-cli 3.0.16__tar.gz → 3.0.17__tar.gz

{drydock_cli-3.0.16/drydock_cli.egg-info → drydock_cli-3.0.17}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 3.0.16
+Version: 3.0.17
 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.16 → drydock_cli-3.0.17}/drydock/__init__.py

@@ -7,4 +7,4 @@ try:
     __version__ = _version("drydock-cli")
 except PackageNotFoundError:
     # Running from source before install (e.g. PYTHONPATH dev runs).
-    __version__ = "3.0.16"
+    __version__ = "3.0.17"

{drydock_cli-3.0.16 → drydock_cli-3.0.17}/drydock/tools/__init__.py

@@ -496,10 +496,16 @@ _TODO_DONE = {"x", "X", "✓"}
 _TODO_DOING = {"~", "-", "/", "*"}
 
 
-def parse_todo(tasks: str) -> list[tuple[str, str]]:
-    """Parse a plain-text checklist into (text, status) pairs. Each non-empty
-    line is one task, optionally prefixed with [x] done / [~] in_progress /
-    [ ] pending. Deliberately forgiving (a bare line counts as pending)."""
+def parse_todo(tasks) -> list[tuple[str, str]]:
+    """Parse a checklist into (text, status) pairs. Each non-empty line is one
+    task, optionally prefixed with [x] done / [~] in_progress / [ ] pending.
+    Deliberately forgiving: a bare line counts as pending, and `tasks` may be a
+    newline-joined string OR a list of lines (the model sometimes sends a JSON
+    array instead of one string — handle both rather than crash the TUI)."""
+    if isinstance(tasks, (list, tuple)):
+        tasks = "\n".join(str(t) for t in tasks)
+    elif tasks is not None and not isinstance(tasks, str):
+        tasks = str(tasks)
     out: list[tuple[str, str]] = []
     for raw in (tasks or "").splitlines():
         line = raw.strip().lstrip("-*•").strip()

{drydock_cli-3.0.16 → drydock_cli-3.0.17/drydock_cli.egg-info}/PKG-INFO

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

@@ -40,6 +40,7 @@ tests/test_loop_detect.py
 tests/test_plan_autocontinue.py
 tests/test_providers_unreachable.py
 tests/test_subagent.py
+tests/test_todo.py
 tests/test_tools_undo.py
 tests/test_tui.py
 tests/test_tuning.py

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

drydock_cli-3.0.17/tests/test_todo.py

@@ -0,0 +1,38 @@
+"""Tests for the `todo` checklist tool and its forgiving parser."""
+from __future__ import annotations
+
+from drydock.tools import parse_todo, tool_todo
+
+
+def test_parse_todo_basic_statuses():
+    items = parse_todo("[x] done one\n[~] doing two\n[ ] pending three")
+    assert items == [
+        ("done one", "done"),
+        ("doing two", "in_progress"),
+        ("pending three", "pending"),
+    ]
+
+
+def test_parse_todo_accepts_list_input():
+    # The model sometimes sends a JSON array instead of one newline string —
+    # this must NOT crash (it took down the whole TUI before).
+    items = parse_todo([
+        "[~] Explore existing tests and run them",
+        "[ ] Add edge-case tests",
+        "[ ] Verify all tests pass",
+    ])
+    assert [s for _, s in items] == ["in_progress", "pending", "pending"]
+    assert items[0][0] == "Explore existing tests and run them"
+
+
+def test_parse_todo_none_and_empty():
+    assert parse_todo(None) == []
+    assert parse_todo("") == []
+    assert parse_todo([]) == []
+
+
+def test_tool_todo_accepts_list(tmp_path):
+    cfg: dict = {}
+    out = tool_todo({"tasks": ["[ ] a", "[x] b"]}, cfg)
+    assert "2 tasks" in out and "1 done" in out
+    assert len(cfg["_todo"]) == 2