fix: address SonarCloud findings in invoke module

- Prefix unused ctx parameter in StateChartInvoker.run() with underscore
- Remove unnecessary async from _spawn_one_async (no await in body)
- Add explicit return after swallowing CancelledError with rationale comment
- Add comment to empty on_cancel() in test to explain intent

Author: fgmacedo

statemachine/invoke.py

@@ -179,7 +179,7 @@ def __init__(self, child_class: "type[StateChart]"):
         self._child_class = child_class
         self._child: "StateChart | None" = None
 
-    def run(self, ctx: "InvokeContext") -> Any:
+    def run(self, _ctx: "InvokeContext") -> Any:
         self._child = self._child_class()
         # The child machine starts automatically in its constructor.
         # If it has final states, it will terminate on its own.
@@ -357,7 +357,7 @@ async def spawn_pending_async(self):
             )
         self._states_to_invoke.clear()
 
-    async def _spawn_one_async(self, callback: "CallbackWrapper", **kwargs):
+    def _spawn_one_async(self, callback: "CallbackWrapper", **kwargs):
         state: "State" = kwargs["state"]
         ctx = self._make_context(state)
         invocation = Invocation(invokeid=ctx.invokeid, state_id=state.id, ctx=ctx)
@@ -394,7 +394,9 @@ async def _run_async_handler(
                     internal=True,
                 )
         except asyncio.CancelledError:
-            pass
+            # Intentionally swallowed: the owning state was exited, so this
+            # invocation was cancelled — there is nothing to propagate.
+            return
         except Exception as e:
             if not ctx.cancelled.is_set():
                 self.sm.send("error.execution", error=e, internal=True)

tests/test_invoke.py

@@ -111,7 +111,7 @@ def run(self, ctx: InvokeContext):
                 return "invoker_result"
 
             def on_cancel(self):
-                pass
+                pass  # no-op: only verifying the protocol is satisfied
 
         assert isinstance(MyInvoker(), IInvoke)