@rowsncolumns/toolkit 24.5.0 → 24.5.3

package/dist/cjs/excel-parser/helpers.d.ts

@@ -2,15 +2,17 @@ import type { GridRange } from "@rowsncolumns/common-types";
 import type { Column, FilterSpec, SortSpecs, TableTotalsRowFunction } from "@rowsncolumns/spreadsheet";
 export declare const getRangeFromAddressRef: (ref: string) => GridRange | undefined;
 /**
- * Convert an xlsx column width (in units of max-digit-width of the
- * default workbook font, Calibri 11 → MDW=7px) to pixels.
+ * OOXML `<col width>` (and `<sheetFormatPr defaultColWidth>`) → pixels.
  *
- * Excel's effective rendered pixel count includes a 5-pixel padding
- * baked into the column box: `pixels = floor(width × MDW + padding)`.
- * The default width "8.43" renders as 64 px (8.43 × 7 + 5 ≈ 64.01),
- * matching Excel exactly. The "pure" ECMA-376 §18.3.1.13 formula omits
- * the 5px padding; we keep the +5 form so our pixel output matches
- * Excel's rendered widths and round-trips with our exporter.
+ * The stored width is already in *padded* character units: Excel writes
+ * `Truncate((chars × MDW + 5) / MDW × 256) / 256` — the default 8.43-character
+ * column is stored as 9.140625 — and renders it back as
+ * `Truncate(((256 × width + Truncate(128 / MDW)) / 256) × MDW)`
+ * (ECMA-376 §18.3.1.13), i.e. 9.140625 → 64 px. Adding the 5 px padding a
+ * second time (the previous `floor(width × 7 + 5)`) drew every Excel-authored
+ * column ~5 px wider than Excel does and grew widths by 0.71 characters per
+ * import/export cycle (the exporter stores `px / 7`). MDW is 7 px for the
+ * Calibri 11 / Aptos Narrow 12 defaults at 96 dpi.
  */
 export declare function excelColumnWidthToPixels(width: number): number;
 export declare function findMax(arr: number[]): number;

