Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: client tool response message #3

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

louisjoecodes
Copy link
Collaborator

@louisjoecodes louisjoecodes commented Nov 26, 2024

Fixes client tool message response:

  • Server expects result not response
  • Server expects a response from client_tool_result in all case. If the user defines a client tool like this:
     clientTools: {
                triggerBrowserAlert: async (paramaters: {
                    message: string
                }) => {
                    alert(paramaters.message)
                }

it would cause the conversation to crash and end. Therefore we now have a default response of Client tool execution successful.

@louisjoecodes louisjoecodes merged commit 733f7b1 into main Nov 26, 2024
3 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants