From e879016879139560ded9a2383add119299e47b6e Mon Sep 17 00:00:00 2001
From: DavidHerran <davidherran@dreamhousestudios.co>
Date: Mon, 4 May 2026 16:53:20 -0500
Subject: [PATCH] fix: pass locale explicitly to next-intl helpers
 (DYNAMIC_SERVER_USAGE)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Production logs were spamming "DYNAMIC_SERVER_USAGE" errors and every
ISR-cached page (home, news, heritage, applications, news/[slug]) was
500-ing. Root cause: when force-dynamic was replaced with revalidate=60
last commit batch, the implicit locale-resolution path through
next-intl's getTranslations() / getLocale() / getMessages() became a
problem. Without an explicit locale arg, next-intl reads cookies()
under the hood — which trips Next.js's "dynamic API used inside a
static / ISR render" guard and aborts with DYNAMIC_SERVER_USAGE.

Fix: pass the locale (already known from the URL via params) every
time we call next-intl on the server.

CHANGES
- src/components/layout/Footer.tsx
  - Now accepts `locale` as a prop instead of awaiting getLocale()
  - getTranslations({ locale, namespace: "Footer" })
- src/app/[locale]/layout.tsx
  - Passes locale prop to <Footer locale={locale} />
  - getMessages({ locale }) instead of getMessages()
- src/components/sections/PatrizioLegacy.tsx
  - Accepts `locale` prop, passes it to getTranslations
- src/app/[locale]/page.tsx
  - Renders <PatrizioLegacy locale={locale} />
- src/app/[locale]/news/page.tsx (body)
  - getTranslations({ locale, namespace: "NewsHub" })
- src/app/[locale]/heritage/page.tsx (body)
  - getTranslations({ locale, namespace: "HeritagePage" })

The generateMetadata / generateStaticParams paths in news/[slug] and
heritage already passed locale correctly — only the page bodies and
shared components were leaking dynamic reads.

Net effect: all five ISR pages render statically again, the error spam
in `docker compose logs app` clears, and /en/applications/digital-print
loads its server component without falling through to the error boundary.
---
 src/app/[locale]/heritage/page.tsx         |  4 ++--
 src/app/[locale]/layout.tsx                |  4 ++--
 src/app/[locale]/news/page.tsx             |  4 ++--
 src/app/[locale]/page.tsx                  |  2 +-
 src/components/layout/Footer.tsx           | 10 ++++++----
 src/components/sections/PatrizioLegacy.tsx | 12 +++++++-----
 6 files changed, 20 insertions(+), 16 deletions(-)

