-
Notifications
You must be signed in to change notification settings - Fork 131
feat: Add support for AG-UI #143
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
base: main
Are you sure you want to change the base?
Conversation
Adds an AG-UI "bridge" to enable Strands Agents to be the backend for an agentic frontend like Copilot Kit.
Addresses #140 |
Based on the current implementation, this looks like several related features are part of this. Specifically, I think we need we're seeing gaps in:
|
Thanks for the review @zastrowm. Yes! This PR as-is would be valuable but not sufficient. I'm continuing to work on a sample/docs updates, etc. I'll review those other bits to see if the agent state and built-in tools work could align as well. |
logger = logging.getLogger(__name__) | ||
|
||
|
||
class AGUIEventType(str, Enum): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like these are already defined in the AG-UI Python SDK - should we use that Enum instead: https://github.com/ag-ui-protocol/ag-ui/blob/d53b012ff8051420905d85bb1a443c2729616d88/python-sdk/ag_ui/core/events.py#L29
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree, but I'd assumed the team would want to avoid having all strands agents depend on ag_ui (it's barely older than strands itself, and not all agents will have a web-based UI). Would you prefer: 1) Taking a hard-dependency on ag_ui and adding a bridge directly in the core package, or 2) Having an ag_ui example that could potentially turn into a strands-ui package if it succeeds?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree on avoiding a direct dependency - at this point we'd be looking towards an example/sample.
There's still a lot of open questions around the other features (#33, #31, multi-agent) which the team is starting work towards and knowing the gaps we have is really useful - but it's still too early for a direct integration.
Hey @adlio this is awesome, I'm with the org behind AG-UI and I'd love to chat more about this integration to make sure it's easy for our users to build with it. |
Adds an AG-UI "bridge" to enable Strands Agents to be the backend for an agentic frontend like Copilot Kit.
Description
Posting this for draft review and considering... continuing to work on docs and examples for using this.
Documentation PR
TBD
Type of Change
Testing
Used this version of Strands to build an interactive web-based Tic Tac Toe against Claude via Bedrock.
Checklist
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.