fix: serve empty 500 responses for non-html requests when a build err…

…or occurs

package.json

@@ -1,6 +1,6 @@
 {
   "name": "vite-live-preview",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Vite build with preview.",
   "license": "ISC",
   "repository": {

src/middleware/error.ts

@@ -21,12 +21,18 @@ export default ({ base, getError }: Options): Connect.NextHandleFunction => {
   const clientSrc = JSON.stringify(path.posix.join(base, CLIENT_SCRIPT_NAME));
 
   return (req, res, next) => {
-    if (!req.headers.accept?.includes('html')) return next();
-
     const error = getError();
 
     if (!error) return next();
 
+    if (!req.headers.accept?.includes('html')) {
+      res.statusCode = 500;
+      res.end();
+      debug?.(`served empty error response for "${req.url}".`);
+
+      return;
+    }
+
     const message = ansiHtml(htmlEscape(error.message));
     const html = TEMPLATE_ERROR_HTML
       .replace(/(?=<\/head>)|$/iu, `<script crossorigin="" src=${clientSrc}></script>\n`)

src/plugin/build.ts

@@ -63,6 +63,7 @@ export default ({ reload = true, config, plugins }: LivePreviewOptions = {}): Pl
   let server: PreviewServer | undefined;
   let error: Error | undefined;
   let deferToBuild: DeferredPromise<void> | undefined;
+  let reloadDelay: NodeJS.Timeout | undefined;
   let deferToRequests: DeferredPromise<void> | undefined;
   let deferToRequestsDelay: NodeJS.Timeout | undefined;
   let activeRequestCount = 0;
@@ -121,6 +122,8 @@ export default ({ reload = true, config, plugins }: LivePreviewOptions = {}): Pl
     async buildStart() {
       if (!enabled) return;
 
+      clearTimeout(reloadDelay);
+
       // Delay the build if requests are in progress.
       await deferToRequests?.promise;
 
@@ -166,11 +169,14 @@ export default ({ reload = true, config, plugins }: LivePreviewOptions = {}): Pl
         }
 
         if (reload) {
-          debug?.(`sending page-reload to ${sockets.size} clients...`);
-          sockets.forEach((socket) => {
-            socket.send(JSON.stringify({ type: 'page-reload' }));
-            debug?.(`sent page-reload.`);
-          });
+          clearTimeout(reloadDelay);
+          reloadDelay = setTimeout(() => {
+            debug?.(`sending page-reload to ${sockets.size} clients...`);
+            sockets.forEach((socket) => {
+              socket.send(JSON.stringify({ type: 'page-reload' }));
+              debug?.(`sent page-reload.`);
+            });
+          }, 1000).unref();
         }
 
         return;