Remove legacy tool call loop detection

Author: Mateusz

.gitignore

@@ -51,6 +51,8 @@ tmp/
 tmp_*.py
 tmp_*.md
 tmp_*.txt
+*.err
+*.out
 
 # OS-specific files
 .DS_Store

docs/architecture/middleware-parity-analysis.md

@@ -12,7 +12,6 @@ This document analyzes all response middleware in the codebase to identify featu
 | ThinkTagsFixMiddleware | Yes | Yes | PARTIAL | Different logic paths |
 | EditPrecisionResponseMiddleware | Yes | Yes | FULL | Same logic for both |
 | ToolCallReactorMiddleware | Yes | Yes | PARTIAL | Different lifecycle handling |
-| ToolCallLoopDetectionMiddleware | Yes | Yes | PARTIAL | Different lifecycle reset |
 | EmptyResponseMiddleware | No | Yes | GAP | Skips streaming entirely |
 | StructuredOutputMiddleware | No | Yes | GAP | Skips streaming entirely |
 | JsonRepairMiddleware | No | Yes | GAP | Skips streaming, uses processor |
@@ -64,14 +63,6 @@ This document analyzes all response middleware in the codebase to identify featu
 - **Gap**: Lines 100-102 clear state for non-streaming only
 - **Migration Priority**: High
 
-#### ToolCallLoopDetectionMiddleware
-- **Location**: `src/core/services/tool_call_loop_middleware.py`
-- **Feature**: Prevents tool call loops
-- **Streaming**: Maintains lifecycle across chunks
-- **Non-streaming**: Clears lifecycle on each call (line 134)
-- **Gap**: Lifecycle handling differs
-- **Migration Priority**: High
-
 ### PARITY GAPS (Missing Streaming Support)
 
 #### EmptyResponseMiddleware
@@ -131,7 +122,6 @@ Added streaming support to middleware that previously lacked it:
 The following middleware have complex streaming state management and are deferred from full migration:
 - `ThinkTagsFixMiddleware` - 700+ lines with sophisticated buffer management
 - `ToolCallReactorMiddleware` - Complex lifecycle state handling
-- `ToolCallLoopDetectionMiddleware` - Session-based lifecycle tracking
 - `EditPrecisionResponseMiddleware` - Already has full parity
 
 These can be wrapped using `MiddlewareToFeatureAdapter` for registry integration.

src/core/commands/handlers/__init__.py

@@ -18,9 +18,6 @@
     "reasoning_aliases",
     "reasoning_handlers",
     "set_command_handler",
-    "tool_loop_max_repeats_command_handler",
-    "tool_loop_mode_command_handler",
-    "tool_loop_ttl_command_handler",
     "unset_command_handler",
 ]
 

src/core/commands/handlers/loop_detection_command_handler.py

@@ -75,52 +75,3 @@ async def handle(self, command: Command, session: Session) -> CommandResult:
             ),
             new_state=session.state,
         )
-
-
-@command("tool-loop-detection")
-class ToolLoopDetectionCommandHandler(ICommandHandler):
-    """Handler for the `/tool-loop-detection` command."""
-
-    @property
-    def command_name(self) -> str:
-        return "tool-loop-detection"
-
-    @property
-    def description(self) -> str:
-        return "Enable or disable tool loop detection."
-
-    @property
-    def format(self) -> str:
-        return "tool-loop-detection(enabled=true|false)"
-
-    @property
-    def examples(self) -> list[str]:
-        return [
-            "!/tool-loop-detection(enabled=true)",
-            "!/tool-loop-detection(enabled=false)",
-        ]
-
-    async def handle(self, command: Command, session: Session) -> CommandResult:
-        enabled, invalid = _parse_bool_argument(command.args)
-        if enabled is None:
-            return CommandResult(
-                success=False,
-                message=(
-                    "Error: Invalid value. Please use 'true' or 'false'."
-                    if invalid is not None
-                    else "Error: Please provide a value (true/false)."
-                ),
-            )
-
-        new_config = session.state.loop_config.with_tool_loop_detection_enabled(enabled)
-        session.state = session.state.with_loop_config(new_config)
-
-        return CommandResult(
-            success=True,
-            message=(
-                "Tool loop detection enabled"
-                if enabled
-                else "Tool loop detection disabled"
-            ),
-            new_state=session.state,
-        )