feat: shoudTranslate

Author: maxprilutskiy

packages/cli/src/cli/loaders/index.spec.ts

@@ -1070,6 +1070,7 @@ user.password=Contraseña
     it("should respect shouldTranslate: false flag", async () => {
       setupFileMocks();
 
+      // Create input with a key that has shouldTranslate: false
       const input = `{
   "sourceLanguage": "en",
   "strings": {
@@ -1103,11 +1104,13 @@ user.password=Contraseña
       const xcodeXcstringsLoader = createBucketLoader("xcode-xcstrings", "i18n/[locale].xcstrings", { isCacheRestore: false, defaultLocale: "en" });
       xcodeXcstringsLoader.setDefaultLocale("en");
 
+      // Pull data and verify that the shouldTranslate: false entry is skipped
       const data = await xcodeXcstringsLoader.pull("en");
 
       expect(data).toHaveProperty("normal_key", "This should be translated");
       expect(data).not.toHaveProperty("do_not_translate");
 
+      // Now push a translation for the normal key
       const payload = {
         "normal_key": "Ceci devrait être traduit"
       };
@@ -1118,15 +1121,22 @@ user.password=Contraseña
       const writeFileCall = (fs.writeFile as any).mock.calls[0];
       const writtenContent = JSON.parse(writeFileCall[1]);
 
+      // Verify that the normal key was translated
       expect(writtenContent.strings.normal_key.localizations.fr.stringUnit.value).toBe("Ceci devrait être traduit");
+      
+      // Verify that the do_not_translate key was preserved with its shouldTranslate: false flag
       expect(writtenContent.strings.do_not_translate).toHaveProperty("shouldTranslate", false);
+      
+      // Check that the do_not_translate key has no localizations for the target language
       expect(writtenContent.strings.do_not_translate.localizations).not.toHaveProperty("fr");
 
+      // Now also test pushing with empty {} content to verify the flag is still preserved
       await xcodeXcstringsLoader.push("fr", {});
 
       const secondWriteFileCall = (fs.writeFile as any).mock.calls[1];
       const secondWrittenContent = JSON.parse(secondWriteFileCall[1]);
 
+      // Verify the flag is still there
       expect(secondWrittenContent.strings.do_not_translate).toHaveProperty("shouldTranslate", false);
     });
   });