Skip to content

test(client): cover reconnect assistant backfill#82

Merged
adamfgr merged 1 commit into
mainfrom
codex/p0-reconnect-backfill
Jul 6, 2026
Merged

test(client): cover reconnect assistant backfill#82
adamfgr merged 1 commit into
mainfrom
codex/p0-reconnect-backfill

Conversation

@wizzoapp

@wizzoapp wizzoapp Bot commented Jul 6, 2026

Copy link
Copy Markdown

Summary

  • add a client-runtime regression for the 7am failure mode: client receives an initial thread snapshot, disconnects, server canonical state gains a persisted assistant message, replacement session reconnects, and the reconnect snapshot renders the missed assistant message
  • keeps the receive/backfill contract explicit; current implementation already resubscribes and applies snapshots on replacement sessions

Refs ADA-77.

Evidence

  • vp test run src/state/threads-sync.test.ts --reporter verbose (8 passed)
  • vp run --filter @t3tools/client-runtime typecheck
  • Factory gate passed: static checks + clean Codex gpt-5.5 autoreview.

@wizzoapp wizzoapp Bot marked this pull request as ready for review July 6, 2026 07:56
@wizzoapp wizzoapp Bot marked this pull request as draft July 6, 2026 14:44
@wizzoapp wizzoapp Bot marked this pull request as ready for review July 6, 2026 14:44

@adamfgr adamfgr left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Auto-approved by wizzo-approve: all policy conditions verified against HEAD 3b7c197.

@adamfgr adamfgr added this pull request to the merge queue Jul 6, 2026
Merged via the queue into main with commit 820ef81 Jul 6, 2026
12 checks passed
@adamfgr adamfgr deleted the codex/p0-reconnect-backfill branch July 6, 2026 15:00
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