[recovery] fix: setRequestLocale ordering in wallets page (static→dynamic error)

[pettinarip]

Error

[ERROR] Error: Page changed from static to dynamic at runtime /api/wallets, reason: headers
    ... at get requestLocale ...

• URL: /api/wallets
• Hit count: 1 (first seen 2026-06-02)
• Function: ___netlify-server-handler

Root cause

/api/wallets is matched by app/[locale]/wallets/page.tsx with locale="api" — a bot/invalid path where api is captured as the [locale] dynamic segment. api is not a valid locale, so it is not produced by generateStaticParams and the route renders on-demand.

The page called getTranslations("page-wallets") before setRequestLocale(locale), and generateMetadata never called setRequestLocale at all. With no locale set, next-intl falls back to reading the locale from request headers, which opts the route into dynamic rendering. Next.js then throws Page changed from static to dynamic at runtime, reason: headers (stack shows get requestLocale) instead of rendering statically / returning a clean response.

This is the same bug class as the videos [slug] page fixed in #18785 — a distinct route/URL, so a distinct fix.

Fix

Call setRequestLocale(locale) first (before any other next-intl API) in both the page component and generateMetadata, mirroring #18785.

Affected files

• app/[locale]/wallets/page.tsx

Verification

• pnpm type-check — passes
• pnpm exec eslint app/[locale]/wallets/page.tsx — clean

🤖 Generated with Claude Code

[netlify]

✅ Deploy Preview for ethereumorg ready!

Name	Link
🔨 Latest commit	f529f47
🔍 Latest deploy log	https://app.netlify.com/projects/ethereumorg/deploys/6a57b6aaf4fa1400081d0039
😎 Deploy Preview	https://deploy-preview-18797.ethereum.it
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	7 paths audited Performance: 62 (🟢 up 6 from production) Accessibility: 95 (no change from production) Best Practices: 100 (no change from production) SEO: 98 (no change from production) PWA: 59 (🔴 down 1 from production) View the detailed breakdown and full score reports

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[github-actions]

🔎 First-pass review — ✅ Looks mergeable

Minimal, correct next-intl ordering fix: setRequestLocale(locale) now runs before any other next-intl API (getTranslations, getMessages) in both Page and generateMetadata. That's the right fix for the static → dynamic (reason: headers) error — without a locale set, next-intl reads the locale from request headers and forces on-demand dynamic rendering. Mirrors the established #18785 fix for the videos [slug] route.

Verified against the file: setRequestLocale was already imported (no import churn), no unused vars, and generateMetadata — which previously never set the locale at all — now does. Base branch dev and scope (single file, +6/−1) are appropriate.

Analysis

• Lane: code (app/[locale]/wallets/page.tsx)
• Checked: call ordering of next-intl APIs vs setRequestLocale; both Page and generateMetadata now call it first. No unused variables/params. No raw Intl.* introduced. No ChainName or component-variant surface touched. Imports clean.
• Non-blocking: none.
• CI: not asserted here — let the pipeline confirm; nothing in the diff should regress it.

Generated by PR Reviewer (team) for #18797 · 33 AIC · ⌖ 11.3 AIC · ⊞ 5.2K · ◷