fix(windows): hide npm launcher proxy child

[Ingwannu]

Summary

• Set windowsHide: true on the final Node-to-Bun npm launcher spawn.
• Prevent headless Windows starts and restarts from allocating a visible, closable console for the long-running proxy child.
• Add a source-scoped regression for the exact Bun child spawn boundary.

Closes #1236

Verification

• taskset -c 0-1 bun test tests/ocx-launcher-source.test.ts tests/ocx-launcher-runtime.test.ts tests/shutdown-launcher.test.ts — 13 passed and 2 platform skips.
• taskset -c 0-1 bun run typecheck — passed.
• taskset -c 0-1 bun run privacy:scan — passed.
• The launcher integration checks cover signal forwarding, proxy teardown, and Codex config restoration after the spawn-option change.

Checklist

• Scope stays focused and avoids unrelated cleanup.
• Docs or release notes were updated when needed.
• Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

Summary by CodeRabbit

• Bug Fixes

  • Prevented an unnecessary console window from appearing when launching the application headlessly on Windows.
  • Preserved standard input, output, and error stream behavior for the long-running process.

• Tests

  • Added coverage to verify the Windows launch behavior and inherited console streams.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 9503d82e-e84a-4026-b67c-e5e82ccfaa74

📥 Commits

Reviewing files that changed from the base of the PR and between fdc47db and 4c40c56.

📒 Files selected for processing (2)

• bin/ocx.mjs
• tests/ocx-launcher-source.test.ts

---

📝 Walkthrough

Walkthrough

The npm launcher now sets windowsHide: true when it spawns Bun. A source-level test verifies that the final spawn preserves inherited stdio and enables Windows console suppression.

Changes

Windows launcher behavior

Layer / File(s)	Summary
Configure and validate hidden Bun spawning bin/ocx.mjs, tests/ocx-launcher-source.test.ts	The Bun child spawn now sets windowsHide: true while retaining inherited stdio. The source-level test verifies both spawn options.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• lidge-jun/opencodex#568: Updates and tests the ocx launcher child-process spawn behavior.
• lidge-jun/opencodex#861: Modifies the same launcher and source test around Bun child-process behavior.
• lidge-jun/opencodex#887: Modifies the same launcher and source test around Bun runtime launching.

Suggested reviewers: lidge-jun, wibias

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the Windows npm launcher fix implemented by this pull request.
Linked Issues check	✅ Passed	The changes implement issue #1236 by adding windowsHide: true to the Bun spawn and adding a regression test for the required behavior.
Out of Scope Changes check	✅ Passed	The changes are limited to the npm launcher and its regression test, with no unrelated code or scope expansion.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch agent/fix-1236-windows-launcher-hide

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

✅ Deterministic PR hygiene checks passed.