Align website docs journey with current APIs

[blove]

Summary

• align homepage, docs landing, prompts, llms.txt, and adapter chooser with current provideAgent/injectAgent package flows
• add adapter chooser to docs search/sidebar/mobile navigation and fix stale sitemap route
• clarify licensing and AG-UI durability claims, regenerate API docs from updated source comments
• harden Claude review for docs-heavy PRs by making the advisory job non-blocking and raising review turn budget

Verification

• npm run generate-api-docs
• git diff --check
• npx vitest run apps/website/src/lib/docs.spec.ts apps/website/src/components/landing/Hero.spec.tsx
• npx nx lint website
• npx nx e2e website
• npx nx build website

[github-actions]

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

[github-actions]

Claude finished @blove's task in 2m 3s —— View job

---

PR Review #756 — Align website docs journey with current APIs

• Gather context and read CLAUDE.md
• Read PR diff and changed files
• Review navigation/sidebar changes (Nav.tsx, DocsSidebar.tsx, DocsSearch.tsx)
• Review docs config and route changes
• Review library source changes (chat.component.ts, agent-lifecycle-registry.ts)
• Review workflow changes (claude-review.yml)
• Post findings

---

Summary

This PR is clean. No correctness bugs, security issues, type-safety problems, or Angular/RxJS pitfalls found.

What was reviewed (non-generated files):

• Nav.tsx / DocsSidebar.tsx / DocsSearch.tsx — specialDocsPages is now surfaced in mobile nav, the docs sidebar, and search. The SearchablePage interface correctly makes library, section, and slug optional so special pages (with no library affiliation) compose cleanly with library pages. The default-results filter !p.library || p.library === library correctly shows adapter chooser in every library context. data-active={pathname === page.path || undefined} correctly omits the attribute when false.

• Nav.tsx state sync — initialMobileLibrary via docsConfig.find() and the useEffect to sync on navigation are correct. The ?.id after find() is technically redundant (find already guards with lib.id === activeLibrary) but harmless.

• docs-config.ts — title and description are now required on SpecialDocsPage. Correct — both fields are used at construction time in search and sidebar.

• site-metadata.ts — Sitemap routes fixed: /angular → /langgraph, /ag-ui added. Stale route removed.

• positioning.ts — LangGraph + AG-UI proof point now links to /docs/choosing-an-adapter instead of the LangGraph-specific concepts page. Better entry point.

• claude-review.yml — continue-on-error: true makes the advisory review non-blocking. --max-turns 30 gives more budget for docs-heavy PRs. Both are intentional and appropriate.

• chat.component.ts / agent-lifecycle-registry.ts — Comment-only corrections (agent() → this.agent() / "configured agent instance"). Accurate.

• llms.txt, page.tsx, ag-ui/page.tsx, docs/page.tsx — Consistent API naming (injectAgent() over agent()), install commands updated to include all required peer packages, licensing copy corrected. The AG-UI page correctly removes interrupt() from the shared contract bullets since that's backend-dependent.

One observation (not a blocker): choosing-an-adapter/index.mdx still ends with "Both adapters are MIT-licensed and live in the same monorepo." This is technically correct (it refers to @threadplane/langgraph and @threadplane/ag-ui, not @threadplane/chat), but the newly added "Next steps" link to the Chat quickstart sits just below it. A reader scanning the page could conflate the licensing claim with @threadplane/chat. Worth a parenthetical like "(the @threadplane/chat UI package has separate commercial licensing)" if the copy ever gets another pass — but not a PR blocker.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
threadplane	Ready	Preview, Comment	Jul 6, 2026 7:55pm

[github-actions]

Claude finished @blove's task in 1m 49s —— View job

---

PR Review #756 — Align website docs journey with current APIs

• Gather context and read CLAUDE.md
• Read PR diff and changed files
• Review navigation/sidebar changes (Nav.tsx, DocsSidebar.tsx, DocsSearch.tsx)
• Review docs config and route changes
• Review library source changes (chat.component.ts, agent-lifecycle-registry.ts)
• Review workflow changes (claude-review.yml)
• Post findings

---

Summary

This PR is clean. No correctness bugs, security issues, type-safety problems, or Angular/RxJS pitfalls found.

Navigation / Search / Sidebar

Nav.tsx — initialMobileLibrary correctly validates against docsConfig (using find(...)?. id ?? 'langgraph') instead of the raw activeLibrary string. The useEffect syncing mobileLibrary on activeLibrary change is correct. The currentLib && guard at the section nav block protects all currentLib.* accesses. The demoUrl null-check refactor (if (!demoUrl) return;) is a minor type-safety improvement. The analytics call for special-page links correctly passes library: 'unknown' since those pages aren't library-scoped.

DocsSearch.tsx — SearchablePage interface correctly makes library, section, and slug optional for special pages. The key={page.href} switch is correct (previous composite key depended on non-optional fields that are now optional). Default-results filter !library || !p.library || p.library === library correctly shows special pages in every library context. Optional chaining on description?.toLowerCase(), slug?.toLowerCase(), section?.toLowerCase() is correct.

DocsSidebar.tsx — data-active={pathname === page.path || undefined} correctly omits the attribute when the path doesn't match (standard React idiom). usePathname() import added alongside the existing useRouter() — correct.

Docs config and routes

docs-config.ts — title and description are now required on SpecialDocsPage. Both are used at construction time in search (description) and sidebar (title), so requiring them at the type level is correct.

site-metadata.ts — Sitemap fixed: /angular → /langgraph, /ag-ui added. Correct.

positioning.ts — LangGraph + AG-UI proof point links to /docs/choosing-an-adapter instead of the LangGraph-specific concepts page. Better entry point for the adapter decision.

Content and copy

Hero.tsx / Hero.spec.tsx — Install command updated to npm install @threadplane/chat @threadplane/langgraph. Spec updated to match new button text and clipboard value. Consistent.

ChatLandingCodeShowcase.tsx — The snippet now correctly shows provideAgent in the component's providers array and injectAgent() in the class field — the Angular DI pattern. Previously showed the old agent() call directly in the class field. Accurate.

llms.txt / docs/page.tsx / ag-ui/page.tsx / page.tsx — API names (injectAgent() over agent()), install commands, and licensing copy are consistent across all surfaces. AG-UI page correctly removes interrupt() from the shared-contract bullets since that's backend-dependent.

claude-review.yml — continue-on-error: true makes the advisory review non-blocking. --max-turns 30 gives more budget for docs-heavy PRs. Both appropriate.

One observation (non-blocking)

choosing-an-adapter/index.mdx still ends with "Both adapters are MIT-licensed and live in the same monorepo." The newly added "Chat quickstart" link sits directly below it. A reader scanning the page could conflate the licensing claim with @threadplane/chat. Worth a parenthetical like "(note: @threadplane/chat is commercially licensed for production)" on a future pass — not a blocker for this PR.