ninja-terminals 2.1.0 → 2.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ninja-terminals",
-  "version": "2.1.0",
+  "version": "2.1.2",
   "description": "Multi-terminal Claude Code orchestrator with DAG task management, permission hooks, and resilience",
   "main": "server.js",
   "bin": {

package/public/app.js

@@ -339,11 +339,23 @@ function createTerminalUI(termData) {
   progressFill.style.width = `${progress || 0}%`;
   progressTrack.appendChild(progressFill);
 
+  // Close button (X)
+  const closeBtn = document.createElement('button');
+  closeBtn.className = 'close-btn';
+  closeBtn.innerHTML = '×';
+  closeBtn.title = 'Close terminal';
+  closeBtn.addEventListener('mousedown', (e) => {
+    e.preventDefault();
+    e.stopPropagation();
+    closeTerminal(id);
+  });
+
   header.appendChild(labelEl);
   header.appendChild(stateEl);
   header.appendChild(elapsedEl);
   header.appendChild(spacer);
   header.appendChild(actionsEl);
+  header.appendChild(closeBtn);
   header.appendChild(progressTrack);
 
   // Double-click header to maximize/restore
@@ -1010,6 +1022,13 @@ async function startApp() {
   setupSidebar();
   setupAddTask();
   setupLearnings();
+  setupAddTerminal();
+
+  // Clear all button
+  const clearBtn = document.getElementById('clear-all-btn');
+  if (clearBtn) {
+    clearBtn.addEventListener('click', clearAllTerminals);
+  }
 
   // Load existing terminals
   try {
@@ -1047,6 +1066,53 @@ async function startApp() {
 
 // ── Learnings Module ───────────────────────────────────────────
 
+function setupAddTerminal() {
+  const btn = document.getElementById('add-terminal-btn');
+  if (!btn) return;
+
+  // Store last used directory
+  let lastCwd = localStorage.getItem('ninja-last-cwd') || '/Users/davidmini/Desktop/Projects';
+
+  btn.addEventListener('click', async () => {
+    try {
+      const cwd = prompt('Working directory:', lastCwd);
+      if (!cwd) return;
+
+      lastCwd = cwd;
+      localStorage.setItem('ninja-last-cwd', cwd);
+
+      const res = await fetch(`${API_BASE}/api/terminals`, {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json', ...auth.getAuthHeader() },
+        body: JSON.stringify({ cwd }),
+      });
+
+      if (!res.ok) {
+        const err = await res.json();
+        alert(`Failed to create terminal: ${err.error || 'Unknown error'}`);
+        return;
+      }
+
+      const terminal = await res.json();
+      createTerminalUI(terminal);
+      addFeedEntry(`Terminal added: T${terminal.id}`);
+    } catch (err) {
+      console.error('Failed to add terminal:', err);
+      alert('Failed to add terminal');
+    }
+  });
+}
+
+async function clearAllTerminals() {
+  if (!confirm('Close all terminals and start fresh?')) return;
+
+  const ids = Array.from(state.terminals.keys());
+  for (const id of ids) {
+    await closeTerminal(id);
+  }
+  addFeedEntry('All terminals cleared');
+}
+
 function setupLearnings() {
   const btn = document.getElementById('learnings-btn');
   const overlay = document.getElementById('learnings-overlay');

package/public/index.html

@@ -60,6 +60,8 @@
         <div class="logo-card">
           <span class="logo">NINJA TERMINALS</span>
           <span class="logo-sub">Multi-Agent Orchestrator</span>
+          <button id="add-terminal-btn" class="add-terminal-btn" title="Add Terminal">+</button>
+          <button id="clear-all-btn" class="clear-all-btn" title="Clear All Terminals">🗑</button>
           <button id="learnings-btn" class="learnings-btn" title="View Session Learnings">🧠</button>
           <button id="logout-btn" class="logout-btn" title="Sign out">Logout</button>
         </div>

package/public/style.css

@@ -976,3 +976,60 @@ main {
 .learnings-btn:hover {
   background: var(--border);
 }
+
+/* ── Terminal Close Button ─────────────────────── */
+
+.close-btn {
+  background: none;
+  border: none;
+  color: var(--text-muted);
+  font-size: 20px;
+  line-height: 1;
+  padding: 2px 8px;
+  cursor: pointer;
+  opacity: 0.5;
+  transition: opacity 0.15s, color 0.15s;
+  margin-left: 8px;
+}
+
+.close-btn:hover {
+  opacity: 1;
+  color: #ff6b6b;
+}
+
+/* ── Add Terminal Button ─────────────────────────── */
+
+.add-terminal-btn {
+  background: var(--feed-t1);
+  border: none;
+  color: var(--bg);
+  font-size: 20px;
+  font-weight: bold;
+  line-height: 1;
+  padding: 4px 10px;
+  border-radius: 4px;
+  cursor: pointer;
+  transition: background 0.15s, transform 0.15s;
+}
+
+.add-terminal-btn:hover {
+  background: var(--feed-t2);
+  transform: scale(1.1);
+}
+
+/* ── Clear All Button ─────────────────────────────── */
+
+.clear-all-btn {
+  background: none;
+  border: none;
+  font-size: 16px;
+  padding: 4px 8px;
+  cursor: pointer;
+  opacity: 0.6;
+  transition: opacity 0.15s, transform 0.15s;
+}
+
+.clear-all-btn:hover {
+  opacity: 1;
+  transform: scale(1.1);
+}

package/server.js

@@ -346,14 +346,30 @@ app.get('/health', (_req, res) => {
 
 // ── Session Endpoints ───────────────────────────────────────
 
-// Create session — validates token and spawns terminals
+// Create session — validates token and returns existing or spawns terminals
 app.post('/api/session', requireAuth, (req, res) => {
   try {
     const { tier, terminalsMax, features, token } = req.ninjaUser;
 
-    // Clear any existing session
+    // If session already exists with same token, return existing terminals
+    if (activeSession && activeSession.token === token) {
+      const existingTerminals = activeSession.terminalIds
+        .map(id => terminals.get(id))
+        .filter(Boolean)
+        .map(t => ({ id: t.id, label: t.label, status: t.status, cwd: t.cwd }));
+
+      console.log(`[session] Returning existing session: tier=${tier}, terminals=${existingTerminals.length}`);
+
+      return res.json({
+        tier,
+        terminalsMax,
+        features,
+        terminals: existingTerminals,
+      });
+    }
+
+    // Clear any existing session with different token
     if (activeSession) {
-      // Kill existing terminals
       for (const id of activeSession.terminalIds) {
         const terminal = terminals.get(id);
         if (terminal) {
@@ -364,7 +380,7 @@ app.post('/api/session', requireAuth, (req, res) => {
       }
     }
 
-    // Create new session
+    // Create new session (but don't auto-spawn terminals - let user add them)
     activeSession = {
       token,
       tier,
@@ -374,28 +390,14 @@ app.post('/api/session', requireAuth, (req, res) => {
       createdAt: Date.now(),
     };
 
-    // Spawn terminals up to the tier limit
-    const cwd = req.body?.cwd || DEFAULT_CWD;
-    const spawnedTerminals = [];
-
-    for (let i = 0; i < terminalsMax; i++) {
-      const terminal = spawnTerminal(`T${i + 1}`, [], cwd, tier);
-      activeSession.terminalIds.push(terminal.id);
-      spawnedTerminals.push({
-        id: terminal.id,
-        label: terminal.label,
-        status: terminal.status,
-        cwd: terminal.cwd,
-      });
-    }
-
-    console.log(`[session] Created session: tier=${tier}, terminals=${terminalsMax}`);
+    console.log(`[session] Created new session: tier=${tier}, terminalsMax=${terminalsMax}`);
 
+    // Return empty terminals - user can add via + button
     res.json({
       tier,
       terminalsMax,
       features,
-      terminals: spawnedTerminals,
+      terminals: [],
     });
   } catch (err) {
     res.status(500).json({ error: 'Failed to create session', detail: err.message });