Skip to content

fix(cloud-agent-next): suppress warm preparation rows - #5809

Merged
eshurakov merged 1 commit into
mainfrom
eshurakov/gallant-heron
Sep 1, 2026
Merged

eshurakov merged 1 commit into
mainfrom
eshurakov/gallant-heron

Conversation

@eshurakov

Copy link
Copy Markdown
Contributor

Summary

  • Keep warm direct-credential reattachment, but omit preparation events and attach preparation metadata when the session is already attached to the same wrapper.
  • Preserve cold-start/rebuild preparation, credential refresh ordering, runtime fences, and visible delivery failures.
  • Cover Cloudflare and Vercel warm refresh, pending/completed visibility, snapshot replay, and failed refresh without prompt delivery.

Control-plane flow only. Existing saved preparation rows are unchanged.

Verification

  • Red/green: all three regression cases failed on unwanted preparation broadcasts before the fix.
  • pnpm --filter cloud-agent-next test src/sandbox-session/session-message-queue.test.ts src/sandbox-session/control-plane-preparing.test.ts src/sandbox-session/control-dispatch.test.ts src/session/preparation-progress.test.ts src/session/preparation-history.test.ts — 246 passed.
  • pnpm -C services/cloud-agent-next/wrapper exec bun test ./src/control/apply-attach.test.ts — 52 passed.
  • pnpm --filter cloud-agent-next typecheck — passed for service and wrapper.
  • pnpm --filter cloud-agent-next lint and explicit regression-test lint — passed.
  • Changed-file formatting and git diff --check — passed.

No live-provider, Workers integration, or browser E2E run.

@kilo-code-bot

kilo-code-bot Bot commented Sep 1, 2026 •

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • services/cloud-agent-next/src/sandbox-session/SandboxSession.ts
  • services/cloud-agent-next/src/sandbox-session/session-message-queue.test.ts

Reviewed by grok-4.6 · Input: 67.2K · Output: 11.8K · Cached: 597.2K

Review guidance: REVIEW.md from base branch main

@eshurakov
eshurakov enabled auto-merge (squash) September 1, 2026 19:11
@eshurakov
eshurakov merged commit 9d03ca3 into main Sep 1, 2026
17 checks passed
@eshurakov
eshurakov deleted the eshurakov/gallant-heron branch September 1, 2026 19:12
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