@cj-tech-master/excelts 6.1.0 → 6.1.1

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

@@ -138,7 +138,8 @@ class Worksheet {
             }
             name = name.substring(0, 31);
         }
-        if (this._workbook.worksheets.find(ws => ws && ws.name.toLowerCase() === name.toLowerCase())) {
+        const nameLower = name.toLowerCase();
+        if (this._workbook.worksheets.find(ws => ws && ws !== this && ws.name.toLowerCase() === nameLower)) {
             throw new WorksheetNameError(`Worksheet name already exists: ${name}`);
         }
         this._name = name;

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

@@ -4,15 +4,17 @@ class HyperlinkXform extends BaseXform {
         return "hyperlink";
     }
     render(xmlStream, model) {
-        if (this.isInternalLink(model)) {
+        if (model.target && isInternalLink(model.target)) {
+            // Internal link: use location attribute only (no relationship)
+            // Strip the leading "#" — OOXML location attribute is without "#"
             xmlStream.leafNode("hyperlink", {
                 ref: model.address,
-                "r:id": model.rId,
                 tooltip: model.tooltip,
-                location: model.target
+                location: model.target.slice(1)
             });
         }
         else {
+            // External link: use r:id relationship reference
             xmlStream.leafNode("hyperlink", {
                 ref: model.address,
                 "r:id": model.rId,
@@ -27,9 +29,13 @@ class HyperlinkXform extends BaseXform {
                 rId: node.attributes["r:id"],
                 tooltip: node.attributes.tooltip
             };
-            // This is an internal link
+            // Internal link: location attribute stores the target without "#"
+            // Normalize: always store as "#Location" in the model regardless of
+            // whether the source had a leading "#" (our old buggy output) or not
+            // (correct OOXML from Excel or the fixed writer).
             if (node.attributes.location) {
-                this.model.target = node.attributes.location;
+                const loc = node.attributes.location;
+                this.model.target = loc.startsWith("#") ? loc : `#${loc}`;
             }
             return true;
         }
@@ -39,9 +45,13 @@ class HyperlinkXform extends BaseXform {
     parseClose() {
         return false;
     }
-    isInternalLink(model) {
-        // @example: Sheet2!D3, return true
-        return !!(model.target && /^[^!]+![a-zA-Z]+[\d]+$/.test(model.target));
-    }
 }
-export { HyperlinkXform };
+/**
+ * Internal hyperlinks start with "#" (e.g. "#Sheet2!A1").
+ * This matches Excel's convention and the OOXML spec where internal links
+ * use the `location` attribute instead of a relationship.
+ */
+function isInternalLink(target) {
+    return target.startsWith("#");
+}
+export { HyperlinkXform, isInternalLink };

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

@@ -3,11 +3,11 @@ import { XmlStream } from "../../../utils/xml-stream.js";
 import { RelType } from "../../rel-type.js";
 import { Merges } from "./merges.js";
 import { BaseXform } from "../base-xform.js";
+import { isInternalLink, HyperlinkXform } from "./hyperlink-xform.js";
 import { ListXform } from "../list-xform.js";
 import { RowXform } from "./row-xform.js";
 import { ColXform } from "./col-xform.js";
 import { DimensionXform } from "./dimension-xform.js";
-import { HyperlinkXform } from "./hyperlink-xform.js";
 import { MergeCellXform } from "./merge-cell-xform.js";
 import { DataValidationsXform } from "./data-validations-xform.js";
 import { SheetPropertiesXform } from "./sheet-properties-xform.js";
@@ -158,6 +158,11 @@ class WorkSheetXform extends BaseXform {
             return `rId${r.length + 1}`;
         }
         model.hyperlinks.forEach(hyperlink => {
+            // Internal links (e.g. "#Sheet2!A1") use the location attribute only,
+            // no relationship is needed in the .rels file.
+            if (isInternalLink(hyperlink.target)) {
+                return;
+            }
             const rId = nextRid(rels);
             hyperlink.rId = rId;
             rels.push({
@@ -594,8 +599,13 @@ class WorkSheetXform extends BaseXform {
         }, {});
         options.hyperlinkMap = (model.hyperlinks ?? []).reduce((h, hyperlink) => {
             if (hyperlink.rId) {
+                // External link: resolve target from relationship
                 h[hyperlink.address] = rels[hyperlink.rId].Target;
             }
+            else if (hyperlink.target) {
+                // Internal link: target was restored from location attribute (with "#" prefix)
+                h[hyperlink.address] = hyperlink.target;
+            }
             return h;
         }, {});
         options.formulae = {};

package/dist/iife/excelts.iife.js

@@ -1,5 +1,5 @@
 /*!
-* @cj-tech-master/excelts v6.1.0
+* @cj-tech-master/excelts v6.1.1
 * TypeScript Excel Workbook Manager - Read and Write xlsx and csv Files.
 * (c) 2026 cjnoname
 * Released under the MIT License
@@ -5000,8 +5000,12 @@ var ExcelTS = (function(exports) {
 			if (name === "History") throw new WorksheetNameError("The name \"History\" is protected. Please use a different name.");
 			if (/[*?:/\\[\]]/.test(name)) throw new WorksheetNameError(`Worksheet name ${name} cannot include any of the following characters: * ? : \\ / [ ]`);
 			if (/(^')|('$)/.test(name)) throw new WorksheetNameError(`The first or last character of worksheet name cannot be a single quotation mark: ${name}`);
-			if (name && name.length > 31) name = name.substring(0, 31);
-			if (this._workbook.worksheets.find((ws) => ws && ws.name.toLowerCase() === name.toLowerCase())) throw new WorksheetNameError(`Worksheet name already exists: ${name}`);
+			if (name && name.length > 31) {
+				console.warn(`Worksheet name ${name} exceeds 31 chars. This will be truncated`);
+				name = name.substring(0, 31);
+			}
+			const nameLower = name.toLowerCase();
+			if (this._workbook.worksheets.find((ws) => ws && ws !== this && ws.name.toLowerCase() === nameLower)) throw new WorksheetNameError(`Worksheet name already exists: ${name}`);
 			this._name = name;
 		}
 		/**
@@ -10812,6 +10816,52 @@ var ExcelTS = (function(exports) {
 		}
 	};
 	//#endregion
+	//#region src/modules/excel/xlsx/xform/sheet/hyperlink-xform.ts
+	var HyperlinkXform = class extends BaseXform {
+		get tag() {
+			return "hyperlink";
+		}
+		render(xmlStream, model) {
+			if (model.target && isInternalLink(model.target)) xmlStream.leafNode("hyperlink", {
+				ref: model.address,
+				tooltip: model.tooltip,
+				location: model.target.slice(1)
+			});
+			else xmlStream.leafNode("hyperlink", {
+				ref: model.address,
+				"r:id": model.rId,
+				tooltip: model.tooltip
+			});
+		}
+		parseOpen(node) {
+			if (node.name === "hyperlink") {
+				this.model = {
+					address: node.attributes.ref,
+					rId: node.attributes["r:id"],
+					tooltip: node.attributes.tooltip
+				};
+				if (node.attributes.location) {
+					const loc = node.attributes.location;
+					this.model.target = loc.startsWith("#") ? loc : `#${loc}`;
+				}
+				return true;
+			}
+			return false;
+		}
+		parseText() {}
+		parseClose() {
+			return false;
+		}
+	};
+	/**
+	* Internal hyperlinks start with "#" (e.g. "#Sheet2!A1").
+	* This matches Excel's convention and the OOXML spec where internal links
+	* use the `location` attribute instead of a relationship.
+	*/
+	function isInternalLink(target) {
+		return target.startsWith("#");
+	}
+	//#endregion
 	//#region src/modules/excel/xlsx/xform/sheet/cell-xform.ts
 	function getValueType(v) {
 		if (v === null || v === void 0) return Enums.ValueType.Null;
@@ -11338,45 +11388,6 @@ var ExcelTS = (function(exports) {
 		}
 	};
 	//#endregion
-	//#region src/modules/excel/xlsx/xform/sheet/hyperlink-xform.ts
-	var HyperlinkXform = class extends BaseXform {
-		get tag() {
-			return "hyperlink";
-		}
-		render(xmlStream, model) {
-			if (this.isInternalLink(model)) xmlStream.leafNode("hyperlink", {
-				ref: model.address,
-				"r:id": model.rId,
-				tooltip: model.tooltip,
-				location: model.target
-			});
-			else xmlStream.leafNode("hyperlink", {
-				ref: model.address,
-				"r:id": model.rId,
-				tooltip: model.tooltip
-			});
-		}
-		parseOpen(node) {
-			if (node.name === "hyperlink") {
-				this.model = {
-					address: node.attributes.ref,
-					rId: node.attributes["r:id"],
-					tooltip: node.attributes.tooltip
-				};
-				if (node.attributes.location) this.model.target = node.attributes.location;
-				return true;
-			}
-			return false;
-		}
-		parseText() {}
-		parseClose() {
-			return false;
-		}
-		isInternalLink(model) {
-			return !!(model.target && /^[^!]+![a-zA-Z]+[\d]+$/.test(model.target));
-		}
-	};
-	//#endregion
 	//#region src/modules/excel/xlsx/xform/sheet/merge-cell-xform.ts
 	var MergeCellXform = class extends BaseXform {
 		get tag() {
@@ -13646,6 +13657,7 @@ var ExcelTS = (function(exports) {
 				return `rId${r.length + 1}`;
 			}
 			model.hyperlinks.forEach((hyperlink) => {
+				if (isInternalLink(hyperlink.target)) return;
 				const rId = nextRid(rels);
 				hyperlink.rId = rId;
 				rels.push({
@@ -13992,6 +14004,7 @@ var ExcelTS = (function(exports) {
 			}, {});
 			options.hyperlinkMap = (model.hyperlinks ?? []).reduce((h, hyperlink) => {
 				if (hyperlink.rId) h[hyperlink.address] = rels[hyperlink.rId].Target;
+				else if (hyperlink.target) h[hyperlink.address] = hyperlink.target;
 				return h;
 			}, {});
 			options.formulae = {};
@@ -15504,10 +15517,7 @@ var ExcelTS = (function(exports) {
 				case "m": return "<m />";
 				case "d": return `<d v="${formatDateForExcel(value.value)}" />`;
 				case "e": return `<e v="${xmlEncode(value.value)}" />`;
-				default: {
-					const _exhaustive = value;
-					throw new Error(`Unhandled record value type: ${_exhaustive.type}`);
-				}
+				default: throw new Error(`Unhandled record value type: ${value.type}`);
 			}
 		}
 		renderTableNew(sourceBodyRows, cacheFields) {
@@ -19670,7 +19680,6 @@ var ExcelTS = (function(exports) {
 	createStreamFormatChecker("decompression", "gzip");
 	createStreamFormatChecker("compression", "deflate");
 	createStreamFormatChecker("decompression", "deflate");
-	new Uint8Array([120, 1]), new Uint8Array([120, 1]), new Uint8Array([120, 94]), new Uint8Array([120, 94]), new Uint8Array([120, 94]), new Uint8Array([120, 94]), new Uint8Array([120, 156]), new Uint8Array([120, 218]), new Uint8Array([120, 218]), new Uint8Array([120, 218]);
 	//#endregion
 	//#region src/modules/archive/shared/errors.ts
 	/**
@@ -19771,10 +19780,9 @@ var ExcelTS = (function(exports) {
 	function resolvePoolOptions(options) {
 		const maxWorkers = options?.maxWorkers ?? DEFAULT_POOL_OPTIONS.maxWorkers;
 		const minWorkers = options?.minWorkers ?? DEFAULT_POOL_OPTIONS.minWorkers;
-		const validatedMinWorkers = Math.min(minWorkers, maxWorkers);
 		return {
 			maxWorkers: Math.max(1, maxWorkers),
-			minWorkers: Math.max(0, validatedMinWorkers),
+			minWorkers: Math.max(0, Math.min(minWorkers, maxWorkers)),
 			idleTimeout: Math.max(0, options?.idleTimeout ?? DEFAULT_POOL_OPTIONS.idleTimeout),
 			useTransferables: options?.useTransferables ?? DEFAULT_POOL_OPTIONS.useTransferables,
 			workerUrl: options?.workerUrl
@@ -21121,18 +21129,6 @@ self.onmessage = async function(event) {
 	function decompressSync(data) {
 		return inflateRaw(data);
 	}
-	new Uint8Array([
-		31,
-		139,
-		8,
-		0,
-		0,
-		0,
-		0,
-		0,
-		0,
-		255
-	]);
 	//#endregion
 	//#region src/modules/archive/compression/streaming-compress.browser.ts
 	/**
@@ -29911,6 +29907,13 @@ self.onmessage = async function(event) {
 			return this._hyperlinksProxy || (this._hyperlinksProxy = new HyperlinksProxy(this));
 		}
 		addHyperlink(hyperlink) {
+			if (isInternalLink(hyperlink.target)) {
+				this._hyperlinks.push({
+					address: hyperlink.address,
+					target: hyperlink.target
+				});
+				return;
+			}
 			const relationship = {
 				Target: hyperlink.target,
 				Type: RelType.Hyperlink,
@@ -30710,7 +30713,10 @@ self.onmessage = async function(event) {
 		getWorksheet(id) {
 			if (id === void 0) return this._worksheets.find(() => true);
 			if (typeof id === "number") return this._worksheets[id];
-			if (typeof id === "string") return this._worksheets.find((ws) => ws?.name === id);
+			if (typeof id === "string") {
+				const idLower = id.toLowerCase();
+				return this._worksheets.find((ws) => ws?.name?.toLowerCase() === idLower);
+			}
 		}
 		addStyles() {
 			return new Promise((resolve) => {
@@ -32389,9 +32395,11 @@ onmessage = async (ev) => {
 								break;
 							case "hyperlink":
 								if (inHyperlinks) {
+									const loc = node.attributes.location;
 									const hyperlink = {
 										ref: node.attributes.ref,
-										rId: node.attributes["r:id"]
+										rId: node.attributes["r:id"],
+										target: loc ? loc.startsWith("#") ? loc : `#${loc}` : void 0
 									};
 									if (emitHyperlinks) (worksheetEvents ||= []).push({
 										eventType: "hyperlink",
@@ -33528,10 +33536,7 @@ onmessage = async (ev) => {
 				modified: true,
 				extras: row.splice(expectedCols)
 			};
-			default: {
-				const _never = columnMore;
-				throw new Error(`Unknown columnMore strategy: ${_never}`);
-			}
+			default: throw new Error(`Unknown columnMore strategy: ${columnMore}`);
 		}
 		switch (columnLess) {
 			case "error": return {
@@ -33547,10 +33552,7 @@ onmessage = async (ev) => {
 					errorCode: "TooFewFields",
 					modified: true
 				};
-			default: {
-				const _never = columnLess;
-				throw new Error(`Unknown columnLess strategy: ${_never}`);
-			}
+			default: throw new Error(`Unknown columnLess strategy: ${columnLess}`);
 		}
 	}
 	/**
@@ -37149,7 +37151,10 @@ onmessage = async (ev) => {
 		getWorksheet(id) {
 			if (id === void 0) return this._worksheets.find(Boolean);
 			if (typeof id === "number") return this._worksheets[id];
-			if (typeof id === "string") return this._worksheets.find((worksheet) => worksheet && worksheet.name === id);
+			if (typeof id === "string") {
+				const idLower = id.toLowerCase();
+				return this._worksheets.find((worksheet) => worksheet && worksheet.name.toLowerCase() === idLower);
+			}
 		}
 		/**
 		* Return a clone of worksheets in order