Update key concepts docs - #2711
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.
|
Here's a visual recap of what changed: Open the full interactive recap
|
9d58d60 to
3b7217f
Compare
…order benefits section
…pts pages Fixes guard:i18n-catalogs — the rewritten key-concepts.mdx locale pages carried several link titles, headings, a diagram, and a bash comment verbatim from the English source instead of translating them: - "Related docs" card titles (A2A Protocol, Agent Surfaces, Context Awareness, Native Chat UI, Skills Guide, What Is Agent-Native?) and their inline-link occurrences elsewhere in the page, now matching each linked page's own translated title. - The German/Spanish/French/Hindi/Japanese/Korean/Portuguese/Chinese 'Core actions for quick database inspection' bash comment. - hi-IN-specific stragglers: the SQL-store table/diagram labels, 'Agent action' / 'Change event' / 'Query refetch' diagram nodes, and the OAuth credentials / view-selection-drafts data model notes. - hi-IN's agent-surfaces.mdx title/H1, which was still untranslated and blocked translating its own key-concepts.mdx backlink.
3b7217f to
7921751
Compare
There was a problem hiding this comment.
Builder reviewed your changes — looks good ✅
Review Details
Incremental Code Review Summary
The latest PR head retains the Key Concepts rewrite, all ten locale translations, and the Hindi Agent Surfaces translation correction. The current diff remains sound: MDX components and fences are balanced, anchors and internal links are valid, locale structure is consistent, documentation validation and production checks pass, and the Core patch changeset is correctly formatted.
Review Result
- ✅ No new clear documentation rendering, link, translation, code-example, formatting, or release issues found.
- ✅ Previously reported unmatched tags and formatting failures remain fixed.
- ✅ Agents report passing docs tests, block validation, link checks, and formatting checks.
Risk remains low because this PR changes documentation and a patch changeset only.
🧪 Browser testing: Will run after this review (PR touches rendered documentation/UI surfaces).
|
Per Claude:
So I think we can merge. Can you check my thinking, @steve8708? |

Rework the Key Concepts doc for clarity, and translate into all 10 locales
Substantially edits
key-concepts.mdxfor clarity and readability, then brings all 10 locale translations up to date with the rewrite (they were untouched by this and had drifted out of sync).Content changes:
Cardsgrid); converted "Five rules," "The four-area checklist," "Why not call an LLM inline," "Frames," and "Related building blocks" away fromCards/questions into plain lists, bold-headed sections, or tables, sinceCardshad become overused for content that isn't a navigation griddb-schema/db-query/useDbSynccode samples actually do, how the building-blocks table maps to the file tree above it, and a definition for "contract" on its first useTranslations:
ar-SA,de-DE,es-ES,fr-FR,hi-IN,ja-JP,ko-KR,pt-BR,zh-CN,zh-TWfrom scratch against the new English source'navigation'string literal in Korean, translatedview-screen/navigateidentifiers in Traditional Chinese that should have stayed untranslated)Includes a changeset (
@agent-native/core: patch).