refactor(ui): complete design system hard cutover - #2714
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
@bitfun/uidesign-system contractsType and Areas
Type: refactor / UI/UX / regression fix / test
Areas: design system, Web UI, desktop startup Appearance projection, MiniApp Appearance projection, repository validation scripts
Motivation / Impact
This removes the dual legacy/new component path and leaves one public design-system boundary. Product surfaces now share canonical anatomy and semantic tokens, while Web UI retains product behavior and state ownership. The follow-up refinements also restore consistent compact control geometry, selected states, identity accents, and configuration layout behavior.
Verification
pnpm run design-system:check— passedpnpm run check:web— passedpnpm exec vitest run src/app/global-search/globalSearchArchitecture.test.ts src/flow_chat/components/ChatInputWorkspaceStripLayout.test.ts src/flow_chat/components/HarnessProfileSelector.test.tsx src/infrastructure/appearance/builtins/appearancePresetOutput.test.ts src/infrastructure/appearance/builtins/startupAppearanceBootstrap.test.ts src/infrastructure/config/components/common/ConfigPageLayout.test.tsxfromsrc/web-ui— 6 files, 68 tests passednode --test scripts/generate-startup-appearance-bootstrap.test.mjs— 2 tests passedgit diff --check origin/1.0.0-explore...HEAD— passedNo browser automation, E2E, or mock visual validation was run. Remote workspace, remote control, Peer Device Mode, and Detached Dispatch were not exercised because this change does not alter their transport, runtime, or persisted contracts.
Reviewer Notes
This is an intentional hard cutover with no legacy compatibility layer. Local design/QA notes and scratch artifacts were excluded from the PR.
Checklist