etch-loop 0.4.1__tar.gz → 0.4.3__tar.gz

{etch_loop-0.4.1 → etch_loop-0.4.3}/PKG-INFO

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

{etch_loop-0.4.1 → etch_loop-0.4.3}/pyproject.toml

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

etch_loop-0.4.3/src/etch/__init__.py

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

{etch_loop-0.4.1 → etch_loop-0.4.3}/src/etch/display.py

@@ -216,6 +216,8 @@ class EtchDisplay:
             title = f"[{AMBER}]- stopped (max iterations)[/{AMBER}]"
         elif reason == "no_changes":
             title = f"[{GREEN}]+ clean — fixer found nothing[/{GREEN}]"
+        elif reason == "build_failed":
+            title = f"[{RED}]x build failed[/{RED}]"
         else:
             title = f"[{FG}]done[/{FG}]"
 

{etch_loop-0.4.1 → etch_loop-0.4.3}/src/etch/loop.py

@@ -66,58 +66,11 @@ def run(
     }
     last_breaker_signal: str | None = None
     last_breaker_output: str | None = None
-    last_runner_output: str | None = None
     iteration_log: list[dict] = []
+    final_runner_entry: dict = {}
 
     with display.EtchDisplay(target=str(prompt_path.parent)) as disp:
 
-        # ── Runner helper — called at every clean exit point ──────────────────
-        def try_runner(iter_entry: dict) -> str:
-            """Run the runner phase if configured.
-
-            Returns:
-                "skip"   — no RUN.md, proceed with clean exit
-                "clear"  — runner passed, proceed with clean exit
-                "issues" — runner failed, continue the loop
-                "error"  — agent error, break the loop
-            """
-            nonlocal last_runner_output
-            if not run_text:
-                return "skip"
-
-            disp.start_phase("runner")
-            runner_start = time.monotonic()
-            try:
-                runner_output = agent.run(run_text, verbose=verbose)
-            except AgentError as exc:
-                disp.finish_phase("runner", status="error", detail=str(exc),
-                                  duration=time.monotonic() - runner_start, success=False)
-                return "error"
-
-            runner_duration = time.monotonic() - runner_start
-            runner_signal = signals.parse(runner_output)
-            runner_detail = (
-                signals.extract_summary(runner_output)
-                or signals.extract_finding(runner_output)
-            )
-
-            if runner_signal == "clear":
-                disp.finish_phase("runner", status="all clear",
-                                  detail=runner_detail or "build passed",
-                                  duration=runner_duration, success=True)
-                iter_entry["runner"] = {"status": "all clear", "detail": runner_detail}
-                last_runner_output = None
-                return "clear"
-            else:
-                disp.record_issue()
-                stats["issues"] += 1
-                disp.finish_phase("runner", status="build failed",
-                                  detail=runner_detail or "build failed",
-                                  duration=runner_duration, success=False)
-                iter_entry["runner"] = {"status": "build failed", "detail": runner_detail}
-                last_runner_output = runner_output
-                return "issues"
-
         # ── Main loop ─────────────────────────────────────────────────────────
         for iteration in range(1, max_iterations + 1):
             stats["iterations"] = iteration
@@ -148,19 +101,9 @@ def run(
                                   detail=scanner_detail or "nothing to fix",
                                   duration=scanner_duration, success=True)
                 iter_entry["scanner"] = {"status": "all clear", "detail": scanner_detail}
