neex 0.1.6 → 0.1.7

package/README.md

@@ -6,7 +6,7 @@
 </picture>
 </a>
 
-# Neex v0.1.6
+# Neex v0.1.7
 
 ### 🚀 Neex: The Modern Build System for Polyrepo-in-Monorepo Architecture
 

package/dist/src/cli.js

@@ -197,11 +197,11 @@ function cli() {
             process.exit(1);
         }
     });
-    // Watch command (Nodemon functionality)
+    // Dev command (Nodemon functionality - formerly watch)
     program
-        .command('watch [commands...]') // Made commands optional
-        .alias('w')
-        .description('Run commands with file watching (nodemon functionality)')
+        .command('dev [commands...]') // Made commands optional
+        .alias('d')
+        .description('Run commands with file watching and auto-restart (nodemon functionality)')
         .option('-c, --no-color', 'Disable colored output')
         .option('-t, --no-timing', 'Hide timing information')
         .option('-p, --no-prefix', 'Hide command prefix')
@@ -222,11 +222,11 @@ function cli() {
                 const foundCommand = await findDefaultCommand();
                 if (foundCommand) {
                     effectiveCommands = [foundCommand];
-                    console.log(chalk_1.default.blue(`${figures_1.default.info} No command specified for 'neex w', using default: "${foundCommand}"`));
+                    console.log(chalk_1.default.blue(`${figures_1.default.info} No command specified for 'neex dev', using default: "${foundCommand}"`));
                 }
                 else {
-                    console.error(chalk_1.default.red(`${figures_1.default.cross} No command specified for 'neex w' and no default script (dev, start) or main file found in package.json.`));
-                    console.error(chalk_1.default.yellow(`${figures_1.default.pointer} Please specify a command to run (e.g., neex w "npm run dev") or define a "dev" or "start" script in your package.json.`));
+                    console.error(chalk_1.default.red(`${figures_1.default.cross} No command specified for 'neex dev' and no default script (dev, start) or main file found in package.json.`));
+                    console.error(chalk_1.default.yellow(`${figures_1.default.pointer} Please specify a command to run (e.g., neex dev "npm run dev") or define a "dev" or "start" script in your package.json.`));
                     process.exit(1);
                 }
             }
@@ -268,7 +268,7 @@ function cli() {
                 // If isLikelyCommandOrScript is true, or if the file auto-detection didn't apply,
                 // effectiveCommands remains as the user provided it (joined later by DevRunner).
             }
