docs: update Agent Surfaces doc - #2608
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Visual recap — screenshot failedA recap was published, but the PR-comment screenshot could not be captured or uploaded. Open the interactive recap directly: Open the full interactive recap Diagnostic: light: page.screenshot: Timeout 30000ms exceeded. Call log: - taking page screenshot - waiting for fonts to load...
|
|
looks great - i may have missed this in the prior PR but we need to make sure the localized versions of any updated docs are updated as well |
There was a problem hiding this comment.
Builder reviewed your changes and found 1 potential issue 🔴
Review Details
Code Review Summary
This incremental update addresses the earlier localization gap by updating all ten localized Agent Surfaces pages and the baseline, while retaining the intended English restructuring and docs-nav reorder. The section order and anchors remain coherent, and the localized pages now include the new inline UI guidance. However, the latest translation commit introduces a high-severity MDX parsing problem in the Traditional Chinese page, and two documentation accuracy issues remain from the previous review. The Traditional Chinese code examples also contain typographic quote characters that readers cannot copy as valid TypeScript. Risk level: Low (documentation/navigation change), although the zh-TW rendering defect is blocking for that locale.
Key findings
- 🔴 HIGH — zh-TW uses typographic quotes in JSX/HTML component attributes, making the localized MDX invalid.
- 🟡 MEDIUM — zh-TW code samples use typographic quotes as TypeScript delimiters.
- 🟡 MEDIUM — Full application documentation still claims “No polling,” despite SSE with polling fallback.
- 🟡 MEDIUM — Manage Agent is documented as personal-only even though Connections renders both personal and organization MCP servers.
The prior missing-localization issue is resolved in this commit. The docs server is running and ready for browser verification.
🧪 Browser testing: Will run after this review (PR touches UI code)
What changed
Restructures
agent-surfaces.mdxso the detail sections match the order of the "Choose a starting point" table: Rich chat → Native inline UI → Generated inline UI → Full application → Embedded sidecar → Automation-first → Rich chat on your agent.Specifics
###subsection under Full application, where it belongs### How it connectsheading before the Embedded sidecar diagramdocsNavItems.ts)