Skip to content

fix(app-showcase): drop wrong-context userFilters from showcase_task.tabular + reclassify P0-30#2564

Merged
os-zhuang merged 1 commit into
mainfrom
fix/showcase-userfilters-adr0053
Jul 4, 2026
Merged

fix(app-showcase): drop wrong-context userFilters from showcase_task.tabular + reclassify P0-30#2564
os-zhuang merged 1 commit into
mainfrom
fix/showcase-userfilters-adr0053

Conversation

@baozhoutao

Copy link
Copy Markdown
Contributor

Fixes #2563

What

Dogfood finding P0-30 ("view userFilters quick filters don't render at all") was investigated against objectui source and is by design, not a rendering bug: objectui ADR-0053 reserves userFilters for page lists (InterfaceListPage, "filters" mode); on an object list view ("views" mode) the console suppresses them — the view switcher is the only nav control there. The real defects were:

  1. the drop was silent (ADR-0053 phase-4 guardrail never landed) → filed as objectui#2219, fixed by objectui#2220 (one-shot console warning);
  2. our own showcase carried the wrong-context block — showcase_task.tabular defined userFilters under a stale ADR-0047 comment, while the same file's header already states the ADR-0053 rule and the page examples (task-workbench / task-triage / active-projects) demo the correct usage.

Changes

  • examples/app-showcase/src/views/task.view.ts — remove the userFilters block from the tabular view; comment now points at the ADR-0053 split (named views on the object, userFilters in the page examples).
  • examples/app-showcase/docs/test/console-list-form-capability-report-2026-07.md — reclassify P0-30 (by-design suppression + silent-drop DX gap + wrong-context metadata) and mark P0-29 fixed upstream (objectui#2218).

Verification

Against the running showcase dev stack (port 3777):

  • /api/v1/meta/app?id=showcase_app and /api/v1/meta/objects/showcase_task contain zero userFilters occurrences after the change;
  • browser check: showcase_task.tabular renders its own columns/tab correctly, no ADR-0053 suppression warning on a clean load.

🤖 Generated with Claude Code

@vercel

vercel Bot commented Jul 4, 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 4, 2026 6:54am

Request Review

@github-actions github-actions Bot added documentation Improvements or additions to documentation size/s and removed documentation Improvements or additions to documentation labels Jul 4, 2026
@os-zhuang
os-zhuang merged commit 92bcf84 into main Jul 4, 2026
15 checks passed
@os-zhuang
os-zhuang deleted the fix/showcase-userfilters-adr0053 branch July 4, 2026 12:04
…tabular (#2563)

objectui ADR-0053 reserves userFilters for page lists ("filters" mode);
on an object list view ("views" mode) the console suppresses them by
design — a showcase view demoing a field the console is designed to
ignore is actively misleading (it's how dogfood finding P0-30 got
filed). Point the comment at the ADR split instead: named views on the
object, userFilters in the task-workbench/task-triage/active-projects
page examples.

Also reclassify P0-30 in the test report (by-design suppression; the
silent drop is fixed by objectui#2220) and mark P0-29 fixed upstream
(objectui#2218).

Fixes #2563

Co-Authored-By: Claude <noreply@anthropic.com>
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.

app-showcase: showcase_task.tabular carries a wrong-context userFilters block that ADR-0053 suppresses (P0-30 reclassified)

2 participants