Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
0579bbd to
ad88d23
Compare
ad88d23 to
a1d3226
Compare
a021d9c to
30ab8cb
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 30ab8cb. Configure here.
30ab8cb to
c251c12
Compare

Note
High Risk
Changes cache schema epoch, authoritative projection composition, and local list authority for production Documents filters—incorrect supplement/patch handling could cause stale membership or unsafe local results until network fallback.
Overview
Ships browser-side
soup-flat-v2predicate projections so Documents tabs can re-evaluate locally after realtime/backfill writes, using direct GraphQL fields plus decodedcacheProjectionsupplements (subtype + email-attachment facts).The WASM cache path now composes complete v2
Replacemutations (orPatchwhen mutations omit projection fields) from query/subscription/backfill responses, rejects invalid required backfill capsules without advancing revision/checkpoints, and switchesentityFiltercompilation tocompile_soup_flat_v2.CACHE_SCHEMA_COMPATIBILITY_EPOCHis bumped to 2 to invalidate stale v1 index state.Mixed-deployment safety:
dssGraphqlFetchretries withoutcacheProjectionon legacy servers, exposesgraphqlSoupProjectionSupported(), and flat Soup local authority stays off when projection isn’t supported.apply_authoritative_projection_patchpreserves unmentioned server-owned facts on settlement/partial writes.Large planning docs (
entity-filter-cache-index-plan, server-minted projection plan, v1 manifest) are removed; related plans are retargeted to the optimistic/pagination prerequisites.Reviewed by Cursor Bugbot for commit 6359ae5. Bugbot is set up for automated code reviews on this repo. Configure here.