fix(lineage): clear stale focused graph during navigation - #612
Conversation
|
Warning Review limit reachedNext included review available in 22 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
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 |
|
Exact-head 09204a4 keeps the focused graph null until the authorized per-post fetch resolves, so navigation shows the existing Loading lineage state instead of a transient global graph. The now-unused global graph fetch/state were deleted. Focused isolation tests, lint, and production build pass. |
…as role=status Its resolved empty/unavailable state carried role="status" like sibling panels' transient loading text does, so mounting it inside the Board's collapsed Advanced Review Tools details collided with every other status region on the page (4 failing App.test.tsx assertions). RankingsPanel's own resolved placeholders carry no ARIA role for the same reason -- only the "Loading..." state announces.
Outcome
Follow-up to protected #609: clears the previous focused graph immediately when the selected post changes, so a new post cannot briefly display the prior post’s isolation reason while its own authorized graph loads.
Verification
The explicit 15-second test budget accommodates the heavily contended local host; the test itself completed in seconds once its worker started.