Unify `MessageComposer` state and expose read-only StateFlows on ViewModels#6279

Merged

andremion merged 4 commits intov7from

remove-derived-state-in-message-composer

Mar 23, 2026

Commits on Mar 20, 2026

Refactor `MessageComposerController` to use `MessageComposerState` as the single source of truth for UI properties including `cooldownTimer`, `validationErrors`, `mentionSuggestions`, `commandSugge…
andremion
committed
Restrict direct modification of state in `MessageComposerController` by exposing immutable `StateFlow` instead of `MutableStateFlow`.
andremion
committed
saveDraftMessage(previousMode) is suspend (network .await()). While it ran, _state.value.messageMode was still the old mode, so overlapping setMessageMode calls could read stale mode.
andremion
committed
_state was storing the same MutableSet instance as MessageComposerState. Further += / clear() changed that object in place, so you lost clear “snapshot” semantics and StateFlow might not emit when …
andremion
committed