fix: use str(state) instead of state.value — Python binding returns string not enum

bkrabach · bkrabach · commit 70154411b61d · 2026-04-02T21:24:23.000-07:00
diff --git a/amplifier_module_loop_basic/__init__.py b/amplifier_module_loop_basic/__init__.py
@@ -103,7 +103,7 @@ async def execute(
                         CANCEL_REQUESTED,
                         {
                             "orchestrator": "loop-basic",
-                            "state": coordinator.cancellation.state.value,
+                            "state": str(coordinator.cancellation.state),
                             "turn_count": iteration,
                         },
                     )
@@ -285,7 +285,7 @@ async def execute(
                                 CANCEL_REQUESTED,
                                 {
                                     "orchestrator": "loop-basic",
-                                    "state": coordinator.cancellation.state.value,
+                                    "state": str(coordinator.cancellation.state),
                                     "turn_count": iteration,
                                 },
                             )
@@ -599,7 +599,7 @@ async def execute_single_tool(
                                 CANCEL_REQUESTED,
                                 {
                                     "orchestrator": "loop-basic",
-                                    "state": coordinator.cancellation.state.value,
+                                    "state": str(coordinator.cancellation.state),
                                     "turn_count": iteration,
                                 },
                             )
@@ -640,7 +640,7 @@ async def execute_single_tool(
                                 CANCEL_REQUESTED,
                                 {
                                     "orchestrator": "loop-basic",
-                                    "state": coordinator.cancellation.state.value,
+                                    "state": str(coordinator.cancellation.state),
                                     "turn_count": iteration,
                                 },
                             )

Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,7 @@ async def execute(`
`103`	`103`	`CANCEL_REQUESTED,`
`104`	`104`	`{`
`105`	`105`	`"orchestrator": "loop-basic",`
`106`		`- "state": coordinator.cancellation.state.value,`
	`106`	`+ "state": str(coordinator.cancellation.state),`
`107`	`107`	`"turn_count": iteration,`
`108`	`108`	`},`
`109`	`109`	`)`
`@@ -285,7 +285,7 @@ async def execute(`
`285`	`285`	`CANCEL_REQUESTED,`
`286`	`286`	`{`
`287`	`287`	`"orchestrator": "loop-basic",`
`288`		`- "state": coordinator.cancellation.state.value,`
	`288`	`+ "state": str(coordinator.cancellation.state),`
`289`	`289`	`"turn_count": iteration,`
`290`	`290`	`},`
`291`	`291`	`)`
`@@ -599,7 +599,7 @@ async def execute_single_tool(`
`599`	`599`	`CANCEL_REQUESTED,`
`600`	`600`	`{`
`601`	`601`	`"orchestrator": "loop-basic",`
`602`		`- "state": coordinator.cancellation.state.value,`
	`602`	`+ "state": str(coordinator.cancellation.state),`
`603`	`603`	`"turn_count": iteration,`
`604`	`604`	`},`
`605`	`605`	`)`
`@@ -640,7 +640,7 @@ async def execute_single_tool(`
`640`	`640`	`CANCEL_REQUESTED,`
`641`	`641`	`{`
`642`	`642`	`"orchestrator": "loop-basic",`
`643`		`- "state": coordinator.cancellation.state.value,`
	`643`	`+ "state": str(coordinator.cancellation.state),`
`644`	`644`	`"turn_count": iteration,`
`645`	`645`	`},`
`646`	`646`	`)`