Skip to content

chatsync: fixed delayed messages during reconnect catch-up - #140

Merged
Fizzadar merged 3 commits into
mainfrom
highest/plat-38874
Sep 25, 2026
Merged

Fizzadar merged 3 commits into
mainfrom
highest/plat-38874

Conversation

@highesttt

@highesttt highesttt commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

When xchat reconnects, it may need to backfill missed messages while disconnected while live messages are waiting to be sent.
Previously, the behavior was to backfill every chat before delivering any new messages when xchat got disconnected, so a message could be delayed by the backfill from other chats.
This PR changes the behavior to prioritize live messages over backfilling for missed messages between reconnect.

Checklist

@highesttt
highesttt requested a review from Fizzadar September 17, 2026 20:37

@Fizzadar Fizzadar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Some comments, for PRs like this a bit more description would really help to understand it. As-is I really don't have much an idea of what the change is attempting.

Comment thread pkg/twittermeow/websocket.go Outdated
Comment thread pkg/twittermeow/websocket.go Outdated
Comment thread pkg/twittermeow/websocket.go Outdated
Comment thread pkg/connector/xchat_reconnect.go
@highesttt

Copy link
Copy Markdown
Contributor Author

Some comments, for PRs like this a bit more description would really help to understand it. As-is I really don't have much an idea of what the change is attempting.

added a description

@Fizzadar
Fizzadar merged commit 107440c into main Sep 25, 2026
11 checks passed
@Fizzadar
Fizzadar deleted the highest/plat-38874 branch September 25, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants