@cj-tech-master/excelts 9.5.7 → 9.5.8

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

@@ -2289,6 +2289,16 @@ class Workbook {
         this._tableNames.clear();
         value.worksheets.forEach(worksheetModel => {
             const { id, name, state } = worksheetModel;
+            // API invariant: `_worksheets` is keyed by a positive integer
+            // sheet id. A worksheet model with a missing or non-integer id
+            // would be stored under a string pseudo key like `"undefined"`
+            // or `"NaN"`, making it unreachable via `getWorksheet(name)`
+            // (issue #166). The xlsx reconciler enforces the same invariant
+            // before reaching this point; programmatic callers assigning
+            // `model` directly with a malformed payload land here instead.
+            if (!Number.isInteger(id) || id <= 0) {
+                return;
+            }
             const orderNo = value.sheets && value.sheets.findIndex(ws => ws.id === id);
             const worksheet = (this._worksheets[id] = new Worksheet({
                 id,

package/dist/browser/modules/excel/xlsx/xform/book/sheet-xform.d.ts

@@ -1,14 +1,24 @@
 import type { WorksheetState } from "../../../types.js";
 import { BaseXform } from "../base-xform.js";
 interface SheetModel {
-    id: number;
+    id: number | undefined;
     name: string;
     state: WorksheetState;
-    rId: string;
+    rId: string | undefined;
 }
 declare class WorksheetXform extends BaseXform {
+    relationshipsPrefixes: readonly string[];
     render(xmlStream: any, model: SheetModel): void;
     parseOpen(node: any): boolean;
+    /**
+     * Locate the relationship id on a `<sheet>` element. Tries every
+     * prefix the workbook root bound to the relationships namespace,
+     * then any prefix the `<sheet>` element itself rebinds locally.
+     * Returns `undefined` if no relationship id is present — callers
+     * (the workbook reconciler) will treat such a `<sheet>` as a
+     * half-broken declaration that can't be bound to a worksheet part.
+     */
+    private _extractRelId;
     parseText(): void;
     parseClose(): boolean;
 }

package/dist/browser/modules/excel/xlsx/xform/book/sheet-xform.js

@@ -1,10 +1,34 @@
 import { BaseXform } from "../base-xform.js";
 const VALID_STATES = new Set(["visible", "hidden", "veryHidden"]);
+const RELATIONSHIPS_NS = "http://schemas.openxmlformats.org/officeDocument/2006/relationships";
 function parseWorksheetState(raw) {
     const state = raw || "visible";
     return VALID_STATES.has(state) ? state : "visible";
 }
+function parseSheetId(raw) {
+    if (raw === undefined) {
+        return undefined;
+    }
+    const id = parseInt(raw, 10);
+    // OOXML constrains `sheetId` to a positive integer. Anything that
+    // doesn't parse to one — empty string, alphabetic, zero, negative,
+    // overflowing — must not propagate as `NaN`/`0`/`-1` because those
+    // would later seed pseudo keys like `_worksheets["NaN"]` (the same
+    // family of bug as issue #166's `_worksheets["undefined"]`).
+    return Number.isInteger(id) && id > 0 ? id : undefined;
+}
 class WorksheetXform extends BaseXform {
+    constructor() {
+        super(...arguments);
+        // All prefixes the workbook root binds to the OOXML relationships
+        // namespace. Conventionally a workbook declares `xmlns:r=…`, so the
+        // sheet uses `r:id="rId1"`. But the prefix is only a label and a
+        // workbook may legally bind any prefix (or several) to that
+        // namespace. `WorkbookXform` populates this list from the
+        // `<workbook>` root; `r` is the safe fallback when the workbook
+        // declares no relationships binding at all.
+        this.relationshipsPrefixes = ["r"];
+    }
     render(xmlStream, model) {
         xmlStream.leafNode("sheet", {
             name: model.name,
@@ -18,14 +42,44 @@ class WorksheetXform extends BaseXform {
         if (node.name === "sheet") {
             this.model = {
                 name: node.attributes.name,
-                id: parseInt(node.attributes.sheetId, 10),
+                id: parseSheetId(node.attributes.sheetId),
                 state: parseWorksheetState(node.attributes.state),
-                rId: node.attributes["r:id"]
+                rId: this._extractRelId(node)
             };
             return true;
         }
         return false;
     }
+    /**
+     * Locate the relationship id on a `<sheet>` element. Tries every
+     * prefix the workbook root bound to the relationships namespace,
+     * then any prefix the `<sheet>` element itself rebinds locally.
+     * Returns `undefined` if no relationship id is present — callers
+     * (the workbook reconciler) will treat such a `<sheet>` as a
+     * half-broken declaration that can't be bound to a worksheet part.
+     */
+    _extractRelId(node) {
+        const attrs = node.attributes ?? {};
+        for (const prefix of this.relationshipsPrefixes) {
+            const value = attrs[`${prefix}:id`];
+            if (value !== undefined) {
+                return value;
+            }
+        }
+        // Local-scope fallback: a `<sheet>` element occasionally redeclares
+        // the relationships namespace under a fresh prefix. Scan its own
+        // attributes for `xmlns:X="…/relationships"` and look up `X:id`.
+        for (const attrName of Object.keys(attrs)) {
+            if (attrName.startsWith("xmlns:") && attrs[attrName] === RELATIONSHIPS_NS) {
+                const localPrefix = attrName.slice("xmlns:".length);
+                const candidate = attrs[`${localPrefix}:id`];
+                if (candidate !== undefined) {
+                    return candidate;
+                }
+            }
+        }
+        return undefined;
+    }
     parseText() { }
     parseClose() {
         return false;

package/dist/browser/modules/excel/xlsx/xform/book/workbook-xform.d.ts

@@ -5,10 +5,17 @@ declare class WorkbookXform extends BaseXform {
     map: {
         [key: string]: any;
     };
+    /**
+     * The `<sheet>` xform shared with the `sheets` ListXform. Held as a
+     * field so `parseOpen` can pass workbook-level state (the prefixes
+     * bound to the OOXML relationships namespace) into it.
+     */
+    private readonly _sheetXform;
     constructor();
     prepare(model: any): void;
     render(xmlStream: any, model: any): void;
     parseOpen(node: any): boolean;
+    private static _findRelationshipsPrefixes;
     parseText(text: string): void;
     parseClose(name: string): boolean;
     reconcile(model: any): void;

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

@@ -1,4 +1,5 @@
 import { colCache } from "../../../utils/col-cache.js";
+import { resolveRelTarget } from "../../../utils/ooxml-paths.js";
 import { BaseXform } from "../base-xform.js";
 import { DefinedNamesXform } from "./defined-name-xform.js";
 import { ExternalReferenceXform } from "./external-reference-xform.js";
@@ -14,6 +15,12 @@ import { StdDocAttributes } from "../../../../xml/writer.js";
 class WorkbookXform extends BaseXform {
     constructor() {
         super();
+        /**
+         * The `<sheet>` xform shared with the `sheets` ListXform. Held as a
+         * field so `parseOpen` can pass workbook-level state (the prefixes
+         * bound to the OOXML relationships namespace) into it.
+         */
+        this._sheetXform = new WorksheetXform();
         this.map = {
             fileVersion: WorkbookXform.STATIC_XFORMS.fileVersion,
             workbookPr: new WorkbookPropertiesXform(),
@@ -23,7 +30,7 @@ class WorkbookXform extends BaseXform {
                 count: false,
                 childXform: new WorkbookViewXform()
             }),
-            sheets: new ListXform({ tag: "sheets", count: false, childXform: new WorksheetXform() }),
+            sheets: new ListXform({ tag: "sheets", count: false, childXform: this._sheetXform }),
             definedNames: new ListXform({
                 tag: "definedNames",
                 count: false,
@@ -171,6 +178,11 @@ class WorkbookXform extends BaseXform {
         }
         switch (node.name) {
             case "workbook":
+                // Capture every prefix the workbook root binds to the OOXML
+                // relationships namespace so nested `<sheet>` parsing can
+                // read the relationship id under any of them. Falls back to
+                // the conventional `r` if the workbook declares no binding.
+                this._sheetXform.relationshipsPrefixes = WorkbookXform._findRelationshipsPrefixes(node);
                 return true;
             default:
                 this.parser = this.map[node.name];
@@ -180,6 +192,17 @@ class WorkbookXform extends BaseXform {
                 return true;
         }
     }
+    static _findRelationshipsPrefixes(node) {
+        const RELATIONSHIPS_NS = "http://schemas.openxmlformats.org/officeDocument/2006/relationships";
+        const attrs = node.attributes ?? {};
+        const prefixes = [];
+        for (const attrName of Object.keys(attrs)) {
+            if (attrName.startsWith("xmlns:") && attrs[attrName] === RELATIONSHIPS_NS) {
+                prefixes.push(attrName.slice("xmlns:".length));
+            }
+        }
+        return prefixes.length > 0 ? prefixes : ["r"];
+    }
     parseText(text) {
         if (this.parser) {
             this.parser.parseText(text);
@@ -247,7 +270,7 @@ class WorkbookXform extends BaseXform {
                 sheetPosition += 1;
                 return;
             }
-            const target = `xl/${rel.Target.replace(/^(\s|\/xl\/)+/, "")}`;
+            const target = resolveRelTarget("xl", rel.Target);
             // Check if this is a chartsheet
             const chartsheetMatch = /xl\/chartsheets\/sheet(\d+)\.xml/.exec(target);
             if (chartsheetMatch) {
@@ -279,6 +302,29 @@ class WorkbookXform extends BaseXform {
         });
         // Store reconciled chartsheets on the model
         model.chartsheetsList = chartsheetsList;
+        // Drop unbound worksheet parts. The reader (xlsx.browser.ts)
+        // collects every `xl/worksheets/sheetN.xml` it sees in the zip,
+        // because zip entries arrive in arbitrary order relative to
+        // workbook.xml. The authoritative `<sheets>` list is only
+        // available here, post-parse — so this is the first point at
+        // which we can prune worksheet parts that no `<sheet>` element
+        // claims through a working rel binding.
+        //
+        // Without pruning, such worksheets propagate downstream with
+        // `id`/`name`/`state` all `undefined`, landing under the literal
+        // string key `"undefined"` in `Workbook._worksheets` and becoming
+        // unreachable via `getWorksheet(name)` (issue #166). OOXML treats
+        // the workbook's `<sheets>` element as the single source of truth
+        // for which parts belong to the workbook; we follow that contract
+        // strictly. Genuinely-cursed workbooks reach this branch only when
+        // their `<sheet>` declarations are themselves missing or broken;
+        // namespace-prefix and Target-path quirks are handled upstream
+        // (the relationships-prefix lookup in sheet-xform and
+        // resolveRelTarget above), so a normal Excel-authored file never
+        // loses sheets here.
+        if (Array.isArray(model.worksheets)) {
+            model.worksheets = model.worksheets.filter((ws) => ws && Number.isInteger(ws.id) && ws.id > 0);
+        }
         // reconcile print areas
         const definedNames = [];
         if (model.definedNames) {

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

@@ -2292,6 +2292,16 @@ class Workbook {
         this._tableNames.clear();
         value.worksheets.forEach(worksheetModel => {
             const { id, name, state } = worksheetModel;
+            // API invariant: `_worksheets` is keyed by a positive integer
+            // sheet id. A worksheet model with a missing or non-integer id
+            // would be stored under a string pseudo key like `"undefined"`
+            // or `"NaN"`, making it unreachable via `getWorksheet(name)`
+            // (issue #166). The xlsx reconciler enforces the same invariant
+            // before reaching this point; programmatic callers assigning
+            // `model` directly with a malformed payload land here instead.
+            if (!Number.isInteger(id) || id <= 0) {
+                return;
+            }
             const orderNo = value.sheets && value.sheets.findIndex(ws => ws.id === id);
             const worksheet = (this._worksheets[id] = new worksheet_1.Worksheet({
                 id,

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

@@ -3,11 +3,35 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.WorksheetXform = void 0;
 const base_xform_1 = require("../base-xform.js");
 const VALID_STATES = new Set(["visible", "hidden", "veryHidden"]);
+const RELATIONSHIPS_NS = "http://schemas.openxmlformats.org/officeDocument/2006/relationships";
 function parseWorksheetState(raw) {
     const state = raw || "visible";
     return VALID_STATES.has(state) ? state : "visible";
 }
+function parseSheetId(raw) {
+    if (raw === undefined) {
+        return undefined;
+    }
+    const id = parseInt(raw, 10);
+    // OOXML constrains `sheetId` to a positive integer. Anything that
+    // doesn't parse to one — empty string, alphabetic, zero, negative,
+    // overflowing — must not propagate as `NaN`/`0`/`-1` because those
+    // would later seed pseudo keys like `_worksheets["NaN"]` (the same
+    // family of bug as issue #166's `_worksheets["undefined"]`).
+    return Number.isInteger(id) && id > 0 ? id : undefined;
+}
 class WorksheetXform extends base_xform_1.BaseXform {
+    constructor() {
+        super(...arguments);
+        // All prefixes the workbook root binds to the OOXML relationships
+        // namespace. Conventionally a workbook declares `xmlns:r=…`, so the
+        // sheet uses `r:id="rId1"`. But the prefix is only a label and a
+        // workbook may legally bind any prefix (or several) to that
+        // namespace. `WorkbookXform` populates this list from the
+        // `<workbook>` root; `r` is the safe fallback when the workbook
+        // declares no relationships binding at all.
+        this.relationshipsPrefixes = ["r"];
+    }
     render(xmlStream, model) {
         xmlStream.leafNode("sheet", {
             name: model.name,
@@ -21,14 +45,44 @@ class WorksheetXform extends base_xform_1.BaseXform {
         if (node.name === "sheet") {
             this.model = {
                 name: node.attributes.name,
-                id: parseInt(node.attributes.sheetId, 10),
+                id: parseSheetId(node.attributes.sheetId),
                 state: parseWorksheetState(node.attributes.state),
-                rId: node.attributes["r:id"]
+                rId: this._extractRelId(node)
             };
             return true;
         }
         return false;
     }
+    /**
+     * Locate the relationship id on a `<sheet>` element. Tries every
+     * prefix the workbook root bound to the relationships namespace,
+     * then any prefix the `<sheet>` element itself rebinds locally.
+     * Returns `undefined` if no relationship id is present — callers
+     * (the workbook reconciler) will treat such a `<sheet>` as a
+     * half-broken declaration that can't be bound to a worksheet part.
+     */
+    _extractRelId(node) {
+        const attrs = node.attributes ?? {};
+        for (const prefix of this.relationshipsPrefixes) {
+            const value = attrs[`${prefix}:id`];
+            if (value !== undefined) {
+                return value;
+            }
+        }
+        // Local-scope fallback: a `<sheet>` element occasionally redeclares
+        // the relationships namespace under a fresh prefix. Scan its own
+        // attributes for `xmlns:X="…/relationships"` and look up `X:id`.
+        for (const attrName of Object.keys(attrs)) {
+            if (attrName.startsWith("xmlns:") && attrs[attrName] === RELATIONSHIPS_NS) {
+                const localPrefix = attrName.slice("xmlns:".length);
+                const candidate = attrs[`${localPrefix}:id`];
+                if (candidate !== undefined) {
+                    return candidate;
+                }
+            }
+        }
+        return undefined;
+    }
     parseText() { }
     parseClose() {
         return false;

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

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.WorkbookXform = void 0;
 const col_cache_1 = require("../../../utils/col-cache.js");
+const ooxml_paths_1 = require("../../../utils/ooxml-paths.js");
 const base_xform_1 = require("../base-xform.js");
 const defined_name_xform_1 = require("./defined-name-xform.js");
 const external_reference_xform_1 = require("./external-reference-xform.js");
@@ -17,6 +18,12 @@ const writer_1 = require("../../../../xml/writer.js");
 class WorkbookXform extends base_xform_1.BaseXform {
     constructor() {
         super();
+        /**
+         * The `<sheet>` xform shared with the `sheets` ListXform. Held as a
+         * field so `parseOpen` can pass workbook-level state (the prefixes
+         * bound to the OOXML relationships namespace) into it.
+         */
+        this._sheetXform = new sheet_xform_1.WorksheetXform();
         this.map = {
             fileVersion: WorkbookXform.STATIC_XFORMS.fileVersion,
             workbookPr: new workbook_properties_xform_1.WorkbookPropertiesXform(),
@@ -26,7 +33,7 @@ class WorkbookXform extends base_xform_1.BaseXform {
                 count: false,
                 childXform: new workbook_view_xform_1.WorkbookViewXform()
             }),
-            sheets: new list_xform_1.ListXform({ tag: "sheets", count: false, childXform: new sheet_xform_1.WorksheetXform() }),
+            sheets: new list_xform_1.ListXform({ tag: "sheets", count: false, childXform: this._sheetXform }),
             definedNames: new list_xform_1.ListXform({
                 tag: "definedNames",
                 count: false,
@@ -174,6 +181,11 @@ class WorkbookXform extends base_xform_1.BaseXform {
         }
         switch (node.name) {
             case "workbook":
+                // Capture every prefix the workbook root binds to the OOXML
+                // relationships namespace so nested `<sheet>` parsing can
+                // read the relationship id under any of them. Falls back to
+                // the conventional `r` if the workbook declares no binding.
+                this._sheetXform.relationshipsPrefixes = WorkbookXform._findRelationshipsPrefixes(node);
                 return true;
             default:
                 this.parser = this.map[node.name];
@@ -183,6 +195,17 @@ class WorkbookXform extends base_xform_1.BaseXform {
                 return true;
         }
     }
+    static _findRelationshipsPrefixes(node) {
+        const RELATIONSHIPS_NS = "http://schemas.openxmlformats.org/officeDocument/2006/relationships";
+        const attrs = node.attributes ?? {};
+        const prefixes = [];
+        for (const attrName of Object.keys(attrs)) {
+            if (attrName.startsWith("xmlns:") && attrs[attrName] === RELATIONSHIPS_NS) {
+                prefixes.push(attrName.slice("xmlns:".length));
+            }
+        }
+        return prefixes.length > 0 ? prefixes : ["r"];
+    }
     parseText(text) {
         if (this.parser) {
             this.parser.parseText(text);
@@ -250,7 +273,7 @@ class WorkbookXform extends base_xform_1.BaseXform {
                 sheetPosition += 1;
                 return;
             }
-            const target = `xl/${rel.Target.replace(/^(\s|\/xl\/)+/, "")}`;
+            const target = (0, ooxml_paths_1.resolveRelTarget)("xl", rel.Target);
             // Check if this is a chartsheet
             const chartsheetMatch = /xl\/chartsheets\/sheet(\d+)\.xml/.exec(target);
             if (chartsheetMatch) {
@@ -282,6 +305,29 @@ class WorkbookXform extends base_xform_1.BaseXform {
         });
         // Store reconciled chartsheets on the model
         model.chartsheetsList = chartsheetsList;
+        // Drop unbound worksheet parts. The reader (xlsx.browser.ts)
+        // collects every `xl/worksheets/sheetN.xml` it sees in the zip,
+        // because zip entries arrive in arbitrary order relative to
+        // workbook.xml. The authoritative `<sheets>` list is only
+        // available here, post-parse — so this is the first point at
+        // which we can prune worksheet parts that no `<sheet>` element
+        // claims through a working rel binding.
+        //
+        // Without pruning, such worksheets propagate downstream with
+        // `id`/`name`/`state` all `undefined`, landing under the literal
+        // string key `"undefined"` in `Workbook._worksheets` and becoming
+        // unreachable via `getWorksheet(name)` (issue #166). OOXML treats
+        // the workbook's `<sheets>` element as the single source of truth
+        // for which parts belong to the workbook; we follow that contract
+        // strictly. Genuinely-cursed workbooks reach this branch only when
+        // their `<sheet>` declarations are themselves missing or broken;
+        // namespace-prefix and Target-path quirks are handled upstream
+        // (the relationships-prefix lookup in sheet-xform and
+        // resolveRelTarget above), so a normal Excel-authored file never
+        // loses sheets here.
+        if (Array.isArray(model.worksheets)) {
+            model.worksheets = model.worksheets.filter((ws) => ws && Number.isInteger(ws.id) && ws.id > 0);
+        }
         // reconcile print areas
         const definedNames = [];
         if (model.definedNames) {

package/dist/esm/modules/excel/workbook.browser.js

@@ -2289,6 +2289,16 @@ class Workbook {
         this._tableNames.clear();
         value.worksheets.forEach(worksheetModel => {
             const { id, name, state } = worksheetModel;
+            // API invariant: `_worksheets` is keyed by a positive integer
+            // sheet id. A worksheet model with a missing or non-integer id
+            // would be stored under a string pseudo key like `"undefined"`
+            // or `"NaN"`, making it unreachable via `getWorksheet(name)`
+            // (issue #166). The xlsx reconciler enforces the same invariant
+            // before reaching this point; programmatic callers assigning
+            // `model` directly with a malformed payload land here instead.
+            if (!Number.isInteger(id) || id <= 0) {
+                return;
+            }
             const orderNo = value.sheets && value.sheets.findIndex(ws => ws.id === id);
             const worksheet = (this._worksheets[id] = new Worksheet({
                 id,

package/dist/esm/modules/excel/xlsx/xform/book/sheet-xform.js

@@ -1,10 +1,34 @@
 import { BaseXform } from "../base-xform.js";
 const VALID_STATES = new Set(["visible", "hidden", "veryHidden"]);
+const RELATIONSHIPS_NS = "http://schemas.openxmlformats.org/officeDocument/2006/relationships";
 function parseWorksheetState(raw) {
     const state = raw || "visible";
     return VALID_STATES.has(state) ? state : "visible";
 }
+function parseSheetId(raw) {
+    if (raw === undefined) {
+        return undefined;
+    }
+    const id = parseInt(raw, 10);
+    // OOXML constrains `sheetId` to a positive integer. Anything that
+    // doesn't parse to one — empty string, alphabetic, zero, negative,
+    // overflowing — must not propagate as `NaN`/`0`/`-1` because those
+    // would later seed pseudo keys like `_worksheets["NaN"]` (the same
+    // family of bug as issue #166's `_worksheets["undefined"]`).
+    return Number.isInteger(id) && id > 0 ? id : undefined;
+}
 class WorksheetXform extends BaseXform {
+    constructor() {
+        super(...arguments);
+        // All prefixes the workbook root binds to the OOXML relationships
+        // namespace. Conventionally a workbook declares `xmlns:r=…`, so the
+        // sheet uses `r:id="rId1"`. But the prefix is only a label and a
+        // workbook may legally bind any prefix (or several) to that
+        // namespace. `WorkbookXform` populates this list from the
+        // `<workbook>` root; `r` is the safe fallback when the workbook
+        // declares no relationships binding at all.
+        this.relationshipsPrefixes = ["r"];
+    }
     render(xmlStream, model) {
         xmlStream.leafNode("sheet", {
             name: model.name,
@@ -18,14 +42,44 @@ class WorksheetXform extends BaseXform {
         if (node.name === "sheet") {
             this.model = {
                 name: node.attributes.name,
-                id: parseInt(node.attributes.sheetId, 10),
+                id: parseSheetId(node.attributes.sheetId),
                 state: parseWorksheetState(node.attributes.state),
-                rId: node.attributes["r:id"]
+                rId: this._extractRelId(node)
             };
             return true;
         }
         return false;
     }
+    /**
+     * Locate the relationship id on a `<sheet>` element. Tries every
+     * prefix the workbook root bound to the relationships namespace,
+     * then any prefix the `<sheet>` element itself rebinds locally.
+     * Returns `undefined` if no relationship id is present — callers
+     * (the workbook reconciler) will treat such a `<sheet>` as a
+     * half-broken declaration that can't be bound to a worksheet part.
+     */
+    _extractRelId(node) {
+        const attrs = node.attributes ?? {};
+        for (const prefix of this.relationshipsPrefixes) {
+            const value = attrs[`${prefix}:id`];
+            if (value !== undefined) {
+                return value;
+            }
+        }
+        // Local-scope fallback: a `<sheet>` element occasionally redeclares
+        // the relationships namespace under a fresh prefix. Scan its own
+        // attributes for `xmlns:X="…/relationships"` and look up `X:id`.
+        for (const attrName of Object.keys(attrs)) {
+            if (attrName.startsWith("xmlns:") && attrs[attrName] === RELATIONSHIPS_NS) {
+                const localPrefix = attrName.slice("xmlns:".length);
+                const candidate = attrs[`${localPrefix}:id`];
+                if (candidate !== undefined) {
+                    return candidate;
+                }
+            }
+        }
+        return undefined;
+    }
     parseText() { }
     parseClose() {
         return false;

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

@@ -1,4 +1,5 @@
 import { colCache } from "../../../utils/col-cache.js";
+import { resolveRelTarget } from "../../../utils/ooxml-paths.js";
 import { BaseXform } from "../base-xform.js";
 import { DefinedNamesXform } from "./defined-name-xform.js";
 import { ExternalReferenceXform } from "./external-reference-xform.js";
@@ -14,6 +15,12 @@ import { StdDocAttributes } from "../../../../xml/writer.js";
 class WorkbookXform extends BaseXform {
     constructor() {
         super();
+        /**
+         * The `<sheet>` xform shared with the `sheets` ListXform. Held as a
+         * field so `parseOpen` can pass workbook-level state (the prefixes
+         * bound to the OOXML relationships namespace) into it.
+         */
+        this._sheetXform = new WorksheetXform();
         this.map = {
             fileVersion: WorkbookXform.STATIC_XFORMS.fileVersion,
             workbookPr: new WorkbookPropertiesXform(),
@@ -23,7 +30,7 @@ class WorkbookXform extends BaseXform {
                 count: false,
                 childXform: new WorkbookViewXform()
             }),
-            sheets: new ListXform({ tag: "sheets", count: false, childXform: new WorksheetXform() }),
+            sheets: new ListXform({ tag: "sheets", count: false, childXform: this._sheetXform }),
             definedNames: new ListXform({
                 tag: "definedNames",
                 count: false,
@@ -171,6 +178,11 @@ class WorkbookXform extends BaseXform {
         }
         switch (node.name) {
             case "workbook":
+                // Capture every prefix the workbook root binds to the OOXML
+                // relationships namespace so nested `<sheet>` parsing can
+                // read the relationship id under any of them. Falls back to
+                // the conventional `r` if the workbook declares no binding.
+                this._sheetXform.relationshipsPrefixes = WorkbookXform._findRelationshipsPrefixes(node);
                 return true;
             default:
                 this.parser = this.map[node.name];
@@ -180,6 +192,17 @@ class WorkbookXform extends BaseXform {
                 return true;
         }
     }
+    static _findRelationshipsPrefixes(node) {
+        const RELATIONSHIPS_NS = "http://schemas.openxmlformats.org/officeDocument/2006/relationships";
+        const attrs = node.attributes ?? {};
+        const prefixes = [];
+        for (const attrName of Object.keys(attrs)) {
+            if (attrName.startsWith("xmlns:") && attrs[attrName] === RELATIONSHIPS_NS) {
+                prefixes.push(attrName.slice("xmlns:".length));
+            }
+        }
+        return prefixes.length > 0 ? prefixes : ["r"];
+    }
     parseText(text) {
         if (this.parser) {
             this.parser.parseText(text);
@@ -247,7 +270,7 @@ class WorkbookXform extends BaseXform {
                 sheetPosition += 1;
                 return;
             }
-            const target = `xl/${rel.Target.replace(/^(\s|\/xl\/)+/, "")}`;
+            const target = resolveRelTarget("xl", rel.Target);
             // Check if this is a chartsheet
             const chartsheetMatch = /xl\/chartsheets\/sheet(\d+)\.xml/.exec(target);
             if (chartsheetMatch) {
@@ -279,6 +302,29 @@ class WorkbookXform extends BaseXform {
         });
         // Store reconciled chartsheets on the model
         model.chartsheetsList = chartsheetsList;
+        // Drop unbound worksheet parts. The reader (xlsx.browser.ts)
+        // collects every `xl/worksheets/sheetN.xml` it sees in the zip,
+        // because zip entries arrive in arbitrary order relative to
+        // workbook.xml. The authoritative `<sheets>` list is only
+        // available here, post-parse — so this is the first point at
+        // which we can prune worksheet parts that no `<sheet>` element
+        // claims through a working rel binding.
+        //
+        // Without pruning, such worksheets propagate downstream with
+        // `id`/`name`/`state` all `undefined`, landing under the literal
+        // string key `"undefined"` in `Workbook._worksheets` and becoming
+        // unreachable via `getWorksheet(name)` (issue #166). OOXML treats
+        // the workbook's `<sheets>` element as the single source of truth
+        // for which parts belong to the workbook; we follow that contract
+        // strictly. Genuinely-cursed workbooks reach this branch only when
+        // their `<sheet>` declarations are themselves missing or broken;
+        // namespace-prefix and Target-path quirks are handled upstream
+        // (the relationships-prefix lookup in sheet-xform and
+        // resolveRelTarget above), so a normal Excel-authored file never
+        // loses sheets here.
+        if (Array.isArray(model.worksheets)) {
+            model.worksheets = model.worksheets.filter((ws) => ws && Number.isInteger(ws.id) && ws.id > 0);
+        }
         // reconcile print areas
         const definedNames = [];
         if (model.definedNames) {