Skip to content

Reuse workflow handlers in nested foreach#340

Merged
chubes4 merged 2 commits into
mainfrom
fix/workflow-nested-handler-context
Jun 18, 2026
Merged

Reuse workflow handlers in nested foreach#340
chubes4 merged 2 commits into
mainfrom
fix/workflow-nested-handler-context

Conversation

@chubes4

@chubes4 chubes4 commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Pass the active workflow step handler map through handler context during step execution.
  • Make default foreach execution reuse that active handler map for nested steps instead of rebuilding filtered defaults.
  • Add smoke coverage for a constructor-injected custom handler inside a nested foreach body.

Testing

  • php tests/workflow-runner-smoke.php
  • composer test
  • composer phpstan

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (GPT-5.5)
  • Used for: implemented and tested nested workflow handler context propagation.

@chubes4 chubes4 merged commit 7c1fcbd into main Jun 18, 2026
2 checks passed
@chubes4 chubes4 deleted the fix/workflow-nested-handler-context branch June 18, 2026 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant