@shbernal/ts-xlsx 1.3.1 → 2.1.0

package/dist/io/xlsx/sheet-properties.js

@@ -1,6 +1,7 @@
 import { encodeAddress } from '../../core/address.js';
+import { isPageOrder, isPageOrientation, } from '../../core/page-setup.js';
 import { SHEET_PROTECTION_FLAGS } from '../../core/protection.js';
-import { attr, boolAttr, escapeAttr, escapeText, numberText } from '../../xml/xml.js';
+import { boolAttr, checkedToken, escapeAttr, escapeSpreadsheetText, numAttr, numberText, } from '../../xml/xml.js';
 import { colorAttrs } from './color-xml.js';
 export function sheetViewsXml(view, active) {
     const selected = active ? ' tabSelected="1"' : '';
@@ -99,7 +100,7 @@ export function headerFooterXml(hf) {
     if (differentFirst)
         attrs += ' differentFirst="1"';
     const body = children
-        .map(({ tag, key }) => `<${tag}>${escapeText(hf[key])}</${tag}>`)
+        .map(({ tag, key }) => `<${tag}>${escapeSpreadsheetText(hf[key])}</${tag}>`)
         .join('');
     return `<headerFooter${attrs}>${body}</headerFooter>`;
 }
@@ -127,12 +128,16 @@ export function pageMarginsXml(margins) {
     return `<pageMargins ${attrs}/>`;
 }
 export function pageSetupXml(pageSetup, printerSettingsRelId) {
-    const attrs = attr('paperSize', pageSetup.paperSize) +
-        attr('scale', pageSetup.scale) +
-        attr('fitToWidth', pageSetup.fitToWidth) +
-        attr('fitToHeight', pageSetup.fitToHeight) +
-        (pageSetup.pageOrder !== undefined ? ` pageOrder="${pageSetup.pageOrder}"` : '') +
-        (pageSetup.orientation !== undefined ? ` orientation="${pageSetup.orientation}"` : '') +
+    const attrs = numAttr('paperSize', pageSetup.paperSize) +
+        numAttr('scale', pageSetup.scale) +
+        numAttr('fitToWidth', pageSetup.fitToWidth) +
+        numAttr('fitToHeight', pageSetup.fitToHeight) +
+        (pageSetup.pageOrder === undefined
+            ? ''
+            : ` pageOrder="${checkedToken(pageSetup.pageOrder, isPageOrder, 'page order')}"`) +
+        (pageSetup.orientation === undefined
+            ? ''
+            : ` orientation="${checkedToken(pageSetup.orientation, isPageOrientation, 'page orientation')}"`) +
         (printerSettingsRelId !== null ? ` r:id="${printerSettingsRelId}"` : '');
     return attrs === '' ? '' : `<pageSetup${attrs}/>`;
 }
@@ -141,8 +146,7 @@ export function pageBreaksXml(breaks, element) {
         return '';
     const brks = breaks
         .map((brk) => {
-        const maxAttr = brk.max !== undefined ? ` max="${brk.max}"` : '';
-        return `<brk id="${brk.id}"${maxAttr} man="1"/>`;
+        return `<brk${numAttr('id', brk.id)}${numAttr('max', brk.max)} man="1"/>`;
     })
         .join('');
     return `<${element} count="${breaks.length}" manualBreakCount="${breaks.length}">${brks}</${element}>`;

package/dist/io/xlsx/styles.d.ts

@@ -1,22 +1,10 @@
-import { type Alignment, type Border, type DifferentialStyle, type Fill, type Font, type NamedCellStyle, type Protection, type TableStyleTable } from '../../core/style.ts';
+import { type DifferentialStyle, type Font, type NamedCellStyle, type TableStyleTable } from '../../core/style.ts';
 import { type TableStyle } from '../../core/table-style.ts';
-/** A cell's style facets as the writer composes them: cell overrides atop row/column defaults. */
-export interface CellStyle {
-    readonly fill?: Fill | undefined;
-    readonly numFmt?: string | undefined;
-    readonly font?: Font | undefined;
-    readonly border?: Border | undefined;
-    readonly alignment?: Alignment | undefined;
-    readonly protection?: Protection | undefined;
-    /** The quote-prefix flag — an attribute on the xf, not a shared sub-table entry. */
-    readonly quotePrefix?: boolean | undefined;
-    /** The `xfId` link into `cellStyleXfs` — the named cell style this format inherits from (0 = Normal). */
-    readonly xfId?: number | undefined;
-}
+import type { XfStyle } from '../style/xf-style.ts';
 /** What a {@link StyleRegistry} needs from its workbook before any style is interned. */
 export interface StyleRegistryOptions {
     /**
-     * The font every cell naming none of its own renders in — emitted as id 0. Take it from
+     * The font every cell naming none of its own renders in, emitted as id 0. Take it from
      * {@link Workbook.defaultFont}, which resolves the authored/declared/theme chain and guarantees a
      * complete entry. Omitted, the registry falls back to the Office default.
      */
@@ -28,7 +16,7 @@ export interface StyleRegistryOptions {
      *
      * The reader flattens font 0 onto every xf that names it, so a cell that merely inherited the
      * file's default arrives carrying it as a concrete face. That face is an artefact of reading, not
-     * an authored intent — in the source file the cell said nothing about its font — so it must
+     * an authored intent (in the source file the cell said nothing about its font) so it must
      * collapse back to id 0 and follow the new default rather than pin itself to the old one through a
      * custom entry.
      */
@@ -41,7 +29,7 @@ export declare class StyleRegistry {
      * The `<cellXfs>` index for a composed cell/row/column style. A style with no facet needs
      * no entry and resolves to the default xf 0, so its owner emits no `s` attribute at all.
      */
-    styleId(style: CellStyle): number;
+    styleId(style: XfStyle): number;
     /**
      * Seed the named cell styles (`<cellStyleXfs>`/`<cellStyles>`) read from a file, in place of the
      * lone default, interning each style's facets into the shared sub-tables so its `fillId`/`fontId`/…
@@ -59,7 +47,7 @@ export declare class StyleRegistry {
      * rule's `dxfId`, and every `<tableStyleElement dxfId="…">` inside a preserved `<tableStyle>` (see
      * {@link seedTableStyles}). Those constructs are carried as opaque XML precisely *because* the
      * indices they name do not move. Renumbering, reordering, or de-duplicating the seeded entries
-     * would silently re-point every one of them at a different format — a change no schema check and no
+     * would silently re-point every one of them at a different format: a change no schema check and no
      * round-trip of our own can catch, because the file stays perfectly valid and merely renders wrong.
      */
     seedDifferentialStyles(fragments: readonly string[]): void;
@@ -88,13 +76,13 @@ export declare class StyleRegistry {
      * {@link seedTableStyles}, whose preserved definitions these append after.
      *
      * A definition here **replaces** a preserved one of the same name. Two `<tableStyle>` elements
-     * sharing a name is ambiguous — a table's `tableStyleInfo/@name` would reach whichever a consumer
-     * happened to index first — so authoring a name the source already used is read as overriding it,
+     * sharing a name is ambiguous, since a table's `tableStyleInfo/@name` would reach whichever a
+     * consumer happened to index first, so authoring a name the source already used is read as overriding it,
      * which is what asking for it means.
      */
     addTableStyle(style: TableStyle): void;
     /**
-     * Intern an authored differential style, returning the `<dxfs>` index that references it — a
+     * Intern an authored differential style, returning the `<dxfs>` index that references it: a
      * conditional-formatting rule's `dxfId`, or a table style element's. Identical styles collapse to
      * one entry, whichever feature asked for them, so a highlight rule and a table style's header row
      * painted the same way share a single `<dxf>`.
@@ -107,4 +95,3 @@ export declare class StyleRegistry {
     toXml(): string;
 }
 export declare function fontXml(font: Font, nameTag?: 'name' | 'rFont'): string;
-export declare function dxfXml(style: DifferentialStyle): string;

package/dist/io/xlsx/styles.js

@@ -1,8 +1,7 @@
 import { isBorderStyle, isFillPatternType, isFontScheme, isFontVerticalAlignment, isHorizontalAlignment, isNamedUnderlineStyle, isVerticalAlignment, } from '../../core/style.js';
 import { TABLE_STYLE_ELEMENT_TYPES } from '../../core/table-style.js';
-import { AuthoringError } from '../../errors.js';
-import { escapeAttr, XML_DECLARATION } from '../../xml/xml.js';
 import { decodeEntities } from '../../xml/xml-read.js';
+import { assertRepresentable, checkedToken, escapeAttr, numberText, XML_DECLARATION, } from '../../xml/xml.js';
 import { colorAttrs } from './color-xml.js';
 import { MARKUP_COMPATIBILITY_NS, SPREADSHEETML_NS } from './namespaces.js';
 const RESERVED_FILL_COUNT = 2;
@@ -21,6 +20,36 @@ const DEFAULT_FORMAT = {
     quotePrefix: false,
     xfId: 0,
 };
+class InternTable {
+    #entries = [];
+    #idByKey = new Map();
+    #base;
+    constructor(base) {
+        this.#base = base;
+    }
+    get entries() {
+        return this.#entries;
+    }
+    get size() {
+        return this.#entries.length;
+    }
+    intern(key, entry) {
+        const existing = this.#idByKey.get(key);
+        if (existing !== undefined)
+            return existing;
+        const id = this.#base + this.#entries.length;
+        this.#entries.push(entry);
+        this.#idByKey.set(key, id);
+        return id;
+    }
+    seed(entry) {
+        const id = this.#base + this.#entries.length;
+        this.#entries.push(entry);
+        if (!this.#idByKey.has(entry))
+            this.#idByKey.set(entry, id);
+        return id;
+    }
+}
 export class StyleRegistry {
     #defaultFontBody;
     #font0Bodies;
@@ -31,22 +60,17 @@ export class StyleRegistry {
             ? [this.#defaultFontBody]
             : [this.#defaultFontBody, fontXml(options.declaredDefaultFont)]);
     }
-    #fillXml = [];
-    #fillIdBySignature = new Map();
-    #numFmtCodes = [];
-    #numFmtIdByCode = new Map();
-    #fontXml = [];
-    #fontIdBySignature = new Map();
-    #borderXml = [];
-    #borderIdBySignature = new Map();
+    #fills = new InternTable(RESERVED_FILL_COUNT);
+    #numFmts = new InternTable(CUSTOM_NUMFMT_BASE);
+    #fonts = new InternTable(RESERVED_FONT_COUNT);
+    #borders = new InternTable(RESERVED_BORDER_COUNT);
     #formats = [DEFAULT_FORMAT];
     #xfIndexBySignature = new Map();
     #cellStyleXfs = [DEFAULT_FORMAT];
     #cellStyleNames = [
         { name: 'Normal', builtinId: 0, xfId: 0 },
     ];
-    #dxfXml = [];
-    #dxfIndexByFragment = new Map();
+    #dxfs = new InternTable(0);
     #indexedColors = [];
     #mruColors = [];
     #tableStyles = { styles: [] };
@@ -94,12 +118,8 @@ export class StyleRegistry {
         });
     }
     seedDifferentialStyles(fragments) {
-        for (const fragment of fragments) {
-            const index = this.#dxfXml.length;
-            this.#dxfXml.push(fragment);
-            if (!this.#dxfIndexByFragment.has(fragment))
-                this.#dxfIndexByFragment.set(fragment, index);
-        }
+        for (const fragment of fragments)
+            this.#dxfs.seed(fragment);
     }
     seedIndexedColors(fragments) {
         this.#indexedColors.length = 0;
@@ -128,69 +148,36 @@ export class StyleRegistry {
     }
     differentialStyleId(style) {
         const fragment = dxfXml(style);
-        let index = this.#dxfIndexByFragment.get(fragment);
-        if (index === undefined) {
-            index = this.#dxfXml.length;
-            this.#dxfXml.push(fragment);
-            this.#dxfIndexByFragment.set(fragment, index);
-        }
-        return index;
+        return this.#dxfs.intern(fragment, fragment);
     }
     #internFill(fill) {
-        const signature = fillSignature(fill);
-        let id = this.#fillIdBySignature.get(signature);
-        if (id === undefined) {
-            id = RESERVED_FILL_COUNT + this.#fillXml.length;
-            this.#fillXml.push(patternFillXml(fill, { solidBgFallback: true }));
-            this.#fillIdBySignature.set(signature, id);
-        }
-        return id;
+        return this.#fills.intern(fillSignature(fill), patternFillXml(fill, { solidBgFallback: true }));
     }
     #internNumFmt(code) {
-        let id = this.#numFmtIdByCode.get(code);
-        if (id === undefined) {
-            id = CUSTOM_NUMFMT_BASE + this.#numFmtCodes.length;
-            this.#numFmtCodes.push(code);
-            this.#numFmtIdByCode.set(code, id);
-        }
-        return id;
+        return this.#numFmts.intern(code, code);
     }
     #internFont(font) {
         const xml = fontXml(font);
         if (xml === '' || this.#font0Bodies.has(xml))
             return 0;
-        let id = this.#fontIdBySignature.get(xml);
-        if (id === undefined) {
-            id = RESERVED_FONT_COUNT + this.#fontXml.length;
-            this.#fontXml.push(`<font>${xml}</font>`);
-            this.#fontIdBySignature.set(xml, id);
-        }
-        return id;
+        return this.#fonts.intern(xml, `<font>${xml}</font>`);
     }
     #internBorder(border) {
         const xml = borderXml(border);
-        if (xml === DEFAULT_BORDER)
-            return 0;
-        let id = this.#borderIdBySignature.get(xml);
-        if (id === undefined) {
-            id = RESERVED_BORDER_COUNT + this.#borderXml.length;
-            this.#borderXml.push(xml);
-            this.#borderIdBySignature.set(xml, id);
-        }
-        return id;
+        return xml === DEFAULT_BORDER ? 0 : this.#borders.intern(xml, xml);
     }
     toXml() {
-        const fillCount = RESERVED_FILL_COUNT + this.#fillXml.length;
+        const fillCount = RESERVED_FILL_COUNT + this.#fills.size;
         const fills = '<fill><patternFill patternType="none"/></fill>' +
             '<fill><patternFill patternType="gray125"/></fill>' +
-            this.#fillXml.join('');
+            this.#fills.entries.join('');
         const cellXfs = this.#formats.map((format) => xfXml(format, format.xfId)).join('');
         const cellStyleXfs = this.#cellStyleXfs.map((format) => xfXml(format, null)).join('');
         const cellStyles = this.#cellStyleNames.map(cellStyleTag).join('');
-        const fontCount = RESERVED_FONT_COUNT + this.#fontXml.length;
-        const fonts = `<font>${this.#defaultFontBody}</font>` + this.#fontXml.join('');
-        const borderCount = RESERVED_BORDER_COUNT + this.#borderXml.length;
-        const borders = DEFAULT_BORDER + this.#borderXml.join('');
+        const fontCount = RESERVED_FONT_COUNT + this.#fonts.size;
+        const fonts = `<font>${this.#defaultFontBody}</font>${this.#fonts.entries.join('')}`;
+        const borderCount = RESERVED_BORDER_COUNT + this.#borders.size;
+        const borders = DEFAULT_BORDER + this.#borders.entries.join('');
         return (XML_DECLARATION +
             `<styleSheet xmlns="${SPREADSHEETML_NS}"${this.#foreignNamespaceAttrs()}>` +
             this.#numFmtsXml() +
@@ -245,17 +232,17 @@ export class StyleRegistry {
         return `<colors>${indexed}${mru}</colors>`;
     }
     #dxfsXml() {
-        if (this.#dxfXml.length === 0)
+        if (this.#dxfs.size === 0)
             return '<dxfs count="0"/>';
-        return `<dxfs count="${this.#dxfXml.length}">${this.#dxfXml.join('')}</dxfs>`;
+        return `<dxfs count="${this.#dxfs.size}">${this.#dxfs.entries.join('')}</dxfs>`;
     }
     #numFmtsXml() {
-        if (this.#numFmtCodes.length === 0)
+        if (this.#numFmts.size === 0)
             return '';
-        const entries = this.#numFmtCodes
+        const entries = this.#numFmts.entries
             .map((code, i) => `<numFmt numFmtId="${CUSTOM_NUMFMT_BASE + i}" formatCode="${escapeFormatCode(code)}"/>`)
             .join('');
-        return `<numFmts count="${this.#numFmtCodes.length}">${entries}</numFmts>`;
+        return `<numFmts count="${this.#numFmts.size}">${entries}</numFmts>`;
     }
 }
 function isDefaultFormat(format) {
@@ -292,12 +279,6 @@ function cellStyleTag(entry) {
     const builtin = entry.builtinId === undefined ? '' : ` builtinId="${entry.builtinId}"`;
     return `<cellStyle name="${escapeAttr(entry.name)}" xfId="${entry.xfId}"${builtin}/>`;
 }
-function checkedToken(value, isValid, kind) {
-    if (!isValid(value)) {
-        throw new AuthoringError(`Invalid ${kind} ${JSON.stringify(value)}: not a value the OOXML enumeration allows`);
-    }
-    return value;
-}
 function alignmentAttrs(alignment) {
     const parts = [];
     if (alignment.horizontal !== undefined && alignment.horizontal !== 'general') {
@@ -307,17 +288,17 @@ function alignmentAttrs(alignment) {
         parts.push(`vertical="${checkedToken(alignment.vertical, isVerticalAlignment, 'vertical alignment')}"`);
     }
     if (alignment.textRotation !== undefined && alignment.textRotation !== 0) {
-        parts.push(`textRotation="${numberAttr(alignment.textRotation)}"`);
+        parts.push(`textRotation="${numberText(alignment.textRotation)}"`);
     }
     if (alignment.wrapText)
         parts.push('wrapText="1"');
     if (alignment.indent !== undefined && alignment.indent !== 0) {
-        parts.push(`indent="${numberAttr(alignment.indent)}"`);
+        parts.push(`indent="${numberText(alignment.indent)}"`);
     }
     if (alignment.shrinkToFit)
         parts.push('shrinkToFit="1"');
     if (alignment.readingOrder !== undefined && alignment.readingOrder !== 0) {
-        parts.push(`readingOrder="${numberAttr(alignment.readingOrder)}"`);
+        parts.push(`readingOrder="${numberText(alignment.readingOrder)}"`);
     }
     return parts.join(' ');
 }
@@ -349,20 +330,20 @@ export function fontXml(font, nameTag = 'name') {
         parts.push(`<vertAlign val="${checkedToken(font.vertAlign, isFontVerticalAlignment, 'font vertical alignment')}"/>`);
     }
     if (font.size !== undefined)
-        parts.push(`<sz val="${numberAttr(font.size)}"/>`);
+        parts.push(`<sz val="${numberText(font.size)}"/>`);
     if (font.color !== undefined)
         parts.push(`<color ${colorAttrs(font.color)}/>`);
     if (font.name !== undefined)
         parts.push(`<${nameTag} val="${escapeAttr(font.name)}"/>`);
     if (font.family !== undefined)
-        parts.push(`<family val="${numberAttr(font.family)}"/>`);
+        parts.push(`<family val="${numberText(font.family)}"/>`);
     if (font.charset !== undefined)
-        parts.push(`<charset val="${numberAttr(font.charset)}"/>`);
+        parts.push(`<charset val="${numberText(font.charset)}"/>`);
     if (font.scheme !== undefined && font.scheme !== 'none')
         parts.push(`<scheme val="${checkedToken(font.scheme, isFontScheme, 'font scheme')}"/>`);
     return parts.join('');
 }
-export function dxfXml(style) {
+function dxfXml(style) {
     const parts = [];
     if (style.font !== undefined) {
         const font = fontXml(style.font);
@@ -380,18 +361,18 @@ export function dxfXml(style) {
 }
 function gradientFillXml(fill) {
     const attrs = (fill.gradient === 'path' ? ' type="path"' : '') +
-        (fill.degree ? ` degree="${numberAttr(fill.degree)}"` : '') +
+        (fill.degree ? ` degree="${numberText(fill.degree)}"` : '') +
         insetAttr('left', fill.left) +
         insetAttr('right', fill.right) +
         insetAttr('top', fill.top) +
         insetAttr('bottom', fill.bottom);
     const stops = fill.stops
-        .map((stop) => `<stop position="${numberAttr(stop.position)}"><color ${colorAttrs(stop.color)}/></stop>`)
+        .map((stop) => `<stop position="${numberText(stop.position)}"><color ${colorAttrs(stop.color)}/></stop>`)
         .join('');
     return `<gradientFill${attrs}>${stops}</gradientFill>`;
 }
 function insetAttr(name, value) {
-    return value ? ` ${name}="${numberAttr(value)}"` : '';
+    return value ? ` ${name}="${numberText(value)}"` : '';
 }
 function underlineXml(underline) {
     if (underline === undefined || underline === false || underline === 'none')
@@ -400,12 +381,6 @@ function underlineXml(underline) {
         return '<u/>';
     return `<u val="${checkedToken(underline, isNamedUnderlineStyle, 'underline style')}"/>`;
 }
-function numberAttr(value) {
-    if (!Number.isFinite(value)) {
-        throw new AuthoringError(`cannot serialise a non-finite font metric (${value})`);
-    }
-    return String(value);
-}
 function borderXml(border) {
     const attrs = (border.diagonalUp ? ' diagonalUp="1"' : '') + (border.diagonalDown ? ' diagonalDown="1"' : '');
     return (`<border${attrs}>` +
@@ -425,6 +400,7 @@ function edgeXml(tag, edge) {
     return `<${tag} style="${style}"><color ${colorAttrs(edge.color)}/></${tag}>`;
 }
 function escapeFormatCode(code) {
+    assertRepresentable(code);
     return code
         .replace(/&/g, '&amp;')
         .replace(/</g, '&lt;')

package/dist/io/xlsx/tables.d.ts

@@ -2,8 +2,8 @@ import { type Table, type TableOptions } from '../../core/table.ts';
 export declare function tableXml(table: Table, id: number): string;
 /**
  * Parse a `<table>` part into the options that reconstruct it, or `undefined` when the XML is not a
- * usable table (no name, no ref, or no columns — Excel treats such a part as corrupt, so we drop it
- * rather than fabricate a degenerate table). Duplicate column names are not resolved here — the
+ * usable table (no name, no ref, or no columns: Excel treats such a part as corrupt, so we drop it
+ * rather than fabricate a degenerate table). Duplicate column names are not resolved here, since the
  * {@link Table} constructor disambiguates them, so authoring and loading share one implementation.
  */
 export declare function parseTable(xml: string): TableOptions | undefined;

package/dist/io/xlsx/tables.js

@@ -1,7 +1,7 @@
-import { decodeRange, encodeAddress } from '../../core/address.js';
+import { encodeAddress, tryDecodeRange } from '../../core/address.js';
 import { isTotalsRowFunction, } from '../../core/table.js';
+import { boolPresent, localName, numInteger, parseXml, TextCapture } from '../../xml/xml-read.js';
 import { boolAttr, escapeAttr, escapeText, XML_DECLARATION } from '../../xml/xml.js';
-import { localName, parseXml } from '../../xml/xml-read.js';
 import { NS } from './relationships.js';
 export function tableXml(table, id) {
     const name = escapeAttr(table.name);
@@ -55,9 +55,6 @@ function tableColumnXml(column, id) {
     }
     return `<tableColumn ${attrs}/>`;
 }
-function parseOoxmlBool(value) {
-    return value !== '0' && value !== 'false';
-}
 export function parseTable(xml) {
     let name;
     let displayName;
@@ -68,21 +65,18 @@ export function parseTable(xml) {
     let style;
     let hasAutoFilter = false;
     const columns = [];
-    let inTotalsFormula = false;
-    let totalsFormula = '';
+    const totalsFormula = new TextCapture('totalsRowFormula');
     parseXml(xml, {
-        onOpen(elementName, attrs) {
+        onOpen(elementName, attrs, selfClosing) {
             switch (localName(elementName)) {
                 case 'table':
                     name = attrs.name ?? attrs.displayName;
                     displayName = attrs.displayName ?? attrs.name;
                     ref = attrs.ref;
-                    if (attrs.headerRowCount !== undefined)
-                        headerRowCount = Number(attrs.headerRowCount);
-                    if (attrs.totalsRowCount !== undefined)
-                        totalsRowCount = Number(attrs.totalsRowCount);
+                    headerRowCount = numInteger(attrs.headerRowCount, 0) ?? headerRowCount;
+                    totalsRowCount = numInteger(attrs.totalsRowCount, 0) ?? totalsRowCount;
                     if (attrs.totalsRowShown !== undefined)
-                        totalsRowShown = parseOoxmlBool(attrs.totalsRowShown);
+                        totalsRowShown = boolPresent(attrs.totalsRowShown);
                     break;
                 case 'autoFilter':
                     hasAutoFilter = true;
@@ -92,13 +86,13 @@ export function parseTable(xml) {
                     if (attrs.name !== undefined)
                         captured.name = attrs.name;
                     if (attrs.showFirstColumn !== undefined)
-                        captured.showFirstColumn = parseOoxmlBool(attrs.showFirstColumn);
+                        captured.showFirstColumn = boolPresent(attrs.showFirstColumn);
                     if (attrs.showLastColumn !== undefined)
-                        captured.showLastColumn = parseOoxmlBool(attrs.showLastColumn);
+                        captured.showLastColumn = boolPresent(attrs.showLastColumn);
                     if (attrs.showRowStripes !== undefined)
-                        captured.showRowStripes = parseOoxmlBool(attrs.showRowStripes);
+                        captured.showRowStripes = boolPresent(attrs.showRowStripes);
                     if (attrs.showColumnStripes !== undefined) {
-                        captured.showColumnStripes = parseOoxmlBool(attrs.showColumnStripes);
+                        captured.showColumnStripes = boolPresent(attrs.showColumnStripes);
                     }
                     style = captured;
                     break;
@@ -117,27 +111,28 @@ export function parseTable(xml) {
                     break;
                 }
                 case 'totalsRowFormula':
-                    inTotalsFormula = true;
-                    totalsFormula = '';
+                    totalsFormula.open('totalsRowFormula', selfClosing);
                     break;
             }
         },
         onText(text) {
-            if (inTotalsFormula)
-                totalsFormula += text;
+            totalsFormula.text(text);
         },
         onClose(elementName) {
-            if (localName(elementName) !== 'totalsRowFormula')
+            const formula = totalsFormula.close(localName(elementName));
+            if (formula === undefined)
                 return;
-            inTotalsFormula = false;
             const column = columns[columns.length - 1];
             if (column !== undefined)
-                column.totalsRowFormula = totalsFormula;
+                column.totalsRowFormula = formula;
         },
     });
     if (name === undefined || ref === undefined || columns.length === 0)
         return undefined;
-    const { top, left, bottom } = decodeRange(ref);
+    const decoded = tryDecodeRange(ref);
+    if (decoded === undefined)
+        return undefined;
+    const { top, left, bottom } = decoded;
     if (top === undefined || left === undefined || bottom === undefined)
         return undefined;
     const headerRow = headerRowCount !== 0;

package/dist/io/xlsx/theme-xml.d.ts

@@ -0,0 +1,37 @@
+import { type ThemeColorScheme, type ThemeFontScheme, type ThemeOverrides } from '../../core/theme.ts';
+/**
+ * Extract the colour scheme from a theme part. Returns only the slots the part actually declares in a
+ * colour model this reader understands; an unrecognised one is dropped rather than guessed at, so a
+ * caller can tell "the theme says nothing here" from "the theme says black".
+ *
+ * Reads the `<a:clrScheme>` block alone. A theme carries a font scheme and a format scheme too, but
+ * neither participates in resolving a colour, and scanning the whole part would let a `<a:srgbClr>`
+ * buried in a gradient stop masquerade as a scheme slot.
+ */
+export declare function parseThemeColorScheme(themeXml: string): ThemeColorScheme;
+/** Extract the major/minor latin typefaces from a theme part's `<a:fontScheme>`. */
+export declare function parseThemeFontScheme(themeXml: string): ThemeFontScheme;
+/**
+ * Apply authored colour/font overrides to a theme part, returning the new part text.
+ *
+ * Surgical by design: the base part rides through untouched except for the `<a:clrScheme>` and
+ * `<a:fontScheme>` blocks, and within those, only what the caller actually named. The format scheme,
+ * the gradients, line styles and effect styles that make a theme look like a theme, is left exactly
+ * as it was, because nobody hand-authors `fillStyleLst` gradient stops from a spreadsheet API and
+ * regenerating it would replace a designer's work with the Office default.
+ *
+ * A slot the caller did not override keeps its **verbatim source element**, not a re-serialisation of
+ * its value. That matters for `dk1`/`lt1`, which Excel writes as `<a:sysClr val="windowText"
+ * lastClr="000000"/>`: rewriting those as `<a:srgbClr>` would pin them to one machine's resolved
+ * window colours and break dark-mode following.
+ */
+export declare function applyThemeOverrides(baseXml: string, overrides: ThemeOverrides): string;
+/**
+ * The theme part a workbook with no theme of its own ships: the standard Office theme.
+ *
+ * A spreadsheet must carry one even when nobody configured it: the stylesheet's own default font
+ * references `theme="1"`, which a consumer can only resolve against this part, so the two travel
+ * together. It is also the base {@link applyThemeOverrides} authors on top of when a workbook was
+ * built from scratch rather than read from a file.
+ */
+export declare const DEFAULT_THEME_XML: string;