drydock-cli 2.6.9__tar.gz → 2.6.11__tar.gz

drydock_cli-2.6.11/.auto_release.lock

@@ -0,0 +1 @@
+3937857

{drydock_cli-2.6.9 → drydock_cli-2.6.11}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 2.6.9
+Version: 2.6.11
 Summary: Drydock — nautical CLI coding agent
 Project-URL: Homepage, https://github.com/fbobe321/drydock
 Project-URL: Repository, https://github.com/fbobe321/drydock

{drydock_cli-2.6.9 → drydock_cli-2.6.11}/drydock/cli/textual_ui/widgets/banner/banner.py

@@ -88,11 +88,14 @@ class Banner(Static):
         )
 
     def _format_meta_counts(self) -> str:
-        return (
-            f"{self.state.models_count} model{'s' if self.state.models_count != 1 else ''}"
-            f" · {self.state.mcp_servers_count} MCP server{'s' if self.state.mcp_servers_count != 1 else ''}"
-            f" · {self.state.skills_count} skill{'s' if self.state.skills_count != 1 else ''}"
-        )
+        parts = []
+        if self.state.models_count:
+            parts.append(f"{self.state.models_count} model{'s' if self.state.models_count != 1 else ''}")
+        if self.state.mcp_servers_count:
+            parts.append(f"{self.state.mcp_servers_count} MCP server{'s' if self.state.mcp_servers_count != 1 else ''}")
+        if self.state.skills_count:
+            parts.append(f"{self.state.skills_count} skill{'s' if self.state.skills_count != 1 else ''}")
+        return " · ".join(parts) if parts else ""
 
     def _format_plan(self) -> str:
         return (

{drydock_cli-2.6.9 → drydock_cli-2.6.11}/drydock/core/agent_loop.py

@@ -1721,6 +1721,8 @@ class AgentLoop:
                 "## Rules\n"
                 "- Use absolute imports: `from package.module import X`\n"
                 "- Always create `__init__.py` and `__main__.py`\n"
+                "- Create ALL files listed in the PRD before stopping\n"
+                "- Do NOT stop after creating just __init__.py — continue to the next file\n"
                 "- NEVER ask 'should I proceed' or 'would you like me to' — JUST DO IT\n"
                 "- After creating a file, immediately create the next one\n"
             )

{drydock_cli-2.6.9 → drydock_cli-2.6.11}/drydock/core/prompts/gemma4.md

@@ -23,4 +23,6 @@ Rules:
 - Keep responses under 50 words. Code speaks for itself.
 - NEVER ask "would you like me to proceed" or "shall I continue" — JUST DO IT.
 - After creating/editing a file, move to the next one. Do not stop.
-- Follow the EXACT CLI interface specified in the PRD. If the PRD says `--roman XIV`, use `--roman` not `--input`. Match argument names, subcommands, and flags exactly as documented.
+- Follow the EXACT CLI interface specified in the PRD. Match argument names, subcommands, and flags exactly.
+- Every subcommand in the PRD must have a working handler — not just argparse registration.
+- After creating files, run python3 -m package_name [subcommand] to verify each one works.

{drydock_cli-2.6.9 → drydock_cli-2.6.11}/drydock/core/tools/builtins/search_replace.py

