Skip to content

feat(temporal): Routed a session's work to the workers serving its project tree. - #15

Closed
moetemp wants to merge 1 commit into
moe/l2-tool-activitiesfrom
moe/worker-affinity
Closed

moetemp wants to merge 1 commit into
moe/l2-tool-activitiesfrom
moe/worker-affinity

Conversation

@moetemp

@moetemp moetemp commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator

This PR adds worker affinity, off by default, behind OPENCODE_TEMPORAL_WORKTREE_AFFINITY=1.

Without it every worker polls one queue, and a worker drawing a session whose tree it has never seen rebuilds it from snapshot packs on its first step. Affinity avoids the rebuild by routing: the queue name is derived from the project worktree (queue.ts), and only workers serving that tree poll it. It is keyed on the project worktree rather than the session directory, so a session started from a subfolder does not wait on a queue nobody polls.

It trades availability for latency, which is why it is opt-in: a session whose tree has no worker polling waits rather than being served elsewhere. Both processes log the queue they use, and the key is derived from the path alone, never from the filesystem, so a client and a worker that do not see the same disk still agree.

Tests: packages/temporal/test/queue.test.ts.

@github-actions

Copy link
Copy Markdown

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

Without affinity any worker draws any session and rebuilds the tree from snapshot packs on its first step. Behind `OPENCODE_TEMPORAL_WORKTREE_AFFINITY=1` the queue is derived from the project worktree, so only the workers standing in that tree poll it, at the cost of a session waiting when none of them is up.
@moetemp
moetemp force-pushed the moe/l2-tool-activities branch from 039a3b1 to c5cfa3b Compare September 15, 2026 01:01
@moetemp
moetemp force-pushed the moe/worker-affinity branch from ef21ffc to e0f2a64 Compare September 15, 2026 01:01
@github-actions

Copy link
Copy Markdown

This pull request has been automatically closed because it was not updated to meet our contributing guidelines within the 2-hour window.

Feel free to open a new pull request that follows our guidelines.

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