From 5ae869f17e4b2d7d7ae66c7075756e87d5fe5185 Mon Sep 17 00:00:00 2001
From: bluwy <bjornlu.dev@gmail.com>
Date: Thu, 24 Oct 2024 20:47:58 +0800
Subject: [PATCH] Clean vite setup

---
 site/src/pages/Package.svelte |  7 +++++++
 site/vite.config.js           | 32 ++++----------------------------
 2 files changed, 11 insertions(+), 28 deletions(-)

diff --git a/site/src/pages/Package.svelte b/site/src/pages/Package.svelte
index 96d3522..6560f5a 100644
--- a/site/src/pages/Package.svelte
+++ b/site/src/pages/Package.svelte
@@ -1,4 +1,5 @@
 <script>
+  import { onDestroy } from 'svelte'
   import githubLogo from '../assets/github.svg'
   import gitlabLogo from '../assets/gitlab.svg'
   import gitLogo from '../assets/git.svg'
@@ -26,6 +27,12 @@
   let error = $state('')
   let status = $state('')
 
+  onDestroy(() => {
+    if (worker) {
+      worker.terminate()
+    }
+  })
+
   function createWorker() {
     const worker = new Worker(new URL('../utils/worker.js', import.meta.url), {
       type: 'module'
diff --git a/site/vite.config.js b/site/vite.config.js
index ed1b57a..1808485 100644
--- a/site/vite.config.js
+++ b/site/vite.config.js
@@ -11,13 +11,10 @@ export default defineConfig({
     // TODO: Fix this in Vite
     entries: ['**/*.html', './src/utils/worker.js']
   },
-  plugins: [
-    spaFallbackWithDot(),
-    serveAnalysisJson(),
-    unocss(),
-    svelte(),
-    markdown()
-  ],
+  plugins: [serveAnalysisJson(), unocss(), svelte(), markdown()],
+  esbuild: {
+    legalComments: 'none'
+  },
   build: {
     rollupOptions: {
       input: {
@@ -28,27 +25,6 @@ export default defineConfig({
   }
 })
 
-/**
- * Vite doesn't handle fallback html with dot (.), see https://github.com/vitejs/vite/issues/2415
- * TODO: Review the PR in Vite
- * @returns {import('vite').Plugin}
- */
-function spaFallbackWithDot() {
-  return {
-    name: 'spa-fallback-with-dot',
-    configureServer(server) {
-      return () => {
-        server.middlewares.use((req, res, next) => {
-          if (req.url.includes('.') && !req.url.endsWith('.html')) {
-            req.url = '/index.html'
-          }
-          next()
-        })
-      }
-    }
-  }
-}
-
 const analysisJsonUrl = new URL(
   '../analysis/cache/_results.json',
   import.meta.url