@cj-tech-master/excelts 5.1.17 → 5.1.18

package/dist/cjs/modules/archive/unzip/stream.base.js

@@ -668,7 +668,6 @@ function streamUntilValidatedDataDescriptor(options) {
                                 if (written > 0) {
                                     source.discard(written);
                                     bytesEmitted += written;
-                                    available -= written;
                                     scanner.onConsume(written);
                                 }
                                 if (waitingDrain) {
@@ -678,7 +677,6 @@ function streamUntilValidatedDataDescriptor(options) {
                             else {
                                 const ok = output.write(source.read(idx));
                                 bytesEmitted += idx;
-                                available -= idx;
                                 scanner.onConsume(idx);
                                 if (!ok) {
                                     waitingDrain = true;
@@ -979,8 +977,17 @@ async function readFileRecord(opts, io, emitter, inflateFactory, state, threshol
             }
         }
     }
-    const extra = (0, exports.parseExtraField)(extraFieldData, vars);
-    vars.lastModifiedDateTime = resolveZipEntryLastModifiedDateTime(vars, extra);
+    const zipVars = {
+        uncompressedSize: vars.uncompressedSize ?? 0,
+        compressedSize: vars.compressedSize ?? 0
+    };
+    const extra = (0, exports.parseExtraField)(extraFieldData, zipVars);
+    vars.lastModifiedDateTime = resolveZipEntryLastModifiedDateTime({
+        flags: vars.flags,
+        uncompressedSize: zipVars.uncompressedSize,
+        lastModifiedDate: vars.lastModifiedDate,
+        lastModifiedTime: vars.lastModifiedTime
+    }, extra);
     entry.vars = vars;
     entry.extraFields = extra;
     entry.__autodraining = autodraining;

package/dist/cjs/modules/archive/unzip/stream.browser.js

@@ -179,7 +179,7 @@ class BrowserInflateRaw extends _stream_1.Duplex {
             this.writer.abort(error ?? undefined).catch(() => { });
         }
         this.reader.cancel(error ?? undefined).catch(() => { });
-        return super.destroy(error);
+        return super.destroy(error ?? undefined);
     }
 }
 // =============================================================================
@@ -398,7 +398,7 @@ class WorkerInflateRaw extends _stream_1.Duplex {
             }
         }
         this._terminateWorker();
-        return super.destroy(error);
+        return super.destroy(error ?? undefined);
     }
 }
 // =============================================================================
@@ -444,7 +444,7 @@ class FallbackInflateRaw extends _stream_1.Duplex {
     destroy(error) {
         this._finished = true;
         this.chunks = [];
-        return super.destroy(error);
+        return super.destroy(error ?? undefined);
     }
 }
 // =============================================================================

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

