@uniformdev/transformer 1.1.51 → 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 ?? ""]
           );
         }
       }
@@ -5259,6 +5312,7 @@ var FieldRemoverService = class {
       compositionsDir,
       compositionPatternsDir,
       componentPatternsDir,
+      contentTypesDir,
       componentType,
       parameterId,
       whatIf,
@@ -5269,6 +5323,7 @@ var FieldRemoverService = class {
     const fullCompositionsDir = this.fileSystem.resolvePath(rootDir, compositionsDir);
     const fullCompositionPatternsDir = this.fileSystem.resolvePath(rootDir, compositionPatternsDir);
     const fullComponentPatternsDir = this.fileSystem.resolvePath(rootDir, componentPatternsDir);
+    const fullContentTypesDir = this.fileSystem.resolvePath(rootDir, contentTypesDir);
     let allIdsToRemove = [parameterId];
     try {
       const { component } = await this.componentService.loadComponent(fullComponentsDir, componentType, findOptions);
@@ -5283,6 +5338,13 @@ var FieldRemoverService = class {
       this.logger.info(`Cascade-removing ${childIds.length} child field(s) [${childIds.join(", ")}] along with "${parameterId}"`);
     }
     this.logger.info(`Removing field "${parameterId}" from instances of ${componentType}`);
+    const contentTypesModified = await this.removeFieldFromContentTypes(
+      fullContentTypesDir,
+      componentType,
+      allIdsToRemove,
+      whatIf,
+      strict
+    );
     const compositionsResult = await this.removeFieldInDirectory(
       fullCompositionsDir,
       componentType,
@@ -5291,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,
@@ -5299,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,
@@ -5307,31 +5375,101 @@ 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("");
     this.logger.info(
-      `Summary: ${totalFiles} file(s) (${totalInstances} instance(s)) updated.`
+      `Summary: ${contentTypesModified} content type(s), ${totalFiles} file(s) (${totalInstances} instance(s)) updated.`
     );
     return {
       compositionsModified: compositionsResult.filesModified,
       compositionPatternsModified: compositionPatternsResult.filesModified,
       componentPatternsModified: componentPatternsResult.filesModified,
+      contentTypesModified,
       instancesUpdated: totalInstances
     };
   }
+  async removeFieldFromContentTypes(contentTypesDir, componentType, fieldIds, whatIf, strict) {
+    let files;
+    try {
+      files = await this.fileSystem.findFiles(contentTypesDir, "**/*.{json,yaml,yml}");
+    } catch {
+      return 0;
+    }
+    if (files.length === 0) {
+      return 0;
+    }
+    const isWildcard = componentType === "*";
+    let contentTypesModified = 0;
+    for (const filePath of files) {
+      let contentType;
+      try {
+        contentType = await this.fileSystem.readFile(filePath);
+      } catch {
+        continue;
+      }
+      if (!contentType?.id || !contentType.fields) {
+        continue;
+      }
+      if (!isWildcard && !this.compareIds(contentType.id, componentType, strict)) {
+        continue;
+      }
+      const removedFieldIds = [];
+      contentType.fields = contentType.fields.filter((field) => {
+        const shouldRemove = fieldIds.some((id) => this.compareIds(field.id, id, strict));
+        if (shouldRemove) {
+          removedFieldIds.push(field.id);
+        }
+        return !shouldRemove;
+      });
+      if (removedFieldIds.length === 0) {
+        continue;
+      }
+      const entryName = contentType.entryName;
+      if (entryName && removedFieldIds.some((id) => this.compareIds(id, entryName, strict))) {
+        this.logger.info(`Clearing entryName "${entryName}" on content type "${contentType.id}" (field was removed)`);
+        contentType.entryName = "";
+      }
+      const relativePath = this.fileSystem.getBasename(filePath);
+      this.logger.action(
+        whatIf,
+        "UPDATE",
+        `contentType/${relativePath} (removed ${removedFieldIds.length} field(s): ${removedFieldIds.join(", ")})`
+      );
+      if (!whatIf) {
+        await this.fileSystem.writeFile(filePath, contentType);
+      }
+      contentTypesModified++;
+    }
+    return contentTypesModified;
+  }
   async removeFieldInDirectory(directory, componentType, parameterIds, whatIf, strict, label) {
     let files;
     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 {
@@ -5342,6 +5480,7 @@ var FieldRemoverService = class {
       if (!composition?.composition) {
         continue;
       }
+      validCompositionDocuments++;
       const treeCount = this.removeFieldFromTree(
         composition.composition,
         componentType,
@@ -5369,7 +5508,7 @@ var FieldRemoverService = class {
         instancesUpdated += totalCount;
       }
     }
-    return { filesModified, instancesUpdated };
+    return { filesModified, instancesUpdated, totalFilesMatched, validCompositionDocuments };
   }
   removeFieldFromTree(node, componentType, parameterIds, strict) {
     let count = 0;
@@ -5395,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) {
@@ -5404,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,
@@ -5413,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 ?? ""]
           );
         }
       }
@@ -5477,6 +5655,8 @@ function createRemoveFieldCommand() {
       parameterId: opts.parameterId
     };
     const logger = new Logger();
+    logger.info(`componentType: ${options.componentType}`);
+    logger.info(`parameterId: ${options.parameterId}`);
     const fileSystem = new FileSystemService();
     const componentService = new ComponentService(fileSystem);
     const remover = new FieldRemoverService(fileSystem, componentService, logger);
@@ -5485,7 +5665,8 @@ function createRemoveFieldCommand() {
     const aggregate = {
       compositionsModified: 0,
       compositionPatternsModified: 0,
-      componentPatternsModified: 0
+      componentPatternsModified: 0,
+      contentTypesModified: 0
     };
     try {
       for (const componentType of componentTypes) {
@@ -5495,6 +5676,7 @@ function createRemoveFieldCommand() {
           compositionsDir: options.compositionsDir,
           compositionPatternsDir: options.compositionPatternsDir,
           componentPatternsDir: options.componentPatternsDir,
+          contentTypesDir: options.contentTypesDir,
           componentType,
           parameterId: options.parameterId,
           whatIf: options.whatIf ?? false,
@@ -5503,9 +5685,10 @@ function createRemoveFieldCommand() {
         aggregate.compositionsModified += result.compositionsModified;
         aggregate.compositionPatternsModified += result.compositionPatternsModified;
         aggregate.componentPatternsModified += result.componentPatternsModified;
+        aggregate.contentTypesModified += result.contentTypesModified;
       }
       logger.success(
-        `Removed field: ${aggregate.compositionsModified} composition(s), ${aggregate.compositionPatternsModified} composition pattern(s), ${aggregate.componentPatternsModified} component pattern(s) updated`
+        `Removed field: ${aggregate.contentTypesModified} content type(s), ${aggregate.compositionsModified} composition(s), ${aggregate.compositionPatternsModified} composition pattern(s), ${aggregate.componentPatternsModified} component pattern(s) updated`
       );
     } catch (error) {
       if (error instanceof TransformError) {
@@ -7704,7 +7887,7 @@ function createClearSlotCommand() {
 // package.json
 var package_default = {
   name: "@uniformdev/transformer",
-  version: "1.1.51",
+  version: "1.1.53",
   description: "CLI tool for transforming Uniform.dev serialization files offline",
   type: "module",
   bin: {