From 164606dc7cf8c3bb380be113adfddb78bd661ce3 Mon Sep 17 00:00:00 2001 From: librelad Date: Sun, 31 May 2026 00:41:16 +0100 Subject: [PATCH] docs(webui): refresh stale features/ path comments after the components/ rename MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Comment-only tidy from the feng-shui audit — no code behavior changes. The features/ directory was renamed to components/ during modularization, but several header banners and inline comments still named the old path: - 6 component module headers (admin/tasks/backup/dashboard/updater/index.js + updater/js/updater-page.js) now name their real components//… path - core/kernel/js/spa.js + core/tasks/js/task-router.js comments - backend/routes/features.js doc-banner (drop a components// folder …) - core/update-notifier/css/update-notifier.css header (js/update-notifier.js) Guarded the rewrite so the LIVE /api/features/list endpoint name (feature- registry.js sources + backend route) is untouched — only stale 'features/' directory references were updated. Signed-off-by: librelad --- containers/libreportal/backend/routes/features.js | 4 ++-- containers/libreportal/frontend/components/admin/index.js | 2 +- containers/libreportal/frontend/components/backup/index.js | 2 +- containers/libreportal/frontend/components/dashboard/index.js | 2 +- containers/libreportal/frontend/components/tasks/index.js | 2 +- containers/libreportal/frontend/components/updater/index.js | 2 +- .../frontend/components/updater/js/updater-page.js | 2 +- containers/libreportal/frontend/core/kernel/js/spa.js | 2 +- containers/libreportal/frontend/core/tasks/js/task-router.js | 2 +- .../frontend/core/update-notifier/css/update-notifier.css | 2 +- 10 files changed, 11 insertions(+), 11 deletions(-) diff --git a/containers/libreportal/backend/routes/features.js b/containers/libreportal/backend/routes/features.js index 5f938e4..70ab516 100644 --- a/containers/libreportal/backend/routes/features.js +++ b/containers/libreportal/backend/routes/features.js @@ -12,9 +12,9 @@ const FEATURES_DIR = path.join(__dirname, '..', '..', 'frontend', 'components'); Walks frontend/components// and returns one entry per directory that contains a feature.json — the WebUI's page manifest, discovered from the folders themselves (exactly how /api/themes/list discovers themes). Drop a - features// folder in and its page appears; delete it and the page is + components// folder in and its page appears; delete it and the page is gone — no central edit. The navigation kernel fetches this and falls back to - the checked-in features/manifest.dev.json if the API is unavailable. + the checked-in components/manifest.dev.json if the API is unavailable. Each feature.json declares: id, routes[], optional module (self-registering index.js), optional handler (legacy fallback method), navId, nav{}, and order diff --git a/containers/libreportal/frontend/components/admin/index.js b/containers/libreportal/frontend/components/admin/index.js index c519435..5ac9c03 100644 --- a/containers/libreportal/frontend/components/admin/index.js +++ b/containers/libreportal/frontend/components/admin/index.js @@ -1,4 +1,4 @@ -// features/admin/index.js — the whole Admin area as one feature module. +// components/admin/index.js — the whole Admin area as one feature module. // // Admin owns every /admin* sub-route: /admin & /admin/dashboard (overview // board), /admin/config/ (config-form pages), /admin/system[/] diff --git a/containers/libreportal/frontend/components/backup/index.js b/containers/libreportal/frontend/components/backup/index.js index 4d15482..1121955 100644 --- a/containers/libreportal/frontend/components/backup/index.js +++ b/containers/libreportal/frontend/components/backup/index.js @@ -1,4 +1,4 @@ -// features/backup/index.js — Backup Center as a self-contained feature module. +// components/backup/index.js — Backup Center as a self-contained feature module. // // FIRST page migrated to the feature-module contract (docs/frontend-modularization.md). // The kernel drives mount()/unmount() for the /backup route instead of diff --git a/containers/libreportal/frontend/components/dashboard/index.js b/containers/libreportal/frontend/components/dashboard/index.js index 4e325d2..d133e19 100644 --- a/containers/libreportal/frontend/components/dashboard/index.js +++ b/containers/libreportal/frontend/components/dashboard/index.js @@ -1,4 +1,4 @@ -// features/dashboard/index.js — the landing Dashboard as a feature module. +// components/dashboard/index.js — the landing Dashboard as a feature module. // // dashboard.js exposes bare global functions (loadInstalledApps, // loadDashboardData live in dashboard.js / data-loader.js, both eager core diff --git a/containers/libreportal/frontend/components/tasks/index.js b/containers/libreportal/frontend/components/tasks/index.js index 147ffef..b412c66 100644 --- a/containers/libreportal/frontend/components/tasks/index.js +++ b/containers/libreportal/frontend/components/tasks/index.js @@ -1,4 +1,4 @@ -// features/tasks/index.js — the Tasks page as a feature module. +// components/tasks/index.js — the Tasks page as a feature module. // // window.tasksManager is a system-loader singleton (its "task-system" component // loads all task scripts, starts the shared SSE bus, then news TasksManager). diff --git a/containers/libreportal/frontend/components/updater/index.js b/containers/libreportal/frontend/components/updater/index.js index 7459809..d174b76 100644 --- a/containers/libreportal/frontend/components/updater/index.js +++ b/containers/libreportal/frontend/components/updater/index.js @@ -1,4 +1,4 @@ -// features/updater/index.js — App Updater feature module. +// components/updater/index.js — App Updater feature module. // // Per-app version tracking + CVE/security scanning + disaster-recovery // (snapshot-before-update and rollback). Built in the same shape as the backup diff --git a/containers/libreportal/frontend/components/updater/js/updater-page.js b/containers/libreportal/frontend/components/updater/js/updater-page.js index 2ae50ae..38dbf5c 100644 --- a/containers/libreportal/frontend/components/updater/js/updater-page.js +++ b/containers/libreportal/frontend/components/updater/js/updater-page.js @@ -1,4 +1,4 @@ -// features/updater/updater-page.js — App Updater controller. +// components/updater/js/updater-page.js — App Updater controller. // // Surfaces, per installed app: version state (current -> available), security // posture (CVEs by severity), and disaster-recovery readiness (a snapshot is diff --git a/containers/libreportal/frontend/core/kernel/js/spa.js b/containers/libreportal/frontend/core/kernel/js/spa.js index 9e11c9d..a580e3a 100755 --- a/containers/libreportal/frontend/core/kernel/js/spa.js +++ b/containers/libreportal/frontend/core/kernel/js/spa.js @@ -256,7 +256,7 @@ class LibrePortalSPAClean { // (The dashboard data-reload that used to live here is now folded into the // dashboard feature module's mount(), so it fires exactly once per - // navigation. See features/dashboard/index.js.) + // navigation. See components/dashboard/index.js.) this.isLoading = true; //console.log('🧭 Navigating to:', path); diff --git a/containers/libreportal/frontend/core/tasks/js/task-router.js b/containers/libreportal/frontend/core/tasks/js/task-router.js index ebc8595..aaed16b 100755 --- a/containers/libreportal/frontend/core/tasks/js/task-router.js +++ b/containers/libreportal/frontend/core/tasks/js/task-router.js @@ -69,7 +69,7 @@ class TaskRouter { case 'tool': return await this.actions.runTool(params.appName, params.toolName, params.toolArgs, params.toolLabel); - // App Updater (features/updater) — version/CVE refresh + update/rollback, + // App Updater (components/updater) — version/CVE refresh + update/rollback, // each routed to the locked-down `libreportal updater` CLI as a task. case 'updater_check': return await this.actions.updaterCheck(); diff --git a/containers/libreportal/frontend/core/update-notifier/css/update-notifier.css b/containers/libreportal/frontend/core/update-notifier/css/update-notifier.css index c05446f..48d23e5 100644 --- a/containers/libreportal/frontend/core/update-notifier/css/update-notifier.css +++ b/containers/libreportal/frontend/core/update-notifier/css/update-notifier.css @@ -1,5 +1,5 @@ /* Update Notifier — topbar badge, dashboard banner, and details panel. - Driven by js/components/update-notifier.js. Colours come from the active + Driven by js/update-notifier.js. Colours come from the active theme tokens (with safe fallbacks) so it tracks every palette. */ /* ---- Topbar badge -------------------------------------------------------- */