novaprime 1.5.1 → 1.6.0

package/bin/novaprime.js

@@ -5,6 +5,7 @@ const ui = require('../src/ui');
 const { c } = ui;
 const { ask, close, boxInput } = require('../src/prompt');
 const { runTurn, fetchMe } = require('../src/agent');
+const tools = require('../src/tools');
 const pkg = require('../package.json');
 
 // Validate the key against the server before saving — a wrong key never logs in.
@@ -82,7 +83,8 @@ async function repl() {
   let me = await fetchMe(config.getServer(), cfg.key);
   console.clear(); // hide login/clutter — start clean with the header at the top
   ui.banner(meToBanner(cfg, me));
-  process.on('SIGINT', () => { console.log(c.muted('\n  bye')); process.exit(0); });
+  process.on('exit', () => { try { tools.stopBackground(); } catch (_) {} }); // stop bg dev servers on quit
+  process.on('SIGINT', () => { try { tools.stopBackground(); } catch (_) {} console.log(c.muted('\n  bye')); process.exit(0); });
 
   ui.hint('  Tip: press ' + c.indigo('Shift+Tab') + c.dim(' for auto-accept (no permission prompts), or answer ') + c.indigo('a') + c.dim(' at any prompt.'));
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "novaprime",
-  "version": "1.5.1",
+  "version": "1.6.0",
   "description": "NovaPrime — an AI coding assistant in your terminal, powered by GLM.",
   "bin": {
     "novaprime": "bin/novaprime.js"

package/src/agent.js

@@ -17,9 +17,9 @@ const SYSTEM_PROMPT =
   `Use clear markdown: short paragraphs, bullet lists, and fenced code blocks with a language tag. ` +
   `When asked to create or change files, call the tool RIGHT AWAY with at most a one-line intro — ` +
   `do NOT write long explanations or feature lists before creating the file. Keep any summary to 1-2 short lines after. ` +
-  `NEVER start dev servers or long-running/blocking processes (e.g. "npm run dev", "npm start", "vite", "next dev", ` +
-  `"nodemon", watchers) — they block the session forever. After creating the files, finish, then tell the user the ` +
-  `commands to run it themselves (e.g. cd into the folder and run "npm run dev", then open the localhost URL). ` +
+  `You MAY start the dev server when the project is ready — run it with run_command (e.g. "cd <project> && npm run dev"). ` +
+  `Dev/watch servers are automatically run in the BACKGROUND and the tool returns the localhost URL, so they do NOT block. ` +
+  `After it starts, give the user the localhost URL to open in their browser, and do NOT start it a second time. ` +
   `Be concise and warm. Current OS: ${os.platform()}. Working directory: ${process.cwd()}.`;
 
 // Fetch read-only account info for the header (name, plan, usage). Never throws.

package/src/tools.js

@@ -1,13 +1,67 @@
 'use strict';
 const fs = require('fs');
+const os = require('os');
 const path = require('path');
-const { spawnSync } = require('child_process');
+const { spawnSync, spawn } = require('child_process');
 const { ask } = require('./prompt');
 const { c, tool } = require('./ui');
 const mode = require('./mode');
 
 const MAX_OUTPUT = 20000; // cap tool output sent back to the model
 
+// dev/watch servers: long-running, so we run them DETACHED in the background and
+// hand back the localhost URL instead of blocking the session.
+const DEV_SERVER_RE = /\b(npm|pnpm|yarn|bun)\s+(run\s+)?(dev|start|serve|watch)\b|\bvite\b(?!\s+build)|\bnext\s+dev\b|\bnodemon\b|webpack-dev-server|webpack\s+serve|\bnpx\s+serve\b|\bhttp-server\b|\blive-server\b|php\s+-S|python\s+-m\s+http\.server|flask\s+run|rails\s+s(erver)?\b|\bng\s+serve\b/i;
+const URL_RE = /https?:\/\/(?:localhost|127\.0\.0\.1)(?::\d+)?\/?\S*/i;
+const bgServers = []; // { pid, cmd, child } — kept so they can be stopped on exit
+
+// Start a long-running command (dev server) in the background. We DON'T await it —
+// we stream its output to a log, grab the localhost URL, and return right away so the
+// session never blocks. It keeps running as a child of the CLI for the whole session.
+async function runBackground(cmd) {
+  const logPath = path.join(os.tmpdir(), 'novaprime-dev-' + Date.now() + '.log');
+  let log = '';
+  let child;
+  try {
+    child = spawn(cmd, { shell: true, windowsHide: true });
+  } catch (err) {
+    return 'ERROR starting dev server: ' + err.message;
+  }
+  bgServers.push({ pid: child.pid, cmd, child });
+  const onData = (d) => { const s = d.toString(); log += s; if (log.length > MAX_OUTPUT) log = log.slice(-MAX_OUTPUT); try { fs.appendFileSync(logPath, s); } catch (_) {} };
+  if (child.stdout) child.stdout.on('data', onData);
+  if (child.stderr) child.stderr.on('data', onData);
+  let earlyExit = null;
+  child.on('error', (e) => { log += '\n[spawn error] ' + e.message; });
+  child.on('exit', (code) => { earlyExit = code; });
+
+  const url = await new Promise((resolve) => {
+    const start = Date.now();
+    const t = setInterval(() => {
+      const m = log.match(URL_RE);
+      if (m) { clearInterval(t); resolve(m[0]); }
+      else if (earlyExit !== null) { clearInterval(t); resolve(null); }   // it crashed on startup
+      else if (Date.now() - start >= 10000) { clearInterval(t); resolve(null); }
+    }, 250);
+  });
+
+  if (earlyExit !== null && !url) {
+    console.log(c.amber('  ! dev server exited (code ' + earlyExit + ')'));
+    return 'The dev command "' + cmd + '" exited immediately (code ' + earlyExit + ') instead of staying up. It likely failed. Read the log and fix the problem, then start it again.\n\nOutput:\n' + clip(log);
+  }
+  console.log(c.green('  ✓ dev server running in background') + c.dim('  (pid ' + child.pid + ')'));
+  if (url) console.log(c.green('  → ') + c.white(url) + c.dim('  — open this in your browser'));
+  else console.log(c.dim('  · still starting… log: ' + logPath));
+  return 'OK: started "' + cmd + '" in the BACKGROUND (pid ' + child.pid + '), so the session is NOT blocked and the server keeps running. ' +
+    (url ? 'It is serving at ' + url + ' — tell the user to open that URL in their browser. ' : 'It is still booting; tell the user to watch for the localhost URL. ') +
+    'Do NOT start it again.\n\nStartup log:\n' + clip(log);
+}
+
+// Stop any background dev servers (called when the CLI exits).
+function stopBackground() {
+  for (const s of bgServers) { try { s.child.kill(); } catch (_) {} }
+}
+
 // Permission gate. Returns true if allowed. In auto-accept mode it never asks.
 // Answering "a" (always) flips auto-accept ON for the rest of the session.
 async function allow(label) {
@@ -125,18 +179,13 @@ async function execute(name, input) {
       }
       case 'run_command': {
         const cmd = String(input.command || '');
-        // Refuse blocking dev servers / watchers — they never exit and freeze the session.
-        if (/\b(npm|pnpm|yarn|bun)\s+(run\s+)?(dev|start|serve|watch)\b|\bvite\b(?!\s+build)|\bnext\s+dev\b|\bnodemon\b|webpack-dev-server|webpack\s+serve|\bnpx\s+serve\b|\bhttp-server\b|\blive-server\b|php\s+-S|python\s+-m\s+http\.server|flask\s+run|rails\s+s(erver)?\b|\bng\s+serve\b/i.test(cmd)) {
-          console.log('');
-          console.log(c.amber('  ! ') + c.amber('skipped dev server: ') + c.white(cmd));
-          console.log(c.dim('    (long-running — start it yourself in another terminal when ready)'));
-          return 'SKIPPED: "' + cmd + '" is a long-running dev/watch server and was NOT run (it would block the session forever). Do NOT start dev servers. Finish the work, then tell the user how to run it themselves in a separate terminal, e.g. `cd <project>` then `npm run dev`, and to open the localhost URL it prints.';
-        }
+        const isDevServer = DEV_SERVER_RE.test(cmd);
         console.log('');
         console.log(c.red('  ╭─ permission · run command ') + c.dim('───────────────────'));
-        console.log(c.red('  │ ') + c.bold(input.command));
+        console.log(c.red('  │ ') + c.bold(cmd) + (isDevServer ? c.dim('   (runs in background)') : ''));
         if (!(await allow(c.red('  ╰─ run this command?')))) { console.log(c.dim('  · skipped')); return 'DENIED: user did not allow running the command.'; }
-        const r = spawnSync(input.command, { shell: true, encoding: 'utf8', timeout: 1000 * 120 });
+        if (isDevServer) return await runBackground(cmd); // long-running: detach so it doesn't block
+        const r = spawnSync(cmd, { shell: true, encoding: 'utf8', timeout: 1000 * 120 });
         const out = (r.stdout || '') + (r.stderr || '');
         return clip(`exit_code=${r.status}\n${out}`.trim());
       }
@@ -148,4 +197,4 @@ async function execute(name, input) {
   }
 }
 
-module.exports = { definitions, execute };
+module.exports = { definitions, execute, runBackground, stopBackground };