@contrail/transform-data 1.3.0-alpha.3 → 1.3.1

package/CHANGELOG.md

@@ -0,0 +1,13 @@
+# Changelog
+
+All notable changes to `@contrail/transform-data` are documented here.
+
+Format follows [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
+Versioning follows [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+
+## [Unreleased]
+
+## [1.3.0] - (initial changelog entry)
+
+- Initial changelog. Prior releases did not include a changelog.
+  See git history for changes predating this entry.

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.shouldOverwriteExisting) {
+        const shouldWriteProperty = options.shouldOverwriteExisting || !Object.prototype.hasOwnProperty.call(current, finalKey);
+        if (shouldWriteProperty) {
             current[finalKey] = value;
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/transform-data",
-  "version": "1.3.0-alpha.3",
+  "version": "1.3.1",
   "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": [],
@@ -46,7 +47,7 @@
   },
   "dependencies": {
     "@contrail/sdk": "^1.5.3",
-    "@contrail/util": "^1.1.18",
+    "@contrail/util": "^1.3.0",
     "node-fetch": "^3.2.10",
     "require-from-url": "^3.1.3"
   }

package/.claude/settings.local.json

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