@@ -76,11 +76,8 @@ class Anchor {
             : 640000;
     }
     get rowHeight() {
-        return this.worksheet &&
-            this.worksheet.getRow(this.nativeRow + 1) &&
-            this.worksheet.getRow(this.nativeRow + 1).height
-            ? Math.floor(this.worksheet.getRow(this.nativeRow + 1).height * 10000)
-            : 180000;
+        const height = this.worksheet?.getRow(this.nativeRow + 1)?.height;
+        return height ? Math.floor(height * 10000) : 180000;
     }
     get model() {
         return {

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

@@ -37,11 +37,11 @@ class Cell {
     }
     // help GC by removing cyclic (and other) references
     destroy() {
-        delete this.style;
-        delete this._value;
-        delete this._row;
-        delete this._column;
-        delete this._address;
+        this.style = undefined;
+        this._value = undefined;
+        this._row = undefined;
+        this._column = undefined;
+        this._address = undefined;
     }
     // =========================================================================
     // Styles stuff
@@ -167,7 +167,7 @@ class Cell {
         if (this._value.type !== Cell.Types.Merge) {
             return false;
         }
-        return this._value.isMergedTo(master);
+        return this._value.isMergedTo ? this._value.isMergedTo(master) : false;
     }
     get master() {
         if (this.type === Cell.Types.Merge) {
@@ -249,7 +249,7 @@ class Cell {
         return this._value.result;
     }
     get formulaType() {
-        return this._value.formulaType;
+        return this._value.formulaType ?? enums_1.Enums.FormulaType.None;
     }
     // =========================================================================
     // Name stuff
@@ -611,7 +611,7 @@ class MergeValue {
         this._master.releaseMergeRef();
     }
     toString() {
-        return this.value.toString();
+        return this.value != null ? this.value.toString() : "";
     }
 }
 class FormulaValue {
@@ -744,7 +744,9 @@ class FormulaValue {
             const { worksheet } = this.cell;
             const master = worksheet.findCell(this.model.sharedFormula);
             this._translatedFormula =
-                master && (0, shared_formula_1.slideFormula)(master.formula, master.address, this.model.address);
+                master && master.formula
+                    ? (0, shared_formula_1.slideFormula)(master.formula, master.address, this.model.address)
+                    : undefined;
         }
         return this._translatedFormula;
     }

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

@@ -113,7 +113,7 @@ class Column {
         }
         this._key = value;
         if (value) {
-            this._worksheet.setColumnKey(this._key, this);
+            this._worksheet.setColumnKey(value, this);
         }
     }
     /**
@@ -138,7 +138,7 @@ class Column {
      * Indicate the collapsed state based on outlineLevel
      */
     get collapsed() {
-        return !!(this._outlineLevel && this._outlineLevel >= this._worksheet.properties.outlineLevelCol);
+        return !!(this._outlineLevel && this._outlineLevel >= (this._worksheet.properties.outlineLevelCol ?? 0));
     }
     toString() {
         return JSON.stringify({

package/dist/cjs/modules/excel/defined-names.js

@@ -110,7 +110,7 @@ class DefinedNames {
     }
     _explore(matrix, cell) {
         cell.mark = false;
-        const { sheetName } = cell;
+        const sheetName = cell.sheetName; // Always set for cells in defined names
         const range = new range_1.Range(cell.row, cell.col, cell.row, cell.col, sheetName);
         let x;
         let y;
@@ -163,7 +163,7 @@ class DefinedNames {
         });
         const ranges = matrix
             .map((cell) => cell.mark && this._explore(matrix, cell))
-            .filter(Boolean)
+            .filter((range) => Boolean(range))
             .map((range) => range.$shortRange);
         return {
             name,

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

@@ -7,7 +7,7 @@ class Note {
         this.note = note;
     }
     get model() {
-        let value = null;
+        let value;
         switch (typeof this.note) {
             case "string":
                 value = {

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

@@ -37,9 +37,9 @@ class Row {
      * Helps GC by breaking cyclic references
      */
     destroy() {
-        delete this._worksheet;
-        delete this._cells;
-        delete this.style;
+        this._worksheet = undefined;
+        this._cells = undefined;
+        this.style = undefined;
     }
     findCell(colNumber) {
         return this._cells[colNumber - 1];
@@ -361,7 +361,7 @@ class Row {
         this._outlineLevel = value;
     }
     get collapsed() {
-        return !!(this._outlineLevel && this._outlineLevel >= this._worksheet.properties.outlineLevelRow);
+        return !!(this._outlineLevel && this._outlineLevel >= (this._worksheet.properties.outlineLevelRow ?? 0));
     }
     // =========================================================================
     get model() {
@@ -427,6 +427,9 @@ class Row {
                         };
                     }
                     previousAddress = address;
+                    if (!address) {
+                        break;
+                    }
                     const cell = this.getCellEx(address);
                     cell.model = cellModel;
                     break;

package/dist/cjs/modules/excel/stream/worksheet-reader.js

@@ -270,7 +270,10 @@ class WorksheetReader extends _stream_1.EventEmitter {
                                 break;
                             case "row":
                                 if (row) {
-                                    this._dimensions.expandRow(row);
+                                    this._dimensions.expandRow({
+                                        number: row.number,
+                                        dimensions: row.dimensions ?? undefined
+                                    });
                                     (worksheetEvents || (worksheetEvents = [])).push({ eventType: "row", value: row });
                                 }
                                 row = null;

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

@@ -1,7 +1,86 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.Table = void 0;
+exports.sanitizeTableName = sanitizeTableName;
 const col_cache_1 = require("./utils/col-cache.js");
+/**
+ * Maximum length for an Excel defined name (and therefore table name).
+ */
+const MAX_TABLE_NAME_LENGTH = 255;
+/**
+ * Matches an A1-style cell reference pattern like A1, Z99, XFD1048576.
+ * Excel rejects table names that match this pattern.
+ */
+const CELL_REF_PATTERN = /^[A-Za-z]{1,3}\d+$/;
+/**
+ * Matches an R1C1-style cell reference, e.g. R1C1, R100C200.
+ * Must have at least one digit after R and at least one digit after C
+ * to be considered a cell reference. Bare "RC" is NOT a cell reference.
+ */
+const R1C1_PATTERN = /^[Rr]\d+[Cc]\d+$/;
+/**
+ * Single-character names that Excel reserves for row/column navigation.
+ * Per Microsoft docs: "You cannot use the uppercase and lowercase characters
+ * 'C', 'c', 'R', or 'r' as a defined name."
+ */
+const RESERVED_SINGLE_CHARS = new Set(["C", "c", "R", "r"]);
+/**
+ * Sanitize a table name to comply with OOXML defined name rules
+ * (ECMA-376, 4th edition, Part 1, §18.5.1.2).
+ *
+ * Rules enforced (per Microsoft documentation):
+ * - First character must be a letter (any script), underscore (_), or backslash (\)
+ * - Subsequent characters may be letters, digits, underscores, or periods (.)
+ * - Backslash is only valid as the first character
+ * - Spaces are replaced with underscores
+ * - Other invalid characters are stripped
+ * - Single-character names "C", "c", "R", "r" are prefixed with _
+ * - Names that look like cell references (e.g. A1, R1C1) are prefixed with _
+ * - Maximum 255 characters
+ * - Empty result falls back to "_Table"
+ *
+ * This library applies these rules automatically so that generated files
+ * always comply with the OOXML schema, avoiding Excel "repair" dialogs.
+ */
+function sanitizeTableName(name) {
+    // Replace all whitespace characters (space, tab, newline, etc.) with underscores
+    let sanitized = name.replace(/\s/g, "_");
+    // Preserve a leading backslash (valid only as first character per spec)
+    let leadingBackslash = false;
+    if (sanitized.startsWith("\\")) {
+        leadingBackslash = true;
+        sanitized = sanitized.slice(1);
+    }
+    // Strip characters not valid in defined names.
+    // Subsequent characters: Unicode letters, digits, underscore, period.
+    // Backslash is NOT valid in subsequent positions.
+    sanitized = sanitized.replace(/[^\p{L}\p{N}_.]/gu, "");
+    // Re-attach leading backslash
+    if (leadingBackslash) {
+        sanitized = `\\${sanitized}`;
+    }
+    // Ensure the first character is valid (letter, underscore, or backslash)
+    if (sanitized.length > 0 && !/^[\p{L}_\\]/u.test(sanitized[0])) {
+        sanitized = `_${sanitized}`;
+    }
+    // Truncate to max length
+    if (sanitized.length > MAX_TABLE_NAME_LENGTH) {
+        sanitized = sanitized.slice(0, MAX_TABLE_NAME_LENGTH);
+    }
+    // Fallback if empty after sanitization
+    if (sanitized.length === 0) {
+        return "_Table";
+    }
+    // Avoid reserved single-character names (C, c, R, r)
+    if (sanitized.length === 1 && RESERVED_SINGLE_CHARS.has(sanitized)) {
+        sanitized = `_${sanitized}`;
+    }
+    // Avoid names that look like cell references
+    if (CELL_REF_PATTERN.test(sanitized) || R1C1_PATTERN.test(sanitized)) {
+        sanitized = `_${sanitized}`;
+    }
+    return sanitized;
+}
 class Column {
     constructor(table, column, index) {
         this.table = table;
@@ -139,16 +218,26 @@ class Table {
         assign(table, "headerRow", true);
         assign(table, "totalsRow", false);
         assign(table, "style", {});
-        assign(table.style, "theme", "TableStyleMedium2");
-        assign(table.style, "showFirstColumn", false);
-        assign(table.style, "showLastColumn", false);
-        assign(table.style, "showRowStripes", false);
-        assign(table.style, "showColumnStripes", false);
+        const style = table.style;
+        assign(style, "theme", "TableStyleMedium2");
+        assign(style, "showFirstColumn", false);
+        assign(style, "showLastColumn", false);
+        assign(style, "showRowStripes", false);
+        assign(style, "showColumnStripes", false);
+        // Sanitize table name and displayName to comply with OOXML defined name rules.
+        // Excel UI rejects invalid names; here we auto-correct to avoid "repair" dialogs.
+        if (table.name) {
+            table.name = sanitizeTableName(table.name);
+        }
+        if (table.displayName) {
+            table.displayName = sanitizeTableName(table.displayName);
+        }
         const assert = (test, message) => {
             if (!test) {
                 throw new Error(message);
             }
         };
+        assert(!!table.name, "Table must have a name");
         assert(!!table.ref, "Table must have ref");
         assert(!!table.columns, "Table must have column definitions");
         assert(!!table.rows, "Table must have row definitions");
@@ -224,7 +313,7 @@ class Table {
                     const formula = this.getFormula(column);
                     if (formula) {
                         cell.value = {
-                            formula: column.totalsRowFormula,
+                            formula,
                             result: column.totalsRowResult
                         };
                     }
@@ -266,7 +355,7 @@ class Table {
                     const formula = this.getFormula(column);
                     if (formula) {
                         cell.value = {
-                            formula: column.totalsRowFormula,
+                            formula,
                             result: column.totalsRowResult
                         };
                     }
@@ -394,13 +483,13 @@ class Table {
         return this.table.name;
     }
     set name(value) {
-        this.table.name = value;
+        this.table.name = sanitizeTableName(value);
     }
     get displayName() {
         return this.table.displayName || this.table.name;
     }
     set displayName(value) {
-        this.table.displayName = value;
+        this.table.displayName = sanitizeTableName(value);
     }
     get headerRow() {
         return this.table.headerRow;
@@ -415,33 +504,48 @@ class Table {
         this._assign(this.table, "totalsRow", value);
     }
     get theme() {
-        return this.table.style.theme;
+        return this.table.style?.theme;
     }
     set theme(value) {
+        if (!this.table.style) {
+            this.table.style = {};
+        }
         this.table.style.theme = value;
     }
     get showFirstColumn() {
-        return this.table.style.showFirstColumn;
+        return this.table.style?.showFirstColumn;
     }
     set showFirstColumn(value) {
+        if (!this.table.style) {
+            this.table.style = {};
+        }
         this.table.style.showFirstColumn = value;
     }
     get showLastColumn() {
-        return this.table.style.showLastColumn;
+        return this.table.style?.showLastColumn;
     }
     set showLastColumn(value) {
+        if (!this.table.style) {
+            this.table.style = {};
+        }
         this.table.style.showLastColumn = value;
     }
     get showRowStripes() {
-        return this.table.style.showRowStripes;
+        return this.table.style?.showRowStripes;
     }
     set showRowStripes(value) {
+        if (!this.table.style) {
+            this.table.style = {};
+        }
         this.table.style.showRowStripes = value;
     }
     get showColumnStripes() {
-        return this.table.style.showColumnStripes;
+        return this.table.style?.showColumnStripes;
     }
     set showColumnStripes(value) {
+        if (!this.table.style) {
+            this.table.style = {};
+        }
         this.table.style.showColumnStripes = value;
     }
 }

package/dist/cjs/modules/excel/utils/sheet-utils.js

@@ -588,7 +588,7 @@ function bookAppendSheet(workbook, worksheet, name) {
     });
     // Copy column properties
     worksheet.columns?.forEach((col, idx) => {
-        if (col && newWs.columns[idx]) {
+        if (col && newWs.columns?.[idx]) {
             if (col.width) {
                 newWs.getColumn(idx + 1).width = col.width;
             }

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

@@ -475,9 +475,9 @@ class Worksheet {
     _copyStyle(src, dest, styleEmpty = false) {
         const rSrc = this.getRow(src);
         const rDst = this.getRow(dest);
-        rDst.style = (0, copy_style_1.copyStyle)(rSrc.style);
+        rDst.style = (0, copy_style_1.copyStyle)(rSrc.style) ?? {};
         rSrc.eachCell({ includeEmpty: styleEmpty }, (cell, colNumber) => {
-            rDst.getCell(colNumber).style = (0, copy_style_1.copyStyle)(cell.style);
+            rDst.getCell(colNumber).style = (0, copy_style_1.copyStyle)(cell.style) ?? {};
         });
         rDst.height = rSrc.height;
     }
@@ -1053,7 +1053,8 @@ class Worksheet {
      */
     addTable(model) {
         const table = new table_1.Table(this, model);
-        this.tables[model.name] = table;
+        // Use table.name (sanitized by Table.validate()) as the key
+        this.tables[table.name] = table;
         return table;
     }
     /**
@@ -1170,7 +1171,7 @@ class Worksheet {
     }
     set model(value) {
         this.name = value.name;
-        this._columns = column_1.Column.fromModel(this, value.cols);
+        this._columns = column_1.Column.fromModel(this, value.cols ?? []);
         this._parseRows(value);
         this._parseMergeCells(value);
         this.dataValidations = new data_validations_1.DataValidations(value.dataValidations);

package/dist/cjs/modules/excel/xlsx/xform/drawing/ctrl-prop-xform.js

@@ -25,7 +25,7 @@ class CtrlPropXform extends base_xform_1.BaseXform {
         return "formControlPr";
     }
     render(xmlStream, model) {
-        const renderModel = model || this.model;
+        const renderModel = (model || this.model);
         const attrs = {
             xmlns: "http://schemas.microsoft.com/office/spreadsheetml/2009/9/main",
             objectType: "CheckBox",

package/dist/cjs/modules/excel/xlsx/xform/drawing/vml-drawing-xform.js

@@ -20,9 +20,11 @@ class VmlDrawingXform extends base_xform_1.BaseXform {
      * Render VML drawing containing both notes and form controls
      */
     render(xmlStream, model) {
-        const renderModel = model || this.model;
-        const hasComments = renderModel.comments && renderModel.comments.length > 0;
-        const hasFormControls = renderModel.formControls && renderModel.formControls.length > 0;
+        const renderModel = (model || this.model);
+        const comments = renderModel.comments;
+        const formControls = renderModel.formControls;
+        const hasComments = comments && comments.length > 0;
+        const hasFormControls = formControls && formControls.length > 0;
         xmlStream.openXml(xml_stream_1.XmlStream.StdDocAttributes);
         xmlStream.openNode(this.tag, VmlDrawingXform.DRAWING_ATTRIBUTES);
         // Shape layout - shared by both notes and form controls
@@ -62,14 +64,13 @@ class VmlDrawingXform extends base_xform_1.BaseXform {
         }
         // Render comment shapes
         if (hasComments) {
-            const comments = renderModel.comments;
             for (let i = 0; i < comments.length; i++) {
                 this.map["v:shape"].render(xmlStream, comments[i], i);
             }
         }
         // Render form control shapes
         if (hasFormControls) {
-            for (const control of renderModel.formControls) {
+            for (const control of formControls) {
                 this._renderCheckboxShape(xmlStream, control);
             }
         }

package/dist/cjs/modules/stream/streams.browser.js

@@ -1718,7 +1718,7 @@ class Transform extends event_emitter_1.EventEmitter {
                 return;
             }
             const result = userTransform.call(this, chunk);
-            if (result && typeof result.then === "function") {
+            if (result instanceof Promise) {
                 const awaited = await result;
                 if (awaited !== undefined) {
                     this.push(awaited);
@@ -1775,7 +1775,7 @@ class Transform extends event_emitter_1.EventEmitter {
                 return;
             }
             const result = userFlush.call(this);
-            if (result && typeof result.then === "function") {
+            if (result instanceof Promise) {
                 const awaited = await result;
                 if (awaited !== undefined && awaited !== null) {
                     this.push(awaited);
@@ -3381,14 +3381,22 @@ function duplexPair(options) {
             stream2.push(chunk);
         }
         stream2.push(null);
-        return originalEnd1(typeof chunk === "function" ? chunk : undefined);
+        if (typeof chunk === "function") {
+            const cb = chunk;
+            return originalEnd1(cb);
+        }
+        return originalEnd1();
     };
     stream2.end = function (chunk) {
         if (chunk !== undefined && typeof chunk !== "function") {
             stream1.push(chunk);
         }
         stream1.push(null);
-        return originalEnd2(typeof chunk === "function" ? chunk : undefined);
+        if (typeof chunk === "function") {
+            const cb = chunk;
+            return originalEnd2(cb);
+        }
+        return originalEnd2();
     };
     return [stream1, stream2];
 }

package/dist/cjs/utils/utils.base.js

@@ -107,7 +107,7 @@ function xmlEncode(text) {
     let lastIndex = 0;
     for (let i = regexResult.index; i < text.length; i++) {
         const charCode = text.charCodeAt(i);
-        let escape = null;
+        let escape;
         switch (charCode) {
             case 34: // "
                 escape = "&quot;";

package/dist/esm/modules/archive/unzip/stream.base.js

@@ -646,7 +646,6 @@ export function streamUntilValidatedDataDescriptor(options) {
                                 if (written > 0) {
                                     source.discard(written);
                                     bytesEmitted += written;
-                                    available -= written;
                                     scanner.onConsume(written);
                                 }
                                 if (waitingDrain) {
@@ -656,7 +655,6 @@ export function streamUntilValidatedDataDescriptor(options) {
                             else {
                                 const ok = output.write(source.read(idx));
                                 bytesEmitted += idx;
-                                available -= idx;
                                 scanner.onConsume(idx);
                                 if (!ok) {
                                     waitingDrain = true;
@@ -957,8 +955,17 @@ async function readFileRecord(opts, io, emitter, inflateFactory, state, threshol
             }
         }
     }
-    const extra = parseExtraField(extraFieldData, vars);
-    vars.lastModifiedDateTime = resolveZipEntryLastModifiedDateTime(vars, extra);
+    const zipVars = {
+        uncompressedSize: vars.uncompressedSize ?? 0,
+        compressedSize: vars.compressedSize ?? 0
+    };
+    const extra = parseExtraField(extraFieldData, zipVars);
+    vars.lastModifiedDateTime = resolveZipEntryLastModifiedDateTime({
+        flags: vars.flags,
+        uncompressedSize: zipVars.uncompressedSize,
+        lastModifiedDate: vars.lastModifiedDate,
+        lastModifiedTime: vars.lastModifiedTime
+    }, extra);
     entry.vars = vars;
     entry.extraFields = extra;
     entry.__autodraining = autodraining;

package/dist/esm/modules/archive/unzip/stream.browser.js

@@ -174,7 +174,7 @@ class BrowserInflateRaw extends Duplex {
             this.writer.abort(error ?? undefined).catch(() => { });
         }
         this.reader.cancel(error ?? undefined).catch(() => { });
-        return super.destroy(error);
+        return super.destroy(error ?? undefined);
     }
 }
 // =============================================================================
@@ -393,7 +393,7 @@ class WorkerInflateRaw extends Duplex {
             }
         }
         this._terminateWorker();
-        return super.destroy(error);
+        return super.destroy(error ?? undefined);
     }
 }
 // =============================================================================
@@ -439,7 +439,7 @@ class FallbackInflateRaw extends Duplex {
     destroy(error) {
         this._finished = true;
         this.chunks = [];
-        return super.destroy(error);
+        return super.destroy(error ?? undefined);
     }
 }
 // =============================================================================

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

@@ -73,11 +73,8 @@ class Anchor {
             : 640000;
     }
     get rowHeight() {
-        return this.worksheet &&
-            this.worksheet.getRow(this.nativeRow + 1) &&
-            this.worksheet.getRow(this.nativeRow + 1).height
-            ? Math.floor(this.worksheet.getRow(this.nativeRow + 1).height * 10000)
-            : 180000;
+        const height = this.worksheet?.getRow(this.nativeRow + 1)?.height;
+        return height ? Math.floor(height * 10000) : 180000;
     }
     get model() {
         return {