A channel's newest posts miss texts that arrived after its status was cached #296

Closed
opened 2026-10-22 19:52:42 +00:00 by littlefox · 1 comment
Owner

Found while building #282.

Kom::channel_page(c, None, n) starts from the cached highest_local_no of c (uconfs), and new-text messages didn't update that cache. So a channel's newest page, and the thread list's new threads, missed texts written after the conference status was first fetched, until the cache entry was evicted.

Fix (in #282's PR): on_async raises the cached highest_local_no of each recipient conference to the new text's local number. Covered by the channel_threads integration test (a late reply is listed).

Found while building #282. `Kom::channel_page(c, None, n)` starts from the cached `highest_local_no` of `c` (`uconfs`), and new-text messages didn't update that cache. So a channel's newest page, and the thread list's new threads, missed texts written after the conference status was first fetched, until the cache entry was evicted. Fix (in #282's PR): `on_async` raises the cached `highest_local_no` of each recipient conference to the new text's local number. Covered by the `channel_threads` integration test (a late reply is listed).
littlefox added this to the 0.7: Threads milestone 2026-10-22 19:52:42 +00:00
Author
Owner

Fixed in #297 (on_async NewText raises each recipient's cached highest_local_no). Covered by the devserver channel_threads test, which failed before the fix. Main run #485 green.

Fixed in #297 (on_async NewText raises each recipient's cached highest_local_no). Covered by the devserver channel_threads test, which failed before the fix. Main run #485 green.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
littlefox/neokom#296
No description provided.