node-pptx-templater 2.0.0-alpha.0 → 2.0.0-alpha.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-pptx-templater",
-  "version": "2.0.0-alpha.0",
+  "version": "2.0.0-alpha.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",
   "module": "./src/index.mjs",
@@ -209,4 +209,4 @@
   "publishConfig": {
     "access": "public"
   }
-}
+}

package/src/core/OutputWriter.js

@@ -25,6 +25,8 @@ class OutputWriter {
   #zipManager
   /** @private @type {ContentTypesManager} */
   #contentTypesManager
+  /** @private @type {RelationshipManager} */
+  #relationshipManager
 
   /** @type {boolean} */
   debugZip = false
@@ -32,10 +34,12 @@ class OutputWriter {
   /**
    * @param {ZipManager} zipManager
    * @param {ContentTypesManager} contentTypesManager
+   * @param {RelationshipManager} [relationshipManager]
    */
-  constructor(zipManager, contentTypesManager) {
+  constructor(zipManager, contentTypesManager, relationshipManager = null) {
     this.#zipManager = zipManager
     this.#contentTypesManager = contentTypesManager
+    this.#relationshipManager = relationshipManager
   }
 
   /**
@@ -71,9 +75,18 @@ class OutputWriter {
     if (slideManager && typeof slideManager.flush === 'function') {
       slideManager.flush()
     }
+    // Complete all pending async mutations (slide duplication, chart writes, etc.)
+    // BEFORE structural normalization so sldIdLst/rels match the final slide set.
+    await zipManager.waitForPendingWrites()
+    if (
+      slideManager &&
+      typeof slideManager.normalizeStructure === 'function' &&
+      this.#relationshipManager
+    ) {
+      await slideManager.normalizeStructure(this.#relationshipManager, this.#contentTypesManager)
+    }
     await this.#flushAllSlides(slideManager, zipManager)
     this.#contentTypesManager.flush(zipManager)
-    await zipManager.waitForPendingWrites()
   }
 
   /**

package/src/core/PPTXTemplater.js

@@ -186,7 +186,11 @@ class PPTXTemplater {
     this.#textManager = new TextManager(this.#xmlParser)
     this.#templateEngine = new TemplateEngine(this.#xmlParser)
     this.#zOrderManager = new ZOrderManager(this.#xmlParser)
-    this.#outputWriter = new OutputWriter(this.#zipManager, this.#contentTypesManager)
+    this.#outputWriter = new OutputWriter(
+      this.#zipManager,
+      this.#contentTypesManager,
+      this.#relationshipManager
+    )
 
     this.#profiler = {
       enabled: false,
@@ -966,8 +970,14 @@ class PPTXTemplater {
    */
   cloneSlide(sourceSlideNumber, atPosition) {
     this.#assertLoaded()
-    this.#slideManager.cloneSlide(sourceSlideNumber, atPosition, this.#relationshipManager)
-    return this
+    const promise = this.#slideManager.cloneSlide(
+      sourceSlideNumber,
+      atPosition,
+      this.#relationshipManager,
+      this.#mediaManager
+    )
+    this.#zipManager.addPendingPromise(promise)
+    return promise.then(() => this)
   }
 
   /**
@@ -1467,8 +1477,14 @@ class PPTXTemplater {
    */
   duplicateSlide(slideIndex, atPosition) {
     this.#assertLoaded()
-    this.#slideManager.duplicateSlide(slideIndex, atPosition, this.#relationshipManager)
-    return this
+    const promise = this.#slideManager.duplicateSlide(
+      slideIndex,
+      atPosition,
+      this.#relationshipManager,
+      this.#mediaManager
+    )
+    this.#zipManager.addPendingPromise(promise)
+    return promise.then(() => this)
   }
 
   /**

package/src/core/ValidationEngine.js

@@ -38,6 +38,11 @@ class ValidationEngine {
     errors.push(...presRelResult.errors.map(e => `Presentation relationship error: ${e}`))
     warnings.push(...presRelResult.warnings.map(w => `Presentation relationship warning: ${w}`))
 
+    // 3. Verify sldIdLst r:id values resolve to slide relationships in presentation.xml.rels
+    const sldIdResult = this.validatePresentationSlideIds(ppt)
+    errors.push(...sldIdResult.errors)
+    warnings.push(...sldIdResult.warnings)
+
     return {
       valid: errors.length === 0,
       errors,
@@ -206,6 +211,41 @@ class ValidationEngine {
     }
   }
 
+  /**
+   * Ensures each p:sldId entry references a valid slide relationship in presentation.xml.rels.
+   *
+   * @param {PPTXTemplater} ppt
+   * @returns {{ valid: boolean, errors: string[], warnings: string[] }}
+   */
+  static validatePresentationSlideIds(ppt) {
+    const errors = []
+    const warnings = []
+
+    const slides = ppt.slideManager.getAllSlideInfo()
+    if (slides.length === 0) {
+      return { valid: true, errors, warnings }
+    }
+
+    const presRels = ppt.relationshipManager
+      .getRelationships('ppt/presentation.xml')
+      .filter(r => r.type.endsWith('/slide'))
+    const slideRelIds = new Set(presRels.map(r => r.id))
+
+    for (const slide of slides) {
+      if (!slideRelIds.has(slide.relationshipId)) {
+        errors.push(
+          `Slide ${slide.index} relationshipId "${slide.relationshipId}" is not defined in presentation.xml.rels`
+        )
+      }
+    }
+
+    return {
+      valid: errors.length === 0,
+      errors,
+      warnings,
+    }
+  }
+
   /**
    * Validates relationship mappings for a specific part.
    *

package/src/managers/ContentTypesManager.js

@@ -150,6 +150,19 @@ class ContentTypesManager {
     return this.#contentTypesObj.Types.Override.some(o => o['@_PartName'] === normalizedPart)
   }
 
+  /**
+   * Gets the override content type for a specific part name, if registered.
+   *
+   * @param {string} partName - Absolute or relative path.
+   * @returns {string|null}
+   */
+  getOverrideContentType(partName) {
+    const normalizedPart = partName.startsWith('/') ? partName : `/${partName}`
+    const overrides = this.#contentTypesObj?.Types?.Override || []
+    const existing = overrides.find(o => o['@_PartName'] === normalizedPart)
+    return existing ? existing['@_ContentType'] : null
+  }
+
   /**
    * Serializes back to [Content_Types].xml and writes to ZIP.
    *

package/src/managers/MediaManager.js

@@ -234,6 +234,44 @@ class MediaManager {
     return zipPath
   }
 
+  /**
+   * Copies an existing media part to a new unique file without deduplication.
+   * Used when cloning slides so the copy does not share media relationships with the source.
+   *
+   * @param {string} sourceZipPath - Absolute ZIP path to the source media file.
+   * @returns {Promise<string>} ZIP path of the new media file.
+   */
+  async copyMediaAsNewPart(sourceZipPath) {
+    const data = await this.#zipManager.readBinaryFile(sourceZipPath)
+    if (!data) {
+      throw new PPTXError(`Cannot clone slide: media not found at ${sourceZipPath}`)
+    }
+
+    const fileName = sourceZipPath.split('/').pop()
+    const ext = fileName.includes('.') ? fileName.split('.').pop().toLowerCase() : 'png'
+    const mimeType = EXT_TO_MIME[ext] || 'application/octet-stream'
+
+    let mediaId = this.#nextMediaId
+    let zipPath = `ppt/media/image${mediaId}.${ext}`
+    while (this.#zipManager.hasFile(zipPath)) {
+      mediaId++
+      zipPath = `ppt/media/image${mediaId}.${ext}`
+    }
+    this.#nextMediaId = mediaId + 1
+
+    this.#zipManager.writeBinaryFile(zipPath, data)
+    this.#mediaRegistry.set(zipPath, {
+      zipPath,
+      hash: null,
+      mimeType,
+      size: data.length,
+    })
+    this.#registerContentType(ext, mimeType)
+
+    logger.debug(`Cloned media to new part: ${zipPath}`)
+    return zipPath
+  }
+
   /**
    * Generates the slide XML snippet for an image element.
    *

package/src/managers/RelationshipManager.js

@@ -407,6 +407,27 @@ class RelationshipManager {
     }
   }
 
+  /**
+   * Sets the relationships array for a part in memory.
+   *
+   * @param {string} partPath
+   * @param {Relationship[]} rels
+   */
+  setRelationships(partPath, rels) {
+    const key = this.#getNormalizedKey(partPath)
+    this.#relationships.set(key, rels)
+  }
+
+  /**
+   * Flushes the relationships of a part.
+   *
+   * @param {string} partPath
+   */
+  flushRelationships(partPath) {
+    const key = this.#getNormalizedKey(partPath)
+    this.#flushRels(key, partPath)
+  }
+
   /**
    * Scans all relationships and removes those pointing to missing internal targets.
    * This is part of the repair functionality.