Skip to content

feat: update react with new agent api #189

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

Merged
merged 3 commits into from
Mar 10, 2025
Merged

feat: update react with new agent api #189

merged 3 commits into from
Mar 10, 2025

Conversation

yanxi0830
Copy link
Contributor

@yanxi0830 yanxi0830 commented Mar 10, 2025

What does this PR do?

  • Upgrade client side ReAct lib with new API
  • Resolves issue with tool_prompt_format
  • Keeps backward compat

Test Plan

python -m examples.agents.react_agent localhost 8321

@yanxi0830 yanxi0830 merged commit ac9d1e2 into main Mar 10, 2025
2 checks passed
@yanxi0830 yanxi0830 deleted the react_upgrade branch March 10, 2025 18:06
yanxi0830 added a commit to meta-llama/llama-stack-apps that referenced this pull request Mar 10, 2025
# What does this PR do?

- update to follow api in
meta-llama/llama-stack-client-python#189

## Feature/Issue validation/testing/test plan

```
python -m examples.agents.react_agent localhost 8321
```


## Sources

Please link relevant resources if necessary.


## Before submitting
- [ ] This PR fixes a typo or improves the docs (you can dismiss the
other checks if that's the case).
- [ ] Did you read the [contributor
guideline](https://github.com/meta-llama/llama-stack-apps/blob/main/CONTRIBUTING.md#pull-requests),
      Pull Request section?
- [ ] Was this discussed/approved via a Github issue? Please add a link
      to it if that's the case.
- [ ] Did you make sure to update the documentation with your changes?
- [ ] Did you write any new necessary tests?

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

Successfully merging this pull request may close these issues.

3 participants