node-pptx-templater 1.1.1 → 1.1.2

package/CHANGELOG.md

@@ -5,6 +5,22 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.1.3] - 2026-06-22
+
+### Fixed
+
+- **Layout Preservation in `addCellShape()`**: Fixed a critical layout bug where querying cell bounds or adding cell shapes dynamically (via `addCellShape()` or `updateCellShape()`) mutated table row heights inside the XML. The sizing/positioning logic now runs in a layout-only mode, keeping the table XML completely untouched and respecting original template custom row heights.
+
+## [1.1.2] - 2026-06-22
+
+### Fixed
+
+- **Slide Link Relationships Target Paths**: Fixed PowerPoint "Repair Mode" errors when adding slide-to-slide hyperlinks (via `addSlideLink` or `addImageLink`). The relative target path resolves correctly to the sibling filename `slideX.xml` instead of using the redundant parent prefix `../slides/slideX.xml` (which exited and re-entered the same directory, violating PowerPoint's relative path resolution rules).
+
+### Added
+
+- **Redundant Traversal Validation**: Added automated check in `ValidationEngine` to identify and error on redundant relative path traversals inside relationship files (e.g. referencing `../slides/slideX.xml` from a source slide file already located in `ppt/slides/`).
+
 ## [1.1.0] - 2026-06-12
 
 ### Added

package/README.md

@@ -172,27 +172,51 @@ const meta = await ppt.getTableRows('SalesTable', { includeMetadata: true });
 ```
 
 ### Table Cell Shapes
-Cell shapes are overlay graphics anchored within table cells. They are positioned absolutely based on cell bounds, and **never** modify row heights, column widths, cell margins, or trigger table reflow. Offsets (`x`, `y`) are relative to the cell's top-left corner. Oversized shapes are scaled down proportionally to fit inside the cell.
+Cell shapes are overlay graphics anchored within table cells. They are positioned absolutely based on cell bounds, and **never** modify row heights, column widths, cell margins, or trigger table reflow.
+
+#### Merged Cell Support
+When targeting a merged cell (spanned using `rowSpan` or `colSpan`), `addCellShape()` dynamically resolves the **actual rendered bounds** of the entire merged cell region (summing the width and height of the spanned rows/columns). The shape is positioned and aligned relative to this final merged boundary. Any offset `x` and `y` specified is interpreted relative to the top-left corner of the resolved merged cell region.
+
+#### Alignment & Presets
+Shapes can be aligned dynamically within the cell (or merged region) using preset positions or explicit alignments:
+* **Presets**: Use `position` values like `'top-left'`, `'center'`, `'bottom-right'`, etc.
+* **Alignment Options**: Set `alignX: 'left' | 'center' | 'right'` and `alignY: 'top' | 'middle' | 'bottom'`. If no alignments or offsets are provided, shapes default to true centering (`'center'`/`'middle'`).
+* **Offsets & Padding**: Offsets (`x`, `y`) are applied relative to the resolved alignment. For instance, when `alignX: 'right'` is used, the shape is placed at `cellRight - shapeWidth - x` (defaulting to a `5px` padding if `x` is omitted).
 
 ```javascript
-// Add a simple indicator
-await ppt.addCellShape('SalesTable', 2, 1, {
+// Add a status icon centered in a merged cell (rowSpan=2, colSpan=2)
+await ppt.addCellShape('StatusTable', 1, 1, {
   type: 'circle',
   width: 12,
   height: 12,
-  fill: '#10B981' // Green status dot
+  fill: '#10B981', // Green status dot
+  alignX: 'center',
+  alignY: 'middle' // Visually centered in the merged cell
 });
 
-// Add a badge with text and custom offsets
-await ppt.addCellShape('SalesTable', 1, 2, {
+// Add a badge with text and explicit offsets inside a merged cell
+await ppt.addCellShape('StatusTable', 1, 1, {
   type: 'badge',
-  text: 'Active',
-  fill: '#3B82F6',
-  x: 4,
-  y: 2,
+  text: 'Urgent',
+  fill: '#EF4444',
+  alignX: 'left',
+  alignY: 'top',
+  x: 5,
+  y: 3,
   width: 50,
   height: 16
 });
+
+// Add an indicator with bottom-right alignment and custom padding
+await ppt.addCellShape('StatusTable', 1, 1, {
+  type: 'icon',
+  icon: 'up',
+  size: 14,
+  alignX: 'right',
+  alignY: 'bottom',
+  x: 4,
+  y: 2
+});
 ```
 
 ### XML Folder Workflow

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-pptx-templater",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "High-performance, low-level PowerPoint (PPTX) OpenXML template engine for Node.js. Dynamically replace text, insert images, update charts (with Excel workbook data caching), and merge table cells without PowerPoint corruption or Repair Mode prompts.",
   "main": "./src/index.js",
   "type": "commonjs",

package/src/core/PPTXTemplater.js

@@ -2727,6 +2727,8 @@ class PPTXTemplater {
 
   /**
    * Dynamically adds a shape inside a table cell based on cell coordinates.
+   * Cell shapes are overlay graphics anchored independently of the table layout,
+   * and adding a cell shape never modifies row heights, column widths, or table dimensions.
    *
    * @param {string} tableId - Table name or shape ID.
    * @param {number} rowIndex - 0-based row index.

package/src/core/ValidationEngine.js

@@ -237,6 +237,18 @@ class ValidationEngine {
         if (!ppt.zipManager.hasFile(resolved)) {
           errors.push(`Relationship ${rel.id} points to non-existent file: ${resolved}`)
         }
+
+        // Catch redundant relative path traversals (e.g., '../slides/slide1.xml' from 'ppt/slides/slide2.xml')
+        const sourceDir = partPath.split('/').slice(0, -1).join('/')
+        const resolvedDir = resolved.split('/').slice(0, -1).join('/')
+        if (sourceDir === resolvedDir) {
+          const expectedTarget = resolved.split('/').pop()
+          if (rel.target !== expectedTarget) {
+            errors.push(
+              `Relationship "${rel.id}" in ${relsPath} uses redundant relative target "${rel.target}" (expected "${expectedTarget}")`
+            )
+          }
+        }
       }
     }
 

package/src/managers/HyperlinkManager.js

@@ -100,7 +100,7 @@ class HyperlinkManager {
     const targetInfo = slideManager.getSlideInfo(targetSlideIndex)
 
     // Build relative target path from source to target slide
-    const relativePath = `../slides/${targetInfo.zipPath.split('/').pop()}`
+    const relativePath = targetInfo.zipPath.split('/').pop()
 
     // Add relationship pointing to the target slide
     const rId = relationshipManager.addRelationship(
@@ -130,7 +130,7 @@ class HyperlinkManager {
     const sourceInfo = slideManager.getSlideInfo(sourceSlideIndex)
     const targetInfo = slideManager.getSlideInfo(targetSlideIndex)
 
-    const relativePath = `../slides/${targetInfo.zipPath.split('/').pop()}`
+    const relativePath = targetInfo.zipPath.split('/').pop()
     const rId = relationshipManager.addRelationship(
       sourceInfo.zipPath,
       REL_TYPES.SLIDE,
@@ -167,7 +167,7 @@ class HyperlinkManager {
     const sourceInfo = slideManager.getSlideInfo(sourceSlideIndex)
     const targetInfo = slideManager.getSlideInfo(targetSlideIndex)
 
-    const relativePath = `../slides/${targetInfo.zipPath.split('/').pop()}`
+    const relativePath = targetInfo.zipPath.split('/').pop()
     const rId = relationshipManager.addRelationship(
       sourceInfo.zipPath,
       REL_TYPES.SLIDE,

package/src/managers/TableManager.js

@@ -223,7 +223,7 @@ class TableManager {
       )
     }
 
-    this.#calculateRowHeights(slideIndex, tableId, slideManager, tblObj)
+    this.#calculateRowHeights(slideIndex, tableId, slideManager, tblObj, true)
 
     if (cellShapes) {
       this.#processCellShapes(
@@ -796,8 +796,6 @@ class TableManager {
   getCellBounds(slideIndex, tableId, rowIndex, colIndex, slideManager) {
     const { tblObj, frameObj } = this.#getTableContext(slideIndex, tableId, slideManager)
 
-    this.#calculateRowHeights(slideIndex, tableId, slideManager, tblObj)
-
     const xfrm = frameObj['p:xfrm']
     const tableX = xfrm?.['a:off']?.['@_x'] ? parseInt(xfrm['a:off']['@_x'], 10) : 0
     const tableY = xfrm?.['a:off']?.['@_y'] ? parseInt(xfrm['a:off']['@_y'], 10) : 0
@@ -807,7 +805,7 @@ class TableManager {
     const colWidths = gridColsArr.map(col => parseInt(col['@_w'] || 0, 10))
 
     const trsArr = tblObj['a:tr'] || []
-    const rowHeights = trsArr.map(row => parseInt(row['@_h'] || 0, 10))
+    const rowHeights = this.#calculateRowHeights(slideIndex, tableId, slideManager, tblObj, false)
 
     const parent = this.getMergeParent(slideIndex, tableId, rowIndex, colIndex, slideManager)
     const pr = parent.row
@@ -1936,8 +1934,6 @@ class TableManager {
       slideManager
     )
 
-    this.#calculateRowHeights(slideIndex, tableId, slideManager, tblObj)
-
     const xfrm = frameObj['p:xfrm']
     const tableX = xfrm?.['a:off']?.['@_x'] ? parseInt(xfrm['a:off']['@_x'], 10) : 0
     const tableY = xfrm?.['a:off']?.['@_y'] ? parseInt(xfrm['a:off']['@_y'], 10) : 0
@@ -1947,7 +1943,7 @@ class TableManager {
     const colWidths = gridColsArr.map(col => parseInt(col['@_w'] || 0, 10))
 
     const trsArr = tblObj['a:tr'] || []
-    const rowHeights = trsArr.map(row => parseInt(row['@_h'] || 0, 10))
+    const rowHeights = this.#calculateRowHeights(slideIndex, tableId, slideManager, tblObj, false)
 
     const parent = this.getMergeParent(slideIndex, tableId, rowIndex, colIndex, slideManager)
     const pr = parent.row
@@ -2021,8 +2017,6 @@ class TableManager {
       slideManager
     )
 
-    this.#calculateRowHeights(slideIndex, tableId, slideManager, tblObj)
-
     const shapes = shapeManager.getShapes(slideIndex, slideManager)
     const prefix = `cellshape_${resolvedTableId}_${rowIndex}_${colIndex}_${shapeIndex}`
     const matchingShapes = shapes.filter(
@@ -2046,7 +2040,7 @@ class TableManager {
     const colWidths = gridColsArr.map(col => parseInt(col['@_w'] || 0, 10))
 
     const trsArr = tblObj['a:tr'] || []
-    const rowHeights = trsArr.map(row => parseInt(row['@_h'] || 0, 10))
+    const rowHeights = this.#calculateRowHeights(slideIndex, tableId, slideManager, tblObj, false)
 
     const parent = this.getMergeParent(slideIndex, tableId, rowIndex, colIndex, slideManager)
     const pr = parent.row
@@ -2158,7 +2152,7 @@ class TableManager {
     }
   }
 
-  #calculateRowHeights(slideIndex, tableId, slideManager, tblObj) {
+  #calculateRowHeights(slideIndex, tableId, slideManager, tblObj, writeToXml = true) {
     const trsArr = tblObj['a:tr'] || []
     if (trsArr.length === 0) return []
 
@@ -2172,7 +2166,7 @@ class TableManager {
     // Initialize rowHeights with original height or a safe minimum floor of 228600 EMUs (~24px/pt)
     const rowHeights = trsArr.map(row => {
       const h = parseInt(row['@_h'] || 0, 10)
-      return Math.max(h, 228600)
+      return h > 0 ? h : 228600
     })
 
     // Helper to get paragraph font size
@@ -2295,7 +2289,9 @@ class TableManager {
         }
 
         const totalCellHeight_emu = marT + marB + textHeight_emu
-        cellHeights[r][c] = Math.max(totalCellHeight_emu, 228600)
+        const rowTemplateHeight = parseInt(row['@_h'] || 0, 10)
+        const minFloor = rowTemplateHeight > 0 ? rowTemplateHeight : 228600
+        cellHeights[r][c] = Math.max(totalCellHeight_emu, minFloor)
       }
     }
 
@@ -2346,8 +2342,10 @@ class TableManager {
     }
 
     // Update row heights in XML
-    for (let r = 0; r < numRows; r++) {
-      trsArr[r]['@_h'] = String(rowHeights[r])
+    if (writeToXml) {
+      for (let r = 0; r < numRows; r++) {
+        trsArr[r]['@_h'] = String(rowHeights[r])
+      }
     }
 
     return rowHeights