feat: defer member/team registries on detail routes (#156)

[wab]

Closes #156.

Summary

Move non-critical loader work off the blocking await on the three detail routes (blog, jobs, clients). Primary content (<h1>, body, hero, sidebar) paints first; member registry resolution streams in afterwards via <Suspense> + <Await>.

Per-route scope

Route	Deferred	Kept sync (why)
blog/$slug	author footer (member registry)	article, toc, intro
($lang)/jobs/$slug	hiring contact footer	article, sections, JSON-LD (crawlers need it on first paint)
($lang)/clients/$slug	team footer	article, resolvedTools (above-the-fold sidebar)

Changes

• app/routes/_main.blog.$slug.tsx — loadMemberRegistry() no longer awaited; author returned as Promise<Member | null>.
• app/components/blog/blog-article.tsx — author footer wrapped in <Suspense fallback={null}><Await>.
• app/routes/_main.($lang).jobs.$slug.tsx — same pattern for contact. ld (JSON-LD) stays sync.
• app/routes/_main.($lang).clients.$slug.tsx — loadMemberRegistry() deferred for resolvedTeam. loadToolRegistry() stays sync (sidebar metas).
• app/components/stories/story-article.tsx — StoryTeamBlock wrapped in <Suspense>.

Tests

• app/routes/_main.blog.$slug.test.ts — happy-path split into "primaries sync" + "author is a Promise resolving to expected value".
• app/routes/_main.($lang).jobs.$slug.test.ts — new test asserting contact is a Promise; ld stays sync.
• app/routes/_main.($lang).clients.$slug.test.ts — new file, 4 cases: 404 (no slug), 404 (fetch failure), sync primaries, deferred team.

All 414 tests pass (pnpm test:run). Typecheck + Biome clean.

Measurement plan

Single-run lab PSI is too noisy to credit a defer-pattern win (we showed ±30-50 % LCP variance across consecutive runs on this site). The real signal lives in Vercel Speed Insights p75 INP / TBT once production traffic accumulates.

• Re-check Vercel Speed Insights on the 3 detail routes ~7 days post-merge (≈ 2026-05-22)
• Update docs/project/performance-baseline.md Changelog with delta vs 2026-05-13 site-wide RUM

Test plan

• pnpm typecheck && pnpm check && pnpm test:run all green locally
• CI green on PR
• Manual smoke on preview deploy: blog detail, jobs detail, clients detail — verify footer (author / hiring contact / team) renders after primary content with no layout shift
• Networktab: verify no waterfall regression on detail routes

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
ocobo	Ready	Preview	May 15, 2026 9:56am

[wab]

Code review feedback addressed in c3ef9c8.

H1 (blocking) — loader concurrency restored. Fire loadMemberRegistry() before awaiting the primary fetch in all 3 routes so the registry call overlaps with fetchBlogpost / fetchJob / fetchStory (matches the parallelism of the original Promise.all).

H2 (blocking) — errorElement={null} on all 3 <Await> boundaries. Prevents a future-throwing registry resolver from escalating to the root ErrorBoundary mid-stream.

Not addressed (low priority, can be follow-ups):

• M1 — component-level RTL tests for the Suspense/Await wiring. Loader-level assertions cover the deferred contract; render tests would add belt-and-braces.
• M2 — outcome.data as unknown as ... cast. Worth a separate invokeLoader<T> generic refactor across all loader tests, not in this PR.
• N1 — resolveDeferred helper to dedupe the loadMemberRegistry().then(resolver) pattern. Optional, current spelling is clear.
• N2 — explicit expect(data.article).not.toBeInstanceOf(Promise) assertions. Nice-to-have boundary lock; not load-bearing.

All 414 tests passing locally (pnpm test:run). Route tests specifically 40/40. Typecheck + Biome clean.