drydock-cli 3.0.20__tar.gz → 3.0.21__tar.gz

{drydock_cli-3.0.20/drydock_cli.egg-info → drydock_cli-3.0.21}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 3.0.20
+Version: 3.0.21
 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.20 → drydock_cli-3.0.21}/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.20"
+    __version__ = "3.0.21"

{drydock_cli-3.0.20 → drydock_cli-3.0.21}/drydock/tools/__init__.py

@@ -401,6 +401,18 @@ def tool_edit(params: dict, config: dict) -> str:
         if count > 1:
             return f"Error: old_string found {count} times in {fp}. Add more context to make it unique."
         updated = content.replace(old, new, 1)
+        if updated == content:
+            # No-op edit (old_string == new_string, or the replacement changes
+            # nothing). Reporting "Edited" here is a false success that invites
+            # the model to re-issue the identical call forever. Tell it plainly
+            # that nothing changed and to move on — this is what breaks the
+            # same-file edit-thrash loop at its source.
+            return (
+                f"No change: that edit leaves {fp} byte-for-byte identical "
+                f"(old_string and new_string are effectively the same). Nothing "
+                f"was written. Do NOT repeat this edit — the file already has "
+                f"this content, so move on to your next step."
+            )
         Path(fp).write_text(updated, encoding="utf-8")
         _record_undo(config, fp, content)
         result = f"Edited {fp}: replaced {len(old)} chars with {len(new)} chars"

{drydock_cli-3.0.20 → drydock_cli-3.0.21/drydock_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 3.0.20
+Version: 3.0.21
 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.20 → drydock_cli-3.0.21}/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.20"
+version = "3.0.21"
 description = "Drydock — a local, provider-agnostic terminal coding agent for local LLMs"
 readme = "README.md"
 requires-python = ">=3.12"

{drydock_cli-3.0.20 → drydock_cli-3.0.21}/tests/test_guards_and_tools.py

@@ -263,6 +263,19 @@ def test_tool_edit_fuzzy_applies_unique_near_miss(tmp_path):
     assert "compute2" in fp.read_text()
 
 
+def test_tool_edit_noop_when_old_equals_new(tmp_path):
+    # old_string == new_string changes nothing. Reporting "Edited" is a false
+    # success that loops a weak model; it must say "No change" and not write.
+    fp = tmp_path / "f.py"
+    fp.write_text("a = 1\n")
+    out = tool_edit(
+        {"file_path": str(fp), "old_string": "a = 1", "new_string": "a = 1"}, {}
+    )
+    assert "No change" in out and "Edited" not in out
+    assert "move on" in out.lower()
+    assert fp.read_text() == "a = 1\n"  # untouched
+
+
 def test_tool_edit_fuzzy_refuses_indent_mismatch(tmp_path):
     # If the near-miss differs in INDENT, applying the model's new_string would
     # corrupt the file — so it must error (and show the real text) instead.