Skip to content

Fix inconsistent speaker label handling in clip transcripts#38

Merged
andmoredev merged 2 commits into
try-to-fix-some-stufffrom
copilot/sub-pr-37
Feb 18, 2026
Merged

Fix inconsistent speaker label handling in clip transcripts#38
andmoredev merged 2 commits into
try-to-fix-some-stufffrom
copilot/sub-pr-37

Conversation

Copilot AI commented Feb 18, 2026

Copy link
Copy Markdown
Contributor

When generating clip transcripts, speaker labels were inconsistently applied: SRT-sourced text omitted the segment speaker label while fallback text included it.

Changes

  • Modified transcript generation to always prepend segment speaker labels regardless of text source (SRT or fallback)
  • Ensures consistent [Speaker]: text format across all transcript segments
// Before: speaker label only applied to fallback
if (relevantEntries.length > 0) {
  const text = relevantEntries.map(e => e.text).join(' ');
  return text;  // Missing speakerLabel
}
const text = segment.transcript || '';
return `${speakerLabel}${text}`;

// After: speaker label applied consistently
if (relevantEntries.length > 0) {
  const text = relevantEntries.map(e => e.text).join(' ');
  return `${speakerLabel}${text}`;
}
const text = segment.transcript || '';
return `${speakerLabel}${text}`;

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: andmoredev <33256364+andmoredev@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP: Address feedback on apply claude code fixes PR Fix inconsistent speaker label handling in clip transcripts Feb 18, 2026
Copilot AI requested a review from andmoredev February 18, 2026 19:49
@andmoredev andmoredev marked this pull request as ready for review February 18, 2026 19:50
@andmoredev andmoredev merged commit be4c1b6 into try-to-fix-some-stuff Feb 18, 2026
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