Skip to content

Commit 2c929ac

Browse files
authored
Fix bail not being defined (#510)
1 parent 70fdb7e commit 2c929ac

File tree

1 file changed

+13
-11
lines changed

1 file changed

+13
-11
lines changed

packages/builder-vite/index.ts

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,20 @@ function iframeMiddleware(options: ExtendedOptions, server: ViteDevServer): Requ
4444
};
4545
}
4646

47+
let server: ViteDevServer;
48+
49+
export async function bail(e?: Error): Promise<void> {
50+
try {
51+
return await server.close();
52+
} catch (err) {
53+
console.warn('unable to close vite server');
54+
}
55+
56+
throw e;
57+
}
58+
4759
export const start: ViteBuilder['start'] = async ({ startTime, options, router, server: devServer }) => {
48-
const server = await createViteServer(options as ExtendedOptions, devServer);
60+
server = await createViteServer(options as ExtendedOptions, devServer);
4961

5062
// Just mock this endpoint (which is really Webpack-specific) so we don't get spammed with 404 in browser devtools
5163
// 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,
5769
router.use(await iframeMiddleware(options as ExtendedOptions, server));
5870
router.use(server.middlewares);
5971

60-
async function bail(e?: Error): Promise<void> {
61-
try {
62-
return await server.close();
63-
} catch (err) {
64-
console.warn('unable to close vite server');
65-
}
66-
67-
throw e;
68-
}
69-
7072
return {
7173
bail,
7274
stats: null,

0 commit comments

Comments
 (0)