@cj-tech-master/excelts 4.0.4-canary.20260109045014.00b0344 → 4.0.4-canary.20260109050555.4f97ebb

package/dist/browser/modules/excel/table.d.ts

@@ -52,8 +52,12 @@ declare class Table {
     set model(value: TableModel);
     cacheState(): void;
     commit(): void;
-    addRow(values: CellValue[], rowNumber?: number): void;
-    removeRows(rowIndex: number, count?: number): void;
+    addRow(values: CellValue[], rowNumber?: number, options?: {
+        commit?: boolean;
+    }): void;
+    removeRows(rowIndex: number, count?: number, options?: {
+        commit?: boolean;
+    }): void;
     getColumn(colIndex: number): Column;
     addColumn(column: TableColumnProperties, values: CellValue[], colIndex?: number): void;
     removeColumns(colIndex: number, count?: number): void;

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

@@ -57,6 +57,26 @@ class Table {
         this.worksheet = worksheet;
         if (table) {
             this.table = table;
+            // When loading tables from xlsx, Excel stores table ranges and cell values in the worksheet,
+            // but may not embed row data into the table definition. Hydrate rows from the worksheet so
+            // table mutations (e.g. addRow) can correctly expand table ranges and serialize.
+            if (Array.isArray(table.rows) && table.rows.length === 0 && table.tableRef) {
+                const decoded = colCache.decode(table.tableRef);
+                if ("dimensions" in decoded) {
+                    const startRow = decoded.top + (table.headerRow === false ? 0 : 1);
+                    const endRow = decoded.bottom - (table.totalsRow === true ? 1 : 0);
+                    if (endRow >= startRow) {
+                        for (let r = startRow; r <= endRow; r++) {
+                            const row = worksheet.getRow(r);
+                            const values = [];
+                            for (let c = decoded.left; c <= decoded.right; c++) {
+                                values.push(row.getCell(c).value);
+                            }
+                            table.rows.push(values);
+                        }
+                    }
+                }
+            }
             // check things are ok first
             this.validate();
             this.store();
@@ -133,9 +153,10 @@ class Table {
         const { row, col } = table.tl;
         assert(row > 0, "Table must be on valid row");
         assert(col > 0, "Table must be on valid col");
-        const { width, filterHeight, tableHeight } = this;
-        // autoFilterRef is a range that includes optional headers only
-        table.autoFilterRef = colCache.encode(row, col, row + filterHeight - 1, col + width - 1);
+        const { width, tableHeight } = this;
+        // autoFilterRef is a single-row range that targets the header row only.
+        // Excel uses this for filter buttons; including data rows can break filter rendering.
+        table.autoFilterRef = colCache.encode(row, col, row, col + width - 1);
         // tableRef is a range that includes optional headers and totals
         table.tableRef = colCache.encode(row, col, row + tableHeight - 1, col + width - 1);
         table.columns.forEach((column, i) => {
@@ -304,8 +325,9 @@ class Table {
             }
         }
         this.store();
+        this._cache = undefined;
     }
-    addRow(values, rowNumber) {
+    addRow(values, rowNumber, options) {
         // Add a row of data, either insert at rowNumber or append
         this.cacheState();
         if (rowNumber === undefined) {
@@ -314,11 +336,17 @@ class Table {
         else {
             this.table.rows.splice(rowNumber, 0, values);
         }
+        if (options?.commit !== false) {
+            this.commit();
+        }
     }
-    removeRows(rowIndex, count = 1) {
+    removeRows(rowIndex, count = 1, options) {
         // Remove a rows of data
         this.cacheState();
         this.table.rows.splice(rowIndex, count);
+        if (options?.commit !== false) {
+            this.commit();
+        }
     }
     getColumn(colIndex) {
         const column = this.table.columns[colIndex];

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

@@ -60,6 +60,26 @@ class Table {
         this.worksheet = worksheet;
         if (table) {
             this.table = table;
+            // When loading tables from xlsx, Excel stores table ranges and cell values in the worksheet,
+            // but may not embed row data into the table definition. Hydrate rows from the worksheet so
+            // table mutations (e.g. addRow) can correctly expand table ranges and serialize.
+            if (Array.isArray(table.rows) && table.rows.length === 0 && table.tableRef) {
+                const decoded = col_cache_1.colCache.decode(table.tableRef);
+                if ("dimensions" in decoded) {
+                    const startRow = decoded.top + (table.headerRow === false ? 0 : 1);
+                    const endRow = decoded.bottom - (table.totalsRow === true ? 1 : 0);
+                    if (endRow >= startRow) {
+                        for (let r = startRow; r <= endRow; r++) {
+                            const row = worksheet.getRow(r);
+                            const values = [];
+                            for (let c = decoded.left; c <= decoded.right; c++) {
+                                values.push(row.getCell(c).value);
+                            }
+                            table.rows.push(values);
+                        }
+                    }
+                }
+            }
             // check things are ok first
             this.validate();
             this.store();
@@ -136,9 +156,10 @@ class Table {
         const { row, col } = table.tl;
         assert(row > 0, "Table must be on valid row");
         assert(col > 0, "Table must be on valid col");
-        const { width, filterHeight, tableHeight } = this;
-        // autoFilterRef is a range that includes optional headers only
-        table.autoFilterRef = col_cache_1.colCache.encode(row, col, row + filterHeight - 1, col + width - 1);
+        const { width, tableHeight } = this;
+        // autoFilterRef is a single-row range that targets the header row only.
+        // Excel uses this for filter buttons; including data rows can break filter rendering.
+        table.autoFilterRef = col_cache_1.colCache.encode(row, col, row, col + width - 1);
         // tableRef is a range that includes optional headers and totals
         table.tableRef = col_cache_1.colCache.encode(row, col, row + tableHeight - 1, col + width - 1);
         table.columns.forEach((column, i) => {
@@ -307,8 +328,9 @@ class Table {
             }
         }
         this.store();
+        this._cache = undefined;
     }
-    addRow(values, rowNumber) {
+    addRow(values, rowNumber, options) {
         // Add a row of data, either insert at rowNumber or append
         this.cacheState();
         if (rowNumber === undefined) {
@@ -317,11 +339,17 @@ class Table {
         else {
             this.table.rows.splice(rowNumber, 0, values);
         }
+        if (options?.commit !== false) {
+            this.commit();
+        }
     }
-    removeRows(rowIndex, count = 1) {
+    removeRows(rowIndex, count = 1, options) {
         // Remove a rows of data
         this.cacheState();
         this.table.rows.splice(rowIndex, count);
+        if (options?.commit !== false) {
+            this.commit();
+        }
     }
     getColumn(colIndex) {
         const column = this.table.columns[colIndex];

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

@@ -57,6 +57,26 @@ class Table {
         this.worksheet = worksheet;
         if (table) {
             this.table = table;
+            // When loading tables from xlsx, Excel stores table ranges and cell values in the worksheet,
+            // but may not embed row data into the table definition. Hydrate rows from the worksheet so
+            // table mutations (e.g. addRow) can correctly expand table ranges and serialize.
+            if (Array.isArray(table.rows) && table.rows.length === 0 && table.tableRef) {
+                const decoded = colCache.decode(table.tableRef);
+                if ("dimensions" in decoded) {
+                    const startRow = decoded.top + (table.headerRow === false ? 0 : 1);
+                    const endRow = decoded.bottom - (table.totalsRow === true ? 1 : 0);
+                    if (endRow >= startRow) {
+                        for (let r = startRow; r <= endRow; r++) {
+                            const row = worksheet.getRow(r);
+                            const values = [];
+                            for (let c = decoded.left; c <= decoded.right; c++) {
+                                values.push(row.getCell(c).value);
+                            }
+                            table.rows.push(values);
+                        }
+                    }
+                }
+            }
             // check things are ok first
             this.validate();
             this.store();
@@ -133,9 +153,10 @@ class Table {
         const { row, col } = table.tl;
         assert(row > 0, "Table must be on valid row");
         assert(col > 0, "Table must be on valid col");
-        const { width, filterHeight, tableHeight } = this;
-        // autoFilterRef is a range that includes optional headers only
-        table.autoFilterRef = colCache.encode(row, col, row + filterHeight - 1, col + width - 1);
+        const { width, tableHeight } = this;
+        // autoFilterRef is a single-row range that targets the header row only.
+        // Excel uses this for filter buttons; including data rows can break filter rendering.
+        table.autoFilterRef = colCache.encode(row, col, row, col + width - 1);
         // tableRef is a range that includes optional headers and totals
         table.tableRef = colCache.encode(row, col, row + tableHeight - 1, col + width - 1);
         table.columns.forEach((column, i) => {
@@ -304,8 +325,9 @@ class Table {
             }
         }
         this.store();
+        this._cache = undefined;
     }
-    addRow(values, rowNumber) {
+    addRow(values, rowNumber, options) {
         // Add a row of data, either insert at rowNumber or append
         this.cacheState();
         if (rowNumber === undefined) {
@@ -314,11 +336,17 @@ class Table {
         else {
             this.table.rows.splice(rowNumber, 0, values);
         }
+        if (options?.commit !== false) {
+            this.commit();
+        }
     }
-    removeRows(rowIndex, count = 1) {
+    removeRows(rowIndex, count = 1, options) {
         // Remove a rows of data
         this.cacheState();
         this.table.rows.splice(rowIndex, count);
+        if (options?.commit !== false) {
+            this.commit();
+        }
     }
     getColumn(colIndex) {
         const column = this.table.columns[colIndex];

package/dist/iife/excelts.iife.js

@@ -1,5 +1,5 @@
 /*!
- * @cj-tech-master/excelts v4.0.4-canary.20260109045014.00b0344
+ * @cj-tech-master/excelts v4.0.4-canary.20260109050555.4f97ebb
  * TypeScript Excel Workbook Manager - Read and Write xlsx and csv Files.
  * (c) 2026 cjnoname
  * Released under the MIT License
@@ -2309,6 +2309,19 @@ var ExcelTS = (function(exports) {
 			this.worksheet = worksheet;
 			if (table) {
 				this.table = table;
+				if (Array.isArray(table.rows) && table.rows.length === 0 && table.tableRef) {
+					const decoded = colCache.decode(table.tableRef);
+					if ("dimensions" in decoded) {
+						const startRow = decoded.top + (table.headerRow === false ? 0 : 1);
+						const endRow = decoded.bottom - (table.totalsRow === true ? 1 : 0);
+						if (endRow >= startRow) for (let r = startRow; r <= endRow; r++) {
+							const row = worksheet.getRow(r);
+							const values = [];
+							for (let c = decoded.left; c <= decoded.right; c++) values.push(row.getCell(c).value);
+							table.rows.push(values);
+						}
+					}
+				}
 				this.validate();
 				this.store();
 			}
@@ -2363,8 +2376,8 @@ var ExcelTS = (function(exports) {
 			const { row, col } = table.tl;
 			assert(row > 0, "Table must be on valid row");
 			assert(col > 0, "Table must be on valid col");
-			const { width, filterHeight, tableHeight } = this;
-			table.autoFilterRef = colCache.encode(row, col, row + filterHeight - 1, col + width - 1);
+			const { width, tableHeight } = this;
+			table.autoFilterRef = colCache.encode(row, col, row, col + width - 1);
 			table.tableRef = colCache.encode(row, col, row + tableHeight - 1, col + width - 1);
 			table.columns.forEach((column, i) => {
 				assert(!!column.name, `Column ${i} must have a name`);
@@ -2491,15 +2504,18 @@ var ExcelTS = (function(exports) {
 				}
 			}
 			this.store();
+			this._cache = void 0;
 		}
-		addRow(values, rowNumber) {
+		addRow(values, rowNumber, options) {
 			this.cacheState();
 			if (rowNumber === void 0) this.table.rows.push(values);
 			else this.table.rows.splice(rowNumber, 0, values);
+			if (options?.commit !== false) this.commit();
 		}
-		removeRows(rowIndex, count = 1) {
+		removeRows(rowIndex, count = 1, options) {
 			this.cacheState();
 			this.table.rows.splice(rowIndex, count);
+			if (options?.commit !== false) this.commit();
 		}
 		getColumn(colIndex) {
 			const column = this.table.columns[colIndex];