Skip to content

fix(chat): stop loading missing thread messages forever - #76

Merged
enisze merged 2 commits into
mainfrom
feature/fix-chat-message-loading-bug
Oct 1, 2026
Merged

enisze merged 2 commits into
mainfrom
feature/fix-chat-message-loading-bug

Conversation

@enisze

@enisze enisze commented Sep 5, 2026

Copy link
Copy Markdown
Owner

A stale thread shell can outlive its detail projection. When that happens, clients repeatedly request a missing snapshot every 250 ms and leave the chat permanently showing “Loading messages…”.

Mark server-confirmed missing thread details as deleted, attach a structured not_found reason to the wire error, and resume from the retained cursor so the client does not restart the missing snapshot request. The client keeps message-based compatibility with older servers.

Verification:

  • vp test run packages/client-runtime/src/state/threads-sync.test.ts
  • vp test run apps/server/src/server.test.ts -t "identifies a missing thread snapshot"
  • scoped contracts and client-runtime typechecks
  • targeted lint and formatting
  • unsigned arm64 DMG build

Created with GPT-5.6 Sol via the Codex harness in T3 Code.

@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Sep 7, 2026
# Conflicts:
#	packages/client-runtime/src/state/threads.ts
@enisze
enisze merged commit d52e0cb into main Oct 1, 2026
6 of 15 checks passed
@enisze
enisze deleted the feature/fix-chat-message-loading-bug branch October 1, 2026 16:07
@github-actions github-actions Bot added the size:S label Oct 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant