Repository navigation
Conversation
|
结论:自动门禁已完成,等待人工批准并合入。 BOE 开发临时规则已启用:门禁与清单完成后,由 Rui-Sun 人工 Merge 视为已批准;其他合入者仍需有效 GitHub Approve。 CR:passed;待处理 0;接受风险 0。 查看结果 图片差异解释:
合入后 Photo 基准更新清单:
BOE 结果仅证明上述实际范围;完整报告和原始差异继续保留。 |
| /** 缓存命中前也核对完整 Rush 配置,不能借旧依赖掩盖本次变化。 */ | ||
| async function assertOfflineConfiguration(root, sha) { | ||
| if ( | ||
| git(root, ['diff', sha, '--', 'common/config/rush', 'rush.json']) || |
There was a problem hiding this comment.
[P2] 离线共享依赖前还需核对项目的 package.json
当两侧项目 package.json 的依赖声明有差异而 Rush 目录及 rush.json 不变时,这里仍判定配置一致,随后 offlineDependencies 直接把候选项目的 node_modules 复制到基线工作树。用相同锁文件的最小 Git 复现中,把基线 packages/vchart/package.json 的 alpha 换为候选侧的 beta 后,函数未报错,基线只得到 beta 而缺少 alpha;这是依赖链接复现,未运行真实图表构建。请在共享链接或命中缓存前校验相关项目的依赖清单与基线一致(或保证两侧各自按清单准备依赖),避免错误依赖参与视觉比较。
已确认修复 (8ccc7de): 原触发条件是基线和候选的项目 package.json 不同、Rush 配置及锁文件相同;现在离线共享链接前和基线缓存命中前均调用配置校验,校验将根目录及各级 package.json 纳入与固定 SHA 的 diff,并检查未跟踪清单。当前版本的确定性测试实际将候选 packages/vchart/package.json 改为不同依赖,验证两处调用均以“依赖配置不同”拒绝,原来错误复用依赖的路径已被阻断;未执行真实图表构建。
原生视觉入口增加
--offline,要求显式固定完整基线提交,并在共享依赖或命中缓存前核对完整 Rush 配置及根目录、各项目的package.json。锁文件相同也不能复用不同的项目依赖声明;两侧工程链接分别指向各自源码,可在无网络环境完成真实基线比较。同时校准混合系列与截断轴标签的交互定位:柱体优先使用未被面积遮挡的内点,轴标签使用实际可命中的目标。保留原始数据、spec、悬停及反向状态、完整文本提示断言。
执行结束时记录真实源码快照;执行或清理期间发生编辑、HEAD 变化或快照读取失败时,报告明确记录执行错误。
已完成验证:
Node.js 22 离线配置、缓存、目标定位和结束快照确定性测试:4/4;Linux 隔离运行同样通过。
原生 CLI
--check通过,显式清单为 264 项。三项受影响用例各连续运行五轮,截图哈希分别一致;抑制动作及移除状态/禁用提示的六次反例均被拒绝。
当前提交的全量原生视觉及 BOE 集成结果见下方自动维护的工作流区块。
本 PR 用于独立工作流接入验证,待四项门禁完成后由维护者合入。
VisActor 工作流(BOE):awaiting_approval
当前提交:
8ccc7de1634dd8d1fa296b837b6eee18a0d6e133门禁、差异解释和 Photo 基准更新清单