Skip to content

TT-7276 Enhance WSAudioPlayer and WavRecorder for improved audio preview handling#293

Merged
sarahentzel merged 2 commits intodevelopfrom
TT-7276-stop
May 6, 2026
Merged

TT-7276 Enhance WSAudioPlayer and WavRecorder for improved audio preview handling#293
sarahentzel merged 2 commits intodevelopfrom
TT-7276-stop

Conversation

@gtryus
Copy link
Copy Markdown
Contributor

@gtryus gtryus commented May 5, 2026

  • Introduced a timeslice constant for live waveform preview in WSAudioPlayer.
  • Added logic to suppress late preview ticks during recording stops.
  • Updated WavRecorder to emit only delta chunks for previews, optimizing performance.
  • Implemented new tests for audio media recorder and delta chunk handling to ensure functionality.

Greg Trihus added 2 commits May 5, 2026 16:24
…iew handling

- Introduced a timeslice constant for live waveform preview in WSAudioPlayer.
- Added logic to suppress late preview ticks during recording stops.
- Updated WavRecorder to emit only delta chunks for previews, optimizing performance.
- Implemented new tests for audio media recorder and delta chunk handling to ensure functionality.
…mproved preview functionality

- AudioMediaRecorder: emit only current timeslice chunk
- WavRecorder: delta-only preview ticks with overlap guard
- WavRecorder: replace setTimeout(100) with worklet recordingComplete promise
- WSAudioPlayer: stale preview guard with generation counter
- Timeslice constant and usage
- Updated RECORD_PREVIEW_TIMESLICE_MS in WSAudioPlayer to enhance waveform preview responsiveness.
- Modified AudioMediaRecorder to emit an accumulated blob for each data available event, ensuring decodable previews.
-  New tests for delta chunk logic and AudioMediaRecorder
@gtryus gtryus requested a review from sarahentzel May 6, 2026 01:51
@sarahentzel sarahentzel merged commit b11dd26 into develop May 6, 2026
2 checks passed
@sarahentzel sarahentzel deleted the TT-7276-stop branch May 6, 2026 17:18
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