-            console.log(chalk_1.default.blue(`${figures_1.default.info} Starting development server with file watching (neex w) for command(s): ${effectiveCommands.map(cmd => `"${cmd}"`).join(' && ')}...`));
+            console.log(chalk_1.default.blue(`${figures_1.default.info} Starting development server with file watching (neex dev) for command(s): ${effectiveCommands.map(cmd => `"${cmd}"`).join(' && ')}...`));
             const watchPaths = options.watch || ['./'];
             const ignorePatterns = options.ignore || [
                 'node_modules/**', '.git/**', '*.log', 'dist/**', 'build/**',
@@ -276,7 +276,7 @@ function cli() {
             ];
             const extensions = options.ext || ['js', 'mjs', 'json', 'ts', 'tsx', 'jsx'];
             devRunner = new dev_runner_js_1.DevRunner({
-                runnerName: 'neex w',
+                runnerName: 'neex dev',
                 parallel: false,
                 color: options.color,
                 showTiming: options.timing,
@@ -299,10 +299,10 @@ function cli() {
         }
         catch (error) {
             if (error instanceof Error) {
-                console.error(chalk_1.default.red(`${figures_1.default.cross} Watch Error: ${error.message}`));
+                console.error(chalk_1.default.red(`${figures_1.default.cross} Dev Error: ${error.message}`));
             }
             else {
-                console.error(chalk_1.default.red(`${figures_1.default.cross} An unknown watch error occurred`));
+                console.error(chalk_1.default.red(`${figures_1.default.cross} An unknown dev error occurred`));
             }
             process.exit(1);
         }
@@ -492,7 +492,7 @@ function cli() {
         .command('logs [id]')
         .description('Show process logs')
         .option('-f, --follow', 'Follow log output')
-        .option('-n, --lines <number>', 'Number of lines to show', parseInt)
+        .option('-n, --lines <number>', 'Number of lines to show', (value) => parseInt(value, 10), 15)
         .action(async (id, options) => {
         try {
             if (!processManager) {
@@ -500,7 +500,7 @@ function cli() {
                 await processManager.load();
             }
             if (id) {
-                const logs = await processManager.logs(id, options.lines || 100);
+                const logs = await processManager.logs(id, options.lines);
                 if (logs.length === 0) {
                     console.log(chalk_1.default.yellow(`${figures_1.default.info} No logs found for process ${id}`));
                 }
@@ -680,7 +680,7 @@ WantedBy=multi-user.target
         program.help();
     }
     // Graceful shutdown handling
-    const handleSignal = (signal) => {
+    const handleSignal = async (signal) => {
         console.log(`\n${chalk_1.default.yellow(`${figures_1.default.warning} Received ${signal}. Cleaning up...`)}`);
         if (devRunner && devRunner.isActive()) {
             devRunner.stop();
@@ -689,13 +689,13 @@ WantedBy=multi-user.target
             cleanupRunner();
         }
         if (processManager) {
-            processManager.dispose();
+            await processManager.dispose();
         }
         setTimeout(() => process.exit(0), 500);
     };
-    process.on('SIGINT', () => handleSignal('SIGINT'));
-    process.on('SIGTERM', () => handleSignal('SIGTERM'));
-    process.on('SIGQUIT', () => handleSignal('SIGQUIT'));
+    process.on('SIGINT', () => handleSignal('SIGINT').catch(err => console.error('SIGINT handler error:', err)));
+    process.on('SIGTERM', () => handleSignal('SIGTERM').catch(err => console.error('SIGTERM handler error:', err)));
+    process.on('SIGQUIT', () => handleSignal('SIGQUIT').catch(err => console.error('SIGQUIT handler error:', err)));
 }
 exports.default = cli;
 // Helper function to format uptime

package/dist/src/process-manager.js

@@ -29,26 +29,32 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.ProcessManager = void 0;
 // src/process-manager.ts - PM2 alternative process manager
 const child_process_1 = require("child_process");
-const fs = __importStar(require("fs/promises"));
+const fsFull = __importStar(require("fs")); // Renamed to fsFull to avoid conflicts
+const fs_1 = require("fs");
 const path = __importStar(require("path"));
 const events_1 = require("events");
 const logger_1 = __importDefault(require("./logger"));
 class ProcessManager extends events_1.EventEmitter {
     constructor(configPath) {
         super();
+        this.logStreams = new Map();
         this.processes = new Map();
         this.isMonitoring = false;
         this.configPath = configPath || path.join(process.cwd(), '.neex', 'processes.json');
-        this.ensureConfigDir();
+        this.logDir = path.join(path.dirname(this.configPath), 'logs');
+        this.ensureConfigDirAndLogDir(); // Renamed and updated
+        // loadConfig will be called by CLI commands as needed before operations like list, stop, etc.
         this.startMonitoring();
     }
-    async ensureConfigDir() {
+    async ensureConfigDirAndLogDir() {
         const dir = path.dirname(this.configPath);
         try {
-            await fs.mkdir(dir, { recursive: true });
+            await fs_1.promises.mkdir(dir, { recursive: true });
+            await fs_1.promises.mkdir(this.logDir, { recursive: true });
         }
         catch (error) {
-            // Directory might already exist
+            logger_1.default.printLine(`Failed to create config/log directories: ${error.message}`, 'error');
+            // Directory might already exist or other error
         }
     }
     async saveConfig() {
@@ -59,7 +65,7 @@ class ProcessManager extends events_1.EventEmitter {
                 created_at: proc.created_at,
                 restarts: proc.restarts
             }));
-            await fs.writeFile(this.configPath, JSON.stringify(config, null, 2));
+            await fs_1.promises.writeFile(this.configPath, JSON.stringify(config, null, 2));
         }
         catch (error) {
             logger_1.default.printLine(`Failed to save config: ${error.message}`, 'error');
@@ -67,7 +73,7 @@ class ProcessManager extends events_1.EventEmitter {
     }
     async loadConfig() {
         try {
-            const data = await fs.readFile(this.configPath, 'utf-8');
+            const data = await fs_1.promises.readFile(this.configPath, 'utf-8');
             const configs = JSON.parse(data);
             for (const config of configs) {
                 if (config.id && !this.processes.has(config.id)) {
@@ -99,8 +105,39 @@ class ProcessManager extends events_1.EventEmitter {
         }
         return id;
     }
+    closeLogStreams(id) {
+        var _a, _b;
+        const streams = this.logStreams.get(id);
+        if (streams) {
+            (_a = streams.out) === null || _a === void 0 ? void 0 : _a.end();
+            (_b = streams.err) === null || _b === void 0 ? void 0 : _b.end();
+            this.logStreams.delete(id);
+        }
+    }
     async startProcess(processInfo) {
         const { config } = processInfo;
+        // Setup log files
+        const defaultLogPath = (type) => path.join(this.logDir, `${processInfo.id}.${type}.log`);
+        config.out_file = config.out_file || defaultLogPath('out');
+        config.error_file = config.error_file || defaultLogPath('err');
+        this.closeLogStreams(processInfo.id); // Ensure any previous streams for this ID are closed (e.g., on restart)
+        let outStream;
+        let errStream;
+        try {
+            if (config.out_file) {
+                await fs_1.promises.mkdir(path.dirname(config.out_file), { recursive: true }); // Ensure directory for custom log file exists
+                outStream = fsFull.createWriteStream(config.out_file, { flags: 'a' });
+                this.logStreams.set(processInfo.id, { ...this.logStreams.get(processInfo.id), out: outStream });
+            }
+            if (config.error_file) {
+                await fs_1.promises.mkdir(path.dirname(config.error_file), { recursive: true }); // Ensure directory for custom log file exists
+                errStream = fsFull.createWriteStream(config.error_file, { flags: 'a' });
+                this.logStreams.set(processInfo.id, { ...this.logStreams.get(processInfo.id), err: errStream });
+            }
+        }
+        catch (streamError) {
+            logger_1.default.printLine(`Failed to open log streams for ${processInfo.id}: ${streamError.message}`, 'error');
+        }
         try {
             processInfo.status = 'launching';
             this.emit('process:launching', processInfo);
@@ -121,7 +158,18 @@ class ProcessManager extends events_1.EventEmitter {
             processInfo.pid = childProcess.pid;
             processInfo.status = 'online';
             processInfo.created_at = new Date();
+            // Allow parent to exit independently of the child
+            if (childProcess.pid) { // unref only if process spawned successfully
+                childProcess.unref();
+            }
             this.emit('process:start', processInfo);
+            // Pipe to log files if streams are available
+            if (outStream && childProcess.stdout) {
+                childProcess.stdout.pipe(outStream);
+            }
+            if (errStream && childProcess.stderr) {
+                childProcess.stderr.pipe(errStream);
+            }
             // Handle stdout
             if (childProcess.stdout) {
                 childProcess.stdout.on('data', (data) => {
@@ -147,6 +195,8 @@ class ProcessManager extends events_1.EventEmitter {
                 processInfo.status = code === 0 ? 'stopped' : 'errored';
                 processInfo.process = undefined;
                 processInfo.pid = undefined;
+                // Note: Streams are not closed here on 'exit' to allow logs to be read after a crash.
+                // They are closed on explicit stop/delete or neex shutdown.
                 this.emit('process:exit', { processInfo, code, signal });
                 // Auto-restart if enabled
                 if (config.autorestart && processInfo.restarts < (config.max_restarts || 10)) {
@@ -252,6 +302,7 @@ class ProcessManager extends events_1.EventEmitter {
         processInfo.status = 'stopped';
         processInfo.process = undefined;
         processInfo.pid = undefined;
+        this.closeLogStreams(id);
         await this.saveConfig();
         this.emit('process:stop', processInfo);
     }
@@ -278,9 +329,22 @@ class ProcessManager extends events_1.EventEmitter {
         if (processInfo.status === 'online') {
             await this.stop(id);
         }
+        const currentProcessInfo = this.processes.get(id);
+        if (currentProcessInfo) {
+            if (currentProcessInfo.status === 'online' || currentProcessInfo.status === 'launching' || currentProcessInfo.status === 'stopping') {
+                // stop() will call closeLogStreams
+            }
+            else {
+                this.closeLogStreams(id); // Ensure closed for other states before deleting
+            }
+        }
+        // Optional: Delete actual log files if `delete` implies full data removal
+        // if (config.out_file) await fsPromises.unlink(config.out_file).catch(() => {});
+        // if (config.error_file) await fsPromises.unlink(config.error_file).catch(() => {});
         this.processes.delete(id);
         await this.saveConfig();
-        this.emit('process:delete', processInfo);
+        if (currentProcessInfo)
+            this.emit('process:delete', currentProcessInfo);
     }
     async list() {
         return Array.from(this.processes.values());
@@ -295,10 +359,42 @@ class ProcessManager extends events_1.EventEmitter {
         }
         return this.list();
     }
-    async logs(id, lines = 100) {
-        // TODO: Implement log file reading
-        // For now, return empty array
-        return [];
+    async logs(id, lines = 15) {
+        let processInfo = this.processes.get(id);
+        if (!processInfo) {
+            await this.loadConfig(); // Attempt to load if not in memory
+            processInfo = this.processes.get(id);
+            if (!processInfo) {
+                throw new Error(`Process ${id} not found`);
+            }
+        }
+        return this.readLogFiles(processInfo.config, lines);
+    }
+    async readLogFiles(config, linesToRead) {
+        const finalMessages = [];
+        const { out_file, error_file } = config;
+        const readFileLastLines = async (filePath) => {
+            if (!filePath)
+                return [];
+            try {
+                await fs_1.promises.access(filePath);
+                const content = await fs_1.promises.readFile(filePath, 'utf-8');
+                const fileLines = content.split('\n').filter((line) => line.trim() !== '');
+                return fileLines.slice(-linesToRead);
+            }
+            catch (error) {
+                // Log file might not exist or not be readable, return empty
+                return [];
+            }
+        };
+        const outLogs = await readFileLastLines(out_file);
+        const errLogs = await readFileLastLines(error_file);
+        outLogs.forEach(log => finalMessages.push(log));
+        if (errLogs.length > 0 && outLogs.length > 0) {
+            finalMessages.push('--- STDERR ---'); // Separator
+        }
+        errLogs.forEach(log => finalMessages.push(log));
+        return finalMessages;
     }
     async save() {
         await this.saveConfig();
@@ -322,7 +418,7 @@ class ProcessManager extends events_1.EventEmitter {
     getProcess(id) {
         return this.processes.get(id);
     }
-    dispose() {
+    async dispose() {
         this.stopMonitoring();
         this.stopAll();
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "neex",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "description": "The Modern Build System for Polyrepo-in-Monorepo Architecture",
   "main": "dist/src/index.js",
   "types": "dist/src/index.d.ts",