`aa40a8b`

Recognize develop todos as mutations

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

SHA: aa40a8bbded4a530916374ecfb53f9a348ead186
Parents: ef7bc99
Tree: de7a758

2 changed files

Status	File	+	-
M	`src/loader/runtime/tool_batches.py`	22	5
M	`tests/test_tool_batches.py`	6	2

src/loader/runtime/tool_batches.pymodified

  _MUTATION_TODO_HINTS = (
      "create",
      "creating",
 +    "develop",
 +    "developing",
 +    "populate",
 +    "populating",
 +    "build",
 +    "building",
      "update",
      "updating",
      "edit",
              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,
              project_root=self.context.project_root,
              messages=list(getattr(self.context.session, "messages", []) or []),
+         )
 +        if (
 +            not has_file_artifact_progress
 +            and _is_pure_directory_creation_tool_call(tool_call)
 +        ):
 +            if (
 +                next_pending
 +                and _todo_is_mutation_step(next_pending)
 +                and resume_target is not None
 +                and resume_target.suffix
 +            ):
 +                self.context.queue_steering_message(
 +                    f"Directory setup is complete. Continue with the next pending item: `{next_pending}`."
 +                    + resume_suffix
 +                    + " Do not reread older reference files before that mutation."
 +                )
 +            return
          use_persistent_handoff = _should_use_persistent_missing_artifact_handoff(
              dod,
              project_root=self.context.project_root,

tests/test_tool_batches.pymodified

  @pytest.mark.asyncio
 -async def test_tool_batch_runner_missing_artifact_nudge_stays_quiet_after_setup_mkdir(
 +async def test_tool_batch_runner_missing_artifact_nudge_names_next_file_after_setup_mkdir(
      temp_dir: Path,
  ) -> None:
      async def assess_confidence(
          consecutive_errors=0,
+     )
 -    assert persistent_messages == []
 +    assert persistent_messages
 +    message = persistent_messages[-1]
 +    assert "Directory setup is complete." in message
 +    assert "Continue with the next pending item: `Develop the main index.html file with proper structure`." in message
 +    assert "Resume by creating `index.html` now." in message
      assert ephemeral_messages == []