Skip to content

Correlate each onTrigger body await on its own id - #224

Merged
alexanderguy merged 6 commits into
mainfrom
intr-402-correlate-ontrigger-body-signals-for-turn-isolation-and
Oct 12, 2026
Merged

alexanderguy merged 6 commits into
mainfrom
intr-402-correlate-ontrigger-body-signals-for-turn-isolation-and

Conversation

@alexanderguy

Copy link
Copy Markdown
Member

Summary

  • Each onTrigger body awaitSignal publishes a correlationId on the deployment run's relay SignalAwaited. A delivery completes that park only when the id and the author signal name both match.
  • A signal for one correlation does not satisfy a later await of the same name. Two sections may await that name at once. A name-only delivery completes a top-level gate, a deployment-container loop, and a relay park that has no correlation id.
  • The admin client lists each live park. One park keeps Approve. Several parks are chosen by step, and a correlated park is posted with its correlationId. Delivery still requires manage on the deployment run.

Verification

  • gmake all WF_PARALLEL=2 exits 0 (lint, tsc -b, the admin UI bundle, unit tests, the workflow-deploy suite, and hub/db tests). Two live Google GenAI tests skip.
  • The workflow-deploy suite shows a name-only delivery leaving a new onTrigger park waiting, the matching correlation completing it, a late correlation leaving the next turn, two sections receiving separate deliveries, and a name-only delivery completing a top-level gate beside a section.

Closes INTR-402

@alexanderguy

Copy link
Copy Markdown
Member Author

Self-review (code-review skill) returned clean.

@linear

linear Bot commented Oct 12, 2026

Copy link
Copy Markdown

INTR-402

@alexanderguy
alexanderguy merged commit 3dc2186 into main Oct 12, 2026
6 checks passed
@alexanderguy
alexanderguy deleted the intr-402-correlate-ontrigger-body-signals-for-turn-isolation-and branch October 12, 2026 22:47
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