Skip to content

feat(verify): 一条命令验证四个悬浮面板修复都真实生效 - #16

Merged
Ryuu-64 merged 1 commit into
mainfrom
feat/demo-fix-verification
Sep 24, 2026
Merged

Ryuu-64 merged 1 commit into
mainfrom
feat/demo-fix-verification

Conversation

@Ryuu-64

@Ryuu-64 Ryuu-64 commented Sep 24, 2026

Copy link
Copy Markdown
Owner

这个 PR 做什么

新增 scripts/demo-fix.mjs:一条命令验证本仓库的四个悬浮面板修复都真实生效。

node scripts/demo-fix.mjs

它分两步:

  1. 对本仓库的 client.js 跑行为测试(单测 + 在 jsdom 里装载真实 bundle 的 e2e);
  2. 把同一套测试指向上游原版 client.js 再跑一遍。

第二步是关键:只在本仓库跑通不算证据——必须证明同一套测试能抓到上游的缺陷,否则测试可能什么都没测。

本机实测结果

=== 1) 本仓库 client.js 的行为测试 ===
    tests 20 | pass 20 | fail 0   ✔ 全绿

=== 2) 同一套测试跑上游原版 client.js(证明能抓到缺陷)===
    tests 20 | pass 1 | fail 19   ✔ 有失败,说明测试有效

输出末尾会列出四个修复与上游 PR 的对应关系,作为本仓库的修复总览:

修复 内容 上游 PR
[A] 归属模型 tipOwner / clearTip / presentTip #65(同思路,他人提交)
[B] 跨间隙改用实时指针位置判定 #66
[C] 宽限定时器按归属收敛 #65 同思路
[D] 共享容器监听器收敛 #67

为什么单独开这个 PR

这个仓库的 main 本身就是「已修复」状态(相对于上游 1.4.10),所以不存在「把修复合并进 main」的差异。缺的是一个可评审、可追溯的改动来说明这个 fork 到底改了什么、怎么验证——就是这个脚本。

关联

新增 scripts/demo-fix.mjs:对本仓库 client.js 跑行为测试,再把**同一套测试**
指向上游原版 client.js 跑一遍。第二步是关键——只在本仓库跑通不算证据,必须
证明同一套测试能抓到上游的缺陷,否则测试可能什么都没测。

本机实测:

  本仓库:  tests 20 | pass 20 | fail 0    ✔ 全绿
  上游原版: tests 20 | pass  1 | fail 19   ✔ 有失败(测试有效)

输出里同时列出四个修复与上游 PR 的对应关系,作为本仓库的修复总览。

对应 issue:#2 #3 #4 #5
上游 PR:#65([A]/[C] 同思路,他人提交)、#66([B])、#67([D])
@Ryuu-64
Ryuu-64 merged commit 2d36f93 into main Sep 24, 2026
@Ryuu-64
Ryuu-64 deleted the feat/demo-fix-verification branch September 24, 2026 06:11
Ryuu-64 pushed a commit that referenced this pull request Sep 24, 2026
finalize-repo.mjs:合并已修复项对应的 PR 并把 issue 置为 completed,
保留尚未触发待办(#10 内核兼容)为 open。本次执行结果:
  合并 PR #16;关闭 #2 #3 #4 #5;#10 保持 open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant