Skip to content

fix(desktop): persist transcript-only batch output#5276

Merged
ComputelessComputer merged 1 commit into
mainfrom
fix/persist-transcript-only-batch
May 19, 2026
Merged

fix(desktop): persist transcript-only batch output#5276
ComputelessComputer merged 1 commit into
mainfrom
fix/persist-transcript-only-batch

Conversation

@ComputelessComputer
Copy link
Copy Markdown
Collaborator

@ComputelessComputer ComputelessComputer commented May 19, 2026

Synthesize word timings for transcript-only batch responses and surface empty batch output as a no-speech error.


Note

Medium Risk
Moderate risk: changes core batch-transcription result handling and introduces synthesized timing data plus a new failure path when no words are produced.

Overview
Batch transcription now handles transcript-only outputs. When batch/streamed responses contain a transcript but no words, the listener synthesizes WordEntry timings (using segment start/duration or overall metadata duration) so results can still be persisted.

Empty batch output is now treated as an error. handleBatchResponse returns a boolean and runBatchSession rejects/marks the session failed with EMPTY_BATCH_TRANSCRIPT_ERROR when no words are produced; tests were added/updated to cover transcript-only persistence and the new no-speech failure behavior.

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

Synthesize word timings for transcript-only batch responses and surface empty batch output as a no-speech error.
@netlify
Copy link
Copy Markdown

netlify Bot commented May 19, 2026

Deploy Preview for old-char canceled.

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

@ComputelessComputer ComputelessComputer merged commit 1e87ec4 into main May 19, 2026
10 checks passed
@ComputelessComputer ComputelessComputer deleted the fix/persist-transcript-only-batch branch May 19, 2026 14:28
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