Skip to content

Conversation

Guikingone
Copy link
Contributor

Q A
Bug fix? no
New feature? yes
Docs? yes
Issues Fix none
License MIT

Hi 👋🏻

This PR aims to introduce commands to handle message store lifecycle, it also introduce the configuration for message stores.

@OskarStark OskarStark added AI Bundle Issues & PRs about the AI integration bundle Chat Issues & PRs about the AI Chat component labels Oct 6, 2025
@carsonbot carsonbot changed the title [Chat|AIBundle] Add lifecycle commands [AI Bundle][Chat][Chat|AIBundle] Add lifecycle commands Oct 6, 2025
@OskarStark OskarStark changed the title [AI Bundle][Chat][Chat|AIBundle] Add lifecycle commands [AI Bundle][Chat] Add lifecycle commands Oct 6, 2025
@Guikingone Guikingone force-pushed the feat/message_store_commands branch from 3fe87f8 to b20fb42 Compare October 6, 2025 17:48
@chr-hertel
Copy link
Member

I feel like this is coming in early - is it motivated by an actual need of yours or rather a hypothesis?
(just a curious question 😆 )

@Guikingone
Copy link
Contributor Author

It's the continuation of the Chat migration process, as stores can be initialized/dropped, we need to have commands to handle the booting phase of them, I also integrated the configuration to ease the usage of stores in current applications (until we merge the Chat refactoring), IMHO, merging commands before having "external" bridges is not an issue (Meilisearch should be ready later this week BTW).

And yes, I'm currently hitting a wall in a project that requires a message store 😅

@Guikingone Guikingone force-pushed the feat/message_store_commands branch 2 times, most recently from 66d46e4 to 27185aa Compare October 7, 2025 15:47
@Guikingone Guikingone force-pushed the feat/message_store_commands branch from 27185aa to 0185201 Compare October 7, 2025 15:50
@Guikingone Guikingone marked this pull request as ready for review October 7, 2025 15:50
@OskarStark
Copy link
Contributor

Thank you Guillaume.

@OskarStark OskarStark merged commit fb1cddb into symfony:main Oct 7, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AI Bundle Issues & PRs about the AI integration bundle Chat Issues & PRs about the AI Chat component Feature New feature Status: Reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants