Skip to content

feat(chat): load earlier history from the minimap - #836

Open
grubwithu wants to merge 1 commit into
agegr:mainfrom
grubwithu:feat/minimap-load-earlier
Open

grubwithu wants to merge 1 commit into
agegr:mainfrom
grubwithu:feat/minimap-load-earlier

Conversation

@grubwithu

Copy link
Copy Markdown

The minimap only indexed the loaded page, and older pages were fetched solely by the top sentinel when the main chat area scrolled up. A single agent turn can span hundreds of entries, so the default 50-entry page often holds zero or one user turn and the minimap loses its navigation value.

Add a "Load earlier" row at the top of the minimap preview. Extract the sentinel's fetch into one guarded loadOlderPage shared by both triggers, so the row reuses the in-flight guard and scroll anchoring. The preview panel now also renders when no turn is loaded yet but earlier history exists, which is exactly the huge-single-turn case.

Verified in a 1428-entry session: 28 loads take the minimap from 1 to all 19 user turns, after which the row disappears; scrolling to the top still paginates through the sentinel.

The minimap only indexed the loaded page, and older pages were fetched
solely by the top sentinel when the main chat area scrolled up. A single
agent turn can span hundreds of entries, so the default 50-entry page
often holds zero or one user turn and the minimap loses its navigation
value.

Add a "Load earlier" row at the top of the minimap preview. Extract the
sentinel's fetch into one guarded loadOlderPage shared by both triggers,
so the row reuses the in-flight guard and scroll anchoring. The preview
panel now also renders when no turn is loaded yet but earlier history
exists, which is exactly the huge-single-turn case.

Verified in a 1428-entry session: 28 loads take the minimap from 1 to
all 19 user turns, after which the row disappears; scrolling to the top
still paginates through the sentinel.
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