Fix incorrect paths to sample code in MCP guides#1157
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates documentation for MCP Apps integration, including a path update in the dynamic MCP guide and the renaming of the 'Contact Multi-Surface Sample' to 'MCP App Standalone Sample' with corresponding path updates. Feedback recommends using relative links instead of absolute GitHub URLs for internal repository references to ensure consistency across different environments.
|
|
||
| - **A2A Agent Server**: | ||
| - Path: [`samples/agent/adk/contact_multiple_surfaces/`](https://github.com/google/A2UI/tree/main/samples/agent/adk/contact_multiple_surfaces/) | ||
| - Path: [`samples/agent/adk/mcp-apps-in-a2ui-sample/`](https://github.com/google/A2UI/tree/main/samples/agent/adk/mcp-apps-in-a2ui-sample/) |
There was a problem hiding this comment.
Consider using a relative link instead of an absolute GitHub URL for internal repository references. This ensures the link remains functional across different branches, forks, and local environments.
| - Path: [`samples/agent/adk/mcp-apps-in-a2ui-sample/`](https://github.com/google/A2UI/tree/main/samples/agent/adk/mcp-apps-in-a2ui-sample/) | |
| - Path: [`samples/agent/adk/mcp-apps-in-a2ui-sample/`](../../samples/agent/adk/mcp-apps-in-a2ui-sample/) |
| - Command: `uv run .` (requires `GEMINI_API_KEY` in `.env`) | ||
| - **Lit Client App**: | ||
| - Path: [`samples/client/lit/contact/`](https://github.com/google/A2UI/tree/main/samples/client/lit/contact/) | ||
| - Path: [`samples/client/lit/mcp-apps-in-a2ui-sample/`](https://github.com/google/A2UI/tree/main/samples/client/lit/mcp-apps-in-a2ui-sample/) |
There was a problem hiding this comment.
Consider using a relative link instead of an absolute GitHub URL for internal repository references. This ensures the link remains functional across different branches, forks, and local environments.
| - Path: [`samples/client/lit/mcp-apps-in-a2ui-sample/`](https://github.com/google/A2UI/tree/main/samples/client/lit/mcp-apps-in-a2ui-sample/) | |
| - Path: [`samples/client/lit/mcp-apps-in-a2ui-sample/`](../../samples/client/lit/mcp-apps-in-a2ui-sample/) |
This PR fixes incorrect paths to sample code in mcp-apps-in-a2ui-surface.md and guide_a2ui_dynamic_mcp.md. This was discovered during onboarding review.