Skip to content

fix: resolve edge handlers on successor ready, not on predecessor done - #438

Merged
shentongmartin merged 1 commit into
mainfrom
fix/data_only_mapping
Sep 2, 2025
Merged

fix: resolve edge handlers on successor ready, not on predecessor done#438
shentongmartin merged 1 commit into
mainfrom
fix/data_only_mapping

Conversation

@shentongmartin

Copy link
Copy Markdown
Contributor

What type of PR is this?

fix an issue when a workflow node A 'indirectly maps' to another node B across branch, the field mapping on edge is eagerly resolved as soon as node A returns, which bypasses the branch control and may cause unexpected behavior.

Comment thread compose/dag.go
Change-Id: I0b0a785fda789c2c8812d20b79408ea0082c03a8
@github-actions

github-actions Bot commented Sep 2, 2025

Copy link
Copy Markdown

📊 Coverage Report:

File coverage threshold (20%) satisfied:	PASS
Package coverage threshold (30%) satisfied:	PASS
Total coverage threshold (83%) satisfied:	PASS
Total test coverage: 83.9% (4840/5768)

Comment thread compose/dag.go
@shentongmartin
shentongmartin merged commit 562f366 into main Sep 2, 2025
13 checks passed
@shentongmartin
shentongmartin deleted the fix/data_only_mapping branch September 2, 2025 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants