@uniformdev/transformer 1.1.19 → 1.1.21

package/dist/index.d.ts

@@ -198,6 +198,15 @@ declare class ComponentService {
         filePath: string;
     }>;
     saveComponent(filePath: string, component: ComponentDefinition): Promise<void>;
+    private sortComponentArrays;
+    sortParametersByGroup<T extends {
+        id: string;
+        type: string;
+        typeConfig?: {
+            childrenParams?: string[];
+            [key: string]: unknown;
+        };
+    }>(parameters: T[]): T[];
     findParameter(component: ComponentDefinition, parameterName: string, options?: FindOptions$1): Parameter | undefined;
     isGroupParameter(parameter: Parameter): boolean;
     resolveGroupParameters(component: ComponentDefinition, groupParameter: Parameter, options?: FindOptions$1): Parameter[];
@@ -488,7 +497,7 @@ declare function computeGuidHash(guidOrSeed: string): string;
 
 /**
  * Recursively walks through a value and regenerates all `_id` properties that contain UUIDs.
- * Uses computeGuidHash(originalValue + jsonPath) to produce deterministic new IDs.
+ * Uses computeGuidHash(jsonPath) to produce deterministic new IDs based purely on position.
  * Also creates a deep clone of the value — the original is not modified.
  */
 declare function regenerateIds<T>(value: T, basePath: string): T;

package/dist/index.js

@@ -179,7 +179,59 @@ var ComponentService = class {
     throw new ComponentNotFoundError(componentType, componentsDir);
   }
   async saveComponent(filePath, component) {
-    await this.fileSystem.writeFile(filePath, component);
+    const sorted = this.sortComponentArrays(component);
+    await this.fileSystem.writeFile(filePath, sorted);
+  }
+  sortComponentArrays(component) {
+    const result = { ...component };
+    if (result.parameters) {
+      result.parameters = this.sortParametersByGroup(result.parameters);
+    }
+    if (result.slots) {
+      result.slots = [...result.slots].sort((a, b) => a.id.localeCompare(b.id, void 0, { sensitivity: "base" })).map((slot) => {
+        if (slot.allowedComponents) {
+          return { ...slot, allowedComponents: [...slot.allowedComponents].sort((a, b) => a.localeCompare(b, void 0, { sensitivity: "base" })) };
+        }
+        return slot;
+      });
+    }
+    const variants = result.variants;
+    if (Array.isArray(variants)) {
+      result.variants = [...variants].sort((a, b) => a.id.localeCompare(b.id, void 0, { sensitivity: "base" }));
+    }
+    return result;
+  }
+  sortParametersByGroup(parameters) {
+    const compare = (a, b) => a.localeCompare(b, void 0, { sensitivity: "base" });
+    const groups = [];
+    const childToGroup = /* @__PURE__ */ new Set();
+    for (const param of parameters) {
+      if (param.type === "group") {
+        groups.push(param);
+        if (param.typeConfig?.childrenParams) {
+          param.typeConfig.childrenParams = [...param.typeConfig.childrenParams].sort(compare);
+        }
+        for (const childId of param.typeConfig?.childrenParams ?? []) {
+          childToGroup.add(childId.toLowerCase());
+        }
+      }
+    }
+    const ungrouped = parameters.filter(
+      (p) => p.type !== "group" && !childToGroup.has(p.id.toLowerCase())
+    );
+    const sortedGroups = [...groups].sort((a, b) => compare(a.id, b.id));
+    const result = [];
+    result.push(...[...ungrouped].sort((a, b) => compare(a.id, b.id)));
+    for (const group of sortedGroups) {
+      result.push(group);
+      for (const childId of group.typeConfig?.childrenParams ?? []) {
+        const child = parameters.find((p) => p.id.toLowerCase() === childId.toLowerCase());
+        if (child) {
+          result.push(child);
+        }
+      }
+    }
+    return result;
   }
   findParameter(component, parameterName, options = {}) {
     const { strict = false } = options;
@@ -726,6 +778,12 @@ var CompositionConverterService = class {
         }
       }
     }
+    for (const contentType of contentTypeMap.values()) {
+      contentType.fields = this.componentService.sortParametersByGroup(contentType.fields);
+    }
+    for (const contentType of flattenContentTypeMap.values()) {
+      contentType.fields = this.componentService.sortParametersByGroup(contentType.fields);
+    }
     for (const [typeName, contentType] of contentTypeMap) {
       const filePath = this.fileSystem.joinPath(contentTypesDirFull, `${typeName}.json`);
       const fieldCount = contentType.fields.filter((f) => f.type !== "contentReference").length;
@@ -1011,7 +1069,7 @@ function walkAndRegenerate(value, currentPath) {
     for (const [key, val] of Object.entries(obj)) {
       const childPath = `${currentPath}.${key}`;
       if (key === "_id" && typeof val === "string" && UUID_PATTERN.test(val)) {
-        result[key] = computeGuidHash(val + childPath);
+        result[key] = computeGuidHash(childPath);
       } else {
         result[key] = walkAndRegenerate(val, childPath);
       }