Skip to content

chore: bump objectui to 6c1ad9e20624 (pulls in #2620 fixes)#2627

Merged
os-zhuang merged 1 commit into
mainfrom
claude/verify-objectui-2620-fixes
Jul 5, 2026
Merged

chore: bump objectui to 6c1ad9e20624 (pulls in #2620 fixes)#2627
os-zhuang merged 1 commit into
mainfrom
claude/verify-objectui-2620-fixes

Conversation

@os-zhuang

Copy link
Copy Markdown
Contributor

Summary

Bumps .objectui-sha to pull in objectstack-ai/objectui#2254, which resolved most of the objectui-side items tracked in #2620 (follow-ups from the #2616 Showcase UX pass).

Verified live against a --fresh --seed-admin instance with the console rebuilt at this SHA (pnpm objectui:refresh):

Item Status
A1 — Sidebar nav labels (Projects/Tasks/Accounts/Settings) Fixed — explicit plural labels now win over i18n object labels
A3 — Approvals "Mark Done" Fixed — no unbound header button; per-row action does a real write, toast timing matches
B1 — Wizard submitBehavior Still broken — see below
B2 — "Overdue Nd" on START DATE Fixed — past start dates now render as neutral "Nd ago"
C2 — Work Map tile-load fallback Fixed — visible banner on MapLibre load failure instead of a silent blank map
D — Command Center chart width/labels ⚠️ Partially fixed — see below
E — console warnings (bare-name fallback, AuthProvider, ai/agents spam) Fixed — none observed across a navigation sweep

Residual issues found during verification (not fixed by objectui#2254 despite being in scope)

  • B1: completing the wizard still leaves the filled step-3 form fully re-submittable. Clicked "Create" twice in a row → confirmed via GET /api/v1/data/showcase_project that two duplicate records were created. The submitBehavior.title/message text now correctly surfaces in a toast (so the spec-bridge fix landed), but the wizard doesn't swap to a confirmation panel or otherwise block re-submission the way a non-wizard form's thank-you behavior implies.
  • D: measured the actual chart <svg> bounding boxes via DOM query — the Command Center's row-A charts render at exactly 100px wide (same collapse class as originally reported, if not worse). Also found literal "task_count" text nodes still rendered as a caption/tooltip label under/in each chart, even though the x-axis category labels (e.g. "To Do", "Urgent") are now correctly humanized.

Filed objectstack-ai/objectui#<TBD> with the concrete repro/evidence for both (screenshots + measured bounding boxes + API before/after diffs) — see linked issue.

Test plan

  • OBJECTUI_ROOT=/workspace/objectui pnpm objectui:refresh — console rebuilt clean, bundle canary present
  • pnpm build — 70/70 tasks green
  • Live browser verification (Playwright + Chromium) of all 7 items above, with screenshots and API-level before/after checks for the two still-broken items

Note: packages/console/dist is gitignored (build artifact) — this PR only changes the .objectui-sha pin; CI/release tooling rebuilds the console from that pin.


Generated by Claude Code

feat(app-shell,plugin-view,plugin-form): record task flows as derived overlays + return invariants (framework#2604 Step 2) (#2256)

objectui@6c1ad9e206249735e8ebf7b77fca75313da28070
@vercel

vercel Bot commented Jul 5, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
spec Ready Ready Preview, Comment Jul 5, 2026 7:12am

Request Review

@os-zhuang
os-zhuang marked this pull request as ready for review July 5, 2026 07:53
@os-zhuang
os-zhuang merged commit 27b5745 into main Jul 5, 2026
15 checks passed
@os-zhuang
os-zhuang deleted the claude/verify-objectui-2620-fixes branch July 5, 2026 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants