feat: enhance admin dashboard startup process and improve log file watching reliability

Author: kaifcoder

bin/lib/admin.js

@@ -659,9 +659,14 @@ export async function startAdminDashboard(options = {}) {
 
     // Auto-open browser if requested
     if (options.open !== false) {
-      const openCmd = process.platform === 'darwin' ? 'open' :
-                     process.platform === 'win32' ? 'start' : 'xdg-open';
-      spawn(openCmd, [`http://localhost:${port}`], { detached: true, stdio: 'ignore' });
+      const url = `http://localhost:${port}`;
+      if (process.platform === 'win32') {
+        // Use cmd start to open default browser on Windows; needs shell invocation
+        spawn('cmd', ['/c', 'start', '', url], { detached: true, stdio: 'ignore' });
+      } else {
+        const openCmd = process.platform === 'darwin' ? 'open' : 'xdg-open';
+        spawn(openCmd, [url], { detached: true, stdio: 'ignore' });
+      }
     }
   });
 

bin/lib/dev.js

@@ -4,26 +4,17 @@ import chalk from 'chalk';
 import { spawn } from 'node:child_process';
 import http from 'http';
 import { initializeServiceLogs } from './logs.js';
+import { fileURLToPath } from 'url';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+const cliEntry = path.resolve(__dirname, '..', 'index.js');
 
 // Start admin dashboard
 function startAdminDashboard(cwd) {
   console.log(chalk.cyan('🎛️  Starting admin dashboard on http://localhost:9000'));
 
-  // Try to find create-polyglot command
-  let adminCmd = 'npx';
-  let adminArgs = ['create-polyglot', 'admin', '--port', '9000'];
-  
-  // Try global command first
-  try {
-    const { execSync } = require('child_process');
-    execSync('which create-polyglot', { stdio: 'ignore' });
-    adminCmd = 'create-polyglot';
-    adminArgs = ['admin', '--port', '9000'];
-  } catch (e) {
-    // Use npx as fallback
-  }
-  
-  const adminProcess = spawn(adminCmd, adminArgs, {
+  const adminProcess = spawn(process.execPath, [cliEntry, 'admin', '--port', '9000', '--no-open'], {
     cwd,
     env: process.env,
     stdio: 'pipe'

bin/lib/logs.js

@@ -612,7 +612,9 @@ export class LogFileWatcher {
       if (fs.existsSync(serviceDir)) {
         // Ensure .logs directory exists so watcher can subscribe immediately
         const logsDir = ensureLogsDir(serviceDir);
-        watchPaths.push(path.join(logsDir, '*.log'));
+        // Watch the directory itself, not glob pattern - more reliable
+        const watchPath = logsDir.replace(/\\/g, '/');
+        watchPaths.push(watchPath);
       }
     }
 
@@ -621,21 +623,35 @@ export class LogFileWatcher {
       return;
     }
 
+    console.log(chalk.cyan('[LogWatcher] Watch paths:'), watchPaths);
+
     // Initialize cache with current logs
     await this.loadInitialLogs(cfg.services);
 
-    // Start chokidar watcher
+    // Start chokidar watcher - watch directories, filter for .log files in handlers
     this.watcher = chokidar.watch(watchPaths, {
-      ignored: /(^|[\/\\])\../, // ignore dotfiles
+      ignored: ['**/node_modules/**', '**/.git/**', '**/.DS_Store'],
       persistent: true,
-      ignoreInitial: false
+      ignoreInitial: false,
+      usePolling: true, // Enable polling for more reliable file watching on Windows
+      interval: 1000,   // Poll every second
+      depth: 0          // Only watch files in the immediate directory, not subdirectories
     });
 
     this.watcher
-      .on('add', (filePath) => this.handleFileChange('add', filePath))
-      .on('change', (filePath) => this.handleFileChange('change', filePath))
-      .on('unlink', (filePath) => this.handleFileChange('unlink', filePath))
-      .on('error', (error) => console.error('Log watcher error:', error));
+      .on('add', (filePath) => {
+        if (!filePath.endsWith('.log')) return; // Only process .log files
+        this.handleFileChange('add', filePath);
+      })
+      .on('change', (filePath) => {
+        if (!filePath.endsWith('.log')) return; // Only process .log files
+        this.handleFileChange('change', filePath);
+      })
+      .on('unlink', (filePath) => {
+        if (!filePath.endsWith('.log')) return; // Only process .log files
+        this.handleFileChange('unlink', filePath);
+      })
+      .on('error', (error) => console.error(chalk.red('Log watcher error:'), error));
 
     this.isWatching = true;
     console.log(chalk.green('📁 Started watching log files with chokidar'));
@@ -678,7 +694,9 @@ export class LogFileWatcher {
   async handleFileChange(event, filePath) {
     try {
       const serviceName = this.getServiceNameFromLogPath(filePath);
-      if (!serviceName) return;
+      if (!serviceName) {
+        return;
+      }
 
       if (event === 'unlink') {
         // File deleted - clear cache for this service

scripts/dev-basic.cjs

@@ -1,4 +1,4 @@
-const { spawn } = require('node:child_process');
+const { spawn, spawnSync } = require('node:child_process');
 const fs = require('fs');
 const path = require('path');
 
@@ -81,28 +81,60 @@ export class Logger {
 }
 
 // Start admin dashboard
-function startAdminDashboard() {
-  console.log('🎛️  Starting admin dashboard...');
-  
-  // Try to find create-polyglot globally or in node_modules
-  let adminCmd = 'npx';
-  let adminArgs = ['create-polyglot', 'admin', '--port', '9000'];
-  
-  // Fallback: check if create-polyglot is available globally
+function resolveCliEntry() {
   try {
-    const { execSync } = require('child_process');
-    execSync('which create-polyglot', { stdio: 'ignore' });
-    adminCmd = 'create-polyglot';
-    adminArgs = ['admin', '--port', '9000'];
+    return require.resolve('create-polyglot/bin/index.js', { paths: [root] });
   } catch (e) {
-    // Use npx as fallback
+    // ignore resolution failure
+  }
+
+  const nodeModulesCli = path.join(root, 'node_modules', 'create-polyglot', 'bin', 'index.js');
+  if (fs.existsSync(nodeModulesCli)) {
+    return nodeModulesCli;
+  }
+
+  const repoCli = path.resolve(__dirname, '../bin/index.js');
+  if (fs.existsSync(repoCli)) {
+    return repoCli;
   }
+
+  return null;
+}
+
+function startAdminDashboard() {
+  console.log('🎛️  Starting admin dashboard...');
 
-  const adminProcess = spawn(adminCmd, adminArgs, {
+  const spawnOptions = {
     cwd: root,
     env: process.env,
-    stdio: 'pipe'
-  });
+    stdio: 'pipe',
+    shell: process.platform === 'win32'
+  };
+
+  const localCli = resolveCliEntry();
+  let adminCmd;
+  let adminArgs;
+
+  if (localCli) {
+    adminCmd = process.execPath;
+    adminArgs = [localCli, 'admin', '--port', '9000', '--no-open'];
+    spawnOptions.shell = false;
+  } else {
+    const check = spawnSync('create-polyglot', ['--version'], {
+      stdio: 'ignore',
+      shell: spawnOptions.shell
+    });
+
+    if (check.status === 0 && !check.error) {
+      adminCmd = 'create-polyglot';
+      adminArgs = ['admin', '--port', '9000', '--no-open'];
+    } else {
+      adminCmd = 'npx';
+      adminArgs = ['create-polyglot', 'admin', '--port', '9000', '--no-open'];
+    }
+  }
+
+  const adminProcess = spawn(adminCmd, adminArgs, spawnOptions);
 
   adminProcess.stdout.on('data', d => process.stdout.write(`[admin] ${d}`));
   adminProcess.stderr.on('data', d => process.stderr.write(`[admin] ${d}`));