Skip to content

docs: add guide for MCP Apps integration in A2UI surface#995

Merged
sugoi-yuzuru merged 1 commit intomainfrom
guide
Mar 26, 2026
Merged

docs: add guide for MCP Apps integration in A2UI surface#995
sugoi-yuzuru merged 1 commit intomainfrom
guide

Conversation

@sugoi-yuzuru
Copy link
Copy Markdown
Collaborator

@sugoi-yuzuru sugoi-yuzuru commented Mar 26, 2026

Description

Add a guide for MCP Apps integration into A2UI Surfaces.

Pre-launch Checklist

If you need help, consider asking for advice on the discussion board.

@github-project-automation github-project-automation bot moved this to Todo in A2UI Mar 26, 2026
@sugoi-yuzuru sugoi-yuzuru requested review from dmandar, wrenj and zeroasterisk and removed request for zeroasterisk March 26, 2026 17:02
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 introduces a new guide for MCP Apps integration in A2UI surfaces, detailing the double-iframe isolation pattern, security rationale, and providing usage examples and testing instructions. Feedback includes a suggestion to simplify the input binding logic in the documentation's code examples using the nullish coalescing operator and a recommendation to align the navigation title in mkdocs.yaml with the document's main heading for consistency.

Comment on lines +59 to +65
inputBinding(
'content',
() => ('content' in properties && properties['content']) || undefined,
),
inputBinding('title', () => ('title' in properties && properties['title']) || undefined),
],
},
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

The inputBinding logic for content and title can be simplified using the nullish coalescing operator (??). This improves readability and conciseness, and explicitly handles cases where the property might be null by converting it to undefined, aligning with common Angular input binding patterns.

For example, ('content' in properties && properties['content']) || undefined can be replaced with properties['content'] ?? undefined.

Suggested change
inputBinding(
'content',
() => ('content' in properties && properties['content']) || undefined,
),
inputBinding('title', () => ('title' in properties && properties['title']) || undefined),
],
},
inputBinding(
'content',
() => properties['content'] ?? undefined,
),
inputBinding('title', () => properties['title'] ?? undefined),

- Authoring Custom Components: guides/authoring-components.md
- Theming & Styling: guides/theming.md
- A2UI over MCP: guides/a2ui_over_mcp.md
- MCP Apps integration in A2UI Surface: guides/mcp-apps-in-a2ui-surface.md
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

The navigation entry title "MCP Apps integration in A2UI Surface" is inconsistent with the document's main heading "# MCP Apps Integration in A2UI Surfaces". For better consistency and user experience, it's recommended to align the navigation title with the document's actual title.

      - MCP Apps Integration in A2UI Surfaces: guides/mcp-apps-in-a2ui-surface.md

Comment on lines 58 to +59
- A2UI over MCP: guides/a2ui_over_mcp.md
- MCP Apps integration in A2UI Surface: guides/mcp-apps-in-a2ui-surface.md
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

mcp apps in a2ui surfaces

@sugoi-yuzuru sugoi-yuzuru merged commit b3da32a into main Mar 26, 2026
11 checks passed
@sugoi-yuzuru sugoi-yuzuru deleted the guide branch March 26, 2026 19:29
@github-project-automation github-project-automation bot moved this from Todo to Done in A2UI Mar 26, 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.

2 participants