node-pptx-templater 1.1.1 → 1.1.3

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.3",
   "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",
@@ -37,6 +37,7 @@
     "example:table-extraction": "node examples/table-extraction.js",
     "example:nested-rows": "node examples/nested-table-rows.js",
     "example:xml-folder": "node examples/xml-folder-workflow.js",
+    "example:all": "node scripts/run-examples-check.js",
     "prepublishOnly": "npm run lint && npm run test"
   },
   "keywords": [

package/src/core/OutputWriter.js

@@ -161,11 +161,11 @@ class OutputWriter {
           : e.compressionMethod === 0
             ? 'STORE'
             : `UNKNOWN(${e.compressionMethod})`
-      console.log(e.name)
-      console.log(`compressed: ${e.compressedSize}`)
-      console.log(`uncompressed: ${e.uncompressedSize}`)
-      console.log(`crc: ${e.crc32}`)
-      console.log(`method: ${methodStr}`)
+      logger.info(e.name)
+      logger.info(`compressed: ${e.compressedSize}`)
+      logger.info(`uncompressed: ${e.uncompressedSize}`)
+      logger.info(`crc: ${e.crc32}`)
+      logger.info(`method: ${methodStr}`)
     })
     logger.info('--- End of ZIP debug output ---')
   }

package/src/core/PPTXTemplater.js

@@ -2654,6 +2654,16 @@ class PPTXTemplater {
     return this.#shapeManager.validateShape(options)
   }
 
