drydock-cli 2.5.6__tar.gz → 2.5.7__tar.gz

{drydock_cli-2.5.6 → drydock_cli-2.5.7}/CLAUDE.md

@@ -9,9 +9,12 @@ Drydock is a local CLI coding agent (fork of mistral-vibe, Apache 2.0).
 - **Hardware:** 2x RTX 4060 Ti 16GB, Gemma 4 26B MoE (A4B) via vLLM Docker at localhost:8000
 - **Server:** remus (Ubuntu 22.04, user: bobef)
 - **Active model:** Gemma 4 26B-A4B-it-AWQ-4bit (only 4B active params, ~70 tok/s)
-- **TUI results:** 88/90 PRD projects pass (98%) through the real TUI
-- **SWE-bench:** 70% file match (v3 baseline), 58% (v3 tuned — tighter prompt hurt)
-- **Priority:** TUI experience first, SWE-bench second
+- **TUI build tests:** 303/320 pass --help (94%) — BUT --help IS MEANINGLESS
+- **TUI real functional tests:** 35/63 pass (56%) — the honest number
+- **SWE-bench:** 70% file match (v3 baseline)
+- **Priority:** TUI experience first. Fix drydock bugs, don't simplify PRDs.
+- **370 PRDs** at /data3/drydock_test_projects/ — the benchmark suite
+- **Current version:** v2.5.6 (PyPI + GitHub + user's env)
 
 ## Build & Test
 
@@ -279,21 +282,22 @@ v3 is a from-scratch rewrite using nano-claude-code as foundation. 4 core files,
 
 ### TUI Test Harness
 - **Script:** `scripts/tui_test.py` — pexpect-based, drives the REAL TUI binary
-- **Auto-approves** tool permission prompts
-- **Detects:** write loops, API errors, tool call counts, file creation, --help pass/fail
-- **100 PRDs:** at `/data3/drydock_test_projects/` (88/90 pass = 98%)
-
-### PRD Format (what works)
-- Short (under 30 lines)
-- 4-6 files max
-- Clear package name and file list
-- Test command: `python3 -m package_name --help`
-- Stdlib only (no external deps)
-
-### What breaks PRDs
-- Too many files (8+) — model runs out of turns
-- Complex parsers (custom YAML/XML from scratch)
-- Features requiring external APIs or databases
+- **Real test runner:** `run_real_tests.sh` — functional testing, not just --help
+- **Results:** `real_test_results.txt` — honest pass/fail with functional verification
+- **370 PRDs** at `/data3/drydock_test_projects/` covering CLI tools, integration, edge cases, best practices
+
+### Test Levels
+1. **Build test (--help):** package runs — MEANINGLESS alone
+2. **Functional test:** PRD test cases verified with correct output — THE REAL TEST
+3. **Acceptance test:** specific expected outputs verified
+4. **Quality test:** code quality checks (docstrings, no eval, error handling)
+
+### Critical Testing Rules
+- PRD failures = DryDock bugs. Fix the agent, NOT the PRD.
+- --help passing proves nothing about code quality
+- Safety mechanisms must be ADVISORY not BLOCKING
+- NEVER add circuit breakers that prevent legitimate work
+- The model should be able to answer questions without being forced to edit files
 
 ## When Compacting
 

{drydock_cli-2.5.6 → drydock_cli-2.5.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: drydock-cli
-Version: 2.5.6
+Version: 2.5.7
 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.6 → drydock_cli-2.5.7}/drydock/core/tools/builtins/search_replace.py

@@ -166,6 +166,17 @@ class SearchReplace(
 
             await self._write_file(file_path, modified_content)
 
+            # Auto-verify syntax for Python files
+            if file_path.suffix == ".py":
+                try:
+                    import ast
+                    ast.parse(modified_content)
+                except SyntaxError as e:
+                    block_result.warnings.append(
+                        f"⚠ SYNTAX ERROR after edit at line {e.lineno}: {e.msg}. "
+                        f"Fix this before continuing."
+                    )
+
         yield SearchReplaceResult(
             file=str(file_path),
             blocks_applied=block_result.applied,

{drydock_cli-2.5.6 → drydock_cli-2.5.7}/drydock/core/tools/builtins/write_file.py

@@ -92,12 +92,28 @@ class WriteFile(
 
         await self._write_file(args, file_path)
 
-        yield WriteFileResult(
+        # Auto-verify syntax for Python files
+        syntax_warning = ""
+        if file_path.suffix == ".py":
+            try:
+                import ast
+                ast.parse(args.content)
+            except SyntaxError as e:
+                syntax_warning = (
+                    f"\n\n⚠ SYNTAX ERROR in {file_path.name} line {e.lineno}: {e.msg}"
+                    f"\n  {e.text.rstrip() if e.text else ''}"
+                    f"\n  Fix this before moving to the next file."
+                )
+
+        result = WriteFileResult(
             path=str(file_path),
             bytes_written=content_bytes,
             file_existed=file_existed,
             content=args.content,
         )
+        if syntax_warning:
+            result.content = syntax_warning  # Override content with the warning
+        yield result
 
     _BINARY_EXTENSIONS = frozenset({
         ".pptx", ".xlsx", ".docx", ".pdf", ".zip", ".tar", ".gz", ".bz2",

{drydock_cli-2.5.6 → drydock_cli-2.5.7}/pyproject.toml

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