Skip to content

refactor(ui): migrate to Base UI and simplify effects - #5

Merged
AzureLandin merged 13 commits into
masterfrom
refactor/base-ui
Oct 21, 2026
Merged

AzureLandin merged 13 commits into
masterfrom
refactor/base-ui

Conversation

@AzureLandin

Copy link
Copy Markdown
Owner

Summary

  • replace the remaining Radix select with @base-ui/react@1.9.0
  • migrate dialogs, alert dialogs, menus, context menus, toast, popover and form controls to Base UI primitives
  • split SettingsModal into focused section components
  • remove all backdrop-filter / glass surfaces and animation of heavy shadows
  • define the missing --bg-surface token and ignore local worktrees in lint/format

Validation

  • npm run typecheck
  • npm test (25 files / 299 tests)
  • npm run lint (0 errors)
  • npm run build:wails
  • wails build -tags webkit2_41 -platform linux/amd64
  • CI matrix green on Windows/macOS/Linux for the pushed branch

Notes

This is stacked on the Eino migration branch. Terminal (xterm) and CodeMirror stay as dedicated components; session tabs remain custom to avoid unmounting terminal instances.

Pin @base-ui/react 1.9.0 as the headless component foundation for the incremental UI migration.
Use solid panel and elevated surfaces, remove backdrop-filter tokens and fallbacks, and add a root stacking context for future portals.
Replace the custom modal focus trap, escape and overlay handling with Base UI Dialog and AlertDialog while preserving visual classes and animation modes.
Use Base UI context menus, menu items, toast and popover primitives for SFTP/terminal context menus, notifications and the transfer center.
Replace Radix Select with Base UI Select, use Base UI field primitives in host and settings forms, and split SettingsModal into section components.
Drop the retired Radix dependency, remove stale selectors, define the missing surface token, and update test setup comments.
Keep parallel migration worktrees out of repository-wide ESLint and Prettier runs.
Fail the Linux job when any expected package is missing and assert the RPM and pacman artifacts in the CI structure test.
Retry hdiutil create after detaching a stale NodeShell mount, and cover the retry in the CI structure test.
@AzureLandin
AzureLandin changed the base branch from refactor/eino-agent to master October 21, 2026 11:40
@AzureLandin
AzureLandin merged commit 9c46cf3 into master Oct 21, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant