Skip to content

Scope transcript render subscriptions#5565

Merged
ComputelessComputer merged 1 commit into
mainfrom
fix/scoped-transcript-render-subscriptions
Jun 12, 2026
Merged

Scope transcript render subscriptions#5565
ComputelessComputer merged 1 commit into
mainfrom
fix/scoped-transcript-render-subscriptions

Conversation

@ComputelessComputer

@ComputelessComputer ComputelessComputer commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Replace broad transcript render and export table subscriptions with session-scoped row listener data.


Note

Medium Risk
Touches live transcript UI, export, and render request assembly; incorrect row scoping could miss updates or over-subscribe, but behavior is covered by existing and updated tests.

Overview
Introduces render-request-hooks so live footer, transcript renderer, and export paths share one way to build RenderTranscriptRequest data without subscribing to whole Tinybase tables.

Subscription model: useSessionTranscriptRenderData / useTranscriptRenderData resolve session transcript IDs and participant mapping IDs from indexes, then useSyncExternalStore bumps a revision when addRowListener fires on only the relevant transcripts, mapping_session_participant, and humans rows. Human rows are limited to session participants, speaker-assignment hints on those transcripts, and the current user—not every human in the store.

Call-site cleanup: during-session, export-data, and renderer/data-hooks drop duplicated useTable + buildRenderTranscriptRequestFromStore logic; the live footer builds transcriptIdByWordId from shared transcriptRows. useTranscriptOffset now depends on useTranscriptRowsRevision instead of the full transcripts table.

render-transcript: adds buildRenderTranscriptRequestFromRows and collectAssignedHumanIdsFromTranscriptRows (exported types for rows/humans). Tests cover assigned-human collection; during-session tests mock addRowListener / delListener for the new hook path.

Reviewed by Cursor Bugbot for commit 6d35b7c. Bugbot is set up for automated code reviews on this repo. Configure here.

Replace broad transcript render and export table subscriptions with session-scoped row listener data.
@netlify

netlify Bot commented Jun 12, 2026

Copy link
Copy Markdown

Deploy Preview for old-char canceled.

Name Link
🔨 Latest commit 6d35b7c
🔍 Latest deploy log https://app.netlify.com/projects/old-char/deploys/6a2bc5b0617023000839d52b

@ComputelessComputer ComputelessComputer merged commit d9a4a49 into main Jun 12, 2026
10 checks passed
@ComputelessComputer ComputelessComputer deleted the fix/scoped-transcript-render-subscriptions branch June 12, 2026 09:46
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