Page through the graph window and load more without losing your place - #40
Merged
Conversation
The window is deterministic, so the next 300 is well defined. Both window routes accept an offset and return page metadata with honest totals; pages are globally consistent (page N plus N plus 1 equals the first 2N window, proven by walking the whole index with no duplicates or gaps) and cost the same at offset 18,000 as at 0. A bounded POST returns edges induced among a set of loaded ids so an appended page connects to what is already on screen. The explorer gains Previous and Next with a nodes A to B of M indicator, and Load next N, which appends while preserving the viewport and positions, chunking induced-edge requests under the cap and carrying the project scope on every chunk after review caught it missing. Offset persists with the other view state and Back returns to the prior page. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested in use: since the canvas shows a bounded window, users should be able to get the next 300, and the next.
Review found one blocker: Load more requested induced edges without the project scope. Fixed with a test locking the request body.
Verification: pnpm turbo build 21/21, test 36/36, typecheck 20/20. Graph 164 tests, api 311, dashboard 97.
🤖 Generated with Claude Code