Skip to content

Add scoped live Voice transcripts#326134

Open
mirimadahmed wants to merge 1 commit into
microsoft:mainfrom
mirimadahmed:scoped-live-transcripts
Open

Add scoped live Voice transcripts#326134
mirimadahmed wants to merge 1 commit into
microsoft:mainfrom
mirimadahmed:scoped-live-transcripts

Conversation

@mirimadahmed

Copy link
Copy Markdown
Contributor

No description provided.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 2d389cdd-c948-4cd8-b17c-c038f35c825d
Copilot AI review requested due to automatic review settings July 16, 2026 10:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds turn-scoped live voice transcripts to prevent stale or duplicate transcription updates.

Changes:

  • Adds turn IDs, revisions, validation, and scoped lifecycle tracking.
  • Handles finalization, barge-in, reconnect, and auto-end scenarios.
  • Adds service and controller tests.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
voiceSessionController.test.ts Tests scoped transcript lifecycle behavior.
voiceClientService.test.ts Tests wire metadata validation and translation.
voiceClientService.ts (common) Extends transcription events with scope metadata.
voiceSessionController.ts Tracks and filters scoped transcription turns.
voiceClientService.ts (browser) Parses and validates scoped wire events.

this._autoListenSuppressed = false;
this._clearAutoListenTimer();
this._pttCurrentTurnId = generateUuid();
this._beginTranscriptionTurn(this._pttCurrentTurnId);
}

private _handleBargeIn(): void {
this._resetTranscriptionTurn();
Comment on lines +290 to +292
if (!status || (msg.turn_id !== undefined && !turnId) || (msg.revision !== undefined && revision === undefined)) {
break;
}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants