From 340aa5448fb435229bd3c1bc38424645003ea8c8 Mon Sep 17 00:00:00 2001 From: iris Date: Wed, 2 Sep 2026 03:07:43 +0200 Subject: [PATCH] frontend: finish the dashboard comment-density sweep Continues #3901 (dashboard round 1 was #3906, tabs.js/swarm.js). Scanned the rest of packages/dashboard/src for the same 'moved to X'/ 'now lives in Y' pattern: five more hits in schedules.js, common.js, logs.js (two, one duplicating the other), core.js, and swarm.js. All trimmed to state the current location as a fact rather than narrating the move; one inline duplicate (logs.js's tab-default comment restating the file-header pointer) dropped outright. --- frontend/packages/dashboard/src/common.js | 6 +++--- frontend/packages/dashboard/src/core.js | 12 ++++++------ frontend/packages/dashboard/src/logs.js | 4 ++-- frontend/packages/dashboard/src/schedules.js | 5 ++--- frontend/packages/dashboard/src/swarm.js | 2 +- 5 files changed, 14 insertions(+), 15 deletions(-) diff --git a/frontend/packages/dashboard/src/common.js b/frontend/packages/dashboard/src/common.js index 52b7643f..803f338d 100644 --- a/frontend/packages/dashboard/src/common.js +++ b/frontend/packages/dashboard/src/common.js @@ -35,9 +35,9 @@ export const form = (action, btnClass, btnLabel, confirmMsg, extra = {}, opts = return f; }; -// `bindAsyncForms` (the `data-async` form submit interceptor) now lives in -// `@hive/shared/forms.js` alongside `asyncBtn` — both the dashboard and the -// per-agent UI import it directly from there rather than through this file. +// `bindAsyncForms` (the `data-async` form submit interceptor) and +// `asyncBtn` live in `@hive/shared/forms.js`, not here — both the +// dashboard and the per-agent UI import them directly from there. // `truncate`, `fmtAgo`, `fmtElapsed`, `fmtDuration` + the `paintAtomic` // render helper live in the dashboard-internal `./util.js`, not here — diff --git a/frontend/packages/dashboard/src/core.js b/frontend/packages/dashboard/src/core.js index 28df8e9c..4f1a3843 100644 --- a/frontend/packages/dashboard/src/core.js +++ b/frontend/packages/dashboard/src/core.js @@ -1,10 +1,10 @@ -// C0R3 page entry (/core.html). The host/coordinator surface carved out -// of the dashboard's old SYST3M tab into its own standalone page so the -// dashboard tab strip stays lean. Two sub-tabs via -// (default = K3PT ST4T3): kept state (tombstones) and container load. +// C0R3 page entry (/core.html). The host/coordinator surface, its own +// standalone page so the dashboard tab strip stays lean. Two sub-tabs +// via (default = K3PT ST4T3): kept state (tombstones) +// and container load. // -// Rebuild queue + meta inputs have moved to /builds.html (the build -// lifecycle hub). This page is its own esbuild bundle with its own +// Rebuild queue + meta inputs live at /builds.html (the build lifecycle +// hub), not here. This page is its own esbuild bundle with its own // runtime: it cold-loads /api/state and subscribes to /dashboard/stream // (the same broker event channel the dashboard uses), maintaining its // own copy of the tombstones state. diff --git a/frontend/packages/dashboard/src/logs.js b/frontend/packages/dashboard/src/logs.js index 3bdefc15..38099922 100644 --- a/frontend/packages/dashboard/src/logs.js +++ b/frontend/packages/dashboard/src/logs.js @@ -1,5 +1,5 @@ // /logs.html entry point: log viewer with three sub-tabs (AGENT, INFRA, -// SYSTEM). Build log history has moved to /builds.html. +// SYSTEM). Build log history lives at /builds.html, not here. // AGENT — per-container journald viewer, backed by GET /api/journal/{name} // INFRA — infra-container journald viewer (hive-ci, hive-forge, …), // same API but full machine journal only (no unit filter) @@ -230,7 +230,7 @@ import '@hive/shared/hive-tab-strip.js'; // Wire the shared tab strip now that fetchSystem + the element refs it // needs are defined. Its initial show() paints the active pane and, if // the deep-linked tab is SYSTEM, kicks off the lazy fetch via onShow. - // Default: AGENT (build logs moved to /builds.html). + // Default: AGENT. logTabs = document.getElementById('logs-tabbar').configure({ tabs: [ { id: 'agent', label: 'AGENT' }, diff --git a/frontend/packages/dashboard/src/schedules.js b/frontend/packages/dashboard/src/schedules.js index 27c18bfc..8c0500fe 100644 --- a/frontend/packages/dashboard/src/schedules.js +++ b/frontend/packages/dashboard/src/schedules.js @@ -10,9 +10,8 @@ // tracking); reads the shared agent roster from state.js for the // target-chip pickers. Render + format helpers come from util.js. // -// Note: per-agent reminders moved to the in-container store (reminders -// migration) and are no longer listed here — they surface via -// get_loose_ends / the todos pill on the agent page instead. +// Note: per-agent reminders live in the in-container store, not here — +// they surface via get_loose_ends / the todos pill on the agent page. import { $ } from './common.js'; import { el } from '@hive/shared/dom.js'; diff --git a/frontend/packages/dashboard/src/swarm.js b/frontend/packages/dashboard/src/swarm.js index 9de7a244..eafefb05 100644 --- a/frontend/packages/dashboard/src/swarm.js +++ b/frontend/packages/dashboard/src/swarm.js @@ -543,7 +543,7 @@ function buildContainerLi(c, node, opts) { body.append(head); // Per-card action buttons (R3ST4RT / ST0P / ST4RT / R3BU1LD / - // DESTR0Y / PURG3) moved to the selection bar — see + // DESTR0Y / PURG3) live on the selection bar, not here — see // docs/web-ui.md::Selection bar. The contextual `needs update ↻` // chip in the head row stays — it's a state-hint, not an // action button.