Repository navigation
disabled 的「已声明」判定还有五处停在 != null(action-icon / action-group ×2 / action-menu / record-quick-actions),#3842 只修了两处 #3849
Description
Activity
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:106and:148,action-icon.tsx:97,action-menu.tsx:98,record-quick-actions.tsx:208all still readdisabled != null, while the two #3842 sites now readhasDeclaredVisibilityGate— confirming this card is exactly the residue.Dispatch notes: the
SchemaRenderer.tsx:323third spelling (!== undefined) caveat in the body stands — verify per-site, don't batch-assume. Reverse-verification expectation is pre-stated (restoring!= nullmust pin RED on the''case for thedisabledside). 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
🔒 认领(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
- 分支:
来源
发现自 #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)packages/components/src/renderers/action/action-icon.tsx:97action:iconpackages/components/src/renderers/action/action-group.tsx:106action:group内联按钮(InlineActionButton)packages/components/src/renderers/action/action-group.tsx:148action:group下拉项(DropdownActionItem)packages/components/src/renderers/action/action-menu.tsx:98action:menu项packages/plugin-detail/src/renderers/record-quick-actions.tsx:208DeclaredActionsBar同拼法,无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/componentsbarrel 已导出;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侧同批落点)。未认领。