Skip to content

docs(rfc): feedback on dynamic workflows RFC#3481

Open
dcieslak19973 wants to merge 1 commit into
coder:mainfrom
dcieslak19973:rfc-feedback-dynamic-workflows
Open

docs(rfc): feedback on dynamic workflows RFC#3481
dcieslak19973 wants to merge 1 commit into
coder:mainfrom
dcieslak19973:rfc-feedback-dynamic-workflows

Conversation

@dcieslak19973
Copy link
Copy Markdown

This adds a short review of the dynamic workflows RFC (added in #3431) as a companion doc in rfc/.

The v1 scope looks right: a bounded, single-player, deterministic agent-orchestration engine. This feedback does not argue for adding features to v1. It proposes keeping four abstractions one notch more general than v1 strictly needs, so later use cases can ride the same engine without a rewrite:

  1. External-input steps — let a durable step resolve to a recorded result supplied by a human or an external event, not only an agent task. Unlocks human-in-the-loop approval, after-action reviews, and bounded incident investigation later.
  2. Addressable run identity + event bus — keep runs observable independent of the launching chat (the RFC already anticipates dashboard/list and background views).
  3. Determinism boundary, written down — affirm deterministic replay, and note that journaling external inputs is what keeps reactive-but-bounded runs inside it.
  4. Pluggable AgentExecutor — keep the execution path behind an interface (mux TaskService as the sole v1 implementation) so non-mux backends (e.g. Anthropic Managed Agents) can fulfill steps later. A clean decoupling/testability win regardless.

Each ask is near-zero v1 build cost, and the doc explicitly lists what it is not asking for.

Only adds rfc/20260529_dynamic-workflows.feedback.md — no code changes.

🤖 Generated with Claude Code

Companion review of the dynamic workflows RFC (coder#3431). Endorses the narrow v1 scope and proposes four low-cost extension seams -- external/human-input steps, addressable run identity + event bus, an explicit determinism boundary, and a pluggable AgentExecutor -- so human-in-the-loop approval, after-action reviews, bounded incident investigation, and non-mux execution backends stay open without expanding v1.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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