diff --git a/src/app/[locale]/heritage/page.tsx b/src/app/[locale]/heritage/page.tsx
index d6781d6..d79bafc 100644
--- a/src/app/[locale]/heritage/page.tsx
+++ b/src/app/[locale]/heritage/page.tsx
@@ -197,8 +197,8 @@ export default async function HeritagePage({ params }: { params: Promise<{ local
   const resolvedParams = await params;
   const locale = resolvedParams.locale;
 
-  // 🔥 Cargamos el diccionario para los textos fijos
-  const t = await getTranslations("HeritagePage");
+  // Pass locale explicitly so getTranslations stays static-friendly under ISR.
+  const t = await getTranslations({ locale, namespace: "HeritagePage" });
 
   let rawSections: any[] = [];
   try {
diff --git a/src/app/[locale]/layout.tsx b/src/app/[locale]/layout.tsx
index 68d268c..207508e 100644
--- a/src/app/[locale]/layout.tsx
+++ b/src/app/[locale]/layout.tsx
@@ -72,7 +72,7 @@ export default async function RootLayout({
   }
 
   const [messages, branding, social] = await Promise.all([
-    getMessages(),
+    getMessages({ locale }),
     getBranding(),
     getSocialLinks(),
   ]);
@@ -114,7 +114,7 @@ export default async function RootLayout({
             {children}
           </div>
 
-          <Footer />
+          <Footer locale={locale} />
           <SilentObserver />
 
         </NextIntlClientProvider>
diff --git a/src/app/[locale]/news/page.tsx b/src/app/[locale]/news/page.tsx
index c2c6788..5eda7b6 100644
--- a/src/app/[locale]/news/page.tsx
+++ b/src/app/[locale]/news/page.tsx
@@ -31,8 +31,8 @@ export default async function NewsHub({ params }: { params: Promise<{ locale: st
   const resolvedParams = await params;
   const locale = resolvedParams.locale;
 
-  // 🔥 LLAMAMOS AL DICCIONARIO
-  const t = await getTranslations("NewsHub");
+  // Pass locale explicitly so getTranslations stays static-friendly under ISR.
+  const t = await getTranslations({ locale, namespace: "NewsHub" });
 
   let rawArticles: any[] = [];
   try {
diff --git a/src/app/[locale]/page.tsx b/src/app/[locale]/page.tsx
index 8b27c3f..82a8eb4 100644
--- a/src/app/[locale]/page.tsx
+++ b/src/app/[locale]/page.tsx
@@ -176,7 +176,7 @@ export default async function Home({ params }: { params: Promise<{ locale: strin
         <ApplicationsDashboard dbApps={dbApps} />
         <GlobalOperations dbNodes={mapNodes} dbApps={dbApps} />
         <OurStory dbTimeline={dbTimeline} />
-        <PatrizioLegacy />
+        <PatrizioLegacy locale={locale} />
         <div className="h-64 w-full"></div>
       </div>
     </main>
diff --git a/src/components/layout/Footer.tsx b/src/components/layout/Footer.tsx
index 02f15bc..2362109 100644
--- a/src/components/layout/Footer.tsx
+++ b/src/components/layout/Footer.tsx
@@ -2,14 +2,16 @@ import { Link } from "@/i18n/routing";
 import { Linkedin, Instagram, Youtube, Mail, Phone } from "lucide-react";
 import { prisma } from "@/lib/prisma";
 import { getLocalizedData } from "@/lib/i18nHelper";
-import { getTranslations, getLocale } from "next-intl/server";
+import { getTranslations } from "next-intl/server";
 import { getFooterSettings, getSocialLinks } from "@/lib/siteSettings";
 
 import { AiContactButton, AiFooterLink } from "@/components/ui/AiTriggerButton";
 
-export default async function Footer() {
-  const locale = await getLocale();
-  const t = await getTranslations("Footer");
+// `locale` comes from the parent layout. Reading it via getLocale() here
+// would call cookies()/headers() under the hood, breaking ISR with
+// DYNAMIC_SERVER_USAGE — passing it explicitly keeps everything static.
+export default async function Footer({ locale }: { locale: string }) {
+  const t = await getTranslations({ locale, namespace: "Footer" });
 
   const [footer, social] = await Promise.all([
     getFooterSettings(locale),
diff --git a/src/components/sections/PatrizioLegacy.tsx b/src/components/sections/PatrizioLegacy.tsx
index c2fc28e..43b4b31 100644
--- a/src/components/sections/PatrizioLegacy.tsx
+++ b/src/components/sections/PatrizioLegacy.tsx
@@ -1,10 +1,12 @@
 import { ArrowRight } from "lucide-react";
-import { Link } from "@/i18n/routing"; 
-// 🔥 IMPORTAMOS LA VERSIÓN DE SERVIDOR
-import { getTranslations } from "next-intl/server"; 
+import { Link } from "@/i18n/routing";
+import { getTranslations } from "next-intl/server";
 
-export default async function PatrizioLegacy() {
-  const t = await getTranslations("PatrizioLegacy"); 
+// `locale` is passed in from the parent server component (the page).
+// Calling getTranslations without an explicit locale forces next-intl to
+// read cookies/headers, which trips DYNAMIC_SERVER_USAGE under ISR.
+export default async function PatrizioLegacy({ locale }: { locale: string }) {
+  const t = await getTranslations({ locale, namespace: "PatrizioLegacy" });
 
   return (
     <section id="legacy" className="relative w-full max-w-7xl mx-auto px-6 py-32 md:py-48 z-10">