docs: promote Apps above Core Architecture, rewrite Templates doc - #2786
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 |
8234c91 to
b18a8d2
Compare
… doc
- Move the per-app docs section (Chat, Mail, Calendar, etc.) up in the
sidebar to sit right after Overview, instead of buried at the very
bottom below Toolkit and Advanced Runtime.
- Fold cloneable-saas.mdx (Templates) and pure-agent-apps.mdx
(Automation-First Apps) into that same section as its two landing
pages, ahead of the per-app groups — they already cross-link each
other as the two app archetypes, so they belong together with the
catalog they introduce.
- Rewrite cloneable-saas.mdx for tone and clarity: cut marketing
language ("Agent-native flips that", "SaaS-grade", "superpowers",
rhetorical question chains, repeated "already" emphasis) in favor of
direct, concise statements. Fixed a confusing, nav-position-dependent
sentence in the intro (referencing docs "in the sidebar below it")
to instead name what the doc covers and link to the sibling doc for
the other archetype.
Verified: typecheck clean, full packages/docs test suite passes
(including the test asserting the Apps section stays wired to the
featured template catalog), full pnpm guards suite passes, and
confirmed the new sidebar order renders correctly in a live docs dev
server.
- Add a changeset entry for the Apps-section nav reorder and Templates doc rewrite from the previous commit. - Translate the 18 changed passages (intro, catalog pointers, What you get / What you do / Why this is practical sections, hosted-version and skill call-outs, and the developer CLI section) into ar-SA, de-DE, es-ES, fr-FR, hi-IN, ja-JP, ko-KR, pt-BR, zh-CN, and zh-TW, matching each locale's existing terminology and register rather than literally transliterating the cut marketing language. - Left everything else in each locale file untouched (Community Templates section, app catalog table, diagrams, unchanged steps). A few locales also had pre-existing translation gaps unrelated to this change (a missing third intro paragraph in de-DE/zh-CN/fr-FR/ zh-TW/ja-JP, a mistranslated "Auth" in hi-IN, a mistranslated "docs" in zh-TW) that got naturally fixed along the way. Verified: oxfmt --check clean on all 11 files, guard:i18n-catalogs clean, full pnpm guards suite passes, and every locale's diff is an identical 48 lines — confirming scoped, consistent edits across all 10 languages.
b18a8d2 to
e1ec17e
Compare
There was a problem hiding this comment.
Builder reviewed your changes — looks good ✅
Review Details
Incremental Code Review Summary
I reviewed the current PR #2786 head and confirmed it matches the state covered by the prior review: the Apps sidebar reorder, English Templates rewrite, ten localized MDX rewrites, and the core patch changeset. The latest code-review passes found no new delta requiring additional feedback; navigation metadata, localized frontmatter, docs links, and changeset structure remain valid.
Risk assessment: Low. The PR is limited to documentation content, localization, navigation configuration, and release metadata. It does not alter backend behavior, APIs, authentication, data access, or runtime logic. No previous findings require resolution.
Key Findings
No actionable findings.
✅ Current HEAD is consistent with the previously reviewed documentation changes.
✅ All affected docs routes and localized pages remain reachable in HTTP-level checks.
✅ Intended sidebar ordering and navigation slugs remain intact.
🧪 Browser testing: Skipped — browser automation tools were unavailable in this session. The docs server and affected routes were reachable, but interactive visual evidence could not be captured.
|
thanks @kapunahelewong! |

What changed
Nav reorg (docsNavItems.ts)
cloneable-saas.mdx(Templates) andpure-agent-apps.mdx(Automation-First Apps) moved out of the Overview section into this same Apps section, as its first two items — they already cross-link each other as the two app archetypes (full-UI template vs. no-browser-UI automation), so they now sit with the catalog they introduce instead of being orphaned pages under Overview.Content rewrite (cloneable-saas.mdx)
/apps.Verification
oxfmt --checkclean on both changed files.tsc --noEmitclean inpackages/docs.packages/docstest suite passes (28 files, 162 tests), includingtemplates-routes.test.ts, which specifically asserts the "Apps" section stays wired to the featured template catalog.pnpm guardssuite passes.Notes for reviewers
docsNavItems.tsdiff is large in line count (~1200 lines) because moving a ~400-line section within the file shows as a delete+add of everything after it — it's a reorder, not a rewrite of the per-app group definitions themselves (those are unchanged).cloneable-saas) were left as-is to avoid touching the corresponding i18n keys across 10 locale catalogs; only English content changed.🤖 Generated with Claude Code