@uniformdev/transformer 1.1.52 → 1.1.53

package/dist/cli/index.js

@@ -4977,6 +4977,9 @@ var ParameterRemoverService = class {
       strict,
       "composition"
     );
+    this.logger.info(
+      `Compositions (${compositionsDir}): ${compositionsResult.totalFilesMatched} file(s) matched, ${compositionsResult.validCompositionDocuments} with a composition root.`
+    );
     const compositionPatternsResult = await this.removeParameterInDirectory(
       fullCompositionPatternsDir,
       componentType,
@@ -4985,6 +4988,9 @@ var ParameterRemoverService = class {
       strict,
       "compositionPattern"
     );
+    this.logger.info(
+      `Composition patterns (${compositionPatternsDir}): ${compositionPatternsResult.totalFilesMatched} file(s) matched, ${compositionPatternsResult.validCompositionDocuments} with a composition root.`
+    );
     const componentPatternsResult = await this.removeParameterInDirectory(
       fullComponentPatternsDir,
       componentType,
@@ -4993,6 +4999,9 @@ var ParameterRemoverService = class {
       strict,
       "componentPattern"
     );
+    this.logger.info(
+      `Component patterns (${componentPatternsDir}): ${componentPatternsResult.totalFilesMatched} file(s) matched, ${componentPatternsResult.validCompositionDocuments} with a composition root.`
+    );
     const totalFiles = compositionsResult.filesModified + compositionPatternsResult.filesModified + componentPatternsResult.filesModified;
     const totalInstances = compositionsResult.instancesUpdated + compositionPatternsResult.instancesUpdated + componentPatternsResult.instancesUpdated;
     this.logger.info("");
@@ -5012,13 +5021,25 @@ var ParameterRemoverService = class {
     try {
       files = await this.fileSystem.findFiles(directory, "**/*.{json,yaml,yml}");
     } catch {
-      return { filesModified: 0, instancesUpdated: 0 };
+      return {
+        filesModified: 0,
+        instancesUpdated: 0,
+        totalFilesMatched: 0,
+        validCompositionDocuments: 0
+      };
     }
+    const totalFilesMatched = files.length;
     if (files.length === 0) {
-      return { filesModified: 0, instancesUpdated: 0 };
+      return {
+        filesModified: 0,
+        instancesUpdated: 0,
+        totalFilesMatched: 0,
+        validCompositionDocuments: 0
+      };
     }
     let filesModified = 0;
     let instancesUpdated = 0;
+    let validCompositionDocuments = 0;
     for (const filePath of files) {
       let composition;
       try {
@@ -5029,6 +5050,7 @@ var ParameterRemoverService = class {
       if (!composition?.composition) {
         continue;
       }
+      validCompositionDocuments++;
       const count = this.removeParameterFromTree(
         composition.composition,
         componentType,
@@ -5056,7 +5078,7 @@ var ParameterRemoverService = class {
         instancesUpdated += totalCount;
       }
     }
-    return { filesModified, instancesUpdated };
+    return { filesModified, instancesUpdated, totalFilesMatched, validCompositionDocuments };
   }
   removeParameterFromTree(node, componentType, parameterIds, strict) {
     let count = 0;
@@ -5098,24 +5120,54 @@ var ParameterRemoverService = class {
       componentType,
       parameterIds,
       strict,
-      counter
+      counter,
+      [composition.composition._id]
     );
     count += counter.count;
+    const slotTreeCounter = { count: 0 };
+    this.removeParameterFromOverrideSlotTrees(
+      composition.composition._overrides,
+      componentType,
+      parameterIds,
+      strict,
+      slotTreeCounter
+    );
+    count += slotTreeCounter.count;
     return count;
   }
-  removeOverridesForMatchingInstances(node, overrides, componentType, parameterIds, strict, counter) {
+  removeParameterFromOverrideSlotTrees(overrides, componentType, parameterIds, strict, counter) {
+    for (const entry of Object.values(overrides)) {
+      const slots = entry.slots;
+      if (!slots) continue;
+      for (const slotInstances of Object.values(slots)) {
+        if (!Array.isArray(slotInstances)) continue;
+        for (const instance of slotInstances) {
+          counter.count += this.removeParameterFromTree(instance, componentType, parameterIds, strict);
+        }
+      }
+    }
+  }
+  removeOverridesForMatchingInstances(node, overrides, componentType, parameterIds, strict, counter, idPath) {
     if (node.slots) {
       for (const slotInstances of Object.values(node.slots)) {
         if (!Array.isArray(slotInstances)) continue;
         for (const instance of slotInstances) {
           if (this.compareIds(instance.type, componentType, strict) && instance._id) {
-            const instanceOverrides = overrides[instance._id];
-            if (instanceOverrides?.parameters) {
-              const removed = this.removeKeysFromMap(instanceOverrides.parameters, parameterIds, strict);
-              if (removed > 0) {
-                counter.count++;
+            const compoundKey = [...idPath, instance._id].join("|");
+            const keysToTry = compoundKey === instance._id ? [compoundKey] : [compoundKey, instance._id];
+            let removedAny = false;
+            for (const overrideKey of keysToTry) {
+              const instanceOverrides = overrides[overrideKey];
+              if (instanceOverrides?.parameters) {
+                const removed = this.removeKeysFromMap(instanceOverrides.parameters, parameterIds, strict);
+                if (removed > 0) {
+                  removedAny = true;
+                }
               }
             }
+            if (removedAny) {
+              counter.count++;
+            }
           }
           this.removeOverridesForMatchingInstances(
             instance,
@@ -5123,7 +5175,8 @@ var ParameterRemoverService = class {
             componentType,
             parameterIds,
             strict,
-            counter
+            counter,
+            [...idPath, instance._id ?? ""]
           );
         }
       }
@@ -5300,6 +5353,9 @@ var FieldRemoverService = class {
       strict,
       "composition"
     );
+    this.logger.info(
+      `Compositions (${compositionsDir}): ${compositionsResult.totalFilesMatched} file(s) matched, ${compositionsResult.validCompositionDocuments} with a composition root.`
+    );
     const compositionPatternsResult = await this.removeFieldInDirectory(
       fullCompositionPatternsDir,
       componentType,
@@ -5308,6 +5364,9 @@ var FieldRemoverService = class {
       strict,
       "compositionPattern"
     );
+    this.logger.info(
+      `Composition patterns (${compositionPatternsDir}): ${compositionPatternsResult.totalFilesMatched} file(s) matched, ${compositionPatternsResult.validCompositionDocuments} with a composition root.`
+    );
     const componentPatternsResult = await this.removeFieldInDirectory(
       fullComponentPatternsDir,
       componentType,
@@ -5316,6 +5375,9 @@ var FieldRemoverService = class {
       strict,
       "componentPattern"
     );
+    this.logger.info(
+      `Component patterns (${componentPatternsDir}): ${componentPatternsResult.totalFilesMatched} file(s) matched, ${componentPatternsResult.validCompositionDocuments} with a composition root.`
+    );
     const totalFiles = compositionsResult.filesModified + compositionPatternsResult.filesModified + componentPatternsResult.filesModified;
     const totalInstances = compositionsResult.instancesUpdated + compositionPatternsResult.instancesUpdated + componentPatternsResult.instancesUpdated;
     this.logger.info("");
@@ -5389,13 +5451,25 @@ var FieldRemoverService = class {
     try {
       files = await this.fileSystem.findFiles(directory, "**/*.{json,yaml,yml}");
     } catch {
-      return { filesModified: 0, instancesUpdated: 0 };
+      return {
+        filesModified: 0,
+        instancesUpdated: 0,
+        totalFilesMatched: 0,
+        validCompositionDocuments: 0
+      };
     }
+    const totalFilesMatched = files.length;
     if (files.length === 0) {
-      return { filesModified: 0, instancesUpdated: 0 };
+      return {
+        filesModified: 0,
+        instancesUpdated: 0,
+        totalFilesMatched: 0,
+        validCompositionDocuments: 0
+      };
     }
     let filesModified = 0;
     let instancesUpdated = 0;
+    let validCompositionDocuments = 0;
     for (const filePath of files) {
       let composition;
       try {
@@ -5406,6 +5480,7 @@ var FieldRemoverService = class {
       if (!composition?.composition) {
         continue;
       }
+      validCompositionDocuments++;
       const treeCount = this.removeFieldFromTree(
         composition.composition,
         componentType,
@@ -5433,7 +5508,7 @@ var FieldRemoverService = class {
         instancesUpdated += totalCount;
       }
     }
-    return { filesModified, instancesUpdated };
+    return { filesModified, instancesUpdated, totalFilesMatched, validCompositionDocuments };
   }
   removeFieldFromTree(node, componentType, parameterIds, strict) {
     let count = 0;
@@ -5459,8 +5534,8 @@ var FieldRemoverService = class {
     }
     let count = 0;
     if (this.compareIds(composition.composition.type, componentType, strict)) {
-      const rootId = composition.composition._id;
-      const rootOverrides = composition.composition._overrides[rootId];
+      const rootId2 = composition.composition._id;
+      const rootOverrides = composition.composition._overrides[rootId2];
       if (rootOverrides?.parameters) {
         const removed = this.removeKeysFromMap(rootOverrides.parameters, parameterIds, strict);
         if (removed > 0) {
@@ -5468,8 +5543,16 @@ var FieldRemoverService = class {
         }
       }
     }
+    const rootId = composition.composition._id;
     count += this.removeOverridesForMatchingInstances(
       composition.composition,
+      composition.composition._overrides,
+      componentType,
+      parameterIds,
+      strict,
+      [rootId]
+    );
+    count += this.removeFieldFromOverrideSlotTrees(
       composition.composition._overrides,
       componentType,
       parameterIds,
@@ -5477,27 +5560,58 @@ var FieldRemoverService = class {
     );
     return count;
   }
-  removeOverridesForMatchingInstances(node, overrides, componentType, parameterIds, strict) {
+  /**
+   * Walks every _overrides entry's `slots` trees (if present) and removes parameter keys from
+   * matching component instances. Uniform often stores nested slot patches only here.
+   */
+  removeFieldFromOverrideSlotTrees(overrides, componentType, parameterIds, strict) {
+    let count = 0;
+    for (const entry of Object.values(overrides)) {
+      const slots = entry.slots;
+      if (!slots) continue;
+      for (const slotInstances of Object.values(slots)) {
+        if (!Array.isArray(slotInstances)) continue;
+        for (const instance of slotInstances) {
+          count += this.removeFieldFromTree(instance, componentType, parameterIds, strict);
+        }
+      }
+    }
+    return count;
+  }
+  /**
+   * Looks up override entries by compound key (ancestor _id chain joined with |) and,
+   * for backward compatibility, by instance _id alone when it differs from the compound key.
+   */
+  removeOverridesForMatchingInstances(node, overrides, componentType, parameterIds, strict, idPath) {
     let count = 0;
     if (node.slots) {
       for (const slotInstances of Object.values(node.slots)) {
         if (!Array.isArray(slotInstances)) continue;
         for (const instance of slotInstances) {
           if (this.compareIds(instance.type, componentType, strict) && instance._id) {
-            const instanceOverrides = overrides[instance._id];
-            if (instanceOverrides?.parameters) {
-              const removed = this.removeKeysFromMap(instanceOverrides.parameters, parameterIds, strict);
-              if (removed > 0) {
-                count++;
+            const compoundKey = [...idPath, instance._id].join("|");
+            const keysToTry = compoundKey === instance._id ? [compoundKey] : [compoundKey, instance._id];
+            let removedAny = false;
+            for (const overrideKey of keysToTry) {
+              const instanceOverrides = overrides[overrideKey];
+              if (instanceOverrides?.parameters) {
+                const removed = this.removeKeysFromMap(instanceOverrides.parameters, parameterIds, strict);
+                if (removed > 0) {
+                  removedAny = true;
+                }
               }
             }
+            if (removedAny) {
+              count++;
+            }
           }
           count += this.removeOverridesForMatchingInstances(
             instance,
             overrides,
             componentType,
             parameterIds,
-            strict
+            strict,
+            [...idPath, instance._id ?? ""]
           );
         }
       }
@@ -7773,7 +7887,7 @@ function createClearSlotCommand() {
 // package.json
 var package_default = {
   name: "@uniformdev/transformer",
-  version: "1.1.52",
+  version: "1.1.53",
   description: "CLI tool for transforming Uniform.dev serialization files offline",
   type: "module",
   bin: {