brandly / brandly/Lax

move old messages out of memory

Open
#17 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
137
Forks
23
PR merge metrics
No merged PRs in 30d

Description

this thing will just keep sucking up messages and eating up memory as long as you leave it open. it also gets increasingly stressful for react to render larger conversations. (improving `MessageList` rendering significantly improved react performance, but switching to a long conversation still hangs slightly)

in the short term, it'd be easy to cap messages per convo and just forget about old messages.

eventually, it'd be nice if this cached old messages to disk. we'd have to coordinate reading them in as you scrolled and such, which would take some work. rejoining a channel and seeing messages from when you were last online would be cool tho

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by tracing MessageList and the conversation message state to understand where messages accumulate and are rendered. Define the short-term cap for stored messages and verify that long conversations no longer retain or render an unbounded history; disk caching and loading while scrolling are described as future work.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, react
Domain
desktop, frontend, performance
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.