-
Notifications
You must be signed in to change notification settings - Fork 9
feat: add chat container #600
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
0d15650 to
924a70b
Compare
051e889 to
94c9e65
Compare
d16d18e to
287eb7d
Compare
9f9540a to
19d404a
Compare
|
Did some improvements and changed it to use the same entrypoint after discussing with @kfenner , will split into two PRs |
3b23d72 to
6b82cff
Compare
6b82cff to
4cd6a1e
Compare
|
@kfenner @spike-rabbit Refactored out the model-API wrapper to #998 , ready to review |
4cd6a1e to
bbc251a
Compare
bbc251a to
4160a81
Compare
1df7aa4 to
85d877c
Compare
85d877c to
771a752
Compare
kfenner
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Killusions Love it 💜
LGTM 👍
|
🎉 This PR is included in version 48.4.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
This adds the chat-container (fully declarative with auto-scroll), potential follow-ups are an ai-chat-container wrapper (model based with a nice signal-based API for streaming / loading state), dogfooded) in #998 (tool message is here #967 )
The chat input is in #948 (merged), the base components (merged) in #877, the file upload directive (merged) in #932
Rough Architecture (ignore the ai chat container for now and the tool message, not a part of this)
