fix: skip applescript when no Chromium browser found (fixes vitejs#11205

) (vitejs#11406)
futurGH · Feb 26, 2023 · 7a7ffd8 · 7a7ffd8
1 parent 762c5c7
commit 7a7ffd8
Showing 1 changed file with 13 additions and 11 deletions.
diff --git a/packages/vite/src/node/server/openBrowser.ts b/packages/vite/src/node/server/openBrowser.ts
@@ -86,17 +86,19 @@ function startBrowserProcess(browser: string | undefined, url: string) {
         preferredOSXBrowser && ps.includes(preferredOSXBrowser)
           ? preferredOSXBrowser
           : supportedChromiumBrowsers.find((b) => ps.includes(b))
-      // Try our best to reuse existing tab with AppleScript
-      execSync(
-        `osascript openChrome.applescript "${encodeURI(
-          url,
-        )}" "${openedBrowser}"`,
-        {
-          cwd: join(VITE_PACKAGE_DIR, 'bin'),
-          stdio: 'ignore',
-        },
-      )
-      return true
+      if (openedBrowser) {
+        // Try our best to reuse existing tab with AppleScript
+        execSync(
+          `osascript openChrome.applescript "${encodeURI(
+            url,
+          )}" "${openedBrowser}"`,
+          {
+            cwd: join(VITE_PACKAGE_DIR, 'bin'),
+            stdio: 'ignore',
+          },
+        )
+        return true
+      }
     } catch (err) {
       // Ignore errors
     }