@ricsam/excel-to-formula-engine 0.2.1 → 0.3.0

package/README.md

@@ -150,6 +150,29 @@ await excelToFormulaEngine(file, {
 | Number formats, hyperlinks | Cell metadata | Stored for your UI; the engine does not interpret them. |
 | Merges, frozen panes, column widths, hidden sheets | Sheet metadata | Same — stored for your UI to apply. |
 
+### Effective cell fonts
+
+Existing worksheet cells resolve their formatting in this order: explicit cell
+`s` (including `s="0"`), row `s` when `customFormat` is enabled, column `style`,
+then workbook cell XF 0. Column ranges remain sparse, including full-sheet
+ranges. Named-style font inheritance (`cellStyleXfs` / `xfId`) is resolved;
+`applyFont="0"` selects the base font, while an explicit applied font replaces
+it as a unit. Supported font decorations include explicit false values so a
+plain font can clear inherited bold, italic, or underline.
+
+The resolved XF 0 is also retained as `workbookMetadata.defaultCellStyle`
+(engine units: font sizes are pixels). This is additive metadata for clients;
+actual imported cells receive coalesced direct styles without requiring a new
+engine API. `styles:false` disables both. Empty coordinates not present in the
+worksheet are not materialized. Older imported snapshots do not retain XF 0
+and cannot reconstruct it losslessly from neighboring styles.
+
+Literal Excel font names are retained as `fontFamily` (clients must provide
+those fonts). This is not full Excel rendering fidelity: theme font substitution,
+table style themes, rich-text runs, and named-style inheritance of non-font
+properties are not translated by this change. Double/accounting underline
+variants become a single boolean underline.
+
 ### Formulas the engine cannot evaluate
 
 The engine implements around 40 functions. A formula using anything else — an

package/dist/index.cjs

@@ -280,6 +280,11 @@ function parseWorksheet(pkg, partPath, sharedStrings) {
 function parseCells(worksheet, sharedStrings, result) {
   const sheetData = worksheet["sheetData"];
   const shared = new Map;
+  const columnStyles = asArray(worksheet["cols"]?.["col"]).map((col) => ({
+    min: numAttr(col, "min"),
+    max: numAttr(col, "max"),
+    style: numAttr(col, "style")
+  })).filter((col) => col.min !== undefined && col.max !== undefined && col.style !== undefined);
   for (const rawRow of asArray(sheetData?.["row"])) {
     const row = rawRow;
     const rowNumber = numAttr(row, "r");
@@ -297,11 +302,14 @@ function parseCells(worksheet, sharedStrings, result) {
       if (!address) {
         continue;
       }
-      const styleIndex = numAttr(cell, "s");
+      const explicitStyle = numAttr(cell, "s");
+      const rowStyle = boolAttr(row, "customFormat") ? numAttr(row, "s") : undefined;
+      const columnStyle = columnStyles.findLast((col) => address.colIndex + 1 >= col.min && address.colIndex + 1 <= col.max)?.style;
+      const styleIndex = explicitStyle ?? rowStyle ?? columnStyle ?? 0;
       const type = attr(cell, "t") ?? "n";
       const formula = readFormula(cell, address, shared);
       const { kind, value } = readValue(cell, type, sharedStrings);
-      if (kind === "empty" && formula === undefined && styleIndex === undefined) {
+      if (kind === "empty" && formula === undefined && explicitStyle === undefined && rowStyle === undefined && columnStyle === undefined) {
         continue;
       }
       result.cells.push({
@@ -691,14 +699,19 @@ function parseStyles(pkg) {
   const fills = asArray(styleSheet["fills"]?.["fill"]);
   const borders = asArray(styleSheet["borders"]?.["border"]);
   const cellXfs = asArray(styleSheet["cellXfs"]?.["xf"]);
+  const styleXfs = asArray(styleSheet["cellStyleXfs"]?.["xf"]);
   const formats = cellXfs.map((rawXf) => {
     const xf = rawXf;
     const style = {};
-    if (boolAttr(xf, "applyFont", true)) {
-      const font = fonts[numAttr(xf, "fontId") ?? -1];
-      if (font) {
-        applyFont(style, font, theme, indexedColors);
-      }
+    const baseXf = styleXfs[numAttr(xf, "xfId") ?? -1];
+    const baseFontId = boolAttr(baseXf, "applyFont", true) ? numAttr(baseXf, "fontId") ?? 0 : 0;
+    const fontId = boolAttr(xf, "applyFont", true) ? numAttr(xf, "fontId") ?? baseFontId : baseFontId;
+    const font = fonts[fontId];
+    if (font) {
+      style.bold = false;
+      style.italic = false;
+      style.underline = false;
+      applyFont(style, font, theme, indexedColors);
     }
     if (boolAttr(xf, "applyFill", true)) {
       const fill = fills[numAttr(xf, "fillId") ?? -1];
@@ -743,14 +756,18 @@ function parseStyles(pkg) {
   return { formats, differentialFormats };
 }
 function applyFont(style, font, theme, indexedColors) {
-  if (font["b"] !== undefined && isOn(font["b"])) {
-    style.bold = true;
+  const fontFamily = attr(font["name"], "val");
+  if (fontFamily) {
+    style.fontFamily = fontFamily;
+  }
+  if (font["b"] !== undefined) {
+    style.bold = isOn(font["b"]);
   }
-  if (font["i"] !== undefined && isOn(font["i"])) {
-    style.italic = true;
+  if (font["i"] !== undefined) {
+    style.italic = isOn(font["i"]);
   }
-  if (font["u"] !== undefined && isOn(font["u"])) {
-    style.underline = true;
+  if (font["u"] !== undefined) {
+    style.underline = attr(font["u"], "val") !== "none";
   }
   const size = numAttr(font["sz"], "val");
   if (size !== undefined) {
@@ -1110,6 +1127,7 @@ function contractThisRowReferences(formulaBody, containingTableName) {
 var import_formula_engine2 = require("@ricsam/formula-engine");
 function translateConditionalFormat(rule, options) {
   const diagnostics = [];
+  const ranges = rule.ranges.map((reference) => reference.replace(/\$/g, ""));
   const areas = [];
   for (const reference of rule.ranges) {
     const range = rangeReferenceToSpreadsheetRange(reference);
@@ -1121,14 +1139,16 @@ function translateConditionalFormat(rule, options) {
     return { diagnostics };
   }
   const anchor = rule.ranges[0]?.split(":")[0]?.replace(/\$/g, "") ?? "A1";
-  if (rule.type === "colorScale") {
-    return translateColorScale(rule, areas, options, diagnostics);
-  }
-  const condition = buildFormulaCondition(rule, anchor, options, diagnostics);
-  if (!condition) {
-    return { diagnostics };
+  const result = rule.type === "colorScale" ? translateColorScale(rule, areas, options, diagnostics).result : withCondition(areas, buildFormulaCondition(rule, anchor, options, diagnostics));
+  for (const diagnostic of diagnostics) {
+    if (diagnostic.ranges === undefined && ranges.length > 0) {
+      diagnostic.ranges = ranges;
+    }
   }
-  return { result: { areas, condition }, diagnostics };
+  return { result, diagnostics };
+}
+function withCondition(areas, condition) {
+  return condition ? { areas, condition } : undefined;
 }
 function buildFormulaCondition(rule, anchor, options, diagnostics) {
   const color = ruleColor(rule, options);
@@ -1441,6 +1461,7 @@ async function convertWorkbook(source, options = {}) {
     cellStyles: cellStyles.length > 0 ? cellStyles : undefined,
     conditionalStyles: conditionalStyles.length > 0 ? conditionalStyles : undefined,
     workbookMetadata: {
+      defaultCellStyle: importStyles ? styleTable.formats[0]?.style : undefined,
       fileName: options.fileName,
       sheetOrder: sheetEntries.map((entry) => entry.name),
       diagnostics: diagnostics.length > 0 ? diagnostics : undefined
@@ -1900,4 +1921,4 @@ function withCsvFileName(source, options) {
   };
 }
 
-//# debugId=5AA8F8A121F02B2D64756E2164756E21
+//# debugId=F34603A2925533A764756E2164756E21