drydock-cli 2.6.68__tar.gz → 2.6.72__tar.gz

drydock_cli-2.6.72/.auto_release.lock

@@ -0,0 +1 @@
+952015

{drydock_cli-2.6.68 → drydock_cli-2.6.72}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 2.6.68
+Version: 2.6.72
 Summary: Local-first CLI coding agent — tested with Gemma 4 26B via vLLM
 Project-URL: Homepage, https://github.com/fbobe321/drydock
 Project-URL: Repository, https://github.com/fbobe321/drydock

{drydock_cli-2.6.68 → drydock_cli-2.6.72}/drydock/core/agents/models.py

@@ -202,7 +202,8 @@ DIAGNOSTIC = AgentProfile(
     agent_type=AgentType.SUBAGENT,
     max_turns=80,
     overrides={
-        "enabled_tools": ["grep", "read_file", "bash", "glob"],
+        "enabled_tools": ["grep", "read_file", "bash", "glob",
+                          "web_search", "web_fetch"],
         "system_prompt_id": "diagnostic",
     },
 )
@@ -242,12 +243,17 @@ BUILDER = AgentProfile(
         "enabled_tools": [
             "read_file", "write_file", "search_replace",
             "glob", "grep", "bash",
+            # web tools — when stuck on a difficult problem, the model can
+            # google for examples, error-message solutions, or API docs.
+            "web_search", "web_fetch",
         ],
         "system_prompt_id": "builder",
         "tools": {
             "write_file": {"permission": "always"},
             "search_replace": {"permission": "always"},
             "bash": {"permission": "always"},
+            "web_search": {"permission": "always"},
+            "web_fetch": {"permission": "always"},
         },
     },
 )

{drydock_cli-2.6.68 → drydock_cli-2.6.72}/drydock/core/llm/format.py

@@ -174,7 +174,13 @@ class APIToolFormatHandler:
 
         # Tools the model hallucinates that should be silently ignored
         # rather than generating a visible error that confuses the user.
-        _IGNORE_TOOLS = {"exit_plan_mode", "enter_plan_mode", "plan_mode"}
+        _IGNORE_TOOLS = {
+            "exit_plan_mode", "enter_plan_mode", "plan_mode",
+            # Gemma 4 confabulated tools — observed in user sessions and
+            # mini_db fresh build (called ralph_repo_index 5 times).
+            "ralph_repo_index", "repo_index", "index_repo",
+            "list_mcp_resources", "list_resources", "search_resources",
+        }
 
         for parsed_call in parsed.tool_calls:
             tool_class = active_tools.get(parsed_call.tool_name)

{drydock_cli-2.6.68 → drydock_cli-2.6.72}/drydock/core/prompts/gemma4.md

@@ -2,7 +2,13 @@ You are DryDock, a CLI coding agent. You write code, fix bugs, and build project
 
 ACT IMMEDIATELY. Your FIRST response must be a tool call — not text. Do NOT explain, plan, or ask. Call a tool NOW.
 
-Your tools: read_file, write_file, search_replace, grep, glob, bash, task.
+Your tools: read_file, write_file, search_replace, grep, glob, bash, task, web_search, web_fetch.
+
+WHEN TO USE WEB TOOLS (use sparingly, NOT for every task):
+- Stuck on an error you've tried to fix 2+ times without progress: `web_search` for the exact error message
+- Need an API example you don't remember (e.g. "how do I parse TOML with stdlib?"): `web_search`
+- Found a URL to a relevant SO post or doc page: `web_fetch` to read it
+- DO NOT web-search for things you already know how to do. Write the code first.
 
 DELEGATION (only for genuinely large tasks)
 

{drydock_cli-2.6.68 → drydock_cli-2.6.72}/drydock/core/tools/builtins/write_file.py

@@ -84,6 +84,53 @@ def _check_main_module_entry(tree) -> str | None:
     return f"imports {sorted(entry_names)} but never calls any of them"
 
 
