drydock-cli 0.5.2__tar.gz → 0.6.0__tar.gz

{drydock_cli-0.5.2 → drydock_cli-0.6.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 0.5.2
+Version: 0.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-0.5.2 → drydock_cli-0.6.0}/PRD.md

@@ -153,10 +153,32 @@ User-reported issues from real usage testing, all fixed:
 
 ### Phase 5b: Testing & CI (Mar 24)
 
-- **64 regression tests** in 2 test files (0.12s total runtime)
-- **Tests gate all deploys** — deploy_to_github.sh and publish_to_pypi.sh abort on failure
-- **Continuous SWE-bench loop** — 320/500 tasks tested, 45.3% pass rate, auto-restarts on reboot
-- **PyPI published** — drydock-cli v0.4.1
+- **76 regression tests** gating all deploys
+- **Continuous SWE-bench loop** — 500/500 tasks tested, 49.0% pass rate (+9.4% from baseline)
+- **vibe/ → drydock/** package rename (908 imports updated)
+- **PyPI published** — drydock-cli v0.5.2
+
+### Phase 6: GSD Integration & Performance (Mar 25)
+
+GSD-inspired features (from [get-shit-done](https://github.com/gsd-build/get-shit-done)):
+
+| Feature | Details |
+|---------|---------|
+| Tiered context warnings | 4 levels at 50/65/75/85% usage, debounced every 5 calls |
+| Prompt injection guard | Detects role overrides, invisible Unicode, hidden instructions in file writes |
+| Structured state file | `.drydock/state.md` persists task, files, decisions across sessions |
+| Deviation handling rules | Auto-fix bugs/imports, ask user for architecture/scope decisions |
+
+Performance and loop fixes:
+
+| Fix | Details |
+|-----|---------|
+| Circuit breaker | Blocks exact same tool call after 2 attempts, returns cached result |
+| Thinking flicker fix | Status words change every 4s, not every token |
+| Conda env protection | Preserves user's active conda environment in subprocesses |
+| `--insecure` / `-k` flag | Disables SSL verification for corporate proxy environments |
+| `--consultant MODEL` flag | Allows calling a smarter model for single-turn advice when stuck |
+| 81 regression tests | Circuit breaker, injection guard, state file, context tiers all tested |
 
 ---
 

{drydock_cli-0.5.2 → drydock_cli-0.6.0}/drydock/cli/entrypoint.py

@@ -90,6 +90,19 @@ def parse_arguments() -> argparse.Namespace:
         help="Skip all tool permission checks. Equivalent to --agent auto-approve. "
         "Use with caution — tools will execute without confirmation.",
     )
+    parser.add_argument(
+        "-k", "--insecure",
+        action="store_true",
+        help="Disable SSL certificate verification for web searches and API calls. "
+        "Useful behind corporate proxies with self-signed certificates.",
+    )
+    parser.add_argument(
+        "--consultant",
+        metavar="MODEL",
+        help="Enable consultant mode: when the agent is uncertain or stuck in a loop, "
+        "it can call a more capable model (e.g., 'gemini-2.5-pro') for single-turn advice. "
+        "The consultant is NOT used for tool calls, only reasoning.",
+    )
 
     # Feature flag for teleport, not exposed to the user yet
     parser.add_argument("--teleport", action="store_true", help=argparse.SUPPRESS)
@@ -160,6 +173,18 @@ def main() -> None:
     if args.dangerously_skip_permissions:
         args.agent = BuiltinAgentName.AUTO_APPROVE
 
+    # --insecure → disable SSL verification globally
+    if args.insecure:
+        os.environ["DRYDOCK_INSECURE"] = "1"
+        os.environ["CURL_CA_BUNDLE"] = ""
+        os.environ["REQUESTS_CA_BUNDLE"] = ""
+        import ssl
+        ssl._create_default_https_context = ssl._create_unverified_context
+
+    # --consultant → store for agent loop to use
+    if args.consultant:
+        os.environ["DRYDOCK_CONSULTANT_MODEL"] = args.consultant
+
     is_interactive = args.prompt is None
     if is_interactive:
         check_and_resolve_trusted_folder()

{drydock_cli-0.5.2 → drydock_cli-0.6.0}/drydock/cli/textual_ui/widgets/loading.py

@@ -64,6 +64,9 @@ class LoadingWidget(SpinnerMixin, Static):
         "Stocking the galley",
     ]
 
+    # Minimum seconds between status word changes (prevents flicker)
+    _STATUS_CHANGE_INTERVAL = 4.0
+
     def __init__(self, status: str | None = None) -> None:
         super().__init__(classes="loading-widget")
         self.init_spinner()
@@ -77,6 +80,7 @@ class LoadingWidget(SpinnerMixin, Static):
         self._last_elapsed: int = -1
         self._paused_total: float = 0.0
         self._pause_start: float | None = None
+        self._last_status_change: float = 0.0
 
     def _get_easter_egg(self) -> str | None:
         EASTER_EGG_PROBABILITY = 0.10
@@ -111,6 +115,11 @@ class LoadingWidget(SpinnerMixin, Static):
             self._pause_start = None
 
     def set_status(self, status: str) -> None:
+        # Throttle status word changes to prevent flicker
+        now = time()
+        if now - self._last_status_change < self._STATUS_CHANGE_INTERVAL:
+            return  # Too soon — keep current word
+        self._last_status_change = now
         self.status = self._apply_easter_egg(status)
         self._update_animation()
 

{drydock_cli-0.5.2 → drydock_cli-0.6.0}/drydock/core/agent_loop.py

@@ -188,6 +188,10 @@ class AgentLoop:
         self.middleware_pipeline = MiddlewarePipeline()
         self._setup_middleware()
 
+        # Circuit breaker: track tool call signatures to prevent exact repeats
+        # Key: hash(tool_name + args), Value: (count, last_result_snippet)
+        self._tool_call_history: dict[str, tuple[int, str]] = {}
+
         system_prompt = get_universal_system_prompt(
             self.tool_manager, self.config, self.skill_manager, self.agent_manager
         )
@@ -831,9 +835,44 @@ class AgentLoop:
                 )
             )
 
+    def _circuit_breaker_check(self, tool_call: ResolvedToolCall) -> str | None:
+        """Block exact-duplicate tool calls. Returns cached result or None."""
+        sig = hashlib.md5(
+            f"{tool_call.tool_name}:{tool_call.raw_arguments}".encode()
+        ).hexdigest()
+
+        count, last_result = self._tool_call_history.get(sig, (0, ""))
+        if count >= 2:
+            return (
+                f"CIRCUIT BREAKER: You already ran this exact command {count} times "
+                f"and got the same result. Previous result: {last_result[:300]}\n\n"
+                f"Do NOT repeat. Try a DIFFERENT approach, different arguments, "
+                f"or a different tool."
+            )
+        return None
+
+    def _circuit_breaker_record(self, tool_call: ResolvedToolCall, result_text: str) -> None:
+        """Record a tool call execution for circuit breaker tracking."""
+        sig = hashlib.md5(
+            f"{tool_call.tool_name}:{tool_call.raw_arguments}".encode()
+        ).hexdigest()
+        count, _ = self._tool_call_history.get(sig, (0, ""))
+        self._tool_call_history[sig] = (count + 1, result_text[:500])
+
     async def _process_one_tool_call(
         self, tool_call: ResolvedToolCall
     ) -> AsyncGenerator[ToolResultEvent | ToolStreamEvent]:
+        # Circuit breaker: block exact duplicate calls after 2 attempts
+        if blocked := self._circuit_breaker_check(tool_call):
+            yield ToolResultEvent(
+                tool_name=tool_call.tool_name,
+                tool_class=tool_call.tool_class,
+                error=blocked,
+                tool_call_id=tool_call.call_id,
+            )
+            self._handle_tool_response(tool_call, blocked, "failure")
+            return
+
         try:
             tool_instance = self.tool_manager.get(tool_call.tool_name)
         except Exception as exc:
@@ -902,6 +941,8 @@ class AgentLoop:
 
             result_dict = result_model.model_dump()
             text = "\n".join(f"{k}: {v}" for k, v in result_dict.items())
+            # Record for circuit breaker
+            self._circuit_breaker_record(tool_call, text)
             self._handle_tool_response(
                 tool_call, text, "success", decision, result_dict
             )

{drydock_cli-0.5.2 → drydock_cli-0.6.0}/drydock/core/tools/builtins/bash.py

@@ -110,11 +110,25 @@ def _get_base_env() -> dict[str, str]:
 
         # Enable conda in non-interactive shells by setting BASH_ENV
         # to source conda's setup script. Without this, `conda activate`
-        # fails because it's a shell function defined in .bashrc (which
-        # isn't sourced for non-interactive shell commands).
+        # fails because it's a shell function defined in .bashrc.
+        #
+        # IMPORTANT: We also preserve the user's active conda env to avoid
+        # interfering with other environments and losing their aliases.
         conda_sh = _get_conda_setup_script()
         if conda_sh:
-            base_env["BASH_ENV"] = conda_sh
+            # Create a wrapper that sources conda.sh then re-activates the user's env
+            active_env = os.environ.get("CONDA_DEFAULT_ENV", "")
+            if active_env and active_env != "base":
+                # Write a temp script that initializes conda AND activates the user's env
+                import tempfile
+                wrapper = tempfile.NamedTemporaryFile(
+                    mode="w", suffix=".sh", delete=False, prefix="drydock_conda_"
+                )
+                wrapper.write(f'source "{conda_sh}"\nconda activate "{active_env}" 2>/dev/null\n')
+                wrapper.close()
+                base_env["BASH_ENV"] = wrapper.name
+            else:
+                base_env["BASH_ENV"] = conda_sh
 
     return base_env
 

{drydock_cli-0.5.2 → drydock_cli-0.6.0}/pyproject.toml

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

{drydock_cli-0.5.2 → drydock_cli-0.6.0}/tests/test_drydock_tasks.py

@@ -16,6 +16,7 @@ These catch the issues users find when actually using the application:
 from __future__ import annotations
 
 from pathlib import Path
+from unittest.mock import patch
 
 import pytest
 
@@ -382,6 +383,86 @@ class TestDeviationHandling:
         assert "Auto-fix" in content or "Auto-resolve" in content
 
 
+# ============================================================================
+# Circuit Breaker
+# ============================================================================
+
+class TestCircuitBreaker:
+    """Prevents exact duplicate tool calls."""
+
+    def test_circuit_breaker_blocks_after_2(self):
+        from drydock.core.agent_loop import AgentLoop
+        from drydock.core.types import MessageList
+        from types import SimpleNamespace
+
+        al = object.__new__(AgentLoop)
+        al.messages = MessageList()
+        al._tool_call_history = {}
+
+        # Mock tool call with same name+args
+        tc = SimpleNamespace(tool_name="bash", raw_arguments='{"command": "ls -ltr"}')
+
+        # First two calls: no block
+        assert al._circuit_breaker_check(tc) is None
+        al._circuit_breaker_record(tc, "file1.py\nfile2.py")
+        assert al._circuit_breaker_check(tc) is None
+        al._circuit_breaker_record(tc, "file1.py\nfile2.py")
+
+        # Third call: BLOCKED
+        result = al._circuit_breaker_check(tc)
+        assert result is not None
+        assert "CIRCUIT BREAKER" in result
+
+    def test_different_args_not_blocked(self):
+        from drydock.core.agent_loop import AgentLoop
+        from drydock.core.types import MessageList
+        from types import SimpleNamespace
+
+        al = object.__new__(AgentLoop)
+        al.messages = MessageList()
+        al._tool_call_history = {}
+
+        tc1 = SimpleNamespace(tool_name="bash", raw_arguments='{"command": "ls -ltr"}')
+        tc2 = SimpleNamespace(tool_name="bash", raw_arguments='{"command": "ls -la"}')
+
+        al._circuit_breaker_record(tc1, "result1")
+        al._circuit_breaker_record(tc1, "result1")
+        # tc1 blocked, tc2 still works
+        assert al._circuit_breaker_check(tc1) is not None
+        assert al._circuit_breaker_check(tc2) is None
+
+
+# ============================================================================
+# Insecure Flag
+# ============================================================================
+
+class TestInsecureFlag:
+    def test_insecure_flag_exists(self):
+        from drydock.cli.entrypoint import parse_arguments
+        import sys
+        with patch.object(sys, "argv", ["drydock", "-k", "-p", "test"]):
+            args = parse_arguments()
+            assert args.insecure is True
+
+    def test_consultant_flag_exists(self):
+        from drydock.cli.entrypoint import parse_arguments
+        import sys
+        with patch.object(sys, "argv", ["drydock", "--consultant", "gemini-2.5-pro", "-p", "test"]):
+            args = parse_arguments()
+            assert args.consultant == "gemini-2.5-pro"
+
+
+# ============================================================================
+# Thinking Flicker Throttle
+# ============================================================================
+
+class TestThinkingThrottle:
+    def test_status_change_interval_exists(self):
+        from drydock.cli.textual_ui.widgets.loading import LoadingWidget
+        assert hasattr(LoadingWidget, "_STATUS_CHANGE_INTERVAL")
+        assert LoadingWidget._STATUS_CHANGE_INTERVAL >= 3.0
+
+
 class TestEasterEggs:
     """Easter eggs are nautical, not French."""