Sync public snapshot from freebuff-private

Source: CodebuffAI/freebuff-private@fdace2e259edc724e584ed22389e388fa4cf0597

Author: github-actions[bot]

bun.lock

@@ -52,6 +52,7 @@
     "terminal-image": "^4.1.0",
     "ts-pattern": "^5.9.0",
     "unified": "^11.0.0",
+    "wsl-utils": "^0.1.0",
     "yoga-layout": "^3.2.1",
     "zod": "^4.2.1",
     "zustand": "^5.0.8"

cli/package.json

@@ -1,6 +1,8 @@
+import fs from 'fs'
 import os from 'os'
 
 import open from 'open'
+import { isWsl, powerShellPathFromWsl } from 'wsl-utils'
 
 import { getCliEnv } from './env'
 import { logger } from './logger'
@@ -11,12 +13,30 @@ import { logger } from './logger'
  * On headless Linux (no DISPLAY or WAYLAND_DISPLAY), calling `open()` spawns
  * `xdg-open` which can crash the entire process — even inside a try/catch —
  * because the child process may trigger fatal signals. This wrapper detects
- * headless environments and skips the call entirely.
+ * headless environments and skips the call entirely. WSL is exempt: there
+ * `open()` goes through powershell.exe, which needs no display.
+ *
+ * On WSL, `open()` spawns powershell.exe from the Windows mount. If Windows
+ * interop is disabled (no access to /mnt/c), that spawn fails with ENOENT.
+ * Under Bun the failure is delivered before the `open()` promise resolves and
+ * cannot be caught from here — not by try/catch, and not by an 'error'
+ * listener attached afterward — so the only reliable defense is checking that
+ * powershell.exe exists before calling `open()`. `wsl-utils` is what `open`
+ * itself uses to build the path, so the check matches its behavior exactly.
  *
  * @returns `true` if the browser was (likely) opened, `false` if skipped.
  */
 export async function safeOpen(url: string): Promise<boolean> {
-  if (os.platform() === 'linux') {
+  if (isWsl) {
+    const powershellPath = await powerShellPathFromWsl()
+    if (!fs.existsSync(powershellPath)) {
+      logger.warn(
+        { powershellPath },
+        'WSL detected but powershell.exe is not accessible (Windows interop disabled?). Skipping browser open.',
+      )
+      return false
+    }
+  } else if (os.platform() === 'linux') {
     const env = getCliEnv()
     const hasDisplay = Boolean(env.DISPLAY || env.WAYLAND_DISPLAY)
     if (!hasDisplay) {
@@ -28,7 +48,13 @@ export async function safeOpen(url: string): Promise<boolean> {
   }
 
   try {
-    await open(url)
+    const subprocess = await open(url)
+    // With the default `wait: false`, spawn failures can surface on the
+    // child's 'error' event after the promise resolves; without a listener
+    // they become uncaught exceptions that kill the process.
+    subprocess.once('error', (err) => {
+      logger.error(err, 'Failed to open browser')
+    })
     return true
   } catch (err) {
     logger.error(err, 'Failed to open browser')