@cj-tech-master/excelts 5.0.6 → 5.1.0

package/dist/cjs/modules/excel/utils/string-buf.js

@@ -36,7 +36,7 @@ class StringBuf {
         return this._buffer;
     }
     reset(position) {
-        position = position || 0;
+        position = position ?? 0;
         this._buffer = undefined;
         this._inPos = position;
     }

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

@@ -233,8 +233,8 @@ class Workbook {
     // ===========================================================================
     get model() {
         return {
-            creator: this.creator || "Unknown",
-            lastModifiedBy: this.lastModifiedBy || "Unknown",
+            creator: this.creator ?? "Unknown",
+            lastModifiedBy: this.lastModifiedBy ?? "Unknown",
             lastPrinted: this.lastPrinted,
             created: this.created,
             modified: this.modified,
@@ -296,10 +296,9 @@ class Workbook {
         this._definedNames.model = value.definedNames;
         this.views = value.views;
         this._themes = value.themes;
-        this.media = value.media || [];
+        this.media = value.media ?? [];
         // Handle pivot tables - either newly created or loaded from file
-        // Loaded pivot tables come from loadedPivotTables after reconciliation
-        this.pivotTables = value.pivotTables || value.loadedPivotTables || [];
+        this.pivotTables = value.pivotTables ?? [];
         // Preserve passthrough files (charts, etc.) for round-trip preservation
         this._passthrough = value.passthrough || {};
         // Preserve raw drawing data for drawings with chart references

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

@@ -24,12 +24,12 @@ class Worksheet {
         options = options || {};
         this._workbook = options.workbook;
         // in a workbook, each sheet will have a number
-        this.id = options.id || 0;
-        this.orderNo = options.orderNo || 0;
+        this.id = options.id ?? 0;
+        this.orderNo = options.orderNo ?? 0;
         // and a name - use the setter to ensure validation and truncation
         this.name = options.name || `sheet${this.id}`;
         // add a state
-        this.state = options.state || "visible";
+        this.state = options.state ?? "visible";
         // rows allows access organised by row. Sparse array of arrays indexed by row-1, col
         // Note: _rows is zero based. Must subtract 1 to go from cell.row to index
         this._rows = [];
@@ -89,8 +89,8 @@ class Worksheet {
         }, options.headerFooter);
         this.dataValidations = new data_validations_1.DataValidations();
         // for freezepanes, split, zoom, gridlines, etc
-        this.views = options.views || [];
-        this.autoFilter = options.autoFilter || null;
+        this.views = options.views ?? [];
+        this.autoFilter = options.autoFilter ?? null;
         // for images, etc
         this._media = [];
         // worksheet protection
@@ -253,7 +253,7 @@ class Worksheet {
         if (inserts.length > 0) {
             // must iterate over all rows whether they exist yet or not
             for (let i = 0; i < nRows; i++) {
-                const insertValues = inserts.map(insert => insert[i] || null);
+                const insertValues = inserts.map(insert => insert[i] ?? null);
                 const row = this.getRow(i + 1);
                 row.splice(start, count, ...insertValues);
             }
@@ -933,7 +933,7 @@ class Worksheet {
         };
         // =================================================
         // columns
-        model.cols = column_1.Column.toModel(this.columns || []);
+        model.cols = column_1.Column.toModel(this.columns ?? []);
         // ==========================================================
         // Rows
         const rows = (model.rows = []);
@@ -981,8 +981,8 @@ class Worksheet {
         this.properties = value.properties;
         this.pageSetup = value.pageSetup;
         this.headerFooter = value.headerFooter;
-        this.rowBreaks = value.rowBreaks || [];
-        this.colBreaks = value.colBreaks || [];
+        this.rowBreaks = value.rowBreaks ?? [];
+        this.colBreaks = value.colBreaks ?? [];
         this.views = value.views;
         this.autoFilter = value.autoFilter;
         this._media = value.media.map(medium => new image_1.Image(this, medium));

package/dist/cjs/modules/excel/xlsx/xform/base-xform.js

@@ -65,7 +65,7 @@ class BaseXform {
     // ============================================================
     reset() {
         // to make sure parses don't bleed to next iteration
-        this.model = null;
+        this.model = undefined;
         // if we have a map - reset them too
         if (this.map) {
             Object.values(this.map).forEach(xform => {

package/dist/cjs/modules/excel/xlsx/xform/book/defined-name-xform.js

@@ -61,7 +61,7 @@ function isValidRange(range) {
         return false;
     }
     // Extract the cell reference part (after the sheet name if present)
-    const cellRef = range.split("!").pop() || "";
+    const cellRef = range.split("!").pop() ?? "";
     // Must match one of the valid patterns
     if (!cellRangeRegexp.test(cellRef) &&
         !rowRangeRegexp.test(cellRef) &&
@@ -97,7 +97,7 @@ function extractRanges(parsedText) {
         if (!item) {
             return;
         }
-        const quotes = (item.match(/'/g) || []).length;
+        const quotes = (item.match(/'/g) ?? []).length;
         if (!quotes) {
             if (quotesOpened) {
                 last += `${item},`;

package/dist/cjs/modules/excel/xlsx/xform/book/workbook-view-xform.js

@@ -5,10 +5,10 @@ const base_xform_1 = require("../base-xform.js");
 class WorkbookViewXform extends base_xform_1.BaseXform {
     render(xmlStream, model) {
         const attributes = {
-            xWindow: model.x || 0,
-            yWindow: model.y || 0,
-            windowWidth: model.width || 12000,
-            windowHeight: model.height || 24000,
+            xWindow: model.x ?? 0,
+            yWindow: model.y ?? 0,
+            windowWidth: model.width ?? 12000,
+            windowHeight: model.height ?? 24000,
             firstSheet: model.firstSheet,
             activeTab: model.activeTab
         };

package/dist/cjs/modules/excel/xlsx/xform/book/workbook-xform.js

@@ -77,7 +77,7 @@ class WorkbookXform extends base_xform_1.BaseXform {
         if (printAreas.length) {
             model.definedNames = model.definedNames.concat(printAreas);
         }
-        (model.media || []).forEach((medium, i) => {
+        (model.media ?? []).forEach((medium, i) => {
             // assign name
             medium.name = medium.type + (i + 1);
         });
@@ -91,7 +91,19 @@ class WorkbookXform extends base_xform_1.BaseXform {
         this.map.sheets.render(xmlStream, model.sheets);
         this.map.definedNames.render(xmlStream, model.definedNames);
         this.map.calcPr.render(xmlStream, model.calcProperties);
-        this.map.pivotCaches.render(xmlStream, model.pivotTables);
+        // R9-B6: Deduplicate pivot caches by cacheId before rendering.
+        // Multiple pivot tables may share the same cache, but workbook.xml should
+        // only list each cache once in <pivotCaches>.
+        const pivotTables = model.pivotTables ?? [];
+        const seenCacheIds = new Set();
+        const uniquePivotCaches = pivotTables.filter((pt) => {
+            if (seenCacheIds.has(pt.cacheId)) {
+                return false;
+            }
+            seenCacheIds.add(pt.cacheId);
+            return true;
+        });
+        this.map.pivotCaches.render(xmlStream, uniquePivotCaches);
         xmlStream.closeNode();
     }
     parseOpen(node) {
@@ -143,7 +155,7 @@ class WorkbookXform extends base_xform_1.BaseXform {
         }
     }
     reconcile(model) {
-        const rels = (model.workbookRels || []).reduce((map, rel) => {
+        const rels = (model.workbookRels ?? []).reduce((map, rel) => {
             map[rel.Id] = rel;
             return map;
         }, {});
@@ -151,7 +163,7 @@ class WorkbookXform extends base_xform_1.BaseXform {
         const worksheets = [];
         let worksheet;
         let index = 0;
-        (model.sheets || []).forEach((sheet) => {
+        (model.sheets ?? []).forEach((sheet) => {
             const rel = rels[sheet.rId];
             if (!rel) {
                 return;

package/dist/cjs/modules/excel/xlsx/xform/comment/style/vml-position-xform.js

@@ -9,7 +9,7 @@ class VmlPositionXform extends base_xform_1.BaseXform {
         this.model = {};
     }
     get tag() {
-        return (this._model && this._model.tag) || "";
+        return this._model?.tag ?? "";
     }
     render(xmlStream, model, type) {
         if (type && model === type[2]) {

package/dist/cjs/modules/excel/xlsx/xform/comment/style/vml-protection-xform.js

@@ -9,7 +9,7 @@ class VmlProtectionXform extends base_xform_1.BaseXform {
         this.text = "";
     }
     get tag() {
-        return (this._model && this._model.tag) || "";
+        return this._model?.tag ?? "";
     }
     render(xmlStream, model) {
         xmlStream.leafNode(this.tag, null, model);

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

@@ -16,7 +16,7 @@ class VmlShapeXform extends base_xform_1.BaseXform {
         return "v:shape";
     }
     render(xmlStream, model, index) {
-        xmlStream.openNode("v:shape", VmlShapeXform.V_SHAPE_ATTRIBUTES(model, index || 0));
+        xmlStream.openNode("v:shape", VmlShapeXform.V_SHAPE_ATTRIBUTES(model, index ?? 0));
         xmlStream.leafNode("v:fill", { color2: "infoBackground [80]" });
         xmlStream.leafNode("v:shadow", { color: "none [81]", obscured: "t" });
         xmlStream.leafNode("v:path", { "o:connecttype": "none" });

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

@@ -14,7 +14,7 @@ class VmlTextboxXform extends base_xform_1.BaseXform {
         return `${(parseFloat(value.toString()) * multiple).toFixed(2)}${unit}`;
     }
     reverseConversionUnit(inset) {
-        return (inset || "").split(",").map(margin => {
+        return (inset ?? "").split(",").map(margin => {
             return Number(parseFloat(this.conversionUnit(parseFloat(margin), 0.1, "")).toFixed(2));
         });
     }

package/dist/cjs/modules/excel/xlsx/xform/core/app-xform.js

@@ -24,7 +24,7 @@ class AppXform extends base_xform_1.BaseXform {
         xmlStream.leafNode("ScaleCrop", undefined, "false");
         this.map.HeadingPairs.render(xmlStream, model.worksheets);
         this.map.TitlesOfParts.render(xmlStream, model.worksheets);
-        this.map.Company.render(xmlStream, model.company || "");
+        this.map.Company.render(xmlStream, model.company ?? "");
         this.map.Manager.render(xmlStream, model.manager);
         xmlStream.leafNode("LinksUpToDate", undefined, "false");
         xmlStream.leafNode("SharedDoc", undefined, "false");

package/dist/cjs/modules/excel/xlsx/xform/core/content-types-xform.js

@@ -11,7 +11,7 @@ class ContentTypesXform extends base_xform_1.BaseXform {
         xmlStream.openXml(xml_stream_1.XmlStream.StdDocAttributes);
         xmlStream.openNode("Types", ContentTypesXform.PROPERTY_ATTRIBUTES);
         const mediaHash = {};
-        (model.media || []).forEach((medium) => {
+        (model.media ?? []).forEach((medium) => {
             if (medium.type === "image") {
                 const imageType = medium.extension;
                 if (!mediaHash[imageType]) {
@@ -40,18 +40,31 @@ class ContentTypesXform extends base_xform_1.BaseXform {
                 ContentType: "application/vnd.openxmlformats-officedocument.spreadsheetml.worksheet+xml"
             });
         });
-        if ((model.pivotTables || []).length) {
+        if ((model.pivotTables ?? []).length) {
+            // R9-B6: Deduplicate cache content types by cacheId. When multiple pivot tables
+            // share the same cache, the cache definition/records files are written only once.
+            const writtenCacheIds = new Set();
             // Add content types for each pivot table
-            (model.pivotTables || []).forEach((pivotTable) => {
+            (model.pivotTables ?? []).forEach((pivotTable) => {
                 const n = pivotTable.tableNumber;
-                xmlStream.leafNode("Override", {
-                    PartName: (0, ooxml_paths_1.toContentTypesPartName)((0, ooxml_paths_1.pivotCacheDefinitionPath)(n)),
-                    ContentType: "application/vnd.openxmlformats-officedocument.spreadsheetml.pivotCacheDefinition+xml"
-                });
-                xmlStream.leafNode("Override", {
-                    PartName: (0, ooxml_paths_1.toContentTypesPartName)((0, ooxml_paths_1.pivotCacheRecordsPath)(n)),
-                    ContentType: "application/vnd.openxmlformats-officedocument.spreadsheetml.pivotCacheRecords+xml"
-                });
+                const cacheId = pivotTable.cacheId;
+                if (!writtenCacheIds.has(cacheId)) {
+                    writtenCacheIds.add(cacheId);
+                    xmlStream.leafNode("Override", {
+                        PartName: (0, ooxml_paths_1.toContentTypesPartName)((0, ooxml_paths_1.pivotCacheDefinitionPath)(n)),
+                        ContentType: "application/vnd.openxmlformats-officedocument.spreadsheetml.pivotCacheDefinition+xml"
+                    });
+                    // R9-B5: Only register cacheRecords content type when the file actually exists.
+                    // Loaded pivot tables may lack cacheRecords (e.g. OLAP sources).
+                    const hasCacheRecords = pivotTable.isLoaded ? !!pivotTable.cacheRecords : true;
+                    if (hasCacheRecords) {
+                        xmlStream.leafNode("Override", {
+                            PartName: (0, ooxml_paths_1.toContentTypesPartName)((0, ooxml_paths_1.pivotCacheRecordsPath)(n)),
+                            ContentType: "application/vnd.openxmlformats-officedocument.spreadsheetml.pivotCacheRecords+xml"
+                        });
+                    }
+                }
+                // Each pivot table always has its own file
                 xmlStream.leafNode("Override", {
                     PartName: (0, ooxml_paths_1.toContentTypesPartName)((0, ooxml_paths_1.pivotTablePath)(n)),
                     ContentType: "application/vnd.openxmlformats-officedocument.spreadsheetml.pivotTable+xml"

package/dist/cjs/modules/excel/xlsx/xform/drawing/base-cell-anchor-xform.js

@@ -13,7 +13,7 @@ class BaseCellAnchorXform extends base_xform_1.BaseXform {
                 this.reset();
                 this.model = {
                     range: {
-                        editAs: node.attributes.editAs || "oneCell"
+                        editAs: node.attributes.editAs ?? "oneCell"
                     }
                 };
                 break;

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

@@ -22,8 +22,8 @@ class ExtXform extends base_xform_1.BaseXform {
     parseOpen(node) {
         if (node.name === this.tag) {
             this.model = {
-                width: parseInt(node.attributes.cx || "0", 10) / EMU_PER_PIXEL_AT_96_DPI,
-                height: parseInt(node.attributes.cy || "0", 10) / EMU_PER_PIXEL_AT_96_DPI
+                width: parseInt(node.attributes.cx ?? "0", 10) / EMU_PER_PIXEL_AT_96_DPI,
+                height: parseInt(node.attributes.cy ?? "0", 10) / EMU_PER_PIXEL_AT_96_DPI
             };
             return true;
         }

package/dist/cjs/modules/excel/xlsx/xform/drawing/one-cell-anchor-xform.js

@@ -23,7 +23,7 @@ class OneCellAnchorXform extends base_cell_anchor_xform_1.BaseCellAnchorXform {
         this.map["xdr:pic"].prepare(model.picture, options);
     }
     render(xmlStream, model) {
-        xmlStream.openNode(this.tag, { editAs: model.range.editAs || "oneCell" });
+        xmlStream.openNode(this.tag, { editAs: model.range.editAs ?? "oneCell" });
         this.map["xdr:from"].render(xmlStream, model.range.tl);
         this.map["xdr:ext"].render(xmlStream, model.range.ext);
         this.map["xdr:pic"].render(xmlStream, model.picture);

package/dist/cjs/modules/excel/xlsx/xform/drawing/two-cell-anchor-xform.js

@@ -40,7 +40,7 @@ class TwoCellAnchorXform extends base_cell_anchor_xform_1.BaseCellAnchorXform {
                     : {})
             });
         }
-        xmlStream.openNode(this.tag, { editAs: model.range.editAs || "oneCell" });
+        xmlStream.openNode(this.tag, { editAs: model.range.editAs ?? "oneCell" });
         this.map["xdr:from"].render(xmlStream, model.range.tl);
         this.map["xdr:to"].render(xmlStream, model.range.br);
         if (model.picture) {

package/dist/cjs/modules/excel/xlsx/xform/list-xform.js

@@ -9,7 +9,7 @@ class ListXform extends base_xform_1.BaseXform {
         this.always = !!options.always;
         this.count = options.count;
         this.empty = options.empty;
-        this.$count = options.$count || "count";
+        this.$count = options.$count ?? "count";
         this.$ = options.$;
         this.childXform = options.childXform;
         this.maxItems = options.maxItems;
@@ -27,10 +27,10 @@ class ListXform extends base_xform_1.BaseXform {
         if (this.always || (model && model.length)) {
             xmlStream.openNode(this.tag, this.$);
             if (this.count) {
-                xmlStream.addAttribute(this.$count, (model && model.length) || 0);
+                xmlStream.addAttribute(this.$count, model?.length ?? 0);
             }
             const { childXform } = this;
-            (model || []).forEach((childModel, index) => {
+            (model ?? []).forEach((childModel, index) => {
                 childXform.render(xmlStream, childModel, index);
             });
             xmlStream.closeNode();

package/dist/cjs/modules/excel/xlsx/xform/pivot-table/cache-field-xform.js

@@ -2,7 +2,11 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.CacheFieldXform = void 0;
 const base_xform_1 = require("../base-xform.js");
+const pivot_table_1 = require("../../../pivot-table.js");
 const utils_1 = require("../../../../../utils/utils.js");
+const raw_xml_collector_1 = require("./raw-xml-collector.js");
+/** Attribute keys on <cacheField> that are individually parsed (not collected into extraAttrs). */
+const KNOWN_CACHE_FIELD_KEYS = new Set(["name", "numFmtId"]);
 /**
  * Xform for parsing individual <cacheField> elements within a pivot cache definition.
  *
@@ -25,8 +29,9 @@ const utils_1 = require("../../../../../utils/utils.js");
 class CacheFieldXform extends base_xform_1.BaseXform {
     constructor() {
         super();
-        this.model = null;
         this.inSharedItems = false;
+        this.fieldGroupCollector = new raw_xml_collector_1.RawXmlCollector("fieldGroup");
+        this.model = null;
     }
     get tag() {
         return "cacheField";
@@ -34,89 +39,123 @@ class CacheFieldXform extends base_xform_1.BaseXform {
     reset() {
         this.model = null;
         this.inSharedItems = false;
+        this.fieldGroupCollector.reset();
     }
     parseOpen(node) {
         const { name, attributes } = node;
+        // Collect fieldGroup XML verbatim for roundtrip preservation
+        if (this.fieldGroupCollector.active) {
+            this.fieldGroupCollector.feedOpen(name, attributes);
+            return true;
+        }
         switch (name) {
-            case "cacheField":
+            case "cacheField": {
                 // Initialize the model with field name
+                // Collect unknown attributes into extraAttrs bag for roundtrip preservation
+                const extraAttrs = {};
+                for (const [k, v] of Object.entries(attributes)) {
+                    if (!KNOWN_CACHE_FIELD_KEYS.has(k)) {
+                        extraAttrs[k] = String(v);
+                    }
+                }
                 this.model = {
-                    name: (0, utils_1.xmlDecode)(attributes.name || ""),
-                    sharedItems: null
+                    name: attributes.name ?? "",
+                    sharedItems: null,
+                    numFmtId: attributes.numFmtId,
+                    isLoaded: true,
+                    extraAttrs: Object.keys(extraAttrs).length > 0 ? extraAttrs : undefined
                 };
                 break;
+            }
+            case "fieldGroup":
+                // Start collecting fieldGroup XML for roundtrip preservation
+                if (this.model) {
+                    this.fieldGroupCollector.start(attributes);
+                }
+                break;
             case "sharedItems":
                 this.inSharedItems = true;
                 // Store numeric field metadata
                 if (this.model) {
-                    this.model.containsNumber = attributes.containsNumber === "1";
-                    this.model.containsInteger = attributes.containsInteger === "1";
+                    // R8-B4: Store containsNumber/containsInteger as raw strings for roundtrip fidelity.
+                    // Previously "0" was lost because it was parsed as undefined.
+                    if (attributes.containsNumber !== undefined) {
+                        this.model.containsNumber = attributes.containsNumber;
+                    }
+                    if (attributes.containsInteger !== undefined) {
+                        this.model.containsInteger = attributes.containsInteger;
+                    }
+                    // R8-B7: Guard against NaN from malformed minValue/maxValue
                     if (attributes.minValue !== undefined) {
-                        this.model.minValue = parseFloat(attributes.minValue);
+                        const parsed = parseFloat(attributes.minValue);
+                        if (Number.isFinite(parsed)) {
+                            this.model.minValue = parsed;
+                        }
                     }
                     if (attributes.maxValue !== undefined) {
-                        this.model.maxValue = parseFloat(attributes.maxValue);
+                        const parsed = parseFloat(attributes.maxValue);
+                        if (Number.isFinite(parsed)) {
+                            this.model.maxValue = parsed;
+                        }
+                    }
+                    // Preserve original sharedItems type attributes for roundtrip fidelity.
+                    // These are stored as raw strings so we can re-emit them exactly as loaded.
+                    if (attributes.containsSemiMixedTypes !== undefined) {
+                        this.model.containsSemiMixedTypes = attributes.containsSemiMixedTypes;
+                    }
+                    if (attributes.containsNonDate !== undefined) {
+                        this.model.containsNonDate = attributes.containsNonDate;
+                    }
+                    if (attributes.containsString !== undefined) {
+                        this.model.containsString = attributes.containsString;
                     }
-                    // Initialize sharedItems array if count > 0 (for both string and numeric fields)
-                    const count = parseInt(attributes.count || "0", 10);
-                    if (count > 0) {
-                        this.model.sharedItems = [];
+                    if (attributes.containsBlank !== undefined) {
+                        this.model.containsBlank = attributes.containsBlank;
                     }
-                    else {
-                        // No count means no individual items (pure numeric field)
-                        this.model.sharedItems = null;
+                    if (attributes.containsDate !== undefined) {
+                        this.model.containsDate = attributes.containsDate;
                     }
+                    if (attributes.containsMixedTypes !== undefined) {
+                        this.model.containsMixedTypes = attributes.containsMixedTypes;
+                    }
+                    // Initialize sharedItems array when sharedItems element is present.
+                    // count="0" means an explicitly empty shared items list (not the same as absent).
+                    // No count attribute also gets an empty array since the <sharedItems> element
+                    // itself is present — child elements (<s>, <n>, etc.) will be pushed into it.
+                    this.model.sharedItems = [];
                 }
                 break;
             case "s":
-                // String value in sharedItems
-                if (this.inSharedItems && this.model?.sharedItems !== null) {
-                    // Decode XML entities in the value
-                    const value = (0, utils_1.xmlDecode)(attributes.v || "");
-                    this.model.sharedItems.push(value);
-                }
-                break;
             case "n":
-                // Numeric value in sharedItems (less common, but possible)
-                if (this.inSharedItems && this.model?.sharedItems !== null) {
-                    const value = parseFloat(attributes.v || "0");
-                    this.model.sharedItems.push(value);
-                }
-                break;
             case "b":
-                // Boolean value in sharedItems
-                if (this.inSharedItems && this.model?.sharedItems !== null) {
-                    const value = attributes.v === "1";
-                    this.model.sharedItems.push(value);
-                }
-                break;
             case "e":
-                // Error value in sharedItems
-                if (this.inSharedItems && this.model?.sharedItems !== null) {
-                    const value = `#${attributes.v || "ERROR!"}`;
-                    this.model.sharedItems.push(value);
-                }
-                break;
             case "m":
-                // Missing/null value in sharedItems
-                if (this.inSharedItems && this.model?.sharedItems !== null) {
-                    this.model.sharedItems.push(null);
-                }
-                break;
             case "d":
-                // Date value in sharedItems
-                if (this.inSharedItems && this.model?.sharedItems !== null) {
-                    const value = new Date(attributes.v || "");
-                    this.model.sharedItems.push(value);
+                // Shared item value — push to sharedItems array if we're inside <sharedItems>
+                if (this.inSharedItems && this.model && this.model.sharedItems !== null) {
+                    this.model.sharedItems.push(parseSharedItemValue(name, attributes));
                 }
                 break;
         }
         return true;
     }
-    parseText(_text) {
-        // No text content in cacheField elements
+    parseText(text) {
+        // Forward text to active collector (B3 fix: text nodes in fieldGroup)
+        if (this.fieldGroupCollector.active) {
+            this.fieldGroupCollector.feedText(text);
+        }
     }
     parseClose(name) {
+        // Handle fieldGroup collection — close tags
+        if (this.fieldGroupCollector.active) {
+            if (this.fieldGroupCollector.feedClose(name)) {
+                if (this.model) {
+                    this.model.fieldGroupXml = this.fieldGroupCollector.result;
+                }
+                this.fieldGroupCollector.reset();
+            }
+            return true;
+        }
         switch (name) {
             case "cacheField":
                 // End of this cacheField element
@@ -129,3 +168,46 @@ class CacheFieldXform extends base_xform_1.BaseXform {
     }
 }
 exports.CacheFieldXform = CacheFieldXform;
+// ---------------------------------------------------------------------------
+// Helpers
+// ---------------------------------------------------------------------------
+/**
+ * Convert a SAX-parsed shared item element into its typed value.
+ *
+ * Tag mapping:
+ * - `s` → string
+ * - `n` → number
+ * - `b` → boolean
+ * - `e` → PivotErrorValue
+ * - `m` → null (missing)
+ * - `d` → Date (UTC)
+ */
+function parseSharedItemValue(tag, attributes) {
+    switch (tag) {
+        case "s":
+            return attributes.v ?? "";
+        case "n":
+            // Missing v attribute → treat as null (missing) rather than fabricating 0
+            if (attributes.v === undefined || attributes.v === "") {
+                return null;
+            }
+            return parseFloat(attributes.v);
+        case "b":
+            return attributes.v === "1";
+        case "e":
+            return new pivot_table_1.PivotErrorValue(attributes.v ?? "");
+        case "m":
+            return null;
+        case "d": {
+            // Missing/empty v attribute → treat as missing value (null) to avoid Invalid Date
+            if (!attributes.v) {
+                return null;
+            }
+            // R8-B13: Guard against Invalid Date from malformed date strings
+            const date = (0, utils_1.parseOoxmlDate)(attributes.v);
+            return isNaN(date.getTime()) ? null : date;
+        }
+        default:
+            return null;
+    }
+}