Skip to content

Delay dictation overlay until audio capture is ready#701

Draft
supfeer wants to merge 1 commit into
altic-dev:mainfrom
supfeer:agent/delay-dictation-overlay-after-audio-ready
Draft

Delay dictation overlay until audio capture is ready#701
supfeer wants to merge 1 commit into
altic-dev:mainfrom
supfeer:agent/delay-dictation-overlay-after-audio-ready

Conversation

@supfeer

@supfeer supfeer commented Jul 23, 2026

Copy link
Copy Markdown

Description

Starts audio capture before presenting the dictation UI so the beginning of a spoken phrase is retained.

This PR:

  • marks capture ready only after the first PCM buffer is accepted
  • waits 500 ms of active recording before showing the recording overlay
  • plays the configured dictation start sound when the overlay appears
  • cancels delayed presentation when capture stops or fails
  • keeps app-context capture and AI prewarming on the first-audio boundary

Type of Change

  • 🐞 Bug fix
  • ✨ New feature
  • 💥 Breaking change
  • 🧹 Chore
  • 📝 Documentation update

Related Issue or Discussion

Related to #477, #236, and #592.

Testing

  • Tested on Intel Mac
  • Tested on Apple Silicon Mac
  • Tested on macOS version: 26.5.2
  • Ran linter locally: swiftlint --strict --config .swiftlint.yml Sources
  • Ran formatter locally: swiftformat --config .swiftformat Sources
  • Ran tests locally:

Additional validation:

  • Signed Debug build completed successfully.
  • Installed and launched locally with com.FluidApp.local.

Screenshots / Video

  • No UI/visual changes; screenshots/video are not applicable.

The existing overlay appearance and layout are unchanged. This PR only changes when the existing overlay and start sound are presented relative to active audio capture.

Notes

  • Audio acquisition begins before the 500 ms presentation delay.
  • The delay is measured from the first accepted PCM buffer, not from the hotkey event.

@github-actions github-actions Bot added needs PR template Pull request is missing required template content. needs screenshots Pull request needs screenshot or video evidence. labels Jul 23, 2026
@github-actions

Copy link
Copy Markdown

The PR Policy check is blocking this PR because required template information is missing.

Please update the PR description with:

  • Related Issue or Discussion
  • Screenshots / Video

Visual files detected:

  • Sources/Fluid/ContentView.swift
  • Sources/Fluid/Services/MenuBarManager.swift

Screenshots or video are required for UI, UX, settings, onboarding, overlay, menu bar, or visual behavior changes. If this PR has no visual changes, check the no-visual-change box in the template.

If this remains incomplete for 48 hours after opening, the PR may be closed.

@github-actions github-actions Bot removed needs PR template Pull request is missing required template content. needs screenshots Pull request needs screenshot or video evidence. labels Jul 23, 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.

1 participant