From fb0e4b44393bae3c54f099bea87dfea19854d058 Mon Sep 17 00:00:00 2001
From: Peter Stone <thepeterstone@gmail.com>
Date: Fri, 13 Mar 2026 05:24:20 +0000
Subject: fix: enable Gemini file writing by passing --yolo and -p flags

GeminiRunner.buildArgs was missing --yolo (auto-approve all tools)
so the gemini CLI only registered 3 tools (read_file, write_todos,
cli_help) and write_file was not available. Agents that needed to
create files silently failed (exit 0, no files written).

Also switch instructions from bare positional arg to -p flag, which
is required for non-interactive headless mode.

Update preamble tests to match file-based summary approach
(CLAUDOMATOR_SUMMARY_FILE) kept from the merge conflict resolution.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 web/app.js                        |  7 +++----
 web/test/new-task-button.test.mjs | 24 ++++++++++++++++++++++++
 2 files changed, 27 insertions(+), 4 deletions(-)
 create mode 100644 web/test/new-task-button.test.mjs

(limited to 'web')

diff --git a/web/app.js b/web/app.js
index 3121f28..187795f 100644
--- a/web/app.js
+++ b/web/app.js
@@ -278,6 +278,9 @@ export function filterActiveTasks(tasks) {
   return tasks.filter(t => _PANEL_ACTIVE_STATES.has(t.state));
 }
 
+// The New Task button is always visible regardless of active tab.
+export function newTaskButtonShouldShowOnTab(_tab) { return true; }
+
 export function filterTasksByTab(tasks, tab) {
   if (tab === 'active')      return tasks.filter(t => ACTIVE_STATES.has(t.state));
   if (tab === 'interrupted') return tasks.filter(t => INTERRUPTED_STATES.has(t.state));
@@ -2198,10 +2201,6 @@ function switchTab(name) {
     }
   });
 
-  // Show/hide the header New Task button (only relevant on tasks tab)
-  document.getElementById('btn-new-task').style.display =
-    name === 'tasks' ? '' : 'none';
-
   if (name === 'running') {
     fetchTasks().then(renderRunningView).catch(() => {
       const currentEl = document.querySelector('.running-current');
diff --git a/web/test/new-task-button.test.mjs b/web/test/new-task-button.test.mjs
new file mode 100644
index 0000000..45a3548
--- /dev/null
+++ b/web/test/new-task-button.test.mjs
@@ -0,0 +1,24 @@
+// new-task-button.test.mjs — visibility contract for the New Task button
+//
+// The New Task button lives in the global header and must be visible on all tabs.
+// Run with: node --test web/test/new-task-button.test.mjs
+
+import { describe, it } from 'node:test';
+import assert from 'node:assert/strict';
+import { newTaskButtonShouldShowOnTab } from '../app.js';
+
+const ALL_TABS = ['tasks', 'active', 'running', 'stats'];
+
+describe('new task button visibility', () => {
+  for (const tab of ALL_TABS) {
+    it(`is visible on "${tab}" tab`, () => {
+      assert.equal(newTaskButtonShouldShowOnTab(tab), true, `expected button to be visible on tab "${tab}"`);
+    });
+  }
+
+  it('is visible on any unknown future tab', () => {
+    assert.equal(newTaskButtonShouldShowOnTab('help'), true);
+    assert.equal(newTaskButtonShouldShowOnTab('templates'), true);
+    assert.equal(newTaskButtonShouldShowOnTab(''), true);
+  });
+});
-- 
cgit v1.2.3