Chat with tools and cards #113
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the chat response handling to support multiple response types (text, cards, tool calls) while updating dependency versions and removing a custom network adapter in favor of ApolloAdapter. Key changes include refactoring the ChatAgent API with new response item structures, updating adapter usage in the UI, and adding Docker configuration for chat-agent services.
Reviewed Changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| hypermode-chat/src/app/page.tsx | Updated network adapter usage and added card action handling in the UI |
| hypermode-chat/src/app/chat-agent-adapter.tsx | Removed the custom network adapter code |
| hypermode-chat/package.json | Upgraded ChatKit dependencies from alpha.4 to alpha.6 |
| chat-agent/docker-dgraph.md | Added Docker setup instructions for Dgraph |
| chat-agent/docker-compose.yml | Introduced Docker Compose configuration for chat-agent and Dgraph services |
| chat-agent/api-go/myChat.go | Modified fact saving mutation query and updated query composition |
| chat-agent/api-go/main.go | Adjusted conversation creation and chat response parsing to support the new design |
| chat-agent/api-go/chatAgent.go | Overhauled ChatAgent to support multiple response types and tool call integration |
Files not reviewed (1)
- hypermode-chat/package-lock.json: Language not supported
Comments suppressed due to low confidence (1)
chat-agent/api-go/chatAgent.go:252
- [nitpick] Consider renaming 'loops' to 'attemptCount' for improved clarity on its purpose in the tool response generation loop.
loops := 0
| fmt.Printf("Error marshaling state: %v\n", err) | ||
| return nil | ||
| } | ||
|
|
||
| stateStr := string(data) | ||
| return &stateStr |
There was a problem hiding this comment.
Consider using a structured logging mechanism or propagating the error instead of printing directly to stdout for more robust error handling in GetState.
| fmt.Printf("Error marshaling state: %v\n", err) | |
| return nil | |
| } | |
| stateStr := string(data) | |
| return &stateStr | |
| log.Printf("Error marshaling state: %v", err) | |
| return nil, err | |
| } | |
| stateStr := string(data) | |
| return &stateStr, nil |
Copilot uses AI. Check for mistakes.
This PR refactors the return type of
CreateConversationto return an object of different types of responses, such as, text, cards and tool calls.Supporting this work is chatkit pr and release: