drydock-cli 2.5.9__tar.gz → 2.6.0__tar.gz

{drydock_cli-2.5.9 → drydock_cli-2.6.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 2.5.9
+Version: 2.6.0
 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.5.9 → drydock_cli-2.6.0}/drydock/cli/cli.py

@@ -163,33 +163,9 @@ def run_cli(args: argparse.Namespace) -> None:
             if not hasattr(args, 'initial_prompt') or not args.initial_prompt:
                 args.initial_prompt = args.prompt or stdin_prompt
 
-        # Disable problematic tools for models that can't handle complex schemas
-        try:
-            active = config.get_active_model()
-            if "gemma" in active.name.lower():
-                config.disabled_tools = [
-                    *config.disabled_tools,
-                    "ask_user_question",
-                    "todo",
-                    "task_create",
-                    "task_update",
-                    "task",
-                    "invoke_skill",
-                    "tool_search",
-                    "cron_create",
-                    "cron_delete",
-                    "cron_list",
-                    "enter_worktree",
-                    "exit_worktree",
-                    "exit_plan_mode",
-                    "list_mcp_resources",
-                    "read_mcp_resource",
-                    "notebook_edit",
-                    "web_fetch",
-                    "web_search",
-                ]
-        except (ValueError, AttributeError):
-            pass
+        # No tools disabled — let the model use whatever it wants.
+        # Tools with validation errors (empty args) will just return errors
+        # and the model will self-correct.
 
         # Disable streaming for Gemma 4 — streaming tool call accumulation
         # produces empty arguments (14 write_file calls → 0 files)

{drydock_cli-2.5.9 → drydock_cli-2.6.0}/drydock/core/agent_loop.py

@@ -489,16 +489,7 @@ class AgentLoop:
             should_break_loop = False
             tool_turns = 0
             api_error_count = 0
-            repeat_warnings = 0
             has_made_edit = False  # Track if model has used search_replace/write_file
-            text_without_action = 0  # Consecutive text responses without tool calls
-            bash_count = 0  # Track consecutive bash calls without an edit
-            search_replace_failures = 0  # Track failed search_replace attempts
-            files_explored: list[str] = []  # Track files read for context state
-            files_modified: list[str] = []  # Track files edited for blast radius
-            context_summary_injected = False
-            self._temp_override: float | None = None  # Temperature bump on loop detection
-            self._failed_approaches: list[str] = []  # Persistent failed approach tracker
             while not should_break_loop:
                 # Loop protection: prevent infinite tool-call loops
                 tool_turns += 1
@@ -774,114 +765,8 @@ class AgentLoop:
 
                 should_break_loop = last_message.role != Role.tool
 
-                # Handle empty response (no content AND no tools) — retry
-                if (should_break_loop and last_message.role == Role.assistant
-                        and not last_message.content and not last_message.tool_calls):
-                    empty_response_count = getattr(self, '_empty_responses', 0) + 1
-                    self._empty_responses = empty_response_count
-                    if empty_response_count <= 3:
-                        should_break_loop = False
-                        self._inject_system_note(
-                            "Your response was empty. You MUST call a tool. "
-                            "Start by reading the relevant file with read_file, "
-                            "or run the failing command with bash to see the error."
-                        )
-                        logger.warning("Empty model response — nudging (attempt %d)", empty_response_count)
-                        continue
-                    # After 3 empty responses, let it exit
-
-                # Text response without edits — this is FINE for questions,
-                # explanations, running code, etc. Only nudge if the model
-                # is clearly asking for permission instead of acting.
-                if should_break_loop and not has_made_edit:
-                    last_text = (last_message.content or "").lower()
-                    is_asking = any(kw in last_text for kw in (
-                        "shall i", "would you like me to",
-                        "ready to begin", "please confirm",
-                    ))
-                    if is_asking and tool_turns == 0:
-                        # Model asked without doing anything — nudge once
-                        should_break_loop = False
-                        self._inject_system_note(
-                            "Act immediately. Do not ask for confirmation."
-                        )
-                        logger.info("Model gave text without editing — nudging (attempt %d)", text_without_action)
-
-                # If model has been investigating for too long without making an edit,
-                # force it to act (catches grep/read_file loops that don't trigger
-                # the text-without-action check because they DO use tools)
-                if not should_break_loop and not has_made_edit and tool_turns >= 15:
-                    if tool_turns == 15:
-                        nudge_text = (
-                            "You have spent 15 tool calls investigating without making an edit. "
-                            "You MUST use search_replace on your next turn. Pick the most likely target "
-                            "file based on what you've seen and make your fix attempt NOW."
-                        )
-                    elif tool_turns % 5 == 0:
-                        nudge_text = (
-                            f"WARNING: {tool_turns} tool calls without editing. "
-                            "Use search_replace IMMEDIATELY. No more investigation."
-                        )
-                    else:
-                        nudge_text = None
-                    if nudge_text:
-                        self._inject_system_note(nudge_text)
-
-                # Loop guidance — retrospection, NEVER stop
-                if not should_break_loop:
-                    rep = self._check_tool_call_repetition()
-                    if rep == "FORCE_STOP" or (rep and rep.startswith("WARNING")):
-                        # Prune duplicate tool calls to give model fresh context
-                        self._prune_repeated_tool_calls()
-
-                        # Temperature bump: force model to explore different paths
-                        self._temp_override = 0.5
-
-                        # Track failed approach for accumulator
-                        self._record_failed_approach()
-
-                        # RETROSPECTION: Summarize recent actions and let the model
-                        # decide its own next step
-                        retro = self._build_retrospection()
-                        if retro:
-                            # Include failed approaches if any
-                            if self._failed_approaches:
-                                approaches = "\n".join(f"  - {a}" for a in self._failed_approaches[-5:])
-                                retro += f"\n\nApproaches already tried (don't repeat these):\n{approaches}"
-
-                            # Detect same-file write loop — compact and restart fresh
-                            stuck_file = self._detect_stuck_file()
-                            if stuck_file:
-                                # Compact: list files already created, drop old messages
-                                existing_files = self._list_created_files()
-
-                                # Keep only the first user message and last 2 messages
-                                if len(self.messages) > 5:
-                                    first_user = None
-                                    for msg in self.messages:
-                                        if msg.role == Role.user:
-                                            first_user = msg
-                                            break
-                                    kept = []
-                                    if first_user:
-                                        kept.append(first_user)
-                                    kept.extend(self.messages[-2:])
-                                    self.messages.reset(kept)
-
-                                # Inject summary of what's done
-                                summary = (
-                                    f"CONTEXT RESET — You were stuck rewriting {stuck_file}.\n"
-                                    f"Files already created: {', '.join(existing_files) if existing_files else 'none'}\n"
-                                    f"Continue creating the REMAINING files. Do not rewrite files that already exist."
-                                )
-                                self._inject_system_note(summary)
-                                # Skip the generic retrospection — the summary is more useful
-                                continue
-
-                            self._inject_system_note(retro)
-                    else:
-                        # Loop broken — reset temperature to normal
-                        self._temp_override = None
+                # No circuit breakers, no loop detection, no forced nudges.
+                # The model works on its own. The only hard stop is MAX_TOOL_TURNS.
 
                 if user_cancelled:
                     return
@@ -1437,7 +1322,7 @@ class AgentLoop:
         try:
             start_time = time.perf_counter()
             # Use temperature override if set (loop detection bumps it)
-            temp = getattr(self, '_temp_override', None) or active_model.temperature
+            temp = active_model.temperature
             result = await self.backend.complete(
                 model=active_model,
                 messages=self.messages,
@@ -1487,7 +1372,7 @@ class AgentLoop:
             usage = LLMUsage()
             chunk_agg = LLMChunk(message=LLMMessage(role=Role.assistant))
             # Use temperature override if set (loop detection bumps it)
-            temp = getattr(self, '_temp_override', None) or active_model.temperature
+            temp = active_model.temperature
             async for chunk in self.backend.complete_streaming(
                 model=active_model,
                 messages=self.messages,

{drydock_cli-2.5.9 → drydock_cli-2.6.0}/pyproject.toml

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