From 1bc9fe93ec4ca82295e77f2c8068bc700ef94367 Mon Sep 17 00:00:00 2001
From: SnapOtter <snapotter.hq@gmail.com>
Date: Sat, 11 Jul 2026 15:00:13 +0800
Subject: [PATCH] feat(landing-i18n): recall a returning visitor's chosen
 language

Render-blocking head script that routes a returning visitor to the language they explicitly picked (stored on switch). Only on default English pages; skips English-only tool-detail pages; crawler-safe. Admin-merged: landing-only PR, the skipped Integration matrix jobs never report the required contexts (known repo deadlock). Logic verified across 10 scenarios; landing build + typecheck clean.
---
 apps/landing/src/layouts/Base.astro | 29 ++++++++++++++++++++++++++++-
 1 file changed, 28 insertions(+), 1 deletion(-)

diff --git a/apps/landing/src/layouts/Base.astro b/apps/landing/src/layouts/Base.astro
index 204448e9..4effe96b 100644
--- a/apps/landing/src/layouts/Base.astro
+++ b/apps/landing/src/layouts/Base.astro
@@ -4,7 +4,7 @@ import "@/styles/globals.css";
 import { TOOLS } from "@snapotter/shared";
 import LaunchBanner from "@/components/LaunchBanner.astro";
 import MachineTranslationBanner from "@/components/MachineTranslationBanner.astro";
-import { t } from "@/i18n";
+import { LANDING_LOCALES, t } from "@/i18n";
 import { altLinks, dirFor, ogLocale } from "@/lib/i18n-page";
 
 interface Props {
@@ -43,6 +43,33 @@ const htmlDir = dirFor(locale);
     <meta charset="utf-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1" />
 
+    <!-- Locale recall: send a returning visitor to the language they explicitly picked
+         (stored on switch in LanguageSwitcher). Render-blocking so there is no flash.
+         Crawler-safe: bots have no localStorage. Only acts on the default English pages,
+         and never on the English-only tool-detail pages (which have no localized version),
+         so it can never redirect to a URL that does not exist. -->
+    <script
+      is:inline
+      define:vars={{
+        recallCurrent: locale,
+        recallPath: path,
+        recallLocales: LANDING_LOCALES.filter((c) => c !== "en"),
+      }}
+    >
+      (function () {
+        try {
+          if (recallCurrent !== "en") return;
+          var stored = localStorage.getItem("snapotter:locale");
+          if (!stored || recallLocales.indexOf(stored) === -1) return;
+          var segs = recallPath.split("/").filter(Boolean);
+          if (segs[0] === "tools" && segs.length === 3) return;
+          var target = "/" + stored + (recallPath === "/" ? "/" : recallPath);
+          if (target.charAt(target.length - 1) !== "/") target += "/";
+          location.replace(target + location.search + location.hash);
+        } catch (e) {}
+      })();
+    </script>
+
     <!-- TEMP: 2.0 launch banner dismissal. Render-blocking so a returning visitor
          who closed the banner never sees it flash. Remove with the banner. -->
     <script is:inline>