Skip to content

fix(windows): hide console windows for AEC, text-monitor, and mic-listener#1234

Open
stantheman0128 wants to merge 1 commit into
OpenWhispr:mainfrom
stantheman0128:fix/win-aec-spawn-hide
Open

fix(windows): hide console windows for AEC, text-monitor, and mic-listener#1234
stantheman0128 wants to merge 1 commit into
OpenWhispr:mainfrom
stantheman0128:fix/win-aec-spawn-hide

Conversation

@stantheman0128

Copy link
Copy Markdown

Summary

  • Several native Windows helpers were spawned without windowsHide: meeting AEC, text-edit monitor, and windows-mic-listener.exe.
  • Those flashes show up during meetings and dictation flows on Windows.
  • Pass windowsHide: true on those spawn sites (same pattern as whisper/ffmpeg/sidecar helpers).

AI assistance

This PR was prepared with AI assistance (Cursor/Grok). Human author: Stan Shih (@stantheman0128). Please review carefully.

Verification / Evidence

Focused tests:

node --test test/helpers/windows-helper-spawn-hide.test.js
# 3 pass, 0 fail

Test plan

  • Source regression for AEC, text-monitor, and Win32 mic-listener spawns
  • Maintainer: start a meeting / dictation edit path on Windows and confirm no helper console flash

…tener

Native Windows helper binaries were spawned without windowsHide, flashing
consoles during meeting AEC, text-edit monitoring, and mic activity detection.

Co-authored-by: Cursor <cursoragent@cursor.com>
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