Skip to content

🤖 feat: default workflow agents to exec#3611

Merged
ThomasK33 merged 2 commits into
mainfrom
workflow-exec-default
Jun 23, 2026
Merged

🤖 feat: default workflow agents to exec#3611
ThomasK33 merged 2 commits into
mainfrom
workflow-exec-default

Conversation

@ThomasK33

@ThomasK33 ThomasK33 commented Jun 23, 2026

Copy link
Copy Markdown
Member

Summary

Default workflow-owned child agents to exec and let workflow authors opt individual steps into explore, model aliases, and thinking levels. Deep Research now keeps read-only discovery on explore while running verification/synthesis through exec agents that inherit the configured exec sub-agent defaults.

Background

Workflow agents previously defaulted to explore, which made workflows safe for read-only orchestration but too restrictive for stages that need execution-oriented behavior. Deep Research in particular benefits from read-only explore agents for source discovery and exec agents for adversarial verification.

Implementation

  • Added a shared DEFAULT_WORKFLOW_AGENT_ID = "exec" used by CLI, ORPC, and AI service workflow adapters.
  • Extended workflow agent() options so model and thinking are normalized into task-level modelString and thinkingLevel overrides before launch/replay hashing when workflow authors explicitly set them.
  • Updated Deep Research to explicitly use explore for scoping/search/fetch and exec for verification and synthesis without hardcoded model/thinking overrides.
  • Updated workflow-authoring/deep-research docs and regenerated built-in skill content.

Validation

  • bun test src/node/services/workflows/WorkflowTaskServiceAdapter.test.ts src/node/services/workflows/WorkflowRunner.test.ts
  • bun scripts/gen_builtin_skills.ts check
  • make fmt-check
  • make typecheck
  • MUX_ESLINT_CONCURRENCY=1 make lint
  • MUX_ESLINT_CONCURRENCY=1 make static-check

Risks

Moderate workflow behavior change: workflows that relied on omitted agent options now run exec by default. Read-only stages should explicitly set agentId: "explore"; Deep Research has been updated accordingly.


Generated with mux • Model: openai:gpt-5.5 • Thinking: xhigh • Cost: $19.38

Default workflow child agents now use exec unless a workflow step explicitly selects another agent. Deep Research pins its discovery stages to explore and uses exec with fable/high for verification and synthesis.

---

_Generated with `mux` • Model: `openai:gpt-5.5` • Thinking: `xhigh` • Cost: `4.90`_

<!-- mux-attribution: model=openai:gpt-5.5 thinking=xhigh costs=14.90 -->
@ThomasK33

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🚀

Reviewed commit: ce293a4e1e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ThomasK33 ThomasK33 added this pull request to the merge queue Jun 23, 2026
@ThomasK33 ThomasK33 removed this pull request from the merge queue due to a manual request Jun 23, 2026
Remove hardcoded verifier and synthesis model/thinking overrides so Deep Research exec steps inherit the configured exec sub-agent defaults.

---

_Generated with `mux` • Model: `openai:gpt-5.5` • Thinking: `xhigh` • Cost: `9.38`_

<!-- mux-attribution: model=openai:gpt-5.5 thinking=xhigh costs=19.38 -->
@ThomasK33

Copy link
Copy Markdown
Member Author

@codex review

Updated Deep Research to remove the hardcoded fable/high model and thinking overrides so exec verification/synthesis steps inherit the configured exec sub-agent defaults.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. What shall we delve into next?

Reviewed commit: d2bc6ed6ec

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ThomasK33 ThomasK33 added this pull request to the merge queue Jun 23, 2026
Merged via the queue into main with commit 34f8360 Jun 23, 2026
40 of 42 checks passed
@ThomasK33 ThomasK33 deleted the workflow-exec-default branch June 23, 2026 16:26
mux-bot Bot added a commit that referenced this pull request Jun 23, 2026
Dedupe the identical inline argument object declared by WorkflowTaskServiceLike.create and createMany into a named WorkflowTaskCreateArgs interface so the agent-task creation shape has a single source of truth. PR #3611 just added onRefusal to both copies, demonstrating the drift risk. Behavior-preserving: pure type-level refactor, no runtime change.
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