fix(app-builder): route sessions through cloud-agent-next#3021
Merged
fix(app-builder): route sessions through cloud-agent-next#3021
Conversation
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
Contributor
Author
Code Review SummaryStatus: 1 Issue Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
Other Observations (not in diff)Issues found in unchanged code that cannot receive inline comments:
Files Reviewed (20 files)
No new inline comments were added in this incremental pass; the existing App Builder legacy-session warning remains the only active finding. Fix these issues in Kilo Cloud Reviewed by gpt-5.5-2026-04-23 · 2,858,519 tokens |
eshurakov
force-pushed
the
session/agent_f1ad3e6a-fe7f-4657-b188-bb76a2412e0f
branch
2 times, most recently
from
1a6067b to
4f208a9
Compare
eshurakov
reviewed
May 5, 2026
eshurakov
force-pushed
the
session/agent_f1ad3e6a-fe7f-4657-b188-bb76a2412e0f
branch
2 times, most recently
from
87f291c to
7347276
Compare
…treaming - Default app_builder_project_sessions.worker_version to 'v2' and remove the PostHog feature-flag gate so every new project runs on cloud-agent-next. - Drop the v1 cloud-agent branches from the send/start/interrupt service paths; legacy v1 sessions are now read-only and the backend always upgrades to a fresh v2 session when the user sends a message. - Remove the prepareLegacySession tRPC endpoint and WebSocket streaming coordinator on the client. Ended v1 sessions load their R2 history lazily via a new getLegacySessionMessages tRPC query (fired from the ExpandableSessionBlock via loadMessages), so getProject no longer fans out N R2 reads on every project load. - Simplify v1 session plumbing: v1 messages.ts keeps only the helpers the upgrade path uses; streaming.ts destructures cloudAgentSessionId and drops the partial-message bookkeeping that was only meaningful while v1 had a live WebSocket. - Restore preview polling on v2 stream completion: active v2 sessions built from existing project data were missing the onStreamComplete callback, so follow-up messages on an established v2 session would not restart preview polling after streaming transitioned to idle. - Drop redundant workerVersion parameter from onSessionChanged since App Builder is always on cloud-agent-next now. - Fix dev app-builder tunnel script to write APP_BUILDER_URL to apps/web/.env.development.local, which is what Next.js dev actually reads (the repo-root file is ignored when vercel env pull has created the apps/web copy as a real file instead of a symlink).
eshurakov
force-pushed
the
session/agent_f1ad3e6a-fe7f-4657-b188-bb76a2412e0f
branch
from
7347276 to
31d1ca9
Compare
jeanduplessis
approved these changes
May 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
prepareLegacySessionendpoints and set new App Builder session rows to default tov2.Verification