+def _check_bare_raise_outside_except(tree) -> list[str]:
+    """Detect `raise` without an argument outside of an except block.
+
+    Bare `raise` is only valid inside an except handler (re-raises the
+    active exception). Outside an except, it fails at runtime with
+    'No active exception to reraise'. This is a common Gemma 4 mistake:
+    writing `if not found: raise` when `raise SomeError(...)` was intended.
+
+    Returns a list of line-number hints for each problem found.
+    """
+    import ast
+
+    problems: list[str] = []
+
+    class _Visitor(ast.NodeVisitor):
+        def __init__(self):
+            self.in_except = 0
+            self.in_func = 0
+
+        def visit_ExceptHandler(self, node):
+            self.in_except += 1
+            self.generic_visit(node)
+            self.in_except -= 1
+
+        def visit_FunctionDef(self, node):
+            self.in_func += 1
+            # bare raise inside a function but outside except is still bad
+            # (the function can be called from anywhere, not just except).
+            self.generic_visit(node)
+            self.in_func -= 1
+
+        def visit_AsyncFunctionDef(self, node):
+            self.visit_FunctionDef(node)
+
+        def visit_Raise(self, node):
+            if node.exc is None and self.in_except == 0:
+                problems.append(
+                    f"line {node.lineno}: bare `raise` outside any except block "
+                    f"(will fail with 'No active exception to reraise' at runtime)"
+                )
+            # Don't recurse — raise has no sub-Raise nodes
+            self.generic_visit(node)
+
+    _Visitor().visit(tree)
+    return problems
+
+
 def _check_missing_sibling_imports(tree, file_path: Path) -> set[str]:
     """Detect imports of sibling modules that don't exist on disk yet.
 
@@ -310,6 +357,24 @@ class WriteFile(
                             f"Without it, `python3 -m {file_path.parent.name}` will "
                             f"exit silently with no output."
                         )
+                # Catch bare `raise` outside except. Found in ACE v2 build —
+                # drydock wrote `if not found: raise` in cli.py, which fails
+                # at runtime with "No active exception to reraise".
+                bare_raises = _check_bare_raise_outside_except(tree)
+                if bare_raises:
+                    raise_warning = (
+                        f"\n\n⚠ {file_path.name} has bare `raise` outside of "
+                        f"any except block:\n"
+                        + "\n".join(f"  {p}" for p in bare_raises[:3])
+                        + "\n  A bare `raise` only works inside an `except` handler. "
+                        + "Use `raise SpecificError(...)` with an explicit exception, "
+                        + "or return/print an error instead."
+                    )
+                    if syntax_warning:
+                        syntax_warning += raise_warning
+                    else:
+                        syntax_warning = raise_warning
+
                 # Catch missing-import bug: file imports `from .x import Y` or
                 # `from pkg.x import Y` but x.py doesn't exist on disk yet.
                 # Real bug found in minivc build — __init__.py imported `cli`

{drydock_cli-2.6.68 → drydock_cli-2.6.72}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "drydock-cli"
-version = "2.6.68"
+version = "2.6.72"
 description = "Local-first CLI coding agent — tested with Gemma 4 26B via vLLM"
 readme = "README.md"
 requires-python = ">=3.12"

{drydock_cli-2.6.68 → drydock_cli-2.6.72}/scripts/ralph_loop.py

@@ -290,8 +290,19 @@ def ralph(cwd: Path, pkg: str, max_iterations: int = 5,
 
     # Baseline: current state of tests
     p, f, _ = run_functional_tests(cwd)
+    # ALSO check whether the package directory even exists on disk. Tests can
+    # "pass" via false positives when the package is missing entirely (grep
+    # for absence-patterns matches when the command output is empty because
+    # of a ModuleNotFoundError). Only treat the build as "existing" if the
+    # package dir has at least an __init__.py.
+    pkg_init = cwd / pkg / "__init__.py"
+    package_exists = pkg_init.exists()
     print(f"  Baseline ({'no build' if fresh_build else 'existing build'}): "
-          f"{p}/{p+f} tests pass")
+          f"{p}/{p+f} tests pass, package_exists={package_exists}")
+    if not package_exists:
+        # Treat as if there's no build — force full rebuild
+        print(f"  [{pkg}/__init__.py missing → treating as fresh build]")
+        p = 0
     baseline_pass = p
     baseline_fail = f
 

drydock_cli-2.6.68/.auto_release.lock

@@ -1 +0,0 @@
-922492