`47db0c5`

Advance first empty child retry

Authored by mfwolffe <wolffemf@dukes.jmu.edu> 1 week ago

SHA: 47db0c5d3f5e6ed3958393efb3f2f079217619f0
Parents: d37b2b4
Tree: e6467a6

2 changed files

Status	File	+	-
M	`src/loader/runtime/repair.py`	1	3
M	`tests/test_repair.py`	12	12

src/loader/runtime/repair.pymodified

          retry_number: int,
          max_empty_retries: int,
      ) -> str | None:
 -        if retry_number < 2:
 -            return None
          if not self._has_confirmed_output_file_progress(dod):
              return None
          if self._has_confirmed_substantive_output_file_progress(dod):
          retry_number: int,
          outline_label: str | None,
      ) -> str | None:
 -        if not require_first_substantive_output or retry_number < 2:
 +        if not require_first_substantive_output or retry_number < 1:
              return None
          if target.suffix.lower() not in {".html", ".htm"}:
              return None

tests/test_repair.pymodified

+     )
 -def test_first_substantive_retry_activates_on_second_empty_turn(
 +def test_first_substantive_retry_activates_on_first_empty_turn(
      temp_dir: Path,
  ) -> None:
      context = build_context(
      decision = repairer.handle_empty_response(
          task="Create a multi-file nginx guide.",
          original_task=None,
 -        empty_retry_count=2,
 +        empty_retry_count=1,
          max_empty_retries=4,
          dod=dod,
+     )
      assert decision.should_continue is True
      assert decision.retry_message is not None
 -    assert "Next missing planned artifact: `introduction.html`" in decision.retry_message
      assert (
 -        "Resume with this exact next step: continue `Write the introduction chapter` "
 -        "by creating `introduction.html`."
 +        "Continue `Write the introduction chapter` by creating `introduction.html`."
          in decision.retry_message
+     )
      assert "Next declared output under `chapters/`" not in decision.retry_message
      assert (
 -        f"Prefer one `write(content=...)` call for `{(chapters / 'introduction.html').resolve(strict=False)}` "
 -        "before more research."
 +        f'Emit this tool shape now: `write(file_path="{display_runtime_path(chapters / "introduction.html")}", content="...")`.'
          in decision.retry_message
+     )
      assert decision.should_continue is True
      assert decision.retry_message is not None
 -    assert "Next missing planned artifact: `01-introduction.html`" in decision.retry_message
      assert (
 -        "Resume with this exact next step: continue `Write the introduction chapter` "
 -        "by creating `01-introduction.html`."
 +        "Continue `Write the introduction chapter` by creating `01-introduction.html`."
 +        in decision.retry_message
 +    )
 +    assert (
 +        f'Emit this tool shape now: `write(file_path="{display_runtime_path(chapters / "01-introduction.html")}", content="...")`.'
          in decision.retry_message
+     )
      assert "Next observed output pattern under `chapters/`" not in decision.retry_message
      assert (
 -        "It mirrors the observed filename pattern from another `chapters/` directory "
 -        "you already inspected."
 +        f"You already read `{display_runtime_path(reference_chapters / '01-introduction.html')}`; "
 +        "reuse its overall structure as the starting pattern for this new file, then adapt the "
 +        "content to the current target."
          in decision.retry_message
+     )