Skip to content

refactor(chat-store): extract message storage responsibility - #367

Merged
Andiveli merged 2 commits into
refactor/chat-store-hydrationfrom
refactor/chat-store-storage
Aug 28, 2026
Merged

refactor(chat-store): extract message storage responsibility#367
Andiveli merged 2 commits into
refactor/chat-store-hydrationfrom
refactor/chat-store-storage

Conversation

@Andiveli

Copy link
Copy Markdown
Owner

Summary

  • Extract message/chat storage from the chat-store coordinator.
  • Preserve existing ingestion and WhatsApp event behavior.

Changes

  • Add src/app/chat_store/storage.rs with focused boundary coverage.
  • Keep existing App methods and delegate ownership through the child module.

Testing

  • Focused storage boundary test
  • cargo test app::chat_store --lib
  • cargo test app::whatsapp_events --lib
  • cargo test app::message_ingestion --lib
  • git diff --check

Chain Context

beta
└── refactor/chat-store-srp (tracker)
    └── refactor/chat-store-hydration (#363)
        └── 📍 refactor/chat-store-storage

Start: hydration extraction.
End: message/chat storage has a dedicated owner.
Follow-up: isolate read-state and receipt coordination.
Out of scope: behavior, persistence schema, UI, and bridge changes.

Closes #361

@Andiveli Andiveli added the type:refactor Code refactoring label Aug 27, 2026
@Andiveli
Andiveli merged commit 962540b into refactor/chat-store-hydration Aug 28, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:refactor Code refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant