Skip to content

feat(host): a chat write is confirmed once its transaction reaches the followed chat - #4253

Closed
charleslpan wants to merge 1 commit into
luke/chat-live-followfrom
luke/chat-write-txid-wait
Closed

charleslpan wants to merge 1 commit into
luke/chat-live-followfrom
luke/chat-write-txid-wait

Conversation

@charleslpan

@charleslpan charleslpan commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Part of making workspace chat live over Neon Realtime. Stacked on #3923. One change: the host waits for a chat write's txid before it answers, as Neon's mutations guide does.

What changes

Tests

  • New compose-coding-agents.test.ts case:
    • a read-to answering a txid waits until that transaction arrives on the followed read
    • a no-op write answers at once
    • a write to a workspace that isn't followed answers at once
  • Checks were run one at a time on the stack's top: tsc for each touched package, biome, oxlint, knip, discarded-effect, the repository checks, and the affected vitest suites.

Made with Luke

View guided diff Turn on auto-fix

…e followed chat

As Neon's mutations guide says: where the service answers a write's txid
and the window follows that workspace's chat over Realtime, the host waits
up to ten seconds for the transaction to arrive on the followed read (the
TxWait liveRows hands out) before it answers, so the chat pushed to the
window already holds the write. A write that changed nothing answers no
txid and waits for nothing, as does one to a chat not followed or polled;
past the wait the write is answered anyway, since it committed.

Co-authored-by: luke-agent[bot] <339870089+luke-agent[bot]@users.noreply.github.com>
@vercel

vercel Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
luke-web Ready Ready Preview Oct 20, 2026 10:12pm UTC

Request Review

@luke-agent

luke-agent Bot commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Closing for now: the developer moved every area onto the central live layer with Neon Realtime off (LUKE_REALTIME, #4267; held polls, #4272), and this change only does something with Realtime on. The branch is kept, so the PR can be reopened and rebased when LUKE_REALTIME goes back on.

This branch was successfully deployed

1 active deployment
Preview — 4f24be74 Deployed Oct 20, 2026 by vercel[bot]
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.

1 participant