@@ -131,6 +131,27 @@ class SearchReplace(
 
         original_content = await self._read_file(file_path)
 
+        # Detect placeholder/omission patterns that would delete code
+        PLACEHOLDER_PATTERNS = [
+            "# rest of code", "# ... rest", "# unchanged",
+            "// rest of code", "// ... rest", "// unchanged",
+            "# TODO: implement", "pass  # placeholder",
+            "...",  # bare ellipsis as entire replacement
+        ]
+        for block in search_replace_blocks:
+            replacement = block.replace.strip()
+            if replacement in PLACEHOLDER_PATTERNS or any(
+                p in replacement.lower() for p in [
+                    "rest of code", "rest of the code", "unchanged",
+                    "remaining code", "code continues", "etc.",
+                ]
+            ):
+                raise ToolError(
+                    f"Your replacement contains a placeholder ('{replacement[:50]}') "
+                    f"that would delete existing code. Provide the COMPLETE replacement "
+                    f"code, not a summary. If the code is unchanged, don't edit it."
+                )
+
         block_result = self._apply_blocks(
             original_content,
             search_replace_blocks,

{drydock_cli-2.6.9 → drydock_cli-2.6.11}/drydock/core/tools/builtins/todo.py

@@ -32,8 +32,8 @@ class TodoPriority(StrEnum):
 
 
 class TodoItem(BaseModel):
-    id: str
-    content: str
+    id: str = Field(default_factory=lambda: str(__import__('time').time_ns()))
+    content: str = ""
     status: TodoStatus = TodoStatus.PENDING
     priority: TodoPriority = TodoPriority.MEDIUM
 

{drydock_cli-2.6.9 → drydock_cli-2.6.11}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "drydock-cli"
-version = "2.6.9"
+version = "2.6.11"
 description = "Drydock — nautical CLI coding agent"
 readme = "README.md"
 requires-python = ">=3.12"

{drydock_cli-2.6.9 → drydock_cli-2.6.11}/tests/test_user_issues.py

@@ -127,3 +127,85 @@ class TestAPIRecovery:
         ml.reset(kept)
         assert len(ml) == 6
         assert ml[0].role == Role.user
+
+
+class TestTodoMissingFields:
+    """#60: Todo should handle missing fields gracefully."""
+
+    def test_missing_id(self):
+        from drydock.core.tools.builtins.todo import TodoItem
+        item = TodoItem.model_validate({"content": "test task"})
+        assert item.content == "test task"
+        assert item.id  # auto-generated
+
+    def test_missing_content(self):
+        from drydock.core.tools.builtins.todo import TodoItem
+        item = TodoItem.model_validate({"id": "1"})
+        assert item.id == "1"
+        assert item.content == ""
+
+    def test_completely_empty(self):
+        from drydock.core.tools.builtins.todo import TodoItem
+        item = TodoItem.model_validate({})
+        assert item.id  # auto-generated
+        assert item.status.value == "pending"
+
+
+class TestThinkingMode:
+    """Verify thinking mode is properly configured."""
+
+    def test_thinking_in_generic_backend(self):
+        """Generic backend source should reference enable_thinking."""
+        content = open("/data3/drydock/drydock/core/llm/backend/generic.py").read()
+        assert "enable_thinking" in content
+
+    def test_thinking_values_in_settings(self):
+        """Settings should define thinking as a Literal type."""
+        content = open("/data3/drydock/drydock/core/config/_settings.py").read()
+        assert "thinking" in content
+
+
+class TestNonStreaming:
+    """Verify non-streaming config exists for Gemma 4."""
+
+    def test_streaming_flag_in_cli(self):
+        """CLI source should have streaming control logic."""
+        content = open("/data3/drydock/drydock/cli/cli.py").read()
+        assert "enable_streaming" in content
+
+
+class TestSha256NotMd5:
+    """Verify no md5 anywhere in codebase."""
+
+    def test_no_md5_in_agent_loop(self):
+        import inspect
+        from drydock.core.agent_loop import AgentLoop
+        source = inspect.getsource(AgentLoop)
+        assert "hashlib.md5" not in source
+
+    def test_no_md5_in_manager(self):
+        content = open("/data3/drydock/drydock/core/tools/manager.py").read()
+        assert "hashlib.md5" not in content
+
+
+class TestPlaceholderDetection:
+    """#62: Detect placeholder replacements that would delete code."""
+
+    def test_rest_of_code_detected(self):
+        """'# rest of code' should be rejected."""
+        from drydock.core.tools.builtins.search_replace import SearchReplace
+        blocks = SearchReplace._parse_search_replace_blocks(
+            '<<<<<<< SEARCH\ndef foo():\n    return 1\n=======\n# rest of code\n>>>>>>> REPLACE'
+        )
+        assert len(blocks) == 1
+        # The placeholder check happens at execution time, not parse time
+        assert blocks[0].replace.strip() == "# rest of code"
+
+    def test_normal_replacement_ok(self):
+        """Normal code replacement should not be flagged."""
+        from drydock.core.tools.builtins.search_replace import SearchReplace
+        blocks = SearchReplace._parse_search_replace_blocks(
+            '<<<<<<< SEARCH\ndef foo():\n    return 1\n=======\ndef foo():\n    return 2\n>>>>>>> REPLACE'
+        )
+        assert len(blocks) == 1
+        assert "return 2" in blocks[0].replace

drydock_cli-2.6.9/.auto_release.lock

@@ -1 +0,0 @@
-3928923