pptx-viewer-core 1.1.46 → 1.1.47

package/dist/index.d.mts

@@ -2576,7 +2576,7 @@ declare function decomposeSmartArt(smartArtData: PptxSmartArtData, containerBoun
  * @module smartart-editing-node-ops
  */
 
-/** Reset the edit counter (useful in tests). */
+/** No-op kept for API compatibility; id generation no longer needs a counter. */
 declare function resetSmartArtEditCounter(): void;
 /**
  * Add a new node to a SmartArt diagram after a given sibling.
@@ -2613,7 +2613,10 @@ declare function demoteSmartArtNode(data: PptxSmartArtData, nodeId: string): Ppt
 /**
  * Add a new node as a child of a given parent.
  *
- * If `parentId` is undefined, the node is added as a root-level item.
+ * If `parentId` is undefined, the node is added as a root-level item, using
+ * the diagram's own root/doc id (inferred from an existing top-level node)
+ * so it ends up as a sibling of the other top-level items rather than an
+ * unparented node the save pipeline can't anchor into the diagram XML.
  * If `text` is undefined, a default label is generated.
  *
  * Returns a new PptxSmartArtData with the node inserted and
@@ -6737,14 +6740,6 @@ declare class PptxHandlerRuntime$V extends PptxHandlerRuntime$W {
      * carries no colour transform.
      */
     protected regenerateSmartArtColorPart(slidePath: string, smartArtData: SmartArtPptxElement['smartArtData']): Promise<void>;
-    /**
-     * Merge the in-memory quick style back into `ppt/diagrams/quickStyles*.xml`.
-     *
-     * Resolves the part via the SmartArt `styleRelId` relationship, merges
-     * surgically, and skips gracefully when the rel or part is absent. No-op
-     * when the in-memory data carries no quick style.
-     */
-    protected regenerateSmartArtQuickStylePart(slidePath: string, smartArtData: SmartArtPptxElement['smartArtData']): Promise<void>;
     /**
      * Read a SmartArt diagram part by slide relationship id, locate its root
      * definition element by local name, apply a surgical merge callback, and

package/dist/index.d.ts

@@ -2576,7 +2576,7 @@ declare function decomposeSmartArt(smartArtData: PptxSmartArtData, containerBoun
  * @module smartart-editing-node-ops
  */
 
-/** Reset the edit counter (useful in tests). */
+/** No-op kept for API compatibility; id generation no longer needs a counter. */
 declare function resetSmartArtEditCounter(): void;
 /**
  * Add a new node to a SmartArt diagram after a given sibling.
@@ -2613,7 +2613,10 @@ declare function demoteSmartArtNode(data: PptxSmartArtData, nodeId: string): Ppt
 /**
  * Add a new node as a child of a given parent.
  *
- * If `parentId` is undefined, the node is added as a root-level item.
+ * If `parentId` is undefined, the node is added as a root-level item, using
+ * the diagram's own root/doc id (inferred from an existing top-level node)
+ * so it ends up as a sibling of the other top-level items rather than an
+ * unparented node the save pipeline can't anchor into the diagram XML.
  * If `text` is undefined, a default label is generated.
  *
  * Returns a new PptxSmartArtData with the node inserted and
@@ -6737,14 +6740,6 @@ declare class PptxHandlerRuntime$V extends PptxHandlerRuntime$W {
      * carries no colour transform.
      */
     protected regenerateSmartArtColorPart(slidePath: string, smartArtData: SmartArtPptxElement['smartArtData']): Promise<void>;
-    /**
-     * Merge the in-memory quick style back into `ppt/diagrams/quickStyles*.xml`.
-     *
-     * Resolves the part via the SmartArt `styleRelId` relationship, merges
-     * surgically, and skips gracefully when the rel or part is absent. No-op
-     * when the in-memory data carries no quick style.
-     */
-    protected regenerateSmartArtQuickStylePart(slidePath: string, smartArtData: SmartArtPptxElement['smartArtData']): Promise<void>;
     /**
      * Read a SmartArt diagram part by slide relationship id, locate its root
      * definition element by local name, apply a surgical merge callback, and

package/dist/index.js

@@ -17952,12 +17952,19 @@ function resolveLayoutFromRawType(layoutType) {
 }
 
 // src/core/utils/smartart-editing-node-ops.ts
-var editNodeCounter = 0;
 function nextModelId() {
-  return `smartart-node-${Date.now()}-${++editNodeCounter}`;
+  return `{${generateFontGuid()}}`;
 }
 function resetSmartArtEditCounter() {
-  editNodeCounter = 0;
+}
+function findRootParentId(nodes) {
+  const nodeIds = new Set(nodes.map((n) => n.id));
+  for (const node of nodes) {
+    if (node.parentId && !nodeIds.has(node.parentId)) {
+      return node.parentId;
+    }
+  }
+  return void 0;
 }
 function addSmartArtNode(data, text, afterNodeId) {
   const newId = nextModelId();
@@ -17966,6 +17973,7 @@ function addSmartArtNode(data, text, afterNodeId) {
     const sibling = data.nodes.find((n) => n.id === afterNodeId);
     parentId = sibling?.parentId;
   }
+  parentId ??= findRootParentId(data.nodes);
   const newNode = {
     id: newId,
     text,
@@ -18118,19 +18126,20 @@ function demoteSmartArtNode(data, nodeId) {
   };
 }
 function addSmartArtNodeAsChild(data, parentId, text) {
+  const resolvedParentId = parentId ?? findRootParentId(data.nodes);
   const newId = nextModelId();
   const label = text ?? `Item ${data.nodes.length + 1}`;
   const newNode = {
     id: newId,
     text: label,
-    parentId
+    parentId: resolvedParentId
   };
   const nodes = [...data.nodes, newNode];
   const connections = [...data.connections ?? []];
-  if (parentId) {
-    const maxSrcOrd = connections.filter((c) => c.sourceId === parentId).reduce((max, c) => Math.max(max, c.srcOrd ?? 0), -1);
+  if (resolvedParentId) {
+    const maxSrcOrd = connections.filter((c) => c.sourceId === resolvedParentId).reduce((max, c) => Math.max(max, c.srcOrd ?? 0), -1);
     connections.push({
-      sourceId: parentId,
+      sourceId: resolvedParentId,
       destId: newId,
       type: "parOf",
       srcOrd: maxSrcOrd + 1,
@@ -33444,12 +33453,6 @@ function applySmartArtColorTransform(colorsDef, transform, getLocalName) {
     return false;
   }
   let mutated = false;
-  if (transform.name && transform.name.length > 0) {
-    if (colorsDef["@_title"] !== transform.name) {
-      colorsDef["@_title"] = transform.name;
-      mutated = true;
-    }
-  }
   const styleLblKey = findKey11(colorsDef, "styleLbl", getLocalName);
   if (!styleLblKey) {
     return mutated;
@@ -33483,54 +33486,6 @@ function applySmartArtColorTransform(colorsDef, transform, getLocalName) {
   return mutated;
 }
 
-// src/core/core/runtime/smartart-quickstyle-builder.ts
-function applySmartArtQuickStyle(styleDef, quickStyle) {
-  if (!quickStyle || !quickStyle.name || quickStyle.name.length === 0) {
-    return false;
-  }
-  if (styleDef["@_title"] === quickStyle.name) {
-    return false;
-  }
-  styleDef["@_title"] = quickStyle.name;
-  return true;
-}
-
-// src/core/core/runtime/smartart-save-chrome.ts
-function asObject2(value) {
-  return value && typeof value === "object" && !Array.isArray(value) ? value : {};
-}
-function applySmartArtChrome(dataModel, chrome, getLocalName) {
-  if (!chrome || !chrome.backgroundColor && !chrome.outlineColor && (chrome.outlineWidth === null || chrome.outlineWidth === void 0)) {
-    return;
-  }
-  const findKey13 = (obj, name) => Object.keys(obj).find((k) => getLocalName(k) === name);
-  if (chrome.backgroundColor) {
-    const hex8 = chrome.backgroundColor.replace("#", "");
-    const bgKey = findKey13(dataModel, "bg") ?? "dgm:bg";
-    const bg = asObject2(dataModel[bgKey]);
-    const fillKey = findKey13(bg, "solidFill") ?? "a:solidFill";
-    bg[fillKey] = { "a:srgbClr": { "@_val": hex8 } };
-    dataModel[bgKey] = bg;
-  }
-  const hasOutlineWidth = chrome.outlineWidth !== null && chrome.outlineWidth !== void 0;
-  if (chrome.outlineColor || hasOutlineWidth) {
-    const wholeKey = findKey13(dataModel, "whole") ?? "dgm:whole";
-    const whole = asObject2(dataModel[wholeKey]);
-    const lnKey = findKey13(whole, "ln") ?? "a:ln";
-    const ln = asObject2(whole[lnKey]);
-    if (hasOutlineWidth) {
-      ln["@_w"] = String(Math.round(chrome.outlineWidth * 12700));
-    }
-    if (chrome.outlineColor) {
-      const hex8 = chrome.outlineColor.replace("#", "");
-      const fillKey = findKey13(ln, "solidFill") ?? "a:solidFill";
-      ln[fillKey] = { "a:srgbClr": { "@_val": hex8 } };
-    }
-    whole[lnKey] = ln;
-    dataModel[wholeKey] = whole;
-  }
-}
-
 // src/core/core/runtime/smartart-style-xml.ts
 function localName(key) {
   const idx = key.indexOf(":");
@@ -33785,9 +33740,58 @@ function mergeSmartArtPointXml(existingPts, nodes) {
   }
   return merged;
 }
-function buildSmartArtConnectionXml(connections) {
+function newSmartArtGuid() {
+  return `{${generateFontGuid()}}`;
+}
+function normalizeConnType(type) {
+  return type && type !== "parOf" ? type : "";
+}
+function connectionKey(conn) {
+  return [
+    conn.srcId,
+    conn.destId,
+    normalizeConnType(conn.type),
+    conn.srcOrd ?? "",
+    conn.destOrd ?? ""
+  ].join(" ");
+}
+function mergeSmartArtConnectionXml(existingCxns, connections) {
+  const existingByKey = /* @__PURE__ */ new Map();
+  for (const cxn of existingCxns) {
+    if (!cxn || typeof cxn !== "object") {
+      continue;
+    }
+    const srcOrdRaw = parseInt(String(cxn["@_srcOrd"] ?? ""), 10);
+    const destOrdRaw = parseInt(String(cxn["@_destOrd"] ?? ""), 10);
+    const key = connectionKey({
+      srcId: String(cxn["@_srcId"] || ""),
+      destId: String(cxn["@_destId"] || ""),
+      type: cxn["@_type"] ? String(cxn["@_type"]) : void 0,
+      srcOrd: Number.isFinite(srcOrdRaw) ? srcOrdRaw : void 0,
+      destOrd: Number.isFinite(destOrdRaw) ? destOrdRaw : void 0
+    });
+    const queue = existingByKey.get(key);
+    if (queue) {
+      queue.push(cxn);
+    } else {
+      existingByKey.set(key, [cxn]);
+    }
+  }
   return connections.map((conn) => {
+    const key = connectionKey({
+      srcId: conn.sourceId,
+      destId: conn.destId,
+      type: conn.type,
+      srcOrd: conn.srcOrd,
+      destOrd: conn.destOrd
+    });
+    const queue = existingByKey.get(key);
+    const match = queue?.shift();
+    if (match) {
+      return match;
+    }
     const cxnNode = {
+      "@_modelId": newSmartArtGuid(),
       "@_srcId": conn.sourceId,
       "@_destId": conn.destId
     };
@@ -33804,6 +33808,204 @@ function buildSmartArtConnectionXml(connections) {
   });
 }
 
+// src/core/core/runtime/smartart-node-synthesis.ts
+function prSetOf(pt2) {
+  const prSet = pt2["dgm:prSet"];
+  return prSet && typeof prSet === "object" && !Array.isArray(prSet) ? prSet : {};
+}
+function findPresPointForAssoc(pts, assocId) {
+  return pts.find(
+    (pt2) => pt2["@_type"] === "pres" && String(prSetOf(pt2)["@_presAssocID"] || "") === assocId
+  );
+}
+function maxPresParOfOrd(cxns, srcId) {
+  return cxns.filter((c) => c["@_type"] === "presParOf" && c["@_srcId"] === srcId).reduce((max, c) => Math.max(max, parseInt(String(c["@_srcOrd"] ?? ""), 10) || 0), -1);
+}
+function countNodePresPoints(pts) {
+  return pts.filter((pt2) => pt2["@_type"] === "pres" && prSetOf(pt2)["@_presName"] === "node").length;
+}
+function emptyTextBody() {
+  return { "a:bodyPr": {}, "a:lstStyle": {}, "a:p": { "a:endParaRPr": { "@_lang": "en-US" } } };
+}
+function buildContentPoint(node) {
+  const pt2 = { "@_modelId": node.id };
+  pt2["dgm:prSet"] = node.text ? {} : { "@_phldrT": "[Text]" };
+  pt2["dgm:spPr"] = {};
+  pt2["dgm:t"] = shouldRebuildFromRuns(node) ? buildPointFromRuns(node.runs) : buildPointText(node.text ?? "");
+  return pt2;
+}
+function buildTransPoint(id, type) {
+  return { "@_modelId": id, "@_type": type, "dgm:prSet": {}, "dgm:t": emptyTextBody() };
+}
+function buildPresNodePoint(id, assocId, idx, cnt) {
+  return {
+    "@_modelId": id,
+    "@_type": "pres",
+    "dgm:prSet": {
+      "dgm:presLayoutVars": { "dgm:bulletEnabled": { "@_val": "1" } },
+      "@_presAssocID": assocId,
+      "@_presName": "node",
+      "@_presStyleLbl": "node1",
+      "@_presStyleIdx": String(idx),
+      "@_presStyleCnt": String(cnt)
+    },
+    "dgm:spPr": {}
+  };
+}
+function buildPresSibTransPoint(id, assocId) {
+  return {
+    "@_modelId": id,
+    "@_type": "pres",
+    "dgm:prSet": { "@_presAssocID": assocId, "@_presName": "sibTrans", "@_presStyleCnt": "0" },
+    "dgm:spPr": {}
+  };
+}
+function buildParOfConnection(srcId, destId, srcOrd, parTransId, sibTransId) {
+  return {
+    "@_modelId": newSmartArtGuid(),
+    "@_srcId": srcId,
+    "@_destId": destId,
+    "@_srcOrd": String(srcOrd),
+    "@_destOrd": "0",
+    "@_parTransId": parTransId,
+    "@_sibTransId": sibTransId
+  };
+}
+function buildPresOfConnection(srcId, destId, presId) {
+  return {
+    "@_modelId": newSmartArtGuid(),
+    "@_type": "presOf",
+    "@_srcId": srcId,
+    "@_destId": destId,
+    "@_srcOrd": "0",
+    "@_destOrd": "0",
+    "@_presId": presId
+  };
+}
+function buildPresParOfConnection(srcId, destId, srcOrd, presId) {
+  return {
+    "@_modelId": newSmartArtGuid(),
+    "@_type": "presParOf",
+    "@_srcId": srcId,
+    "@_destId": destId,
+    "@_srcOrd": String(srcOrd),
+    "@_destOrd": "0",
+    "@_presId": presId
+  };
+}
+function synthesizeNewSmartArtStructuralPoints(existingPts, existingCxns, nodes, connections) {
+  const existingContentIds = new Set(
+    existingPts.filter(isContentPoint).map((pt2) => String(pt2["@_modelId"] || ""))
+  );
+  const newNodes = nodes.filter((n) => n.id && !existingContentIds.has(n.id));
+  if (newNodes.length === 0) {
+    return { pts: existingPts, cxns: existingCxns, extraConnections: [] };
+  }
+  const presId = existingCxns.find((c) => c["@_presId"])?.["@_presId"];
+  if (!presId) {
+    return { pts: existingPts, cxns: existingCxns, extraConnections: [] };
+  }
+  const pts = [...existingPts];
+  const cxns = [...existingCxns];
+  const extraConnections = [];
+  const connectionByChildId = /* @__PURE__ */ new Map();
+  for (const conn of connections ?? []) {
+    if (!conn.type || conn.type === "parOf") {
+      connectionByChildId.set(conn.destId, conn);
+    }
+  }
+  for (const node of newNodes) {
+    const parentId = node.parentId;
+    const parentPres = parentId ? findPresPointForAssoc(pts, parentId) : void 0;
+    if (!parentId || !parentPres) {
+      continue;
+    }
+    const parTransId = newSmartArtGuid();
+    const sibTransId = newSmartArtGuid();
+    const presNodeId = newSmartArtGuid();
+    const presSibTransId = newSmartArtGuid();
+    pts.push(buildContentPoint(node));
+    pts.push(buildTransPoint(parTransId, "parTrans"));
+    pts.push(buildTransPoint(sibTransId, "sibTrans"));
+    const idx = countNodePresPoints(pts);
+    pts.push(buildPresNodePoint(presNodeId, node.id, idx, idx + 1));
+    pts.push(buildPresSibTransPoint(presSibTransId, sibTransId));
+    const total = idx + 1;
+    for (const pt2 of pts) {
+      if (pt2["@_type"] === "pres" && prSetOf(pt2)["@_presName"] === "node") {
+        prSetOf(pt2)["@_presStyleCnt"] = String(total);
+      }
+    }
+    const srcOrd = connectionByChildId.get(node.id)?.srcOrd ?? 0;
+    cxns.push(buildParOfConnection(parentId, node.id, srcOrd, parTransId, sibTransId));
+    cxns.push(buildPresOfConnection(node.id, presNodeId, presId));
+    extraConnections.push({
+      sourceId: node.id,
+      destId: presNodeId,
+      type: "presOf",
+      srcOrd: 0,
+      destOrd: 0
+    });
+    const parentPresId = String(parentPres["@_modelId"]);
+    const baseOrd = maxPresParOfOrd(cxns, parentPresId) + 1;
+    cxns.push(buildPresParOfConnection(parentPresId, presNodeId, baseOrd, presId));
+    cxns.push(buildPresParOfConnection(parentPresId, presSibTransId, baseOrd + 1, presId));
+    extraConnections.push(
+      {
+        sourceId: parentPresId,
+        destId: presNodeId,
+        type: "presParOf",
+        srcOrd: baseOrd,
+        destOrd: 0
+      },
+      {
+        sourceId: parentPresId,
+        destId: presSibTransId,
+        type: "presParOf",
+        srcOrd: baseOrd + 1,
+        destOrd: 0
+      }
+    );
+  }
+  return { pts, cxns, extraConnections };
+}
+
+// src/core/core/runtime/smartart-save-chrome.ts
+function asObject2(value) {
+  return value && typeof value === "object" && !Array.isArray(value) ? value : {};
+}
+function applySmartArtChrome(dataModel, chrome, getLocalName) {
+  if (!chrome || !chrome.backgroundColor && !chrome.outlineColor && (chrome.outlineWidth === null || chrome.outlineWidth === void 0)) {
+    return;
+  }
+  const findKey13 = (obj, name) => Object.keys(obj).find((k) => getLocalName(k) === name);
+  if (chrome.backgroundColor) {
+    const hex8 = chrome.backgroundColor.replace("#", "");
+    const bgKey = findKey13(dataModel, "bg") ?? "dgm:bg";
+    const bg = asObject2(dataModel[bgKey]);
+    const fillKey = findKey13(bg, "solidFill") ?? "a:solidFill";
+    bg[fillKey] = { "a:srgbClr": { "@_val": hex8 } };
+    dataModel[bgKey] = bg;
+  }
+  const hasOutlineWidth = chrome.outlineWidth !== null && chrome.outlineWidth !== void 0;
+  if (chrome.outlineColor || hasOutlineWidth) {
+    const wholeKey = findKey13(dataModel, "whole") ?? "dgm:whole";
+    const whole = asObject2(dataModel[wholeKey]);
+    const lnKey = findKey13(whole, "ln") ?? "a:ln";
+    const ln = asObject2(whole[lnKey]);
+    if (hasOutlineWidth) {
+      ln["@_w"] = String(Math.round(chrome.outlineWidth * 12700));
+    }
+    if (chrome.outlineColor) {
+      const hex8 = chrome.outlineColor.replace("#", "");
+      const fillKey = findKey13(ln, "solidFill") ?? "a:solidFill";
+      ln[fillKey] = { "a:srgbClr": { "@_val": hex8 } };
+    }
+    whole[lnKey] = ln;
+    dataModel[wholeKey] = whole;
+  }
+}
+
 // src/core/core/runtime/PptxHandlerRuntimeSaveDocumentParts.ts
 var PptxHandlerRuntime24 = class _PptxHandlerRuntime extends PptxHandlerRuntime23 {
   /** Pending SmartArt data updates to process during save. */
@@ -33856,29 +34058,34 @@ var PptxHandlerRuntime24 = class _PptxHandlerRuntime extends PptxHandlerRuntime2
         const ptListKey = Object.keys(dataModel).find(
           (k) => this.compatibilityService.getXmlLocalName(k) === "ptLst"
         );
-        if (ptListKey) {
-          const ptList = dataModel[ptListKey];
-          const ptKey = Object.keys(ptList || {}).find(
-            (k) => this.compatibilityService.getXmlLocalName(k) === "pt"
+        const cxnListKey = Object.keys(dataModel).find(
+          (k) => this.compatibilityService.getXmlLocalName(k) === "cxnLst"
+        );
+        const ptList = ptListKey ? dataModel[ptListKey] : void 0;
+        const cxnList = cxnListKey ? dataModel[cxnListKey] : void 0;
+        const ptKey = ptList ? Object.keys(ptList).find((k) => this.compatibilityService.getXmlLocalName(k) === "pt") : void 0;
+        const cxnKey = cxnList ? Object.keys(cxnList).find((k) => this.compatibilityService.getXmlLocalName(k) === "cxn") : void 0;
+        let existingPts = ptKey && ptList ? this.ensureArray(ptList[ptKey]) : [];
+        let existingCxns = cxnKey && cxnList ? this.ensureArray(cxnList[cxnKey]) : [];
+        let desiredConnections = smartArtData.connections;
+        if (ptKey && cxnKey) {
+          const synthesized = synthesizeNewSmartArtStructuralPoints(
+            existingPts,
+            existingCxns,
+            smartArtData.nodes,
+            smartArtData.connections
           );
-          if (ptKey) {
-            const existingPts = this.ensureArray(ptList[ptKey]);
-            ptList[ptKey] = mergeSmartArtPointXml(existingPts, smartArtData.nodes);
+          existingPts = synthesized.pts;
+          existingCxns = synthesized.cxns;
+          if (synthesized.extraConnections.length > 0) {
+            desiredConnections = [...desiredConnections ?? [], ...synthesized.extraConnections];
           }
         }
-        if (smartArtData.connections && smartArtData.connections.length > 0) {
-          const cxnListKey = Object.keys(dataModel).find(
-            (k) => this.compatibilityService.getXmlLocalName(k) === "cxnLst"
-          );
-          if (cxnListKey) {
-            const cxnList = dataModel[cxnListKey];
-            const cxnKey = Object.keys(cxnList || {}).find(
-              (k) => this.compatibilityService.getXmlLocalName(k) === "cxn"
-            );
-            if (cxnKey) {
-              cxnList[cxnKey] = buildSmartArtConnectionXml(smartArtData.connections);
-            }
-          }
+        if (ptKey && ptList) {
+          ptList[ptKey] = mergeSmartArtPointXml(existingPts, smartArtData.nodes);
+        }
+        if (cxnKey && cxnList && desiredConnections && desiredConnections.length > 0) {
+          cxnList[cxnKey] = mergeSmartArtConnectionXml(existingCxns, desiredConnections);
         }
         applySmartArtChrome(
           dataModel,
@@ -33890,7 +34097,6 @@ var PptxHandlerRuntime24 = class _PptxHandlerRuntime extends PptxHandlerRuntime2
         console.warn(`Failed to save SmartArt data at ${dataPartPath}:`, e);
       }
       await this.regenerateSmartArtColorPart(slidePath, smartArtData);
-      await this.regenerateSmartArtQuickStylePart(slidePath, smartArtData);
     }
     this.pendingSmartArtUpdates = void 0;
   }
@@ -33919,26 +34125,6 @@ var PptxHandlerRuntime24 = class _PptxHandlerRuntime extends PptxHandlerRuntime2
       )
     );
   }
-  /**
-   * Merge the in-memory quick style back into `ppt/diagrams/quickStyles*.xml`.
-   *
-   * Resolves the part via the SmartArt `styleRelId` relationship, merges
-   * surgically, and skips gracefully when the rel or part is absent. No-op
-   * when the in-memory data carries no quick style.
-   */
-  async regenerateSmartArtQuickStylePart(slidePath, smartArtData) {
-    const quickStyle = smartArtData?.quickStyle;
-    if (!smartArtData?.styleRelId || !quickStyle) {
-      return;
-    }
-    await this.mergeSmartArtDiagramPart(
-      slidePath,
-      smartArtData.styleRelId,
-      "styleDef",
-      "quick style",
-      (styleDef) => applySmartArtQuickStyle(styleDef, quickStyle)
-    );
-  }
   /**
    * Read a SmartArt diagram part by slide relationship id, locate its root
    * definition element by local name, apply a surgical merge callback, and
@@ -44099,7 +44285,7 @@ var PptxHandlerRuntime70 = class _PptxHandlerRuntime extends PptxHandlerRuntime6
     const pointList = this.xmlLookupService.getChildByLocalName(dataModel, "ptLst");
     const points2 = this.xmlLookupService.getChildrenArrayByLocalName(pointList, "pt");
     const { parsedConnections, parentByNodeId } = this.parseSmartArtConnections(dataModel);
-    const nodes = points2.map((point) => {
+    const nodes = points2.filter(isContentPoint).map((point) => {
       const pointId = String(point?.["@_modelId"] || "").trim();
       if (pointId.length === 0) {
         return null;
@@ -44108,14 +44294,11 @@ var PptxHandlerRuntime70 = class _PptxHandlerRuntime extends PptxHandlerRuntime6
       const textValues = [];
       this.collectLocalTextValues(point, "t", textValues);
       const resolvedText = textValues.find((entry) => entry.trim().length > 0);
-      if (!resolvedText) {
-        return null;
-      }
       const runs = this.extractSmartArtNodeRuns(point);
       const style = this.extractSmartArtNodeStyle(point);
       return {
         id: pointId,
-        text: resolvedText.trim(),
+        text: resolvedText ? resolvedText.trim() : "",
         parentId: parentByNodeId.get(pointId),
         nodeType,
         runs,

package/dist/index.mjs

@@ -17947,12 +17947,19 @@ function resolveLayoutFromRawType(layoutType) {
 }
 
 // src/core/utils/smartart-editing-node-ops.ts
-var editNodeCounter = 0;
 function nextModelId() {
-  return `smartart-node-${Date.now()}-${++editNodeCounter}`;
+  return `{${generateFontGuid()}}`;
 }
 function resetSmartArtEditCounter() {
-  editNodeCounter = 0;
+}
+function findRootParentId(nodes) {
+  const nodeIds = new Set(nodes.map((n) => n.id));
+  for (const node of nodes) {
+    if (node.parentId && !nodeIds.has(node.parentId)) {
+      return node.parentId;
+    }
+  }
+  return void 0;
 }
 function addSmartArtNode(data, text, afterNodeId) {
   const newId = nextModelId();
@@ -17961,6 +17968,7 @@ function addSmartArtNode(data, text, afterNodeId) {
     const sibling = data.nodes.find((n) => n.id === afterNodeId);
     parentId = sibling?.parentId;
   }
+  parentId ??= findRootParentId(data.nodes);
   const newNode = {
     id: newId,
     text,
@@ -18113,19 +18121,20 @@ function demoteSmartArtNode(data, nodeId) {
   };
 }
 function addSmartArtNodeAsChild(data, parentId, text) {
+  const resolvedParentId = parentId ?? findRootParentId(data.nodes);
   const newId = nextModelId();
   const label = text ?? `Item ${data.nodes.length + 1}`;
   const newNode = {
     id: newId,
     text: label,
-    parentId
+    parentId: resolvedParentId
   };
   const nodes = [...data.nodes, newNode];
   const connections = [...data.connections ?? []];
-  if (parentId) {
-    const maxSrcOrd = connections.filter((c) => c.sourceId === parentId).reduce((max, c) => Math.max(max, c.srcOrd ?? 0), -1);
+  if (resolvedParentId) {
+    const maxSrcOrd = connections.filter((c) => c.sourceId === resolvedParentId).reduce((max, c) => Math.max(max, c.srcOrd ?? 0), -1);
     connections.push({
-      sourceId: parentId,
+      sourceId: resolvedParentId,
       destId: newId,
       type: "parOf",
       srcOrd: maxSrcOrd + 1,
@@ -33439,12 +33448,6 @@ function applySmartArtColorTransform(colorsDef, transform, getLocalName) {
     return false;
   }
   let mutated = false;
-  if (transform.name && transform.name.length > 0) {
-    if (colorsDef["@_title"] !== transform.name) {
-      colorsDef["@_title"] = transform.name;
-      mutated = true;
-    }
-  }
   const styleLblKey = findKey11(colorsDef, "styleLbl", getLocalName);
   if (!styleLblKey) {
     return mutated;
@@ -33478,54 +33481,6 @@ function applySmartArtColorTransform(colorsDef, transform, getLocalName) {
   return mutated;
 }
 
-// src/core/core/runtime/smartart-quickstyle-builder.ts
-function applySmartArtQuickStyle(styleDef, quickStyle) {
-  if (!quickStyle || !quickStyle.name || quickStyle.name.length === 0) {
-    return false;
-  }
-  if (styleDef["@_title"] === quickStyle.name) {
-    return false;
-  }
-  styleDef["@_title"] = quickStyle.name;
-  return true;
-}
-
-// src/core/core/runtime/smartart-save-chrome.ts
-function asObject2(value) {
-  return value && typeof value === "object" && !Array.isArray(value) ? value : {};
-}
-function applySmartArtChrome(dataModel, chrome, getLocalName) {
-  if (!chrome || !chrome.backgroundColor && !chrome.outlineColor && (chrome.outlineWidth === null || chrome.outlineWidth === void 0)) {
-    return;
-  }
-  const findKey13 = (obj, name) => Object.keys(obj).find((k) => getLocalName(k) === name);
-  if (chrome.backgroundColor) {
-    const hex8 = chrome.backgroundColor.replace("#", "");
-    const bgKey = findKey13(dataModel, "bg") ?? "dgm:bg";
-    const bg = asObject2(dataModel[bgKey]);
-    const fillKey = findKey13(bg, "solidFill") ?? "a:solidFill";
-    bg[fillKey] = { "a:srgbClr": { "@_val": hex8 } };
-    dataModel[bgKey] = bg;
-  }
-  const hasOutlineWidth = chrome.outlineWidth !== null && chrome.outlineWidth !== void 0;
-  if (chrome.outlineColor || hasOutlineWidth) {
-    const wholeKey = findKey13(dataModel, "whole") ?? "dgm:whole";
-    const whole = asObject2(dataModel[wholeKey]);
-    const lnKey = findKey13(whole, "ln") ?? "a:ln";
-    const ln = asObject2(whole[lnKey]);
-    if (hasOutlineWidth) {
-      ln["@_w"] = String(Math.round(chrome.outlineWidth * 12700));
-    }
-    if (chrome.outlineColor) {
-      const hex8 = chrome.outlineColor.replace("#", "");
-      const fillKey = findKey13(ln, "solidFill") ?? "a:solidFill";
-      ln[fillKey] = { "a:srgbClr": { "@_val": hex8 } };
-    }
-    whole[lnKey] = ln;
-    dataModel[wholeKey] = whole;
-  }
-}
-
 // src/core/core/runtime/smartart-style-xml.ts
 function localName(key) {
   const idx = key.indexOf(":");
@@ -33780,9 +33735,58 @@ function mergeSmartArtPointXml(existingPts, nodes) {
   }
   return merged;
 }
-function buildSmartArtConnectionXml(connections) {
+function newSmartArtGuid() {
+  return `{${generateFontGuid()}}`;
+}
+function normalizeConnType(type) {
+  return type && type !== "parOf" ? type : "";
+}
+function connectionKey(conn) {
+  return [
+    conn.srcId,
+    conn.destId,
+    normalizeConnType(conn.type),
+    conn.srcOrd ?? "",
+    conn.destOrd ?? ""
+  ].join(" ");
+}
+function mergeSmartArtConnectionXml(existingCxns, connections) {
+  const existingByKey = /* @__PURE__ */ new Map();
+  for (const cxn of existingCxns) {
+    if (!cxn || typeof cxn !== "object") {
+      continue;
+    }
+    const srcOrdRaw = parseInt(String(cxn["@_srcOrd"] ?? ""), 10);
+    const destOrdRaw = parseInt(String(cxn["@_destOrd"] ?? ""), 10);
+    const key = connectionKey({
+      srcId: String(cxn["@_srcId"] || ""),
+      destId: String(cxn["@_destId"] || ""),
+      type: cxn["@_type"] ? String(cxn["@_type"]) : void 0,
+      srcOrd: Number.isFinite(srcOrdRaw) ? srcOrdRaw : void 0,
+      destOrd: Number.isFinite(destOrdRaw) ? destOrdRaw : void 0
+    });
+    const queue = existingByKey.get(key);
+    if (queue) {
+      queue.push(cxn);
+    } else {
+      existingByKey.set(key, [cxn]);
+    }
+  }
   return connections.map((conn) => {
+    const key = connectionKey({
+      srcId: conn.sourceId,
+      destId: conn.destId,
+      type: conn.type,
+      srcOrd: conn.srcOrd,
+      destOrd: conn.destOrd
+    });
+    const queue = existingByKey.get(key);
+    const match = queue?.shift();
+    if (match) {
+      return match;
+    }
     const cxnNode = {
+      "@_modelId": newSmartArtGuid(),
       "@_srcId": conn.sourceId,
       "@_destId": conn.destId
     };
@@ -33799,6 +33803,204 @@ function buildSmartArtConnectionXml(connections) {
   });
 }
 
+// src/core/core/runtime/smartart-node-synthesis.ts
+function prSetOf(pt2) {
+  const prSet = pt2["dgm:prSet"];
+  return prSet && typeof prSet === "object" && !Array.isArray(prSet) ? prSet : {};
+}
+function findPresPointForAssoc(pts, assocId) {
+  return pts.find(
+    (pt2) => pt2["@_type"] === "pres" && String(prSetOf(pt2)["@_presAssocID"] || "") === assocId
+  );
+}
+function maxPresParOfOrd(cxns, srcId) {
+  return cxns.filter((c) => c["@_type"] === "presParOf" && c["@_srcId"] === srcId).reduce((max, c) => Math.max(max, parseInt(String(c["@_srcOrd"] ?? ""), 10) || 0), -1);
+}
+function countNodePresPoints(pts) {
+  return pts.filter((pt2) => pt2["@_type"] === "pres" && prSetOf(pt2)["@_presName"] === "node").length;
+}
+function emptyTextBody() {
+  return { "a:bodyPr": {}, "a:lstStyle": {}, "a:p": { "a:endParaRPr": { "@_lang": "en-US" } } };
+}
+function buildContentPoint(node) {
+  const pt2 = { "@_modelId": node.id };
+  pt2["dgm:prSet"] = node.text ? {} : { "@_phldrT": "[Text]" };
+  pt2["dgm:spPr"] = {};
+  pt2["dgm:t"] = shouldRebuildFromRuns(node) ? buildPointFromRuns(node.runs) : buildPointText(node.text ?? "");
+  return pt2;
+}
+function buildTransPoint(id, type) {
+  return { "@_modelId": id, "@_type": type, "dgm:prSet": {}, "dgm:t": emptyTextBody() };
+}
+function buildPresNodePoint(id, assocId, idx, cnt) {
+  return {
+    "@_modelId": id,
+    "@_type": "pres",
+    "dgm:prSet": {
+      "dgm:presLayoutVars": { "dgm:bulletEnabled": { "@_val": "1" } },
+      "@_presAssocID": assocId,
+      "@_presName": "node",
+      "@_presStyleLbl": "node1",
+      "@_presStyleIdx": String(idx),
+      "@_presStyleCnt": String(cnt)
+    },
+    "dgm:spPr": {}
+  };
+}
+function buildPresSibTransPoint(id, assocId) {
+  return {
+    "@_modelId": id,
+    "@_type": "pres",
+    "dgm:prSet": { "@_presAssocID": assocId, "@_presName": "sibTrans", "@_presStyleCnt": "0" },
+    "dgm:spPr": {}
+  };
+}
+function buildParOfConnection(srcId, destId, srcOrd, parTransId, sibTransId) {
+  return {
+    "@_modelId": newSmartArtGuid(),
+    "@_srcId": srcId,
+    "@_destId": destId,
+    "@_srcOrd": String(srcOrd),
+    "@_destOrd": "0",
+    "@_parTransId": parTransId,
+    "@_sibTransId": sibTransId
+  };
+}
+function buildPresOfConnection(srcId, destId, presId) {
+  return {
+    "@_modelId": newSmartArtGuid(),
+    "@_type": "presOf",
+    "@_srcId": srcId,
+    "@_destId": destId,
+    "@_srcOrd": "0",
+    "@_destOrd": "0",
+    "@_presId": presId
+  };
+}
+function buildPresParOfConnection(srcId, destId, srcOrd, presId) {
+  return {
+    "@_modelId": newSmartArtGuid(),
+    "@_type": "presParOf",
+    "@_srcId": srcId,
+    "@_destId": destId,
+    "@_srcOrd": String(srcOrd),
+    "@_destOrd": "0",
+    "@_presId": presId
+  };
+}
+function synthesizeNewSmartArtStructuralPoints(existingPts, existingCxns, nodes, connections) {
+  const existingContentIds = new Set(
+    existingPts.filter(isContentPoint).map((pt2) => String(pt2["@_modelId"] || ""))
+  );
+  const newNodes = nodes.filter((n) => n.id && !existingContentIds.has(n.id));
+  if (newNodes.length === 0) {
+    return { pts: existingPts, cxns: existingCxns, extraConnections: [] };
+  }
+  const presId = existingCxns.find((c) => c["@_presId"])?.["@_presId"];
+  if (!presId) {
+    return { pts: existingPts, cxns: existingCxns, extraConnections: [] };
+  }
+  const pts = [...existingPts];
+  const cxns = [...existingCxns];
+  const extraConnections = [];
+  const connectionByChildId = /* @__PURE__ */ new Map();
+  for (const conn of connections ?? []) {
+    if (!conn.type || conn.type === "parOf") {
+      connectionByChildId.set(conn.destId, conn);
+    }
+  }
+  for (const node of newNodes) {
+    const parentId = node.parentId;
+    const parentPres = parentId ? findPresPointForAssoc(pts, parentId) : void 0;
+    if (!parentId || !parentPres) {
+      continue;
+    }
+    const parTransId = newSmartArtGuid();
+    const sibTransId = newSmartArtGuid();
+    const presNodeId = newSmartArtGuid();
+    const presSibTransId = newSmartArtGuid();
+    pts.push(buildContentPoint(node));
+    pts.push(buildTransPoint(parTransId, "parTrans"));
+    pts.push(buildTransPoint(sibTransId, "sibTrans"));
+    const idx = countNodePresPoints(pts);
+    pts.push(buildPresNodePoint(presNodeId, node.id, idx, idx + 1));
+    pts.push(buildPresSibTransPoint(presSibTransId, sibTransId));
+    const total = idx + 1;
+    for (const pt2 of pts) {
+      if (pt2["@_type"] === "pres" && prSetOf(pt2)["@_presName"] === "node") {
+        prSetOf(pt2)["@_presStyleCnt"] = String(total);
+      }
+    }
+    const srcOrd = connectionByChildId.get(node.id)?.srcOrd ?? 0;
+    cxns.push(buildParOfConnection(parentId, node.id, srcOrd, parTransId, sibTransId));
+    cxns.push(buildPresOfConnection(node.id, presNodeId, presId));
+    extraConnections.push({
+      sourceId: node.id,
+      destId: presNodeId,
+      type: "presOf",
+      srcOrd: 0,
+      destOrd: 0
+    });
+    const parentPresId = String(parentPres["@_modelId"]);
+    const baseOrd = maxPresParOfOrd(cxns, parentPresId) + 1;
+    cxns.push(buildPresParOfConnection(parentPresId, presNodeId, baseOrd, presId));
+    cxns.push(buildPresParOfConnection(parentPresId, presSibTransId, baseOrd + 1, presId));
+    extraConnections.push(
+      {
+        sourceId: parentPresId,
+        destId: presNodeId,
+        type: "presParOf",
+        srcOrd: baseOrd,
+        destOrd: 0
+      },
+      {
+        sourceId: parentPresId,
+        destId: presSibTransId,
+        type: "presParOf",
+        srcOrd: baseOrd + 1,
+        destOrd: 0
+      }
+    );
+  }
+  return { pts, cxns, extraConnections };
+}
+
+// src/core/core/runtime/smartart-save-chrome.ts
+function asObject2(value) {
+  return value && typeof value === "object" && !Array.isArray(value) ? value : {};
+}
+function applySmartArtChrome(dataModel, chrome, getLocalName) {
+  if (!chrome || !chrome.backgroundColor && !chrome.outlineColor && (chrome.outlineWidth === null || chrome.outlineWidth === void 0)) {
+    return;
+  }
+  const findKey13 = (obj, name) => Object.keys(obj).find((k) => getLocalName(k) === name);
+  if (chrome.backgroundColor) {
+    const hex8 = chrome.backgroundColor.replace("#", "");
+    const bgKey = findKey13(dataModel, "bg") ?? "dgm:bg";
+    const bg = asObject2(dataModel[bgKey]);
+    const fillKey = findKey13(bg, "solidFill") ?? "a:solidFill";
+    bg[fillKey] = { "a:srgbClr": { "@_val": hex8 } };
+    dataModel[bgKey] = bg;
+  }
+  const hasOutlineWidth = chrome.outlineWidth !== null && chrome.outlineWidth !== void 0;
+  if (chrome.outlineColor || hasOutlineWidth) {
+    const wholeKey = findKey13(dataModel, "whole") ?? "dgm:whole";
+    const whole = asObject2(dataModel[wholeKey]);
+    const lnKey = findKey13(whole, "ln") ?? "a:ln";
+    const ln = asObject2(whole[lnKey]);
+    if (hasOutlineWidth) {
+      ln["@_w"] = String(Math.round(chrome.outlineWidth * 12700));
+    }
+    if (chrome.outlineColor) {
+      const hex8 = chrome.outlineColor.replace("#", "");
+      const fillKey = findKey13(ln, "solidFill") ?? "a:solidFill";
+      ln[fillKey] = { "a:srgbClr": { "@_val": hex8 } };
+    }
+    whole[lnKey] = ln;
+    dataModel[wholeKey] = whole;
+  }
+}
+
 // src/core/core/runtime/PptxHandlerRuntimeSaveDocumentParts.ts
 var PptxHandlerRuntime24 = class _PptxHandlerRuntime extends PptxHandlerRuntime23 {
   /** Pending SmartArt data updates to process during save. */
@@ -33851,29 +34053,34 @@ var PptxHandlerRuntime24 = class _PptxHandlerRuntime extends PptxHandlerRuntime2
         const ptListKey = Object.keys(dataModel).find(
           (k) => this.compatibilityService.getXmlLocalName(k) === "ptLst"
         );
-        if (ptListKey) {
-          const ptList = dataModel[ptListKey];
-          const ptKey = Object.keys(ptList || {}).find(
-            (k) => this.compatibilityService.getXmlLocalName(k) === "pt"
+        const cxnListKey = Object.keys(dataModel).find(
+          (k) => this.compatibilityService.getXmlLocalName(k) === "cxnLst"
+        );
+        const ptList = ptListKey ? dataModel[ptListKey] : void 0;
+        const cxnList = cxnListKey ? dataModel[cxnListKey] : void 0;
+        const ptKey = ptList ? Object.keys(ptList).find((k) => this.compatibilityService.getXmlLocalName(k) === "pt") : void 0;
+        const cxnKey = cxnList ? Object.keys(cxnList).find((k) => this.compatibilityService.getXmlLocalName(k) === "cxn") : void 0;
+        let existingPts = ptKey && ptList ? this.ensureArray(ptList[ptKey]) : [];
+        let existingCxns = cxnKey && cxnList ? this.ensureArray(cxnList[cxnKey]) : [];
+        let desiredConnections = smartArtData.connections;
+        if (ptKey && cxnKey) {
+          const synthesized = synthesizeNewSmartArtStructuralPoints(
+            existingPts,
+            existingCxns,
+            smartArtData.nodes,
+            smartArtData.connections
           );
-          if (ptKey) {
-            const existingPts = this.ensureArray(ptList[ptKey]);
-            ptList[ptKey] = mergeSmartArtPointXml(existingPts, smartArtData.nodes);
+          existingPts = synthesized.pts;
+          existingCxns = synthesized.cxns;
+          if (synthesized.extraConnections.length > 0) {
+            desiredConnections = [...desiredConnections ?? [], ...synthesized.extraConnections];
           }
         }
-        if (smartArtData.connections && smartArtData.connections.length > 0) {
-          const cxnListKey = Object.keys(dataModel).find(
-            (k) => this.compatibilityService.getXmlLocalName(k) === "cxnLst"
-          );
-          if (cxnListKey) {
-            const cxnList = dataModel[cxnListKey];
-            const cxnKey = Object.keys(cxnList || {}).find(
-              (k) => this.compatibilityService.getXmlLocalName(k) === "cxn"
-            );
-            if (cxnKey) {
-              cxnList[cxnKey] = buildSmartArtConnectionXml(smartArtData.connections);
-            }
-          }
+        if (ptKey && ptList) {
+          ptList[ptKey] = mergeSmartArtPointXml(existingPts, smartArtData.nodes);
+        }
+        if (cxnKey && cxnList && desiredConnections && desiredConnections.length > 0) {
+          cxnList[cxnKey] = mergeSmartArtConnectionXml(existingCxns, desiredConnections);
         }
         applySmartArtChrome(
           dataModel,
@@ -33885,7 +34092,6 @@ var PptxHandlerRuntime24 = class _PptxHandlerRuntime extends PptxHandlerRuntime2
         console.warn(`Failed to save SmartArt data at ${dataPartPath}:`, e);
       }
       await this.regenerateSmartArtColorPart(slidePath, smartArtData);
-      await this.regenerateSmartArtQuickStylePart(slidePath, smartArtData);
     }
     this.pendingSmartArtUpdates = void 0;
   }
@@ -33914,26 +34120,6 @@ var PptxHandlerRuntime24 = class _PptxHandlerRuntime extends PptxHandlerRuntime2
       )
     );
   }
-  /**
-   * Merge the in-memory quick style back into `ppt/diagrams/quickStyles*.xml`.
-   *
-   * Resolves the part via the SmartArt `styleRelId` relationship, merges
-   * surgically, and skips gracefully when the rel or part is absent. No-op
-   * when the in-memory data carries no quick style.
-   */
-  async regenerateSmartArtQuickStylePart(slidePath, smartArtData) {
-    const quickStyle = smartArtData?.quickStyle;
-    if (!smartArtData?.styleRelId || !quickStyle) {
-      return;
-    }
-    await this.mergeSmartArtDiagramPart(
-      slidePath,
-      smartArtData.styleRelId,
-      "styleDef",
-      "quick style",
-      (styleDef) => applySmartArtQuickStyle(styleDef, quickStyle)
-    );
-  }
   /**
    * Read a SmartArt diagram part by slide relationship id, locate its root
    * definition element by local name, apply a surgical merge callback, and
@@ -44094,7 +44280,7 @@ var PptxHandlerRuntime70 = class _PptxHandlerRuntime extends PptxHandlerRuntime6
     const pointList = this.xmlLookupService.getChildByLocalName(dataModel, "ptLst");
     const points2 = this.xmlLookupService.getChildrenArrayByLocalName(pointList, "pt");
     const { parsedConnections, parentByNodeId } = this.parseSmartArtConnections(dataModel);
-    const nodes = points2.map((point) => {
+    const nodes = points2.filter(isContentPoint).map((point) => {
       const pointId = String(point?.["@_modelId"] || "").trim();
       if (pointId.length === 0) {
         return null;
@@ -44103,14 +44289,11 @@ var PptxHandlerRuntime70 = class _PptxHandlerRuntime extends PptxHandlerRuntime6
       const textValues = [];
       this.collectLocalTextValues(point, "t", textValues);
       const resolvedText = textValues.find((entry) => entry.trim().length > 0);
-      if (!resolvedText) {
-        return null;
-      }
       const runs = this.extractSmartArtNodeRuns(point);
       const style = this.extractSmartArtNodeStyle(point);
       return {
         id: pointId,
-        text: resolvedText.trim(),
+        text: resolvedText ? resolvedText.trim() : "",
         parentId: parentByNodeId.get(pointId),
         nodeType,
         runs,

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pptx-viewer-core",
-	"version": "1.1.46",
+	"version": "1.1.47",
 	"description": "PowerPoint PPTX engine: parse, edit, serialize, and convert .pptx files. Framework-agnostic TypeScript SDK.",
 	"keywords": [
 		"converter",