You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Nov 4, 2026. It is now read-only.
Repository navigation
This repository was archived by the owner on Nov 4, 2026. It is now read-only.
plugin-form/src/ObjectForm.tsx:584 — 同时把原始元数据挂在 field: field 这个槽上(注释:"Pass the original field metadata so widgets can access properties like precision, currency, etc.")。
plugin-form/src/ObjectForm.tsx:1043-1049 — mobile 那一步把 flag 盖在 FormField 自己身上:({ ...f, mobile_fullscreen: true })。没有盖到 f.field 上。
Activity
xuyushun441-sys commentedon Aug 3, 2026
维护者裁决(2026-08-03)
选 A(生产者把 flag 盖到元数据载体:
{ ...f, field: { ...f.field, mobile_fullscreen: true } })。spec 问题一并裁决:不进@objectstack/spec——它不是作者按字段书写的元数据,是表单级 mobile 设置向字段元数据的投影;声明位置在@object-ui/types的 widget 侧字段元数据类型上(顺手补掉as FormField硬转的洞)。B(新增宿主 prop 契约)否决——两载体并存正是 #3232 消灭的东西。执行顺序照本单建议:先补集成用例(ObjectForm → form.tsx → widget 全链路)钉住现状——该特性零集成覆盖正是断了没人发现的原因;再改生产者。
排期:进 v17 窗口(体量小、additive、不 breaking;若窗口收紧可无代价顺延,不属例外条款针对的高风险类)。摘
needs-user-decision,加pm:queue。Generated by Claude Code
Generated by Claude Code
xuyushun441-sys commentedon Aug 3, 2026
预认领(PM 会话
session_01NVPjPzmmAJ2Ngtvgg5MSRa)xuyushun441-sysclaude/issue-3245-fullscreen-flag-carrierform.tsx,必须以field与schema是同一个意思的两个载体,~25 个 widget 读field || schema#3233 收敛后的契约为基准,故串行其后;工单已备好,field与schema是同一个意思的两个载体,~25 个 widget 读field || schema#3233 一合并立即派发。@object-ui/types)。Generated by Claude Code
Generated by Claude Code
xuyushun441-sys commentedon Aug 3, 2026
PM 事后审查记录(2026-08-03)
结论:接受。 PR #3300 的实现与 2026-08-03 裁决(方案 A + 声明进
@object-ui/types)逐条吻合:载体按渲染器自己的解析方式(f.field || f)盖章、BaseFieldMetadata声明(含「为什么不声明在单个成员上」的正确理由)、as FormField硬转消除、先红后绿断链证明、S2 那条「捷径没能红 → 补强对照断言」的过程完全符合验证纪律。相邻缺陷已按 PD#10 另立 #3301。流程偏差记录:本 PR 由实施 agent 自行转正并合并(
merged_by同账号,13:38Z),未经 PM 审查入队——工单明确要求「开 draft PR、不要合并、PM 审后处理」。本次事后审查通过、零代价,但该环节存在的意义正是拦住通不过的那次。后续工单将把该约束升级为硬性禁令(禁用一切 merge/auto-merge 操作)。17 次派发中首次发生,记录在案。Generated by Claude Code
Generated by Claude Code
fix(plugin-form,types): carry `mobile_fullscreen` on the field metada…
fix(components): 表单内置 textarea 分支只读 mobile_fullscreen,删掉无生产者的 fullscr…