plugin-gantt, app-shell designers: an invalid gantt block is warned about and drawn anyway, and a failed spec derivation degrades the designer form instead of failing — refuse loudly #13459

Activity

1. os-zhuang

   added

   bugSomething isn't workingSomething isn't working

   sduiServer-Driven UI runtimeServer-Driven UI runtime

   finding

   domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatobjectui ui stream: fix lands on the published library or apps — objectui execution seat

   on Oct 24, 2026
2. objectstack-fleet

   added

   area:studioChanging a running app without code — authoring, publish, docs and the portalChanging a running app without code — authoring, publish, docs and the portal

   pm:queue

   priority:p2

   and removed

   finding

   on Oct 24, 2026
3. objectstack-fleet commented on Oct 24, 2026

   

   objectstack-fleetBot

   on Oct 24, 2026

   Contributor

   More actions

   Triage: first grade — bug · priority:p2 · domain:ui · area:studio · pm:queue · triage seat · session_01AavokzJ5DndAwitDXvKy4U · 2026-10-24T17:03Z

   ⛔ Not a claim, ⛔ not a dispatch.
   finding comes off.

   • An invalid gantt block is warned about and drawn anyway; it should fail loudly, as the rest of the metadata path does (Prime Directive 12: never a lenient fallback in a consumer). A failed spec derivation in the designer should show an error, not a degraded form.

   ---

   Generated by Claude Code
4. objectstack-fleet

   added

   pm:dispatched

   and removed

   pm:queue

   on Oct 25, 2026
5. objectstack-fleet

   assigned

   os-elon-musk

   on Oct 25, 2026
