ninja-terminals 2.4.5 → 2.4.6

package/cli.js

@@ -225,6 +225,14 @@ process.env.HTTP_PORT          = String(port);
 process.env.DEFAULT_TERMINALS  = String(terminals);
 process.env.DEFAULT_CWD        = cwd;
 
+// Single browser-open path: server.js opens the tab in its listen callback
+// (knows the actual selected port, uses 127.0.0.1). Enable it by default for
+// CLI launches; opt out with --no-open. This replaces cli.js's own opener,
+// which previously fired a second, redundant tab.
+if (!hasFlag('--no-open') && !process.env.NINJA_OPEN_BROWSER) {
+  process.env.NINJA_OPEN_BROWSER = '1';
+}
+
 // Auth env vars
 if (token) {
   process.env.NINJA_AUTH_TOKEN = token;
@@ -233,41 +241,9 @@ if (offline) {
   process.env.NINJA_OFFLINE = '1';
 }
 
-// ── Auto-open browser ────────────────────────────────────────
-
-function openBrowser(url) {
-  const { spawn } = require('child_process');
-  const platform = process.platform;
-  let cmd, cmdArgs;
-
-  if (platform === 'darwin') {
-    cmd = 'open';
-    cmdArgs = [url];
-  } else if (platform === 'win32') {
-    cmd = 'cmd';
-    cmdArgs = ['/c', 'start', url];
-  } else {
-    cmd = 'xdg-open';
-    cmdArgs = [url];
-  }
-
-  spawn(cmd, cmdArgs, { stdio: 'ignore', detached: true }).unref();
-}
-
-// Delay browser open until after the server listen callback fires.
-// server.js calls server.listen() synchronously on require, so we
-// schedule the open after the current tick stack clears. Read the actual
-// selected port from the session file — the server may have rolled to a
-// different port if the preferred one was occupied (on either IP stack).
-setTimeout(() => {
-  let openPort = port;
-  try {
-    const sessionPath = require('path').join(require('os').homedir(), '.ninja', 'session.json');
-    const s = JSON.parse(require('fs').readFileSync(sessionPath, 'utf8'));
-    if (s && s.port) openPort = s.port;
-  } catch { /* fall back to requested port */ }
-  openBrowser(`http://127.0.0.1:${openPort}`);
-}, 1500);
+// Browser auto-open is handled by server.js's listen callback (single path,
+// gated by NINJA_OPEN_BROWSER which we set above). cli.js no longer opens its
+// own tab — that produced a duplicate.
 
 // ── Start the server ─────────────────────────────────────────
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ninja-terminals",
-  "version": "2.4.5",
+  "version": "2.4.6",
   "description": "MCP server for multi-terminal Claude Code orchestration with DAG task management, parallel execution, and self-improvement",
   "main": "server.js",
   "bin": {