`2a6f142`

Quiet setup handoffs

Authored by mfwolffe <wolffemf@dukes.jmu.edu> 2 weeks ago

Status	File	+	-
M	`src/loader/runtime/tool_batches.py`	21	0
M	`tests/test_tool_batches.py`	2	11

src/loader/runtime/tool_batches.pymodified

  from __future__ import annotations
 +import shlex
  from collections.abc import Awaitable, Callable
  from dataclasses import dataclass, field
  from pathlib import Path
              dod,
              project_root=self.context.project_root,
+         )
 +        if (
 +            not has_file_artifact_progress
 +            and _is_pure_directory_creation_tool_call(tool_call)
 +        ):
 +            return
          resume_target = _preferred_resume_target_path(
              dod,
              next_pending=next_pending,
      return f"the successful `{tool_call.name}` result"
 +def _is_pure_directory_creation_tool_call(tool_call: ToolCall) -> bool:
 +    if tool_call.name != "bash":
 +        return False
 +    command = str(tool_call.arguments.get("command", "")).strip()
 +    if not command or any(
 +        operator in command for operator in ("&&", "||", ";", "|", "$(", ">", "<")
 +    ):
 +        return False
 +    try:
 +        parts = shlex.split(command)
 +    except ValueError:
 +        return False
 +    return bool(parts) and parts[0] == "mkdir"
++
++
  def _tool_call_label(tool_call: ToolCall) -> str:
      """Human-readable label for one tool call."""
      name = tool_call.name

tests/test_tool_batches.pymodified

  @pytest.mark.asyncio
 -async def test_tool_batch_runner_missing_artifact_nudge_prefers_pending_index_after_mkdir(
 +async def test_tool_batch_runner_missing_artifact_nudge_stays_quiet_after_setup_mkdir(
      temp_dir: Path,
  ) -> None:
      async def assess_confidence(
          consecutive_errors=0,
+     )
 -    assert persistent_messages
 -    message = persistent_messages[-1]
 -    assert "Next step: create `index.html`." in message
 -    assert (
 -        f"Prefer one `write(file_path=..., content=...)` call for `{(nginx_root / 'index.html').resolve(strict=False)}` now."
 -        in message
 -    )
 -    assert "One declared output artifact is still missing." not in message
 -    assert "Do not reread reference material or spend the next turn on bookkeeping." in message
 -    assert "Resume by creating the next output file under `chapters/` now." not in message
 +    assert persistent_messages == []
      assert ephemeral_messages == []