+  /**
+   * Adds a new shape to the targeted slide(s).
+   *
+   * @param {string|Object} typeOrOptions Either the shape type string (e.g., 'rect', 'ellipse') or a full options object.
+   * @param {Object} [options={}] Additional configuration options if type was specified as a string.
+   * @returns {Promise<PPTXTemplater>} The templater instance for chaining.
+   *
+   * @example
+   * await ppt.addShape('rect', { id: 'MyShape', x: 100, y: 100, width: 200, height: 100 });
+   */
   async addShape(typeOrOptions, options = {}) {
     this.#assertLoaded()
     let resolvedOptions = {}
@@ -2727,6 +2737,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.
@@ -2736,14 +2748,28 @@ class PPTXTemplater {
    */
   async addCellShape(tableId, rowIndex, colIndex, options) {
     this.#assertLoaded()
+    let row = rowIndex
+    let col = colIndex
+    let opts = options
+    if (rowIndex && typeof rowIndex === 'object') {
+      row = rowIndex.row !== undefined ? rowIndex.row : rowIndex.rowIndex
+      col =
+        rowIndex.column !== undefined
+          ? rowIndex.column
+          : rowIndex.col !== undefined
+            ? rowIndex.col
+            : rowIndex.colIndex
+      opts = rowIndex
+    }
+
     const targetIndices = this.#getTargetSlideIndices()
     for (const idx of targetIndices) {
       this.#tableManager.addCellShape(
         idx,
         tableId,
-        rowIndex,
-        colIndex,
-        options,
+        row,
+        col,
+        opts,
         this.#slideManager,
         this.#shapeManager
       )
@@ -2763,15 +2789,36 @@ class PPTXTemplater {
    */
   async updateCellShape(tableId, rowIndex, colIndex, shapeIndex, options) {
     this.#assertLoaded()
+    let row = rowIndex
+    let col = colIndex
+    let shpIdx = shapeIndex
+    let opts = options
+    if (rowIndex && typeof rowIndex === 'object') {
+      row = rowIndex.row !== undefined ? rowIndex.row : rowIndex.rowIndex
+      col =
+        rowIndex.column !== undefined
+          ? rowIndex.column
+          : rowIndex.col !== undefined
+            ? rowIndex.col
+            : rowIndex.colIndex
+      shpIdx =
+        rowIndex.shapeIndex !== undefined
+          ? rowIndex.shapeIndex
+          : rowIndex.shape !== undefined
+            ? rowIndex.shape
+            : colIndex
+      opts = options !== undefined ? options : rowIndex
+    }
+
     const targetIndices = this.#getTargetSlideIndices()
     for (const idx of targetIndices) {
       this.#tableManager.updateCellShape(
         idx,
         tableId,
-        rowIndex,
-        colIndex,
-        shapeIndex,
-        options,
+        row,
+        col,
+        shpIdx,
+        opts,
         this.#slideManager,
         this.#shapeManager
       )
@@ -2790,14 +2837,33 @@ class PPTXTemplater {
    */
   async removeCellShape(tableId, rowIndex, colIndex, shapeIndex) {
     this.#assertLoaded()
+    let row = rowIndex
+    let col = colIndex
+    let shpIdx = shapeIndex
+    if (rowIndex && typeof rowIndex === 'object') {
+      row = rowIndex.row !== undefined ? rowIndex.row : rowIndex.rowIndex
+      col =
+        rowIndex.column !== undefined
+          ? rowIndex.column
+          : rowIndex.col !== undefined
+            ? rowIndex.col
+            : rowIndex.colIndex
+      shpIdx =
+        rowIndex.shapeIndex !== undefined
+          ? rowIndex.shapeIndex
+          : rowIndex.shape !== undefined
+            ? rowIndex.shape
+            : colIndex
+    }
+
     const targetIndices = this.#getTargetSlideIndices()
     for (const idx of targetIndices) {
       this.#tableManager.removeCellShape(
         idx,
         tableId,
-        rowIndex,
-        colIndex,
-        shapeIndex,
+        row,
+        col,
+        shpIdx,
         this.#slideManager,
         this.#shapeManager
       )
@@ -2816,14 +2882,33 @@ class PPTXTemplater {
    */
   getCellShape(tableId, rowIndex, colIndex, shapeIndex) {
     this.#assertLoaded()
+    let row = rowIndex
+    let col = colIndex
+    let shpIdx = shapeIndex
+    if (rowIndex && typeof rowIndex === 'object') {
+      row = rowIndex.row !== undefined ? rowIndex.row : rowIndex.rowIndex
+      col =
+        rowIndex.column !== undefined
+          ? rowIndex.column
+          : rowIndex.col !== undefined
+            ? rowIndex.col
+            : rowIndex.colIndex
+      shpIdx =
+        rowIndex.shapeIndex !== undefined
+          ? rowIndex.shapeIndex
+          : rowIndex.shape !== undefined
+            ? rowIndex.shape
+            : colIndex
+    }
+
     const targetIndices = this.#getTargetSlideIndices()
     for (const idx of targetIndices) {
       const shape = this.#tableManager.getCellShape(
         idx,
         tableId,
-        rowIndex,
-        colIndex,
-        shapeIndex,
+        row,
+        col,
+        shpIdx,
         this.#slideManager,
         this.#shapeManager
       )
@@ -2846,16 +2931,22 @@ class PPTXTemplater {
       typeof tableIdOrObj === 'object'
         ? tableIdOrObj.id || tableIdOrObj.name || tableIdOrObj.tableId
         : tableIdOrObj
+    let row = rowIndex
+    let col = colIndex
+    if (rowIndex && typeof rowIndex === 'object') {
+      row = rowIndex.row !== undefined ? rowIndex.row : rowIndex.rowIndex
+      col =
+        rowIndex.column !== undefined
+          ? rowIndex.column
+          : rowIndex.col !== undefined
+            ? rowIndex.col
+            : rowIndex.colIndex
+    }
+
     const targetIndices = this.#getTargetSlideIndices()
     for (const idx of targetIndices) {
       try {
-        const bounds = this.#tableManager.getCellBounds(
-          idx,
-          tableId,
-          rowIndex,
-          colIndex,
-          this.#slideManager
-        )
+        const bounds = this.#tableManager.getCellBounds(idx, tableId, row, col, this.#slideManager)
         if (bounds) return bounds
       } catch (err) {
         logger.debug(
@@ -2883,17 +2974,33 @@ class PPTXTemplater {
       typeof tableIdOrObj === 'object'
         ? tableIdOrObj.id || tableIdOrObj.name || tableIdOrObj.tableId
         : tableIdOrObj
+    let row = rowIndex
+    let col = colIndex
+    let widthOrOpts = shapeWidthOrOptions
+    let height = shapeHeight
+    if (rowIndex && typeof rowIndex === 'object') {
+      row = rowIndex.row !== undefined ? rowIndex.row : rowIndex.rowIndex
+      col =
+        rowIndex.column !== undefined
+          ? rowIndex.column
+          : rowIndex.col !== undefined
+            ? rowIndex.col
+            : rowIndex.colIndex
+      widthOrOpts = colIndex
+      height = shapeWidthOrOptions
+    }
+
     const targetIndices = this.#getTargetSlideIndices()
     for (const idx of targetIndices) {
       try {
         const pos = this.#tableManager.getCellPosition(
           idx,
           tableId,
-          rowIndex,
-          colIndex,
+          row,
+          col,
           this.#slideManager,
-          shapeWidthOrOptions,
-          shapeHeight
+          widthOrOpts,
+          height
         )
         if (pos) return pos
       } catch (err) {

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
@@ -1304,10 +1302,10 @@ class TableManager {
   }
 
   #expandCellShape(config, cellBounds) {
-    const cellLeft_px = Math.round(cellBounds.left / 9525)
-    const cellTop_px = Math.round(cellBounds.top / 9525)
-    const cellWidth_px = Math.round(cellBounds.width / 9525)
-    const cellHeight_px = Math.round(cellBounds.height / 9525)
+    const cellLeft_px = cellBounds.x
+    const cellTop_px = cellBounds.y
+    const cellWidth_px = cellBounds.width
+    const cellHeight_px = cellBounds.height
 
     const parseLength = (val, maxVal) => {
       if (typeof val === 'string' && val.endsWith('%')) {
@@ -1745,7 +1743,7 @@ class TableManager {
     slideManager,
     shapeManager,
     tblObj,
-    frameObj
+    _frameObj
   ) {
     if (!cellShapes || !shapeManager) return
 
@@ -1763,62 +1761,16 @@ class TableManager {
       }
     }
 
-    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
-
-    const gridCols = tblObj['a:tblGrid']?.['a:gridCol'] || []
-    const gridColsArr = Array.isArray(gridCols) ? gridCols : [gridCols]
-    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 getCellBounds = (r, c) => {
-      const parent = this.getMergeParent(slideIndex, tableId, r, c, slideManager)
-      const pr = parent.row
-      const pc = parent.col
-
-      let cellLeft = tableX
-      for (let idx = 0; idx < pc; idx++) {
-        cellLeft += colWidths[idx] || 0
-      }
-
-      let cellTop = tableY
-      for (let idx = 0; idx < pr; idx++) {
-        cellTop += rowHeights[idx] || 0
-      }
-
-      const parentCell = trsArr[pr]?.['a:tc']?.[pc]
-      const gridSpan = parentCell?.['@_gridSpan'] ? parseInt(parentCell['@_gridSpan'], 10) : 1
-      const rowSpan = parentCell?.['@_rowSpan'] ? parseInt(parentCell['@_rowSpan'], 10) : 1
-
-      let cellWidth = 0
-      for (let idx = 0; idx < gridSpan; idx++) {
-        cellWidth += colWidths[pc + idx] || 0
-      }
-
-      let cellHeight = 0
-      for (let idx = 0; idx < rowSpan; idx++) {
-        cellHeight += rowHeights[pr + idx] || 0
-      }
-
-      return {
-        left: cellLeft,
-        top: cellTop,
-        width: cellWidth,
-        height: cellHeight,
-      }
-    }
-
     const shapesToCreate = []
-    const headerNames = (trsArr[0]?.['a:tc'] || []).map(cell => this.#getCellText(cell).trim())
+    const headerNames = (tblObj['a:tr']?.[0]?.['a:tc'] || []).map(cell =>
+      this.#getCellText(cell).trim()
+    )
 
     for (let i = 0; i < rowsData.length; i++) {
       const rowData = rowsData[i]
       const finalRowIndex = isObjectRows ? i + 1 : i
 
-      const numCols = trsArr[finalRowIndex]?.['a:tc']?.length || 0
+      const numCols = tblObj['a:tr']?.[finalRowIndex]?.['a:tc']?.length || 0
       for (let j = 0; j < numCols; j++) {
         const headerName = headerNames[j]
         let shapeFn = null
@@ -1853,16 +1805,24 @@ class TableManager {
     shapesToCreate.sort((a, b) => (a.config.zIndex || 0) - (b.config.zIndex || 0))
 
     shapesToCreate.forEach(item => {
-      const bounds = getCellBounds(item.rowIndex, item.colIndex)
-      const expandedConfigs = this.#expandCellShape(item.config, bounds)
+      const bounds = this.getCellBounds(
+        slideIndex,
+        tableId,
+        item.rowIndex,
+        item.colIndex,
+        slideManager
+      )
+      if (bounds) {
+        const expandedConfigs = this.#expandCellShape(item.config, bounds)
 
-      expandedConfigs.forEach((expandedConfig, expIdx) => {
-        const finalShapeIndex =
-          expandedConfigs.length > 1 ? `${item.shapeIndex}_${expIdx}` : item.shapeIndex
-        expandedConfig.id = `cellshape_${resolvedTableId}_${item.rowIndex}_${item.colIndex}_${finalShapeIndex}`
+        expandedConfigs.forEach((expandedConfig, expIdx) => {
+          const finalShapeIndex =
+            expandedConfigs.length > 1 ? `${item.shapeIndex}_${expIdx}` : item.shapeIndex
+          expandedConfig.id = `cellshape_${resolvedTableId}_${item.rowIndex}_${item.colIndex}_${finalShapeIndex}`
 
-        shapeManager.addShape(slideIndex, expandedConfig, slideManager)
-      })
+          shapeManager.addShape(slideIndex, expandedConfig, slideManager)
+        })
+      }
     })
   }
 
@@ -1930,55 +1890,13 @@ class TableManager {
   }
 
   addCellShape(slideIndex, tableId, rowIndex, colIndex, options, slideManager, shapeManager) {
-    const { tblObj, frameObj, resolvedTableId } = 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
-
-    const gridCols = tblObj['a:tblGrid']?.['a:gridCol'] || []
-    const gridColsArr = Array.isArray(gridCols) ? gridCols : [gridCols]
-    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 parent = this.getMergeParent(slideIndex, tableId, rowIndex, colIndex, slideManager)
-    const pr = parent.row
-    const pc = parent.col
-
-    let cellLeft = tableX
-    for (let idx = 0; idx < pc; idx++) {
-      cellLeft += colWidths[idx] || 0
-    }
-
-    let cellTop = tableY
-    for (let idx = 0; idx < pr; idx++) {
-      cellTop += rowHeights[idx] || 0
-    }
-
-    const parentCell = trsArr[pr]?.['a:tc']?.[pc]
-    const gridSpan = parentCell?.['@_gridSpan'] ? parseInt(parentCell['@_gridSpan'], 10) : 1
-    const rowSpan = parentCell?.['@_rowSpan'] ? parseInt(parentCell['@_rowSpan'], 10) : 1
-
-    let cellWidth = 0
-    for (let idx = 0; idx < gridSpan; idx++) {
-      cellWidth += colWidths[pc + idx] || 0
-    }
+    const { resolvedTableId } = this.#getTableContext(slideIndex, tableId, slideManager)
 
-    let cellHeight = 0
-    for (let idx = 0; idx < rowSpan; idx++) {
-      cellHeight += rowHeights[pr + idx] || 0
+    const bounds = this.getCellBounds(slideIndex, tableId, rowIndex, colIndex, slideManager)
+    if (!bounds) {
+      throw new PPTXError(`Could not calculate bounds for cell (${rowIndex}, ${colIndex})`)
     }
 
-    const bounds = { left: cellLeft, top: cellTop, width: cellWidth, height: cellHeight }
-
     const shapes = shapeManager.getShapes(slideIndex, slideManager)
     const prefix = `cellshape_${resolvedTableId}_${rowIndex}_${colIndex}_`
     let maxShapeIndex = -1
@@ -2015,13 +1933,7 @@ class TableManager {
     slideManager,
     shapeManager
   ) {
-    const { tblObj, frameObj, resolvedTableId } = this.#getTableContext(
-      slideIndex,
-      tableId,
-      slideManager
-    )
-
-    this.#calculateRowHeights(slideIndex, tableId, slideManager, tblObj)
+    const { resolvedTableId } = this.#getTableContext(slideIndex, tableId, slideManager)
 
     const shapes = shapeManager.getShapes(slideIndex, slideManager)
     const prefix = `cellshape_${resolvedTableId}_${rowIndex}_${colIndex}_${shapeIndex}`
@@ -2037,47 +1949,11 @@ class TableManager {
       shapeManager.deleteShape(slideIndex, s.name, slideManager)
     }
 
-    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
-
-    const gridCols = tblObj['a:tblGrid']?.['a:gridCol'] || []
-    const gridColsArr = Array.isArray(gridCols) ? gridCols : [gridCols]
-    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 parent = this.getMergeParent(slideIndex, tableId, rowIndex, colIndex, slideManager)
-    const pr = parent.row
-    const pc = parent.col
-
-    let cellLeft = tableX
-    for (let idx = 0; idx < pc; idx++) {
-      cellLeft += colWidths[idx] || 0
-    }
-
-    let cellTop = tableY
-    for (let idx = 0; idx < pr; idx++) {
-      cellTop += rowHeights[idx] || 0
-    }
-
-    const parentCell = trsArr[pr]?.['a:tc']?.[pc]
-    const gridSpan = parentCell?.['@_gridSpan'] ? parseInt(parentCell['@_gridSpan'], 10) : 1
-    const rowSpan = parentCell?.['@_rowSpan'] ? parseInt(parentCell['@_rowSpan'], 10) : 1
-
-    let cellWidth = 0
-    for (let idx = 0; idx < gridSpan; idx++) {
-      cellWidth += colWidths[pc + idx] || 0
-    }
-
-    let cellHeight = 0
-    for (let idx = 0; idx < rowSpan; idx++) {
-      cellHeight += rowHeights[pr + idx] || 0
+    const bounds = this.getCellBounds(slideIndex, tableId, rowIndex, colIndex, slideManager)
+    if (!bounds) {
+      throw new PPTXError(`Could not calculate bounds for cell (${rowIndex}, ${colIndex})`)
     }
 
-    const bounds = { left: cellLeft, top: cellTop, width: cellWidth, height: cellHeight }
-
     const expandedConfigs = this.#expandCellShape(options, bounds)
 
     expandedConfigs.forEach((expandedConfig, expIdx) => {
@@ -2158,7 +2034,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 +2048,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 +2171,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 +2224,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

package/src/managers/ZipManager.js

@@ -646,7 +646,11 @@ class ZipManager {
           await fs.writeFile(targetPath, this.#dirtyBinaryFiles.get(relPath))
         } else {
           const srcPath = path.join(this.#folderRoot, relPath)
-          await fs.copy(srcPath, targetPath)
+          const resolvedSrc = path.resolve(srcPath)
+          const resolvedDest = path.resolve(targetPath)
+          if (resolvedSrc !== resolvedDest) {
+            await fs.copy(srcPath, targetPath)
+          }
         }
       }
     } else {