-
Notifications
You must be signed in to change notification settings - Fork 874
feat: add experimental Chat UI #17650
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
base: main
Are you sure you want to change the base?
Conversation
9cc72d3
to
e7559c3
Compare
a7d6a2f
to
54a20db
Compare
6a805f9
to
ae9fbc0
Compare
ae9fbc0
to
9a3fcc9
Compare
9a3fcc9
to
942741e
Compare
args: unknown; | ||
result?: unknown; |
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.
review: changed to unknown
to avoid any
@@ -717,11 +708,12 @@ const FilePreview: FC<{ files: Record<string, string>; prefix?: string }> = | |||
); | |||
}); | |||
|
|||
// TODO: generate these from codersdk/toolsdk.go. |
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.
review: ideally these would be autogenerated, but deferring this for now
export interface ChatContext { | ||
selectedModel: string; | ||
modelConfig: LanguageModelConfig; | ||
|
||
setSelectedModel: (model: string) => void; | ||
} |
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.
review: Ideally I'd like to move this into useAgenticChat
but ran out of time.
Builds on #17570
Frontend portion of https://github.com/coder/coder/tree/chat originally authored by @kylecarbs
Additional changes:
ChatToolInvocation
argument definitions in line with those defined incodersdk/toolsdk
ExperimentAgenticChat
is enabled.