fix(compiler): dictionary merging

maxprilutskiy · maxprilutskiy · commit 8e97256ca4e7 · 2025-06-13T08:23:20.000-07:00
diff --git a/.changeset/eleven-ghosts-roll.md b/.changeset/eleven-ghosts-roll.md
@@ -0,0 +1,7 @@
+---
+"vite-project": patch
+"@lingo.dev/_compiler": patch
+"next-app": patch
+---
+
+fix dictionary merging
diff --git a/demo/next-app/src/lingo/dictionary.js b/demo/next-app/src/lingo/dictionary.js
@@ -5,15 +5,15 @@ export default {
       entries: {
         "0/declaration/body/0/argument": {
           content: {
-            ar: "قم بتعريب تطبيق React الخاص بك بجميع اللغات في دقائق.<element:br></element:br> توسع ليصل إلى الملايين منذ اليوم الأول.",
-            de: "Lokalisieren Sie Ihre React-App in jeder Sprache innerhalb von Minuten.<element:br></element:br> Skalieren Sie vom ersten Tag an auf Millionen.",
-            en: "Localiza tu aplicación React en todos los idiomas en minutos.<element:br></element:br> Escala a millones desde el primer día.",
+            ar: "قم بتوطين تطبيق React الخاص بك بكل لغة في دقائق.<element:br></element:br> قم بالتوسع إلى الملايين من اليوم الأول.",
+            de: "Lokalisieren Sie Ihre React-App in jeder Sprache in Minuten.<element:br></element:br> Skalieren Sie von Anfang an auf Millionen.",
+            en: "Localize your React app in every language in minutes.<element:br></element:br> Scale to millions from day one.",
             es: "Localiza tu aplicación React en cualquier idioma en minutos.<element:br></element:br> Escala a millones desde el primer día.",
-            fr: "Localisez votre application React dans toutes les langues en quelques minutes.<element:br></element:br> Adaptez-vous à des millions d'utilisateurs dès le premier jour.",
-            ja: "数分でReactアプリをあらゆる言語にローカライズ。<element:br></element:br> 初日から数百万規模へスケール。",
-            ko: "몇 분 안에 React 앱을 모든 언어로 현지화하세요.<element:br></element:br> 첫날부터 수백만 규모로 확장하세요.",
-            ru: "Локализуйте своё React-приложение на любой язык за считанные минуты.<element:br></element:br> Масштабируйтесь до миллионов пользователей с первого дня.",
-            zh: "在几分钟内将您的 React 应用程序本地化为每种语言。<element:br></element:br> 从第一天起就支持数百万用户的扩展。",
+            fr: "Localisez votre application React dans toutes les langues en quelques minutes.<element:br></element:br> Évoluez vers des millions dès le premier jour.",
+            ja: "数分でReactアプリをすべての言語にローカライズします。<element:br></element:br> 最初の日から数百万にスケールします。",
+            ko: "몇 분 만에 React 앱을 모든 언어로 로컬라이즈하세요.<element:br></element:br> 첫날부터 수백만 사용자에게 확장할 수 있습니다.",
+            ru: "Локализуйте ваше приложение React на любом языке за считанные минуты.<element:br></element:br> Масштабируйте до миллионов с первого дня.",
+            zh: "在几分钟内将您的React应用本地化为任何语言。<element:br></element:br> 从第一天开始扩展到数百万用户。",
           },
           hash: "2b8865d21f308426a4da45852b27004d",
         },
diff --git a/demo/vite-project/src/lingo/dictionary.js b/demo/vite-project/src/lingo/dictionary.js
@@ -53,9 +53,9 @@ export default {
             es: "¡Bienvenido a tu nueva aplicación de Vite y React! Esta plantilla de inicio incluye todo lo que necesitas para comenzar con Vite y React y Lingo.dev para la internacionalización.",
             fr: "Bienvenue dans votre nouvelle application Vite & React ! Ce modèle de démarrage comprend tout ce dont vous avez besoin pour commencer avec Vite & React et Lingo.dev pour l'internationalisation.",
             ja: "新しいVite＆Reactアプリケーションへようこそ！このスターターテンプレートには、Vite＆ReactとLingo.devを使用した国際化を始めるために必要なすべてが含まれています。",
-            ko: "Vite & React 애플리케이션에 오신 것을 환영합니다! 이 스타터 템플릿에는 Vite & React 및 국제화를 위한 Lingo.dev로 시작하는 데 필요한 모든 것이 포함되어 있습니다.",
+            ko: "새로운 Vite & React 애플리케이션에 오신 것을 환영합니다! 이 스타터 템플릿에는 Vite & React 및 국제화를 위한 Lingo.dev로 시작하는 데 필요한 모든 것이 포함되어 있습니다.",
             ru: "Добро пожаловать в ваше новое приложение на Vite и React! Этот стартовый шаблон включает всё необходимое для начала работы с Vite, React и Lingo.dev для интернационализации.",
-            zh: "欢迎使用全新的 Vite 和 React 应用程序！此入门模板包含了使用 Vite 和 React 以及 Lingo.dev 进行国际化所需的一切。",
+            zh: "欢迎使用您的全新 Vite 和 React 应用程序！此入门模板包含了使用 Vite 和 React 以及 Lingo.dev 进行国际化所需的一切。",
           },
           hash: "c6deea18ed327336cf526f7d5a268d18",
         },
@@ -95,7 +95,7 @@ export default {
             es: "Haz clic en los logos de arriba para obtener más información",
             fr: "Cliquez sur les logos ci-dessus pour en savoir plus",
             ja: "詳細を学ぶには上記のロゴをクリックしてください",
-            ko: "위의 로고를 클릭하여 자세히 알아보세요",
+            ko: "더 자세히 알아보려면 위의 로고를 클릭하세요",
             ru: "Нажмите на логотипы выше, чтобы узнать больше",
             zh: "点击上方的标志以了解更多信息",
           },
diff --git a/packages/compiler/src/lib/lcp/cache.ts b/packages/compiler/src/lib/lcp/cache.ts
@@ -57,53 +57,65 @@ export class LCPCache {
     dictionary: DictionarySchema,
     lcp: LCPSchema,
   ): DictionaryCacheSchema {
-    const files = _(dictionary.files)
-      .mapValues((file, fileName) => ({
-        ...file,
-        entries: _(file.entries)
-          .mapValues((entry, entryName) => {
-            // find if entry exists in current cache, it might contain some locales already
-            const cachedEntry =
-              _.get(currentCache, ["files", fileName, "entries", entryName]) ??
-              {};
-            const hash = _.get(lcp, [
-              "files",
-              fileName,
-              "scopes",
-              entryName,
-              "hash",
-            ]);
-
-            // reuse existing cache entry if its hash matches LCP schema, ensures the cache is up to date
-            const cachedEntryContent =
-              cachedEntry.hash === hash ? cachedEntry.content : {};
-
-            // sorted by keys (locales) to minimize diffs
-            const content = _({
-              ...cachedEntryContent,
-              [dictionary.locale]: entry,
-            })
-              .toPairs()
-              .sortBy([0])
-              .fromPairs()
-              .value();
-            return { content, hash };
-          })
+    // Deep-clone to avoid mutating caller's object
+    const newCache: DictionaryCacheSchema = _.cloneDeep(currentCache);
+
+    for (const [fileName, fileData] of Object.entries(dictionary.files)) {
+      for (const [entryName, entryValue] of Object.entries(fileData.entries)) {
+        const hash = _.get(lcp, [
+          "files",
+          fileName,
+          "scopes",
+          entryName,
+          "hash",
+        ]);
+
+        const existingEntry = _.get(newCache, [
+          "files",
+          fileName,
+          "entries",
+          entryName,
+        ]) || {
+          content: {},
+          hash,
+        };
+
+        // Merge locales and sort them alphabetically to stabilise diffs
+        const mergedContent = _({
+          ...existingEntry.content,
+          [dictionary.locale]: entryValue,
+        })
           .toPairs()
           .sortBy([0])
           .fromPairs()
-          .value(),
-      }))
+          .value();
+
+        _.set(newCache, ["files", fileName, "entries", entryName], {
+          content: mergedContent,
+          hash,
+        });
+      }
+    }
+
+    // Final sort of files and their entries for minimal diffs
+    const sortedFiles = _(newCache.files)
       .toPairs()
       .sortBy([0])
+      .map(([fileName, file]) => {
+        const sortedEntries = _(file.entries)
+          .toPairs()
+          .sortBy([0])
+          .fromPairs()
+          .value();
+        return [fileName, { entries: sortedEntries }];
+      })
       .fromPairs()
       .value();
 
-    const newCache = {
+    return {
       version: dictionary.version,
-      files,
+      files: sortedFiles,
     };
-    return newCache;
   }
 
   // extract dictionary from cache for given locale, validate entry hash from LCP schema
