Repository navigation
Conversation
|
Hey! Your PR title Please update it to start with one of:
Where See CONTRIBUTING.md for details. |
|
This PR doesn't fully meet our contributing guidelines and PR template. What needs to be fixed:
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. |
e301a18 to
b0e96ee
Compare
|
Deployment/update note:
The earlier push hook failure was environmental ( |
|
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. |
Summary
writeSSE()work/event,/global/event, and the workspace control-plane event streammessage.part.deltabus publish logging from info to debug and add relay regression testsRoot cause
message.part.deltastorms were being forwarded one-by-one through async EventEmitter listeners that calledawait stream.writeSSE(...). Since EventEmitter does not await async listeners, slow consumers could accumulate an unbounded backlog of pending writes and payload allocations, eventually wedging the sharedserveinstance.Testing
bun test test/server/relay.test.ts test/acp/event-subscription.test.ts --timeout 30000(frompackages/opencode)bun typecheck(frompackages/opencode)bun turbo typecheck(ran via push hook)Closes #17977.