zyket 1.2.13 → 1.2.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zyket",
-  "version": "1.2.13",
+  "version": "1.2.14",
   "main": "index.js",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1"

package/src/services/express/Express.js

@@ -94,10 +94,6 @@ module.exports = class Express extends Service {
         }
     });
 
-    // In production, serve the built Vite frontend (static assets + SPA fallback).
-    // Registered after API routes so the API keeps precedence.
-    this.#serveFrontend();
-
     // Attach Express to HTTP server - this allows dynamic route registration
     this.#httpServer.removeAllListeners("request");
     this.#httpServer.on("request", this.#app);
@@ -105,34 +101,6 @@ module.exports = class Express extends Service {
     this.#container.get('logger').info(`Express is running on http://localhost:${httpServer.address().port}`);
   }
 
-  #serveFrontend() {
-    const viteEnabled = process.env.VITE_ROOT && process.env.DISABLE_VITE !== 'true';
-    if (process.env.NODE_ENV !== 'production' || !viteEnabled) return;
-
-    const vite = this.#container.has('vite') ? this.#container.get('vite') : null;
-    const distPath = vite?.outDir?.()
-      || path.resolve(process.cwd(), process.env.VITE_ROOT || 'frontend', 'dist');
-    const indexHtml = path.join(distPath, 'index.html');
-
-    if (!fs.existsSync(indexHtml)) {
-      this.#container.get('logger').warn(`Frontend build not found at ${distPath}. Skipping static serving.`);
-      return;
-    }
-
-    this.#app.use(express.static(distPath));
-
-    // SPA fallback: serve index.html for any GET not handled by API routes,
-    // except the Swagger docs path. Uses middleware (Express 5 dropped bare '*').
-    const docsPath = process.env.SWAGGER_PATH || '/docs';
-    this.#app.use((req, res, next) => {
-      if (req.method !== 'GET') return next();
-      if (req.path.startsWith(docsPath)) return next();
-      res.sendFile(indexHtml);
-    });
-
-    this.#container.get('logger').info(`Serving frontend from ${distPath}`);
-  }
-
   async registerRoutes(routes) {
     const methods = ['post', 'get', 'put', 'delete']
     for (const route of routes) {

package/src/services/vite/index.js

@@ -25,12 +25,27 @@ module.exports = class Vite extends Service {
 
     const configFile = this.#resolveConfigFile();
 
-    // In production we build the frontend once and let Express serve the
-    // generated static files. No dev server is started.
+    // In production we build the frontend once and serve it with Vite's own
+    // preview server on its own port (no dev server, no conflict with Express).
     if (this.#isProduction) {
       this.#container.get("logger").info("Building Vite frontend for production...");
       this.#outDir = await buildViteApp({ root: this.#root, configFile });
       this.#container.get("logger").info(`Vite build complete (output: ${this.#outDir})`);
+
+      const { preview } = await import("vite");
+      this.#viteServer = await preview({
+        root: this.#root,
+        configFile,
+        preview: {
+          port: this.#port,
+          strictPort: false,
+        },
+      });
+
+      const previewPort = this.#viteServer.httpServer?.address()?.port ?? this.#port;
+      this.#container.get("logger").info(
+        `Vite preview server running on http://localhost:${previewPort} (root: ${this.#root})`
+      );
       return;
     }