Skip to content

feat: use a util to concatenate dictation transcripts - #24

Open
Juozas (juozaspeleckas) wants to merge 7 commits into
mainfrom
dictation-sdk-util
Open

feat: use a util to concatenate dictation transcripts#24
Juozas (juozaspeleckas) wants to merge 7 commits into
mainfrom
dictation-sdk-util

Conversation

@juozaspeleckas

Copy link
Copy Markdown

Summary

Adds applyDictationTranscript usage to the existing /transcribe endpoints in both SDK showcase apps, so the response includes fully assembled transcript text alongside the raw messages.

Copilot AI review requested due to automatic review settings June 16, 2026 07:32

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

Updates the SDK showcase /transcribe endpoints (TypeScript + .NET) to also return assembled dictation transcript text (committed + interim) alongside the raw WebSocket message stream.

Changes:

  • TypeScript Express /transcribe: accumulate transcript snapshots via applyDictationTranscript() and include committedText / interimText in the JSON response.
  • .NET Web API /transcribe: accumulate transcript snapshots via DictationTranscript.Apply() and include committedText / interimText in the JSON response.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
sdk/typescript/express-web-api/src/routes/transcribe.ts Adds transcript snapshot accumulation using the SDK util and returns assembled text fields.
sdk/dotnet/web-api/Endpoints/TranscribeEndpoint.cs Adds transcript snapshot accumulation in the transcript message subscription and returns assembled text fields.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread sdk/dotnet/web-api/Endpoints/TranscribeEndpoint.cs
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.

2 participants