refactor: move donedata support from SCXML module to core State

Add `donedata` parameter to `State.__init__()` so Python-defined final
states can specify donedata callables directly. The SCXML processor now
passes donedata as a separate key instead of mixing it into enter
callbacks. Also removes the no-op `_processing_loop()` call from
`ExecuteBlock`.

Author: fgmacedo

statemachine/io/__init__.py

@@ -45,6 +45,7 @@ class BaseStateKwargs(TypedDict, total=False):
     parallel: bool
     enter: "str | ActionProtocol | Sequence[str] | Sequence[ActionProtocol]"
     exit: "str | ActionProtocol | Sequence[str] | Sequence[ActionProtocol]"
+    donedata: "ActionProtocol | None"
 
 
 class StateKwargs(BaseStateKwargs, total=False):

statemachine/io/scxml/actions.py

@@ -511,12 +511,9 @@ def __init__(self, content: ExecutableContent):
         self.action_callables = [create_action_callable(action) for action in content.actions]
 
     def __call__(self, *args, **kwargs):
-        machine: StateChart = kwargs["machine"]
         for action in self.action_callables:
             action(*args, **kwargs)
 
-        machine._processing_loop()
-
 
 class DoneDataCallable(CallableAction):
     """Evaluates <donedata> params/content and returns the data for done events."""

statemachine/io/scxml/processor.py

@@ -162,14 +162,14 @@ def _process_state(self, state: State) -> StateDefinition:
         if state.parallel:
             state_dict["parallel"] = True
 
-        # Process enter actions + donedata
+        # Process enter actions
         enter_callables: list = [
             ExecuteBlock(content) for content in state.onentry if not content.is_empty
         ]
-        if state.final and state.donedata:
-            enter_callables.append(DoneDataCallable(state.donedata))
         if enter_callables:
             state_dict["enter"] = enter_callables
+        if state.final and state.donedata:
+            state_dict["donedata"] = DoneDataCallable(state.donedata)
 
         # Process exit actions
         if state.onexit:

statemachine/state.py

@@ -8,6 +8,7 @@
 from .callbacks import CallbackGroup
 from .callbacks import CallbackPriority
 from .callbacks import CallbackSpecList
+from .exceptions import InvalidDefinition
 from .exceptions import StateMachineError
 from .i18n import _
 from .transition import Transition
@@ -190,6 +191,7 @@ def __init__(
         history: "List[HistoryState] | None" = None,
         enter: Any = None,
         exit: Any = None,
+        donedata: Any = None,
         _callbacks: Any = None,
     ):
         self.name = name
@@ -212,6 +214,10 @@ def __init__(
         self.exit = self._specs.grouper(CallbackGroup.EXIT).add(
             exit, priority=CallbackPriority.INLINE
         )
+        if donedata is not None:
+            if not final:
+                raise InvalidDefinition(_("'donedata' can only be specified on final states."))
+            self.enter.add(donedata, priority=CallbackPriority.INLINE)
         self.document_order = 0
         self._init_states()