build(client): use shell instead of vite plugin (#102)

vuejs · Dec 21, 2023 · 2f6abc3 · 2f6abc3
1 parent 8453523
commit 2f6abc3
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 13 deletions.
diff --git a/packages/client/package.json b/packages/client/package.json
@@ -15,7 +15,7 @@
     "node": ">=v14.21.3"
   },
   "scripts": {
-    "build": "nr build:lib && vite build",
+    "build": "tsx ./scripts/pre-build.ts && (nr build:lib & vite build)",
     "build:lib": "vite build --config vite.lib.config.ts",
     "stub": "nr stub:lib && vite build --watch",
     "stub:lib": "vite build --config vite.lib.config.ts --watch"

diff --git a/packages/client/scripts/pre-build.ts b/packages/client/scripts/pre-build.ts
@@ -0,0 +1,15 @@
+import { resolve } from 'node:path'
+import { fileURLToPath } from 'node:url'
+import fse from 'fs-extra'
+
+const __dirname = fileURLToPath(new URL('.', import.meta.url))
+
+async function run() {
+  ;['../../browser-extension/client', '../../electron/client', '../../vite/dist/client'].forEach((dir) => {
+    const absoluteDir = resolve(__dirname, dir)
+    if (fse.existsSync(absoluteDir))
+      fse.removeSync(absoluteDir)
+  })
+}
+
+await run()
diff --git a/packages/client/vite.config.ts b/packages/client/vite.config.ts
@@ -14,14 +14,6 @@ export default defineConfig(mergeConfig(baseConfig, {
         // copy
         const clientFile = resolve(__dirname, './dist')
 
-        // Removed `browser-extension` and `electron` on `build:lib`
-        // Remove `vite/dist/client` on building
-        ;['../vite/dist/client'].forEach((dir) => {
-          const absoluteDir = resolve(__dirname, dir)
-          if (fse.existsSync(absoluteDir))
-            fse.removeSync(absoluteDir)
-        })
-
         ;['../browser-extension/client', '../electron/client', '../vite/dist/client'].forEach((dir) => {
           fse.copySync(clientFile, resolve(__dirname, dir))
         })

diff --git a/packages/client/vite.lib.config.ts b/packages/client/vite.lib.config.ts
@@ -45,10 +45,7 @@ export default defineConfig(mergeConfig(baseConfig, {
         ;['../browser-extension/client', '../electron/client'].forEach((dir) => {
           // NOTE: remember the order of `build:lib` and `build`,
           // if change the order, rmSync must set in `build` stage
-          const absoluteDir = resolve(__dirname, dir)
-          if (fse.existsSync(absoluteDir))
-            fse.removeSync(absoluteDir)
-          fse.copySync(clientFile, absoluteDir)
+          fse.copySync(clientFile, resolve(__dirname, dir))
         })
       },
     },