Skip to content

fix(desktop): hide helper process windows - #40137

Closed
zcxGGmu wants to merge 1 commit into
anomalyco:devfrom
zcxGGmu:hide-helper-windows
Closed

zcxGGmu wants to merge 1 commit into
anomalyco:devfrom
zcxGGmu:hide-helper-windows

Conversation

@zcxGGmu

@zcxGGmu zcxGGmu commented Aug 2, 2026

Copy link
Copy Markdown

Summary

  • Set windowsHide for remaining TUI native helper subprocesses (clipboard/editor)
  • Add a shared desktop main-process helper for hidden subprocess windows
  • Use it for app lookup and custom open-path launches

Fixes #31629

Test Plan

  • bun test test/clipboard.test.ts test/editor.test.ts
  • bun test src/main/apps.test.ts src/main/process-options.test.ts
  • bun run typecheck (packages/tui)
  • bun run lint packages/tui/src/clipboard.ts packages/tui/test/clipboard.test.ts packages/tui/src/editor.ts packages/tui/test/editor.test.ts packages/desktop/src/main/apps.ts packages/desktop/src/main/apps.test.ts packages/desktop/src/main/process-options.ts packages/desktop/src/main/process-options.test.ts packages/desktop/src/main/ipc.ts
  • git diff --cached --check
  • staged secret scan

Note: bun run typecheck from packages/desktop is blocked in this Windows checkout by the existing packages/app/src/custom-elements.d.ts symlink placeholder being materialized as the literal text ../../ui/src/custom-elements.d.ts.

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@github-actions github-actions Bot added the needs:compliance This means the issue will auto-close after 2 hours. label Aug 2, 2026
@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

This pull request has been automatically closed because it was not updated to meet our contributing guidelines within the 2-hour window.

Feel free to open a new pull request that follows our guidelines.

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.

Windows: subprocess spawn (git, codegraph, etc.) flashes console window

2 participants