[codex] refine startup loading copy (#86)

* refine startup loading copy

* fix startup shell copy hydration

* refine startup shell accessibility

* harden startup shell config

Author: zouyonghe

scripts/prepare-resources/startup-shell-copy.test.mjs

@@ -0,0 +1,111 @@
+import { test } from 'node:test';
+import assert from 'node:assert/strict';
+import { readFile } from 'node:fs/promises';
+
+const startupShellPath = new URL('../../ui/index.html', import.meta.url);
+const startupCopyConfigPath = new URL('../../ui/startup-copy.js', import.meta.url);
+
+test('startup shell loads shared copy config, reuses applyStartupMode, and exposes a focused live region', async () => {
+  const source = await readFile(startupShellPath, 'utf8');
+  const configSource = await readFile(startupCopyConfigPath, 'utf8').catch(() => '');
+
+  assert.match(
+    source,
+    /<script src="\.\/startup-copy\.js"><\/script>/,
+    'expected startup shell to load a dedicated startup copy config',
+  );
+
+  assert.match(
+    source,
+    /<span id="startup-status"[^>]*role="status"[^>]*aria-live="polite"[^>]*><\/span>/,
+    'expected the status text element to be the polite live region',
+  );
+  assert.doesNotMatch(
+    source,
+    /aria-atomic="true"/,
+    'expected startup shell to avoid verbose atomic live region announcements',
+  );
+
+  assert.doesNotMatch(
+    source,
+    /<h1 id="startup-title" class="title">[^<]+<\/h1>/,
+    'expected startup title to be populated from STARTUP_COPY instead of duplicated static copy',
+  );
+  assert.doesNotMatch(
+    source,
+    /<p id="startup-desc" class="desc">[^<]+<\/p>/,
+    'expected startup description to be populated from STARTUP_COPY instead of duplicated static copy',
+  );
+  assert.doesNotMatch(
+    source,
+    /<span id="startup-status">[^<]+<\/span>/,
+    'expected startup status to be populated from STARTUP_COPY instead of duplicated static copy',
+  );
+
+  assert.doesNotMatch(
+    source,
+    /const\s+STARTUP_COPY\s*=/,
+    'expected startup copy to be defined in a dedicated config instead of inline',
+  );
+  assert.match(
+    source,
+    /const\s+startupShell\s*=\s*window\.astrbot\.startupShell;/,
+    'expected startup shell to read its shared config from a named astrbot namespace',
+  );
+  assert.match(
+    source,
+    /const\s+\{\s*STARTUP_MODES,\s*STARTUP_COPY\s*\}\s*=\s*startupShell;/,
+    'expected startup shell to destructure startup config from the namespaced object',
+  );
+  assert.doesNotMatch(
+    source,
+    /const\s+initialCopy\s*=/,
+    'expected initial render to reuse applyStartupMode instead of duplicating copy application',
+  );
+  assert.match(
+    source,
+    /applyStartupMode\(STARTUP_MODES\.LOADING\);/,
+    'expected startup shell to initialize through applyStartupMode',
+  );
+  assert.match(
+    source,
+    /if\s*\(status\.textContent\s*===\s*next\.status\)\s*return;/,
+    'expected startup shell to skip duplicate status announcements',
+  );
+
+  assert.match(
+    configSource,
+    /const\s+deepFreeze\s*=\s*\(obj\)\s*=>/,
+    'expected shared startup copy config to use a deepFreeze helper',
+  );
+  assert.match(
+    configSource,
+    /const\s+STARTUP_MODES\s*=\s*\{/,
+    'expected shared startup copy config to define startup modes',
+  );
+  assert.match(
+    configSource,
+    /window\.astrbot\s*=\s*window\.astrbot\s*\|\|\s*\{\};/,
+    'expected shared startup copy config to allocate the astrbot namespace',
+  );
+  assert.match(
+    configSource,
+    /window\.astrbot\.startupShell\s*=\s*deepFreeze\(/,
+    'expected shared startup copy config to expose startup shell under the astrbot namespace',
+  );
+  assert.match(
+    configSource,
+    /STARTUP_COPY:\s*\{/,
+    'expected shared startup copy config to define localized startup copy',
+  );
+  assert.match(
+    configSource,
+    /en:\s*\{/,
+    'expected shared startup copy config to include English startup copy',
+  );
+  assert.match(
+    configSource,
+    /zh:\s*\{/,
+    'expected shared startup copy config to include Chinese startup copy',
+  );
+});

ui/index.html

@@ -133,56 +133,28 @@
     <main class="shell">
       <section class="panel">
         <img class="brand" src="./astrbot-logo.png" alt="AstrBot Logo" />
-        <h1 id="startup-title" class="title">AstrBot Loading</h1>
-        <p id="startup-desc" class="desc">Initializing runtime, please wait.</p>
+        <h1 id="startup-title" class="title"></h1>
+        <p id="startup-desc" class="desc"></p>
         <div class="bar-wrap" aria-hidden="true">
           <div class="bar"></div>
         </div>
         <div class="status">
           <span class="dot" aria-hidden="true"></span>
-          <span id="startup-status">Starting AstrBot...</span>
+          <span id="startup-status" role="status" aria-live="polite"></span>
         </div>
       </section>
     </main>
+    <script src="./startup-copy.js"></script>
     <script>
       (() => {
         const title = document.getElementById("startup-title");
         const desc = document.getElementById("startup-desc");
         const status = document.getElementById("startup-status");
         if (!title || !desc || !status) return;
+        if (!window.astrbot || !window.astrbot.startupShell) return;
 
-        // Keep in sync with startup mode constants in src-tauri/src/startup_mode.rs.
-        const STARTUP_MODES = Object.freeze({
-          LOADING: "loading",
-          PANEL_UPDATE: "panel-update",
-        });
-
-        const STARTUP_COPY = {
-          en: {
-            [STARTUP_MODES.LOADING]: {
-              title: "AstrBot Loading",
-              desc: "Initializing runtime, please wait.",
-              status: "Starting AstrBot...",
-            },
-            [STARTUP_MODES.PANEL_UPDATE]: {
-              title: "AstrBot Updating Panel",
-              desc: "Panel updates detected. Downloading and applying now, please keep the app open.",
-              status: "Downloading AstrBot panel updates...",
-            },
-          },
-          zh: {
-            [STARTUP_MODES.LOADING]: {
-              title: "AstrBot 正在加载",
-              desc: "正在启动，请稍候。",
-              status: "正在启动 AstrBot...",
-            },
-            [STARTUP_MODES.PANEL_UPDATE]: {
-              title: "AstrBot 正在更新面板",
-              desc: "检测到面板更新，正在下载并应用，请耐心等待，不要关闭应用。",
-              status: "正在下载 AstrBot 面板更新...",
-            },
-          },
-        };
+        const startupShell = window.astrbot.startupShell;
+        const { STARTUP_MODES, STARTUP_COPY } = startupShell;
 
         const resolveLocaleKey = () => {
           const locale =
@@ -202,6 +174,7 @@ <h1 id="startup-title" class="title">AstrBot Loading</h1>
           const next = resolveStartupCopy(mode);
           title.textContent = next.title;
           desc.textContent = next.desc;
+          if (status.textContent === next.status) return;
           status.textContent = next.status;
         };
 

ui/startup-copy.js

@@ -0,0 +1,46 @@
+const deepFreeze = (obj) => {
+  for (const key of Object.getOwnPropertyNames(obj)) {
+    const value = obj[key];
+    if (value && typeof value === 'object' && !Object.isFrozen(value)) {
+      deepFreeze(value);
+    }
+  }
+
+  return Object.freeze(obj);
+};
+
+const STARTUP_MODES = {
+  LOADING: 'loading',
+  PANEL_UPDATE: 'panel-update',
+};
+
+window.astrbot = window.astrbot || {};
+window.astrbot.startupShell = deepFreeze({
+  STARTUP_MODES,
+  STARTUP_COPY: {
+    en: {
+      [STARTUP_MODES.LOADING]: {
+        title: 'AstrBot Desktop',
+        desc: 'Preparing the runtime, please wait.',
+        status: 'Starting core services...',
+      },
+      [STARTUP_MODES.PANEL_UPDATE]: {
+        title: 'AstrBot Desktop',
+        desc: 'A new panel version is available and is being downloaded and applied.',
+        status: 'Syncing panel assets...',
+      },
+    },
+    zh: {
+      [STARTUP_MODES.LOADING]: {
+        title: 'AstrBot Desktop',
+        desc: '正在准备运行环境，请稍候。',
+        status: '正在启动核心服务...',
+      },
+      [STARTUP_MODES.PANEL_UPDATE]: {
+        title: 'AstrBot Desktop',
+        desc: '检测到新面板版本，正在下载并应用。',
+        status: '正在同步面板资源...',
+      },
+    },
+  },
+});