Skip to content

fix: worker config on workflow blocks - #426

Merged
LeXwDeX merged 5 commits into
mainfrom
fix/425-worker-config-on
Aug 24, 2026
Merged

fix: worker config on workflow blocks#426
LeXwDeX merged 5 commits into
mainfrom
fix/425-worker-config-on

Conversation

@LeXwDeX

@LeXwDeXLeXwDeX commented Aug 24, 2026

Copy link
Copy Markdown
Owner

Closes#425

Why

模型在用 workflow 工具组合 blocks 图时因 schema 词汇不对称误传块级 worker_config 被 fail-closed 拒绝。按用户决策显式接受该字段(B)并补齐诊断与文档(A')。

What changed

  • WorkflowBlock 可选 worker_config { timeout_ms },编译映射进节点且覆盖 node_defaults
  • FIELD_DRIFT_HINTS 覆盖裸 timeout/timeouts/timeout_ms;字段清单文案更新
  • draft 描述与 guide(blocks) 文档同步;严格未知键拒绝保持不变

Evidence

  • dx-verify 门禁 PASS @ fix/425-worker-config-on @ 9854ec3(typecheck / 定向 dag 测试 / dag-core / lint 棘轮 / 文案一致性 grep)
  • 提交时 pre-commit 全仓 turbo typecheck 29/29

Checklist

  • Why, What changed, and Evidence are filled in.
  • specgit finish exits 0.

Blocks previously could set worker_type but not worker_config, so authors
needing a per-block timeout had to inline low-level nodes — and models kept
reaching for the field anyway (fail-closed rejection with no actionable
hint). Accept it explicitly:
- WorkflowBlock gains optional worker_config { timeout_ms }; compilation
threads it through every expanded node, overriding node_defaults
- FIELD_DRIFT_HINTS covers bare timeout/timeouts/timeout_ms; the drift-hint
field list now includes worker_config
- draft-config description and guide(blocks) document the capability
- strict unknown-key parsing stays fail-closed
Tests: block-level value lands on compiled nodes, node_defaults fallback,
block-over-defaults precedence, unknown keys still rejected with hints.
@LeXwDeX
LeXwDeX marked this pull request as ready for review August 24, 2026 06:09
@LeXwDeX
LeXwDeX merged commit 6dc238e into mainAug 24, 2026
9 checks passed
@LeXwDeX
LeXwDeX deleted the fix/425-worker-config-on branch August 24, 2026 09:26
Sign up for freeto 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.

fix: worker config on workflow blocks

1 participant

@LeXwDeX