6. objectstack-fleet commented on Oct 25, 2026

   

   objectstack-fleetBot

   on Oct 25, 2026

   Contributor

   More actions

   Claim: PM loop round 1 · 2026-10-25T02:01Z
   Session: session_01AdaeVx4N1y9QsiBkJ31nfJ (the card session that works this card; claim written by the domain:ui seat 4 session session_01XWGTsmqrtrt1Zqh1ioWqsT)
   Account: os-elon-musk
   Branch: claude/project-thread-y2g2x6 (platform-assigned at spawn; the card session can push only this branch)
   Worktree: objectui-issue-13459
   Domain: domain:ui
   Seat: domain:ui#4
   File surface: packages/plugin-gantt/src/ObjectGantt.tsx (the GanttConfigSchema.safeParse branch and the refusal it renders); the five derivation sites in packages/app-shell/src/views/metadata-admin/{dashboard,package,page,report,view}-schema.ts; their tests; .changeset/13459-*.md (stop on breach; explain in the report)
   Container & model: M, mode:thread, model: default tier (dispatch-gates --tier --repo objectstack-ai/objectui: no path-derived mandate), effort: high
   Clause-②: no
   Metadata-protocol: narrowed
   Responsibility: objectui's ObjectGantt draws a gantt block that fails GanttConfigSchema, and the metadata-admin designers degrade their form when spec derivation fails | ObjectTimeline's refusal posture ("日期轴永不虚构") | every author whose gantt block or designer form is wrong and is never told
   Thread-read: 6387054722
   Premise: holds — on objectui main c85408f, ObjectGantt.tsx:616 runs GanttConfigSchema.safeParse(config) and goes on with the unvalidated config on failure; failed to derive catch → warn sites sit in the five metadata-admin/*-schema.ts files above
   Serial constraints cleared: seat 1's objectui#13442 / #13449 / #13450 declare no path under plugin-gantt/src/** or app-shell/src/views/metadata-admin/**; seat 4's in-flight PRs (#13480–#13487) touch neither; no other in-flight card names these files
   Writes: all PR, comment and label writes go only through objectstack's relay (scripts/pm/fleet-write); never write with GitHub MCP.

   • Ruling section: filed under the maintainer's 「同意：建议新建：与合并无关、独立成卡的产品/契约项（可现在建）」; triage 6387054722: fail loudly, show an error in the designer instead of a degraded form.
   • One PR from this branch carries both items (the card's "one PR each" is two S halves of one M card; the platform gives one branch).
   • Must vs may: the gantt refusal and the designer derivation are the deliverable. A derivation from spec that fails becomes a failing test (build-time fact); the page shows an error instead of a silently thinner form; a runtime catch stays only where the input is genuinely runtime. ⛔ tzShift.ts (timeZone fallback) is out: the card ties it to a spec-row decision.
   • Clause-② reason: a block that rendered before is refused now; that narrows what renders and widens nothing. A narrowing owes one contract-review-tier review before the queue, which the seat arranges; the dev hangs no labels for it.
7. objectstack-fleet commented on Oct 25, 2026

   

   objectstack-fleetBot

   on Oct 25, 2026

   Contributor

   More actions

   os-dev-report

   {
     "issue": 13459,
     "status": "done",
     "branch": "claude/project-thread-y2g2x6",
     "pr": "https://github.com/objectstack-ai/objectui/pull/13490",
     "session": "session_01AdaeVx4N1y9QsiBkJ31nfJ",
     "premise_still_valid": true,
     "summary": "ObjectGantt now refuses a gantt block that fails GanttConfigSchema: getGanttConfig returns config null plus the parse issues, and the component renders a role=alert refusal naming each issue by its path in the block (gantt.endDateField: ...), instead of drawing the unvalidated block; valid blocks, the flat ObjectView/ListView spelling and the no-binding screen are unchanged. The ten catch-and-warn sites in metadata-admin/{dashboard,package,page,report,view}-schema.ts are removed with their _Failed flags, so a failed derivation from the bundled spec throws: tests go red and at runtime the route ErrorBoundary shows it instead of a thinner form; getter return types narrow to JsonSchema. tzShift.ts untouched per the claim. One PR, both halves, inside the claimed file surface (an i18n-pack draft of the refusal copy was reverted to stay inside it; copy is English like the neighbouring refusal).",
     "tests": "head e26d1cd. New ObjectGantt.invalidBlockRefusal-13459.test.tsx (4) + spec-derivation-13459.test.ts (14): 2 files 18 passed. vitest run packages/plugin-gantt/: Test Files 100 passed | 1 skipped (101), Tests 775 passed | 114 skipped. App-shell narrowed to the 88 test files importing a changed module or its callers: 88 passed, 721 tests passed (blind spot declared in PR body; CI shards the full package). Ablation, fix committed first, restored with git checkout HEAD --, git diff HEAD empty: ObjectGantt.tsx at origin/main -> 3 refusal cases fail, CONTROL passes; view-schema.ts at origin/main -> 3 view-schema throw cases fail. A first hoisted vi.mock('zod') leaked into view-schema.test.ts (3 failures in the 88-file run); replaced with scoped vi.doMock/doUnmock in a3b60c1, re-run green.",
     "gates": "local: eslint --no-inline-config on touched files :: exit 0 (0 errors, pre-existing warnings); type-check @object-ui/plugin-gantt :: exit 0; type-check @object-ui/app-shell :: exit 0 (turbo); check:new-line-citations :: exit 0; check:changeset-claims :: exit 0; check-changeset-no-major :: exit 0; check-changeset-presence :: exit 0; check:i18n-keys :: exit 0; check:vi-mock-specifiers :: exit 0; check:vi-mock-inherit :: exit 0; check:vi-mock-override-shape :: exit 0; check:control-bytes :: exit 0; check:test-path-roots :: exit 0. check:readme-exports NOT MEASURED (needs all dists). objectui has no dispatch-gates.mjs; list hand-derived from package.json. CI check-runs: in_progress at report time.",
     "dogfood_battery": "n/a — no ! in the PR title and Clause-②: no",
     "mcp_calls": "2 — mcp__github__issue_read (get, get_comments); no write tool",
     "api_writes": "3 — relay pr_create objectstack-ai/objectui (#13490, assignee os-elon-musk), relay pr_request_reviewers #13490, relay comment on #13459 (this report)",
     "open_questions": [],
     "out_of_scope_findings": [
       "class: (a) candidate, not filed · reach: NOT MEASURED through a public door · evidence: packages/plugin-timeline/src/ObjectTimeline.tsx runs TimelineExtensionSchema.safeParse(schema) in an effect and only console.warns on failure, the same warn-and-draw shape this card removed from the gantt · dedupe words: TimelineExtensionSchema safeParse warn refusal",
       "carrier: none · noted in PR Acceptance notes"
     ]
   }

   ---

   Generated by Claude Code
8. objectstack-fleet commented on Oct 25, 2026

   

   objectstack-fleetBot

   on Oct 25, 2026

   Contributor

   More actions

   ACCEPT → PR #13490 comment 6392708594 · domain:ui#4 · 2026-10-25T03:14Z. Metadata-protocol is narrowed, so the PR carries needs:contract-review and lands only after the director's contract-review record.
9. objectstack-fleet

   closed this as completedin #13490on Oct 25, 2026
10. objectstack-fleet commented on Oct 25, 2026

    

    objectstack-fleetBot

    on Oct 25, 2026

    Contributor

    More actions

    Landed · domain:ui#4 · 2026-10-25T04:19Z

    PR #13490 merged at 2026-10-25T04:17Z, merge commit 320ec359e4b9eabcfb21022b77da2242e2e4b529, after the director's contract review PASS on the same head. The card closed completed through Fixes; pm:dispatched comes off in this stroke. objectui merges publish nothing (the changesets release is a human act).

    ---

    Generated by Claude Code
11. objectstack-fleet

    removed

    pm:dispatched

    on Oct 25, 2026
12. akarma-synetal

    added a commit that references this issue on Oct 28, 2026

    fix(plugin-gantt, app-shell): refuse an invalid gantt block; designer…

    ...

    This issue will close once commit 320ec35 is merged into the 'main' branch.

    320ec35