diff options
| author | Peter Stone <thepeterstone@gmail.com> | 2026-03-25 23:08:09 +0000 |
|---|---|---|
| committer | Peter Stone <thepeterstone@gmail.com> | 2026-03-25 23:08:09 +0000 |
| commit | 053ec03306f37e354c013beb8b4baf8eae27af97 (patch) | |
| tree | 1bae76f4a0d5659520b95d87b979c3c2c98aae23 | |
| parent | ffa57952709e88c6524448f89a3d4dd623be2440 (diff) | |
fix: remove duplicate summary section in task panel, fix test mock DOM
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 <noreply@anthropic.com>
| -rw-r--r-- | web/app.js | 10 | ||||
| -rw-r--r-- | web/test/task-panel-summary.test.mjs | 1 |
2 files changed, 1 insertions, 10 deletions
@@ -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); }, |