tenseleyflow/loader / 76195cc

Browse files

Keep populate todos off reads

Authored by espadonne
SHA
76195cc1fe81ae7df8fd61b70bd4637960edac0d
Parents
107ddd4
Tree
2e0a112

2 changed files

StatusFile+-
M src/loader/runtime/workflow.py 2 0
M tests/test_workflow.py 33 0
src/loader/runtime/workflow.pymodified
@@ -118,6 +118,8 @@ _MUTATION_STEP_HINTS = (
118118
     "creating",
119119
     "develop",
120120
     "developing",
121
+    "populate",
122
+    "populating",
121123
     "build",
122124
     "building",
123125
     "update",
tests/test_workflow.pymodified
@@ -964,6 +964,39 @@ def test_advance_todos_from_tool_call_does_not_complete_develop_step_from_refere
964964
     assert "Develop the main index.html file for the nginx guide" in dod.pending_items
965965
966966
967
+def test_advance_todos_from_tool_call_does_not_complete_populate_step_from_reference_read() -> None:
968
+    dod = create_definition_of_done("Create a multi-file nginx guide.")
969
+    sync_todos_to_definition_of_done(
970
+        dod,
971
+        [
972
+            {
973
+                "content": "First, examine the existing fortran guide structure and content",
974
+                "active_form": "Working on: First, examine the existing fortran guide structure and content",
975
+                "status": "pending",
976
+            },
977
+            {
978
+                "content": "Populate content for each chapter",
979
+                "active_form": "Working on: Populate content for each chapter",
980
+                "status": "pending",
981
+            },
982
+        ],
983
+    )
984
+
985
+    assert advance_todos_from_tool_call(
986
+        dod,
987
+        ToolCall(
988
+            id="read-reference-chapter",
989
+            name="read",
990
+            arguments={"file_path": "~/Loader/guides/fortran/chapters/01-introduction.html"},
991
+        ),
992
+    )
993
+    assert (
994
+        "First, examine the existing fortran guide structure and content"
995
+        in dod.completed_items
996
+    )
997
+    assert "Populate content for each chapter" in dod.pending_items
998
+
999
+
9671000
 def test_advance_todos_from_tool_call_does_not_complete_linking_step_from_glob() -> None:
9681001
     dod = create_definition_of_done("Create a multi-file nginx guide.")
9691002
     sync_todos_to_definition_of_done(