wip refactor: fold agent sessions on the backend only - #5570
Draft
ehayes2000 wants to merge 16 commits into
Draft
Conversation
Contributor
|
Important Review skippedIgnore keyword(s) in the title. ⛔ Ignored keywords (3)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ehayes2000
force-pushed
the
claude/acp-fold-backend-migration-qc0j64
branch
from
August 11, 2026 20:46
a63f4ce to
50cf0b4
Compare
ehayes2000
force-pushed
the
claude/acp-fold-backend-migration-qc0j64
branch
from
August 11, 2026 20:53
50cf0b4 to
9965314
Compare
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
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.
Rip out the wasm/browser half of the fold. The server already ran a fold machine per live session to keep placeholder rows in step; it is now also the only fold anywhere:
wiremodule: one serializable shape for folded messages, shared by the REST endpoint and the websocket event so the two cannot drift.agent_fold loses its wasm inbound module, cdylib crate type, wasm dependencies and the cli feature split; agent_runtime_protocol loses its transport feature split; the hakari exclusions and workspace-hack memberships for the fold's dependency chain are restored.
The web app no longer folds: the worker, wasm module, hand-written types and the fold stream client are deleted. Channels fetch the folded snapshot and upsert websocket events into a reactive store keyed by agentSessionMessageId. Placeholder-row synthesis (and adoption by the real row) stays, as does hiding duplicate prompts.