@uniformdev/transformer 1.1.2 → 1.1.4

package/dist/cli/index.js

@@ -2422,6 +2422,12 @@ var ComponentAdderService = class {
     }
     return id1.toLowerCase() === id2.toLowerCase();
   }
+  parseParentComponentTypes(parentComponentType) {
+    return parentComponentType.split("|").map((t) => t.trim()).filter((t) => t.length > 0);
+  }
+  matchesAnyParentType(instanceType, parentTypes, strict) {
+    return parentTypes.some((pt) => this.compareIds(instanceType, pt, strict));
+  }
   parseParameters(parameterString) {
     const result = {};
     if (!parameterString) return result;
@@ -2437,25 +2443,9 @@ var ComponentAdderService = class {
   generateId(baseName) {
     return `${baseName}-${randomUUID().slice(0, 8)}`;
   }
-  cloneComponentInstance(instance) {
-    const clone = JSON.parse(JSON.stringify(instance));
-    if (clone._id) {
-      clone._id = this.generateId(clone.type);
-    }
-    if (clone.slots) {
-      for (const slotInstances of Object.values(clone.slots)) {
-        if (Array.isArray(slotInstances)) {
-          for (const nestedInstance of slotInstances) {
-            this.regenerateInstanceIds(nestedInstance);
-          }
-        }
-      }
-    }
-    return clone;
-  }
   regenerateInstanceIds(instance) {
     if (instance._id) {
-      instance._id = this.generateId(instance.type);
+      instance._id = instance._pattern ? randomUUID() : this.generateId(instance.type);
     }
     if (instance.slots) {
       for (const slotInstances of Object.values(instance.slots)) {
@@ -2511,18 +2501,9 @@ var ComponentAdderService = class {
     const fullCompositionPatternsDir = this.fileSystem.resolvePath(rootDir, compositionPatternsDir);
     const fullComponentPatternsDir = this.fileSystem.resolvePath(rootDir, componentPatternsDir);
     const findOptions = { strict };
-    this.logger.info(`Loading parent component: ${parentComponentType}`);
-    const { component: parentComponent, filePath: parentComponentFilePath } = await this.componentService.loadComponent(fullComponentsDir, parentComponentType, findOptions);
-    if (!parentComponent.slots) {
-      parentComponent.slots = [];
-    }
-    let slotDef = parentComponent.slots.find(
-      (s) => this.compareIds(s.id, slot, strict)
-    );
-    if (!slotDef) {
-      this.logger.info(`Slot "${slot}" not found on component "${parentComponentType}", creating it`);
-      slotDef = { id: slot, name: slot, allowedComponents: [] };
-      parentComponent.slots.push(slotDef);
+    const parentTypes = this.parseParentComponentTypes(parentComponentType);
+    if (parentTypes.length === 0) {
+      throw new TransformError("parentComponentType cannot be empty");
     }
     this.logger.info(`Validating new component: ${newComponentType}`);
     try {
@@ -2536,35 +2517,50 @@ var ComponentAdderService = class {
       throw error;
     }
     let allowedComponentsUpdated = false;
-    const slotIndex = parentComponent.slots?.findIndex(
-      (s) => this.compareIds(s.id, slotDef.id, strict)
-    );
-    if (slotIndex !== void 0 && slotIndex >= 0) {
-      const actualSlot = parentComponent.slots[slotIndex];
-      if (!actualSlot.allowedComponents) {
-        actualSlot.allowedComponents = [];
+    for (const parentType of parentTypes) {
+      this.logger.info(`Loading parent component: ${parentType}`);
+      const { component: parentComponent, filePath: parentComponentFilePath } = await this.componentService.loadComponent(fullComponentsDir, parentType, findOptions);
+      if (!parentComponent.slots) {
+        parentComponent.slots = [];
+      }
+      let slotDef = parentComponent.slots.find(
+        (s) => this.compareIds(s.id, slot, strict)
+      );
+      if (!slotDef) {
+        this.logger.info(`Slot "${slot}" not found on component "${parentType}", creating it`);
+        slotDef = { id: slot, name: slot, allowedComponents: [] };
+        parentComponent.slots.push(slotDef);
       }
-      const isAlreadyAllowed = actualSlot.allowedComponents.some(
-        (c) => this.compareIds(c, newComponentType, strict)
+      const slotIndex = parentComponent.slots?.findIndex(
+        (s) => this.compareIds(s.id, slotDef.id, strict)
       );
-      if (!isAlreadyAllowed) {
-        this.logger.action(
-          whatIf,
-          "UPDATE",
-          `Adding "${newComponentType}" to allowedComponents for slot "${slot}" on component "${parentComponentType}"`
+      if (slotIndex !== void 0 && slotIndex >= 0) {
+        const actualSlot = parentComponent.slots[slotIndex];
+        if (!actualSlot.allowedComponents) {
+          actualSlot.allowedComponents = [];
+        }
+        const isAlreadyAllowed = actualSlot.allowedComponents.some(
+          (c) => this.compareIds(c, newComponentType, strict)
         );
-        actualSlot.allowedComponents.push(newComponentType);
-        if (!whatIf) {
-          await this.componentService.saveComponent(parentComponentFilePath, parentComponent);
+        if (!isAlreadyAllowed) {
+          this.logger.action(
+            whatIf,
+            "UPDATE",
+            `Adding "${newComponentType}" to allowedComponents for slot "${slot}" on component "${parentType}"`
+          );
+          actualSlot.allowedComponents.push(newComponentType);
+          if (!whatIf) {
+            await this.componentService.saveComponent(parentComponentFilePath, parentComponent);
+          }
+          allowedComponentsUpdated = true;
         }
-        allowedComponentsUpdated = true;
       }
     }
     const parsedParams = this.parseParameters(paramString);
     const newInstance = this.createComponentInstance(newComponentType, parsedParams);
     const compositionsResult = await this.addComponentToDirectory(
       fullCompositionsDir,
-      parentComponentType,
+      parentTypes,
       slot,
       newInstance,
       whatIf,
@@ -2573,7 +2569,7 @@ var ComponentAdderService = class {
     );
     const compositionPatternsResult = await this.addComponentToDirectory(
       fullCompositionPatternsDir,
-      parentComponentType,
+      parentTypes,
       slot,
       newInstance,
       whatIf,
@@ -2582,7 +2578,7 @@ var ComponentAdderService = class {
     );
     const componentPatternsResult = await this.addComponentToDirectory(
       fullComponentPatternsDir,
-      parentComponentType,
+      parentTypes,
       slot,
       newInstance,
       whatIf,
@@ -2591,7 +2587,7 @@ var ComponentAdderService = class {
     );
     this.logger.info("");
     this.logger.info(
-      `Summary: ${allowedComponentsUpdated ? "1 component definition updated, " : ""}${compositionsResult} composition(s), ${compositionPatternsResult} composition pattern(s), ${componentPatternsResult} component pattern(s) updated. ${compositionsResult + compositionPatternsResult + componentPatternsResult} instance(s) added.`
+      `Summary: ${allowedComponentsUpdated ? `${parentTypes.length} component definition(s) updated, ` : ""}${compositionsResult} composition(s), ${compositionPatternsResult} composition pattern(s), ${componentPatternsResult} component pattern(s) updated. ${compositionsResult + compositionPatternsResult + componentPatternsResult} instance(s) added.`
     );
     return {
       allowedComponentsUpdated,
@@ -2622,18 +2618,9 @@ var ComponentAdderService = class {
     const fullCompositionPatternsDir = this.fileSystem.resolvePath(rootDir, compositionPatternsDir);
     const fullComponentPatternsDir = this.fileSystem.resolvePath(rootDir, componentPatternsDir);
     const findOptions = { strict };
-    this.logger.info(`Loading parent component: ${parentComponentType}`);
-    const { component: parentComponent } = await this.componentService.loadComponent(fullComponentsDir, parentComponentType, findOptions);
-    if (!parentComponent.slots) {
-      parentComponent.slots = [];
-    }
-    let slotDef = parentComponent.slots.find(
-      (s) => this.compareIds(s.id, slot, strict)
-    );
-    if (!slotDef) {
-      this.logger.info(`Slot "${slot}" not found on component "${parentComponentType}", creating it`);
-      slotDef = { id: slot, name: slot, allowedComponents: [] };
-      parentComponent.slots.push(slotDef);
+    const parentTypes = this.parseParentComponentTypes(parentComponentType);
+    if (parentTypes.length === 0) {
+      throw new TransformError("parentComponentType cannot be empty");
     }
     this.logger.info(`Loading component pattern: ${componentPatternId}`);
     const pattern = await this.loadComponentPattern(
@@ -2647,38 +2634,55 @@ var ComponentAdderService = class {
         `Component pattern "${componentPatternId}" has no valid definition or missing type`
       );
     }
-    let allowedComponentsUpdated = false;
     const componentTypeInPattern = patternDefinition.type;
-    const slotIndex = parentComponent.slots?.findIndex(
-      (s) => this.compareIds(s.id, slotDef.id, strict)
-    );
-    if (slotIndex !== void 0 && slotIndex >= 0) {
-      const actualSlot = parentComponent.slots[slotIndex];
-      if (!actualSlot.allowedComponents) {
-        actualSlot.allowedComponents = [];
+    let allowedComponentsUpdated = false;
+    for (const parentType of parentTypes) {
+      this.logger.info(`Loading parent component: ${parentType}`);
+      const { component: parentComponent, filePath: parentComponentFilePath } = await this.componentService.loadComponent(fullComponentsDir, parentType, findOptions);
+      if (!parentComponent.slots) {
+        parentComponent.slots = [];
+      }
+      let slotDef = parentComponent.slots.find(
+        (s) => this.compareIds(s.id, slot, strict)
+      );
+      if (!slotDef) {
+        this.logger.info(`Slot "${slot}" not found on component "${parentType}", creating it`);
+        slotDef = { id: slot, name: slot, allowedComponents: [] };
+        parentComponent.slots.push(slotDef);
       }
-      const isAlreadyAllowed = actualSlot.allowedComponents.some(
-        (c) => this.compareIds(c, componentTypeInPattern, strict)
+      const slotIndex = parentComponent.slots?.findIndex(
+        (s) => this.compareIds(s.id, slotDef.id, strict)
       );
-      if (!isAlreadyAllowed) {
-        this.logger.action(
-          whatIf,
-          "UPDATE",
-          `Adding "${componentTypeInPattern}" to allowedComponents for slot "${slot}" on component "${parentComponentType}"`
+      if (slotIndex !== void 0 && slotIndex >= 0) {
+        const actualSlot = parentComponent.slots[slotIndex];
+        if (!actualSlot.allowedComponents) {
+          actualSlot.allowedComponents = [];
+        }
+        const isAlreadyAllowed = actualSlot.allowedComponents.some(
+          (c) => this.compareIds(c, componentTypeInPattern, strict)
         );
-        actualSlot.allowedComponents.push(componentTypeInPattern);
-        if (!whatIf) {
-          await this.componentService.loadComponent(fullComponentsDir, parentComponentType, findOptions).then(
-            ({ filePath }) => this.componentService.saveComponent(filePath, parentComponent)
+        if (!isAlreadyAllowed) {
+          this.logger.action(
+            whatIf,
+            "UPDATE",
+            `Adding "${componentTypeInPattern}" to allowedComponents for slot "${slot}" on component "${parentType}"`
           );
+          actualSlot.allowedComponents.push(componentTypeInPattern);
+          if (!whatIf) {
+            await this.componentService.saveComponent(parentComponentFilePath, parentComponent);
+          }
+          allowedComponentsUpdated = true;
         }
-        allowedComponentsUpdated = true;
       }
     }
-    const newInstance = this.cloneComponentInstance(patternDefinition);
+    const newInstance = {
+      type: componentTypeInPattern,
+      _pattern: pattern.componentPatternId,
+      _id: randomUUID()
+    };
     const compositionsResult = await this.addComponentToDirectory(
       fullCompositionsDir,
-      parentComponentType,
+      parentTypes,
       slot,
       newInstance,
       whatIf,
@@ -2687,7 +2691,7 @@ var ComponentAdderService = class {
     );
     const compositionPatternsResult = await this.addComponentToDirectory(
       fullCompositionPatternsDir,
-      parentComponentType,
+      parentTypes,
       slot,
       newInstance,
       whatIf,
@@ -2696,7 +2700,7 @@ var ComponentAdderService = class {
     );
     const componentPatternsResult = await this.addComponentToDirectory(
       fullComponentPatternsDir,
-      parentComponentType,
+      parentTypes,
       slot,
       newInstance,
       whatIf,
@@ -2705,7 +2709,7 @@ var ComponentAdderService = class {
     );
     this.logger.info("");
     this.logger.info(
-      `Summary: ${allowedComponentsUpdated ? "1 component definition updated, " : ""}${compositionsResult} composition(s), ${compositionPatternsResult} composition pattern(s), ${componentPatternsResult} component pattern(s) updated. ${compositionsResult + compositionPatternsResult + componentPatternsResult} instance(s) added.`
+      `Summary: ${allowedComponentsUpdated ? `${parentTypes.length} component definition(s) updated, ` : ""}${compositionsResult} composition(s), ${compositionPatternsResult} composition pattern(s), ${componentPatternsResult} component pattern(s) updated. ${compositionsResult + compositionPatternsResult + componentPatternsResult} instance(s) added.`
     );
     return {
       allowedComponentsUpdated,
@@ -2756,7 +2760,7 @@ var ComponentAdderService = class {
     }
     return raw;
   }
-  async addComponentToDirectory(directory, parentComponentType, slot, newInstance, whatIf, strict, dirType) {
+  async addComponentToDirectory(directory, parentTypes, slot, newInstance, whatIf, strict, dirType) {
     const exists = await this.fileSystem.fileExists(directory);
     if (!exists) {
       this.logger.detail(`${dirType} directory does not exist, skipping`);
@@ -2789,7 +2793,7 @@ var ComponentAdderService = class {
         }
         const rootInstance = composition.composition;
         let modified = false;
-        if (this.compareIds(rootInstance.type, parentComponentType, strict)) {
+        if (this.matchesAnyParentType(rootInstance.type, parentTypes, strict)) {
           if (!rootInstance.slots) {
             rootInstance.slots = {};
           }
@@ -2801,7 +2805,7 @@ var ComponentAdderService = class {
         if (rootInstance.slots) {
           const nestedModified = this.addComponentToNestedSlots(
             rootInstance.slots,
-            parentComponentType,
+            parentTypes,
             slot,
             newInstance,
             strict
@@ -2835,12 +2839,12 @@ var ComponentAdderService = class {
     }
     return filesModified;
   }
-  addComponentToNestedSlots(slots, parentComponentType, slot, newInstance, strict) {
+  addComponentToNestedSlots(slots, parentTypes, slot, newInstance, strict) {
     let modified = false;
     for (const slotInstances of Object.values(slots)) {
       if (!Array.isArray(slotInstances)) continue;
       for (const instance of slotInstances) {
-        if (this.compareIds(instance.type, parentComponentType, strict)) {
+        if (this.matchesAnyParentType(instance.type, parentTypes, strict)) {
           if (!instance.slots) {
             instance.slots = {};
           }
@@ -2852,7 +2856,7 @@ var ComponentAdderService = class {
         if (instance.slots) {
           const nestedModified = this.addComponentToNestedSlots(
             instance.slots,
-            parentComponentType,
+            parentTypes,
             slot,
             newInstance,
             strict
@@ -2927,7 +2931,7 @@ function createAddComponentCommand() {
 import { Command as Command8 } from "commander";
 function createAddComponentPatternCommand() {
   const command = new Command8("add-component-pattern");
-  command.description("Adds a component pattern instance to existing component slots across all compositions.").option("--parentComponentType <type>", "The component type that owns the slot").option("--slot <slotId>", "The slot ID where the component pattern will be added").option("--componentPatternId <id>", "The component pattern ID to add").option("--parameters <params>", "Pipe-separated parameter assignments (key1:value1|key2:value2)").hook("preAction", (thisCommand) => {
+  command.description("Adds a component pattern instance to existing component slots across all compositions.").option("--parentComponentType <type>", "The component type that owns the slot").option("--slot <slotId>", "The slot ID where the component pattern will be added").option("--componentPatternId <id>", "The component pattern ID to add").hook("preAction", (thisCommand) => {
     const opts = thisCommand.opts();
     const requiredOptions = [
       { name: "parentComponentType", flag: "--parentComponentType" },
@@ -2945,8 +2949,7 @@ function createAddComponentPatternCommand() {
       ...globalOpts,
       parentComponentType: opts.parentComponentType,
       slot: opts.slot,
-      componentPatternId: opts.componentPatternId,
-      parameters: opts.parameters
+      componentPatternId: opts.componentPatternId
     };
     const logger = new Logger();
     const fileSystem = new FileSystemService();
@@ -2962,7 +2965,6 @@ function createAddComponentPatternCommand() {
         parentComponentType: options.parentComponentType,
         slot: options.slot,
         componentPatternId: options.componentPatternId,
-        parameters: options.parameters,
         whatIf: options.whatIf ?? false,
         strict: options.strict ?? false
       });