docs(a2ui): consolidate and extend A2UI guides#2771
Conversation
Merge the overview + architecture docs into docs/overview.md, and the catalogs + custom-components + built-in catalog README into docs/catalog-guide.md. Both are extended and reader-oriented. overview.md: React-vs-A2UI mental model, end-to-end data-flow diagrams, how a message becomes UI (store/processor/surfaces/resources/signals), the full <A2UI> props and lifecycle, and the export map -- now including the Loading built-in that the old architecture doc omitted. catalog-guide.md: a categorized table of all 20 built-in components, the complete list of 25 basic-catalog functions with their real wire names, the "no catalog/all" rationale, the paste-able all-builtins recipe, custom components with the react hook contract, manifest generation via the public CLI, and the catalog API reference. Also: update the README "More Docs" links, fold the src/catalog READMEs into short stubs that point at the new guide, refresh in-source comment pointers, and repoint the website docs-site config (sync sources, routes, nav, and link-rewrite maps). Simplified Chinese mirrors updated to match.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughConsolidates A2UI docs by adding English and Chinese overview and catalog-guide pages, removing older architecture/catalogs/custom-components files, updating in-package README pointers and test comments, and updating website sidebar generation and navigation to reference the new pages. ChangesA2UI Documentation Restructuring
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/genui/a2ui/docs/overview_zh.md`:
- Line 88: Update the Chinese wording in the table row containing "用与 client
可渲染能力一致的 catalog contract 提示模型" to the suggested, more natural phrasing "使用与
client 可渲染能力一致的 catalog contract 提示模型"; locate the string in overview_zh.md (the
Agent 服务 | Server | 你的应用 row) and replace the verb "用" with "使用" so the cell
reads smoothly.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 02f100c5-d615-4795-8476-109f9953b23b
📒 Files selected for processing (17)
packages/genui/a2ui/README.mdpackages/genui/a2ui/README_zh.mdpackages/genui/a2ui/docs/architecture.mdpackages/genui/a2ui/docs/architecture_zh.mdpackages/genui/a2ui/docs/catalog-guide.mdpackages/genui/a2ui/docs/catalog-guide_zh.mdpackages/genui/a2ui/docs/catalogs.mdpackages/genui/a2ui/docs/catalogs_zh.mdpackages/genui/a2ui/docs/custom-components.mdpackages/genui/a2ui/docs/custom-components_zh.mdpackages/genui/a2ui/docs/overview.mdpackages/genui/a2ui/docs/overview_zh.mdpackages/genui/a2ui/src/catalog/README.mdpackages/genui/a2ui/src/catalog/readme_zh.mdpackages/genui/a2ui/src/index.tspackages/genui/a2ui/test/catalog.test.tswebsite/sidebars/genui.ts
💤 Files with no reviewable changes (6)
- packages/genui/a2ui/docs/architecture_zh.md
- packages/genui/a2ui/docs/custom-components_zh.md
- packages/genui/a2ui/docs/catalogs.md
- packages/genui/a2ui/docs/custom-components.md
- packages/genui/a2ui/docs/architecture.md
- packages/genui/a2ui/docs/catalogs_zh.md
Merging this PR will degrade performance by 76.35%
|
| Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|
| ❌ | basic-performance-large-css |
16.2 ms | 68.6 ms | -76.35% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing PupilTong:claude/romantic-liskov-4efd92 (ef8c0d0) with main (536cbca)
Footnotes
-
26 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩
Summary
Consolidates the six A2UI docs into two reader-oriented guides and extends their content.
genui a2ui generate catalogCLI, and the catalog API reference.Supporting changes
Docs only -- no runtime code changed.
@lynx-js/genui-a2uiis a private package, so no changeset is required.Checklist
Summary by CodeRabbit