Skip to content

fix(components): FilterBuilder 切换 field 时把算子归到新桶内 (#4768) - #4779

Merged
yinlianghui merged 1 commit into
mainfrom
claude/issue-4768-filter-field-switch-operator
Aug 16, 2026
Merged

fix(components): FilterBuilder 切换 field 时把算子归到新桶内 (#4768)#4779
yinlianghui merged 1 commit into
mainfrom
claude/issue-4768-filter-field-switch-operator

Conversation

@yinlianghui

@yinlianghuiyinlianghui commented Aug 16, 2026

Copy link
Copy Markdown
Collaborator

Fixes#4768

问题

FilterBuilder 的算子桶是按字段类型分的，且互不包含：select 桶有 in / notIn，text 桶没有；between 只有日期桶有。切换一行的 field 时组件只写 { field }，算子原样留在了新桶里根本没有的位置——比如 select 改 text 之后行上仍是 in。

Radix 的 SelectValue 是拿实际挂载的 SelectItem 去匹配的，桶里没有该项，trigger 就渲染成空白：用户既看不见也点不到自己正在用的算子，只能删掉整行重来；行本身还在按那个算子过滤。

改动

把「改字段」和「定算子、定 value 形状」合成一次编辑，与 #3958 / PR #4762 在算子侧确立的机制同形（那边是「改算子」和「重整 value 形状」合成一次编辑）：

  • 新桶仍然提供的算子予以保留，value 一并不动——contains 从一个文本列换到另一个文本列不该悄悄变成 equals；
  • 新桶无法提供的算子换成该桶首项，再由 reshapeFilterValue 按新算子的家族重整 value：in 下的列表塌成首项标量，between 的区间保留下界，未填过的 [] 变回 ''。

归属判断走 spec 自己的 normalizeFilterOperator——正是 filterValueArity 已经在用的那次折叠。存量视图读回来可能拼成 not_in，而下拉里列的是别名 notIn，两者是同一个算子，不能因为拼写不同就把作者的算子重置掉。折叠在本组件的算子词表上是单射的（defaultOperators 的 22 个 id 折叠出 22 个互异的规范算子），这才使「按规范拼写比较」是安全的；测试把这条单射性钉住，将来加入的算子一旦破坏它就会红。

renderers / form 未触碰（#4014 在飞）。

反向验证（先预判、后执行、不提交）

预判：只把接线退回 updateCondition(condition.id, { field: value })（helper 不动），前两个 describe 的 9 个 UI 驱动钉转红，「新桶仍提供则不重置」整个 describe 与全部纯函数钉保持绿——后者是防止修复过度（无条件重置会把它们弄红），不是「必须变化」的钉。

实测与预判逐条一致：

 Test Files 1 failed | 1 passed (2)
Tests 9 failed | 37 passed (46)

红的正是预判的 9 条（4 条「切换后算子落回新桶」+ 4 条「value 按新家族重整」+ 1 条 #4762 家族衔接），PR #4762 自己那 24 条钉全绿。随后已还原修复。

验证

Generated by Claude Code

@claude
fix(components): FilterBuilder 切换 field 时把算子归到新桶内 (#4768)
8431fec
算子桶按字段类型划分且互不包含（select 有 in/notIn，text 没有；between
只有日期桶有）。切换一行的 field 只写 { field }，算子就留在了新桶没有的
位置——Radix 的 SelectValue 拿实际挂载的 SelectItem 匹配，于是 trigger
渲染成空白，用户看不见也点不到自己正在用的算子。
现在把「改字段」和「定算子、定 value 形状」合成一次编辑，与 #3958 /
PR #4762 在算子侧的机制同形：新桶仍提供的算子予以保留（value 一并不动），
新桶不提供的换成该桶首项并按新家族重整 value。归属判断走 spec 自己的
normalizeFilterOperator，所以存量视图读回的 not_in 会被认成下拉里列的
notIn，不会被无谓重置；该折叠在本组件的 22 个算子 id 上是单射的，测试
把这条性质一并钉住。
Co-authored-by: Claude <noreply@anthropic.com>
@vercel

vercelBot commented Aug 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
ProjectDeploymentActionsUpdated (UTC)
objectuiIgnoredIgnoredAug 16, 2026 5:53am

Request Review

@yinlianghuiyinlianghui mentioned this pull request Aug 16, 2026
Closed
@github-actions

github-actionsBot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

✅ Console Performance Budget

MetricValueBudget
Main entry (gzip)24.7 KB350 KB
Entry fileindex-wRZFiFXW.js—
StatusPASS—

📦 Bundle Size Report

PackageSizeGzipped
app-shell (index.js)9.56KB3.59KB
app-shell (runtime-config.js)7.42KB2.32KB
app-shell (types.js)0.01KB0.04KB
app-shell (urlParams.js)8.92KB3.41KB
auth (AuthContext.js)0.31KB0.24KB
auth (AuthGuard.js)1.17KB0.53KB
auth (AuthProvider.js)25.13KB5.40KB
auth (AuthShell.js)3.49KB1.40KB
auth (ForgotPasswordForm.js)12.21KB3.45KB
auth (LoginForm.js)18.13KB5.39KB
auth (PreviewBanner.js)0.90KB0.50KB
auth (RegisterForm.js)6.64KB2.21KB
auth (SocialSignInButtons.js)9.60KB3.89KB
auth (UserMenu.js)3.40KB1.22KB
auth (auth-gate-events.js)1.29KB0.66KB
auth (authStyles.js)5.04KB1.72KB
auth (createAuthClient.js)38.46KB10.17KB
auth (createAuthenticatedFetch.js)6.34KB2.43KB
auth (index.js)2.35KB1.07KB
auth (org-roles.js)6.66KB2.78KB
auth (phone-identifier.js)1.11KB0.66KB
auth (types.js)0.59KB0.35KB
auth (useAuth.js)5.02KB0.88KB
auth (useIsWorkspaceAdmin.js)1.61KB0.85KB
collaboration (CommentThread.js)26.07KB7.56KB
collaboration (LiveCursors.js)3.17KB1.27KB
collaboration (PresenceAvatars.js)6.49KB2.64KB
collaboration (PresenceProvider.js)2.79KB1.13KB
collaboration (index.js)1.65KB0.73KB
collaboration (useCollaborationTranslation.js)6.05KB2.52KB
collaboration (useCommentSearch.js)1.98KB0.88KB
collaboration (useConflictResolution.js)7.75KB1.86KB
collaboration (useMentionNotifications.js)1.81KB0.68KB
collaboration (usePresence.js)6.33KB1.84KB
collaboration (useRealtimeSubscription.js)7.91KB2.01KB
components (index.js)495.08KB110.01KB
core (index.js)3.79KB1.52KB
create-plugin (index.js)10.08KB3.26KB
data-objectstack (index.js)157.05KB43.28KB
fields (index.js)231.21KB57.42KB
i18n (LocalizationContext.js)1.76KB0.96KB
i18n (currency.js)1.22KB0.64KB
i18n (i18n.js)4.32KB1.77KB
i18n (index.js)3.35KB1.38KB
i18n (pickLocalized.js)3.69KB1.73KB
i18n (provider.js)23.12KB7.62KB
i18n (useDisplayLocale.js)2.84KB1.45KB
i18n (useObjectLabel.js)27.59KB6.63KB
i18n (useSafeTranslation.js)7.77KB3.13KB
layout (index.js)38.96KB10.85KB
mobile (MobileProvider.js)0.92KB0.49KB
mobile (ResponsiveContainer.js)0.94KB0.38KB
mobile (breakpoints.js)1.51KB0.70KB
mobile (createOfflineDataSource.js)5.61KB1.74KB
mobile (index.js)1.50KB0.62KB
mobile (offlineQueue.js)3.91KB1.35KB
mobile (pwa.js)0.97KB0.49KB
mobile (serviceWorker.js)1.48KB0.62KB
mobile (serviceWorkerSource.js)3.41KB1.48KB
mobile (useBreakpoint.js)1.54KB0.65KB
mobile (useGesture.js)6.96KB1.98KB
mobile (useOfflineSync.js)1.99KB0.72KB
mobile (usePullToRefresh.js)2.53KB0.85KB
mobile (useResponsive.js)0.71KB0.42KB
mobile (useResponsiveConfig.js)1.36KB0.63KB
mobile (useSpecGesture.js)4.32KB1.64KB
mobile (useTouchTarget.js)1.01KB0.54KB
permissions (MePermissionsProvider.js)9.35KB3.31KB
permissions (PermissionContext.js)0.31KB0.25KB
permissions (PermissionGuard.js)0.89KB0.45KB
permissions (PermissionProvider.js)3.89KB1.22KB
permissions (evaluator.js)4.41KB1.44KB
permissions (index.js)0.91KB0.41KB
permissions (store.js)0.91KB0.42KB
permissions (useFieldPermissions.js)1.28KB0.52KB
permissions (usePermissions.js)1.81KB0.83KB
plugin-ai (index.js)15.75KB3.80KB
plugin-calendar (index.js)46.86KB12.91KB
plugin-charts (index.js)64.44KB18.28KB
plugin-chatbot (index.js)181.21KB43.14KB
plugin-dashboard (index.js)128.14KB32.58KB
plugin-designer (index.js)212.39KB42.83KB
plugin-detail (index.js)239.81KB59.97KB
plugin-editor (index.js)2.46KB1.10KB
plugin-form (index.js)114.12KB27.65KB
plugin-gantt (index.js)164.30KB40.02KB
plugin-grid (index.js)192.27KB51.31KB
plugin-kanban (index.js)52.72KB14.54KB
plugin-list (index.js)111.23KB26.97KB
plugin-map (index.js)18.16KB5.81KB
plugin-markdown (index.js)13.72KB4.69KB
plugin-report (index.js)41.38KB11.09KB
plugin-timeline (index.js)26.68KB7.66KB
plugin-tree (index.js)8.50KB2.88KB
plugin-view (index.js)83.81KB20.49KB
providers (DataSourceProvider.js)0.75KB0.39KB
providers (MetadataProvider.js)1.37KB0.59KB
providers (ThemeProvider.js)1.90KB0.85KB
providers (UploadProvider.js)11.71KB3.53KB
providers (index.js)0.44KB0.22KB
providers (types.js)0.01KB0.04KB
react-runtime (index.js)5.67KB2.37KB
react (LazyPluginLoader.js)3.77KB1.33KB
react (SchemaRenderer.js)27.64KB9.44KB
react (data-invalidation.js)5.05KB2.08KB
react (index.js)1.28KB0.68KB
react (schema-input.js)1.45KB0.83KB
react (spec-input.js)0.20KB0.18KB
sdui-parser (codegen.js)4.09KB1.74KB
sdui-parser (index.js)4.47KB2.03KB
sdui-parser (parse.js)10.04KB2.82KB
sdui-parser (types.js)0.29KB0.24KB
sdui-parser (validate.js)4.69KB1.48KB
types (ai.js)0.20KB0.17KB
types (api-types.js)0.20KB0.18KB
types (app.js)2.87KB0.99KB
types (base.js)0.20KB0.18KB
types (blocks.js)0.20KB0.18KB
types (complex.js)0.20KB0.18KB
types (crud.js)0.20KB0.18KB
types (dashboard-filter-alias.js)6.23KB2.74KB
types (data-display.js)0.20KB0.18KB
types (data-protocol.js)0.20KB0.19KB
types (data.js)0.20KB0.18KB
types (designer.js)1.87KB0.85KB
types (disclosure.js)0.20KB0.18KB
types (error-code.js)1.54KB0.88KB
types (feedback.js)0.20KB0.18KB
types (field-types.js)0.20KB0.18KB
types (form.js)0.20KB0.18KB
types (http-retry.js)4.32KB2.02KB
types (index.js)3.05KB1.52KB
types (layout.js)0.20KB0.18KB
types (managed-by.js)0.19KB0.18KB
types (mobile.js)2.59KB1.31KB
types (navigation.js)0.20KB0.18KB
types (objectql.js)0.20KB0.18KB
types (overlay.js)0.20KB0.18KB
types (permissions.js)0.20KB0.18KB
types (plugin-scope.js)0.20KB0.18KB
types (record-components.js)0.20KB0.19KB
types (record-semantics.js)1.28KB0.67KB
types (registry.js)0.20KB0.18KB
types (reports.js)0.20KB0.18KB
types (spec-report.js)5.05KB1.93KB
types (system-fields.js)3.33KB1.54KB
types (theme.js)0.20KB0.18KB
types (ui-action.js)3.40KB1.71KB
types (views.js)0.20KB0.18KB
types (widget.js)0.20KB0.18KB

Size Limits

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

@yinlianghuiClaude

yinlianghui commented Aug 16, 2026

Copy link
Copy Markdown
CollaboratorAuthor

PM 验收(座位 repo:objectui,session_01GTRjn8xBqp75dk7kFupVRt):通过,转 ready 并挂 auto-merge。#4768 落地 —— field 切换后 operator 不在新桶内则重置为首项、value 随新家族重整,「下拉显示空、行带着看不见的 operator」关闭。

核验记录(实物核验 + CI 亲读):

  1. 机制与 fix(components): FilterBuilder 为 in/not_in/between 渲染匹配形状的值输入,家族判定改读 spec 词表 (#3958) #4762 同家族(reconcileOperatorForField 纯 helper),钉③「新桶仍提供则不重置」是防过度修复的守卫 —— 反向验证预判明确区分「必须红的 9 条」与「守卫必须绿」的两类,实测逐条吻合,PR fix(components): FilterBuilder 为 in/not_in/between 渲染匹配形状的值输入,家族判定改读 spec 词表 (#3958) #4762 自身 24 钉全绿不回归。
  2. 契约钉的诚实标注:helper 契约钉(规范拼写折叠、单射性、跨字段类型全量不变式)在 main 上是不编译而非失败 —— 如实写明不套 before-red 模板。
  3. 消费半径外扩到 app-shell fold 整链(59 例绿)。
  4. 门与规程:components 1196 绿;type-check 81/81;十道 gate 绿;fable 0;releases/renderers 零触碰;22 新钉。
  5. CI 亲读终态:20/20 completed、0 失败(shard×4 至 06:03:08Z)。

out-of-scope #4781(field 切换后 value 残留的三选项语义)已入决策箱 —— dev 对「这是产品语义不是契约机械」的判断被 PM 采纳,倾向 B 的论证在卡上待维护者拍板。

Generated by Claude Code

@yinlianghui
yinlianghui marked this pull request as ready for review August 16, 2026 06:10
@yinlianghui
yinlianghui added this pull request to the merge queueAug 16, 2026
Merged via the queue into main with commit c4533dcAug 16, 2026
21 checks passed
@yinlianghui
yinlianghui deleted the claude/issue-4768-filter-field-switch-operator branch August 16, 2026 06:10
@yinlianghuiyinlianghui mentioned this pull request Aug 16, 2026
Merged
akarma-synetal pushed a commit to akarma-synetal/objectui that referenced this pull request Aug 17, 2026
@yinlianghui@claude
fix(components): FilterBuilder 切换 field 时按新列类型重整 value —— 可转则转，不可转清空 (o…
be60815
…bjectstack-ai#4781) (objectstack-ai#4876)
* fix(components): FilterBuilder 切换 field 时按新列类型重整 value —— 可转则转，不可转清空 (objectstack-ai#4781)
切换一行的 field 时，PR objectstack-ai#4779 只在算子家族变化时重整 value 的形状；标量到
标量没有形状问题，值被原样带过去。但字段的**类型**也换了，而值输入框正是
按新类型重绘的：`<input type="number">` 对非数字 value 一律显示为空。于是
text 列的 `equals "acme"` 指到 number 列后，输入框空白，行却仍带着
`"acme"` —— `foldFilterGroupToSpecRules` 原样持久化，实时网格拿
`amount equals "acme"` 去查。与 objectstack-ai#4768 同一形状的「看不见的值」，只是发生在
value 而不是 operator 上。
按维护者 2026-08-16 裁定（方案 B：能转则转、不能转才清）：
- convertibility 一处定义（`convertScalarToFamily`），与 `reshapeFilterValue`
的 carry-if-possible 语义并列；判定为**干净、无歧义**的读数，比
`parseFloat` 严格 —— 后者会把 `"acme"` 读成 `0`，写出用户从未写过的筛选；
- number：`"42"` → `42`，`"42abc"` / `"1,000"` / `Infinity` 清空；
- boolean：只认 `"true"` / `"false"`（两端可往返），`1` / `"yes"` 是约定不是
读数，清空；
- date/datetime/time：只收目标输入框本身能渲染的值，外加唯一一处不丢信息的
截断（无时区时间戳 → 其日期）。裸日期不会被补上午夜去凑 datetime 列：
`equals 2024-03-05T00:00` 是一条看起来已填、实则几乎匹配不到东西的规则；
- 新列装得下的值原样保留（两个 text 列之间、两个数值列之间），空行仍是空行。
`getInputType` 改为读同一张 family 表，使「值被转换成的类型」与「值被编辑的
输入框」不可能各说各话 —— 二者不一致正是本单的病灶。
Co-authored-by: Claude <noreply@anthropic.com>
* refactor(components): family→input 表改为模块私有，不进包的公共 API (objectstack-ai#4781)
`FILTER_INPUT_TYPE_BY_FAMILY` 上一版标了「@internal exported for tests」，
但测试并不导入它 —— 输入框类型是从渲染出来的 DOM 上钉的，那本来就是更硬的
钉法。`packages/components/src/custom/index.ts` 是 `export *`，于是它白白进了
包的公共 API（连 `dist/*.d.ts` 都带上了）。没有消费方要求的东西不进契约。
Co-authored-by: Claude <noreply@anthropic.com>
---------
Co-authored-by: Claude <noreply@anthropic.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

package: componentstests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

FilterBuilder 切换行的 field 后 operator 不重置 —— select 改 text 后 operator 可停留在下拉里已不存在的 in,trigger 显示空

2 participants

@yinlianghui@claude