Skip to content
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

[playground][ui] update tool / tool call schemas switching logic to not require a schema for all providers #5360

Open
Parker-Stafford opened this issue Nov 14, 2024 · 0 comments
Assignees
Labels

Comments

@Parker-Stafford
Copy link
Contributor

Parker-Stafford commented Nov 14, 2024

We currently have tool / toolCall schemas for anthropic and open ai and convert between them when switching providers. This is good and should be the goal for new providers however, we don't want to block adding new providers on getting all of the tool / tool call schemas correct right away.

Solution:
Allow for new providers to be added with no tool / tool call schemas in the UI. With the option to always incrementally add them later for a better (switching between providers) UX if necessary.

Effects:
New providers can start as schema-less which means no linting in json tool / tool call editors and no conversion when switching to that provider (just keep the schema from the previous provider).

Details:

  • Update error handling to show persistent errors when calling subscription or mutation in the playground to allow user to easily see why their call failed (in this case potentially invalid tools) to offer a path to fixing it
  • keep provider lists exhaustive in tool / tool call conversion schemas but add handling for providers with no schema
  • Unknown: how do we convert from a schema-provider to a schema'd one
    • could create a new "empty" (default) tool for that provider (probably a better option)
    • could remove tools
@Parker-Stafford Parker-Stafford self-assigned this Nov 14, 2024
@Parker-Stafford Parker-Stafford converted this from a draft issue Nov 14, 2024
@dosubot dosubot bot added the c/ui label Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 📘 Todo
Development

No branches or pull requests

1 participant