drydock-cli 2.6.10__tar.gz → 2.6.12__tar.gz

drydock_cli-2.6.12/.auto_release.lock

@@ -0,0 +1 @@
+3939228

{drydock_cli-2.6.10 → drydock_cli-2.6.12}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 2.6.10
+Version: 2.6.12
 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.10 → drydock_cli-2.6.12}/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.10 → drydock_cli-2.6.12}/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.10 → drydock_cli-2.6.12}/pyproject.toml

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

{drydock_cli-2.6.10 → drydock_cli-2.6.12}/tests/test_user_issues.py

@@ -149,3 +149,63 @@ class TestTodoMissingFields:
         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.10/.auto_release.lock

@@ -1 +0,0 @@
-3931496