node-pptx-templater 1.1.4 → 1.1.6

package/CHANGELOG.md

@@ -5,22 +5,6 @@ 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,51 +172,27 @@ 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.
-
-#### 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).
+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.
 
 ```javascript
-// Add a status icon centered in a merged cell (rowSpan=2, colSpan=2)
-await ppt.addCellShape('StatusTable', 1, 1, {
+// Add a simple indicator
+await ppt.addCellShape('SalesTable', 2, 1, {
   type: 'circle',
   width: 12,
   height: 12,
-  fill: '#10B981', // Green status dot
-  alignX: 'center',
-  alignY: 'middle' // Visually centered in the merged cell
+  fill: '#10B981' // Green status dot
 });
 
-// Add a badge with text and explicit offsets inside a merged cell
-await ppt.addCellShape('StatusTable', 1, 1, {
+// Add a badge with text and custom offsets
+await ppt.addCellShape('SalesTable', 1, 2, {
   type: 'badge',
-  text: 'Urgent',
-  fill: '#EF4444',
-  alignX: 'left',
-  alignY: 'top',
-  x: 5,
-  y: 3,
+  text: 'Active',
+  fill: '#3B82F6',
+  x: 4,
+  y: 2,
   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.4",
+  "version": "1.1.6",
   "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,8 +37,6 @@
     "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:shape-cells": "node examples/shape-cells.js",
-    "example:all": "node scripts/run-examples-check.js",
     "prepublishOnly": "npm run lint && npm run test"
   },
   "keywords": [

package/src/core/PPTXTemplater.js

@@ -473,10 +473,14 @@ class PPTXTemplater {
         throw new PPTXError(`Destination is a file: ${outputPath}`)
       }
       const files = fs.readdirSync(resolvedOut)
-      if (files.length > 0 && !options.overwrite) {
-        throw new PPTXError(
-          `Destination directory "${outputPath}" is not empty. Set overwrite: true to overwrite.`
-        )
+      if (files.length > 0) {
+        if (!options.overwrite) {
+          throw new PPTXError(
+            `Destination directory "${outputPath}" is not empty. Set overwrite: true to overwrite.`
+          )
+        } else {
+          await fs.emptyDir(resolvedOut)
+        }
       }
     } else {
       await fs.ensureDir(resolvedOut)
@@ -1569,30 +1573,11 @@ class PPTXTemplater {
   }
 
   /**
-   * Appends one or more rows to a table. Supports flat arrays, nested arrays
-   * for rowspan-merged cells, and **Shape Cell** configuration objects that insert
-   * a graphic indicator directly inside a cell.
-   *
-   * ### Shape Cell Object
-   * Instead of a string value you may pass an object with the following keys:
-   *
-   * | Key | Type | Default | Description |
-   * |-----|------|---------|-------------|
-   * | `type` | `string` | **required** | Shape preset: `'circle'`, `'square'`, `'rectangle'`, `'triangle'`, `'diamond'`, `'hexagon'`, `'line'`. |
-   * | `text` | `string` | `''` | Optional label rendered next to the shape. |
-   * | `color` | `string` | `'#4CAF50'` | Fill colour of the shape (hex). |
-   * | `width` | `number` | `14` | Shape width in points. |
-   * | `height` | `number` | `14` | Shape height in points. |
-   * | `position` | `string` | `'center'` | Alignment inside the cell: `'center'`, `'left'`, `'right'`, `'top'`, `'bottom'`, `'top-left'`, `'top-right'`, `'bottom-left'`, `'bottom-right'`. |
-   * | `offsetX` | `number` | `0` | Additional horizontal offset from the resolved anchor (pt). |
-   * | `offsetY` | `number` | `0` | Additional vertical offset from the resolved anchor (pt). |
-   *
-   * The shape is added as a floating overlay anchored to the cell; the table
-   * dimensions (row heights, column widths) are **never mutated**.
+   * Appends one or more rows to a table. Supports flat arrays and nested arrays
+   * for rowspan-merged cells.
    *
    * @param {string} tableId - Table name or shape ID.
-   * @param {Array<string|Array<string>|Object>} rowData - Row data. Each element may be
-   *   a plain string, a nested array (rowspan), or a Shape Cell configuration object.
+   * @param {Array<string|Array<string>>} rowData - Row data. Nested arrays create rowspan cells.
    * @param {Object} [options] - Row insertion options.
    * @param {'rowspan'|'auto'|'none'} [options.mergeStrategy='rowspan'] - How to handle nested arrays.
    *   `'rowspan'` creates OpenXML vertical spans, `'auto'` merges identical adjacent values,
@@ -1605,20 +1590,6 @@ class PPTXTemplater {
    *
    * // Nested row with rowspan
    * ppt.addTableRow('SalesTable', ['Region', ['Q1', 'Q2'], '$5K'], { mergeStrategy: 'rowspan' });
-   *
-   * // KPI indicator – green circle placed in the cell centre, no label
-   * await ppt.addTableRow('StatusTable', [
-   *   'Alice',
-   *   'Engineering',
-   *   { type: 'circle', color: '#4CAF50', width: 12, height: 12, position: 'center' },
-   * ]);
-   *
-   * // Status badge – shape with accompanying text label, aligned to the left
-   * await ppt.addTableRow('StatusTable', [
-   *   'Bob',
-   *   'Design',
-   *   { type: 'circle', color: '#F44336', text: 'Blocked', position: 'left' },
-   * ]);
    */
   addTableRow(tableId, rowData, options = {}) {
     this.#assertLoaded()

package/src/managers/ShapeManager.js

@@ -433,6 +433,14 @@ class ShapeManager {
     if (!options) return options
     const normalized = { ...options }
 
+    // Fallback mapping between id and name for compatibility
+    if (normalized.id === undefined && normalized.name !== undefined) {
+      normalized.id = normalized.name
+    }
+    if (normalized.name === undefined && normalized.id !== undefined) {
+      normalized.name = normalized.id
+    }
+
     // 1. Map color alias to fill
     const fillVal = options.fill !== undefined ? options.fill : options.color
     if (fillVal !== undefined) {

package/src/managers/TableManager.js

@@ -1719,40 +1719,52 @@ class TableManager {
     }
 
     // 2. Determine alignment settings
-    let alignX = undefined
-    let alignY = undefined
-
-    const pos = String(config.position || '')
-      .toLowerCase()
-      .trim()
-    if (pos) {
-      if (pos === 'left') {
-        alignX = 'left'
-        alignY = 'middle'
-      } else if (pos === 'right') {
-        alignX = 'right'
-        alignY = 'middle'
-      } else if (pos === 'center' || pos === 'middle') {
-        alignX = 'center'
-        alignY = 'middle'
-      } else if (pos === 'top') {
-        alignX = 'center'
-        alignY = 'top'
-      } else if (pos === 'bottom') {
-        alignX = 'center'
-        alignY = 'bottom'
-      } else if (pos === 'top-left') {
-        alignX = 'left'
-        alignY = 'top'
-      } else if (pos === 'top-right') {
-        alignX = 'right'
-        alignY = 'top'
-      } else if (pos === 'bottom-left') {
-        alignX = 'left'
-        alignY = 'bottom'
-      } else if (pos === 'bottom-right') {
-        alignX = 'right'
-        alignY = 'bottom'
+    let alignX = config.alignX
+    let alignY = config.alignY
+
+    if (config.position) {
+      switch (config.position) {
+        case 'top-left':
+          if (!alignX) alignX = 'left'
+          if (!alignY) alignY = 'top'
+          break
+        case 'top-center':
+        case 'top':
+          if (!alignX) alignX = 'center'
+          if (!alignY) alignY = 'top'
+          break
+        case 'top-right':
+          if (!alignX) alignX = 'right'
+          if (!alignY) alignY = 'top'
+          break
+        case 'middle-left':
+        case 'left':
+          if (!alignX) alignX = 'left'
+          if (!alignY) alignY = 'middle'
+          break
+        case 'center':
+        case 'middle-center':
+          if (!alignX) alignX = 'center'
+          if (!alignY) alignY = 'middle'
+          break
+        case 'middle-right':
+        case 'right':
+          if (!alignX) alignX = 'right'
+          if (!alignY) alignY = 'middle'
+          break
+        case 'bottom-left':
+          if (!alignX) alignX = 'left'
+          if (!alignY) alignY = 'bottom'
+          break
+        case 'bottom-center':
+        case 'bottom':
+          if (!alignX) alignX = 'center'
+          if (!alignY) alignY = 'bottom'
+          break
+        case 'bottom-right':
+          if (!alignX) alignX = 'right'
+          if (!alignY) alignY = 'bottom'
+          break
       }
     }
 
@@ -2057,7 +2069,7 @@ class TableManager {
     slideManager,
     shapeManager,
     tblObj,
-    _frameObj
+    frameObj
   ) {
     if (!cellShapes || !shapeManager) return
 
@@ -2075,6 +2087,54 @@ 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 = (tblObj['a:tr']?.[0]?.['a:tc'] || []).map(cell =>
       this.#getCellText(cell).trim()
@@ -2204,7 +2264,45 @@ class TableManager {
   }
 
   addCellShape(slideIndex, tableId, rowIndex, colIndex, options, slideManager, shapeManager) {
-    const { resolvedTableId } = this.#getTableContext(slideIndex, tableId, slideManager)
+    const { tblObj, frameObj, resolvedTableId } = this.#getTableContext(
+      slideIndex,
+      tableId,
+      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
+    }
 
     const bounds = this.getCellBounds(slideIndex, tableId, rowIndex, colIndex, slideManager)
     if (!bounds) {
@@ -2259,7 +2357,11 @@ class TableManager {
     slideManager,
     shapeManager
   ) {
-    const { resolvedTableId } = this.#getTableContext(slideIndex, tableId, slideManager)
+    const { tblObj, frameObj, resolvedTableId } = this.#getTableContext(
+      slideIndex,
+      tableId,
+      slideManager
+    )
 
     const shapes = shapeManager.getShapes(slideIndex, slideManager)
     const prefix = `cellshape_${resolvedTableId}_${rowIndex}_${colIndex}_${shapeIndex}`