Convert feedback and changelog modals to imperative base-ui API in LobeHub

FACTAI JUDGMENTDetected 39 days ago

Factual Description

Migrates FeedbackModal and ChangelogModal from a declarative API with external state management to a simpler imperative API from @lobehub/ui/base-ui, removing the need for open/close state plumbing.

Event TypeCode Change

DetectedJun 01, 2026

TopicConsumer AI Applications

Core Technical Contributions

Simplifies modal management in the LobeHub UI codebase by replacing a declarative modal API and a zustand store with a direct imperative API, reducing state complexity in layout components.

@lobehub/ui/base-uicreateModaluseFeedbackModalFeedbackModalChangelogModal

AI Impact Judgment

Developers working on the LobeHub UI will have a simpler and less error-prone way to manage modals, eliminating the need to manage open/close state across multiple layout files. This reduces the chance of UI state bugs where modals might not open or close correctly. The refactor also cleans up related test mocks. Further observation is needed to see if this imperative pattern is adopted for other modals in the codebase.

Confidence0%

Importance65

Evidence1

Raw Evidence Links

Github Pull Requestlobehub/lobehub PR #15396: ♻️ refactor(modal): convert feedback & changelog modals to base-ui imperative API

Migrate `FeedbackModal` and `ChangelogModal` from the declarative `@lobehub/ui` `Modal` API + a zustand `useFeedbackModal` store to the `@lobehub/ui/base-ui` imperative `createModal()` API. Call sites now invoke `openFeedbackModal()` / `openChangelogModal()` directly — no more open/close state plumbing through `(main)/_layout` or `(mobile)/me/(home)`.