Skip to content

Extract workflow step executor#335

Merged
chubes4 merged 2 commits into
mainfrom
cook/workflow-step-executor-20260618a
Jun 18, 2026
Merged

Extract workflow step executor#335
chubes4 merged 2 commits into
mainfrom
cook/workflow-step-executor-20260618a

Conversation

@chubes4

@chubes4 chubes4 commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add an internal workflow run context and step executor to share deterministic step resolution, dispatch, normalization, and context updates.
  • Route top-level workflow steps and foreach nested steps through the shared executor.
  • Route agent steps through the shared ability dispatcher path and preserve agents/chat missing handling.
  • Extend workflow runner smoke coverage for agent step error propagation alongside existing top-level and foreach ability coverage.

Testing

  • php tests/workflow-runner-smoke.php
  • php -l src/Workflows/class-wp-agent-workflow-run-context.php && php -l src/Workflows/class-wp-agent-workflow-step-executor.php && php -l src/Workflows/class-wp-agent-workflow-runner.php && php -l tests/workflow-runner-smoke.php && php -l agents-api.php
  • composer phpstan

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (openai/gpt-5.5)
  • Used for: Drafted the internal workflow step executor refactor and smoke test updates; Chris to review and validate before merge.

@chubes4 chubes4 merged commit 4a65e0c into main Jun 18, 2026
2 checks passed
@chubes4 chubes4 deleted the cook/workflow-step-executor-20260618a branch June 18, 2026 01:23
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