Skip to content

feat(server): system messages #303

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

edvinbrobeck
Copy link
Contributor

Description

This PR simply adds SystemMessage to the mapping in PrismChatController.
I was using Open WebUI with a knowledge base, and the "RAG context" is added with 'role' => 'system'.

@edvinbrobeck edvinbrobeck changed the title Add support for system messages in chat server controller fix: Add support for system messages in chat server controller Apr 9, 2025
@edvinbrobeck edvinbrobeck changed the title fix: Add support for system messages in chat server controller fix: Add support for system messages in server chat controller Apr 9, 2025
@edvinbrobeck edvinbrobeck force-pushed the fix/server-chatcontroller-system-message-mapping branch from 61cf750 to 8116e58 Compare April 9, 2025 07:09
Copy link
Contributor

@sixlive sixlive left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AWESOME!! Thank you! I meant to do this at somepoint but lost track of it, thanks!

@sixlive sixlive changed the title fix: Add support for system messages in server chat controller feat(server): system messages Apr 9, 2025
@sixlive sixlive merged commit 3ae583d into prism-php:main Apr 9, 2025
14 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.

2 participants