etch-loop 0.4.8__tar.gz → 0.5.0__tar.gz

{etch_loop-0.4.8 → etch_loop-0.5.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: etch-loop
-Version: 0.4.8
+Version: 0.5.0
 Summary: Run Claude Code in a fix-break loop until your codebase is clean
 License: MIT
 Requires-Python: >=3.11

{etch_loop-0.4.8 → etch_loop-0.5.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "etch-loop"
-version = "0.4.8"
+version = "0.5.0"
 requires-python = ">=3.11"
 description = "Run Claude Code in a fix-break loop until your codebase is clean"
 readme = "README.md"

etch_loop-0.5.0/src/etch/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.5.0"

{etch_loop-0.4.8 → etch_loop-0.5.0}/src/etch/agent.py

@@ -13,6 +13,7 @@ def run(
     prompt: str,
     verbose: bool = False,
     tick_callback: Callable[[str], None] | None = None,
+    timeout: int = 300,
 ) -> str:
     """Run the Claude agent with the given prompt piped to stdin.
 
@@ -58,7 +59,7 @@ def run(
         try:
             process.stdin.write(prompt)
             process.stdin.close()
-        except BrokenPipeError as exc:
+        except OSError as exc:
             stdin_exc.append(exc)
 
     stdin_writer = threading.Thread(target=write_stdin, daemon=True)
@@ -71,11 +72,8 @@ def run(
         process.kill()
         raise AgentError(f"Failed to write prompt to claude stdin: {stdin_exc[0]}") from stdin_exc[0]
 
-    if process.stdout is None:
-        process.kill()
-        raise AgentError("claude subprocess has no stdout")
-
     output_lines: list[str] = []
+    stderr_lines: list[str] = []
     lock = threading.Lock()
 
     def read_stdout() -> None:
@@ -87,23 +85,28 @@ def run(
             if tick_callback is not None:
                 tick_callback(line)
 
+    def read_stderr() -> None:
+        for line in process.stderr:
+            stderr_lines.append(line)
+
     reader = threading.Thread(target=read_stdout, daemon=True)
+    stderr_reader = threading.Thread(target=read_stderr, daemon=True)
     reader.start()
-    reader.join(timeout=300)
+    stderr_reader.start()
+    reader.join(timeout=timeout)
     if reader.is_alive():
         process.kill()
         raise AgentError("claude subprocess timed out (output reader still running)")
 
+    stderr_reader.join(timeout=10)
+
     try:
         process.wait(timeout=10)
     except subprocess.TimeoutExpired:
         process.kill()
         raise AgentError("claude subprocess timed out waiting for exit")
 
-    # Capture stderr for error reporting
-    stderr_output = ""
-    if process.stderr:
-        stderr_output = process.stderr.read().strip()
+    stderr_output = "".join(stderr_lines).strip()
 
     if process.returncode != 0:
         detail = stderr_output or "(no stderr)"

{etch_loop-0.4.8 → etch_loop-0.5.0}/src/etch/analyze.py

@@ -299,10 +299,11 @@ You are a test engineer. The fixer has just made changes. Your job is to write t
 1. You MAY edit test files — that is your job
 2. Do NOT touch production code — only tests
 3. If tests fail because of flawed test logic, fix the test and re-run before reporting
-4. When done, write your summary in this exact format — it appears directly in the terminal:
+4. **After tests pass, delete every test file you created during this session** — leave no temporary test files behind
+5. When done, write your summary in this exact format — it appears directly in the terminal:
    `<etch_summary>wrote 4 tests, all 51 passed</etch_summary>`
    `<etch_summary>2 tests failed — TypeError in test_auth.py:38, production bug in token.py:12</etch_summary>`
-5. End with EXACTLY one of these on its own line:
+6. End with EXACTLY one of these on its own line:
    `ETCH_ALL_CLEAR` — if all tests pass
    `ETCH_ISSUES_FOUND` — if tests reveal a bug in production code
 """

{etch_loop-0.4.8 → etch_loop-0.5.0}/src/etch/display.py

@@ -376,8 +376,8 @@ class InitDisplay:
 
         table = Table.grid(padding=(0, 1))
         table.add_column(width=2)   # symbol
-        table.add_column(width=12)  # label
-        table.add_column()          # detail / scanbar
+        table.add_column(width=20)  # label
+        table.add_column()          # scanbar (only used while scanning)
 
         for sym, color, text in lines:
             table.add_row(

{etch_loop-0.4.8 → etch_loop-0.5.0}/src/etch/loop.py

@@ -50,7 +50,11 @@ def run(
         return
 
     # Runner is optional — None means the phase is skipped
-    run_text = prompt.load_run(prompt_path)
+    try:
+        run_text = prompt.load_run(prompt_path)
+    except PromptError as exc:
+        display.print_error(str(exc))
+        return
 
     if focus:
         scan_text += f"\n\n## User focus\n\nConcentrate on: {focus}\n"
@@ -105,6 +109,15 @@ def run(
                 iteration_log.append(iter_entry)
                 break
 
+            if scanner_signal == "empty":
+                disp.finish_phase("scanner", status="no signal",
+                                  detail="agent produced no output token",
+                                  duration=scanner_duration, success=False)
+                iter_entry["scanner"] = {"status": "no signal", "detail": "agent produced no output token"}
+                stats["reason"] = "agent_error"
+                iteration_log.append(iter_entry)
+                break
+
             disp.finish_phase("scanner", status="issues found",
                               detail=scanner_detail or "issues found",
                               duration=scanner_duration, success=False)
@@ -156,34 +169,34 @@ def run(
                         stats["reason"] = "no_changes"
                         iteration_log.append(iter_entry)
                         break
-                    iteration_log.append(iter_entry)
                     # Fall through to breaker
 
             # ── Commit ────────────────────────────────────────────────────────
-            fixer_summary = (
-                signals.extract_summary(_fixer_output)
-                or signals.extract_commit_message(_fixer_output, fallback="")
-            )
-            commit_msg = signals.extract_commit_message(
-                _fixer_output, fallback=f"fix(edge): iteration {iteration}"
-            )
-            if not no_git and not no_commit:
-                try:
-                    git.commit(commit_msg)
-                except GitError as exc:
-                    disp.finish_phase("fixer", status="commit error", detail=str(exc),
-                                      duration=fixer_duration, success=False)
-                    stats["reason"] = "git_error"
-                    iteration_log.append(iter_entry)
-                    break
+            if no_git or changed:
+                fixer_summary = (
+                    signals.extract_summary(_fixer_output)
+                    or signals.extract_commit_message(_fixer_output, fallback="")
+                )
+                commit_msg = signals.extract_commit_message(
+                    _fixer_output, fallback=f"fix(edge): iteration {iteration}"
+                )
+                if not no_git and not no_commit:
+                    try:
+                        git.commit(commit_msg)
+                    except GitError as exc:
+                        disp.finish_phase("fixer", status="commit error", detail=str(exc),
+                                          duration=fixer_duration, success=False)
+                        stats["reason"] = "git_error"
+                        iteration_log.append(iter_entry)
+                        break
 
-            disp.record_fix()
-            stats["fixes"] += 1
-            status_label = "changed" if (no_git or no_commit) else "committed"
-            fixer_detail = fixer_summary or commit_msg
-            disp.finish_phase("fixer", status=status_label, detail=fixer_detail,
-                              duration=fixer_duration, success=True)
-            iter_entry["fixer"] = {"status": status_label, "detail": fixer_detail}
+                disp.record_fix()
+                stats["fixes"] += 1
+                status_label = "changed" if (no_git or no_commit) else "committed"
+                fixer_detail = fixer_summary or commit_msg
+                disp.finish_phase("fixer", status=status_label, detail=fixer_detail,
+                                  duration=fixer_duration, success=True)
+                iter_entry["fixer"] = {"status": status_label, "detail": fixer_detail}
 
             # ── Breaker phase ─────────────────────────────────────────────────
             disp.start_phase("breaker")
@@ -214,6 +227,14 @@ def run(
                 stats["reason"] = "clear"
                 iteration_log.append(iter_entry)
                 break
+            elif signal == "empty":
+                disp.finish_phase("breaker", status="no signal",
+                                  detail="agent produced no output token",
+                                  duration=breaker_duration, success=False)
+                iter_entry["breaker"] = {"status": "no signal", "detail": "agent produced no output token"}
+                stats["reason"] = "agent_error"
+                iteration_log.append(iter_entry)
+                break
             else:
                 disp.record_issue()
                 stats["issues"] += 1
@@ -232,7 +253,7 @@ def run(
             disp.start_phase("runner")
             runner_start = time.monotonic()
             try:
-                runner_output = agent.run(run_text, verbose=verbose)
+                runner_output = agent.run(run_text, verbose=verbose, timeout=600)
                 runner_duration = time.monotonic() - runner_start
                 runner_signal = signals.parse(runner_output)
                 runner_detail = (

{etch_loop-0.4.8 → etch_loop-0.5.0}/src/etch/templates/RUN.md

@@ -21,9 +21,10 @@ You are a test engineer. The fixer has just made changes. Your job is to write t
 1. You MAY edit test files — that is your job
 2. Do NOT touch production code — only tests
 3. If tests fail because of flawed test logic, fix the test and re-run before reporting
-4. When done, write your summary in this exact format — it appears directly in the terminal:
+4. **After tests pass, delete every test file you created during this session** — leave no temporary test files behind
+5. When done, write your summary in this exact format — it appears directly in the terminal:
    `<etch_summary>wrote 4 tests, all 51 passed</etch_summary>`
    `<etch_summary>2 tests failed — TypeError in test_auth.py:38, production bug in token.py:12</etch_summary>`
-5. End with EXACTLY one of these on its own line:
+6. End with EXACTLY one of these on its own line:
    `ETCH_ALL_CLEAR` — if all tests pass
    `ETCH_ISSUES_FOUND` — if tests reveal a bug in production code

etch_loop-0.4.8/src/etch/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.4.8"