feat(cli): ignoredKeys in typescript being ignored (#1065)

* feat(cli): fix typescript ignoredKeys for export default and exports

* feat(cli): added types

* feat(cli): updated existing test cases for ignoredKeys in typescript

* fix(cli/run): remove unwanted changes as not needed

* fix(cli/loaders): remove unwanted changes as not needed

* Revert "fix(cli/run): remove unwanted changes as not needed"

This reverts commit 518237f.

* feat(cli): remove unnecessary change

* feat(cli): added ignorekeys support for run

* feat: add changeset

* feat(cli): remove unwanted changes

* feat(cli): remove unwanted changes

* feat(cli): remove unwanted changes

* feat(cli): added ignorekeys for typescript and it test

---------

Co-authored-by: Max Prilutskiy <5614659+maxprilutskiy@users.noreply.github.com>
Co-authored-by: Matej Lednicky <matej@lednicky.name>

Author: 3 people

.changeset/few-boats-tap.md

@@ -0,0 +1,7 @@
+---
+"lingo.dev": minor
+---
+
+Add support for `ignoredKeys` in TypeScript loader
+
+The TypeScript loader now fully supports the `ignoredKeys` option, allowing you to exclude specific keys (including nested keys) from localization when using both `export default` and `export const` patterns. This works seamlessly with the `run` method and the CLI, and is compatible with flattened key structures. All related tests now pass.

packages/cli/src/cli/cmd/run/_types.ts

@@ -28,6 +28,7 @@ export type CmdRunTask = {
   injectLocale: string[];
   lockedKeys: string[];
   lockedPatterns: string[];
+  ignoredKeys: string[];
   onlyKeys: string[];
 };
 

packages/cli/src/cli/cmd/run/execute.ts

@@ -127,6 +127,7 @@ function createLoaderForTask(assignedTask: CmdRunTask) {
     },
     assignedTask.lockedKeys,
     assignedTask.lockedPatterns,
+    assignedTask.ignoredKeys,
   );
   bucketLoader.setDefaultLocale(assignedTask.sourceLocale);
 

packages/cli/src/cli/cmd/run/plan.ts

@@ -132,6 +132,7 @@ export default async function plan(
                   injectLocale: bucket.injectLocale || [],
                   lockedKeys: bucket.lockedKeys || [],
                   lockedPatterns: bucket.lockedPatterns || [],
+                  ignoredKeys: bucket.ignoredKeys || [],
                   onlyKeys: input.flags.key || [],
                 });
               }

packages/cli/src/cli/loaders/ignored-keys.spec.ts

@@ -79,8 +79,6 @@ describe("ignored-keys loader", () => {
     // The loader should have re-inserted the ignored keys from the pull input.
     expect(pushResult).toEqual({
       greeting: "hola",
-      meta: "meta es",
-      todo: "todo es",
     });
   });
 
@@ -170,13 +168,15 @@ describe("ignored-keys loader", () => {
     const pushResult = await loader.push(targetLocale, dataToPush);
     expect(pushResult).toEqual({
       greeting: "hola",
-      meta: "meta es",
-      "pages/0/title": "Título 0",
       "pages/0/content": "Contenido Nuveo",
-      "pages/foo/title": "Título Foo",
       "pages/foo/content": "Contenido Nuevo Foo",
       "pages/bar/notitle": "No Título",
       "pages/bar/content": "Contenido Nuevo Bar",
     });
+    expect(pushResult).not.toEqual({
+      meta: "meta es",
+      "pages/0/title": "Título 0",
+      "pages/foo/title": "Título Foo",
+    });
   });
 });

packages/cli/src/cli/loaders/ignored-keys.ts

@@ -14,10 +14,11 @@ export default function createIgnoredKeysLoader(
       return result;
     },
     push: async (locale, data, originalInput, originalLocale, pullInput) => {
-      const ignoredSubObject = _.pickBy(pullInput, (value, key) =>
+      // Ignored keys loader should remove ignored keys from push data too
+      const result = _.omitBy(data, (value, key) =>
         _isIgnoredKey(key, ignoredKeys),
       );
-      const result = _.merge({}, data, ignoredSubObject);
+
       return result;
     },
   });