genbox 1.0.96 → 1.0.98

package/dist/scanner/index.js

@@ -66,6 +66,8 @@ class ProjectScanner {
         const apps = await this.discoverApps(root, structure, compose, options.exclude);
         // Layer 5.5: Apply framework detection to apps (ports, commands, etc.)
         await this.applyFrameworkDefaults(root, apps, frameworks);
+        // Layer 5.6: Ensure unique ports for all apps (avoid collisions)
+        this.ensureUniquePorts(apps);
         // Layer 6: Analyze environment variables (skip if option set)
         const envAnalysis = options.skipEnv
             ? { required: [], optional: [], secrets: [], references: [], sources: [] }
@@ -168,20 +170,41 @@ class ProjectScanner {
                         const hasStartScript = !!(pkg.scripts?.start || pkg.scripts?.dev || pkg.scripts?.serve);
                         const type = this.inferAppTypeFromPackage(entry.name, pkg, hasStartScript);
                         // Try to detect framework
+                        // Priority: fullstack frameworks > meta-frameworks > dev servers > UI libraries
                         let framework;
                         const deps = { ...pkg.dependencies, ...pkg.devDependencies };
+                        // 1. Fullstack/meta-frameworks (most specific)
                         if (deps['next'])
                             framework = 'nextjs';
+                        else if (deps['nuxt'])
+                            framework = 'nuxt';
+                        else if (deps['@remix-run/react'])
+                            framework = 'remix';
+                        else if (deps['astro'])
+                            framework = 'astro';
+                        else if (deps['@sveltejs/kit'])
+                            framework = 'sveltekit';
+                        // 2. Backend frameworks
                         else if (deps['@nestjs/core'])
                             framework = 'nestjs';
+                        else if (deps['fastify'])
+                            framework = 'fastify';
+                        else if (deps['express'])
+                            framework = 'express';
+                        else if (deps['hono'])
+                            framework = 'hono';
+                        // 3. Dev servers (determines port)
+                        else if (deps['vite'])
+                            framework = 'vite';
+                        // 4. UI libraries (least specific - use dev server port if available)
                         else if (deps['react-admin'])
-                            framework = 'react-admin';
+                            framework = 'vite'; // react-admin uses vite
                         else if (deps['react'])
                             framework = 'react';
                         else if (deps['vue'])
                             framework = 'vue';
-                        else if (deps['express'])
-                            framework = 'express';
+                        else if (deps['svelte'])
+                            framework = 'svelte';
                         // Try to detect port from package.json scripts
                         let port;
                         const devScript = pkg.scripts?.dev || pkg.scripts?.start || '';
@@ -368,18 +391,101 @@ class ProjectScanner {
                 if (!app.framework) {
                     app.framework = appFramework.name;
                 }
-                // Apply default port from framework
-                if (!app.port && appFramework.defaultPort) {
-                    app.port = appFramework.defaultPort;
+            }
+            // Priority 1: Detect port from config files (vite.config.ts, etc.)
+            if (!app.port) {
+                const configPort = this.detectPortFromConfig(appPath);
+                if (configPort) {
+                    app.port = configPort;
+                }
+            }
+            // Priority 2: Use framework default from detector
+            if (!app.port && appFramework?.defaultPort) {
+                app.port = appFramework.defaultPort;
+            }
+            // Priority 3: Use hardcoded framework default as last resort
+            if (!app.port && app.framework) {
+                app.port = this.getDefaultPortForFramework(app.framework);
+            }
+        }
+    }
+    /**
+     * Detect port from config files (vite.config.ts, next.config.js, etc.)
+     */
+    detectPortFromConfig(appPath) {
+        const fs = require('fs');
+        const pathModule = require('path');
+        // Try vite.config.ts/js
+        for (const configFile of ['vite.config.ts', 'vite.config.js', 'vite.config.mts', 'vite.config.mjs']) {
+            const configPath = pathModule.join(appPath, configFile);
+            if (fs.existsSync(configPath)) {
+                try {
+                    const content = fs.readFileSync(configPath, 'utf8');
+                    // Match server.port or server: { port: XXXX }
+                    const portMatch = content.match(/server\s*:\s*\{[^}]*port\s*:\s*(\d+)/s) ||
+                        content.match(/port\s*:\s*(\d+)/);
+                    if (portMatch) {
+                        return parseInt(portMatch[1], 10);
+                    }
                 }
+                catch { }
             }
-            else if (app.framework) {
-                // App has framework name but we need to look up its default port
-                const matchingFramework = frameworks.find(f => f.name === app.framework);
-                if (matchingFramework?.defaultPort && !app.port) {
-                    app.port = matchingFramework.defaultPort;
+        }
+        // Try next.config.js/mjs (less common to have port there, but check anyway)
+        // Next.js port is usually in package.json scripts or CLI args
+        // Try nuxt.config.ts/js
+        for (const configFile of ['nuxt.config.ts', 'nuxt.config.js']) {
+            const configPath = pathModule.join(appPath, configFile);
+            if (fs.existsSync(configPath)) {
+                try {
+                    const content = fs.readFileSync(configPath, 'utf8');
+                    const portMatch = content.match(/devServer\s*:\s*\{[^}]*port\s*:\s*(\d+)/s) ||
+                        content.match(/port\s*:\s*(\d+)/);
+                    if (portMatch) {
+                        return parseInt(portMatch[1], 10);
+                    }
                 }
+                catch { }
+            }
+        }
+        return undefined;
+    }
+    /**
+     * Get default port for a framework (fallback if config file doesn't specify)
+     */
+    getDefaultPortForFramework(framework) {
+        const portMap = {
+            nextjs: 3000,
+            nuxt: 3000,
+            vite: 5173,
+            react: 3000,
+            vue: 5173,
+            angular: 4200,
+            nestjs: 3000,
+            express: 3000,
+        };
+        return portMap[framework];
+    }
+    /**
+     * Ensure all apps have unique ports.
+     * When multiple apps have the same port, increment subsequent ones.
+     */
+    ensureUniquePorts(apps) {
+        const usedPorts = new Set();
+        for (const app of apps) {
+            // Skip libraries and apps without ports
+            if (app.type === 'library' || !app.port)
+                continue;
+            let port = app.port;
+            // If port is already used, find the next available one
+            while (usedPorts.has(port)) {
+                port++;
+            }
+            // Update app port if it changed
+            if (port !== app.port) {
+                app.port = port;
             }
+            usedPorts.add(port);
         }
     }
     async detectGit(root) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "genbox",
-  "version": "1.0.96",
+  "version": "1.0.98",
   "description": "Genbox CLI - AI-Powered Development Environments",
   "main": "dist/index.js",
   "bin": {