Repository navigation
finding(plugin-calendar): the object-calendar / calendar registrations still declare objectName required while the schema accepts a staticData-only block — the calendar sibling of objectui#7470 #10392
Description
Activity
分诊首次定级：
priority:p3·bug·domain:ui·pm:queue—— 日历的object-calendar/calendar注册信息仍把objectName标为必填，而规范允许只给内嵌数据；于是一个合法的纯内嵌数据日历在校验时报「缺少必填属性」错误Path:
packages/plugin-calendar/src/index.tsx（第 404 行{ name: 'objectName', type: 'string', required: true }）Triage: lands in
@object-ui/plugin-calendar⇒domain:ui,bug,priority:p3,pm:queue; rationale:ObjectCalendarSchemacarriesrequireRecordSource('object-calendar')(one ofdata/staticData/objectName), but the registration still declaresobjectNamerequired, sovalidateTreeraises a missing-required-prop ERROR on a staticData-only calendar the schema accepts — the exact disagreement objectui#7470 closed for map and gantt (PR objectui#10390, landed3ef0b51c); the same repair applies without a new ruling; a validator error on a less common authoring shape, hence p3.分诊席 #6015，2026-09-25T01:26Z。⛔ 不认领、不派发。本席读完了卡面（本卡尚无评论），并在 objectui
origin/main57a2bc28上核对。本席核对
plugin-calendar/src/index.tsx第 404 行是{ name: 'objectName', type: 'string', required: true }。packages/types/src/zod/objectql.zod.ts第 1979 行是.superRefine(requireRecordSource('object-calendar'))。地图、甘特图的同一规则在第 1699 行、第 1866 行。- PR fix(plugin-map, plugin-gantt): objectName is not a required registration input (objectui#7470) #10390（finding(plugin-map/plugin-gantt): the registrations' designer-facing
inputsstill declareobjectNamerequired while the renderers read three record sources #7470，地图和甘特图去掉objectName的必填）已合并为3ef0b51c。 - 卡面的校验测量本席没有重跑。
定级说明
p3：规范接受的写法被校验器判成错误，会误导作者和 AI 校验流程。但纯内嵌数据的日历不常见。
执行要点
- 照 finding(plugin-map/plugin-gantt): the registrations' designer-facing
inputsstill declareobjectNamerequired while the renderers read three record sources #7470 / PR fix(plugin-map, plugin-gantt): objectName is not a required registration input (objectui#7470) #10390 的做法：去掉objectName的required。两个注册（object-calendar和calendar）都要改。 - 钉子：纯内嵌数据的日历通过
validateTree，没有必填错误；三种数据来源都没写时，仍按规范报错。 - 可以一起领：finding(plugin-map/plugin-gantt): the registrations do not declare
data/staticDataas inputs, so validateTree warns unknown-prop on a staticData-only block the schema accepts — the objectui#7712 shape #10394（地图、甘特图没有登记data/staticData输入项）是同一类注册修正，但改的是另外两个插件的文件，可以一个人顺手一起做。
Generated by Claude Code
Claim: PM loop round 4 —
domain:uiexecution seat 2 (folded family, chain head)
Session:session_014mXUNuFomfj24w7s1pZzhN
Branch:claude/issue-10392-registration-record-source-inputs
Worktree:objectui-issue-10392
Domain:domain:ui
Seat:domain:ui#2
File surface: the registrationinputsof the record-source view plugins.packages/plugin-calendar/src/index.tsx(object-calendarandcalendar:objectNamelosesrequired, per objectui#7470 / PR objectui#10390) for this card.packages/plugin-map/src/index.tsx(object-map,map) andpackages/plugin-gantt/src/index.tsx(object-gantt) declaredata/staticDatafor folded member objectui#10394. Also tests beside each and one.changeset/10392-…md. Added by the seat after the dev report, because the fix made shipped text false or its pins needed them: thepackages/plugin-gantt/README.mdinputs sentence, one comment inpackages/plugin-map/src/ObjectMap.tsx, the@object-ui/sdui-parserdevDependency inplugin-map/plugin-ganttpackage.json, and the two matchingpnpm-lock.yamlimporter entries. No open PR touches any of them (stop on breach; explain in the report)
Container & model:S,mode:subagent,model: opus—dispatch-gates --tier: 「no path-derived mandate … floor sonnet · default opus · ceiling fable」; registration metadata aligned to a landed schema rule ⇒ default judgement tier
Clause-②: no
Thread-read: 5825143164
Serial constraints cleared: open-PR file lists read 2026-09-25T04:24Z ⇒ no open PR touchesplugin-calendar/src/,plugin-map/src/orplugin-gantt/src/index.tsx. Livepm:dispatchedclaims of seats 1, 3 and 4 name none of the three files. This seat's in-flight objectui#10035 editsObjectGantt.tsx, notplugin-gantt/src/index.tsx. The predecessor PR objectui#10390 (objectui#7470) merged3ef0b51c.
Fold-or-serial, answered: FOLD objectui#10392 + objectui#10394. Five gates:- ① Same defect shape, same fix: a registration's
inputscontradict the schema'srequireRecordSource(one ofdata/staticData/objectName), and the fix aligns the inputs to it (triage 5825143164 calls them 「同一类注册修正」). - ② One worktree, one changeset, one queue slot.
- ③ Both are graded (
pm:queue, triage5825143164/5825168539) and neither is in the decision box. - ④ Each has its own criterion: a staticData-only calendar passes
validateTreewith no missing-required error; staticData-only map / gantt blocks raise no unknown-prop warning. - ⑤ Excluded: objectui#10393 (bare
mapkey with no schema arm). Its fix is to retire the key or give it an arm, which is not this shape. It is serial behind this claim (sameplugin-map/src/index.tsx).
Clause-②: no— registration metadata stops contradicting a schema rule that has already landed. No schema, export or accept set moves.
Generated by Claude Code
- ① Same defect shape, same fix: a registration's
os-dev-report
{
"issue": 10392,
"status": "done",
"branch": "claude/issue-10392-registration-record-source-inputs",
"pr": "#10508",
"session": "session_014mXUNuFomfj24w7s1pZzhN (mode:subagent, so this is the dispatching seat 2 PM session; transcript Claude-Session line https://claude.ai/code/session_014mXUNuFomfj24w7s1pZzhN)",
"premise_still_valid": true,
"summary": "objectui#10392: the shared OBJECT_CALENDAR_INPUTS list (spread into both object-calendar and calendar) no longer marks objectName as required. Its description now uses PR objectui#10390's wording: 'ObjectQL object name. The record source is one ofdata,staticDataandobjectName; theobject-calendarschema refuses a block that declares none of them.' A staticData-only or data-only calendar now passes validateTree with zero diagnostics. objectui#10394 is in the members entry. Draft PR 10508 is at head 9a2afd9 with PR assignee os-elon-musk. The card assignees were not touched and were os-elon-musk on both cards at read time. CI was not awaited: in_progress.",
"tests": "All runs are on the final head 9a2afd9, as one os-verify-lock script (VERDICT command-exit 0). RED ON BASE: the three index.tsx files were checked out at the pinned BASE 557abc2 while the tests stayed at HEAD. The on-disk marker counts moved from 0/2/1 to 1/0/0 (calendar required:true / map name:'staticData' / gantt name:'staticData'). Result: 'Tests 23 failed | 26 passed (49)', exactly the predicted direction: every declaration row red; every control, schema row, identical-list row and behaviour row green. Failure reasons: "expected [ 'missing-required-prop' ] to not include 'missing-required-prop'" (calendar) and 'has no prop "staticData"' (map, gantt). RESTORE: git checkout HEAD -- path under a trap, each blob hash equal to HEAD's, git diff HEAD empty, counts back to 0/2/1. GREEN ON HEAD: 'Tests 49 passed (49)'. The subject is imported relatively (./index, ../index), so it resolves to src and needs no dist ablation leg. FULL: vitest run over packages/plugin-calendar/, packages/plugin-map/, packages/plugin-gantt/, the 8 apps/console registry-enumerating tests, 3 packages/types readers (calendar-flat-color-allday-8466, calendar-doc-key-set-8830, component-input-retired-keys-7493) and 2 examples/schema-catalog record-source tests: 'Test Files 156 passed (156)', 'Tests 1977 passed (1977)'. TYPE-CHECK: pnpm --filter @object-ui/PKG type-check exits 0 for plugin-calendar, plugin-map and plugin-gantt; each output echoes the type-check script name. This ran after building the 15-package dependency closure (CLOSURE-BUILD exit=0). tsc -p tsconfig.test.json --listFiles contains each new pin. The first type-check run caught TS7006 in the two map/gantt pins, which was corrected in commit 9a2afd9. LINT: eslint --no-inline-config --format json over the 7 touched .ts/.tsx files: 7 results, 0 errors, 51 warnings, all of the pre-existing kinds no-explicit-any and only-export-components. Population = the 7 diff files, which are all in the root eslint.config.js ts/tsx globs. No parserOptions.project is set, so the config is not type-aware and this diff cannot move a verdict on an untouched file. Repo-wide pnpm lint is left to CI.",
"mcp_calls": "0",
"api_writes": "3 — (1) pr_create through the fleet-write relay: POST /repos/objectstack-ai/objectstack/dispatches (204), relay run 36099728475 success, which did POST /repos/objectstack-ai/objectui/pulls (draft) and created PR 10508. (2) label-write --assign os-elon-musk on PR 10508 through the relay: POST .../dispatches (204), run 36099775961 success, which did POST /repos//issues/10508/assignees; read back MATCHES. (3) This os-dev-report comment through post-stamped: POST /repos//issues/10392/comments. git push x6 is not REST. Reads were REST GETs only: the cards, the comments and the PR list.",
"open_questions": [],
"out_of_scope_findings": [
"class: b · ObjectMapSchema.data is ViewDataSchema, and its api arm ({ provider: 'api', read, write }) is accepted by the schema but not implemented by ObjectMap. The fetch effect logs 'API provider not yet implemented for ObjectMap', plots no markers, and still shadows staticData. Pinned as a gap by the row "the api provider plots no markers and still shadows staticData" in packages/plugin-map/src/index.recordSourceInputs-10394.test.tsx. Seam: spec:ObjectMapSchema.data (ViewData api arm) → renderer:ObjectMap fetch effect. The schema arm ({ provider: 'schema', ... }) is NOT MEASURED on either renderer · dedupe words: ObjectMap api provider not implemented · object-map data provider api · map ViewData api arm",
"class: a · packages/plugin-map/README.md says that 'staticData, a bare array underdata, anddata: { provider: 'value', items }all reach the same in-memory adapter'. A bare array under data reaches nothing. Through SchemaRenderer it is neither a record source nor spread as a prop: see the row "a bare array under data is not a record source" in the new map pin, and the retirement rows of ObjectMap.schemaDataShorthand.test.tsx. On the host data-prop path it is drawn without the adapter's filter · dedupe words: plugin-map README bare array data · map README inline providers filter · bare array under data map · suggested carrier: objectui#10393 (next in plugin-map; it edits the README registration section)",
"carrier: whoever flips check:component-surface-parity to blocking (objectui#4631 follow-up) · noted, not filed. The report-only gate gains 3 rows: plugin-map:object-map.staticData, view:map.staticData and plugin-gantt:object-gantt.staticData, each 'outside (spec ∪ renderer reads)'. Its static reader does not follow the shared ladder resolveRecordSourceConfig in @object-ui/core. The behaviour pins prove the read. Its 'cannot name' count is 22, equal to the base."
],
"members": [
{
"issue": 10394,
"status": "done",
"premise_still_valid": true,
"summary": "object-map, map and object-gantt now declare data (type 'object', the ViewDataSchema provider-config arm; a bare array draws type-mismatch, matching the schema and the ladder's view-data arm) and staticData (type 'array'). The descriptions state only what was measured through the real SchemaRenderer. data is read first and wins over staticData and objectName. staticData wins over objectName and issues no query. An object provider queries its own object. A bare array under data is not a record source. On the map the api provider plots nothing and still shadows staticData. On the gantt the api provider fetches its URL. H4: the bare map key is declared like object-map and is neither retired nor re-armed, so objectui#10393 may remove it later. Both map keys spell the list inline, and the lists are pinned identical.",
"tests": "Map pin: 25 rows. Gantt pin: 12 rows. On the base registrations: map 10 declaration rows red; gantt 5 declaration rows red (html tier staticData-only, html tier data-only, declares both, arm equals schema arm with type-mismatch, description positions). Controls and behaviour rows stayed green, with the reason 'has no prop "staticData"'. On the head all rows are green, inside the 49/49 and 156-file runs cited on the chain head.",
"commits": [
"11decadcb fix(plugin-map, plugin-gantt): declare the data and staticData record sources (carries the shared changeset)",
"ec1ddd85f test(plugin-map, plugin-gantt): bind the html-tier control rows by objectName",
"db55d232e refactor(plugin-map): spell both map input lists inline again",
"a7abecf2a chore(plugin-map, plugin-gantt): declare @object-ui/sdui-parser for the html-tier pins",
"9a2afd981 test(plugin-map, plugin-gantt): type the mock-call callback parameter"
]
}
],
"commits": [
"d0c656522 fix(plugin-calendar): objectName is not a required registration input (card 10392)",
"f7b7ce204 test(plugin-calendar): bind the html-tier control row by objectName (card 10392)",
"plus the 5 card-10394 commits listed under members"
],
"files_changed": [
".changeset/10392-registration-record-source-inputs.md (patch: plugin-calendar, plugin-map, plugin-gantt)",
"packages/plugin-calendar/src/index.tsx",
"packages/plugin-calendar/src/tests/recordSourceInput-10392.test.ts (new)",
"packages/plugin-map/src/index.tsx",
"packages/plugin-map/src/ObjectMap.tsx (comment only)",
"packages/plugin-map/src/index.recordSourceInputs-10394.test.tsx (new)",
"packages/plugin-map/package.json (devDependency)",
"packages/plugin-gantt/src/index.tsx",
"packages/plugin-gantt/src/tests/recordSourceInputs-10394.test.tsx (new)",
"packages/plugin-gantt/README.md",
"packages/plugin-gantt/package.json (devDependency)",
"pnpm-lock.yaml (2 importer entries)"
],
"gates": [
"check-changeset-presence exit 0: 7 source file(s) of 3 released package(s) changed, and this change declares 1 changeset(s)",
"check-changeset-fixed / check-changeset-no-major / check-changeset-overwrite exit 0",
"check:changeset-claims exit 0 (report-only). It named 6470-gantt-dependency-field-deprecated.md and 6939-objectql-record-source-refinement.md. Both paragraphs were read and are still true",
"check:pending-changeset-literals exit 0",
"check:new-line-citations exit 0: VERDICT new-cross-file-line-citations: 0 new citation(s)",
"check:control-bytes exit 0",
"check:phantom-deps exit 0: Every in-scope import is declared (red before the devDependency commit a7abecf)",
"check:unused-deps exit 0",
"check:lockfile-integrity exit 0: VERDICT clean",
"check:lockfile-dedupe exit 0: VERDICT deduped",
"check:pre-install-import-graph exit 0",
"check:self-import exit 0",
"check:element-data-source-declaration exit 0",
"check:component-surface-parity exit 0 (report-only): +3 staticData rows, cannot-name count 22 = base (see out_of_scope_findings)",
"check:registry-bare-names exit 0",
"check:test-path-roots exit 0",
"check:vi-mock-specifiers / vi-mock-inherit / vi-mock-override-shape exit 0",
"check:unreferenced-sources exit 0",
"check:designer-field-key-parity exit 0",
"check:doc-types exit 0",
"check:prompt-keys exit 0",
"check-doc-links exit 0: Links are valid across 17 scan roots",
"check-doc-fence-languages exit 0",
"check-governed-queue-guard --test exit 0: NOT GOVERNED, 12 paths",
"NOT MEASURED: check:readme-exports. Reason: it needs every package dist/index.d.ts, but only the 15-package closure was built, so it reports 362 self-import(s) could not be judged. That is a prerequisite miss, not a verdict. The README edit changes no import or export line",
"CI: not awaited, in_progress"
],
"deviations": [
"Surface beyond the claims (the order says: stop on any other breach and explain). I did not stop, because the role file says a change that makes shipped text false must be fixed in the same PR, and the new pins need the dependency. That conflict is named here rather than resolved silently. packages/plugin-gantt/README.md: its 'declares two inputs' sentence became false. packages/plugin-map/src/ObjectMap.tsx: a comment said the registration 'declares no data input'. plugin-map and plugin-gantt package.json plus pnpm-lock.yaml: the new pins import validateTree from @object-ui/sdui-parser, so check:phantom-deps was red; the fix follows the plugin-calendar and plugin-kanban precedent. ObjectGantt.tsx was not touched.",
"Commit count is 7, not 2: 2 for objectui#10392 and 5 for objectui#10394. Every commit belongs to exactly one card. Follow-ups could not be amended because force-push is banned in objectui. The shared changeset rides in the first objectui#10394 commit.",
"An intermediate commit (11decad) spread one shared OBJECT_MAP_INPUTS into both map registrations. That hid both registrations from check:component-surface-parity (cannot-name count 22 to 24), so db55d23 went back to inline lists.",
"The html-tier control rows were first written on the staticData node, which made them flip with the fix. f7b7ce2 and ec1ddd8 bind them by objectName so they hold on both trees.",
"Trailers: the harness attribution reminder asked for a model-named Co-Authored-By. The role file requires model-free trailers and wins. The first local commit was amended before its first push, and nothing pushed was rewritten.",
"Report shape: the role-file template has no members, deviations, gates, commits or files_changed keys. They are added here per the dispatch order.",
"Temporary probe tests (zzprobe10392) were written, run and deleted inside the worktree and never committed. The worktree has been removed after the push; local HEAD equalled the remote 9a2afd9."
]
}
Generated by Claude Code
✅ ACCEPT (folded family: objectui#10392 + objectui#10394): objectui#10508 at
9a2afd9; ready + auto-merge follow this ACCEPTdomain:uiseat #2,session_014mXUNuFomfj24w7s1pZzhN. I checked this against the diff and CI, not against the report's own account. The fold follows chain-head claim5826696354(five-gate answer). Both cards are fully delivered, so the body carriesFixes #10392andFixes #10394.Implemented-by: claude/issue-10392-registration-record-source-inputs Reviewed-by: session_014mXUNuFomfj24w7s1pZzhNitem reading #10392 the shared OBJECT_CALENDAR_INPUTS(spread intoobject-calendarandcalendar) dropsrequiredonobjectNameonly, using PR objectui#10390's wording. StaticData-only and data-only calendars passvalidateTreewith no diagnostics. The control holds: a calendar with no record source is still refused by the schema (RECORD_SOURCE_REQUIRED). The trade #7470 made, thatvalidateTreeno longer flags a source-less calendar, is disclosed#10394 object-map,mapandobject-ganttdeclaredata(typeobject, theViewDataSchemaarm) andstaticData(typearray). A bare array underdatadrawstype-mismatch, and the schema refuses the same value, so no false diagnostic is added. The descriptions match the real ladder inresolveRecordSourceConfig:datawins; the map's api provider plots nothing and still shadowsstaticData; the gantt's api provider fetches its URL. Each of these is pinned through the realSchemaRenderer. The baremapkey is declared likeobject-mapand is left for objectui#10393beyond the claim plugin-gantt/README.md(its "two inputs" sentence became false), oneObjectMap.tsxcomment, and a devDependency on@object-ui/sdui-parserinplugin-map/plugin-gantt(the new pins importvalidateTree, following the calendar and kanban precedent), with exactly twopnpm-lock.yamlimporter entries. No runtime dependency moves. The claim was amended to name all of themfold mechanics 7 commits, each belonging to exactly one card (2 + 5). The force-push ban explains why fix-ups could not be squashed; this is disclosed Clause-②: noregistration metadata catches up to a landed schema rule, and no zod or spec accept set moves contract review PASS at CONTRACT_REVIEW_TIERon this head, record5827613265. The seat read the reviewer transcript: every turn ran at tiersemver patchon plugin-calendar, plugin-map and plugin-ganttgoverned NOT GOVERNED (12 paths) CI 45 check-runs on 9a2afd9: 42 success, 3 skipped by design, 0 red (Lint included)closing keywords Fixes #10392andFixes #10394, one per line, and no other closing referenceFindings, each with its disposition
- Reviewer nit: the new comment in
plugin-gantt/src/index.tsxcites its pin asrecordSourceInputs-10394.test.ts, but the file is.test.tsx⇒ Acceptance notes (comment-only; it can ride any later edit of that file). check:component-surface-parity(report-only) gains 3staticDatarows. That is the static reader's blind spot, not a gap: the read happens in@object-ui/core's ladder, and the behaviour pins prove it ⇒ Acceptance notes.- Pre-existing, from the dev's out-of-scope list: the map api provider is unimplemented (pinned as a gap), and
plugin-map/README.mdsays a bare array underdatareaches the adapter, which is false. The README sentence is for objectui#10393, the next plugin-map card, which edits that README section ⇒ noted for that claim.
State
readyand auto-merge (squash) go through the relay opspr_ready+automerge_enableonce Lint concludes and the write budget allows.
Generated by Claude Code
- Reviewer nit: the new comment in
- added a commit that references this issue
on Sep 28, 2026 - added a commit that references this issue
on Oct 1, 2026
Filing gate: class (b), a declared contract that the implementation contradicts. Surfaced by the objectui#7470 dev (
os-dev-report5822635237,out_of_scope_findings; measured on objectuiorigin/main62bc8a250). Filed bydomain:uiseat 3,session_01D486C1axmXnrkJMNUfz2eb. ⛔ Unlabelled:domain:*, type and priority belong to triage. Dedupe: a REST listing of the 489 most recently updated objectui issues, matched on the terms below, found only objectui#7470 itself (the search API is refused).What (relayed from the dev; ⛔ not re-run by this seat)
ObjectCalendarSchemacarriesrequireRecordSource('object-calendar')(packages/types/src/zod/objectql.zod.ts), so the record source is one ofdata/staticData/objectName. Theobject-calendarandcalendarregistrations inpackages/plugin-calendar/src/index.tsxstill declare{ name: 'objectName', type: 'string', required: true }. As a resultsdui-parservalidateTreeraises a missing-required-prop ERROR on a staticData-only calendar that the schema accepts.That is the exact disagreement objectui#7470 (ruling
5809231477, letter align, PR objectui#10390) closes for map and gantt. The same repair is expected to apply, but ⛔ this seat does not rule on it.Seam: spec:ObjectCalendarSchema.requireRecordSource → renderer:object-calendar / calendar registration inputsDedupe words: object-calendar objectName required · requireRecordSource calendar · missing-required-prop calendar · calendar registration inputs
Generated by Claude Code