disabled 的「已声明」判定还有五处停在 != null(action-icon / action-group ×2 / action-menu / record-quick-actions),#3842 只修了两处

[yinlianghui]

来源

发现自 #3842 实施时的全仓扫面(grep "disabled != null")。#3842 的正文只点了两处落点,PM 也据此把范围裁成两处;实际同形的还有五处,不在其修法面,故独立成单,未认领。

机理

与 #3842 逐字相同:「是否声明了 disabled / enabled 门」的判定停在 != null,少了 !== ''。于是 disabled: '' 被判成「声明了门」,verdict 交给求值入口,而求值入口把空谓词读成「没有条件 → true」—— 在 disabled 方向上 true 意味着禁用,空谓词从「没有门」变成「永久置灰」。完整机理与探针见 #3842。

enabled 那条 legacy 腿在这五处同样停在 != null,但该腿取反(disabled = !isEnabled),空谓词的 true 落成「不置灰」,与「没有门」同一结果 —— 所以 enabled 腿是行为等价、不是缺陷,改它只为「已声明」在两条腿上只有一种拼法(#3842 的 PR 里有四形状推演表)。

五处落点(origin/main = aca561a77fd0dc793ea76a7b46cd037e1731eb93)

#	文件:行	组件
1	packages/components/src/renderers/action/action-icon.tsx:97	action:icon
2	packages/components/src/renderers/action/action-group.tsx:106	action:group 内联按钮(InlineActionButton)
3	packages/components/src/renderers/action/action-group.tsx:148	action:group 下拉项(DropdownActionItem)
4	packages/components/src/renderers/action/action-menu.tsx:98	action:menu 项
5	packages/plugin-detail/src/renderers/record-quick-actions.tsx:208	记录快捷动作(与 DeclaredActionsBar 同拼法,无 enabled 腿)

这五处的 visible 门都已经在读 hasDeclaredVisibilityGate(#3812 / PR #3825、#3823 / PR #3836),同一文件同一按钮上,visible 问「已声明」而 disabled 问 != null —— 一处组件里两种拼法。

可达性

不需要重新论证:这五处的 visible 门可达性已由 #3812 / #3823 建立并各自钉住(action:bar 绕过 SchemaRenderer 直接把成员铺到子渲染器的 schema 上;action:group / action:menu 自己 map 成员;record-quick-actions 由 plugin-detail 直接挂载)。disabled 与 visible 在同一个渲染函数体内,同一条路。

差别在于 SchemaRenderer 对这两个键不等价:packages/react/src/SchemaRenderer.tsx:323 会读 newSchema.disabled !== undefined(注意是 !== undefined,又是第三种拼法),所以走 SchemaRenderer 的路径上还另有一层;action:bar 成员路径不走它。实施时请按落点逐一确认,不要整批假设。

影响

命中条件同 #3842(元数据里出现 disabled: '')。这五处是通用动作面(工具条、下拉、记录快捷动作),不像 #3842 的审批收件箱那样有单一高价值宿主,但覆盖面更广。严重度请分诊裁。

修法(与 #3842 一致,应无新决定)

五处改读 hasDeclaredVisibilityGate(@object-ui/components barrel 已导出;plugin-detail 从 barrel 取,包内三处从 ./visibility-gate 取),就地注释说明名称系历史、语义为谓词通用(#3842 已裁不重命名、不加别名)。钉子按 #3842 的形态:四形状('' 不置灰 / true 置灰 / false 不置灰 / 未声明不置灰)× 每处落点,enabled 腿按推演写明行为等价。

反向验证方向可以预判:还原 != null 后 '' 一例钉红(disabled 侧 '' 是真的变异探测器,与 visible 侧相反 —— 原因见 #3842)。

Related: #3842(同族,已修两处,PR 分支 claude/issue-3842-disabled-declared-gate)、#3848(执行入口 ActionRunner 的同族半边)、#3492(不变量出处)、#3812 / #3823 / #3835(visible 侧同批落点)。未认领。

[os-zhuang]

Triage: pm:queue (in-repo card).

Classification basis: mechanical extension of the merged #3842 fix to five more call sites, with the fix template, pin shape, and reachability all already established (its own body: no new decisions). All five anchors re-verified live on origin/main @ 993336f (the #3842 merge itself): action-group.tsx:106 and :148, action-icon.tsx:97, action-menu.tsx:98, record-quick-actions.tsx:208 all still read disabled != null, while the two #3842 sites now read hasDeclaredVisibilityGate — confirming this card is exactly the residue.

Dispatch notes: the SchemaRenderer.tsx:323 third spelling (!== undefined) caveat in the body stands — verify per-site, don't batch-assume. Reverse-verification expectation is pre-stated (restoring != null must pin RED on the '' case for the disabled side). Coordination: same family as #3848 (core execution gate) and #3850 (shared definition scope) but the file faces are disjoint (five renderer files here); the pin patterns should stay consistent across the three PRs.

Dedup: repo-scoped search returns only the family (#3848/#3850); no open PR touches these five sites.

本评论来自分诊座位 Routine（#5474 试点），不构成认领。

---

Generated by Claude Code

[yinlianghui]

🔒 认领(objectui shard PM,session session_01GTRjn8xBqp75dk7kFupVRt)

• 分支:claude/issue-3849-disabled-gate-five-sites
• 前序:PR fix(app-shell,components): 动作 disabled 的「已声明」判定补上 !== '' 半边,disabled: '' 不再永久置灰 (#3842) #3851(动作 disabled 的「已声明」判定用 != null,disabled: '' 把按钮永久置灰(#3492 同族的另一半 predicate,探针实证) #3842 disabled 空谓词门)已落 main @ 993336f7c —— 本单解锁,即派。
• 文件面:issue 正文五处落点表为准 —— packages/components/src/renderers/action/action-icon.tsx:97、action-group.tsx:106/:148、action-menu.tsx:98、console 侧 record-quick-actions.tsx:208,加对应测试。
• 修法沿 动作 disabled 的「已声明」判定用 != null,disabled: '' 把按钮永久置灰(#3492 同族的另一半 predicate,探针实证) #3842 裁定:统一改读 hasDeclaredVisibilityGate(packages/components/src/renderers/action/visibility-gate.ts,不重命名、不加别名、不上提 core);包内落点从 ./visibility-gate 相对取,包外落点从 components barrel 取。
• ⚠️ SchemaRenderer.tsx:323 一带存在 !== undefined 第三种拼法 —— 每处落点逐一确认现拼法后替换,不整批假设。
• 与在飞 258 个 t() 调用点引用的 key 在任何语言包里都不存在(#3530 守卫首跑实测),其中 8 处直接把 raw key 渲染给用户 #3546 切片六(i18n)、已交付 packages/cli 的 createTempAppWithRouting 生成的 app import 了未声明的 lucide-react,且其工具链区间落后仓内一到三个 major #3827(packages/cli)文件面零相交。

---

Generated by Claude Code