chore: prevent adding missing entries to wrong locale files

Author: userquin

lunaria/prepare-json-files.ts

@@ -88,7 +88,7 @@ async function mergeLocale(locale: LocaleObject): Promise<void> {
 
   await fs.writeFile(
     path.resolve(`${destFolder}/${locale.code}.json`),
-    JSON.stringify(source, null, 2),
+    `${JSON.stringify(source, null, 2)}\n`,
     'utf-8',
   )
 }

scripts/compare-translations.ts

@@ -215,6 +215,30 @@ const processLocale = async (
 ): Promise<SyncStats> => {
   const filePath = join(LOCALES_DIRECTORY, localeFile)
   const localeInfo = checkJsonName(filePath)
+
+  if (fix) {
+    // prevent update
+    // country locale file: e.g. es or es-ES
+    if (localeInfo.forCountry) {
+      // prevent adding missing keys to target locale: e.g. es-ES
+      // in this case i18n will do the work for us, the target locale should be empty
+      if (localeInfo.mergeLocale) {
+        console.error(
+          `${COLORS.red}Error: Locale "${localeInfo.locale}" cannot be fixed, fix the ${localeInfo.lang} locale instead!${COLORS.reset}`,
+        )
+        process.exit(1)
+      }
+    }
+    // prevent fix country locale, i18n will merge lang file and the target locale:
+    // e.g. es-419: i18n will merge es and es-419 for us
+    else if (localeInfo.mergeLocale) {
+      console.error(
+        `${COLORS.red}Error: Locale "${localeInfo.locale}" cannot be fixed.!${COLORS.reset}`,
+      )
+      process.exit(1)
+    }
+  }
+
   const targetContent = await loadJson(localeInfo)
 
   const stats: SyncStats = {