pptx-viewer-core 1.2.2 → 1.2.3

package/CHANGELOG.md

@@ -4,6 +4,8 @@ All notable changes to this project are documented here.
 This file is generated from [Conventional Commits](https://www.conventionalcommits.org)
 by [git-cliff](https://git-cliff.org); do not edit it by hand.
 
+## [1.2.2](https://github.com/ChristopherVR/pptx-viewer/releases/tag/pptx-viewer-core@1.2.2) - 2026-07-07
+
 ## [1.2.1](https://github.com/ChristopherVR/pptx-viewer/releases/tag/pptx-viewer-core@1.2.1) - 2026-07-06
 
 ## [1.2.0](https://github.com/ChristopherVR/pptx-viewer/releases/tag/pptx-viewer-core@1.2.0) - 2026-07-05

package/README.md

@@ -20,7 +20,7 @@ There is no UI here: this is the engine on its own. Use it directly when you nee
 npm install pptx-viewer-core
 ```
 
-> That's it, nothing else to install. **jszip** (a `.pptx` is a ZIP file, and this reads and writes that container) and **fast-xml-parser** (the slides inside are XML, and this reads and writes it) are regular dependencies of `pptx-viewer-core`, so your package manager pulls them in automatically. Password protection and digital signatures need a few extra packages (`node-forge`, `xml-crypto`, `@xmldom/xmldom`), but only if you actually use those features.
+> That's it, nothing else to install. All runtime dependencies are pulled in automatically by your package manager. Password protection and digital signatures need a few extra packages (`node-forge`, `xml-crypto`, `@xmldom/xmldom`), but only if you actually use those features.
 
 ## What it does
 

package/dist/index.d.mts

@@ -2594,7 +2594,11 @@ declare function addSmartArtNode(data: PptxSmartArtData, text: string, afterNode
 declare function removeSmartArtNode(data: PptxSmartArtData, nodeId: string): PptxSmartArtData;
 /**
  * Update the text of a SmartArt node by ID.
- * Clears drawing shapes to trigger layout reflow.
+ *
+ * When drawing shapes exist, updates the matching shape's text in-place so the
+ * renderer path does not change (avoids polygon-to-chevron downgrade that makes
+ * a pyramid look like stacked bars). When no drawing shapes are present the
+ * family renderer reads text directly from the node array.
  */
 declare function updateSmartArtNodeText(data: PptxSmartArtData, nodeId: string, newText: string): PptxSmartArtData;
 /**

package/dist/index.d.ts

@@ -2594,7 +2594,11 @@ declare function addSmartArtNode(data: PptxSmartArtData, text: string, afterNode
 declare function removeSmartArtNode(data: PptxSmartArtData, nodeId: string): PptxSmartArtData;
 /**
  * Update the text of a SmartArt node by ID.
- * Clears drawing shapes to trigger layout reflow.
+ *
+ * When drawing shapes exist, updates the matching shape's text in-place so the
+ * renderer path does not change (avoids polygon-to-chevron downgrade that makes
+ * a pyramid look like stacked bars). When no drawing shapes are present the
+ * family renderer reads text directly from the node array.
  */
 declare function updateSmartArtNodeText(data: PptxSmartArtData, nodeId: string, newText: string): PptxSmartArtData;
 /**

package/dist/index.js

@@ -17952,6 +17952,9 @@ function resolveLayoutFromRawType(layoutType) {
 }
 
 // src/core/utils/smartart-editing-node-ops.ts
+function markShapesStale(shapes) {
+  return shapes !== void 0 && shapes.length > 0 ? [] : void 0;
+}
 function nextModelId() {
   return `{${generateFontGuid()}}`;
 }
@@ -18005,8 +18008,7 @@ function addSmartArtNode(data, text, afterNodeId) {
     ...data,
     nodes,
     connections: connections.length > 0 ? connections : void 0,
-    // Clear pre-computed shapes to force layout reflow
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 function removeSmartArtNode(data, nodeId) {
@@ -18035,7 +18037,7 @@ function removeSmartArtNode(data, nodeId) {
     ...data,
     nodes,
     connections: connections.length > 0 ? connections : void 0,
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 function updateSmartArtNodeText(data, nodeId, newText) {
@@ -18043,9 +18045,48 @@ function updateSmartArtNodeText(data, nodeId, newText) {
   return {
     ...data,
     nodes,
-    drawingShapes: void 0
+    drawingShapes: patchDrawingShapeText(data.drawingShapes, data.nodes, nodeId, newText)
   };
 }
+function patchDrawingShapeText(shapes, nodes, nodeId, newText) {
+  if (!shapes || shapes.length === 0) {
+    return shapes;
+  }
+  const idx = findDrawingShapeIndex(shapes, nodes, nodeId);
+  if (idx < 0) {
+    return [];
+  }
+  const updated = [...shapes];
+  updated[idx] = { ...updated[idx], text: newText };
+  return updated;
+}
+function findDrawingShapeIndex(shapes, nodes, nodeId) {
+  for (let i = 0; i < shapes.length; i++) {
+    if (shapes[i].id.startsWith("reflow-") && shapes[i].id.endsWith(`-${nodeId}`)) {
+      return i;
+    }
+  }
+  if (shapes.length === nodes.length) {
+    const nodeIdx = nodes.findIndex((n) => n.id === nodeId);
+    if (nodeIdx >= 0) {
+      return nodeIdx;
+    }
+  }
+  const node = nodes.find((n) => n.id === nodeId);
+  if (node?.text) {
+    const text = node.text.trim();
+    const matches = [];
+    for (let i = 0; i < shapes.length; i++) {
+      if (shapes[i].text?.trim() === text) {
+        matches.push(i);
+      }
+    }
+    if (matches.length === 1) {
+      return matches[0];
+    }
+  }
+  return -1;
+}
 function reorderSmartArtNode(data, nodeId, direction) {
   const node = data.nodes.find((n) => n.id === nodeId);
   if (!node) {
@@ -18068,7 +18109,7 @@ function reorderSmartArtNode(data, nodeId, direction) {
   return {
     ...data,
     nodes,
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 function promoteSmartArtNode(data, nodeId) {
@@ -18095,7 +18136,7 @@ function promoteSmartArtNode(data, nodeId) {
     ...data,
     nodes,
     connections: connections.length > 0 ? connections : void 0,
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 function demoteSmartArtNode(data, nodeId) {
@@ -18122,7 +18163,7 @@ function demoteSmartArtNode(data, nodeId) {
     ...data,
     nodes,
     connections: connections.length > 0 ? connections : void 0,
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 function addSmartArtNodeAsChild(data, parentId, text) {
@@ -18150,7 +18191,7 @@ function addSmartArtNodeAsChild(data, parentId, text) {
     ...data,
     nodes,
     connections: connections.length > 0 ? connections : void 0,
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 function reorderSmartArtNodeToIndex(data, nodeId, newIndex) {
@@ -18183,7 +18224,7 @@ function reorderSmartArtNodeToIndex(data, nodeId, newIndex) {
   return {
     ...data,
     nodes,
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 
@@ -18223,9 +18264,61 @@ function setSmartArtNodeStyle(data, nodeId, style) {
   return {
     ...data,
     nodes,
-    drawingShapes: void 0
+    drawingShapes: patchDrawingShapeStyle(data.drawingShapes, data.nodes, nodeId, style)
   };
 }
+function patchDrawingShapeStyle(shapes, nodes, nodeId, style) {
+  if (!shapes || shapes.length === 0) {
+    return shapes;
+  }
+  const idx = findShapeIndexForNode(shapes, nodes, nodeId);
+  if (idx < 0) {
+    return [];
+  }
+  const patch = {};
+  if (style.fillColor !== void 0) {
+    patch.fillColor = style.fillColor;
+  }
+  if (style.lineColor !== void 0) {
+    patch.strokeColor = style.lineColor;
+  }
+  if (style.fontColor !== void 0) {
+    patch.fontColor = style.fontColor;
+  }
+  if (Object.keys(patch).length === 0) {
+    return shapes;
+  }
+  const updated = [...shapes];
+  updated[idx] = { ...updated[idx], ...patch };
+  return updated;
+}
+function findShapeIndexForNode(shapes, nodes, nodeId) {
+  for (let i = 0; i < shapes.length; i++) {
+    if (shapes[i].id.startsWith("reflow-") && shapes[i].id.endsWith(`-${nodeId}`)) {
+      return i;
+    }
+  }
+  if (shapes.length === nodes.length) {
+    const nodeIdx = nodes.findIndex((n) => n.id === nodeId);
+    if (nodeIdx >= 0) {
+      return nodeIdx;
+    }
+  }
+  const node = nodes.find((n) => n.id === nodeId);
+  if (node?.text) {
+    const text = node.text.trim();
+    const matches = [];
+    for (let i = 0; i < shapes.length; i++) {
+      if (shapes[i].text?.trim() === text) {
+        matches.push(i);
+      }
+    }
+    if (matches.length === 1) {
+      return matches[0];
+    }
+  }
+  return -1;
+}
 
 // src/core/utils/smartart-editing-reflow-layouts-geometric.ts
 function reflowCycle(nodes, bounds) {
@@ -20282,20 +20375,24 @@ function switchSmartArtLayout(currentData, newLayoutType) {
   return {
     ...currentData,
     // Clear the raw layoutType string since the user is explicitly
-    // choosing a resolved category — this avoids the heuristic
+    // choosing a resolved category - this avoids the heuristic
     // re-resolve overriding their choice.
     layoutType: newLayoutType,
     resolvedLayoutType: newLayoutType,
-    // Clear the named layout preset — switching category invalidates it
+    // Clear the named layout preset - switching category invalidates it
     layout: void 0,
-    // Clear stale pre-computed drawing shapes from the old layout so the
+    // Mark stale pre-computed drawing shapes from the old layout so the
     // reflow pipeline (rebuildDrawingShapesIfCleared) regenerates them for
-    // the new layout type; otherwise the renderer keeps preferring the old
-    // shapes and the switch has no visible effect.
-    drawingShapes: void 0
+    // the new layout type. If the element never had drawing shapes (freshly
+    // inserted), leave undefined so the family SVG renderer handles it
+    // without triggering a lossy polygon-to-chevron reflow.
+    drawingShapes: markShapesStale2(currentData.drawingShapes)
     // Preserve everything else: nodes, connections, colours, styles, chrome, etc.
   };
 }
+function markShapesStale2(shapes) {
+  return shapes !== void 0 && shapes.length > 0 ? [] : void 0;
+}
 function isSwitchableLayoutType(layoutType) {
   return SWITCHABLE_LAYOUT_TYPES.includes(layoutType);
 }
@@ -37618,6 +37715,9 @@ var PptxHandlerRuntime41 = class _PptxHandlerRuntime extends PptxHandlerRuntime4
       slideLayoutRelationshipType,
       relationshipsNamespace
     });
+    for (const slide of slides) {
+      await this.processSlideForSave(slide, saveSession, saveConstants);
+    }
     const contentTypesXml2 = await this.zip.file("[Content_Types].xml")?.async("string");
     if (contentTypesXml2) {
       const contentTypesData = this.parser.parse(contentTypesXml2);
@@ -37629,9 +37729,6 @@ var PptxHandlerRuntime41 = class _PptxHandlerRuntime extends PptxHandlerRuntime4
       });
       this.zip.file("[Content_Types].xml", this.builder.build(contentTypesData));
     }
-    for (const slide of slides) {
-      await this.processSlideForSave(slide, saveSession, saveConstants);
-    }
     for (const existingCommentPath of saveSession.getExistingCommentPaths()) {
       if (saveSession.isCommentPathActive(existingCommentPath)) {
         continue;

package/dist/index.mjs

@@ -17947,6 +17947,9 @@ function resolveLayoutFromRawType(layoutType) {
 }
 
 // src/core/utils/smartart-editing-node-ops.ts
+function markShapesStale(shapes) {
+  return shapes !== void 0 && shapes.length > 0 ? [] : void 0;
+}
 function nextModelId() {
   return `{${generateFontGuid()}}`;
 }
@@ -18000,8 +18003,7 @@ function addSmartArtNode(data, text, afterNodeId) {
     ...data,
     nodes,
     connections: connections.length > 0 ? connections : void 0,
-    // Clear pre-computed shapes to force layout reflow
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 function removeSmartArtNode(data, nodeId) {
@@ -18030,7 +18032,7 @@ function removeSmartArtNode(data, nodeId) {
     ...data,
     nodes,
     connections: connections.length > 0 ? connections : void 0,
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 function updateSmartArtNodeText(data, nodeId, newText) {
@@ -18038,9 +18040,48 @@ function updateSmartArtNodeText(data, nodeId, newText) {
   return {
     ...data,
     nodes,
-    drawingShapes: void 0
+    drawingShapes: patchDrawingShapeText(data.drawingShapes, data.nodes, nodeId, newText)
   };
 }
+function patchDrawingShapeText(shapes, nodes, nodeId, newText) {
+  if (!shapes || shapes.length === 0) {
+    return shapes;
+  }
+  const idx = findDrawingShapeIndex(shapes, nodes, nodeId);
+  if (idx < 0) {
+    return [];
+  }
+  const updated = [...shapes];
+  updated[idx] = { ...updated[idx], text: newText };
+  return updated;
+}
+function findDrawingShapeIndex(shapes, nodes, nodeId) {
+  for (let i = 0; i < shapes.length; i++) {
+    if (shapes[i].id.startsWith("reflow-") && shapes[i].id.endsWith(`-${nodeId}`)) {
+      return i;
+    }
+  }
+  if (shapes.length === nodes.length) {
+    const nodeIdx = nodes.findIndex((n) => n.id === nodeId);
+    if (nodeIdx >= 0) {
+      return nodeIdx;
+    }
+  }
+  const node = nodes.find((n) => n.id === nodeId);
+  if (node?.text) {
+    const text = node.text.trim();
+    const matches = [];
+    for (let i = 0; i < shapes.length; i++) {
+      if (shapes[i].text?.trim() === text) {
+        matches.push(i);
+      }
+    }
+    if (matches.length === 1) {
+      return matches[0];
+    }
+  }
+  return -1;
+}
 function reorderSmartArtNode(data, nodeId, direction) {
   const node = data.nodes.find((n) => n.id === nodeId);
   if (!node) {
@@ -18063,7 +18104,7 @@ function reorderSmartArtNode(data, nodeId, direction) {
   return {
     ...data,
     nodes,
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 function promoteSmartArtNode(data, nodeId) {
@@ -18090,7 +18131,7 @@ function promoteSmartArtNode(data, nodeId) {
     ...data,
     nodes,
     connections: connections.length > 0 ? connections : void 0,
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 function demoteSmartArtNode(data, nodeId) {
@@ -18117,7 +18158,7 @@ function demoteSmartArtNode(data, nodeId) {
     ...data,
     nodes,
     connections: connections.length > 0 ? connections : void 0,
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 function addSmartArtNodeAsChild(data, parentId, text) {
@@ -18145,7 +18186,7 @@ function addSmartArtNodeAsChild(data, parentId, text) {
     ...data,
     nodes,
     connections: connections.length > 0 ? connections : void 0,
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 function reorderSmartArtNodeToIndex(data, nodeId, newIndex) {
@@ -18178,7 +18219,7 @@ function reorderSmartArtNodeToIndex(data, nodeId, newIndex) {
   return {
     ...data,
     nodes,
-    drawingShapes: void 0
+    drawingShapes: markShapesStale(data.drawingShapes)
   };
 }
 
@@ -18218,9 +18259,61 @@ function setSmartArtNodeStyle(data, nodeId, style) {
   return {
     ...data,
     nodes,
-    drawingShapes: void 0
+    drawingShapes: patchDrawingShapeStyle(data.drawingShapes, data.nodes, nodeId, style)
   };
 }
+function patchDrawingShapeStyle(shapes, nodes, nodeId, style) {
+  if (!shapes || shapes.length === 0) {
+    return shapes;
+  }
+  const idx = findShapeIndexForNode(shapes, nodes, nodeId);
+  if (idx < 0) {
+    return [];
+  }
+  const patch = {};
+  if (style.fillColor !== void 0) {
+    patch.fillColor = style.fillColor;
+  }
+  if (style.lineColor !== void 0) {
+    patch.strokeColor = style.lineColor;
+  }
+  if (style.fontColor !== void 0) {
+    patch.fontColor = style.fontColor;
+  }
+  if (Object.keys(patch).length === 0) {
+    return shapes;
+  }
+  const updated = [...shapes];
+  updated[idx] = { ...updated[idx], ...patch };
+  return updated;
+}
+function findShapeIndexForNode(shapes, nodes, nodeId) {
+  for (let i = 0; i < shapes.length; i++) {
+    if (shapes[i].id.startsWith("reflow-") && shapes[i].id.endsWith(`-${nodeId}`)) {
+      return i;
+    }
+  }
+  if (shapes.length === nodes.length) {
+    const nodeIdx = nodes.findIndex((n) => n.id === nodeId);
+    if (nodeIdx >= 0) {
+      return nodeIdx;
+    }
+  }
+  const node = nodes.find((n) => n.id === nodeId);
+  if (node?.text) {
+    const text = node.text.trim();
+    const matches = [];
+    for (let i = 0; i < shapes.length; i++) {
+      if (shapes[i].text?.trim() === text) {
+        matches.push(i);
+      }
+    }
+    if (matches.length === 1) {
+      return matches[0];
+    }
+  }
+  return -1;
+}
 
 // src/core/utils/smartart-editing-reflow-layouts-geometric.ts
 function reflowCycle(nodes, bounds) {
@@ -20277,20 +20370,24 @@ function switchSmartArtLayout(currentData, newLayoutType) {
   return {
     ...currentData,
     // Clear the raw layoutType string since the user is explicitly
-    // choosing a resolved category — this avoids the heuristic
+    // choosing a resolved category - this avoids the heuristic
     // re-resolve overriding their choice.
     layoutType: newLayoutType,
     resolvedLayoutType: newLayoutType,
-    // Clear the named layout preset — switching category invalidates it
+    // Clear the named layout preset - switching category invalidates it
     layout: void 0,
-    // Clear stale pre-computed drawing shapes from the old layout so the
+    // Mark stale pre-computed drawing shapes from the old layout so the
     // reflow pipeline (rebuildDrawingShapesIfCleared) regenerates them for
-    // the new layout type; otherwise the renderer keeps preferring the old
-    // shapes and the switch has no visible effect.
-    drawingShapes: void 0
+    // the new layout type. If the element never had drawing shapes (freshly
+    // inserted), leave undefined so the family SVG renderer handles it
+    // without triggering a lossy polygon-to-chevron reflow.
+    drawingShapes: markShapesStale2(currentData.drawingShapes)
     // Preserve everything else: nodes, connections, colours, styles, chrome, etc.
   };
 }
+function markShapesStale2(shapes) {
+  return shapes !== void 0 && shapes.length > 0 ? [] : void 0;
+}
 function isSwitchableLayoutType(layoutType) {
   return SWITCHABLE_LAYOUT_TYPES.includes(layoutType);
 }
@@ -37613,6 +37710,9 @@ var PptxHandlerRuntime41 = class _PptxHandlerRuntime extends PptxHandlerRuntime4
       slideLayoutRelationshipType,
       relationshipsNamespace
     });
+    for (const slide of slides) {
+      await this.processSlideForSave(slide, saveSession, saveConstants);
+    }
     const contentTypesXml2 = await this.zip.file("[Content_Types].xml")?.async("string");
     if (contentTypesXml2) {
       const contentTypesData = this.parser.parse(contentTypesXml2);
@@ -37624,9 +37724,6 @@ var PptxHandlerRuntime41 = class _PptxHandlerRuntime extends PptxHandlerRuntime4
       });
       this.zip.file("[Content_Types].xml", this.builder.build(contentTypesData));
     }
-    for (const slide of slides) {
-      await this.processSlideForSave(slide, saveSession, saveConstants);
-    }
     for (const existingCommentPath of saveSession.getExistingCommentPaths()) {
       if (saveSession.isCommentPathActive(existingCommentPath)) {
         continue;

package/package.json

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