fix(sdui): PageComponentSchema.dataSource 逐 block 接线,record_picker 不再丢 view (objectstack#6953) - #3969
Merged
Merged
Conversation
…丢 view (objectstack#6953)
objectstack#5576 把 spec 的 per-element 数据绑定接到了 `list-view`,同一条声明在
其余每一个 page component 上依旧无人消费。两个缺口都是静默的:
- `element:record_picker` 读了五个键里的四个,单独丢掉 `view`。于是
`dataSource: { object: 'account', view: 'hot' }`(spec 自己的例子)构造出全量
account 选择器,而不是 saved view 选出的行 —— 不抛错、不报面板,只是候选列表
比作者写的更宽,用户因此能选到页面声明为范围外的记录。
- `object-grid` / `object-form` / `object-kanban` / `object-calendar` /
`object-chart` / `object-metric` / `record:related_list` 整条绑定都不读:每块
都把 fetch 挂在自己的 `objectName` 上,没有任何地方把 `dataSource.object` 映射
过去。按 spec 写、不额外写 `objectName` 的页面渲染出空 —— 不发请求、无诊断。
spec-valid 的 metadata 渲染出空,即 objectstack#4413 的形状。
新增 `packages/react/src/element-data-source/ElementDataSourceGate.tsx`
(`useElementDataSourceSchema` / `ElementDataSourceGate` / 两个状态面板)。#5576
的公共件本体一行未改;新增这一层只负责最后一跳 —— 把合成结果写到某个 block 真读
的那几个键上,并统一渲染 loading / 无法解析两种非终态。之所以要这一层:`dataSource`
声明在每一个 page component 上,八个 block 需要的是同一张优先级表、只有键名不同,
那是一份映射描述而不是八套算法;一 block 一份副本正是「additional filter criteria」
变成两种方言的路径。语义完全沿用 #5576(view 作基线、组件显式键覆盖 view、绑定显式
键覆盖两者、`filter` 三方 `and` 合成、view 解析不到报错不回退全量、空 `columns: []`
视作未声明)。
映射只写该 block 真读的键:把合成值写到 block 不读的键上,等于把本单要消除的缺陷
往下挪一层、看起来像接好了。所以 kanban 的 `columns`(泳道,不是字段列表)不会被
view 的字段列表覆盖,没有行上限的 block 就把 `limit` 留空。逐 block 覆盖表连同两处
残留缺口写进 `content/docs/guide/data-source.md`。
钉子:9 个新文件 46 例,每块两方向(带 `dataSource`(含 `view`)/ 不带);公共层另
18 例,钉住优先级表本身与「未映射的键一定不被写入」——两条所有 block 都依赖、而任何
单块测试都看不见的性质。
反向验证(先预判方向再跑,变异不提交):
- record_picker 恢复 pre-fix 读法(直读 `schema.dataSource`,丢 `view`):预判合成
相关两例翻红、其余四例绿。实测 2 failed / 4 passed —— 红的是 view 的
filter/sort/cap 与三方 `and` 合成;绿的是「绑定显式键覆盖」(绑定键本来就赢)、
「无法解析的 view 报错」(走 hook 那一肢,变异没碰)、无 view 基线、`properties.object`
简写。
- 公共层去掉 object 映射那一行:预判八个 block 的「查询到达数据层」全部翻红,无绑定
基线与错误面板保持绿。实测 18 failed / 28 passed,跨全部 8 个文件,分布与预判一致。
顺带记录、未在此修:objectstack#7118(`record:related_list.filter` 声明了却无人读,
因此本次故意不映射它的 `filter`,并钉了一条确认「没有任何 filter 到达 RelatedList」
的诚实断言)、objectstack#7119(`object-grid` 的 inputs 是 `filters` 复数、渲染器读
单数 `filter`)、objectstack#7120(`ListViewBlock` 的私有优先级表副本,观察类)、
objectstack#7121(剩余 object-bound public block 仍未消费该绑定)。
`plugin-kanban/src/registration.test.tsx` 顺带从整模块 mock 改成 `importOriginal()`
部分 mock —— 与 `plugin-calendar/src/registration.test.tsx` 同款转换、同一理由
(objectui#3219)。
Claude-Session: https://claude.ai/code/session_01GTRjn8xBqp75dk7kFupVRt
Co-authored-by: Claude <noreply@anthropic.com>
…tack#6953)
绑定与运行时 adapter 同名 —— `SchemaRenderer` 现在已不再 spread `schema.dataSource`
为 prop,但 host(或旧的缓存 bundle)把 BINDING 从 adapter 参数递进来时,取 saved
view 会走到「这个 data source 无法列出保存的视图」,于是一个真实存在的 view 被报成
解析不到。`ListViewBlock` 本来就带这条防御(#5576),现在提到公共层,一次覆盖所有
block。
钉子:把 `{ object, view }` 当 adapter 传入时,状态是 missing 且错误正文点明「无法
列出视图」这一事实,而不是宣称 view 不存在 —— 即没有走 binding-as-adapter 那条路。
Claude-Session: https://claude.ai/code/session_01GTRjn8xBqp75dk7kFupVRt
Co-authored-by: Claude <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
github-actions
Bot
added
documentation
Contributor
✅ Console Performance Budget
📦 Bundle Size Report
Size Limits
|
Collaborator
Author
|
PM 验收(session_01GTRjn8xBqp75dk7kFupVRt):通过,转 ready 并挂 auto-merge。objectstack#6953 落地。 核验记录(head
四条 out-of-scope(os#7118/7119/7120/7121)由 PM 分诊(另评于各单)。 Generated by Claude Code |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes objectstack-ai/objectstack#6953
这个 PR 修什么
objectstack#5576 把 spec 的 per-element 数据绑定(
PageComponentSchema.dataSource={ object, view?, filter?, sort?, limit? })接到了list-view,但同一条声明在其余每一个 page component 上依旧无人消费。本 PR 逐 block 接线,复用 #5576 落地的公共件,并沿用它定下的合成语义。两个缺口都是静默的:
element:record_picker读了五个键里的四个,单独丢掉view。 于是dataSource: { object: 'account', view: 'hot' }(spec 自己的例子)构造出的是全量 account 选择器,而不是 saved view 选出的行。它不抛错、不渲染错误面板 —— 只是候选列表比作者写的更宽,用户因此能选到页面声明为范围外的记录。这是 AI 生成的 metadata 里最藏得住的一类错误。object-grid/object-form/object-kanban/object-calendar/object-chart/object-metric/record:related_list整条绑定都不读。 每一块都把 fetch 挂在自己的objectName上,而没有任何地方把dataSource.object映射过去 —— 于是按 spec 写、不额外写objectName的页面渲染出空 grid / 无字段的表单 / 无卡片的看板 / 空月历 / 空图表 / 静态数字的指标卡,不发请求、也没有任何诊断。spec-valid 的 metadata 渲染出空,就是 objectstack#4413 的形状。接线清单(block → 文件 → 映射了哪些键)
objectview贡献filtersortlimitelement:record_pickerpackages/components/src/renderers/basic/record-picker.tsxobject-grid(+view:grid)packages/plugin-grid/src/index.tsxpagination.pageSizerecord:related_listpackages/plugin-detail/src/renderers/record-related-list.tsxlimitobject-calendar(+view:calendar)packages/plugin-calendar/src/index.tsxobject-kanban(+view:kanban)packages/plugin-kanban/src/index.tsx$top: 100object-chart(+view:chart)packages/plugin-charts/src/ObjectChart.tsxobject-metricpackages/plugin-dashboard/src/index.tsxobject-form(+view:form)packages/plugin-form/src/index.tsx⛔ 的含义是「该 block 没有这个键的读点,所以故意不映射」,不是遗漏。 把合成值写到 block 不读的键上,等于把本单要消除的缺陷(声明了却被丢弃)往下挪一层、看起来像接好了 —— 所以 kanban 的
columns(那是泳道,不是字段列表)永远不会被 view 的字段列表覆盖,没有行上限的 block 就把limit留空。逐 block 覆盖表连同两处残留缺口,写进了content/docs/guide/data-source.md。view那一列写的是「命名的 saved view 在该 block 上实际贡献了什么」。解析不到的 view 名在表里每一个 block 上都渲染配置错误 —— 包括那些从 view 里什么都不取的 block,所以写错名字在任何 block 上都不会静默通过。公共件:一张优先级表,不是八份
新增
packages/react/src/element-data-source/ElementDataSourceGate.tsx,导出useElementDataSourceSchema/ElementDataSourceGate/ElementDataSourceErrorPanel/ElementDataSourceLoadingPanel。#5576 的公共件本体(useElementDataSource、composeElementDataSource等)一行未改;新增的这一层只负责最后一跳:把合成结果写到某个 block 真读的那几个键上,并统一渲染 loading / 无法解析两种非终态。之所以要这一层:
dataSource声明在每一个 page component 上,八个 block 需要的是同一张优先级表、只有键名不同 —— 那是一份映射描述,不是八套算法。一 block 一份副本正是「additional filter criteria」会变成两种方言的路径(一块and合成、下一块直接替换)。语义完全沿用 #5576:view 提供基线 → 组件自身显式键覆盖 view → 绑定的显式键覆盖两者;
filter三方and合成(绑定只能收窄 view,永远不能放宽);view解析不到报错、不回退全量;空columns: []视作「未声明」。plugin-list的ListViewBlock早于本层落地,仍带一份私有副本(两者当前语义一致 —— #5576 整套 60 文件 826 例在本层下未改一行全绿)。把它收敛过来是 objectstack#7120(观察类)。每 block 两方向钉子
新增 9 个测试文件、48 例。每一块都同时钉住「带
dataSource(含view)时按 view 选行 / 正确映射」与「不带时行为与现状完全一致」。公共层另有 19 例,单独钉两条所有 block 都依赖、而任何单块测试都看不见的性质:优先级表本身,以及未映射的键一定不被写入(kanban 的泳道、object-form只写对象名)。反向验证(先预判方向,再跑)
两次变异,都不提交:
record_picker恢复 pre-fix 的读法(直接读schema.dataSource,丢view)。预判:合成相关的两例翻红,其余四例保持绿。实测2 failed | 4 passed—— 红的正是 view 的 filter/sort/cap 与三方and合成;绿的是「绑定显式键覆盖」(绑定键本来就赢,与 view 无关)、「无法解析的 view 报错」(那条走的是 hook 那一肢,变异没碰它)、无 view 基线、以及properties.object简写。18 failed | 28 passed,跨全部 8 个测试文件,分布与预判一致。顺手记录、未在本 PR 修的发现
record:related_list.filter声明了却无人读(RelatedList的列表查询只按父关系过滤)。因此本 PR 故意不映射 它的filter,并钉了一条诚实的反向断言:确认没有任何 filter 到达RelatedList。连带后果(saved view 的 filter 被丢、列表比它引用的 view 更宽)在代码注释与文档里明写。object-grid的inputs声明filters(复数),渲染器只读filter(单数)。ListViewBlock的私有优先级表副本(观察类,finding)。顺带的两处
ElementDataSourceGate把ListViewBlock的「BINDING 不能被当成 adapter」防御(#5576)提到公共层,一次覆盖所有 block:host(或旧的缓存 bundle)把绑定从 adapter 参数递进来时,一个真实存在的 view 会被报成解析不到。已钉住。packages/plugin-kanban/src/registration.test.tsx原本整模块替换@object-ui/react且只列两个导出,于是渲染器多用一个导出就会以「mock 上没有定义 ElementDataSourceGate 导出」失败,而不是报告它本该测的注册行为。改成importOriginal()的部分 mock —— 与plugin-calendar/src/registration.test.tsx已有的同款转换、同一理由(objectui#3219)。验证输出
Generated by Claude Code