test(harmony): 判据按 pi 复核意见补强 —— 手写色登记表、玻璃叠用形状、按标题取小节、遮罩必须被用、废弃 API 清单从 SDK 生成
pi 逐行读了 `cross-client-theme.test.mjs` 与 `Theme.ets` 后指出五处（第一处是真缺口）， 外加一条建议。全部处理，并且**每一处都用变异验证过**。 ## 一（真缺口）：枚举 11 个"必须是系统资源"的名字，挡不住第 12 个新写死的手写色 `static readonly brandSecondary: string = '#123456'` 这种新增**三条判据都碰不到**： A（不在名单里）、B（六位、不是半透明）、裸色值那条（只管 `pages/`）。 原理与当初 14 处 Google 色逃出去是同一条 —— **枚举挡实例，类才挡漂移**， 只是这次枚举的单位是**名字**。补 `A2` 条： - 枚举 `Theme.ets` 里所有 `static readonly X: string = '#……'`，未登记的 → 红； - 名单里已不存在的名字 → 红（名单不能烂成化石）； - `Theme.ets` 里的「手写色登记表」段必须逐个列出这些名字（理由不能只存在于记忆里）； - 自检：把一个**新的**手写色塞进源码字符串，确认这条抓得到。 `Theme.ets` 因此新增登记表（17 项，每项一行理由，按品牌 / 业务语义 / 档位胶囊分组）。 变异：Theme.ets 加 `brandSecondary` → 红。 ## 二：C 的"玻璃只有一处"**说错了自己断言的东西** `assert.equal(glassCalls.length, 1)` 断言的是"全仓共一处"，**不是**"没有嵌套"： 同页两个**并列**玻璃面（没问题）会让它红，真嵌套它没在判。P5 正是悬浮玻璃导航， 到时若把 1 改成 2 就等于不判。改成形状判据： - 收集每个调用点所作用的**组件块**（按括号配对回溯；注意 ArkUI 修饰符是链式的， `X.blur(A).blur(B)` 前面是 `)` 不是 `}` —— 第一版只看一个字符，对这种写法**静默失效**， 是判据自检抓出来的）； - 判两种叠法：同一组件上叠多次（同一块）、以及套在另一层玻璃的子树里； - 每一处玻璃都要在 `GLASS_REGISTRY` 里登记（附一句为什么），名单里的位置若已不存在 → 红。 变异：链式叠两层 → 红；通信页多开一处未登记玻璃 → 红；导航条块内嵌玻璃 → 红。 ## 三：文档小节用 `indexOf('有意差异')` 会**拿错段落** 别的段落正文里出现这四个字，切片就从那里开始，后面所有断言都在**别的段落**上判。 改成**按标题**定位（正则匹配 `^#{2,4}…有意差异…$`），并加了表头列名断言 （WebUI / 鸿蒙 / 为什么）—— 拿错段落时表头不会是这个形状，于是它自己会红。 顺带修掉一个**我自己的**同类毛病：品牌色那行原来用"关键词 + 80 字符窗口"判， 窗口宽度在赌表格单元格字符数（该行两格之和 > 80）。改成**按行取**那一行再断言。 另外把偏移算术的切片换成**按行**切片（偏移差一个字符就会把最后一行拦腰截断， 现象是"品牌色那行只剩 57 字符"这种看着像文案、其实像切片的怪事）。 变异：文件前面插入含「有意差异」的段落 → 仍绿（按标题定位生效）； 再改坏真表里的品牌色行 → 红（确实读的是那一张表）。 ## 四：`Theme.overlay` 只判了"存在"，没判"被用" 没使用点的令牌是自证。补断言：它必须在 `Theme.ets` **之外**有真实使用点 （实测 `SettingsPage.ets` / `MailDetailPage.ets` 两处自绘弹层在用）， 并把 `overlayColor`/`overlayAlpha` 消失的理由写进 §7.12 —— 否则下一个人会当成漏改补回来。 ## 五：材质档次是这次替换里唯一"判据绿但可能观感错"的地方 `COMPONENT_THICK` 的依据（对 THIN / BACKGROUND_* / ULTRA_THICK 的取舍）写进 §7.12， 并把"材质档次在导航条上的实际观感"列为**模拟器起来后第一个要看的项**（间距是数字，材质是判断）。 ## 六（建议）：废弃 API 判据**类化** —— 清单从 SDK 生成 原来是手写"不得再用全局 `promptAction.showToast`"（只挡已踩过的那个）。 现在从 SDK 生成：顶层（花括号深度 0）被标 `@deprecated` 的 `declare function` —— 实测 68 个名字（含 `animateTo` / `getContext` / `px2vp`），配一份**空**的 allow-list。 深度判定是必要的：`declare namespace fileIo { declare function open() }` 里的 `open` 是命名空间成员，算进来会造一堆假红。只算全局调用（排除 `x.name(`）。 变异：调 `px2vp(10)` → 红；`this.px2vp(10)`（成员调用）→ 绿（假阳性自检）。 ## 验证 `hvigorw assembleHap` BUILD SUCCESSFUL；`npm test` 退出码 0 （11 个判据文件全绿 + vitest 258/258）。cross-client-theme 13 → 14 条，harmony-system-api 4 → 5 条。
This commit is contained in:
@ -173,3 +173,108 @@ test('替换成系统方案时要用的那批系统色，先在这里核过（
|
||||
assert.equal(table.get(ghost), undefined, `${ghost} 竟然存在了？那这条边界要重新核一遍`);
|
||||
}
|
||||
});
|
||||
|
||||
/*
|
||||
* ─────────── 废弃 API：**清单从 SDK 生成**，不手写 ───────────
|
||||
*
|
||||
* pi 的建议（2026-09-14）：与其一条条手写"不得再用全局 `promptAction.showToast`"，
|
||||
* 不如从 SDK 的 `@deprecated` 标记**生成**一份清单，再配一份人工 allow-list ——
|
||||
* 新增一条默认判红，除非有人显式放行。这与 `$r('sys.*')` 名字表的做法是同一个套路。
|
||||
*
|
||||
* 手写清单的死法是"只挡已经踩过的那一个"：这次修了 23 处，下一个人写个新的废弃 API
|
||||
* 照样全绿 —— 而 SDK 自己知道哪些废弃了，问它就行。
|
||||
*/
|
||||
|
||||
/** 递归收集 SDK 的 .d.ts（含 api/ 与 component/） */
|
||||
function sdkDtsFiles() {
|
||||
const roots = [
|
||||
join(CLT, 'sdk/default/openharmony/ets/component'),
|
||||
join(CLT, 'sdk/default/openharmony/ets/api')
|
||||
];
|
||||
const out = [];
|
||||
const walk = (dir) => {
|
||||
for (const e of readdirSync(dir, { withFileTypes: true })) {
|
||||
const full = join(dir, e.name);
|
||||
if (e.isDirectory()) walk(full);
|
||||
else if (e.name.endsWith('.d.ts')) out.push(full);
|
||||
}
|
||||
};
|
||||
for (const r of roots) if (existsSync(r)) walk(r);
|
||||
return out;
|
||||
}
|
||||
|
||||
/**
|
||||
* 顶层（不在 namespace/class 里）被标 `@deprecated` 的 `declare function` 名字。
|
||||
*
|
||||
* "顶层"这件事必须判：`declare namespace fileIo { declare function open(...) }` 里的
|
||||
* `open` 是**命名空间成员**，全局调 `open(...)` 并不会走到那个废弃实现 ——
|
||||
* 把它算进来会造出一堆假红（这一版就是按花括号深度筛出来的）。
|
||||
*/
|
||||
function deprecatedGlobalFunctions() {
|
||||
const names = new Set();
|
||||
for (const f of sdkDtsFiles()) {
|
||||
const src = readFileSync(f, 'utf8');
|
||||
// 先切出 depth == 0 的片段
|
||||
let depth = 0;
|
||||
let segStart = 0;
|
||||
const segs = [];
|
||||
for (let i = 0; i < src.length; i++) {
|
||||
const c = src[i];
|
||||
if (c === '{') {
|
||||
if (depth === 0) segs.push([segStart, i]);
|
||||
depth++;
|
||||
} else if (c === '}') {
|
||||
depth--;
|
||||
if (depth === 0) segStart = i + 1;
|
||||
}
|
||||
}
|
||||
if (depth === 0) segs.push([segStart, src.length]);
|
||||
const top = segs.map(([a, b]) => src.slice(a, b)).join('\n');
|
||||
for (const m of top.matchAll(/\/\*\*((?:(?!\*\/)[\s\S])*?)\*\/\s*declare function (\w+)\s*\(/g)) {
|
||||
if (m[1].includes('@deprecated')) names.add(m[2]);
|
||||
}
|
||||
}
|
||||
return names;
|
||||
}
|
||||
|
||||
test('废弃 API：清单**从 SDK 生成**，源码里不得调用（新增一条默认判红）', () => {
|
||||
if (missingSdk.length) return; // 没有 SDK：这一条无从判起（由文件顶部统一报）
|
||||
const deprecated = deprecatedGlobalFunctions();
|
||||
assert.ok(deprecated.size > 20, `要从 SDK 读到一批废弃的全局函数，实际 ${deprecated.size} 个`);
|
||||
// 清单真的抓到了我们关心的东西（否则"生成"这件事本身没生效）
|
||||
for (const must of ['animateTo', 'getContext', 'px2vp']) {
|
||||
assert.ok(deprecated.has(must), `清单里应该有 ${must}（它已被 SDK 标记废弃）`);
|
||||
}
|
||||
// 全局 showToast 不是 `declare function`（它是命名空间成员），由 harmony-logic 那条单钉
|
||||
|
||||
/**
|
||||
* 允许的例外：**每条都要写理由**。
|
||||
* 空名单就是"一处都不许"—— 想加就得在这里写明为什么非用不可。
|
||||
*/
|
||||
const ALLOW = [];
|
||||
|
||||
const walk = (dir, acc = []) => {
|
||||
for (const e of readdirSync(dir, { withFileTypes: true })) {
|
||||
const full = join(dir, e.name);
|
||||
if (e.isDirectory()) walk(full, acc);
|
||||
else if (/\.(ets|ts)$/.test(e.name)) acc.push(full);
|
||||
}
|
||||
return acc;
|
||||
};
|
||||
const files = walk(ETS_DIR);
|
||||
const hits = [];
|
||||
for (const f of files) {
|
||||
const src = readFileSync(f, 'utf8').replace(/\/\*[\s\S]*?\*\//g, '').replace(/^\s*\/\/.*$/gm, '');
|
||||
for (const name of deprecated) {
|
||||
// 只算**全局调用**：排除成员调用 `x.name(`（`.animateTo(` 是我们要求的新写法）
|
||||
const re = new RegExp(`(?<![.\\w])${name}\\s*\\(`, 'g');
|
||||
if (re.test(src)) hits.push(`${f.slice(ETS_DIR.length + 1)}: ${name}`);
|
||||
}
|
||||
}
|
||||
const allowed = hits.filter(h => ALLOW.some(a => h.includes(a.name)));
|
||||
const bad = hits.filter(h => !ALLOW.some(a => h.includes(a.name)));
|
||||
assert.deepEqual(bad, [],
|
||||
`这些地方调了 SDK 已标记废弃的**全局**函数：${bad.join('、')} —— ` +
|
||||
'换成新写法（如 UIContext 上的同名方法），或把它登记进 ALLOW 并写明理由');
|
||||
assert.ok(allowed.length === ALLOW.length || allowed.length >= 0, 'allow-list 命中统计');
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user