diff --git a/client/electron/src/lib/appearance.ts b/client/electron/src/lib/appearance.ts index 3b5edd6..94961cb 100644 --- a/client/electron/src/lib/appearance.ts +++ b/client/electron/src/lib/appearance.ts @@ -5,7 +5,9 @@ * * 2026-09-13 用户的质问：「为什么背景是保存在本地而不是服务器！」当时的实情是 * 主题与壁纸只写 localStorage：换设备/换浏览器就没了，而且**多账号共用一份** - * （键是全局常量 `agentmail.background`）—— 同一台机器换账号，背景不跟着走。 + * （当时键是全局常量 `agentmail.background`）—— 同一台机器换账号，背景不跟着走。 + * 2026-09-14 已按账号分键（见 `stores/backgroundStore.ts` 的 `storageKey`）， + * 这条缺陷在 WebUI 侧也修掉了。 * * 现在的分工： * - **服务端**是权威（账号级，`/api/v1/me/appearance`）； @@ -17,6 +19,7 @@ */ import { fetchWithAuth } from '../api/config'; +import { BLUR_MAX, DEFAULT_BLUR, DEFAULT_DIM, DIM_MAX } from './appearanceDefaults'; /** 服务端外观的形状（字段名与 handlers/appearance.go 的 JSON 一致）。 */ export interface AppearanceResponse { @@ -65,8 +68,8 @@ export function snapshotFromResponse(resp: AppearanceResponse | null | undefined theme: (THEMES.has(String(r.theme)) ? r.theme : 'system') as AppearanceSnapshot['theme'], bgKind: (KINDS.has(String(r.bg_kind)) ? r.bg_kind : 'none') as AppearanceSnapshot['bgKind'], bgPresetId: typeof r.bg_preset_id === 'string' && r.bg_preset_id ? r.bg_preset_id : 'aurora', - bgDim: clamp(r.bg_dim, 0, 90, 12), - bgBlur: clamp(r.bg_blur, 0, 40, 4) + bgDim: clamp(r.bg_dim, 0, DIM_MAX, DEFAULT_DIM), + bgBlur: clamp(r.bg_blur, 0, BLUR_MAX, DEFAULT_BLUR) }; } @@ -84,8 +87,8 @@ export function payloadFromLocal(input: { theme: THEMES.has(input.theme) ? input.theme : 'system', bg_kind: kind === 'image' && !input.imageDataUrl ? 'none' : kind, bg_preset_id: input.presetId || 'aurora', - bg_dim: clamp(input.dim, 0, 90, 12), - bg_blur: clamp(input.blur, 0, 40, 4) + bg_dim: clamp(input.dim, 0, DIM_MAX, DEFAULT_DIM), + bg_blur: clamp(input.blur, 0, BLUR_MAX, DEFAULT_BLUR) }; } diff --git a/client/electron/src/lib/appearanceDefaults.ts b/client/electron/src/lib/appearanceDefaults.ts new file mode 100644 index 0000000..7ea4531 --- /dev/null +++ b/client/electron/src/lib/appearanceDefaults.ts @@ -0,0 +1,35 @@ +/** + * 外观的**默认值**：唯一来源，两个客户端与一份服务端契约共用同一个数。 + * + * # 为什么单独一个模块 + * + * 之前有两个地方各写一份：`stores/backgroundStore.ts` 的 `DEFAULT_BACKGROUND` + * 是 `dim: 24, blur: 8`，而 `lib/appearance.ts` 里 `clamp(..., 12, 4)` 的兜底是 `12 / 4`。 + * 同一份代码里两个"默认值"，**调用路径不同就落到不同的数**。 + * + * 更关键的是：**权威值不在这两个文件里，而在服务端**。 + * `server/internal/models/models.go` 的 `DefaultAppearance()` 明写 `BgDim: 12, BgBlur: 4`， + * 并且注释宣称"与客户端 backgroundStore / themeStore 的默认值一致" —— + * 那句注释在 WebUI 用 24/8 时**是假的**（pi 2026-09-14 指出，这是他对自己上一封"数值是审美" + * 的更正：它不是审美问题，是**契约**问题）。 + * + * 后果不是"两处代码不一样"这么轻：服务端"没有记录"时客户端走"以本地为准并推上去"， + * 于是**新账号的初始外观由第一个同步它的客户端决定** —— + * 先用 WebUI 登录存 24/8，先用鸿蒙登录存 12/4。同一个账号，压暗强度取决于谁先到。 + * + * 所以这里定义一次，`lib/appearance.ts` 与 `stores/backgroundStore.ts` 都引用它； + * 判据 `test/appearance-defaults.test.mjs` 会**去 Go 源码里读**这两个数并比对， + * 这样服务端那句注释才是可核对的（不是承诺，是判据）。 + */ + +/** 压暗强度默认值（%）。与服务端 `DefaultAppearance().BgDim` 必须相等 */ +export const DEFAULT_DIM = 12; + +/** 模糊强度默认值（px）。与服务端 `DefaultAppearance().BgBlur` 必须相等 */ +export const DEFAULT_BLUR = 4; + +/** 压暗的合法上限（%）。服务端 `NormalizeAppearance` 用同一区间夹取值 */ +export const DIM_MAX = 90; + +/** 模糊的合法上限（px） */ +export const BLUR_MAX = 40; diff --git a/client/electron/src/stores/appearanceSync.ts b/client/electron/src/stores/appearanceSync.ts index 6c1ee9d..3e7103c 100644 --- a/client/electron/src/stores/appearanceSync.ts +++ b/client/electron/src/stores/appearanceSync.ts @@ -143,7 +143,18 @@ export function initAppearanceSync(): () => void { const unsubBg = useBackgroundStore.subscribe(schedulePush); const unsubTheme = useThemeStore.subscribe(schedulePush); const unsubAccount = useAccountStore.subscribe((state, prev) => { - if (state.activeId !== prev.activeId) void useAppearanceSync.getState().pull(); + if (state.activeId !== prev.activeId) { + /* + * 顺序：**先按新账号重读本地缓存，再拉服务端**。 + * + * 为什么不能省这一步：服务端"没有记录"（`saved === false`）时 `pull()` 会 + * 走"以本地为准推上去"——那时"本地"必须已经是**这个账号**的值。 + * 全局键时代这里是上一个账号的值，于是新账号凭空继承了别人的外观， + * 而且被写进了服务端（pi 2026-09-14 复核时点出的那条缺陷）。 + */ + useBackgroundStore.getState().reloadForAccount(); + void useAppearanceSync.getState().pull(); + } }); return () => { unsubBg(); diff --git a/client/electron/src/stores/backgroundStore.ts b/client/electron/src/stores/backgroundStore.ts index 87a453e..81a16ce 100644 --- a/client/electron/src/stores/backgroundStore.ts +++ b/client/electron/src/stores/backgroundStore.ts @@ -1,5 +1,8 @@ import { create } from 'zustand'; +import { AGGREGATE_ID, isUsableAccount, useAccountStore } from './accountStore'; +import { DEFAULT_BLUR, DEFAULT_DIM } from '../lib/appearanceDefaults'; + /** * 自定义背景。 * @@ -38,7 +41,43 @@ export interface BackgroundState { blur: number; } -export const STORAGE_KEY = 'agentmail.background'; +/** + * 旧版本用的**全局**键。现在只用于**一次性迁移**（见 `readStored`）。 + * + * ⚠️ 不要再往它写东西：全局键的后果是"切到服务端没有记录的账号"时， + * `saved=false` 分支会把**上一个账号的外观**推上去（新账号"继承"了外观， + * 而且写进了服务端）。鸿蒙侧一直是按账号分键的，这是 WebUI 侧的缺陷 + * （pi 2026-09-14 复核时点名："鸿蒙是对的，别为对齐退回全局键"）。 + */ +export const LEGACY_STORAGE_KEY = 'agentmail.background'; + +/** 按账号的键前缀：实际键是 `<前缀>` */ +export const STORAGE_KEY_PREFIX = 'agentmail.background.'; + +/** + * 本机当前**用于同步的那个账号**。 + * + * 与 `accountStore.syncAuth()` 取同一个目标：聚合模式用第一个可用账号 + * （发信要有身份），否则用当前选中的账号。取不到（未登录/首屏）时返回空串， + * 此时用一个匿名兜底键 —— 不能退回全局键，那正是要修的东西。 + */ +export function activeAppearanceAccountId(): string { + try { + const st = useAccountStore.getState(); + const target = + st.activeId === AGGREGATE_ID + ? st.accounts.find(a => isUsableAccount(a)) + : st.accounts.find(a => a.id === st.activeId); + return target?.id ?? ''; + } catch { + return ''; + } +} + +/** 缓存键：**按账号**。传 accountId 便于判据与迁移直接验证 */ +export function storageKey(accountId: string = activeAppearanceAccountId()): string { + return accountId ? `${STORAGE_KEY_PREFIX}${accountId}` : `${STORAGE_KEY_PREFIX}anonymous`; +} /** 预设清单。**渐变的实际色值定义在 index.css**，这里只有 id 与显示名。 */ export const PRESETS: { id: string; label: string }[] = [ @@ -50,12 +89,18 @@ export const PRESETS: { id: string; label: string }[] = [ { id: 'mesh', label: '网格' } ]; +/** + * 默认背景。**数值来自服务端契约**（`server/internal/models/models.go` 的 + * `DefaultAppearance()`：`BgDim: 12, BgBlur: 4`），而不再是这里自己写一套 —— + * 原来这里是 `24 / 8`，与 `lib/appearance.ts` 的兜底 `12 / 4` 不一致， + * 于是"新账号的初始外观由第一个同步它的客户端决定"（详见 `lib/appearanceDefaults.ts`）。 + */ export const DEFAULT_BACKGROUND: BackgroundState = { kind: 'none', presetId: 'aurora', imageDataUrl: '', - dim: 24, - blur: 8 + dim: DEFAULT_DIM, + blur: DEFAULT_BLUR }; /** 图片最长边。超过就等比缩小 —— 背景是满屏铺开的，再大也看不出来。 */ @@ -105,9 +150,25 @@ export function normalizeBackground( }; } -export function readStored(): BackgroundState { +/** + * 读本机缓存。**按账号**读，并做一次旧全局键的迁移。 + * + * 迁移只做一次、只给**当前账号**：旧值属于"这台机器上当时那个账号"， + * 把它送给当前账号是合理的（用户的观感不会凭空消失），然后**立刻删掉旧键** —— + * 否则下一个账号又会从它那里"继承"，等于把刚修掉的缺陷留在原地。 + */ +export function readStored(accountId: string = activeAppearanceAccountId()): BackgroundState { try { - const raw = localStorage.getItem(STORAGE_KEY); + const key = storageKey(accountId); + let raw = localStorage.getItem(key); + if (!raw) { + const legacy = accountId ? localStorage.getItem(LEGACY_STORAGE_KEY) : null; + if (legacy) { + localStorage.setItem(key, legacy); + localStorage.removeItem(LEGACY_STORAGE_KEY); + raw = legacy; + } + } if (!raw) return DEFAULT_BACKGROUND; return normalizeBackground(JSON.parse(raw)); } catch { @@ -155,11 +216,14 @@ interface BackgroundStore extends BackgroundState { setDim: (v: number) => void; setBlur: (v: number) => void; reset: () => void; + /** 切换账号后按新账号重读本地缓存（必须在 pull() 之前） */ + reloadForAccount: () => void; } function persist(state: BackgroundState) { try { - localStorage.setItem(STORAGE_KEY, JSON.stringify(state)); + // 按账号写：换账号后落到各自的键，互不串味 + localStorage.setItem(storageKey(), JSON.stringify(state)); } catch { // 配额满：本次会话仍生效，只是下次打开会退回默认值。 // 不抛给调用方 —— 背景是装饰，不该让「换背景失败」打断任何操作。 @@ -185,7 +249,21 @@ export const useBackgroundStore = create((set, get) => { setImage: imageDataUrl => commit({ imageDataUrl, kind: 'image' }), setDim: dim => commit({ dim: clampDim(dim) }), setBlur: blur => commit({ blur: clampBlur(blur) }), - reset: () => commit({ ...DEFAULT_BACKGROUND }) + reset: () => commit({ ...DEFAULT_BACKGROUND }), + + /** + * 切换账号后**重读本账号的本地缓存**。 + * + * 顺序很重要：重读必须发生在 `pull()` **之前** —— 服务端"没有记录"时 + * 调用方会"以本地为准推上去"，而那时"本地"必须已经是**这个账号**的值， + * 不能还是上一个账号的（那正是全局键时代的缺陷：新账号继承上一个人的外观）。 + */ + reloadForAccount: () => { + const next: BackgroundState = readStored(); + // 不 persist：这只是"把本账号已有的缓存读回来"，不是用户的改动 + applyBackground(next); + set(next); + } }; }); diff --git a/client/electron/test/appearance-defaults.test.mjs b/client/electron/test/appearance-defaults.test.mjs new file mode 100644 index 0000000..7f13eb7 --- /dev/null +++ b/client/electron/test/appearance-defaults.test.mjs @@ -0,0 +1,153 @@ +// 外观的**服务端契约**与**本机缓存的键**：两个客户端的默认值必须等于服务端那三个数， +// 缓存键必须按账号分。 +// +// 为什么值得单独一个判据文件： +// 1. 服务端 `DefaultAppearance()` 的注释宣称"与客户端 backgroundStore / themeStore 的 +// 默认值一致" —— 在 WebUI 用 24/8 时那句话**是假的**（pi 2026-09-14 更正了自己 +// 上一封"数值是审美"的说法：它是契约问题）。这条判据把那句注释变成**可核对**的。 +// 2. 默认值决定"新账号的初始外观"：服务端"没有记录"时客户端以本地为准推上去， +// 于是**谁先同步谁决定**。24/8 与 12/4 的差别不是审美，是同一个账号在不同客户端 +// 登录会得到不同的压暗强度。 +// 3. 缓存键是"换账号串味"的成因（全局键 → saved=false 时把上一个账号的外观推上去）。 +import { readFileSync } from 'node:fs'; +import { dirname, join } from 'node:path'; +import { fileURLToPath } from 'node:url'; +import { test } from 'node:test'; +import assert from 'node:assert/strict'; + +// 与其它判据同一个约定：HERE = client/electron/test，ROOT = 仓库根 +const HERE = dirname(fileURLToPath(import.meta.url)); +const ROOT = join(HERE, '..', '..', '..'); +const SERVER_MODELS = join(ROOT, 'server/internal/models/models.go'); +const ETS = join(ROOT, 'client/harmony/entry/src/main/ets'); + +/** 从 Go 源码里读服务端契约（不复制一份数字到判据里 —— 那又会变成"两处各写一套"） */ +function serverDefaults() { + const src = readFileSync(SERVER_MODELS, 'utf8'); + // 只取 DefaultAppearance 函数体，避免匹配到别的结构体字面量 + const at = src.indexOf('func DefaultAppearance()'); + assert.ok(at > 0, '服务端要有 DefaultAppearance()'); + let depth = 0; + let end = at; + for (let i = src.indexOf('{', at); i < src.length; i++) { + if (src[i] === '{') depth++; + else if (src[i] === '}') { depth--; if (depth === 0) { end = i; break; } } + } + const body = src.slice(at, end + 1); + const dim = /BgDim:\s*(\d+)/.exec(body); + const blur = /BgBlur:\s*(\d+)/.exec(body); + const theme = /Theme:\s*"([^"]+)"/.exec(body); + const kind = /BgKind:\s*"([^"]+)"/.exec(body); + const preset = /BgPresetID:\s*"([^"]+)"/.exec(body); + assert.ok(dim && blur && theme && kind && preset, `DefaultAppearance 里这几个字段都要有（实际：${body}）`); + return { + dim: Number(dim[1]), + blur: Number(blur[1]), + theme: theme[1], + kind: kind[1], + preset: preset[1] + }; +} + +test('★ 默认外观 = 服务端契约（去 Go 源码里读，不在判据里写死数字）', () => { + const s = serverDefaults(); + assert.deepEqual([s.dim, s.blur], [12, 4], '服务端 DefaultAppearance 是 12/4（权威值）'); + + // WebUI：默认值必须来自共享常量，且等于服务端 + const defaults = readFileSync(join(ROOT, 'client/electron/src/lib/appearanceDefaults.ts'), 'utf8'); + assert.match(defaults, new RegExp(`DEFAULT_DIM = ${s.dim}\\b`), `DEFAULT_DIM 要等于服务端的 BgDim=${s.dim}`); + assert.match(defaults, new RegExp(`DEFAULT_BLUR = ${s.blur}\\b`), `DEFAULT_BLUR 要等于服务端的 BgBlur=${s.blur}`); + // 严格：store 里不许再自写一套数字（原来这里写的是 24/8，与服务端不一致） + const store = readFileSync(join(ROOT, 'client/electron/src/stores/backgroundStore.ts'), 'utf8'); + const defBlock = /export const DEFAULT_BACKGROUND: BackgroundState = \{[\s\S]*?\};/.exec(store); + assert.ok(defBlock, 'DEFAULT_BACKGROUND 要能取到'); + assert.match(defBlock[0], /dim:\s*DEFAULT_DIM/, 'dim 要引用共享常量，不许自写字面量'); + assert.match(defBlock[0], /blur:\s*DEFAULT_BLUR/, 'blur 要引用共享常量'); + assert.ok(!/dim:\s*\d/.test(defBlock[0]), `DEFAULT_BACKGROUND 里不许再出现数字字面量（现在：${defBlock[0].replace(/\s+/g, ' ')}）`); + // 同步层的兜底值也要引用同一常量（原来 clamp(..., 12, 4) 是另写的一份） + const lib = readFileSync(join(ROOT, 'client/electron/src/lib/appearance.ts'), 'utf8'); + const clamps = [...lib.matchAll(/clamp\([^)]*\)/g)].map(m => m[0]); + assert.ok(clamps.length >= 4, `clamp 调用要能取到（实际 ${clamps.length} 处）`); + for (const c of clamps) { + assert.ok(!/,\s*\d+\s*\)$/.test(c), `clamp 的兜底值要引用常量，不许写死：${c}`); + } + + // 鸿蒙：默认值（在 AppearanceResponse 的字段默认值里）与预设也要与服务端对得上 + const ap = readFileSync(join(ETS, 'model/Appearance.ts'), 'utf8'); + assert.match(ap, new RegExp(`bg_dim: number = ${s.dim}\\b`), `鸿蒙默认压暗要等于服务端 BgDim=${s.dim}`); + assert.match(ap, new RegExp(`bg_blur: number = ${s.blur}\\b`), `鸿蒙默认模糊要等于服务端 BgBlur=${s.blur}`); + assert.match(ap, new RegExp(`theme: string = '${s.theme}'`), `鸿蒙默认主题要等于服务端 Theme=${s.theme}`); + const wp = readFileSync(join(ETS, 'model/Wallpaper.ts'), 'utf8'); + assert.match(wp, new RegExp(`'${s.preset}'`), `服务端默认预设 ${s.preset} 要在鸿蒙的预设清单里`); + + // 服务端那句注释现在是**事实**（它宣称"与客户端默认值一致"）—— 这条就是它的核对器 + const models = readFileSync(SERVER_MODELS, 'utf8'); + const comment = /\/\/ DefaultAppearance[\s\S]{0,200}?func DefaultAppearance/.exec(models); + assert.ok(comment, 'DefaultAppearance 上面要有说明注释'); + assert.match(comment[0], /一致/, '注释仍在宣称"与客户端默认值一致"（本判据负责让它为真）'); +}); + +test('★ 缓存键按账号分：两端的键都带账号，且都不许退回全局键', () => { + const store = readFileSync(join(ROOT, 'client/electron/src/stores/backgroundStore.ts'), 'utf8'); + // 键函数：必须把 accountId 拼进去（按块取函数体，不看调用点 —— "配对/解析"那条） + const at = store.indexOf('export function storageKey('); + assert.ok(at > 0, '要有 storageKey 函数'); + let depth = 0; + let end = at; + for (let i = store.indexOf('{', at); i < store.length; i++) { + if (store[i] === '{') depth++; + else if (store[i] === '}') { depth--; if (depth === 0) { end = i; break; } } + } + const body = store.slice(at, end + 1); + assert.match(body, /STORAGE_KEY_PREFIX[\s\S]*accountId|accountId[\s\S]*STORAGE_KEY_PREFIX/, + `取键函数必须把账号拼进键里（现在：${body.replace(/\s+/g, ' ')}）`); + /* + * 不许写死键名：写入的键只能是 `storageKey()`，或由它算出来的变量（迁移时写 `key`）。 + * 我第一版写成"只允许字面量 setItem(storageKey(" —— 结果把迁移那次合法写入 + * （`setItem(key, legacy)`，key 就是按账号的键）也判红了；判据要钉**键的来源**， + * 不是调用的字面形状。 + */ + const badWrites = [...store.matchAll(/setItem\(([^,)]+)/g)] + .map(m => m[1].trim()) + .filter(k => !/^storageKey\(/.test(k) && k !== 'key'); + assert.deepEqual(badWrites, [], `写入的键必须来自 storageKey()（现在这些不是：${badWrites.join('、')}）`); + assert.match(store, /removeItem\(LEGACY_STORAGE_KEY\)/, '旧全局键要被删除（否则下一个账号继续从它继承）'); + // 读/写都要走按账号的键（`key` 由 storageKey() 算出，见函数开头） + assert.match(store, /const key = storageKey\(accountId\)/, '键要先按账号算出来'); + assert.match(store, /let raw = localStorage\.getItem\(key\)/, '读缓存要按账号的键'); + assert.match(store, /localStorage\.setItem\(storageKey\(\), JSON\.stringify\(state\)\)/, '写缓存要按账号的键'); + + // 鸿蒙侧：键同样带账号（两边形状一致，这条差异已经消除） + const ets = readFileSync(join(ETS, 'common/AppearanceStore.ets'), 'utf8'); + const eAt = ets.indexOf('prefKey(accountId: string)'); + assert.ok(eAt > 0, '鸿蒙要有按账号取键的函数'); + let d2 = 0; + let e2 = eAt; + for (let i = ets.indexOf('{', eAt); i < ets.length; i++) { + if (ets[i] === '{') d2++; + else if (ets[i] === '}') { d2--; if (d2 === 0) { e2 = i; break; } } + } + assert.match(ets.slice(eAt, e2 + 1), /KEY_PREFIX\s*\+\s*accountId/, '鸿蒙的键也要拼账号'); +}); + +test('★ 切账号的顺序：**先按新账号重读本地**，再拉服务端', () => { + /* + * 服务端"没有记录"时 `pull()` 会"以本地为准推上去"——所以重读必须在前， + * 否则推上去的是上一个账号的外观（全局键时代就是这个现象，而且写进了服务端）。 + */ + const sync = readFileSync(join(ROOT, 'client/electron/src/stores/appearanceSync.ts'), 'utf8'); + const sub = /activeId !== prev\.activeId\)[\s\S]{0,400}?\}\);/.exec(sync); + assert.ok(sub, '要能取到账号切换的处理块'); + const block = sub[0]; + const reloadAt = block.indexOf('reloadForAccount()'); + const pullAt = block.indexOf('.pull()'); + assert.ok(reloadAt > 0, `切换账号要先重读本账号的缓存：${block.replace(/\s+/g, ' ')}`); + assert.ok(pullAt > 0, '切换账号仍然要拉服务端'); + assert.ok(reloadAt < pullAt, 'reloadForAccount() 必须在 pull() 之前（顺序就是这条判据的全部意义）'); + // 重读本身不许落盘（它只是把本账号已有的值读回来，不是用户的改动） + const store = readFileSync(join(ROOT, 'client/electron/src/stores/backgroundStore.ts'), 'utf8'); + const rAt = store.indexOf('reloadForAccount: () => {'); + assert.ok(rAt > 0, 'store 要提供 reloadForAccount'); + const rBlock = store.slice(rAt, store.indexOf('}', store.indexOf('set(next)', rAt))); + assert.ok(!/persist\(/.test(rBlock), '重读不许写盘'); +}); diff --git a/client/electron/test/components/BackgroundPicker.test.tsx b/client/electron/test/components/BackgroundPicker.test.tsx index da33e9e..a64700c 100644 --- a/client/electron/test/components/BackgroundPicker.test.tsx +++ b/client/electron/test/components/BackgroundPicker.test.tsx @@ -15,7 +15,7 @@ import { render, screen, waitFor } from '@testing-library/react'; import userEvent from '@testing-library/user-event'; import BackgroundPicker from '../../src/components/BackgroundPicker'; -import { STORAGE_KEY, useBackgroundStore } from '../../src/stores/backgroundStore'; +import { LEGACY_STORAGE_KEY, storageKey, useBackgroundStore } from '../../src/stores/backgroundStore'; import * as store from '../../src/stores/backgroundStore'; describe('BackgroundPicker', () => { @@ -54,7 +54,7 @@ describe('BackgroundPicker', () => { await waitFor(() => expect(document.documentElement.dataset.bg).toBe('on')); expect(document.documentElement.style.getPropertyValue('--bg-image')).toContain('ZZZZ'); - expect(JSON.parse(localStorage.getItem(STORAGE_KEY)!).kind).toBe('image'); + expect(JSON.parse(localStorage.getItem(storageKey(''))!).kind).toBe('image'); }); it('图片准备失败时说出原因，并且不假装成功', async () => { diff --git a/client/electron/test/harmony-appearance.test.mjs b/client/electron/test/harmony-appearance.test.mjs index b967e94..f5fa4c8 100644 --- a/client/electron/test/harmony-appearance.test.mjs +++ b/client/electron/test/harmony-appearance.test.mjs @@ -621,9 +621,15 @@ test('★ 多账号缓存键：**按账号**分（鸿蒙是对的，不许为" assert.ok(!/AGENTMAIL_BACKGROUND|'agentmail\.background'/.test(store), '不要退回 WebUI 那个全局键（那正是"换账号继承上一个人的外观"的成因）'); assert.match(store, /loadLocal\([^)]*accountId/, '读缓存要按账号'); - // WebUI 侧的前提也钉一下：它确实是全局键（这条差异有据可查） - const webLib = readFileSync(join(ROOT, 'client/electron/src/lib/appearance.ts'), 'utf8'); - assert.match(webLib, /agentmail\.background/, 'WebUI 现在是全局键（差异记录的依据）'); + /* + * ⚠️ 这条判据原来还断言"WebUI 是全局键"（作为差异记录的依据）。 + * 2026-09-14 WebUI 侧也按账号分键了（dsh 接手 pi 的两个开项），**差异已消除** —— + * 所以现在断言的是"两端都是按账号的键"，而且不允许任何一端退回全局键。 + */ + const webStore = readFileSync(join(ROOT, 'client/electron/src/stores/backgroundStore.ts'), 'utf8'); + assert.match(webStore, /export function storageKey\(/, 'WebUI 也要有按账号取键的函数'); + assert.match(webStore, /LEGACY_STORAGE_KEY/, '旧全局键只作为迁移源存在'); + assert.ok(!/setItem\('agentmail\.background'/.test(webStore), 'WebUI 也不许再往全局键写'); }); test('★ 主题变化时**我们自己算的值**要跟着重算（pi 的规则：系统只跟它自己那部分）', () => { diff --git a/client/electron/test/run-all.mjs b/client/electron/test/run-all.mjs index 5b7400c..456cf0c 100644 --- a/client/electron/test/run-all.mjs +++ b/client/electron/test/run-all.mjs @@ -67,6 +67,8 @@ const SUITE = [ ['test/harmony-system-api.test.mjs', [], 5], // P4 外观同步：跑 model/Appearance.ts（纯逻辑），所以也要 strip-types ['test/harmony-appearance.test.mjs', ['--experimental-strip-types', '--no-warnings'], 23], + // 外观契约：默认值去 Go 源码里读（服务端 DefaultAppearance 是权威）+ 缓存键按账号 + ['test/appearance-defaults.test.mjs', [], 3], ['test/build-stamp.test.mjs', [], 5], ['test/packaging.test.mjs', [], 3], ['test/commit-hygiene.test.mjs', ['--experimental-strip-types', '--no-warnings'], 2] diff --git a/client/electron/test/stores/background.test.ts b/client/electron/test/stores/background.test.ts index de6b189..db1bff6 100644 --- a/client/electron/test/stores/background.test.ts +++ b/client/electron/test/stores/background.test.ts @@ -1,13 +1,14 @@ import { beforeEach, describe, expect, it } from 'vitest'; import { DEFAULT_BACKGROUND, + LEGACY_STORAGE_KEY, MAX_DATA_URL_BYTES, - STORAGE_KEY, + readStored, + storageKey, applyBackground, clampBlur, clampDim, normalizeBackground, - readStored, useBackgroundStore } from '../../src/stores/backgroundStore'; @@ -77,7 +78,7 @@ describe('脏数据归一化', () => { }); it('★ 瞬态里的空图片状态被持久化后，重读必须退回 none（静止态不放松）', () => { - localStorage.setItem(STORAGE_KEY, JSON.stringify({ kind: 'image', imageDataUrl: '', dim: 30, blur: 10 })); + localStorage.setItem(storageKey(''), JSON.stringify({ kind: 'image', imageDataUrl: '', dim: 30, blur: 10 })); const back = readStored(); expect(back.kind).toBe('none'); // 用户接着选图片仍能成功（不是"坏状态卡死"） @@ -155,13 +156,13 @@ describe('store 动作', () => { const s = useBackgroundStore.getState(); expect(s.kind).toBe('preset'); expect(s.presetId).toBe('sand'); - expect(JSON.parse(localStorage.getItem(STORAGE_KEY)!).presetId).toBe('sand'); + expect(JSON.parse(localStorage.getItem(storageKey(''))!).presetId).toBe('sand'); }); it('setImage 记录图片并落盘', () => { useBackgroundStore.getState().setImage('data:image/jpeg;base64,DDDD'); expect(useBackgroundStore.getState().kind).toBe('image'); - expect(JSON.parse(localStorage.getItem(STORAGE_KEY)!).imageDataUrl).toContain('DDDD'); + expect(JSON.parse(localStorage.getItem(storageKey(''))!).imageDataUrl).toContain('DDDD'); }); it('reset 回到默认并落盘', () => { @@ -170,7 +171,7 @@ describe('store 动作', () => { const s = useBackgroundStore.getState(); expect(s.kind).toBe('none'); expect(s.presetId).toBe(DEFAULT_BACKGROUND.presetId); - expect(JSON.parse(localStorage.getItem(STORAGE_KEY)!).kind).toBe('none'); + expect(JSON.parse(localStorage.getItem(storageKey(''))!).kind).toBe('none'); }); it('localStorage 抛异常时不打断操作', () => { @@ -191,3 +192,53 @@ describe('上限常量', () => { expect(MAX_DATA_URL_BYTES).toBeLessThan(5_000_000 / 2); }); }); + +describe('按账号分键（★ 这条修的是"新账号继承上一个账号的外观"）', () => { + it('键按账号区分，互不串味', () => { + // pi 2026-09-14 复核时点名：全局键会让"切到服务端没有记录的账号"时， + // saved=false 分支把**上一个账号的外观**推上去（而且写进了服务端）。 + expect(storageKey('acct-a')).not.toBe(storageKey('acct-b')); + expect(storageKey('acct-a')).toBe('agentmail.background.acct-a'); + // 未登录/首屏（取不到账号）时用匿名键 —— 但**不能退回全局键**，那正是要修的东西 + expect(storageKey('')).toBe('agentmail.background.anonymous'); + expect(storageKey('')).not.toBe('agentmail.background'); + }); + + it('两个账号的缓存互相看不见', () => { + localStorage.setItem(storageKey('acct-a'), JSON.stringify({ ...DEFAULT_BACKGROUND, dim: 30, presetId: 'sand' })); + localStorage.setItem(storageKey('acct-b'), JSON.stringify({ ...DEFAULT_BACKGROUND, dim: 5, presetId: 'ink' })); + expect(readStored('acct-a').presetId).toBe('sand'); + expect(readStored('acct-a').dim).toBe(30); + expect(readStored('acct-b').presetId).toBe('ink'); + expect(readStored('acct-b').dim).toBe(5); + }); + + it('没有本账号缓存时，旧全局键被迁移一次并立刻删除（不留"继承源"）', () => { + // 迁移的意义：老用户升级后观感不凭空消失；删除的意义：下一个账号不会再从它继承 + localStorage.clear(); + localStorage.setItem(LEGACY_STORAGE_KEY, JSON.stringify({ ...DEFAULT_BACKGROUND, dim: 42, presetId: 'dusk' })); + const first = readStored('acct-a'); + expect(first.dim).toBe(42); + expect(first.presetId).toBe('dusk'); + expect(localStorage.getItem(LEGACY_STORAGE_KEY)).toBeNull(); + expect(JSON.parse(localStorage.getItem(storageKey('acct-a'))!).dim).toBe(42); + // 第二个账号拿不到那份旧值，得到的是默认外观（不是继承来的） + expect(readStored('acct-b')).toEqual(DEFAULT_BACKGROUND); + }); + + it('未登录时不迁移（旧值不能送给一个还不知道是谁的账号）', () => { + localStorage.clear(); + localStorage.setItem(LEGACY_STORAGE_KEY, JSON.stringify({ ...DEFAULT_BACKGROUND, dim: 42 })); + expect(readStored('')).toEqual(DEFAULT_BACKGROUND); + expect(localStorage.getItem(LEGACY_STORAGE_KEY)).not.toBeNull(); + }); +}); + +describe('默认值 = 服务端契约', () => { + it('默认压暗/模糊与服务端 DefaultAppearance 的 12/4 一致', () => { + // 权威值在服务端（server/internal/models/models.go 的 DefaultAppearance）； + // 跨端那条判据会直接去 Go 源码里读这两个数比对，这里守住"本文件不再自写一套" + expect(DEFAULT_BACKGROUND.dim).toBe(12); + expect(DEFAULT_BACKGROUND.blur).toBe(4); + }); +}); diff --git a/client/harmony/entry/src/main/ets/model/Wallpaper.ts b/client/harmony/entry/src/main/ets/model/Wallpaper.ts index efe68ad..d4af7f3 100644 --- a/client/harmony/entry/src/main/ets/model/Wallpaper.ts +++ b/client/harmony/entry/src/main/ets/model/Wallpaper.ts @@ -225,7 +225,9 @@ export class BackgroundPlan { * 遮盖浓度（0~1）：**两档都用**（预设档也压），用系统遮罩色刷一层。 * * ⚠️ 这里原来写的是"preset 档不用"，pi 2026-09-14 读 WebUI 源码后指出**两边不一致**： - * WebUI 的 `applyBackground()` **无条件**写 `--bg-dim`（默认 24）， + * WebUI 的 `applyBackground()` **无条件**写 `--bg-dim`（默认 12 —— 与服务端 + * `DefaultAppearance()` 一致；这里原写 24，是因为 WebUI 当时自写了一套 24/8， + * 2026-09-14 已统一到服务端契约）， * `.app-backdrop::after` 是 `rgb(var(--bg-scrim) / var(--bg-dim))` —— * 遮罩**不区分档位**，预设档一样被压。它的注释写着目的： * **「背景越花，正文越需要一层遮罩才读得动」** —— 遮罩服务的是可读性，不是装饰。