Skip to content

Fix incorrect paths to sample code in MCP guides#1157

Merged
jacobsimionato merged 2 commits intomainfrom
fix/mcp-docs-paths
Apr 14, 2026
Merged

Fix incorrect paths to sample code in MCP guides#1157
jacobsimionato merged 2 commits intomainfrom
fix/mcp-docs-paths

Conversation

@zeroasterisk
Copy link
Copy Markdown
Collaborator

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.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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/)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

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.

Suggested change
- 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/)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

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.

Suggested change
- 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/)

@jacobsimionato jacobsimionato merged commit 04088ec into main Apr 14, 2026
13 checks passed
@jacobsimionato jacobsimionato deleted the fix/mcp-docs-paths branch April 14, 2026 20:19
@github-project-automation github-project-automation bot moved this from Todo to Done in A2UI Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants