Skip to content

Hide anonymized posts from profiles - #916

Draft
RhysSullivan wants to merge 3 commits into
mainfrom
codex/fix-anonymous-profile-linkage
Draft

Hide anonymized posts from profiles#916
RhysSullivan wants to merge 3 commits into
mainfrom
codex/fix-anonymous-profile-linkage

Conversation

@RhysSullivan

Copy link
Copy Markdown
Collaborator

Keeps anonymized posts out of real user profile histories. Adds coverage for mixed privacy settings across servers.

@vercel

vercel Bot commented Jul 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
answer-overflow-main-site Ready Ready Preview, Comment Jul 11, 2026 10:37pm

Request Review

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 11, 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 failed
View logs
answeroverflow-main-site 7348180 Jul 11 2026, 10:36 PM

@charliecreates charliecreates 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.

One blocking pagination issue found.

ctx,
filteredMessages,
);
const identifiedPosts = enrichedPosts.filter(

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Blocking: Filtering only after paginate can return an empty page with isDone: false when a user’s first 20 posts are anonymized. SnapshotInfiniteList renders loading skeletons for that state and never mounts Virtuoso, so it never follows continueCursor to reach later identified posts. Filter/skip anonymized rows while filling the requested page (or have the consumer advance empty, non-terminal pages) so profiles with a leading anonymized run do not get stuck indefinitely.

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