fix(frontend): auto-redirect root path to locale based on browser language

- Update proxy.ts matcher to capture all paths including /
- Remove basePath placeholder from Dockerfile (was breaking redirects)
- Extend entrypoint placeholder replacement coverage

Author: damienlaine

frontend/Dockerfile

@@ -19,8 +19,9 @@ COPY . .
 # Disable telemetry during build
 ENV NEXT_TELEMETRY_DISABLED=1
 
-# Build with placeholders - will be replaced at runtime by entrypoint script
-ENV NEXT_PUBLIC_BASE_PATH="/__NEXT_BASEPATH_PLACEHOLDER__"
+# Build without basePath (root deployment)
+# For subpath deployment, rebuild with NEXT_PUBLIC_BASE_PATH set
+ENV NEXT_PUBLIC_BASE_PATH=""
 ENV NEXT_PUBLIC_API_URL="__NEXT_API_URL_PLACEHOLDER__"
 ENV NEXT_PUBLIC_WS_URL="__NEXT_WS_URL_PLACEHOLDER__"
 

frontend/docker-entrypoint.sh

@@ -14,17 +14,21 @@ if [ -n "$BASE_PATH" ]; then
     # Escaped version first (for regex patterns in JSON)
     find /app/.next/static -type f -name "*.json" -exec sed -i "s|${BASEPATH_PLACEHOLDER_ESCAPED}|${ESCAPED_BASE_PATH}|g" {} + 2>/dev/null || true
     find /app/.next/static -type f \( -name "*.js" -o -name "*.json" \) -exec sed -i "s|${BASEPATH_PLACEHOLDER}|${BASE_PATH}|g" {} + 2>/dev/null || true
-    find /app/.next/server -type f \( -name "*.html" -o -name "*.rsc" \) -exec sed -i "s|${BASEPATH_PLACEHOLDER}|${BASE_PATH}|g" {} + 2>/dev/null || true
+    find /app/.next/server -type f \( -name "*.html" -o -name "*.rsc" -o -name "*.meta" -o -name "*.body" \) -exec sed -i "s|${BASEPATH_PLACEHOLDER}|${BASE_PATH}|g" {} + 2>/dev/null || true
     find /app/.next/server -type f -name "*client-reference-manifest.js" -exec sed -i "s|${BASEPATH_PLACEHOLDER}|${BASE_PATH}|g" {} + 2>/dev/null || true
+    # Also handle prerender manifests and action manifests
+    find /app/.next -type f -name "*.json" -exec sed -i "s|${BASEPATH_PLACEHOLDER}|${BASE_PATH}|g" {} + 2>/dev/null || true
 else
     echo "No BASE_PATH set, running at root path"
     sed -i "s|${BASEPATH_PLACEHOLDER}||g" /app/server.js 2>/dev/null || true
     sed -i "s|${BASEPATH_PLACEHOLDER}||g" /app/.next/routes-manifest.json 2>/dev/null || true
     # Escaped version first (for regex patterns in JSON)
     find /app/.next/static -type f -name "*.json" -exec sed -i "s|${BASEPATH_PLACEHOLDER_ESCAPED}||g" {} + 2>/dev/null || true
     find /app/.next/static -type f \( -name "*.js" -o -name "*.json" \) -exec sed -i "s|${BASEPATH_PLACEHOLDER}||g" {} + 2>/dev/null || true
-    find /app/.next/server -type f \( -name "*.html" -o -name "*.rsc" \) -exec sed -i "s|${BASEPATH_PLACEHOLDER}||g" {} + 2>/dev/null || true
+    find /app/.next/server -type f \( -name "*.html" -o -name "*.rsc" -o -name "*.meta" -o -name "*.body" \) -exec sed -i "s|${BASEPATH_PLACEHOLDER}||g" {} + 2>/dev/null || true
     find /app/.next/server -type f -name "*client-reference-manifest.js" -exec sed -i "s|${BASEPATH_PLACEHOLDER}||g" {} + 2>/dev/null || true
+    # Also handle prerender manifests and action manifests
+    find /app/.next -type f -name "*.json" -exec sed -i "s|${BASEPATH_PLACEHOLDER}||g" {} + 2>/dev/null || true
 fi
 
 API_URL="${NEXT_PUBLIC_API_URL:-http://localhost:8000}"

frontend/proxy.ts

@@ -4,6 +4,11 @@ import { routing } from './i18n/routing';
 export default createMiddleware(routing);
 
 export const config = {
-  // Match only internationalized pathnames
-  matcher: ['/', '/(fr|en)/:path*'],
+  matcher: [
+    // Match all pathnames except:
+    // - API routes starting with /api
+    // - Next.js internals (_next)
+    // - Static files with extensions
+    '/((?!api|_next|.*\\..*).*)'
+  ],
 };