qiaoqichaoandClaude Code
f1187218ca
这些缺陷源码守卫与构建都查不出，只有真正渲染才暴露（正是 F11 要防的情况）： 1. 雷达图没有尺寸约束：<svg> 只有 viewBox，撑满整个容器（实测约 1000px 高）， 且轴上没有维度标签。现按仓库既有约定给显式 width/height 并补轴标签。 2. 首屏不显示演示值：真实运行视图按 AC-30 不携带演示估计，导致首屏显示 "暂不显示"，与 13 §7「首屏并列概率（演示值）+ 主审意见」不一致。 现载入演示域运行后取一次基线情景（空选择 → 服务端演示表条目 00）， production 域不请求演示表。 3. 未勾选任何假设时也画出实线情景轮廓（等于把基线复制成"所选情景"）； 现只在存在被假设的维度时画实线。另外维度阶段补上序号，便于对照图例。 守卫脚本相应补 4 条断言（显式尺寸、轴标签、基线虚线、无勾选不画实线）， 并修正被本次改动影响的旧断言。 Co-Authored-By: Claude Code <noreply@anthropic.com>
90 lines
5.7 KiB
JavaScript
90 lines
5.7 KiB
JavaScript
/**
|
||
* T09 预审工作台与沙盘界面源码守卫。
|
||
*
|
||
* 这是**源码断言**，不是浏览器端到端测试（docs/v2/06-validation.md §2 已明确
|
||
* 现有 UI 脚本的性质）。它锁住的是可机械核对的约束：数值只能来自服务端、
|
||
* 演示值只在演示域、雷达只画 3–8 维、乱序响应丢弃、失败保留基线、
|
||
* 图的等价文字与键盘可达。
|
||
*/
|
||
const assert = require("node:assert/strict");
|
||
const fs = require("node:fs");
|
||
const path = require("node:path");
|
||
|
||
const root = path.resolve(__dirname, "..");
|
||
const read = (file) => fs.readFileSync(path.join(root, file), "utf8");
|
||
|
||
const panel = read("src/components/xingbang/PrescreenPanel.tsx");
|
||
const lib = read("src/lib/prescreen-run.ts");
|
||
const workbench = read("src/app/xingbang-workbench/page.tsx");
|
||
|
||
// 1. 挂载在既有工作台内，不新建路由（04-ui-spec §1：不恢复已合并的旧页面）
|
||
assert.match(workbench, /import PrescreenPanel from "@\/components\/xingbang\/PrescreenPanel"/, "工作台应导入预审面板");
|
||
assert.match(workbench, /<PrescreenPanel /, "工作台应渲染预审面板");
|
||
for (const banned of ["prescreen", "sandbox", "simulation"]) {
|
||
assert.ok(!fs.existsSync(path.join(root, "src/app", banned)), `不得为 T09 新建路由 src/app/${banned}`);
|
||
}
|
||
|
||
// 2. 复用既有样式，不新增样式表
|
||
assert.ok(!/^import .*\.css";/m.test(panel), "面板应复用 workspace.css 既有 class，不新增样式表");
|
||
|
||
// 3. 数值只能来自服务端：前端不得复制演示情景表（AC-30/31）
|
||
for (const [name, source] of [["panel", panel], ["lib", lib]]) {
|
||
assert.ok(!/0\.(42|58|51|64)\b/.test(source), `${name} 不得硬编码演示概率值`);
|
||
assert.ok(!/"(00|10|01|11)"\s*:/.test(source), `${name} 不得在客户端复制演示情景表`);
|
||
}
|
||
assert.match(panel, /simulation\.probability_view/, "情景数值必须取自服务端响应");
|
||
|
||
// 4. 演示值只在演示域显示，且紧邻"人工设定"说明（AC-12/30、DESIGN §38）
|
||
assert.match(panel, /const domainName = run\?\.data_domain/, "演示值必须按数据域判定后才显示");
|
||
assert.match(panel, /demoValue\(domainName, simulation\?\.baseline_probability_view \|\| run\?\.probability_view\)/,
|
||
"基线演示值必须取自服务端基线视图，不能由前端合成");
|
||
assert.match(panel, /if \(first && first\.data_domain === "demo"\) await applyScenario\(\[\], first\)/,
|
||
"首屏演示值只对演示域取值，production 域不得请求演示表");
|
||
assert.match(panel, /未完成真实历史校准/, "未校准时必须说明为何不显示概率");
|
||
assert.match(panel, /正式审批由人工决定/, "演示值旁必须有用途边界说明");
|
||
assert.match(lib, /domain !== "demo"/, "lib 必须在非演示域返回 null");
|
||
|
||
// 5. 雷达只对同刻度 3–8 维绘制，其余给完整列表；空维度不补占位轴（AC-44）
|
||
assert.match(lib, /RADAR_MIN = 3/);
|
||
assert.match(lib, /RADAR_MAX = 8/);
|
||
assert.match(lib, /layout: "radar"/);
|
||
assert.match(lib, /DimensionLayout = "empty" \| "list" \| "grouped" \| "radar" \| "faceted"/, "呈现词汇必须与后端 review_plan.presentation 一致");
|
||
assert.match(lib, /EMPTY_DIMENSIONS = "当前方案尚未配置核查维度"/);
|
||
assert.match(panel, /shown\.layout === "radar" && <RadarChart/, "只有判定为雷达时才画雷达");
|
||
assert.match(panel, /shown\.layout === "empty" && <div className="ra-empty">/, "空维度用空状态，不补六轴");
|
||
assert.match(panel, /文字列表与图等价/, "图必须有等价文字列表");
|
||
assert.match(panel, /width=\{\d+\} height=\{\d+\} viewBox=/, "雷达必须给显式尺寸，否则会撑满容器（真渲染验证过的缺陷）");
|
||
assert.match(panel, /<text key=\{`label-\$\{item\.id\}`\}/, "雷达必须有轴标签，图不能只有形状");
|
||
assert.match(panel, /strokeDasharray="4 4"/, "基线必须用虚线，不能只靠颜色区分");
|
||
assert.match(panel, /hasScenario = Array\.from\(scenarioStages\.values\(\)\)\.some\(\(item\) => item\.assumed\)/,
|
||
"没有勾选假设时不得画出实线情景轮廓");
|
||
assert.match(panel, /阶段 \{stage \?\? "—"\} · \{stageName\(stage\)\}/, "维度必须同时给出阶段序号与名称");
|
||
|
||
// 6. 乱序与过期响应必须丢弃（AC-31）
|
||
assert.match(panel, /ticket !== seq\.current/, "旧请求晚到必须丢弃");
|
||
assert.match(panel, /returned !== \[\.\.\.keys\]\.sort\(\)\.join\(","\)/, "结果与当前选择不一致时必须丢弃");
|
||
|
||
// 7. 失败保留基线并标未完成；计算中不得先给乐观增幅
|
||
assert.match(panel, /该情景未完成。基线数值与原有意见保持不变/, "失败必须保留基线");
|
||
assert.match(panel, /不会先给一个乐观增幅/, "计算中不得预告结果");
|
||
assert.match(panel, /不是各项增量的简单相加/, "沙盘必须说明组合非加和");
|
||
|
||
// 8. 键盘可达与无障碍
|
||
assert.match(panel, /<input type="checkbox"/, "假设动作必须是原生可聚焦控件");
|
||
assert.match(panel, /aria-live="polite"/, "变化的数值区必须有 aria-live");
|
||
assert.match(panel, /aria-label=\{`查看 \$\{item\.id\} 的依据`\}/, "维度依据入口必须有可读名称");
|
||
assert.ok(/<svg[^>]*aria-label=/.test(panel.replace(/\n/g, " ")), "雷达必须有文字等价说明");
|
||
|
||
// 9. 来源与数据域用文字＋图标共同区分，不靠颜色单独传意（AC-42）
|
||
assert.match(lib, /text: "历史提炼"/);
|
||
assert.match(lib, /text: "人工创建"/);
|
||
assert.match(lib, /text: "制度导入"/);
|
||
assert.match(lib, /icon: "history"/);
|
||
assert.match(panel, /domain\.text/, "数据域必须以文字呈现");
|
||
|
||
// 10. 阶段是序数，不得换算成概率
|
||
assert.match(lib, /雷达面积不换算成百分比|STAGE_LEGEND/, "必须说明阶段与概率无关");
|
||
assert.ok(!/probability\s*[:=]\s*[0-9]/.test(panel), "面板不得写入任何概率数值");
|
||
|
||
console.log("prescreen-ui: PASS");
|