From 053ec03306f37e354c013beb8b4baf8eae27af97 Mon Sep 17 00:00:00 2001 From: Peter Stone Date: Wed, 25 Mar 2026 23:08:09 +0000 Subject: fix: remove duplicate summary section in task panel, fix test mock DOM MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit renderTaskPanel was rendering the summary twice — once with .task-summary before Overview, and again with .task-summary-text after it. Remove the second (post-Overview) duplicate. Add `dataset: {}` to the test mock DOM so makeMetaItem's state badge path doesn't crash during renderTaskPanel tests. Co-Authored-By: Claude Sonnet 4.6 --- web/app.js | 10 ---------- web/test/task-panel-summary.test.mjs | 1 + 2 files changed, 1 insertion(+), 10 deletions(-) diff --git a/web/app.js b/web/app.js index 1e10ad3..2241c67 100644 --- a/web/app.js +++ b/web/app.js @@ -1798,16 +1798,6 @@ export function renderTaskPanel(task, executions) { overview.appendChild(overviewGrid); content.appendChild(overview); - // ── Summary ── - if (task.summary) { - const summarySection = makeSection("Summary"); - const summaryText = document.createElement("div"); - summaryText.className = "task-summary-text"; - summaryText.textContent = task.summary; - summarySection.appendChild(summaryText); - content.appendChild(summarySection); - } - // ── Agent Config ── const a = task.agent || {}; const agentSection = makeSection('Agent Config'); diff --git a/web/test/task-panel-summary.test.mjs b/web/test/task-panel-summary.test.mjs index 63dd483..1777003 100644 --- a/web/test/task-panel-summary.test.mjs +++ b/web/test/task-panel-summary.test.mjs @@ -18,6 +18,7 @@ function makeMockDOM() { innerHTML: '', hidden: false, children: [], + dataset: {}, _listeners: {}, appendChild(child) { this.children.push(child); return child; }, prepend(...nodes) { this.children.unshift(...nodes); }, -- cgit v1.2.3