Skip to content

fix(graph): stop repeated enumeration cursors - #34

Open
rishibalakrishnan wants to merge 2 commits into
mainfrom
fix/enumeration-duplicates
Open

fix(graph): stop repeated enumeration cursors#34
rishibalakrishnan wants to merge 2 commits into
mainfrom
fix/enumeration-duplicates

Conversation

@rishibalakrishnan

@rishibalakrishnan rishibalakrishnan commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • fetch one extra raw follow or list-member row so terminal pages omit the cursor
  • refill getFollows and getList after moderation/hydration filtering, with a bounded repeated-cursor guard
  • keep the change scoped to following and list-member enumeration; starter-pack members use the list-member path

Problem

The AppView could return the final account together with a cursor pointing back to that same tail. Subsequent requests then repeated the account indefinitely. This was reproduced against jama.com, where Jason Nagata appeared repeatedly after the first page.

Verification

  • regression test failed before the fix on the terminal cursor
  • 37 focused follow/list integration tests pass, including real filtering/refill cases
  • 3 fillPage unit tests pass
  • AppView build/typecheck passes
  • focused ESLint passes
  • full Prettier check and git diff --check pass
  • independently reviewed by Fable and Sol with no blocking findings

Scope

This PR intentionally does not change followers, actor-list enumeration, starter-pack search, or unrelated pagination endpoints.

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