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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 3.0.20
+Version: 3.0.22
 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.22}/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.22"

{drydock_cli-3.0.20 → drydock_cli-3.0.22}/drydock/detect.py

@@ -61,9 +61,11 @@ def onboarding_message(found: list[dict]) -> str:
     if not found:
         ports = ", ".join(b.split("//")[1] for _, b in CANDIDATES)
         return (
-            "No local LLM detected (" + ports + "). Start a server "
-            "(llama.cpp / vLLM / Ollama / LM Studio), or set base_url in "
-            "~/.drydock/config.toml, then restart."
+            "No local LLM detected (" + ports + "). Point Drydock at your model "
+            "right here:\n"
+            "  /model url <http://host:port/v1>     then     /model <model-name>\n"
+            "(saved to ~/.drydock/config.toml). Or start a local server "
+            "(llama.cpp / vLLM / Ollama / LM Studio) and restart."
         )
     best = found[0]
     model = best["models"][0] if best["models"] else "?"

{drydock_cli-3.0.20 → drydock_cli-3.0.22}/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.22}/drydock/tui/app.py

@@ -405,7 +405,8 @@ class DrydockApp(App):
             self._info(
                 "Commands:\n"
                 "  /help            this help\n"
-                "  /model [name]    show or switch the model\n"
+                "  /model           show/set model, provider, endpoint URL\n"
+                "                   /model <name> · /model url <url> · /model provider <p>\n"
                 "  /cwd [path]      show or change the working directory\n"
                 "  /undo            revert the last file write/edit\n"
                 "  /back            rewind the last turn from the model's context\n"
@@ -418,14 +419,61 @@ class DrydockApp(App):
         else:
             self._mount(ErrorMessage(f"unknown command: {cmd} (try /help)"))
 
-    def _cmd_model(self, name: str) -> None:
-        if not name:
-            self._info(f"model: {self.config.get('model')}")
+    def _persist_config(self) -> None:
+        """Save the persistable settings (model/provider/base_url/… — save_file
+        filters to those) to ~/.drydock/config.toml so setup survives restart."""
+        from drydock import config as cfgmod
+
+        ok = cfgmod.save_file(self.config, cfgmod.default_config_path())
+        if not ok:
+            self._mount(ErrorMessage("could not write ~/.drydock/config.toml"))
+
+    def _cmd_model(self, arg: str) -> None:
+        """Model + endpoint setup. Subcommands persist so they survive restart:
+          /model                      show model, provider, endpoint
+          /model <name>               set the model name
+          /model url <base_url>       set the server URL (e.g. http://host:8000/v1)
+          /model provider <name>      vllm | ollama | lmstudio | openai
+        """
+        arg = (arg or "").strip()
+        if not arg:
+            prov = self.config.get("provider") or "(default)"
+            url = self.config.get("base_url") or "(provider default)"
+            self._info(
+                f"model:    {self.config.get('model')}\n"
+                f"provider: {prov}\n"
+                f"endpoint: {url}\n"
+                "Set up:  /model <name>  ·  /model url <http://host:port/v1>  ·  "
+                "/model provider <vllm|ollama|lmstudio|openai>"
+            )
+            return
+        parts = arg.split(maxsplit=1)
+        sub = parts[0].lower()
+        val = parts[1].strip() if len(parts) > 1 else ""
+        if sub == "url":
+            if not val:
+                self._info("usage: /model url <http://host:port/v1>")
+                return
+            self.config["base_url"] = val
+            self._persist_config()
+            self._info(f"endpoint → {val}  (saved). Send a prompt to test it.")
+            return
+        if sub == "provider":
+            from drydock.providers import PROVIDERS
+
+            if val not in PROVIDERS:
+                self._info(f"unknown provider {val!r}. Choose: {', '.join(PROVIDERS)}")
+                return
+            self.config["provider"] = val
+            self._persist_config()
+            self._info(f"provider → {val}  (saved)")
             return
-        self.config["model"] = name
-        self.system = self._build_system(name)  # prompt may be model-specific
+        # Otherwise: set the model name (the whole arg, so names with spaces work).
+        self.config["model"] = arg
+        self.system = self._build_system(arg)  # prompt may be model-specific
+        self._persist_config()
         self._refresh_status()
-        self._info(f"switched model → {name}")
+        self._info(f"model → {arg}  (saved)")
 
     def _cmd_cwd(self, path: str) -> None:
         if not path:

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 3.0.20
+Version: 3.0.22
 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.22}/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.22"
 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.22}/tests/test_detect.py

@@ -71,3 +71,5 @@ def test_onboarding_message_when_nothing_found():
     msg = detect.onboarding_message([])
     assert "No local LLM detected" in msg
     assert "config.toml" in msg
+    # New users must learn how to point Drydock at their model from the TUI.
+    assert "/model url" in msg and "/model" in msg

{drydock_cli-3.0.20 → drydock_cli-3.0.22}/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.