ninja-terminals 2.1.1 → 2.1.3

package/package.json

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

package/public/app.js

@@ -202,10 +202,8 @@ const state = {
 const grid = document.getElementById('grid');
 const sidebar = document.getElementById('sidebar');
 const activityFeed = document.getElementById('activity-feed');
-const taskQueue = document.getElementById('task-queue');
 const statusCounts = document.getElementById('status-counts');
 const statusProgress = document.getElementById('status-progress');
-const addTaskBtn = document.getElementById('add-task-btn');
 const sidebarToggle = document.getElementById('sidebar-toggle');
 
 // ── State Color Map ──────────────────────────────────────────
@@ -347,7 +345,7 @@ function createTerminalUI(termData) {
   closeBtn.addEventListener('mousedown', (e) => {
     e.preventDefault();
     e.stopPropagation();
-    removeTerminal(id);
+    closeTerminal(id);
   });
 
   header.appendChild(labelEl);
@@ -892,41 +890,6 @@ function connectSSE() {
   return evtSource;
 }
 
-// ── Task Queue ───────────────────────────────────────────────
-
-async function fetchTasks() {
-  try {
-    const res = await fetch(`${API_BASE}/api/tasks`, { headers: auth.getAuthHeader() });
-    if (!res.ok) {
-      taskQueue.innerHTML = '<div class="no-tasks">No tasks</div>';
-      return;
-    }
-    const tasks = await res.json();
-    renderTasks(tasks);
-  } catch {
-    taskQueue.innerHTML = '<div class="no-tasks">No tasks</div>';
-  }
-}
-
-function renderTasks(tasks) {
-  if (!tasks || !Array.isArray(tasks) || tasks.length === 0) {
-    taskQueue.innerHTML = '<div class="no-tasks">No tasks</div>';
-    return;
-  }
-
-  taskQueue.innerHTML = '';
-  for (const task of tasks) {
-    const item = document.createElement('div');
-    item.className = 'task-item';
-    const dotClass = task.status || 'pending';
-    item.innerHTML = `
-      <span class="task-dot ${dotClass}"></span>
-      <span class="task-name" title="${escapeHtml(task.name || task.id || '')}">${escapeHtml(task.name || task.id || 'Unnamed')}</span>
-      <span class="task-status">${escapeHtml(task.status || 'pending')}</span>
-    `;
-    taskQueue.appendChild(item);
-  }
-}
 
 // ── Status Polling (fallback + elapsed updates) ──────────────
 
@@ -996,14 +959,6 @@ function setupSidebar() {
   document.body.appendChild(mobileToggle);
 }
 
-// ── Add Task ─────────────────────────────────────────────────
-
-function setupAddTask() {
-  addTaskBtn.addEventListener('click', () => {
-    addNewTerminal();
-  });
-}
-
 // ── Resize Handler ───────────────────────────────────────────
 
 let resizeTimeout;
@@ -1020,7 +975,6 @@ async function startApp() {
 
   // Setup sidebar
   setupSidebar();
-  setupAddTask();
   setupLearnings();
   setupAddTerminal();
 
@@ -1049,15 +1003,9 @@ async function startApp() {
   // Connect SSE for real-time updates
   connectSSE();
 
-  // Fetch initial task queue
-  fetchTasks();
-
   // Poll status every 3 seconds (fallback for SSE + elapsed updates)
   setInterval(pollStatus, 3000);
 
-  // Poll task queue every 5 seconds
-  setInterval(fetchTasks, 5000);
-
   // Initial status bar
   updateStatusBar();
 
@@ -1108,7 +1056,7 @@ async function clearAllTerminals() {
 
   const ids = Array.from(state.terminals.keys());
   for (const id of ids) {
-    await removeTerminal(id);
+    await closeTerminal(id);
   }
   addFeedEntry('All terminals cleared');
 }

package/public/index.html

@@ -77,13 +77,7 @@
         <h3 class="sidebar-title">Activity</h3>
         <div id="activity-feed"></div>
       </div>
-      <hr>
-      <div class="sidebar-section">
-        <h3 class="sidebar-title">Task Queue</h3>
-        <div id="task-queue"></div>
-        <button id="add-task-btn" class="add-task-btn">+ Add Task</button>
-      </div>
-    </aside>
+          </aside>
     <main>
       <div id="grid"></div>
     </main>

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 });