input-kanban 0.0.14 → 0.0.15

package/RELEASE_NOTES.md

@@ -1,5 +1,23 @@
 # Release Notes
 
+## v0.0.15
+
+### Highlights
+
+- Absorb PR #3 recovery hardening without directly merging its older `v0.0.13` branch, preserving the `v0.0.14` Plan Approval Gate and Web layout changes.
+- Route blocked-run dashboard execution actions to `/api/runs/:id/retry`, so `batch_blocked` runs retry failed/unknown tasks instead of hitting the dispatch endpoint.
+- Remove stale Web `workers_completed` / `workers_failed` UI state handling now that backend run status uses `batches_completed` and `batch_blocked`.
+- Harden final judge starts: reject archived/stopped runs, duplicate running judges, and completed judges; failed judges are archived to `judge_attempts/` before retrying.
+- Add short `/api/codex` detection caching to avoid repeatedly spawning Codex detection during frequent dashboard refreshes.
+- Keep task-table Codex session IDs and their copy buttons on one line by widening the session column and using a compact inline layout.
+
+### Verification
+
+- `npm run check` passed locally with 84 tests.
+- `npm pack --dry-run` passed for `input-kanban@0.0.15`.
+- Windows release-candidate validation on `zhangxing_win` passed with 84 tests.
+- Windows Web smoke confirmed `/api/health`, `/api/codex`, Plan Approval UI, compact header copy tools, and one-line Codex session copy layout.
+
 ## v0.0.14
 
 ### Highlights

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "input-kanban",
-  "version": "0.0.14",
+  "version": "0.0.15",
   "type": "module",
   "bin": {
     "input-kanban": "bin/input-kanban.js"

package/public/index.html

@@ -41,20 +41,20 @@
     @media (prefers-reduced-motion: reduce) { button, button.state-active, button.state-pending::after, .refresh-pulse-chip.pulse .refresh-pulse-dot { animation: none !important; transition: none !important; } }
     table { width: 100%; border-collapse: collapse; font-size: 13px; table-layout: fixed; }
     th, td { border-bottom: 1px solid var(--line); padding: 9px 8px; text-align: left; vertical-align: top; }
-    th:nth-child(1), td:nth-child(1) { width: 34%; }
+    th:nth-child(1), td:nth-child(1) { width: 32%; }
     th:nth-child(2), td:nth-child(2) { width: 92px; white-space: nowrap; }
     th:nth-child(3), td:nth-child(3) { width: 132px; }
     th:nth-child(4), td:nth-child(4) { width: 64px; white-space: nowrap; }
     th:nth-child(5), td:nth-child(5) { width: 96px; }
-    th:nth-child(6), td:nth-child(6) { width: 92px; }
+    th:nth-child(6), td:nth-child(6) { width: 116px; }
     th:nth-child(7), td:nth-child(7) { width: 66px; }
     th:nth-child(8), td:nth-child(8) { width: 94px; }
     th { color: #cbd5e1; font-size: 12px; text-transform: uppercase; letter-spacing: .06em; }
     tr:hover { background: #162033; cursor: pointer; }
     .pill { display: inline-block; padding: 3px 8px; border-radius: 999px; font-size: 12px; font-weight: 800; background: var(--gray); line-height: 1.3; }
-    .completed, .judged, .workers_completed, .planned, .batches_completed { background: var(--green); }
+    .completed, .judged, .planned, .batches_completed { background: var(--green); }
     .running, .planning, .judging { background: var(--blue); }
-    .failed, .workers_failed, .plan_failed, .judge_failed, .unknown, .batch_blocked { background: var(--red); }
+    .failed, .plan_failed, .judge_failed, .unknown, .batch_blocked { background: var(--red); }
     .plan_empty { background: var(--orange); }
     .pending, .created { background: var(--gray); }
     .batch-row td { border-top: 3px solid var(--line-strong); background: #101827; font-weight: 800; font-size: 14px; padding-top: 13px; }
@@ -117,7 +117,9 @@
     .run-card-title-actions { display: inline-flex; align-items: center; gap: 3px; }
     .archive-confirm-btn { min-width: 46px; padding: 4px 10px; border-color: rgba(248,113,113,.85); background: var(--red) !important; color: white; font-weight: 900; }
     .icon-svg { width: 14px; height: 14px; display: block; }
-    .session-cell { word-break: break-all; }
+    .session-cell-wrap { display: inline-flex; align-items: center; gap: 5px; white-space: nowrap; }
+    .session-cell { min-width: 0; font-variant-numeric: tabular-nums; }
+    .session-cell-wrap .copy-btn { flex: 0 0 auto; margin-left: 0; }
     .row-actions { display: flex; align-items: center; justify-content: flex-end; gap: 6px; }
     .row-actions .danger, .row-actions .secondary { margin: 0; }
     .status-stack { display: inline-flex; flex-direction: column; align-items: flex-start; gap: 5px; }
@@ -273,7 +275,6 @@ function userFacingErrorMessage(error) {
 const statusText = {
   created: '已创建', pending: '等待中', planning: '拆分中', planned: '已拆分',
   running: '执行中', completed: '已完成', failed: '失败', unknown: '未知',
-  workers_completed: '子任务完成', workers_failed: '子任务失败',
   batches_completed: '批次完成', batch_blocked: '批次阻塞', plan_empty: '拆分为空', stopped: '已停止',
   judging: '验收中', judged: '已验收', plan_failed: '拆分失败', judge_failed: '验收失败'
 };
@@ -586,23 +587,22 @@ function runActionState(key) {
   const anyWorkerRunning = (currentState.tasks || []).some(t => t.status === 'running');
   if (key === 'plan') {
     if (status === 'planning' || currentState.planner?.status === 'running') return { label: '拆分中', disabled: true, state: 'active' };
-    if (status === 'planned' || status === 'running' || status === 'workers_completed' || status === 'batches_completed' || status === 'judging' || status === 'judged') return { label: '已拆分', disabled: true, state: 'done' };
+    if (status === 'planned' || status === 'running' || status === 'batches_completed' || status === 'judging' || status === 'judged') return { label: '已拆分', disabled: true, state: 'done' };
     if (status === 'plan_failed' || status === 'plan_empty') return { label: '重试拆分', disabled: false, state: 'retry' };
     return { label: '拆分', disabled: false, state: '' };
   }
   if (key === 'dispatch') {
     if (status === 'running' || anyWorkerRunning) return { label: '执行中', disabled: true, state: 'active' };
     if (status === 'planned') return { label: planApprovalPending() ? '开始执行' : '执行', disabled: false, state: '' };
-    if (status === 'batch_blocked') return { label: '执行', disabled: false, state: 'retry' };
-    if (status === 'workers_failed') return { label: '重试执行', disabled: false, state: 'retry' };
-    if (['workers_completed','batches_completed','judging','judged'].includes(status)) return { label: '已完成', disabled: true, state: 'done' };
+    if (status === 'batch_blocked') return { label: '重试执行', disabled: false, state: 'retry' };
+    if (['batches_completed','judging','judged'].includes(status)) return { label: '已完成', disabled: true, state: 'done' };
     return { label: '执行', disabled: true, state: 'done' };
   }
   if (key === 'judge') {
     if (status === 'judging' || currentState.judge?.status === 'running') return { label: '验收中', disabled: true, state: 'active' };
     if (status === 'judged') return { label: '已验收', disabled: true, state: 'done' };
     if (status === 'judge_failed') return { label: '重试验收', disabled: false, state: 'retry' };
-    if (status === 'batches_completed' || status === 'workers_completed') return { label: '验收', disabled: false, state: '' };
+    if (status === 'batches_completed') return { label: '验收', disabled: false, state: '' };
     return { label: '验收', disabled: true, state: 'done' };
   }
   if (key === 'stop') {
@@ -716,7 +716,7 @@ function shortSessionId(thread) {
 }
 function sessionCell(thread) {
   if (!thread) return '-';
-  return `<span class="session-cell" title="${esc(thread)}">…${esc(shortSessionId(thread))}</span><button class="copy-btn" title="复制完整 Codex 会话ID" onclick="copySessionId(event, '${esc(thread)}')">⧉</button>`;
+  return `<span class="session-cell-wrap"><span class="session-cell" title="${esc(thread)}">…${esc(shortSessionId(thread))}</span><button class="copy-btn" title="复制完整 Codex 会话ID" onclick="copySessionId(event, '${esc(thread)}')">⧉</button></span>`;
 }
 function taskStartedCell(t) {
   return t?.startedAt ? formatDateTime(t.startedAt) : '-';
@@ -1049,7 +1049,11 @@ async function renameRunLabel(event, runId = selectedRun) {
   });
 }
 async function planRun() { if (selectedRun) await runActionWithPending('plan', async () => { await api(`/api/runs/${selectedRun}/plan`, {method:'POST'}); await refreshSelected(); }); }
-async function dispatchRun() { if (selectedRun) await runActionWithPending('dispatch', async () => { await api(`/api/runs/${selectedRun}/dispatch`, {method:'POST'}); await refreshSelected(); }); }
+async function dispatchRun() {
+  if (!selectedRun) return;
+  const endpoint = currentState?.status === 'batch_blocked' ? 'retry' : 'dispatch';
+  await runActionWithPending('dispatch', async () => { await api(`/api/runs/${selectedRun}/${endpoint}`, {method:'POST'}); await refreshSelected(); });
+}
 async function judgeRun() { if (selectedRun) await runActionWithPending('judge', async () => { await api(`/api/runs/${selectedRun}/judge`, {method:'POST'}); await refreshSelected(); }); }
 async function stopSelectedRun() {
   if (!selectedRun) return;

package/src/orchestrator.js

@@ -500,6 +500,27 @@ async function rotatePlannerAttempt(state, runDir) {
   }];
 }
 
+async function rotateJudgeAttempt(state, runDir) {
+  const judgeDir = roleDir(runDir, 'judge');
+  if (!fs.existsSync(judgeDir)) return;
+  const attemptsDir = path.join(runDir, 'judge_attempts');
+  await ensureDir(attemptsDir);
+  const previousAttempts = (state.judgeAttempts || []).map(item => Number(item.attempt || 0)).filter(Number.isFinite);
+  const attempt = Number(state.judge?.attempt || 0) || Math.max(1, 1 + Math.max(0, ...previousAttempts));
+  const archivedDir = path.join(attemptsDir, `attempt-${String(attempt).padStart(2, '0')}`);
+  await fsp.rm(archivedDir, { recursive: true, force: true });
+  await fsp.rename(judgeDir, archivedDir);
+  state.judgeAttempts = [...(state.judgeAttempts || []), {
+    attempt,
+    status: state.judge?.status,
+    exitCode: state.judge?.exitCode ?? null,
+    startedAt: state.judge?.startedAt,
+    endedAt: state.judge?.endedAt,
+    archivedDir,
+    archivedAt: nowIso()
+  }];
+}
+
 async function rotateWorkerAttempt(state, task) {
   const runDir = pathForRun(state.runId);
   const workerDir = roleDir(runDir, 'worker', task.id);
@@ -792,16 +813,26 @@ export async function startJudge(runId) {
   return await withRunStateLock(runId, async () => {
     const state = await loadRun(runId);
     if (!state) throw new Error(`run not found: ${runId}`);
+    if (state.archived) throw new Error('archived run cannot be judged');
+    if (state.status === 'stopped') throw new Error('stopped run cannot be judged');
     recomputeRunStatus(state);
+    if (state.judge?.status === 'running') throw new Error('judge already running');
+    if (state.judge?.status === 'completed') throw new Error('judge already completed');
     if (!allBatchesCompleted(state) && state.tasks?.length) throw new Error('final judge is allowed only after all batches completed');
-    const outDir = roleDir(pathForRun(runId), 'judge');
+    const runDir = pathForRun(runId);
+    if (state.judge?.status === 'failed') await rotateJudgeAttempt(state, runDir);
+    const outDir = roleDir(runDir, 'judge');
     await ensureDir(outDir);
     const judgeInputPath = path.join(outDir, 'judge_input.json');
     const judgeInput = await buildJudgeInput(state);
     await writeJsonAtomic(judgeInputPath, judgeInput);
     const prompt = defaultJudgePrompt(state, judgeInputPath);
-    const child = await runner.startCodexTask({ runId: state.runId, taskId: 'judge', batchId: 'judge', runStatePath: statePath(pathForRun(runId)), prompt, sandbox: 'read-only', cwd: workspacePathOf(state), outDir });
-    state.judge = { status: 'running', pid: child.pid, startedAt: nowIso(), dir: outDir };
+    const child = await runner.startCodexTask({ runId: state.runId, taskId: 'judge', batchId: 'judge', runStatePath: statePath(runDir), prompt, sandbox: 'read-only', cwd: workspacePathOf(state), outDir });
+    const previousJudgeAttempts = [
+      ...(state.judgeAttempts || []).map(item => Number(item.attempt || 0)),
+      Number(state.judge?.attempt || 0)
+    ].filter(Number.isFinite);
+    state.judge = { status: 'running', pid: child.pid, startedAt: nowIso(), dir: outDir, attempt: 1 + Math.max(0, ...previousJudgeAttempts) };
     state.status = 'judging';
     await saveRun(state);
     child.onExit(async code => {

package/src/server.js

@@ -8,6 +8,8 @@ import { createRun, listRuns, startPlanner, dispatchRun, startJudge, refreshRun,
 import { startAutoScheduler } from './scheduler.js';
 
 const PUBLIC_DIR = path.join(APP_ROOT, 'public');
+const CODEX_INFO_TTL_MS = 30000;
+let codexInfoCache = null;
 
 function send(res, status, body, type = 'application/json') {
   const data = type === 'application/json' ? JSON.stringify(body, null, 2) : body;
@@ -26,6 +28,13 @@ async function readBody(req) {
 function notFound(res) { send(res, 404, { error: 'not found' }); }
 function methodNotAllowed(res) { send(res, 405, { error: 'method not allowed' }); }
 
+async function cachedCodexInfo(nowMs = Date.now()) {
+  if (codexInfoCache && codexInfoCache.expiresAt > nowMs) return codexInfoCache.value;
+  const value = await detectCodexInfo();
+  codexInfoCache = { value, expiresAt: nowMs + CODEX_INFO_TTL_MS };
+  return value;
+}
+
 async function serveStatic(req, res, pathname) {
   let file = pathname === '/' ? path.join(PUBLIC_DIR, 'index.html') : path.join(PUBLIC_DIR, pathname.replace(/^\/+/, ''));
   file = path.resolve(file);
@@ -46,7 +55,7 @@ async function handleApi(req, res, url, appClient) {
       return send(res, 200, { ok: true, version: PACKAGE_VERSION, appRoot: APP_ROOT, runsDir: RUNS_DIR, defaultWorkspace: DEFAULT_WORKSPACE, defaultRepo: DEFAULT_REPO, runner: RUNNER, codexBin: CODEX_BIN });
     }
     if (req.method === 'GET' && url.pathname === '/api/codex') {
-      return send(res, 200, { ok: true, codex: await detectCodexInfo() });
+      return send(res, 200, { ok: true, codex: await cachedCodexInfo() });
     }
     if (parts[1] === 'runs' && parts.length === 2) {
       if (req.method === 'GET') return send(res, 200, { runs: await listRuns({ includeArchived: url.searchParams.get('includeArchived') === '1', workspace: url.searchParams.get('workspace') || '' }) });