From 2c929ac71a9b19796b830bcadfa61dfd1e41ed82 Mon Sep 17 00:00:00 2001
From: Josh Wooding <12938082+joshwooding@users.noreply.github.com>
Date: Sat, 8 Oct 2022 16:55:28 +0100
Subject: [PATCH] Fix bail not being defined (#510)

---
 packages/builder-vite/index.ts | 24 +++++++++++++-----------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/packages/builder-vite/index.ts b/packages/builder-vite/index.ts
index bb813f53..651e25a1 100644
--- a/packages/builder-vite/index.ts
+++ b/packages/builder-vite/index.ts
@@ -44,8 +44,20 @@ function iframeMiddleware(options: ExtendedOptions, server: ViteDevServer): Requ
   };
 }
 
+let server: ViteDevServer;
+
+export async function bail(e?: Error): Promise<void> {
+  try {
+    return await server.close();
+  } catch (err) {
+    console.warn('unable to close vite server');
+  }
+
+  throw e;
+}
+
 export const start: ViteBuilder['start'] = async ({ startTime, options, router, server: devServer }) => {
-  const server = await createViteServer(options as ExtendedOptions, devServer);
+  server = await createViteServer(options as ExtendedOptions, devServer);
 
   // Just mock this endpoint (which is really Webpack-specific) so we don't get spammed with 404 in browser devtools
   // TODO: we should either show some sort of progress from Vite, or just try to disable the whole Loader in the Manager UI.
@@ -57,16 +69,6 @@ export const start: ViteBuilder['start'] = async ({ startTime, options, router,
   router.use(await iframeMiddleware(options as ExtendedOptions, server));
   router.use(server.middlewares);
 
-  async function bail(e?: Error): Promise<void> {
-    try {
-      return await server.close();
-    } catch (err) {
-      console.warn('unable to close vite server');
-    }
-
-    throw e;
-  }
-
   return {
     bail,
     stats: null,