fix(cli): po files --frozen flag (#616)

Fix spec tests.

Author: mathio

.changeset/lazy-flies-draw.md

@@ -0,0 +1,6 @@
+---
+"@lingo.dev/_spec": patch
+"lingo.dev": patch
+---
+
+po files --frozen flag

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

@@ -94,8 +94,8 @@ export default function createBucketLoader(
         createPoLoader(),
         createFlatLoader(),
         createSyncLoader(),
-        createUnlocalizableLoader(options.isCacheRestore, options.returnUnlocalizedKeys),
         createVariableLoader({ type: "python" }),
+        createUnlocalizableLoader(options.isCacheRestore, options.returnUnlocalizedKeys),
       );
     case "properties":
       return composeLoaders(
@@ -127,8 +127,8 @@ export default function createBucketLoader(
         createXcodeXcstringsLoader(options.defaultLocale),
         createFlatLoader(),
         createSyncLoader(),
-        createUnlocalizableLoader(options.isCacheRestore, options.returnUnlocalizedKeys),
         createVariableLoader({ type: "ieee" }),
+        createUnlocalizableLoader(options.isCacheRestore, options.returnUnlocalizedKeys),
       );
     case "yaml":
       return composeLoaders(

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

@@ -24,7 +24,8 @@ function variableExtractLoader(
   return createLoader({
     pull: async (locale, input) => {
       const result: Record<string, VariableExtractionPayload> = {};
-      for (const [key, value] of Object.entries(input)) {
+      const inputValues = _.omitBy(input, _.isEmpty);
+      for (const [key, value] of Object.entries(inputValues)) {
         const matches = value.match(specifierPattern) || [];
         result[key] = result[key] || {
           value,

packages/spec/src/config.spec.ts

@@ -1,5 +1,5 @@
 import { describe, it, expect } from "vitest";
-import { parseI18nConfig, defaultConfig } from "./config"; // Adjust the import path as needed
+import { parseI18nConfig, defaultConfig, LATEST_CONFIG_DEFINITION } from "./config";
 
 // Helper function to create a v0 config
 const createV0Config = () => ({
@@ -71,7 +71,7 @@ describe("I18n Config Parser", () => {
     const result = parseI18nConfig(v0Config);
 
     expect(result["$schema"]).toBeDefined();
-    expect(result.version).toBe(1.4);
+    expect(result.version).toBe(LATEST_CONFIG_DEFINITION.defaultValue.version);
     expect(result.locale).toEqual(defaultConfig.locale);
     expect(result.buckets).toEqual({});
   });
@@ -81,7 +81,7 @@ describe("I18n Config Parser", () => {
     const result = parseI18nConfig(v1Config);
 
     expect(result["$schema"]).toBeDefined();
-    expect(result.version).toBe(1.4);
+    expect(result.version).toBe(LATEST_CONFIG_DEFINITION.defaultValue.version);
     expect(result.locale).toEqual(v1Config.locale);
     expect(result.buckets).toEqual({
       json: {
@@ -119,7 +119,7 @@ describe("I18n Config Parser", () => {
   it("should throw an error for unsupported locales", () => {
     const invalidLocaleConfig = createInvalidLocaleConfig();
     expect(() => parseI18nConfig(invalidLocaleConfig)).toThrow(
-      `\nUnsupported locale: ${invalidLocaleConfig.locale.source}\nUnsupported locale: ${invalidLocaleConfig.locale.targets[1]}`,
+      `\nUnsupported locale: ${invalidLocaleConfig.locale.source}\nUnsupported locale: ${invalidLocaleConfig.locale.targets[1]}`
     );
   });
 });