durable-workflow 2.12.0__tar.gz → 2.12.1__tar.gz

{durable_workflow-2.12.0/src/durable_workflow.egg-info → durable_workflow-2.12.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: durable-workflow
-Version: 2.12.0
+Version: 2.12.1
 Summary: Python client and worker SDK for Durable Workflow Cloud and self-hosted Server
 Author: Durable Workflow Contributors
 License-Expression: MIT

{durable_workflow-2.12.0 → durable_workflow-2.12.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "durable-workflow"
-version = "2.12.0"
+version = "2.12.1"
 description = "Python client and worker SDK for Durable Workflow Cloud and self-hosted Server"
 readme = "README.md"
 requires-python = ">=3.10"
@@ -76,8 +76,8 @@ durable-workflow-replay-conformance = "durable_workflow.replay_conformance:main"
 durable-workflow-workflow-updates-conformance = "durable_workflow.workflow_updates_conformance:main"
 
 [tool.durable-workflow]
-product-train = "2.12.0"
-registry-version = "2.12.0"
+product-train = "2.12.1"
+registry-version = "2.12.1"
 supported-server-versions = "2.8.0"
 worker-protocol-version = "1.20"
 control-plane-version = "2"

{durable_workflow-2.12.0 → durable_workflow-2.12.1}/src/durable_workflow/workflow.py

@@ -4661,6 +4661,7 @@ def _replay_state(
     # Receiver replay uses this exact boundary so interleaved input is applied
     # in durable order without leaking input recorded after the terminal row.
     condition_wait_terminal_indexes: dict[str, int] = {}
+    condition_wait_opened_indexes: dict[str, int] = {}
     # Selection members are bound by their durable sequence instead of being
     # inserted into the ordinary completion-order cursor. This is what lets a
     # loser complete before or after unrelated successor work without changing
@@ -5006,6 +5007,7 @@ def _replay_state(
                 wait_id = payload.get("condition_wait_id")
                 if not isinstance(wait_id, str) or not wait_id:
                     continue
+                condition_wait_opened_indexes.setdefault(wait_id, index)
 
                 if current_wait_id is None:
                     for receiver_index in prefix_receivers:
@@ -5070,7 +5072,14 @@ def _replay_state(
 
             for receiver_index in receivers_since_wait:
                 if receiver_index is not None and receiver_index not in bindings:
-                    bindings[receiver_index] = current_wait_id
+                    if boundary_kind == "condition":
+                        # A terminal row can precede the next physical reopen.
+                        # Later inputs admitted during the first receiver's
+                        # leased replay belong to that reopen, not the wait
+                        # already resolved by the first receiver.
+                        prefix_receivers.append(receiver_index)
+                    else:
+                        bindings[receiver_index] = current_wait_id
             current_wait_id = None
             receivers_since_wait = []
             prefix_can_bind_to_first_wait = boundary_kind == "condition"
@@ -5431,6 +5440,11 @@ def _replay_state(
         terminal_index = condition_wait_terminal_indexes.get(condition_wait_id)
         if terminal_index is not None:
             while pending_receivers and pending_receivers[0].history_index < terminal_index:
+                receiver_wait_id = pending_receivers[0].condition_wait_id
+                receiver_wait_index = (condition_wait_opened_indexes.get(receiver_wait_id)
+                                       if receiver_wait_id is not None else None)
+                if receiver_wait_index is not None and receiver_wait_index > terminal_index:
+                    break
                 _apply_receiver(pending_receivers.pop(0))
             return
         target_index = next(

{durable_workflow-2.12.0 → durable_workflow-2.12.1/src/durable_workflow.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: durable-workflow
-Version: 2.12.0
+Version: 2.12.1
 Summary: Python client and worker SDK for Durable Workflow Cloud and self-hosted Server
 Author: Durable Workflow Contributors
 License-Expression: MIT

{durable_workflow-2.12.0 → durable_workflow-2.12.1}/tests/test_replay_regression_corpus.py

@@ -47,6 +47,7 @@ from tests.test_update_signal_condition_replay import (
     PostConditionReceiversWorkflow,
     UpdateSignalConditionTimerWorkflow,
 )
+from tests.test_wait_condition import SignalCounterUntilFinished
 from tests.test_worker import compatible_cluster_info
 from tests.test_workflow_identity import IdentityAcrossReplay, IdentityContinueAsNew
 
@@ -375,6 +376,7 @@ WORKFLOWS = [
     FencedRecoveryWorkflow,
     PagedRecordedSideEffectsWorkflow,
     PostConditionReceiversWorkflow,
+    SignalCounterUntilFinished,
     RecordedSideEffectWorkflow,
     SelectionAwaitMarkerWorkflow,
     SelectionCancellationQueryWorkflow,

{durable_workflow-2.12.0 → durable_workflow-2.12.1}/tests/test_wait_condition.py

@@ -3,7 +3,7 @@ from __future__ import annotations
 import pytest
 
 from durable_workflow import serializer, workflow
-from durable_workflow.errors import NonDeterministicReplayError
+from durable_workflow.errors import NonDeterministicReplayError, QueryFailed
 from durable_workflow.workflow import (
     CompleteWorkflow,
     WaitCondition,
@@ -567,6 +567,46 @@ class TestReplayWaitCondition:
         assert outcome.commands[0].result == expected
         assert query_state(SignalCounterUntilFinished, history, [], "current") == expected
 
+    @pytest.mark.parametrize("receiver_sequence", [None, 1])
+    @pytest.mark.parametrize("changed_identity", [None, "condition_key", "condition_definition_fingerprint"])
+    def test_signal_during_condition_completion_keeps_its_physical_reopen(
+        self, receiver_sequence: int | None, changed_identity: str | None,
+    ) -> None:
+        history = [
+            {"event_type": "ConditionWaitOpened", "payload": {
+                "condition_wait_id": "wait-first", "condition_key": "done", "sequence": 1,
+            }},
+            _signal_received_event("increment", [3], workflow_sequence=receiver_sequence),
+            _signal_received_event("finish", [], workflow_sequence=receiver_sequence),
+            {"event_type": "SignalApplied", "payload": {"signal_name": "increment"}},
+            {"event_type": "ConditionWaitSatisfied", "payload": {
+                "condition_wait_id": "wait-first", "condition_key": "done", "sequence": 1,
+            }},
+            {"event_type": "ConditionWaitOpened", "payload": {
+                "condition_wait_id": "wait-reopened", "condition_key": "done", "sequence": 2,
+            }},
+        ]
+
+        if changed_identity is not None:
+            history[-1]["payload"][changed_identity] = "changed-definition"
+            with pytest.raises(NonDeterministicReplayError, match="ConditionWaitOpened") as captured:
+                replay(SignalCounterUntilFinished, history, [])
+            assert captured.value.workflow_sequence == 2
+            with pytest.raises(QueryFailed, match="ConditionWaitOpened") as query_error:
+                query_state(SignalCounterUntilFinished, history, [], "current")
+            assert isinstance(query_error.value.__cause__, NonDeterministicReplayError)
+            assert query_error.value.__cause__.workflow_sequence == 2
+            return
+
+        expected = {"count": 3, "done": True, "events": [
+            {"signal": "increment", "amount": 3, "count": 3}, {"signal": "finish", "count": 3},
+        ]}
+        outcome = replay(SignalCounterUntilFinished, history, [])
+        assert len(outcome.commands) == 1
+        assert isinstance(outcome.commands[0], CompleteWorkflow)
+        assert outcome.commands[0].result == expected
+        assert query_state(SignalCounterUntilFinished, history, [], "current") == expected
+
     def test_signal_received_before_reopened_wait_uses_next_wait_when_sequence_absent(self) -> None:
         history = [
             {