@contrail/transform-data 1.3.0-alpha.2 → 1.3.0

package/.claude/settings.local.json

@@ -1,5 +1,9 @@
 {
   "permissions": {
-    "allow": ["Bash(npx jest --no-coverage)", "Bash(git checkout:*)"]
+    "allow": [
+      "Bash(npx jest --no-coverage)",
+      "Bash(git checkout:*)",
+      "Bash(npx jest:*)"
+    ]
   }
 }

package/lib/common/object-path-util.d.ts

@@ -13,7 +13,7 @@ export declare class ObjectPathUtil {
         pathParts: string[];
         value: unknown;
         options: {
-            shouldOverwrite: boolean;
+            shouldOverwriteExisting: boolean;
         };
     }): void;
     private static expandArraySegments;

package/lib/common/object-path-util.js

@@ -44,14 +44,17 @@ class ObjectPathUtil {
     }
     static setValueAtPath({ obj, pathParts, value, options, }) {
         let current = obj;
-        let pathHasArrayIndex = false;
         for (let i = 0; i < pathParts.length - 1; i++) {
             const key = pathParts[i];
             const isCurrentPartArrayIndex = ObjectPathUtil.isArrayIndexPart(key);
             if (isCurrentPartArrayIndex) {
-                pathHasArrayIndex = true;
                 const idx = ObjectPathUtil.getArrayIndex(key);
                 ObjectPathUtil.ensureArrayFilledTo(current, idx);
+                const isNextPartArrayIndex = ObjectPathUtil.isArrayIndexPart(pathParts[i + 1]);
+                const isElementNotArray = isNextPartArrayIndex && !Array.isArray(current[idx]);
+                if (isElementNotArray) {
+                    current[idx] = [];
+                }
                 current = current[idx];
                 continue;
             }
@@ -68,11 +71,14 @@ class ObjectPathUtil {
         if (isFinalKeyArrayIndex) {
             const idx = ObjectPathUtil.getArrayIndex(finalKey);
             ObjectPathUtil.ensureArrayFilledTo(current, idx);
-            current[idx] = value;
+            const isArrayIndexEmpty = current[idx] == null || (typeof current[idx] === 'object' && Object.keys(current[idx]).length === 0);
+            if (options.shouldOverwriteExisting || isArrayIndexEmpty) {
+                current[idx] = value;
+            }
             return;
         }
-        const shouldWriteProperty = pathHasArrayIndex || !Object.prototype.hasOwnProperty.call(current, finalKey);
-        if (shouldWriteProperty || options.shouldOverwrite) {
+        const shouldWriteProperty = options.shouldOverwriteExisting || !Object.prototype.hasOwnProperty.call(current, finalKey);
+        if (shouldWriteProperty) {
             current[finalKey] = value;
         }
     }

package/lib/rekey/rekey-transformer.js

@@ -22,7 +22,7 @@ class RekeyTransformer {
                     obj: row,
                     pathParts: newKeyParts,
                     value: oldValue,
-                    options: { shouldOverwrite: false },
+                    options: { shouldOverwriteExisting: false },
                 });
             }
             else if (rekeyKeepMissingValues) {
@@ -30,7 +30,7 @@ class RekeyTransformer {
                     obj: row,
                     pathParts: newKeyParts,
                     value: undefined,
-                    options: { shouldOverwrite: false },
+                    options: { shouldOverwriteExisting: false },
                 });
             }
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/transform-data",
-  "version": "1.3.0-alpha.2",
+  "version": "1.3.0",
   "description": "Libraries for transforming data",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
@@ -9,6 +9,7 @@
     "format": "prettier --write \"src/**/*.ts\" \"src/**/*.js\"",
     "lint": "tslint --fix -p tsconfig.json",
     "test": "jest",
+    "test-debug": "npx jest --runInBand",
     "test:watch": "jest --watch"
   },
   "keywords": [],