docs: expand AG-UI A2UI guide#1709
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the documentation to guide users on integrating A2UI with various agent frameworks and harnesses using AG-UI. It updates the README, index, and mkdocs configuration, and rewrites the integration guide to include a framework selector and code snippets for ADK, LangGraph, Mastra, Strands, CrewAI, Google Chat, Slack, and Teams. The review feedback highlights several issues in the new documentation, including incorrect relative asset paths, invalid Gemini model identifiers, and a nested list issue where a tool-retrieval function already returning a list was unnecessarily wrapped in another list.
| </div> | ||
|
|
||
| <video width="100%" height="auto" controls playsinline preload="metadata" style="display: block; aspect-ratio: 16/9; object-fit: cover; border-radius: 8px; margin: 24px 0;"> | ||
| <source src="../../assets/ag-ui-a2ui-demo.mp4" type="video/mp4"> |
There was a problem hiding this comment.
The relative path to the asset folder from docs/guides/a2ui-with-any-agent-framework.md should be ../assets/ instead of ../../assets/ because the file is located at a depth of two levels under the docs directory.
| <source src="../../assets/ag-ui-a2ui-demo.mp4" type="video/mp4"> | |
| <source src="../assets/ag-ui-a2ui-demo.mp4" type="video/mp4"> |
| Slack adapter already implements this pattern: | ||
|
|
||
| <video width="100%" height="auto" controls playsinline preload="metadata" style="display: block; aspect-ratio: 16/9; object-fit: cover; border-radius: 8px; margin: 24px 0;"> | ||
| <source src="../../assets/ag-ui-slack-demo.mp4" type="video/mp4"> |
There was a problem hiding this comment.
The relative path to the asset folder from docs/guides/a2ui-with-any-agent-framework.md should be ../assets/ instead of ../../assets/ because the file is located at a depth of two levels under the docs directory.
| <source src="../../assets/ag-ui-slack-demo.mp4" type="video/mp4"> | |
| <source src="../assets/ag-ui-slack-demo.mp4" type="video/mp4"> |
| from langgraph.prebuilt import ToolNode | ||
|
|
||
| gemini = ChatGoogleGenerativeAI( | ||
| model="gemini-3.1-pro", |
| async def chat(self): | ||
| response = await copilotkit_stream( | ||
| await acompletion( | ||
| model="gemini/gemini-3.1-pro", |
| a2ui_tool = get_a2ui_tools({"model": gemini}) | ||
| TOOLS = [a2ui_tool] |
There was a problem hiding this comment.
get_a2ui_tools returns a list of tools. Wrapping it in a list [a2ui_tool] creates a nested list [[tool1, tool2, ...]], which will cause issues when passed to ToolNode. Assign the returned list directly to TOOLS.
| a2ui_tool = get_a2ui_tools({"model": gemini}) | |
| TOOLS = [a2ui_tool] | |
| TOOLS = get_a2ui_tools({"model": gemini}) |
0f92fd6 to
9acd77f
Compare
9acd77f to
6ab3d83
Compare
Summary
Notes
Verification
npx --yes prettier@^3.5.0 --write docs/guides/a2ui-with-any-agent-framework.mdgit diff --checkmkdocs build