Skip to content

fix(app-shell,plugin-list): 关联列表「+ New」消费 create 谓词,行内编辑开关补 update 权限门 (#4646, #4647) - #5145

Merged
yinlianghui merged 1 commit into
mainfrom
claude/issue-4646-toolbar-action-gates
Aug 18, 2026
Merged

fix(app-shell,plugin-list): 关联列表「+ New」消费 create 谓词,行内编辑开关补 update 权限门 (#4646, #4647)#5145
yinlianghui merged 1 commit into
mainfrom
claude/issue-4646-toolbar-action-gates

Conversation

@yinlianghui

@yinlianghui yinlianghui commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

Fixes #4646
Fixes #4647

为什么合单

两卡共享同一张工具栏面(记录页关联列表 / 列表视图工具栏),分诊席在两卡各自的 triage 评论里都留了 serial note:「shares the RecordDetailView/list-toolbar surface with #464x — dispatch serially or to one dev」。两者又是同一个病:声明半已落、渲染半缺失。一席双卡,一个 PR。

前提复核(dist 级证据 → src 级读点)

两卡的证据都是 dist 级(console@17.0.0)。落地前逐条转成 src 级,其中两条需要修正,如实记录:

卡 原始论断 src 级复核结果
#4646 createPredicates 零消费 ✅ 成立。grep -rn "createPredicates" packages/ apps/ → 0 命中;生产者是 spec 的 resolveCrudAffordances,经 @object-ui/core 的 resolveEffectiveCrudAffordances 转出。对照 editPredicates 在 5 个文件有消费/定义点
#4646 「RecordDetailView 的关联列表工具栏仍是旧两轴」 ⚠️ 归属已迁移。该两轴不在 RecordDetailView.tsx,而在 RelatedRecordActionsBridge.tsx(由 RecordDetailView 挂载):create: rawAff.create && can(objectName, 'create')。实质成立,位置更正
#4646 「组件手里有 parentId」 ⚠️ 成立但不足。resolve() 只拿到 parentId,拿不到父记录本身;record.frozen != true 要的是字段不是主键 —— 所以本 PR 必须把 parentRecord 也穿进桥接层
#4646 spec 声明半 ✅ object.zod.ts:1528 确为 union;实装 @objectstack/spec@17.0.0;docblock 语义逐字核对
#4647 gap 1 渲染条件 / 宿主无条件传参 ✅ 逐字成立。ListView.tsx:2453、ObjectView.tsx:2013
#4647 gap 2「editInline 全 dist 只作 i18n 标签键,零消费」 ⚠️ 部分证伪。InterfaceListPage.tsx:431 就有一处消费:inlineEdit: userActions.editInline === true。dist 扫描看不见它,是因为该消费把键映射成了 inlineEdit,键名没作为字符串留存。实质缺口仍在,但范围收窄为「对象列表工具栏路径零消费」

两处修正都不推翻卡的结论,故照常实施;记录在此以免下一位读者重蹈。

#4646 半 —— 关联列表「+ New」消费 create 谓词

镜像模板 = 同面 editPredicates 的既有消费形(RecordDetailView.tsx:1063-1128,即 #4419 / PR #4515 给记录页页头做的那一版),逐条对齐、不自创语义:

轴 记录页页头既有形 本 PR 关联列表工具栏
求值器 useRowPredicate → evalRowPredicate evalRowPredicate(同一个,见下「为何无 hook」)
visibleWhen 缺省 ?? true,布尔短路 同
visibleWhen 失败方向 fail-CLOSED(fallback: false) 同
visibleWhen 声明判定 != null,非真值性(#3492 不变式) 同
disabledWhen 失败方向 fail-SOFT 同
disabledWhen 声明判定 != null 门在求值之外 同
fields 传参 传对象字段定义,让关系字段按存储外键绑定 传父对象字段(被绑定的记录是父)
隐藏 vs 置灰 visibleWhen 折进 affordance 布尔;disabledWhen 走独立 disabled 标志 同

绑定域按 spec docblock:对宿主父记录求值一次(不是逐行)。独立对象列表没有 scope 记录 —— 它根本不经过这个桥接层。

为何 hook-free:resolve 要为可变数量的关联列表在一个 useMemo 内作答,每个子对象一次 useRowPredicate 会把 hook 数量绑到关联列表条数。这与 plugin-grid 的 evalRowActionVisibility 面对的是同一个约束(它的 row loop 逐 action 面对同一件事),故采同一解法;底层求值器两边都是 canonical 的 evalRowPredicate。

新增的 disabled 通道:RelatedRecordHandlers 原本只有「给 / 不给 onCreate」二值,契约明说「省略 = 隐藏该 affordance」。disabledWhen 需要第三态,故新增 createDisabled → RelatedList 的 newDisabled → 按钮 disabled。把 disabledWhen 折进隐藏是错的:它会让同一个键在这个面上的行为与其他所有面不同。

#4647 半

gap 1 挂点选择 —— 渲染侧,不是宿主侧。 理由:

  1. 同 bundle 既有实践就在渲染侧。同文件 ListView.tsx:1059 的批量删除门 resolveEffectiveCrudAffordances(objectDef, effectiveApiOps).delete && canDo(schema.objectName,'delete') 就长在 ListView 自己身上,组件手里已经有 usePermissions 和 objectDef。
  2. 一次设门覆盖全部宿主。挂宿主侧只堵 ObjectView 一个入口;挂渲染侧连 InterfaceListPage 和任何嵌入宿主一起覆盖。

门形与批量删除逐字同构,只把 operation 从 delete 换成 update;第一段写作 isObjectInlineEditable(...),它就是 resolveEffectiveCrudAffordances(...).edit,但用的是说清这个面在问什么的那个名字 —— 记录页正文的双击/铅笔也读同一个 helper,于是列表页和记录页不会对「这个对象的行能否就地编辑」给出两个答案。

三个渲染点,不是一个。 按消费半径清点后发现该 affordance 有三个入口,只堵卡里点名的那个会留下两扇门:

  1. 宽工具栏开关(ListView.tsx:2453)—— 卡里点名的
  2. 紧凑工具栏设置浮层入口(ListView.tsx:2905,showInlineEdit={currentView === 'grid'})—— 此前完全无门,连 onInlineEditChange 都不要求
  3. 交给 grid 的 editable 模式本身(ListView.tsx:1885)—— 存量视图存着 inlineEdit: true 时,只读账号连开关都不用按就直接落进可编辑单元格,「改单元格 → 全部保存 → 403」这条卡里描述的往返原样可达

只堵 1 会让卡自己描述的后果仍然可达,故三处同用一个 inlineEditOffered 判定。

gap 2 缺省语义 —— 如实写明,并未强制 spec 的 .default(false)。

editInline 显式 false ⇒ 不提供;显式 true ⇒ 提供;缺省 ⇒ 沿用宿主既有的 inlineEdit 通道。这是与 spec 声明缺省的一处有意分歧,理由:强制 .default(false) 会一次性抹掉所有存量 console 列表视图的行内编辑 —— 没有任何 legacy 键折叠进 editInline(SHOW_FLAG_TO_USER_ACTION 无此条目),存量视图也都没声明它。这正是相邻 toolbarFlags 块为自己写下的规则(缺省取「这些开关一直以来的行为」;hideFields/rowColor 保持历史 OFF,因为翻转「会在每个存量视图上多长两个按钮」)在会删掉一个入口方向上的同一应用。

该分歧连同「收敛到 spec 缺省」的两条路径,已另立 finding #5144 交维护者裁决,并在测试里钉了一条同名用例,使日后改动缺省是决策而非事故。

反向验证(先书面预判,后跑;commit 后变异、跑完还原)

预判写在 4646-reverse-predictions.md(scratch,已删),三条全部命中,其中 (c) 的细节值得单独说:

(a) 摘掉 create 谓词消费 → 预判 RED,且恰是 6 条依赖谓词的用例;boolean 臂与对象级门用例须保持绿。
实测 6 failed | 19 passed,红的正是预判的 6 条,boolean 臂与对象级门全绿,兄弟文件 effectiveOps 全绿。

(b) 摘掉 can(update) 门 → 预判 RED 3 条。
实测 3 failed | 14 passed,恰为预判三条(无权限失开关 / 存量视图仍进编辑模式 / 紧凑入口),gap-2 用例与 fail-open 用例保持绿。

(c) 自选非显然方向 —— 隐藏 vs 置灰的分界实测。 变异:把 disabledWhen 也折成隐藏(create: … && !createDisabled)。
预判里明确写了一件事:这条变异只会被一条断言抓住,而且不是那条看起来最相关的 —— soft-fail 用例在两种实现下 createDisabled 都是 false,不具区分力;真正把线守住的是 disabled 用例里的 expect(h.onCreate).toBe(true)。
实测 1 failed | 15 passed,失败信息 AssertionError: expected false to be true,正是那一行。

第三条如实记录而非套模板:这条边界目前由单一断言承载。若日后有人在 disabled 用例里只留 createDisabled 的断言、删掉 onCreate 那行,折叠式回归会以「什么都没产出」的方式变绿(#5046 subtraction case 的同款陷阱)。

验证

新增测试:RelatedRecordActionsBridge.createPredicates.test.tsx(16 例)+ ListView.permissions.test.tsx 追加两个 describe(11 例)。

一处测试写法值得记:紧凑工具栏那两例必须先点开 popover、且让 Section 展开(defaultOpen={!!inlineEdit})才有意义 —— 对着关着的浮层断言「不存在」,对任何实现都会绿。正例对照组一开始就是这么红的,红得对。

半径与刻意未动

  • ⛔ 未动 spec、未动 data-adapter 生产侧、未动 packages/react/SchemaRenderer
  • DetailView.tsx:1538/1659 的 (related as any).onNew 是 schema 作者自带回调通道,不经桥接层、不带对象元数据,正确地未受影响
  • changeset:patch,如实写明只读用户的行为变化(UI 从「能进编辑再 403」变「不提供入口」;数据访问权限未变,服务端始终是执行边界)

半径外发现(均已查重、未指派)

家族关系

Generated by Claude Code

@claude
fix(app-shell,plugin-list): 关联列表「+ New」消费 create 谓词,行内编辑开关补 update 权限门 (
23f9394 
#4646, #4647)

同一工具栏面上的两个「已声明未生效」缺口,合单处理。

#4646 —— `createPredicates` 有生产者、零消费者。
spec@17.0.0 把 `userActions.create` 放宽为
`z.union([z.boolean(), RowCrudActionOverrideSchema])`,`resolveCrudAffordances`
随之产出 `createPredicates`,而 objectui 侧无人读取(对照
`editPredicates`/`deletePredicates` 各约 15 处消费点)。症状:父记录进入冻结态
后,子表行内 Edit/Delete 已正确置灰,关联列表工具栏的「+ New」却依然可点,用户
填完整张子表单才拿到服务端 409。

现在关联列表工具栏按 spec docblock 对该键的绑定语义,**对宿主父记录求值一次**
`visibleWhen` / `disabledWhen`,叠加在既有 `o.create ∧ can(child,'create')` 之上。
`visibleWhen` 隐藏按钮、fail-CLOSED;`disabledWhen` 置灰按钮、fail-SOFT ——
求值器、失败方向、隐藏 vs 置灰的分界,逐一镜像记录页页头 edit/delete 的既有消费形
(#4419 / PR #4515)。union 的 boolean 臂原样不动:无谓词即无求值。

#4647 —— 行内编辑开关是该工具栏上唯一无权限门的入口。
渲染条件曾是「grid 视图 ∧ 宿主传了 `onInlineEditChange` ∧ 非紧凑工具栏」,而宿主
无条件传该回调。同一工具栏上 New / Import 会按权限隐藏、批量删除入口 AND 了
`can(obj,'delete')`,唯独行内编辑对只读账号敞开:可开启、可改单元格、可点「全部
保存」,最后由服务端 403 拦下。

现按「对象已解析的 edit affordance ∧ `can(object,'update')`」设门,与同工具栏
批量删除的门形一致,并覆盖该功能的**全部三个渲染点**:宽工具栏开关、紧凑工具栏
设置浮层入口(此前完全无门,连回调都不要求)、以及交给 grid 的 `editable` 模式
本身 —— 否则存量视图上的 `inlineEdit: true` 仍会把只读账号直接丢进可编辑单元格,
连开关都不用按。

同时消费声明已久却无人读取的 `ListViewSchema.userActions.editInline`:显式
`false` 可整体关闭该功能,这是作者此前做不到的。

缺省语义如实说明:`editInline` 缺省时沿用宿主既有的 `inlineEdit` 通道,不强制
spec 的 `.default(false)`。强制该缺省会一次性抹掉所有存量 console 列表视图的开关
(没有任何 legacy 键折叠进 `editInline`,存量视图也都没声明它)。这正是相邻
`toolbarFlags` 块为自己写下的规则 —— 缺省取「这些开关一直以来的行为」——
在「会删掉一个入口」方向上的同一应用。`InterfaceListPage` 作为该键的另一消费者,
把缺省读作 OFF,因为 ADR-0047 界面页没有可沿用的宿主通道。

行为变化(对只读用户):UI 从「能进编辑再 403」变为「不提供入口」。数据访问权限
未变 —— 服务端始终是执行边界,此处只是不再把用户领进注定失败的往返。有权限的账号
行为不变;未挂 `PermissionProvider` 的宿主(独立嵌入、Studio 设计器)保持现状,
因为 `can()` 在那里按设计答 true。

Co-authored-by: Claude <noreply@anthropic.com>
@github-actions

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

✅ Console Performance Budget

Metric Value Budget
Main entry (gzip) 25.3 KB 350 KB
Entry file index-EEeh4W14.js —
Status PASS —

📦 Bundle Size Report

Package Size Gzipped
app-shell (index.js) 9.56KB 3.59KB
app-shell (runtime-config.js) 7.42KB 2.32KB
app-shell (types.js) 0.01KB 0.04KB
app-shell (urlParams.js) 8.92KB 3.41KB
auth (AuthContext.js) 0.31KB 0.24KB
auth (AuthGuard.js) 1.17KB 0.53KB
auth (AuthProvider.js) 25.13KB 5.40KB
auth (AuthShell.js) 3.49KB 1.40KB
auth (ForgotPasswordForm.js) 12.21KB 3.45KB
auth (LoginForm.js) 18.13KB 5.39KB
auth (PreviewBanner.js) 0.90KB 0.50KB
auth (RegisterForm.js) 6.64KB 2.21KB
auth (SocialSignInButtons.js) 9.60KB 3.89KB
auth (UserMenu.js) 3.40KB 1.22KB
auth (auth-gate-events.js) 1.29KB 0.66KB
auth (authStyles.js) 5.04KB 1.72KB
auth (createAuthClient.js) 40.21KB 10.79KB
auth (createAuthenticatedFetch.js) 6.34KB 2.43KB
auth (index.js) 2.71KB 1.22KB
auth (invitation-status.js) 1.22KB 0.70KB
auth (org-roles.js) 6.66KB 2.78KB
auth (phone-identifier.js) 1.11KB 0.66KB
auth (types.js) 0.59KB 0.35KB
auth (useAuth.js) 5.02KB 0.88KB
auth (useIsWorkspaceAdmin.js) 1.61KB 0.85KB
collaboration (CommentThread.js) 26.07KB 7.56KB
collaboration (LiveCursors.js) 3.17KB 1.27KB
collaboration (PresenceAvatars.js) 6.49KB 2.64KB
collaboration (PresenceProvider.js) 2.79KB 1.13KB
collaboration (index.js) 1.65KB 0.73KB
collaboration (useCollaborationTranslation.js) 6.05KB 2.52KB
collaboration (useCommentSearch.js) 1.98KB 0.88KB
collaboration (useConflictResolution.js) 7.75KB 1.86KB
collaboration (useMentionNotifications.js) 1.81KB 0.68KB
collaboration (usePresence.js) 6.33KB 1.84KB
collaboration (useRealtimeSubscription.js) 7.91KB 2.01KB
components (index.js) 502.20KB 112.21KB
core (index.js) 4.11KB 1.62KB
create-plugin (index.js) 10.08KB 3.26KB
data-objectstack (index.js) 159.03KB 44.08KB
fields (index.js) 234.25KB 58.48KB
i18n (LocalizationContext.js) 1.76KB 0.96KB
i18n (currency.js) 1.22KB 0.64KB
i18n (i18n.js) 4.28KB 1.75KB
i18n (index.js) 3.35KB 1.38KB
i18n (pickLocalized.js) 3.69KB 1.73KB
i18n (provider.js) 23.12KB 7.62KB
i18n (useDisplayLocale.js) 2.84KB 1.45KB
i18n (useObjectLabel.js) 27.59KB 6.63KB
i18n (useSafeTranslation.js) 7.77KB 3.13KB
layout (index.js) 39.16KB 10.97KB
mobile (MobileProvider.js) 0.92KB 0.49KB
mobile (ResponsiveContainer.js) 0.94KB 0.38KB
mobile (breakpoints.js) 1.51KB 0.70KB
mobile (createOfflineDataSource.js) 5.61KB 1.74KB
mobile (index.js) 1.50KB 0.62KB
mobile (offlineQueue.js) 3.91KB 1.35KB
mobile (pwa.js) 0.97KB 0.49KB
mobile (serviceWorker.js) 1.48KB 0.62KB
mobile (serviceWorkerSource.js) 3.41KB 1.48KB
mobile (useBreakpoint.js) 1.54KB 0.65KB
mobile (useGesture.js) 6.96KB 1.98KB
mobile (useOfflineSync.js) 1.99KB 0.72KB
mobile (usePullToRefresh.js) 2.53KB 0.85KB
mobile (useResponsive.js) 0.71KB 0.42KB
mobile (useResponsiveConfig.js) 1.36KB 0.63KB
mobile (useSpecGesture.js) 4.32KB 1.64KB
mobile (useTouchTarget.js) 1.01KB 0.54KB
permissions (MePermissionsProvider.js) 9.35KB 3.31KB
permissions (PermissionContext.js) 0.31KB 0.25KB
permissions (PermissionGuard.js) 0.89KB 0.45KB
permissions (PermissionProvider.js) 4.42KB 1.42KB
permissions (evaluator.js) 5.12KB 1.74KB
permissions (index.js) 0.91KB 0.41KB
permissions (store.js) 0.91KB 0.42KB
permissions (useFieldPermissions.js) 1.28KB 0.52KB
permissions (usePermissions.js) 1.81KB 0.83KB
plugin-ai (index.js) 15.75KB 3.80KB
plugin-calendar (index.js) 46.62KB 12.83KB
plugin-charts (index.js) 64.75KB 18.37KB
plugin-chatbot (index.js) 181.21KB 43.14KB
plugin-dashboard (index.js) 127.85KB 32.73KB
plugin-designer (index.js) 212.39KB 42.83KB
plugin-detail (index.js) 241.46KB 60.56KB
plugin-editor (index.js) 2.46KB 1.10KB
plugin-form (index.js) 123.33KB 29.93KB
plugin-gantt (index.js) 164.10KB 39.87KB
plugin-grid (index.js) 197.97KB 53.15KB
plugin-kanban (index.js) 52.72KB 14.54KB
plugin-list (index.js) 111.39KB 27.03KB
plugin-map (index.js) 18.72KB 6.09KB
plugin-markdown (index.js) 13.72KB 4.69KB
plugin-report (index.js) 41.97KB 11.33KB
plugin-timeline (index.js) 26.68KB 7.66KB
plugin-tree (index.js) 8.50KB 2.88KB
plugin-view (index.js) 83.81KB 20.49KB
providers (DataSourceProvider.js) 0.75KB 0.39KB
providers (MetadataProvider.js) 1.37KB 0.59KB
providers (ThemeProvider.js) 1.90KB 0.85KB
providers (UploadProvider.js) 11.66KB 3.50KB
providers (index.js) 0.44KB 0.22KB
providers (types.js) 0.01KB 0.04KB
react-runtime (index.js) 5.62KB 2.34KB
react (LazyPluginLoader.js) 3.77KB 1.33KB
react (SchemaRenderer.js) 31.55KB 10.70KB
react (data-invalidation.js) 5.05KB 2.08KB
react (index.js) 1.28KB 0.68KB
react (schema-input.js) 1.45KB 0.83KB
react (spec-input.js) 0.20KB 0.18KB
sdui-parser (codegen.js) 5.41KB 2.34KB
sdui-parser (index.js) 4.77KB 2.16KB
sdui-parser (input-type.js) 2.84KB 1.40KB
sdui-parser (parse.js) 10.76KB 3.17KB
sdui-parser (provenance.js) 3.66KB 1.82KB
sdui-parser (types.js) 0.29KB 0.24KB
sdui-parser (validate.js) 6.92KB 2.40KB
types (ai.js) 0.20KB 0.17KB
types (api-types.js) 0.20KB 0.18KB
types (app.js) 2.87KB 0.99KB
types (base.js) 0.20KB 0.18KB
types (blocks.js) 0.20KB 0.18KB
types (complex.js) 0.20KB 0.18KB
types (crud.js) 0.20KB 0.18KB
types (dashboard-filter-alias.js) 6.23KB 2.74KB
types (data-display.js) 0.20KB 0.18KB
types (data-protocol.js) 0.20KB 0.19KB
types (data.js) 0.20KB 0.18KB
types (designer.js) 1.87KB 0.85KB
types (disclosure.js) 0.20KB 0.18KB
types (error-code.js) 1.54KB 0.88KB
types (feedback.js) 0.20KB 0.18KB
types (field-types.js) 0.20KB 0.18KB
types (form.js) 0.20KB 0.18KB
types (http-retry.js) 4.32KB 2.02KB
types (index.js) 3.05KB 1.52KB
types (layout.js) 0.20KB 0.18KB
types (managed-by.js) 0.19KB 0.18KB
types (mobile.js) 2.59KB 1.31KB
types (navigation.js) 0.20KB 0.18KB
types (objectql.js) 0.20KB 0.18KB
types (overlay.js) 0.20KB 0.18KB
types (permissions.js) 0.20KB 0.18KB
types (plugin-scope.js) 0.20KB 0.18KB
types (record-components.js) 0.20KB 0.19KB
types (record-semantics.js) 1.28KB 0.67KB
types (registry.js) 0.20KB 0.18KB
types (reports.js) 0.20KB 0.18KB
types (spec-report.js) 5.05KB 1.93KB
types (system-fields.js) 3.33KB 1.54KB
types (theme.js) 0.20KB 0.18KB
types (ui-action.js) 3.40KB 1.71KB
types (views.js) 0.20KB 0.18KB
types (widget.js) 0.20KB 0.18KB

Size Limits

  • ✅ Core packages should be < 50KB gzipped
  • ✅ Component packages should be < 100KB gzipped
  • ⚠️ Plugin packages should be < 150KB gzipped

@yinlianghui Claude

yinlianghui commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator Author

PM 验收:ACCEPT(session_01GTRjn8xBqp75dk7kFupVRt,objectui 分片,批次 23;Fixes #4646 + Fixes #4647 合单)

实物核验:merge-base 958d7573a;head 23f939493;10 files, +867/−11;标识 grep msg/diff 双零,trailer 唯一正确;releases/控制字节(含 ESC 位)零;merge-tree 对当前 main(含刚合入的 PR #5133)零冲突(本 PR 未动 plugin-detail/src/index.tsx,预警的行级冲突不存在)✅。CI 亲读:20/20 check 全 completed 零失败 ✅。

验收要点:

  1. dist→src 转写的两处论断修正如实且关键:旧两轴在 RelatedRecordActionsBridge 而非 RecordDetailView(且 resolve() 拿不到父记录 ⇒ parentRecord+fields 穿桥是必要接缝);Grid toolbar inline-edit toggle is not gated on can(object, 'update'); userActions.editInline is declared in spec but has no consumer #4647 gap 2 的「零消费」被 InterfaceListPage:431 证伪后收窄而非放弃(缺口=对象列表工具栏路径)—— 前提门的正确用法。
  2. 镜像语义按模板真实行为执行:visibleWhen fail-CLOSED / disabledWhen fail-SOFT(RecordDetailView:1097-1128 实测)—— 派发词写的「fail-open」是 PM 转述不准,席位以「⛔ 不自创语义、逐一镜像」主指令为准修正,正确。
  3. 三处半径扩张全部采信:紧凑工具栏浮层 + editable 模式本身(只堵点名处则卡自述的 403 往返仍可达 —— 后果射程内);core UserActionsOverride.create boolean→union(类型跟上 spec 17.0.0 已发布形,import 键刻意不放宽避免声明-未实施 —— 这个不对称是本单最清醒的一格,并产出 importPredicates is produced by the spec resolver and consumed nowhere — the same inverse-symmetry gap as #4646, one key over #5142);createDisabled/newDisabled 三态通道(原契约二值无法表达 disabled)。
  4. (c) 变异的单一断言承载警示(隐藏 vs 置灰分界只由 expect(h.onCreate).toBe(true) 一行守,docs(plugin-calendar): 对齐 README 的导入面与包的真实导出面 #5046 减法陷阱同款)写进 PR —— 有价值的自我暴露。
  5. editInline 缺省语义的非回归读法采信:直接强制 .default(false) 会一次性抹掉所有存量视图的行内编辑,toolbarFlags 块自己的「缺省取历史行为」规则同向适用;同名用例已钉,使日后改缺省是决策而非事故。A/B 收敛题(推荐 B-折叠)入决策箱 —— PM 将把 [finding] userActions.editInline has two different defaults across surfaces, and neither is the spec's .default(false) #5144 升级 needs-user-decision。
  6. 新卡分诊:importPredicates is produced by the spec resolver and consumed nowhere — the same inverse-symmetry gap as #4646, one key over #5142(importPredicates 同族零消费)与 ObjectGrid's editable schema key offers inline editing with no can(object,'update') gate — the SDUI-authored grid path #4647 did not reach #5143(ObjectGrid editable 无 update 门,SDUI 直写路径)均 pm:queue + bug;[finding] userActions.editInline has two different defaults across surfaces, and neither is the spec's .default(false) #5144 finding + needs-user-decision。
  7. 测试写法备忘(紧凑浮层须先展开才断言有意义,正例对照组先红后修)记入案头。

三件套照常:本评论 → undraft → auto-merge(SQUASH)。

Generated by Claude Code

@yinlianghui
yinlianghui marked this pull request as ready for review August 18, 2026 05:20
@yinlianghui
yinlianghui added this pull request to the merge queue Aug 18, 2026
Merged via the queue into main with commit dfc6975 Aug 18, 2026
21 checks passed
@yinlianghui
yinlianghui deleted the claude/issue-4646-toolbar-action-gates branch August 18, 2026 05:20
This was referenced Aug 18, 2026
Closed
Merged
Open
yinlianghui pushed a commit that referenced this pull request Aug 18, 2026
@claude
test(plugin-grid): 钉住 dataConfig 寻址的 grid 只由本门判定 (#5143)
b4ae01e 
反向验证 (c) 查出一条非显然事实,值得留成钉子而不是只写进 PR 正文:
本门与 #4647(PR #5145)ListView 那道**并非字面同一谓词**。ObjectGrid 解析
`objectName = dataConfig.object ?? schema.objectName`,ListView 的
`inlineEditOffered` 只读 `schema.objectName`。因此当对象身份**仅**经
`data: { provider: 'object', object: … }` 传入、无顶层 `objectName` 时,
ListView 的 `schema.objectName ? … : true` 分支会落开,该形态**只由本门**
判定 —— 本门不是 ListView 那道的冗余副本。

新增钉子覆盖该形态(无授权 ⇒ 不进编辑态;有授权 ⇒ 照常)。变异验证:把
`objectName` 收窄回 `schema.objectName`,预判「仅此一例红」,实测恰好 1 红
11 绿 —— 钉子既咬得住又不误伤。

同时订正上一次提交里一句过强的注释:原文写「两门是同一谓词」,与刚验证出
的差异相抵触。改为如实说明幂等只成立于 ListView 路径,以及差异朝安全方向
跑。「上游已经设过门了」正是当初留下这扇门的推理,不该由注释再复述一遍。

纯注释 + 测试,产物行为不变。

Co-authored-by: Claude <noreply@anthropic.com>
This was referenced Aug 18, 2026
Closed
Closed
Merged
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment