Skip to content

feat(graph): auto-paginate consumer graph + loading progress chip#966

Merged
ishaanxgupta merged 1 commit into
mainfrom
mahesh/nova-main-repro-graph
May 18, 2026
Merged

feat(graph): auto-paginate consumer graph + loading progress chip#966
ishaanxgupta merged 1 commit into
mainfrom
mahesh/nova-main-repro-graph

Conversation

@MaheshtheDev

@MaheshtheDev MaheshtheDev commented May 18, 2026

Copy link
Copy Markdown
Member

Accounts with more than 100 documents previously only saw the newest 100 in the memory graph until they manually zoomed out (and even then, only one extra page per zoom). For users with thousands of documents this made the graph appear nearly empty.

Adds an opt-in autoFetchAll flag to useGraphApi that chains fetchNextPage() until every page is loaded, and renders a top-right progress chip ("X memories - Y/Z documents") with a spinner while pages stream in. The chip self-hides on completion. Only the consumer GraphLayoutView opts in; modal and chat-rail variants remain scoped to their existing single-page behaviour.

This is a stopgap for high-memory users. A proper server-side sampling/clustering endpoint is needed to scale past tens of thousands of nodes and will be tracked separately.

Accounts with more than 100 documents previously only saw the newest
100 in the memory graph until they manually zoomed out (and even then,
only one extra page per zoom). For users with thousands of documents
this made the graph appear nearly empty.

Adds an opt-in `autoFetchAll` flag to `useGraphApi` that chains
`fetchNextPage()` until every page is loaded, and renders a top-right
progress chip ("X memories - Y/Z documents") with a spinner while pages
stream in. The chip self-hides on completion. Only the consumer
GraphLayoutView opts in; modal and chat-rail variants remain scoped to
their existing single-page behaviour.

This is a stopgap for high-memory users. A proper server-side
sampling/clustering endpoint is needed to scale past tens of thousands
of nodes and will be tracked separately.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented May 18, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
supermemory-mcp 14e6880 May 18 2026, 05:08 PM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented May 18, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
supermemory-app 14e6880 Commit Preview URL

Branch Preview URL
May 18 2026, 05:08 PM

Copy link
Copy Markdown
Member Author

How to use the Graphite Merge Queue

Add the label Main to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@claude

claude Bot commented May 18, 2026

Copy link
Copy Markdown

Claude finished @MaheshtheDev's task in 1m 17s —— View job


Review Complete

  • Gather context and understand the changes
  • Review the diff for bugs, security issues, and logic errors
  • Check surrounding code for boundary issues
  • Submit review with findings

Summary: Reviewed the auto-pagination implementation for the consumer graph. The code is clean — the useEffect auto-fetch loop properly guards against concurrent fetches, dependencies are correct, and the progress indicator logic is sound. No bugs, security issues, or logic errors found. Score: 10/10

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Overview: Adds opt-in auto-pagination for the consumer memory graph to load all documents progressively, with a progress chip showing loading status.

Issues found: None — this looks good to ship.

The implementation is clean:

  • The useEffect auto-fetch loop in use-graph-api.ts properly guards against concurrent fetches using isPending and isFetchingNextPage
  • Dependencies are correctly specified, and fetchNextPage from TanStack Query is stable
  • Progress indicator logic correctly shows during loading and hides on completion
  • The feature is properly scoped via autoFetchAll flag, leaving modal/chat-rail variants unaffected

Score: 10/10

@ishaanxgupta ishaanxgupta merged commit fbd9f5e into main May 18, 2026
10 checks passed
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.

2 participants