From fd42a54d96fcd3342941caaeb61a4b0d5d3f1b4f Mon Sep 17 00:00:00 2001
From: Peter Stone <thepeterstone@gmail.com>
Date: Fri, 6 Mar 2026 23:55:07 +0000
Subject: recover: restore untracked work from recovery branch (no Gemini
 changes)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Recovered files with no Claude→Agent contamination:
- docs/adr/002-task-state-machine.md
- internal/api/logs.go/logs_test.go: task-level log streaming endpoint
- internal/api/validate.go/validate_test.go: POST /api/tasks/validate
- internal/api/server_test.go, storage/db_test.go: expanded test coverage
- scripts/reset-failed-tasks, reset-running-tasks
- web/app.js, index.html, style.css: frontend improvements
- web/test/: active-tasks-tab, delete-button, filter-tabs, sort-tasks tests

Manually applied from server.go diff (skipping Claude→Agent rename):
- taskLogStore field + validateCmdPath field
- DELETE /api/tasks/{id} route + handleDeleteTask
- GET /api/tasks/{id}/logs/stream route
- POST /api/tasks/{id}/resume route + handleResumeTimedOutTask
- handleCancelTask: allow cancelling PENDING/QUEUED tasks directly

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 web/test/active-tasks-tab.test.mjs | 68 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 68 insertions(+)
 create mode 100644 web/test/active-tasks-tab.test.mjs

(limited to 'web/test/active-tasks-tab.test.mjs')

diff --git a/web/test/active-tasks-tab.test.mjs b/web/test/active-tasks-tab.test.mjs
new file mode 100644
index 0000000..7b68c6f
--- /dev/null
+++ b/web/test/active-tasks-tab.test.mjs
@@ -0,0 +1,68 @@
+// active-tasks-tab.test.mjs — TDD contract tests for filterActiveTasks
+//
+// filterActiveTasks is imported from app.js. Tests are RED until the function
+// is exported from app.js.
+//
+// Run with: node --test web/test/active-tasks-tab.test.mjs
+
+import { describe, it } from 'node:test';
+import assert from 'node:assert/strict';
+import { filterActiveTasks } from '../app.js';
+
+// ── Helpers ────────────────────────────────────────────────────────────────────
+
+function makeTask(state) {
+  return { id: state, name: `task-${state}`, state };
+}
+
+// ── Tests ──────────────────────────────────────────────────────────────────────
+
+describe('filterActiveTasks', () => {
+  it('returns only RUNNING tasks when given mixed states', () => {
+    const tasks = [makeTask('RUNNING'), makeTask('COMPLETED'), makeTask('PENDING')];
+    const result = filterActiveTasks(tasks);
+    assert.equal(result.length, 1);
+    assert.equal(result[0].state, 'RUNNING');
+  });
+
+  it('returns only READY tasks', () => {
+    const tasks = [makeTask('READY'), makeTask('FAILED'), makeTask('TIMED_OUT')];
+    const result = filterActiveTasks(tasks);
+    assert.equal(result.length, 1);
+    assert.equal(result[0].state, 'READY');
+  });
+
+  it('returns only QUEUED tasks', () => {
+    const tasks = [makeTask('QUEUED'), makeTask('CANCELLED'), makeTask('PENDING')];
+    const result = filterActiveTasks(tasks);
+    assert.equal(result.length, 1);
+    assert.equal(result[0].state, 'QUEUED');
+  });
+
+  it('returns only BLOCKED tasks', () => {
+    const tasks = [makeTask('BLOCKED'), makeTask('BUDGET_EXCEEDED'), makeTask('COMPLETED')];
+    const result = filterActiveTasks(tasks);
+    assert.equal(result.length, 1);
+    assert.equal(result[0].state, 'BLOCKED');
+  });
+
+  it('returns all four active states together, excludes PENDING/COMPLETED/FAILED/TIMED_OUT/CANCELLED/BUDGET_EXCEEDED', () => {
+    const allStates = [
+      'RUNNING', 'READY', 'QUEUED', 'BLOCKED',
+      'PENDING', 'COMPLETED', 'FAILED', 'TIMED_OUT', 'CANCELLED', 'BUDGET_EXCEEDED',
+    ];
+    const tasks = allStates.map(makeTask);
+    const result = filterActiveTasks(tasks);
+    assert.equal(result.length, 4, 'exactly 4 active-state tasks should be returned');
+    for (const state of ['RUNNING', 'READY', 'QUEUED', 'BLOCKED']) {
+      assert.ok(result.some(t => t.state === state), `${state} should be included`);
+    }
+    for (const state of ['PENDING', 'COMPLETED', 'FAILED', 'TIMED_OUT', 'CANCELLED', 'BUDGET_EXCEEDED']) {
+      assert.ok(!result.some(t => t.state === state), `${state} should be excluded`);
+    }
+  });
+
+  it('returns empty array for empty input', () => {
+    assert.deepEqual(filterActiveTasks([]), []);
+  });
+});
-- 
cgit v1.2.3