@cj-tech-master/excelts 5.1.1 → 5.1.2-canary.20260211105222.3da3461

package/dist/browser/modules/excel/cell.js

@@ -147,7 +147,7 @@ class Cell {
         if (this.type === Cell.Types.Merge) {
             this._value.release();
             this._value = Value.create(Cell.Types.Null, this);
-            this.style = this._mergeStyle(this._row.style, this._column.style, {});
+            this.style = this._mergeStyle(this._row.style, this._column.style, { ...this.style });
         }
     }
     isMergedTo(master) {

package/dist/browser/modules/excel/xlsx/xform/sheet/worksheet-xform.js

@@ -188,16 +188,29 @@ class WorkSheetXform extends BaseXform {
                 vmlDrawing: `vmlDrawing${model.id}`
             });
         }
-        // Handle pre-loaded drawing (from file read) that may contain charts or other non-image content
-        // This preserves drawings through round-trip even if they don't contain images
-        // Note: Always assign a new rId since rels are rebuilt during write
+        // Handle pre-loaded drawing (from file read) that may contain charts or other non-image content.
+        // Reset anchors and rels so they are rebuilt cleanly from model.media (images) and
+        // model.formControls (shapes) below.  Without this reset, every read-write cycle would
+        // duplicate image anchors because the same images exist in both model.drawing.anchors
+        // (preserved for round-trip) and model.media (the canonical image list).
+        // For chart drawings, rels are preserved because the raw XML passthrough references
+        // original rIds; anchors are still cleared since they are unused for chart drawings.
         if (model.drawing && model.drawing.anchors) {
-            // This is a loaded drawing that needs to be added to relationships
             const drawing = model.drawing;
             drawing.rId = nextRid(rels);
             if (!drawing.name) {
                 drawing.name = `drawing${++options.drawingsCount}`;
             }
+            const hasChartRels = (drawing.rels ?? []).some((rel) => rel.Target && rel.Target.includes("/charts/"));
+            // Anchors are always reset: for chart drawings they are unused (raw XML passthrough),
+            // for normal drawings they are rebuilt from model.media below.
+            drawing.anchors = [];
+            if (!hasChartRels) {
+                // Non-chart drawings: clear rels so image rels are rebuilt from scratch.
+                drawing.rels = [];
+            }
+            // Chart drawings keep their original rels intact since the raw drawing XML
+            // references those rIds directly.
             options.drawings.push(drawing);
             rels.push({
                 Id: drawing.rId,

package/dist/cjs/modules/excel/cell.js

@@ -150,7 +150,7 @@ class Cell {
         if (this.type === Cell.Types.Merge) {
             this._value.release();
             this._value = Value.create(Cell.Types.Null, this);
-            this.style = this._mergeStyle(this._row.style, this._column.style, {});
+            this.style = this._mergeStyle(this._row.style, this._column.style, { ...this.style });
         }
     }
     isMergedTo(master) {

package/dist/cjs/modules/excel/xlsx/xform/sheet/worksheet-xform.js

@@ -191,16 +191,29 @@ class WorkSheetXform extends base_xform_1.BaseXform {
                 vmlDrawing: `vmlDrawing${model.id}`
             });
         }
-        // Handle pre-loaded drawing (from file read) that may contain charts or other non-image content
-        // This preserves drawings through round-trip even if they don't contain images
-        // Note: Always assign a new rId since rels are rebuilt during write
+        // Handle pre-loaded drawing (from file read) that may contain charts or other non-image content.
+        // Reset anchors and rels so they are rebuilt cleanly from model.media (images) and
+        // model.formControls (shapes) below.  Without this reset, every read-write cycle would
+        // duplicate image anchors because the same images exist in both model.drawing.anchors
+        // (preserved for round-trip) and model.media (the canonical image list).
+        // For chart drawings, rels are preserved because the raw XML passthrough references
+        // original rIds; anchors are still cleared since they are unused for chart drawings.
         if (model.drawing && model.drawing.anchors) {
-            // This is a loaded drawing that needs to be added to relationships
             const drawing = model.drawing;
             drawing.rId = nextRid(rels);
             if (!drawing.name) {
                 drawing.name = `drawing${++options.drawingsCount}`;
             }
+            const hasChartRels = (drawing.rels ?? []).some((rel) => rel.Target && rel.Target.includes("/charts/"));
+            // Anchors are always reset: for chart drawings they are unused (raw XML passthrough),
+            // for normal drawings they are rebuilt from model.media below.
+            drawing.anchors = [];
+            if (!hasChartRels) {
+                // Non-chart drawings: clear rels so image rels are rebuilt from scratch.
+                drawing.rels = [];
+            }
+            // Chart drawings keep their original rels intact since the raw drawing XML
+            // references those rIds directly.
             options.drawings.push(drawing);
             rels.push({
                 Id: drawing.rId,

package/dist/esm/modules/excel/cell.js

@@ -147,7 +147,7 @@ class Cell {
         if (this.type === Cell.Types.Merge) {
             this._value.release();
             this._value = Value.create(Cell.Types.Null, this);
-            this.style = this._mergeStyle(this._row.style, this._column.style, {});
+            this.style = this._mergeStyle(this._row.style, this._column.style, { ...this.style });
         }
     }
     isMergedTo(master) {

package/dist/esm/modules/excel/xlsx/xform/sheet/worksheet-xform.js

@@ -188,16 +188,29 @@ class WorkSheetXform extends BaseXform {
                 vmlDrawing: `vmlDrawing${model.id}`
             });
         }
-        // Handle pre-loaded drawing (from file read) that may contain charts or other non-image content
-        // This preserves drawings through round-trip even if they don't contain images
-        // Note: Always assign a new rId since rels are rebuilt during write
+        // Handle pre-loaded drawing (from file read) that may contain charts or other non-image content.
+        // Reset anchors and rels so they are rebuilt cleanly from model.media (images) and
+        // model.formControls (shapes) below.  Without this reset, every read-write cycle would
+        // duplicate image anchors because the same images exist in both model.drawing.anchors
+        // (preserved for round-trip) and model.media (the canonical image list).
+        // For chart drawings, rels are preserved because the raw XML passthrough references
+        // original rIds; anchors are still cleared since they are unused for chart drawings.
         if (model.drawing && model.drawing.anchors) {
-            // This is a loaded drawing that needs to be added to relationships
             const drawing = model.drawing;
             drawing.rId = nextRid(rels);
             if (!drawing.name) {
                 drawing.name = `drawing${++options.drawingsCount}`;
             }
+            const hasChartRels = (drawing.rels ?? []).some((rel) => rel.Target && rel.Target.includes("/charts/"));
+            // Anchors are always reset: for chart drawings they are unused (raw XML passthrough),
+            // for normal drawings they are rebuilt from model.media below.
+            drawing.anchors = [];
+            if (!hasChartRels) {
+                // Non-chart drawings: clear rels so image rels are rebuilt from scratch.
+                drawing.rels = [];
+            }
+            // Chart drawings keep their original rels intact since the raw drawing XML
+            // references those rIds directly.
             options.drawings.push(drawing);
             rels.push({
                 Id: drawing.rId,

package/dist/iife/excelts.iife.js

@@ -1,5 +1,5 @@
 /*!
- * @cj-tech-master/excelts v5.1.1
+ * @cj-tech-master/excelts v5.1.2-canary.20260211105222.3da3461
  * TypeScript Excel Workbook Manager - Read and Write xlsx and csv Files.
  * (c) 2026 cjnoname
  * Released under the MIT License
@@ -768,7 +768,7 @@ var ExcelTS = (function(exports) {
 			if (this.type === Cell.Types.Merge) {
 				this._value.release();
 				this._value = Value.create(Cell.Types.Null, this);
-				this.style = this._mergeStyle(this._row.style, this._column.style, {});
+				this.style = this._mergeStyle(this._row.style, this._column.style, { ...this.style });
 			}
 		}
 		isMergedTo(master) {
@@ -14219,6 +14219,9 @@ var ExcelTS = (function(exports) {
 				const drawing = model.drawing;
 				drawing.rId = nextRid(rels);
 				if (!drawing.name) drawing.name = `drawing${++options.drawingsCount}`;
+				const hasChartRels = (drawing.rels ?? []).some((rel) => rel.Target && rel.Target.includes("/charts/"));
+				drawing.anchors = [];
+				if (!hasChartRels) drawing.rels = [];
 				options.drawings.push(drawing);
 				rels.push({
 					Id: drawing.rId,