Fix handling of keys containing dots (v1.0.3)

- Implement proper escaping for object keys that contain dots
- Use null byte as path separator to avoid conflicts with dots in keys
- Fix issue with files like society-flow that have keys like 'auth.login'
- Update all tests to match new path separator format
- Preserve exact JSON structure without interpreting dots as nested paths

This fixes the 'Cannot create property on string' error when processing
JSON files with ambiguous key names.

Author: SySoIAP

package.json

@@ -1,6 +1,6 @@
 {
   "name": "translator-gemini",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "Fast and efficient JSON i18n translator powered by Google Gemini API with incremental caching support",
   "main": "dist/index.js",
   "bin": {

src/helpers.ts

@@ -25,17 +25,27 @@ export function getDefaultCacheFilePath(): string {
   return path.join(getCacheDirectory(), 'translation-cache.json');
 }
 
+// Use a unique separator that won't appear in normal keys
+const PATH_SEPARATOR = '\x00';
+const DOT_ESCAPE = '\x01';
+
 export function flattenObjectWithPaths(obj: JsonValue, currentPath: string = '', result: Map<string, string> = new Map()): Map<string, string> {
   if (typeof obj === 'string') {
     if (/[a-zA-Z]/.test(obj) && !/^{{.*}}$/.test(obj)) {
       result.set(currentPath, obj);
     }
   } else if (Array.isArray(obj)) {
-    obj.forEach((item, index) => flattenObjectWithPaths(item, `${currentPath}[${index}]`, result));
+    obj.forEach((item, index) => {
+      const newPath = currentPath ? `${currentPath}${PATH_SEPARATOR}[${index}]` : `[${index}]`;
+      flattenObjectWithPaths(item, newPath, result);
+    });
   } else if (typeof obj === 'object' && obj !== null) {
     for (const key in obj) {
       if(Object.prototype.hasOwnProperty.call(obj, key)) {
-        flattenObjectWithPaths(obj[key], currentPath ? `${currentPath}.${key}` : key, result);
+        // Escape dots in the key to preserve them
+        const escapedKey = key.replace(/\./g, DOT_ESCAPE);
+        const newPath = currentPath ? `${currentPath}${PATH_SEPARATOR}${escapedKey}` : escapedKey;
+        flattenObjectWithPaths(obj[key], newPath, result);
       }
     }
   }
@@ -44,19 +54,44 @@ export function flattenObjectWithPaths(obj: JsonValue, currentPath: string = '',
 
 export function unflattenObject(flatMap: Map<string, string>): JsonObject {
   const result: JsonObject = {};
+  
   for (const [path, value] of flatMap.entries()) {
-    const keys = path.match(/[^.[\]]+/g) || [];
-    keys.reduce((acc: any, key: string, index: number) => {
-      if (index === keys.length - 1) {
-        acc[key] = value;
+    const parts = path.split(PATH_SEPARATOR);
+    
+    let current: any = result;
+    for (let i = 0; i < parts.length; i++) {
+      const part = parts[i];
+      const isLast = i === parts.length - 1;
+      
+      if (part.startsWith('[') && part.endsWith(']')) {
+        // Array index
+        const index = parseInt(part.slice(1, -1));
+        if (isLast) {
+          current[index] = value;
+        } else {
+          if (!current[index]) {
+            // Look ahead to determine if next is array or object
+            const nextPart = parts[i + 1];
+            current[index] = nextPart.startsWith('[') ? [] : {};
+          }
+          current = current[index];
+        }
       } else {
-        const nextKeyIsNumeric = /^\d+$/.test(keys[index + 1]);
-        if (!acc[key]) {
-          acc[key] = nextKeyIsNumeric ? [] : {};
+        // Regular key - unescape dots
+        const key = part.replace(new RegExp(DOT_ESCAPE, 'g'), '.');
+        if (isLast) {
+          current[key] = value;
+        } else {
+          if (!current[key]) {
+            // Look ahead to determine if next is array or object
+            const nextPart = parts[i + 1];
+            current[key] = nextPart.startsWith('[') ? [] : {};
+          }
+          current = current[key];
         }
       }
-      return acc[key];
-    }, result);
+    }
   }
+  
   return result;
 }

src/index.ts

@@ -176,7 +176,7 @@ async function main() {
 
   const program = new Command();
   program
-    .version('1.0.2')
+    .version('1.0.3')
     .description('A CLI tool to translate and synchronize JSON i18n files.')
     .argument('<inputFile>', 'Path to the source English JSON file.')
     .requiredOption('-l, --lang <langCode>', 'The target language code.')

tests/unit/cli.test.ts

@@ -7,7 +7,7 @@ describe('CLI Argument Parsing', () => {
   beforeEach(() => {
     program = new Command();
     program
-      .version('1.0.2')
+      .version('1.0.3')
       .description('A CLI tool to translate and synchronize JSON i18n files.')
       .argument('<inputFile>', 'Path to the source English JSON file.')
       .requiredOption('-l, --lang <langCode>', 'The target language code.')
@@ -93,7 +93,7 @@ describe('CLI Argument Parsing', () => {
     const args = ['node', 'translator-gemini', '--version'];
 
     // exitOverride causes it to throw with the version string
-    expect(() => program.parse(args)).toThrow('1.0.2');
+    expect(() => program.parse(args)).toThrow('1.0.3');
 
     mockLog.mockRestore();
   });

tests/unit/dots-escape.test.ts

@@ -0,0 +1,50 @@
+import { flattenObjectWithPaths, unflattenObject } from '../../src/helpers';
+
+describe('Dot escaping in keys', () => {
+  it('should preserve dots in original keys', () => {
+    const obj = {
+      "auth.login": "Login",
+      "auth.logout": "Logout",
+      "menu": {
+        "home": "Home",
+        "auth.login": "Sign In"
+      }
+    };
+    
+    const flattened = flattenObjectWithPaths(obj);
+    const unflattened = unflattenObject(flattened);
+    
+    expect(unflattened).toEqual(obj);
+  });
+
+  it('should handle the society-flow menu structure', () => {
+    const obj = {
+      "menu": {
+        "legal": "Legal",
+        "legal.terms-of-service": "Terms of Service",
+        "legal.privacy-policy": "Privacy Policy"
+      }
+    };
+    
+    const flattened = flattenObjectWithPaths(obj);
+    const unflattened = unflattenObject(flattened);
+    
+    expect(unflattened).toEqual(obj);
+    expect((unflattened.menu as any).legal).toBe("Legal");
+    expect((unflattened.menu as any)["legal.terms-of-service"]).toBe("Terms of Service");
+  });
+
+  it('should handle arrays correctly', () => {
+    const obj = {
+      "items": ["one", "two", "three"],
+      "nested": {
+        "list": ["a", "b"]
+      }
+    };
+    
+    const flattened = flattenObjectWithPaths(obj);
+    const unflattened = unflattenObject(flattened);
+    
+    expect(unflattened).toEqual(obj);
+  });
+});

tests/unit/helpers.test.ts

@@ -109,8 +109,8 @@ describe('Helper Functions', () => {
       };
       const result = flattenObjectWithPaths(obj);
 
-      expect(result.get('user.name')).toBe('John');
-      expect(result.get('user.address.city')).toBe('New York');
+      expect(result.get('user\x00name')).toBe('John');
+      expect(result.get('user\x00address\x00city')).toBe('New York');
       expect(result.size).toBe(2);
     });
 
@@ -120,9 +120,9 @@ describe('Helper Functions', () => {
       };
       const result = flattenObjectWithPaths(obj);
 
-      expect(result.get('items[0]')).toBe('apple');
-      expect(result.get('items[1]')).toBe('banana');
-      expect(result.get('items[2]')).toBe('cherry');
+      expect(result.get('items\x00[0]')).toBe('apple');
+      expect(result.get('items\x00[1]')).toBe('banana');
+      expect(result.get('items\x00[2]')).toBe('cherry');
       expect(result.size).toBe(3);
     });
 
@@ -180,8 +180,8 @@ describe('Helper Functions', () => {
 
     it('should unflatten nested paths', () => {
       const flatMap = new Map([
-        ['user.name', 'John'],
-        ['user.address.city', 'New York']
+        ['user\x00name', 'John'],
+        ['user\x00address\x00city', 'New York']
       ]);
       const result = unflattenObject(flatMap);
 
@@ -197,9 +197,9 @@ describe('Helper Functions', () => {
 
     it('should unflatten array paths', () => {
       const flatMap = new Map([
-        ['items[0]', 'apple'],
-        ['items[1]', 'banana'],
-        ['items[2]', 'cherry']
+        ['items\x00[0]', 'apple'],
+        ['items\x00[1]', 'banana'],
+        ['items\x00[2]', 'cherry']
       ]);
       const result = unflattenObject(flatMap);
 
@@ -210,10 +210,10 @@ describe('Helper Functions', () => {
 
     it('should handle mixed nested structures', () => {
       const flatMap = new Map([
-        ['user.name', 'John'],
-        ['user.tags[0]', 'admin'],
-        ['user.tags[1]', 'developer'],
-        ['settings.theme', 'dark']
+        ['user\x00name', 'John'],
+        ['user\x00tags\x00[0]', 'admin'],
+        ['user\x00tags\x00[1]', 'developer'],
+        ['settings\x00theme', 'dark']
       ]);
       const result = unflattenObject(flatMap);