-                runner_result = try_runner(iter_entry)
-                if runner_result == "error":
-                    stats["reason"] = "agent_error"
-                    iteration_log.append(iter_entry)
-                    break
-                elif runner_result == "issues":
-                    stats["reason"] = "issues"
-                    iteration_log.append(iter_entry)
-                    continue
-                else:  # "clear" or "skip"
-                    stats["reason"] = "no_changes"
-                    iteration_log.append(iter_entry)
-                    break
+                stats["reason"] = "no_changes"
+                iteration_log.append(iter_entry)
+                break
 
             disp.finish_phase("scanner", status="issues found",
                               detail=scanner_detail or "issues found",
@@ -179,12 +122,6 @@ def run(
                     f"{last_breaker_output.strip()}\n\n"
                     f"Also address these if not already covered above.\n"
                 )
-            if last_runner_output:
-                fixer_prompt += (
-                    f"\n\n## Build/test failures from previous iteration\n\n"
-                    f"{last_runner_output.strip()}\n\n"
-                    f"Fix the underlying code issues causing these failures.\n"
-                )
 
             # ── Fixer phase ───────────────────────────────────────────────────
             disp.start_phase("fixer")
@@ -216,19 +153,9 @@ def run(
                                       duration=fixer_duration, success=True)
                     iter_entry["fixer"] = {"status": "no changes", "detail": "nothing to fix"}
                     if last_breaker_signal != "issues":
-                        runner_result = try_runner(iter_entry)
-                        if runner_result == "error":
-                            stats["reason"] = "agent_error"
-                            iteration_log.append(iter_entry)
-                            break
-                        elif runner_result == "issues":
-                            stats["reason"] = "issues"
-                            iteration_log.append(iter_entry)
-                            continue
-                        else:  # "clear" or "skip"
-                            stats["reason"] = "no_changes"
-                            iteration_log.append(iter_entry)
-                            break
+                        stats["reason"] = "no_changes"
+                        iteration_log.append(iter_entry)
+                        break
                     iteration_log.append(iter_entry)
                     # Fall through to breaker
 
@@ -284,19 +211,9 @@ def run(
                                   detail=breaker_detail or "no issues found",
                                   duration=breaker_duration, success=True)
                 iter_entry["breaker"] = {"status": "all clear", "detail": breaker_detail}
-                runner_result = try_runner(iter_entry)
-                if runner_result == "error":
-                    stats["reason"] = "agent_error"
-                    iteration_log.append(iter_entry)
-                    break
-                elif runner_result == "issues":
-                    stats["reason"] = "issues"
-                    iteration_log.append(iter_entry)
-                    continue
-                else:  # "clear" or "skip"
-                    stats["reason"] = "clear"
-                    iteration_log.append(iter_entry)
-                    break
+                stats["reason"] = "clear"
+                iteration_log.append(iter_entry)
+                break
             else:
                 disp.record_issue()
                 stats["issues"] += 1
@@ -310,6 +227,38 @@ def run(
         else:
             stats["reason"] = "max_iterations"
 
+        # ── Runner — final step, only when loop ended cleanly ─────────────────
+        if run_text and stats["reason"] in ("clear", "no_changes"):
+            disp.start_phase("runner")
+            runner_start = time.monotonic()
+            try:
+                runner_output = agent.run(run_text, verbose=verbose)
+                runner_duration = time.monotonic() - runner_start
+                runner_signal = signals.parse(runner_output)
+                runner_detail = (
+                    signals.extract_summary(runner_output)
+                    or signals.extract_finding(runner_output)
+                )
+                if runner_signal == "clear":
+                    disp.finish_phase("runner", status="all clear",
+                                      detail=runner_detail or "build passed",
+                                      duration=runner_duration, success=True)
+                    final_runner_entry = {"status": "all clear", "detail": runner_detail}
+                else:
+                    disp.record_issue()
+                    stats["issues"] += 1
+                    disp.finish_phase("runner", status="build failed",
+                                      detail=runner_detail or "build failed",
+                                      duration=runner_duration, success=False)
+                    final_runner_entry = {"status": "build failed", "detail": runner_detail}
+                    stats["reason"] = "build_failed"
+            except AgentError as exc:
+                disp.finish_phase("runner", status="error",
+                                  detail=str(exc),
+                                  duration=time.monotonic() - runner_start, success=False)
+                final_runner_entry = {"status": "error", "detail": str(exc)}
+                stats["reason"] = "agent_error"
+
         stats["elapsed"] = time.monotonic() - start_time
 
     # Live panel is fully closed before printing anything below
@@ -317,6 +266,8 @@ def run(
 
     # ── Write report ──────────────────────────────────────────────────────────
     try:
+        if final_runner_entry:
+            iteration_log.append({"n": "runner", "runner": final_runner_entry})
         report_path = report.write(stats, iteration_log, output_dir=prompt_path.parent)
         if not no_git and not no_commit and stats["fixes"] > 0:
             try:

{etch_loop-0.4.1 → etch_loop-0.4.3}/src/etch/report.py

@@ -87,6 +87,7 @@ def _reason_label(reason: str) -> str:
         "interrupted": "interrupted",
         "agent_error": "stopped — agent error",
         "git_error": "stopped — git error",
+        "build_failed": "stopped — build/tests failed",
     }.get(reason, reason)
 
 

etch_loop-0.4.1/src/etch/__init__.py

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