fix(web-ui): address CodeRabbit review findings

- remove duplicate agentsPath key in app.js data()
- wrap localStorage.setItem for claudeConfigs in try/catch
- use locale-neutral weekdayKeys in logic layer, localize in computed
- guard weekdayLabels fallback and max division-by-zero
- normalize fallback tooltip tokenTotal to prevent null render

Author: SurviveM

tests/unit/session-usage.test.mjs

@@ -119,7 +119,7 @@ test('buildUsageHourlyHeatmap produces 7x24 grid with correct aggregation', () =
     assert.strictEqual(result.range, '7d');
     assert.strictEqual(result.grid.length, 7);
     assert.strictEqual(result.grid[0].length, 24);
-    assert.strictEqual(result.weekdayLabels.length, 7);
+    assert.strictEqual(result.weekdayKeys.length, 7);
     assert.strictEqual(result.hourLabels.length, 24);
 
     const april6Dow = (new Date(Date.UTC(2026, 3, 6)).getUTCDay() + 6) % 7;

web-ui/app.js

@@ -112,7 +112,6 @@ document.addEventListener('DOMContentLoaded', () => {
                 _pendingCodexApplyOptions: null,
                 agentsContent: '',
                 agentsPath: '',
-                agentsPath: '',
                 agentsExists: false,
                 agentsLineEnding: '\n',
                 agentsLoading: false,

web-ui/logic.sessions.mjs

@@ -352,7 +352,7 @@ export function buildUsageHourlyHeatmap(sessions = [], options = {}) {
         range,
         grid,
         maxSessionCount: Math.max(1, maxSessionCount),
-        weekdayLabels: ['周一', '周二', '周三', '周四', '周五', '周六', '周日'],
+        weekdayKeys: [0, 1, 2, 3, 4, 5, 6],
         hourLabels: Array.from({ length: 24 }, (_, index) => String(index).padStart(2, '0'))
     };
 }

web-ui/modules/app.computed.session.mjs

@@ -588,13 +588,13 @@ export function createSessionComputed() {
             const result = buildUsageHourlyHeatmap(sessions, { range: this.sessionsUsageTimeRange });
             const t = typeof this.t === 'function' ? this.t : null;
             const lang = typeof this.lang === 'string' ? this.lang.trim().toLowerCase() : '';
-            const weekdayLabels = lang === 'en'
-                ? ['Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat', 'Sun']
-                : result.weekdayLabels;
-            const max = result.maxSessionCount;
+            const weekdayLabelsZh = ['周一', '周二', '周三', '周四', '周五', '周六', '周日'];
+            const weekdayLabelsEn = ['Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat', 'Sun'];
+            const weekdayLabels = lang === 'en' ? weekdayLabelsEn : weekdayLabelsZh;
+            const max = Math.max(1, result.maxSessionCount);
             const grid = Array.isArray(result.grid) ? result.grid : [];
             const rows = grid.map((cells, dayIndex) => ({
-                weekday: weekdayLabels[dayIndex],
+                weekday: weekdayLabels[dayIndex] || '',
                 cells: cells.map((cell, hourIndex) => {
                     const ratio = cell.sessionCount > 0 ? (cell.sessionCount / max) : 0;
                     const level = cell.sessionCount <= 0
@@ -609,7 +609,7 @@ export function createSessionComputed() {
                             messages: cell.messageCount,
                             tokens: (cell.tokenTotal || 0).toLocaleString('en-US')
                         })
-                        : `${weekdayLabels[dayIndex]} ${hourLabel}:00 · ${cell.sessionCount} sessions · ${cell.messageCount} messages · ${cell.tokenTotal} tokens`;
+                        : `${weekdayLabels[dayIndex] || ''} ${hourLabel}:00 · ${cell.sessionCount} sessions · ${cell.messageCount} messages · ${(cell.tokenTotal || 0).toLocaleString('en-US')} tokens`;
                     return {
                         hour: hourIndex,
                         hourLabel,

web-ui/modules/app.methods.claude-config.mjs

@@ -37,7 +37,7 @@ export function createClaudeConfigMethods(options = {}) {
         },
 
         saveClaudeConfigs() {
-            localStorage.setItem('claudeConfigs', JSON.stringify(this.claudeConfigs));
+            try { localStorage.setItem('claudeConfigs', JSON.stringify(this.claudeConfigs)); } catch (_) {}
             if (this.currentClaudeConfig) {
                 try { localStorage.setItem('currentClaudeConfig', this.currentClaudeConfig); } catch (_) {}
             }