Skip to content

Comments

feat(cloud): agent server#749

Merged
joshsny merged 1 commit intoagent-server/test-infrastructurefrom
agent-server/core
Feb 4, 2026
Merged

feat(cloud): agent server#749
joshsny merged 1 commit intoagent-server/test-infrastructurefrom
agent-server/core

Conversation

@joshsny
Copy link
Contributor

@joshsny joshsny commented Feb 3, 2026

Adds the agent server that will run in the sandbox.

This has a server that clients can connect to directly to stream events and send commands.


This is part 2 of 2 in a stack made with GitButler:

@joshsny joshsny force-pushed the agent-server/core branch 6 times, most recently from f2831b9 to 86e8a3d Compare February 3, 2026 15:41
@joshsny joshsny force-pushed the agent-server/test-infrastructure branch from e59e816 to 6fa64f4 Compare February 3, 2026 15:46
@joshsny joshsny force-pushed the agent-server/core branch 2 times, most recently from 64cb7b4 to bf049cf Compare February 3, 2026 16:06
@joshsny joshsny marked this pull request as ready for review February 3, 2026 16:20
@joshsny joshsny requested a review from a team as a code owner February 3, 2026 16:20
@joshsny joshsny requested review from a team and tatoalo February 3, 2026 16:47
Copy link
Contributor

@tatoalo tatoalo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, just a couple of nits and maybe adding heartbeat for SSEs to prevent timeouts down the line?

async write(chunk) {
tap.process(chunk);
try {
const writer = underlying.getWriter();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

have you experienced any parallel writes issues with the lock?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i haven't, but it's a good point, added an async mutex to handle it

@joshsny joshsny merged commit f939324 into agent-server/test-infrastructure Feb 4, 2026
9 checks passed
@joshsny joshsny deleted the agent-server/core branch February 4, 2026 13:46
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.

2 participants