drydock-cli 0.8.1__tar.gz → 0.8.2__tar.gz

{drydock_cli-0.8.1 → drydock_cli-0.8.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 0.8.1
+Version: 0.8.2
 Summary: Drydock — nautical CLI coding agent
 Project-URL: Homepage, https://github.com/fbobe321/drydock
 Project-URL: Repository, https://github.com/fbobe321/drydock

{drydock_cli-0.8.1 → drydock_cli-0.8.2}/drydock/core/agent_loop.py

@@ -599,15 +599,35 @@ class AgentLoop:
                         f"Consider stopping to verify your approach."
                     )
 
-                # Bash abuse detection: if model keeps using bash instead of
-                # search_replace/read_file, redirect it to the proper tools
-                if not has_made_edit and bash_count >= 10 and bash_count % 5 == 0:
-                    self._inject_system_note(
-                        f"STOP using bash. You have run {bash_count} bash commands without making an edit. "
-                        "Use search_replace to edit files, not bash. Use read_file to read files, not cat/head. "
-                        "Use grep (the tool) to search, not bash grep. "
-                        "Call search_replace NOW with your fix."
-                    )
+                # Bash abuse detection: model uses bash instead of proper tools
+                # This is the #1 cause of no-patch failures (88% of cases)
+                if not has_made_edit and bash_count >= 5:
+                    if bash_count == 5:
+                        self.messages.append(LLMMessage(
+                            role=Role.user,
+                            content=(
+                                "You have run 5 bash commands without making any edit. "
+                                "STOP using bash for reading/searching. Use the proper tools: "
+                                "read_file (not cat), grep (not bash grep), search_replace (not sed). "
+                                "Call search_replace NOW to fix the bug."
+                            )
+                        ))
+                    elif bash_count == 8:
+                        self.messages.append(LLMMessage(
+                            role=Role.user,
+                            content=(
+                                "WARNING: 8 bash commands, ZERO edits. You are wasting context. "
+                                "Bash does NOT fix bugs — search_replace does. "
+                                "Use search_replace on your NEXT response or the session ends."
+                            )
+                        ))
+                    elif bash_count >= 12:
+                        # Force stop — model is clearly stuck in bash loop
+                        yield AssistantEvent(
+                            content="\n\n[STOPPED: 12+ bash commands without any edit. Use search_replace.]\n",
+                            stopped_by_middleware=True,
+                        )
+                        return
 
                 # 3-Strike Rule: search_replace keeps failing
                 if search_replace_failures >= 3:

{drydock_cli-0.8.1 → drydock_cli-0.8.2}/drydock/core/prompts/cli.md

@@ -161,6 +161,13 @@ Bug Fixing in Open-Source Repos
 
 NEVER edit test files. The bug is in the library source code, not the tests.
 
+CRITICAL — Tool Usage:
+- To EDIT files: use search_replace. NEVER use bash with sed/awk/echo to edit files.
+- To READ files: use read_file. NEVER use bash with cat/head/tail.
+- To SEARCH code: use grep. NEVER use bash with grep/find/rg.
+- bash is ONLY for: running tests, checking git status, installing packages, running scripts.
+- If you have not called search_replace, you have not fixed anything. Bash commands do not fix bugs.
+
 Context Budget:
 Your context window is your most important resource. Every grep result and file read consumes it.
 Performance degrades as context fills. Budget your investigation:

{drydock_cli-0.8.1 → drydock_cli-0.8.2}/drydock/core/tools/builtins/prompts/swe_bench.md

@@ -29,3 +29,10 @@ COMMON PATTERNS:
 - Missing import: grep for where the symbol is defined, add the import
 - Off-by-one: check loop bounds and slice indices
 - Unhashable type: wrap mutable container in tuple() before hashing
+
+AFTER YOUR FIRST EDIT:
+After making your first edit, ALWAYS check:
+1. grep for other files that import/use the function you changed
+2. If you edited a base class, check if subclasses override the method
+3. If you edited a config/settings file, check if there's a related migration
+4. If you edited a backend, check if other backends need the same fix

{drydock_cli-0.8.1 → drydock_cli-0.8.2}/pyproject.toml

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

{drydock_cli-0.8.1 → drydock_cli-0.8.2}/scripts/deploy_to_github.sh

@@ -37,7 +37,7 @@ AUTH_URL="https://${GITHUB_TOKEN}@github.com/fbobe321/drydock.git"
 
 log "Running regression tests..."
 cd "$DRYDOCK_SRC"
-if ! python3 -m pytest tests/test_drydock_regression.py tests/test_drydock_tasks.py tests/test_loop_detection.py tests/test_agent_tasks.py tests/test_integration.py tests/test_user_issues.py tests/test_real_issues.py \
+if ! python3 -m pip install -q "pytest>=9.0" 2>/dev/null; python3 -m pytest tests/test_drydock_regression.py tests/test_drydock_tasks.py tests/test_loop_detection.py tests/test_agent_tasks.py tests/test_integration.py tests/test_user_issues.py tests/test_real_issues.py \
     -p no:xdist -p no:cov --override-ini="addopts=" -q 2>&1; then
     log "TESTS FAILED — deploy aborted. Fix the tests before deploying."
     exit 1

{drydock_cli-0.8.1 → drydock_cli-0.8.2}/scripts/publish_to_pypi.sh

@@ -25,7 +25,7 @@ PYPI_TOKEN=$(cat "$TOKEN_FILE")
 
 # --- Run tests ---
 log "Running regression tests..."
-if ! $PYTHON -m pytest tests/test_drydock_regression.py tests/test_drydock_tasks.py tests/test_loop_detection.py tests/test_agent_tasks.py tests/test_integration.py tests/test_user_issues.py tests/test_real_issues.py \
+if ! $PYTHON -m pip install -q "pytest>=9.0" 2>/dev/null; $PYTHON -m pytest tests/test_drydock_regression.py tests/test_drydock_tasks.py tests/test_loop_detection.py tests/test_agent_tasks.py tests/test_integration.py tests/test_user_issues.py tests/test_real_issues.py \
     -p no:xdist -p no:cov --override-ini="addopts=" -q 2>&1; then
     log "TESTS FAILED — publish aborted."
     exit 1