package/dist/cjs/excel-parser/helpers.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"helpers.d.ts","sourceRoot":"","sources":["../../../excel-parser/helpers.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,4BAA4B,CAAC;AAC5D,OAAO,KAAK,EAEV,MAAM,EACN,UAAU,EACV,SAAS,EACT,sBAAsB,EACvB,MAAM,2BAA2B,CAAC;AAUnC,eAAO,MAAM,sBAAsB,GAAI,KAAK,MAAM,KAAG,SAAS,GAAG,SAIhE,CAAC;AAEF;;;;;;;;;;GAUG;AACH,wBAAgB,wBAAwB,CAAC,KAAK,EAAE,MAAM,UAIrD;AAED,wBAAgB,OAAO,CAAC,GAAG,EAAE,MAAM,EAAE,UAMpC;AAmBD,eAAO,MAAM,wBAAwB,GACnC,OAAO,OAAO,KACb,KAAK,IAAI,sBACwD,CAAC;AAErE;;;;;GAKG;AACH,eAAO,MAAM,sBAAsB,GACjC,QAAQ,GAAG,KACV,IAAI,CAAC,MAAM,EAAE,mBAAmB,GAAG,gBAAgB,GAAG,kBAAkB,CAmB1E,CAAC;AAqBF,wBAAgB,+BAA+B,CAC7C,UAAU,EAAE,GAAG;AACf;;;;GAIG;AACH,eAAe,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,SAAS,EAAE,OAAO,KAAK,MAAM,GAAG,SAAS,gBA0G5E;AAED;;;;GAIG;AACH,wBAAgB,yBAAyB,CAAC,UAAU,EAAE,GAAG,GAAG,GAAG,CAAC,MAAM,CAAC,CAgBtE;AAED;;;;;;GAMG;AACH,wBAAgB,4BAA4B,CAC1C,SAAS,EAAE,GAAG,EACd,KAAK,EAAE,SAAS,EAChB,eAAe,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,SAAS,EAAE,OAAO,KAAK,MAAM,GAAG,SAAS,GAC1E,SAAS,EAAE,CA2Cb;AAED;;GAEG;AACH,wBAAsB,aAAa,CAAC,IAAI,EAAE,IAAI,GAAG,WAAW,wBAkB3D"}
+{"version":3,"file":"helpers.d.ts","sourceRoot":"","sources":["../../../excel-parser/helpers.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,4BAA4B,CAAC;AAC5D,OAAO,KAAK,EAEV,MAAM,EACN,UAAU,EACV,SAAS,EACT,sBAAsB,EACvB,MAAM,2BAA2B,CAAC;AAUnC,eAAO,MAAM,sBAAsB,GAAI,KAAK,MAAM,KAAG,SAAS,GAAG,SAIhE,CAAC;AAEF;;;;;;;;;;;;GAYG;AACH,wBAAgB,wBAAwB,CAAC,KAAK,EAAE,MAAM,UAMrD;AAED,wBAAgB,OAAO,CAAC,GAAG,EAAE,MAAM,EAAE,UAMpC;AAmBD,eAAO,MAAM,wBAAwB,GACnC,OAAO,OAAO,KACb,KAAK,IAAI,sBACwD,CAAC;AAErE;;;;;GAKG;AACH,eAAO,MAAM,sBAAsB,GACjC,QAAQ,GAAG,KACV,IAAI,CAAC,MAAM,EAAE,mBAAmB,GAAG,gBAAgB,GAAG,kBAAkB,CAmB1E,CAAC;AAqBF,wBAAgB,+BAA+B,CAC7C,UAAU,EAAE,GAAG;AACf;;;;GAIG;AACH,eAAe,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,SAAS,EAAE,OAAO,KAAK,MAAM,GAAG,SAAS,gBA0G5E;AAED;;;;GAIG;AACH,wBAAgB,yBAAyB,CAAC,UAAU,EAAE,GAAG,GAAG,GAAG,CAAC,MAAM,CAAC,CAgBtE;AAED;;;;;;GAMG;AACH,wBAAgB,4BAA4B,CAC1C,SAAS,EAAE,GAAG,EACd,KAAK,EAAE,SAAS,EAChB,eAAe,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,SAAS,EAAE,OAAO,KAAK,MAAM,GAAG,SAAS,GAC1E,SAAS,EAAE,CA2Cb;AAED;;GAEG;AACH,wBAAsB,aAAa,CAAC,IAAI,EAAE,IAAI,GAAG,WAAW,wBAkB3D"}

package/dist/cjs/excel-parser/helpers.js

@@ -8,20 +8,23 @@ export var getRangeFromAddressRef = function (ref) {
     return selection.range;
 };
 /**
- * Convert an xlsx column width (in units of max-digit-width of the
- * default workbook font, Calibri 11 → MDW=7px) to pixels.
+ * OOXML `<col width>` (and `<sheetFormatPr defaultColWidth>`) → pixels.
  *
- * Excel's effective rendered pixel count includes a 5-pixel padding
- * baked into the column box: `pixels = floor(width × MDW + padding)`.
- * The default width "8.43" renders as 64 px (8.43 × 7 + 5 ≈ 64.01),
- * matching Excel exactly. The "pure" ECMA-376 §18.3.1.13 formula omits
- * the 5px padding; we keep the +5 form so our pixel output matches
- * Excel's rendered widths and round-trips with our exporter.
+ * The stored width is already in *padded* character units: Excel writes
+ * `Truncate((chars × MDW + 5) / MDW × 256) / 256` — the default 8.43-character
+ * column is stored as 9.140625 — and renders it back as
+ * `Truncate(((256 × width + Truncate(128 / MDW)) / 256) × MDW)`
+ * (ECMA-376 §18.3.1.13), i.e. 9.140625 → 64 px. Adding the 5 px padding a
+ * second time (the previous `floor(width × 7 + 5)`) drew every Excel-authored
+ * column ~5 px wider than Excel does and grew widths by 0.71 characters per
+ * import/export cycle (the exporter stores `px / 7`). MDW is 7 px for the
+ * Calibri 11 / Aptos Narrow 12 defaults at 96 dpi.
  */
 export function excelColumnWidthToPixels(width) {
-    var padding = 5;
-    var charWidth = 7;
-    return Math.floor(width * charWidth + padding);
+    var maxDigitWidth = 7;
+    if (!Number.isFinite(width) || width <= 0)
+        return 0;
+    return Math.trunc(((256 * width + Math.trunc(128 / maxDigitWidth)) / 256) * maxDigitWidth);
 }
 export function findMax(arr) {
     var max = arr[0];

package/dist/cjs/exporter/xlsx/xlsx-export.js

@@ -1444,8 +1444,113 @@ var normalizeMergeRanges = function (ranges) {
     }
     return valid;
 };
+/** Escape `"` as `""` inside a formula string literal. */
+var escapeFormulaText = function (value) { return value.replace(/"/g, '""'); };
+// `xlsxSource` is meant to hold the verbatim OOXML representation, but
+// writers other than the importer have filled it from Excel's JavaScript-API
+// vocabulary: a bare `list.source` literal (`On track,Watch,Off track`),
+// PascalCase operators (`Between`), PascalCase types (`WholeNumber`) and
+// `=`-prefixed formulas. Excel opens a file carrying any of those in repair
+// mode and drops the rule, so the exporter normalizes every field before
+// replaying it, and a type / operator that is not an OOXML token even after
+// normalization (`undefined` below) is never replayed — the rule falls
+// through to the condition synthesizer. Twin of `normalize_source_*` in
+// `rust/crates/rnc-export/src/xlsx/datavalidation.rs` — keep the two in sync
+// so both engines emit identical XML.
+var normalizeXlsxSourceType = function (type) {
+    switch (type.trim().toLowerCase()) {
+        case "list":
+            return "list";
+        case "whole":
+        case "wholenumber":
+            return "whole";
+        case "decimal":
+            return "decimal";
+        case "date":
+            return "date";
+        case "time":
+            return "time";
+        case "textlength":
+            return "textLength";
+        case "custom":
+            return "custom";
+        default:
+            return undefined;
+    }
+};
+var normalizeXlsxSourceOperator = function (operator) {
+    switch (operator.trim().toLowerCase()) {
+        case "between":
+            return "between";
+        case "notbetween":
+            return "notBetween";
+        case "equal":
+        case "equalto":
+            return "equal";
+        case "notequal":
+        case "notequalto":
+            return "notEqual";
+        case "greaterthan":
+            return "greaterThan";
+        case "lessthan":
+            return "lessThan";
+        case "greaterthanorequal":
+        case "greaterthanorequalto":
+            return "greaterThanOrEqual";
+        case "lessthanorequal":
+        case "lessthanorequalto":
+            return "lessThanOrEqual";
+        default:
+            return undefined;
+    }
+};
+// OOXML `<formula1>` text never carries the leading `=` a formula-style
+// source (`=Sheet1!$A$1:$A$5`) has.
+var stripLeadingEquals = function (formula) {
+    var trimmed = formula.trim();
+    return trimmed.startsWith("=") ? trimmed.slice(1) : trimmed;
+};
+// One identifier token — `[A-Za-z_\\][A-Za-z0-9_.]*` — which is what a
+// defined name (`StatusList`, `My_List.v2`) or a single A1 cell (`A1`) looks
+// like; a word list with commas or spaces is not one.
+var XLSX_IDENTIFIER_TOKEN = /^[A-Za-z_\\][A-Za-z0-9_.]*$/;
+// A single A1 cell token: 1–3 letters followed by digits.
+var XLSX_A1_CELL_TOKEN = /^[A-Za-z]{1,3}[0-9]+$/;
+// A function call — `OFFSET(…)`, `INDIRECT(…)` — which Excel writes without
+// a leading `=`.
+var XLSX_FUNCTION_CALL = /^[A-Za-z_\\][A-Za-z0-9_.]*\(.*\)$/;
+// `true` when a list source reads as a reference / formula rather than as
+// list text: it carries a reference marker (`!` sheet qualifier, `:` range,
+// `$` absolute anchor, `[` structured / external ref), is a single A1 cell
+// or defined-name token, or is a function call. Anything else — comma lists,
+// text with spaces, digit-only entries — is a bare literal that Excel only
+// accepts quoted.
+var looksLikeXlsxListReference = function (text) {
+    return text.length > 0 &&
+        (/[!:$[]/.test(text) ||
+            XLSX_A1_CELL_TOKEN.test(text) ||
+            XLSX_IDENTIFIER_TOKEN.test(text) ||
+            XLSX_FUNCTION_CALL.test(text));
+};
+// A `list` source is valid OOXML only as a quoted string literal (`"a,b"`)
+// or a reference (`$A$1:$A$5`, `Sheet1!$A$1:$A$5`, `StatusList`, `OFFSET(…)`).
+// Excel's API hands back a literal list as bare comma-separated text and a
+// reference with a leading `=`, so: quoted → verbatim, `=`-prefixed → the `=`
+// is stripped and the rest replayed verbatim, reference-like → verbatim,
+// anything else → a literal that must be quoted (with `"` doubled inside, as
+// the ONE_OF_LIST branch does).
+var normalizeXlsxSourceListFormula1 = function (formula1) {
+    var trimmed = formula1.trim();
+    if (trimmed.startsWith('"'))
+        return trimmed;
+    if (trimmed.startsWith("="))
+        return trimmed.slice(1).trim();
+    if (looksLikeXlsxListReference(trimmed))
+        return trimmed;
+    return "\"".concat(escapeFormulaText(trimmed), "\"");
+};
 var convertDataValidationRule = function (rule) {
-    var _a, _b, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _s, _t, _u, _v, _w, _x, _y, _z, _0, _1, _2, _3, _4, _5;
+    var _a, _b, _d, _e, _f, _g;
     if (!rule)
         return undefined;
     var allowBlank = rule.allowBlank !== false;
@@ -1462,14 +1567,48 @@ var convertDataValidationRule = function (rule) {
         prompt: (_g = inputMessage === null || inputMessage === void 0 ? void 0 : inputMessage.message) !== null && _g !== void 0 ? _g : undefined,
     };
     // Round-trip path: when the rule was imported from xlsx, replay the
-    // verbatim source representation. Preserves type/operator/formulas
-    // exactly, including `whole`/`textLength`/`time` types and user-written
-    // custom formulas without `INDIRECT("RC",FALSE)` rewriting.
+    // source representation after normalizing it to OOXML vocabulary (see
+    // `normalizeXlsxSource*`). Preserves type/operator/formulas, including
+    // `whole`/`textLength`/`time` types and user-written custom formulas
+    // without `INDIRECT("RC",FALSE)` rewriting. A type / operator that is not
+    // an OOXML token even after normalization (Excel's `Inconsistent`,
+    // `MixedCriteria`) is never written: the rule falls through to the
+    // condition synthesizer.
     var src = rule.xlsxSource;
+    var rejectedListSource;
     if (src && src.type && typeof src.formula1 === "string" && src.formula1) {
-        return __assign({ type: src.type, operator: src.operator, formula1: src.formula1, formula2: src.formula2, showDropDown: src.type === "list" && rule.displayStyle === "plain" ? true : undefined }, baseConfig);
+        var type = normalizeXlsxSourceType(src.type);
+        var operator = src.operator
+            ? normalizeXlsxSourceOperator(src.operator)
+            : undefined;
+        var operatorValid = !src.operator || operator !== undefined;
+        if (type !== undefined && operatorValid) {
+            var formula1 = type === "list"
+                ? normalizeXlsxSourceListFormula1(src.formula1)
+                : stripLeadingEquals(src.formula1);
+            return __assign({ type: type, operator: operator, formula1: formula1, formula2: src.formula2 ? stripLeadingEquals(src.formula2) : undefined, showDropDown: type === "list" && rule.displayStyle === "plain" ? true : undefined }, baseConfig);
+        }
+        if (type === "list") {
+            rejectedListSource = normalizeXlsxSourceListFormula1(src.formula1);
+        }
+    }
+    var synthesized = convertDataValidationCondition(rule, baseConfig);
+    if (synthesized)
+        return synthesized;
+    // A rejected `list` source with no usable condition is still kept — its
+    // normalized source text (quoted literal or reference) is always valid
+    // OOXML. Anything else rejected has nothing valid to write and is skipped.
+    if (rejectedListSource !== undefined) {
+        return __assign({ type: "list", formula1: rejectedListSource, showDropDown: rule.displayStyle === "plain" ? true : undefined }, baseConfig);
     }
-    if (!((_h = rule.condition) === null || _h === void 0 ? void 0 : _h.type)) {
+    return undefined;
+};
+// The condition-based synthesizer: the Excel config for a rule that has no
+// replayable `xlsxSource` (UI-authored, edited after import, or rejected by
+// the round-trip guard above).
+var convertDataValidationCondition = function (rule, baseConfig) {
+    var _a, _b, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _s, _t, _u, _v, _w, _x, _y, _z;
+    if (!((_a = rule.condition) === null || _a === void 0 ? void 0 : _a.type)) {
         // Prompt-only DV (no validation rule, just an input-message tooltip
         // and/or an error alert). Emit it with no type/formula so Excel still
         // shows the tooltip when the cell is selected.
@@ -1479,9 +1618,8 @@ var convertDataValidationRule = function (rule) {
         return undefined;
     }
     var condition = rule.condition;
-    var values = (_j = condition.values) !== null && _j !== void 0 ? _j : [];
+    var values = (_b = condition.values) !== null && _b !== void 0 ? _b : [];
     var currentCell = 'INDIRECT("RC",FALSE)';
-    var escapeFormulaText = function (value) { return value.replace(/"/g, '""'); };
     var ensureFormula = function (value) {
         return value.startsWith("=") ? value : "=".concat(value);
     };
@@ -1505,7 +1643,7 @@ var convertDataValidationRule = function (rule) {
         return __assign({ type: "list", formula1: formula1.startsWith("=") ? formula1 : "=".concat(formula1), showDropDown: rule.displayStyle === "plain" ? true : undefined }, baseConfig);
     }
     if (condition.type === "CUSTOM_FORMULA") {
-        var raw = (_l = (_k = values[0]) === null || _k === void 0 ? void 0 : _k.userEnteredValue) !== null && _l !== void 0 ? _l : "";
+        var raw = (_e = (_d = values[0]) === null || _d === void 0 ? void 0 : _d.userEnteredValue) !== null && _e !== void 0 ? _e : "";
         if (!raw)
             return undefined;
         return __assign({ type: "custom", formula1: ensureFormula(raw) }, baseConfig);
@@ -1520,33 +1658,33 @@ var convertDataValidationRule = function (rule) {
         return __assign({ type: "custom", formula1: "=".concat(currentCell, "<>\"\"") }, baseConfig);
     }
     if (condition.type === "TEXT_EQ" || condition.type === "TEXT_NOT_EQ") {
-        var raw = (_o = (_m = values[0]) === null || _m === void 0 ? void 0 : _m.userEnteredValue) !== null && _o !== void 0 ? _o : "";
+        var raw = (_g = (_f = values[0]) === null || _f === void 0 ? void 0 : _f.userEnteredValue) !== null && _g !== void 0 ? _g : "";
         if (!raw)
             return undefined;
         var op = condition.type === "TEXT_EQ" ? "=" : "<>";
         return __assign({ type: "custom", formula1: "=".concat(currentCell).concat(op, "\"").concat(escapeFormulaText(raw), "\"") }, baseConfig);
     }
     if (condition.type === "TEXT_CONTAINS") {
-        var raw = (_q = (_p = values[0]) === null || _p === void 0 ? void 0 : _p.userEnteredValue) !== null && _q !== void 0 ? _q : "";
+        var raw = (_j = (_h = values[0]) === null || _h === void 0 ? void 0 : _h.userEnteredValue) !== null && _j !== void 0 ? _j : "";
         if (!raw)
             return undefined;
         return __assign({ type: "custom", formula1: "=ISNUMBER(SEARCH(\"".concat(escapeFormulaText(raw), "\",").concat(currentCell, "))") }, baseConfig);
     }
     if (condition.type === "TEXT_NOT_CONTAINS") {
-        var raw = (_t = (_s = values[0]) === null || _s === void 0 ? void 0 : _s.userEnteredValue) !== null && _t !== void 0 ? _t : "";
+        var raw = (_l = (_k = values[0]) === null || _k === void 0 ? void 0 : _k.userEnteredValue) !== null && _l !== void 0 ? _l : "";
         if (!raw)
             return undefined;
         return __assign({ type: "custom", formula1: "=ISERROR(SEARCH(\"".concat(escapeFormulaText(raw), "\",").concat(currentCell, "))") }, baseConfig);
     }
     if (condition.type === "TEXT_STARTS_WITH") {
-        var raw = (_v = (_u = values[0]) === null || _u === void 0 ? void 0 : _u.userEnteredValue) !== null && _v !== void 0 ? _v : "";
+        var raw = (_o = (_m = values[0]) === null || _m === void 0 ? void 0 : _m.userEnteredValue) !== null && _o !== void 0 ? _o : "";
         if (!raw)
             return undefined;
         var text = escapeFormulaText(raw);
         return __assign({ type: "custom", formula1: "=LEFT(".concat(currentCell, ",LEN(\"").concat(text, "\"))=\"").concat(text, "\"") }, baseConfig);
     }
     if (condition.type === "TEXT_ENDS_WITH") {
-        var raw = (_x = (_w = values[0]) === null || _w === void 0 ? void 0 : _w.userEnteredValue) !== null && _x !== void 0 ? _x : "";
+        var raw = (_q = (_p = values[0]) === null || _p === void 0 ? void 0 : _p.userEnteredValue) !== null && _q !== void 0 ? _q : "";
         if (!raw)
             return undefined;
         var text = escapeFormulaText(raw);
@@ -1566,10 +1704,10 @@ var convertDataValidationRule = function (rule) {
         condition.type === "NUMBER_GREATER_THAN_EQ" ||
         condition.type === "NUMBER_LESS" ||
         condition.type === "NUMBER_LESS_THAN_EQ") {
-        var first = (_z = (_y = values[0]) === null || _y === void 0 ? void 0 : _y.userEnteredValue) !== null && _z !== void 0 ? _z : "";
+        var first = (_t = (_s = values[0]) === null || _s === void 0 ? void 0 : _s.userEnteredValue) !== null && _t !== void 0 ? _t : "";
         if (!first)
             return undefined;
-        var second = (_1 = (_0 = values[1]) === null || _0 === void 0 ? void 0 : _0.userEnteredValue) !== null && _1 !== void 0 ? _1 : "";
+        var second = (_v = (_u = values[1]) === null || _u === void 0 ? void 0 : _u.userEnteredValue) !== null && _v !== void 0 ? _v : "";
         var operatorMap = {
             NUMBER_BETWEEN: "between",
             NUMBER_NOT_BETWEEN: "notBetween",
@@ -1606,10 +1744,10 @@ var convertDataValidationRule = function (rule) {
         condition.type === "DATE_ON_OR_AFTER" ||
         condition.type === "DATE_BETWEEN" ||
         condition.type === "DATE_NOT_BETWEEN") {
-        var first = (_3 = (_2 = values[0]) === null || _2 === void 0 ? void 0 : _2.userEnteredValue) !== null && _3 !== void 0 ? _3 : "";
+        var first = (_x = (_w = values[0]) === null || _w === void 0 ? void 0 : _w.userEnteredValue) !== null && _x !== void 0 ? _x : "";
         if (!first)
             return undefined;
-        var second = (_5 = (_4 = values[1]) === null || _4 === void 0 ? void 0 : _4.userEnteredValue) !== null && _5 !== void 0 ? _5 : "";
+        var second = (_z = (_y = values[1]) === null || _y === void 0 ? void 0 : _y.userEnteredValue) !== null && _z !== void 0 ? _z : "";
         var operatorMap = {
             DATE_EQ: "equal",
             DATE_NOT_EQ: "notEqual",

package/dist/esm/excel-parser/helpers.d.ts

@@ -2,15 +2,17 @@ import type { GridRange } from "@rowsncolumns/common-types";
 import type { Column, FilterSpec, SortSpecs, TableTotalsRowFunction } from "@rowsncolumns/spreadsheet";
 export declare const getRangeFromAddressRef: (ref: string) => GridRange | undefined;
 /**
- * Convert an xlsx column width (in units of max-digit-width of the
- * default workbook font, Calibri 11 → MDW=7px) to pixels.
+ * OOXML `<col width>` (and `<sheetFormatPr defaultColWidth>`) → pixels.
  *
- * Excel's effective rendered pixel count includes a 5-pixel padding
- * baked into the column box: `pixels = floor(width × MDW + padding)`.
- * The default width "8.43" renders as 64 px (8.43 × 7 + 5 ≈ 64.01),
- * matching Excel exactly. The "pure" ECMA-376 §18.3.1.13 formula omits
- * the 5px padding; we keep the +5 form so our pixel output matches
- * Excel's rendered widths and round-trips with our exporter.
+ * The stored width is already in *padded* character units: Excel writes
+ * `Truncate((chars × MDW + 5) / MDW × 256) / 256` — the default 8.43-character
+ * column is stored as 9.140625 — and renders it back as
+ * `Truncate(((256 × width + Truncate(128 / MDW)) / 256) × MDW)`
+ * (ECMA-376 §18.3.1.13), i.e. 9.140625 → 64 px. Adding the 5 px padding a
+ * second time (the previous `floor(width × 7 + 5)`) drew every Excel-authored
+ * column ~5 px wider than Excel does and grew widths by 0.71 characters per
+ * import/export cycle (the exporter stores `px / 7`). MDW is 7 px for the
+ * Calibri 11 / Aptos Narrow 12 defaults at 96 dpi.
  */
 export declare function excelColumnWidthToPixels(width: number): number;
 export declare function findMax(arr: number[]): number;

package/dist/esm/excel-parser/helpers.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"helpers.d.ts","sourceRoot":"","sources":["../../../excel-parser/helpers.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,4BAA4B,CAAC;AAC5D,OAAO,KAAK,EAEV,MAAM,EACN,UAAU,EACV,SAAS,EACT,sBAAsB,EACvB,MAAM,2BAA2B,CAAC;AAUnC,eAAO,MAAM,sBAAsB,GAAI,KAAK,MAAM,KAAG,SAAS,GAAG,SAIhE,CAAC;AAEF;;;;;;;;;;GAUG;AACH,wBAAgB,wBAAwB,CAAC,KAAK,EAAE,MAAM,UAIrD;AAED,wBAAgB,OAAO,CAAC,GAAG,EAAE,MAAM,EAAE,UAMpC;AAmBD,eAAO,MAAM,wBAAwB,GACnC,OAAO,OAAO,KACb,KAAK,IAAI,sBACwD,CAAC;AAErE;;;;;GAKG;AACH,eAAO,MAAM,sBAAsB,GACjC,QAAQ,GAAG,KACV,IAAI,CAAC,MAAM,EAAE,mBAAmB,GAAG,gBAAgB,GAAG,kBAAkB,CAmB1E,CAAC;AAqBF,wBAAgB,+BAA+B,CAC7C,UAAU,EAAE,GAAG;AACf;;;;GAIG;AACH,eAAe,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,SAAS,EAAE,OAAO,KAAK,MAAM,GAAG,SAAS,gBA0G5E;AAED;;;;GAIG;AACH,wBAAgB,yBAAyB,CAAC,UAAU,EAAE,GAAG,GAAG,GAAG,CAAC,MAAM,CAAC,CAgBtE;AAED;;;;;;GAMG;AACH,wBAAgB,4BAA4B,CAC1C,SAAS,EAAE,GAAG,EACd,KAAK,EAAE,SAAS,EAChB,eAAe,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,SAAS,EAAE,OAAO,KAAK,MAAM,GAAG,SAAS,GAC1E,SAAS,EAAE,CA2Cb;AAED;;GAEG;AACH,wBAAsB,aAAa,CAAC,IAAI,EAAE,IAAI,GAAG,WAAW,wBAkB3D"}
+{"version":3,"file":"helpers.d.ts","sourceRoot":"","sources":["../../../excel-parser/helpers.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,4BAA4B,CAAC;AAC5D,OAAO,KAAK,EAEV,MAAM,EACN,UAAU,EACV,SAAS,EACT,sBAAsB,EACvB,MAAM,2BAA2B,CAAC;AAUnC,eAAO,MAAM,sBAAsB,GAAI,KAAK,MAAM,KAAG,SAAS,GAAG,SAIhE,CAAC;AAEF;;;;;;;;;;;;GAYG;AACH,wBAAgB,wBAAwB,CAAC,KAAK,EAAE,MAAM,UAMrD;AAED,wBAAgB,OAAO,CAAC,GAAG,EAAE,MAAM,EAAE,UAMpC;AAmBD,eAAO,MAAM,wBAAwB,GACnC,OAAO,OAAO,KACb,KAAK,IAAI,sBACwD,CAAC;AAErE;;;;;GAKG;AACH,eAAO,MAAM,sBAAsB,GACjC,QAAQ,GAAG,KACV,IAAI,CAAC,MAAM,EAAE,mBAAmB,GAAG,gBAAgB,GAAG,kBAAkB,CAmB1E,CAAC;AAqBF,wBAAgB,+BAA+B,CAC7C,UAAU,EAAE,GAAG;AACf;;;;GAIG;AACH,eAAe,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,SAAS,EAAE,OAAO,KAAK,MAAM,GAAG,SAAS,gBA0G5E;AAED;;;;GAIG;AACH,wBAAgB,yBAAyB,CAAC,UAAU,EAAE,GAAG,GAAG,GAAG,CAAC,MAAM,CAAC,CAgBtE;AAED;;;;;;GAMG;AACH,wBAAgB,4BAA4B,CAC1C,SAAS,EAAE,GAAG,EACd,KAAK,EAAE,SAAS,EAChB,eAAe,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,SAAS,EAAE,OAAO,KAAK,MAAM,GAAG,SAAS,GAC1E,SAAS,EAAE,CA2Cb;AAED;;GAEG;AACH,wBAAsB,aAAa,CAAC,IAAI,EAAE,IAAI,GAAG,WAAW,wBAkB3D"}

package/dist/esm/excel-parser/helpers.js

@@ -7,20 +7,23 @@ export const getRangeFromAddressRef = (ref) => {
     return selection.range;
 };
 /**
- * Convert an xlsx column width (in units of max-digit-width of the
- * default workbook font, Calibri 11 → MDW=7px) to pixels.
+ * OOXML `<col width>` (and `<sheetFormatPr defaultColWidth>`) → pixels.
  *
- * Excel's effective rendered pixel count includes a 5-pixel padding
- * baked into the column box: `pixels = floor(width × MDW + padding)`.
- * The default width "8.43" renders as 64 px (8.43 × 7 + 5 ≈ 64.01),
- * matching Excel exactly. The "pure" ECMA-376 §18.3.1.13 formula omits
- * the 5px padding; we keep the +5 form so our pixel output matches
- * Excel's rendered widths and round-trips with our exporter.
+ * The stored width is already in *padded* character units: Excel writes
+ * `Truncate((chars × MDW + 5) / MDW × 256) / 256` — the default 8.43-character
+ * column is stored as 9.140625 — and renders it back as
+ * `Truncate(((256 × width + Truncate(128 / MDW)) / 256) × MDW)`
+ * (ECMA-376 §18.3.1.13), i.e. 9.140625 → 64 px. Adding the 5 px padding a
+ * second time (the previous `floor(width × 7 + 5)`) drew every Excel-authored
+ * column ~5 px wider than Excel does and grew widths by 0.71 characters per
+ * import/export cycle (the exporter stores `px / 7`). MDW is 7 px for the
+ * Calibri 11 / Aptos Narrow 12 defaults at 96 dpi.
  */
 export function excelColumnWidthToPixels(width) {
-    const padding = 5;
-    const charWidth = 7;
-    return Math.floor(width * charWidth + padding);
+    const maxDigitWidth = 7;
+    if (!Number.isFinite(width) || width <= 0)
+        return 0;
+    return Math.trunc(((256 * width + Math.trunc(128 / maxDigitWidth)) / 256) * maxDigitWidth);
 }
 export function findMax(arr) {
     let max = arr[0];

package/dist/esm/exporter/xlsx/xlsx-export.js

@@ -1086,6 +1086,109 @@ const normalizeMergeRanges = (ranges) => {
     }
     return valid;
 };
+/** Escape `"` as `""` inside a formula string literal. */
+const escapeFormulaText = (value) => value.replace(/"/g, '""');
+// `xlsxSource` is meant to hold the verbatim OOXML representation, but
+// writers other than the importer have filled it from Excel's JavaScript-API
+// vocabulary: a bare `list.source` literal (`On track,Watch,Off track`),
+// PascalCase operators (`Between`), PascalCase types (`WholeNumber`) and
+// `=`-prefixed formulas. Excel opens a file carrying any of those in repair
+// mode and drops the rule, so the exporter normalizes every field before
+// replaying it, and a type / operator that is not an OOXML token even after
+// normalization (`undefined` below) is never replayed — the rule falls
+// through to the condition synthesizer. Twin of `normalize_source_*` in
+// `rust/crates/rnc-export/src/xlsx/datavalidation.rs` — keep the two in sync
+// so both engines emit identical XML.
+const normalizeXlsxSourceType = (type) => {
+    switch (type.trim().toLowerCase()) {
+        case "list":
+            return "list";
+        case "whole":
+        case "wholenumber":
+            return "whole";
+        case "decimal":
+            return "decimal";
+        case "date":
+            return "date";
+        case "time":
+            return "time";
+        case "textlength":
+            return "textLength";
+        case "custom":
+            return "custom";
+        default:
+            return undefined;
+    }
+};
+const normalizeXlsxSourceOperator = (operator) => {
+    switch (operator.trim().toLowerCase()) {
+        case "between":
+            return "between";
+        case "notbetween":
+            return "notBetween";
+        case "equal":
+        case "equalto":
+            return "equal";
+        case "notequal":
+        case "notequalto":
+            return "notEqual";
+        case "greaterthan":
+            return "greaterThan";
+        case "lessthan":
+            return "lessThan";
+        case "greaterthanorequal":
+        case "greaterthanorequalto":
+            return "greaterThanOrEqual";
+        case "lessthanorequal":
+        case "lessthanorequalto":
+            return "lessThanOrEqual";
+        default:
+            return undefined;
+    }
+};
+// OOXML `<formula1>` text never carries the leading `=` a formula-style
+// source (`=Sheet1!$A$1:$A$5`) has.
+const stripLeadingEquals = (formula) => {
+    const trimmed = formula.trim();
+    return trimmed.startsWith("=") ? trimmed.slice(1) : trimmed;
+};
+// One identifier token — `[A-Za-z_\\][A-Za-z0-9_.]*` — which is what a
+// defined name (`StatusList`, `My_List.v2`) or a single A1 cell (`A1`) looks
+// like; a word list with commas or spaces is not one.
+const XLSX_IDENTIFIER_TOKEN = /^[A-Za-z_\\][A-Za-z0-9_.]*$/;
+// A single A1 cell token: 1–3 letters followed by digits.
+const XLSX_A1_CELL_TOKEN = /^[A-Za-z]{1,3}[0-9]+$/;
+// A function call — `OFFSET(…)`, `INDIRECT(…)` — which Excel writes without
+// a leading `=`.
+const XLSX_FUNCTION_CALL = /^[A-Za-z_\\][A-Za-z0-9_.]*\(.*\)$/;
+// `true` when a list source reads as a reference / formula rather than as
+// list text: it carries a reference marker (`!` sheet qualifier, `:` range,
+// `$` absolute anchor, `[` structured / external ref), is a single A1 cell
+// or defined-name token, or is a function call. Anything else — comma lists,
+// text with spaces, digit-only entries — is a bare literal that Excel only
+// accepts quoted.
+const looksLikeXlsxListReference = (text) => text.length > 0 &&
+    (/[!:$[]/.test(text) ||
+        XLSX_A1_CELL_TOKEN.test(text) ||
+        XLSX_IDENTIFIER_TOKEN.test(text) ||
+        XLSX_FUNCTION_CALL.test(text));
+// A `list` source is valid OOXML only as a quoted string literal (`"a,b"`)
+// or a reference (`$A$1:$A$5`, `Sheet1!$A$1:$A$5`, `StatusList`, `OFFSET(…)`).
+// Excel's API hands back a literal list as bare comma-separated text and a
+// reference with a leading `=`, so: quoted → verbatim, `=`-prefixed → the `=`
+// is stripped and the rest replayed verbatim, reference-like → verbatim,
+// anything else → a literal that must be quoted (with `"` doubled inside, as
+// the ONE_OF_LIST branch does).
+const normalizeXlsxSourceListFormula1 = (formula1) => {
+    const trimmed = formula1.trim();
+    if (trimmed.startsWith('"'))
+        return trimmed;
+    if (trimmed.startsWith("="))
+        return trimmed.slice(1).trim();
+    if (looksLikeXlsxListReference(trimmed))
+        return trimmed;
+    return `"${escapeFormulaText(trimmed)}"`;
+};
 const convertDataValidationRule = (rule) => {
     if (!rule)
         return undefined;
@@ -1103,20 +1206,58 @@ const convertDataValidationRule = (rule) => {
         prompt: inputMessage?.message ?? undefined,
     };
     // Round-trip path: when the rule was imported from xlsx, replay the
-    // verbatim source representation. Preserves type/operator/formulas
-    // exactly, including `whole`/`textLength`/`time` types and user-written
-    // custom formulas without `INDIRECT("RC",FALSE)` rewriting.
+    // source representation after normalizing it to OOXML vocabulary (see
+    // `normalizeXlsxSource*`). Preserves type/operator/formulas, including
+    // `whole`/`textLength`/`time` types and user-written custom formulas
+    // without `INDIRECT("RC",FALSE)` rewriting. A type / operator that is not
+    // an OOXML token even after normalization (Excel's `Inconsistent`,
+    // `MixedCriteria`) is never written: the rule falls through to the
+    // condition synthesizer.
     const src = rule.xlsxSource;
+    let rejectedListSource;
     if (src && src.type && typeof src.formula1 === "string" && src.formula1) {
+        const type = normalizeXlsxSourceType(src.type);
+        const operator = src.operator
+            ? normalizeXlsxSourceOperator(src.operator)
+            : undefined;
+        const operatorValid = !src.operator || operator !== undefined;
+        if (type !== undefined && operatorValid) {
+            const formula1 = type === "list"
+                ? normalizeXlsxSourceListFormula1(src.formula1)
+                : stripLeadingEquals(src.formula1);
+            return {
+                type,
+                operator,
+                formula1,
+                formula2: src.formula2 ? stripLeadingEquals(src.formula2) : undefined,
+                showDropDown: type === "list" && rule.displayStyle === "plain" ? true : undefined,
+                ...baseConfig,
+            };
+        }
+        if (type === "list") {
+            rejectedListSource = normalizeXlsxSourceListFormula1(src.formula1);
+        }
+    }
+    const synthesized = convertDataValidationCondition(rule, baseConfig);
+    if (synthesized)
+        return synthesized;
+    // A rejected `list` source with no usable condition is still kept — its
+    // normalized source text (quoted literal or reference) is always valid
+    // OOXML. Anything else rejected has nothing valid to write and is skipped.
+    if (rejectedListSource !== undefined) {
         return {
-            type: src.type,
-            operator: src.operator,
-            formula1: src.formula1,
-            formula2: src.formula2,
-            showDropDown: src.type === "list" && rule.displayStyle === "plain" ? true : undefined,
+            type: "list",
+            formula1: rejectedListSource,
+            showDropDown: rule.displayStyle === "plain" ? true : undefined,
             ...baseConfig,
         };
     }
+    return undefined;
+};
+// The condition-based synthesizer: the Excel config for a rule that has no
+// replayable `xlsxSource` (UI-authored, edited after import, or rejected by
+// the round-trip guard above).
+const convertDataValidationCondition = (rule, baseConfig) => {
     if (!rule.condition?.type) {
         // Prompt-only DV (no validation rule, just an input-message tooltip
         // and/or an error alert). Emit it with no type/formula so Excel still
@@ -1129,7 +1270,6 @@ const convertDataValidationRule = (rule) => {
     const { condition } = rule;
     const values = condition.values ?? [];
     const currentCell = 'INDIRECT("RC",FALSE)';
-    const escapeFormulaText = (value) => value.replace(/"/g, '""');
     const ensureFormula = (value) => value.startsWith("=") ? value : `=${value}`;
     const ifErrorWrap = (formula) => `IFERROR(${formula},TRUE)`;
     if (condition.type === "ONE_OF_LIST") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rowsncolumns/toolkit",
-  "version": "24.5.0",
+  "version": "24.5.3",
   "author": "Vinay M",
   "license": "SEE LICENSE IN LICENSE.md",
   "main": "./dist/cjs/index.js",
@@ -45,7 +45,7 @@
     "url": "https://github.com/rowsncolumns/spreadsheet/issues"
   },
   "devDependencies": {
-    "@rowsncolumns/utils": ">=9.4.0",
+    "@rowsncolumns/utils": ">=9.5.1",
     "@types/lodash-es": "^4.17.12"
   },
   "peerDependencies": {
@@ -55,9 +55,9 @@
   "dependencies": {
     "@noble/ciphers": "^0.5.3",
     "@noble/hashes": "^1.4.0",
-    "@rowsncolumns/calculator": ">=7.2.33",
-    "@rowsncolumns/common-types": ">=5.2.34",
-    "@rowsncolumns/date-parser": ">=13.0.1",
+    "@rowsncolumns/calculator": ">=7.3.1",
+    "@rowsncolumns/common-types": ">=5.3.1",
+    "@rowsncolumns/date-parser": ">=13.0.2",
     "@zip.js/zip.js": "^2.8.17",
     "fast-xml-parser": "^5.8.0",
     "fflate": "^0.8.2",