thegeneralist01/archivr
1
Fork 0
mirror of https://github.com/thegeneralist01/archivr synced 2026-10-09 12:55:00 +02:00
Code Issues Activity

fix: guard summary callbacks during entry render

Browse source
This commit is contained in:
archivr-qa 2026-08-24 15:51:53 +02:00
parent 222d237044
commit c541ca8032
No known key found for this signature in database
1 changed files with 6 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

10
frontend/src/components/ContextRail.jsx
View file

@ -77,6 +77,12 @@ export default function ContextRail({ archiveId, selectedEntry, selectedUids, se
const summaryPollAbortRef = useRef(null) const summaryPollAbortRef = useRef(null)
const summaryGenerateAbortRef = useRef(null) const summaryGenerateAbortRef = useRef(null)
const summarySelectionRef = useRef(null) const summarySelectionRef = useRef(null)
// Update before effects run so a request settled during a new selection's
// render can never apply state or refresh the newly selected entry.
const summarySelectionKey = archiveId && detail?.summary?.entry_uid
? `${archiveId}:${detail.summary.entry_uid}`
: null
summarySelectionRef.current = summarySelectionKey
// ── Bulk-panel state ──────────────────────────────────────────────────── // ── Bulk-panel state ────────────────────────────────────────────────────
const isBulk = selectedUids?.size >= 2 const isBulk = selectedUids?.size >= 2
@ -131,10 +137,6 @@ export default function ContextRail({ archiveId, selectedEntry, selectedUids, se
// Seed the summary from the entry detail payload and stop any poll left over // Seed the summary from the entry detail payload and stop any poll left over
// from the previously selected entry. // from the previously selected entry.
useLayoutEffect(() => { useLayoutEffect(() => {
const selectionKey = archiveId && detail?.summary?.entry_uid
? `${archiveId}:${detail.summary.entry_uid}`
: null
summarySelectionRef.current = selectionKey
clearInterval(summaryPollRef.current) clearInterval(summaryPollRef.current)
summaryPollRef.current = null summaryPollRef.current = null
summaryPollAbortRef.current?.abort() summaryPollAbortRef.current?.abort()