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

Support tools role in messages in the pipeline #862

Open
jhrozek opened this issue Jan 31, 2025 · 1 comment
Open

Support tools role in messages in the pipeline #862

jhrozek opened this issue Jan 31, 2025 · 1 comment

Comments

@jhrozek
Copy link
Contributor

jhrozek commented Jan 31, 2025

The open interpreter support in PR #820 added handling of role: tool messages scoped to open interpreter. This was the right thing to scope them to the tool as we wanted to get the tool support in, but we should support that message role in general in our pipeline.

more todo items would probably come out when we do the work but for now I think we should do:

  • when messages with that role are processed (e.g. get_last_user_message_block) do we take the tools role into account
  • make sure we don't touch those in pipeline to avoid breaking the tools

We can add a bunch of tests just against the openai/anthropic api to make sure we follow their APIs, no need to tie this to a specific client.

@yrobla
Copy link
Contributor

yrobla commented Jan 31, 2025

It would also be worth investigating how this role is being used in other tools (if that's used at all) and how to better support it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants