Skip to content

Isolate sidecar conversation per originating workbench - #453

Merged
TheGreatAxios merged 2 commits into
cl-7107-repin-7-ledgerfrom
cl-7168-isolate-agent-conversation-per-workbench
Aug 29, 2026
Merged

TheGreatAxios merged 2 commits into
cl-7107-repin-7-ledgerfrom
cl-7168-isolate-agent-conversation-per-workbench

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Aug 28, 2026 •

Copy link
Copy Markdown
Contributor

Same agent principal. Durable turns nested under agent-state/<agentKey>/<workbenchId>/. A new room starts empty.

Stacked on #461: the room is read from the step request's own Mail input (headers.from, already validated upstream by isMail), so this carries no vendored workflow-host delta — the earlier onInboundMail hook in run-child.ts and its kill-date hash record are gone. A mail-less request (an approval resume) keeps the current binding.

https://linear.app/abklabs/issue/CL-7168

@TheGreatAxios
TheGreatAxios force-pushed the cl-7168-isolate-agent-conversation-per-workbench branch from 207eaee to d1754e3 Compare August 28, 2026 19:08
@TheGreatAxios
TheGreatAxios changed the base branch from main to cl-7107-repin-7-ledger August 28, 2026 19:08
@TheGreatAxios

Copy link
Copy Markdown
Contributor Author

Rebased onto the re-pin stack (#461) and linked into it. While resolving the collision with the re-vendored run-child.ts, the hand-added onInboundMail hook became unnecessary: upstream a8bc06ae delivers the inbound Mail as the step request's input, so the sidecar reads headers.from from req in its own invokeStep wrapper. Vendored tree is now byte-identical to #461; the hash-record and ledger-format commits were dropped. Sidecar: tsc clean, 214 tests pass, check:structural green.

Same agent principal. Durable turns nested under agent-state/<agentKey>/<workbenchId>/. A new room starts empty. https://linear.app/abklabs/issue/CL-7168
…-wins file

Supervisor enqueue overwrote one origin file per mailbox, so two queued
mails bound the first under the second's workbench. The step request
already carries this message's Mail as its input (the object the
workflow-host step invoker sends to the agent); read From from it. A
mail-less request (an approval resume) keeps the current binding.
No vendored workflow-host delta.
@TheGreatAxios
TheGreatAxios force-pushed the cl-7168-isolate-agent-conversation-per-workbench branch from d1754e3 to e290dbd Compare August 28, 2026 19:17
@TheGreatAxios
TheGreatAxios merged commit 5078024 into main Aug 29, 2026
5 checks passed
TheGreatAxios added a commit that referenced this pull request Aug 29, 2026
…ation-per-workbench

Isolate sidecar conversation per originating workbench
@TheGreatAxios
TheGreatAxios deleted the cl-7168-isolate-agent-conversation-per-workbench branch August 29, 2026 14:29
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