console 产线构建:模态表单里 lookup 搜索框无法聚焦/输入——focus-scope 栈竞态把 dialog scope 踢出栈

[baozhoutao]

现象

生产构建的 console(如 @objectstack/console@17.0.0-rc.1)中,新建/编辑记录的模态表单里:

1. lookup 字段的快捷下拉:点搜索框无法输入——输入框拿不到焦点,焦点被同步拽回「选择...」触发按钮,列表始终不过滤;
2. 点「浏览全部」弹出的记录选择表格(Record Picker)同样无法搜索。

「新建用户权限集」表单的 用户(sys_user)lookup 上真机复现;任意对象的 lookup 均受影响。仅产线 bundle 复现,vite dev(HMR)正常,因此长期未被本仓测试发现。

排查(EHR rc.1 后端 + 逐层插桩坐实)

• 服务端无辜:GET /api/v1/data/sys_user?search=zzz 正确返回 0 条(rc.1 与 main 均正常);
• 前端链路(useRecordQuery → $search → rawFind ?search=)代码正确,但请求根本没发出——因为搜索框根本打不进字;
• 插桩显示:input.focus() 时,焦点在 focusout 阶段就被外层 Dialog 的 Radix FocusScope.handleFocusOut 同步拽回触发按钮;
• 弹层(Popover)的 FocusScope 挂载副作用确认执行了 focusScopesStack.add,但外层 dialog scope 的 pause() 从未被调用——它已经不在栈里了;
• 根因是 @radix-ui/react-focus-scope@1.1.16(上游最新 rc 同样存在)的竞态:栈管理 effect 的 cleanup 用 setTimeout(0) 延迟执行 focusScopesStack.remove(scope)。当该 effect 因依赖抖动(container ref 重挂)在组件未卸载的情况下重跑时,顺序变成「重跑先 add → 旧 cleanup 的延时 remove 后执行」,把刚加回去的 scope 永久踢出栈。此后 dialog 的陷阱监听器仍然活着,但任何弹层都无法暂停它 → 弹层内输入框永远无法聚焦。

方案

pnpm patch @radix-ui/react-focus-scope(patches/@radix-ui__react-focus-scope.patch):

• effect 重跑(scope 仍存活)时,取消上一次 cleanup 挂起的延时驱逐(clearTimeout);
• 真正卸载时行为不变(autofocus-on-unmount + 栈移除照旧)。

回归测试 packages/components/src/__tests__/dialog-popover-focus-scope.test.tsx:用「asChild 子元素换 key → container ref 重挂 → effect 必然重跑」确定性复现竞态,未打补丁必红、打补丁转绿;另附 Dialog+Popover 集成用例。

已在 os-tianshun-ehr rc.1 后端 + 修复后产线 console 实测:下拉搜索 zzz→「未找到选项」、Dev→正确过滤;Record Picker 表格弹窗搜索同样恢复。

上游(radix-ui/primitives)最新版本仍未修复该竞态,补丁需保留至上游修复为止。

[xuyushun441-sys]

分诊(objectui 分片 PM,session_01NVPjPzmmAJ2Ngtvgg5MSRa / xuyushun441-sys):这单看起来已经修完了,只是没被自动关闭。

不动你的认领,只把证据摆出来 —— 本单 assignee 是 @baozhoutao,按跨账号规则归你,关闭与否由你定。

证据:修复已在 origin/main(@ ca58e45)

f6e8d78 — fix(deps): #3184 可合并版 —— focus-scope 栈驱逐竞态补丁,解冲突 + 补丁存废说明 (#3200),其提交体内含 Closes #3183,且描述与本单根因逐字一致(setTimeout(0) 延时 focusScopesStack.remove 在 effect 重跑时把刚 add 回去的 scope 永久踢出栈)。

落地物均在 main 上实测存在:

本单方案	main 现状
patches/@radix-ui__react-focus-scope.patch	✅ 存在
回归测试 packages/components/src/__tests__/dialog-popover-focus-scope.test.tsx	✅ 存在
补丁存废说明	✅ patches/README.md

为什么没自动关

Closes #3183 写在 squash 合并后的提交体内层(原 PR #3184 的提交信息被 #3200 带进来),不在 PR #3200 自己的描述里,所以 GitHub 的自动关闭没有触发。这是个纯记账缺口,不是修复缺口。

建议

确认后关闭本单。留着的代价是实打实的:它现在长得像一个未修的产线 P0,下一轮分诊(无论人还是 agent)都要重新走一遍我刚走的这条排查路,而队列里真正没修的单会被它稀释。

如果你认为还有残留没覆盖(比如补丁只解了 Dialog+Popover,Record Picker 表格弹窗那条路径想再回归一次),那更应该把剩余部分重写成一单,而不是让整单挂着。

---

Generated by Claude Code

[os-zhuang]

分诊:关闭(completed) —— 修复已在 origin/main,本单是纯记账缺口。⛔ 未打任何 pm:*(已落地的单不该进状态机),⛔ 未动 assignee、未动认领。

独立复核 08-03 那条建议(对 objectui origin/main 84bd39e 实读,不沿用旧结论):

本单方案的三件落地物	main 现状
patches/@radix-ui__react-focus-scope.patch	✅ 存在,且内容对得上根因——clearTimeout 出现 2 处,即「effect 重跑(scope 仍存活)时取消上一次 cleanup 挂起的延时驱逐」那一改
回归测试 packages/components/src/__tests__/dialog-popover-focus-scope.test.tsx	✅ 存在
补丁存废说明	✅ patches/README.md 存在

读数纪律注记(避免下一位复核者踩同一坑):git merge-base --is-ancestor f6e8d78 origin/main 在本地检出上返回 fatal: Not a valid object name —— 本地是 shallow 检出,解不出那个 commit,而该 fatal 在 &&/|| 链里会被读成「不是祖先」,是 SKILL Operational notes 6 点名的那类假读数。⇒ 判据不取 commit 可达性,取上表三项文件级存在性 + 补丁内容与根因的对应,三项齐备且互相印证。

判「关闭」而非「留着观察」的理由:留着的代价是实打实的——它长得像一个未修的产线 P0(标题写着「产线构建 lookup 搜索框无法聚焦」),每一轮分诊(人或 Routine)都要重走一遍这条排查路,而队列里真正没修的单被它稀释。08-03 已有一次同样的复核并建议关闭,三日无异议;本轮独立复核结论一致,故执行。

⛔ 否决窗口:若你认为仍有残留未覆盖 —— 最可能的一处是正文提到的 Record Picker 表格弹窗那条路径(回归测试覆盖的是 Dialog + Popover 组合),想再单独回归一次 —— 请重开本单或就残留部分另立新单,不要让整单挂着。重开一键即可,本座位不会再次关闭。

上游状态提醒(与关闭无关,但影响补丁存废):正文记录「上游 radix-ui/primitives 最新版本仍未修复该竞态,补丁需保留至上游修复为止」。这条没有任何机制在跟踪 —— 若希望它被跟踪,建议以 patches/README.md 的存废清单为准,或另立一条带重启条件的 pm:on-hold 单;⛔ 分诊座位不代立。

查重(三仓 open issue + PR 各一遍):无其它影子单;#3184 / #3200 是本单的修复 PR 链(Closes #3183 落在 squash 后的提交体内层、不在 PR #3200 描述里,故 GitHub 自动关闭未触发——这就是记账缺口的成因)。

本评论来自分诊座位 Routine(#5474 试点),不构成认领。

---

Generated by Claude Code