ooxml-core 0.21.2 → 0.22.1

package/README.md

@@ -9,7 +9,6 @@
 **Read, edit, validate and write Office Open XML documents in TypeScript.**
 One package, one XML model, every format: Word, PowerPoint, Excel and the shared building blocks beneath them. Visio VSDX parsing and bounded editing are available through `ooxml-core/visio`.
 
-[![license](https://img.shields.io/badge/license-Apache--2.0-blue.svg)](LICENSE)
 [![CI](https://github.com/ChristopherVR/ooxml/actions/workflows/ci.yml/badge.svg)](https://github.com/ChristopherVR/ooxml/actions/workflows/ci.yml)
 [![Contributor Covenant](https://img.shields.io/badge/Contributor%20Covenant-2.1-4baaaa.svg)](CODE_OF_CONDUCT.md)
 
@@ -17,7 +16,7 @@ One package, one XML model, every format: Word, PowerPoint, Excel and the shared
 [**Packages and areas**](#features-and-api-one-package-many-areas)  · 
 [**Install**](#install)  · 
 [**Examples**](#quick-start)  · 
-[**Contributing**](CONTRIBUTING.md)
+[**Contributing**](../../CONTRIBUTING.md)
 
 </div>
 
@@ -63,7 +62,7 @@ The `chart` subpath provides shared chart data calculations (regression, quartil
 blank values and stacked series). The `text` subpath provides script-category
 segmentation primitives. These areas are DOM-free and do not import a product model.
 SVG curve flattening is available from `geometry`. See the
-[PowerPoint reuse audit](docs/pptx-shared-logic-audit.md) for extraction evidence
+[PowerPoint reuse audit](../../docs/pptx-shared-logic-audit.md) for extraction evidence
 and remaining candidates.
 
 ## Install
@@ -110,7 +109,7 @@ const part = resolvePartPath('word/document.xml', rels.get('rId5')!.target);
 
 ## Contributing and roadmap
 
-`ooxml-core` is developed in the [ChristopherVR/ooxml](https://github.com/ChristopherVR/ooxml) repository, which also holds [`ooxml-ui`](https://www.npmjs.com/package/ooxml-ui), the MCP servers and the Word, Excel, Visio and OpenTeams viewers. Its README covers the repository layout, the development workflow and releases, and the [roadmap](https://github.com/ChristopherVR/ooxml/blob/main/docs/roadmap.md) says what comes next; the working agreements are in [AGENTS.md](AGENTS.md) and [CONTRIBUTING.md](CONTRIBUTING.md), and [PROVENANCE.md](PROVENANCE.md) records where each module came from.
+`ooxml-core` is developed in the [ChristopherVR/ooxml](https://github.com/ChristopherVR/ooxml) repository, which also holds [`ooxml-ui`](https://www.npmjs.com/package/ooxml-ui), the MCP servers and the Word, Excel, Visio and OpenTeams viewers. Its README covers the repository layout, the development workflow and releases, and the [roadmap](https://github.com/ChristopherVR/ooxml/blob/main/docs/roadmap.md) says what comes next; the working agreements are in [AGENTS.md](../../AGENTS.md) and [CONTRIBUTING.md](../../CONTRIBUTING.md), and [PROVENANCE.md](../../PROVENANCE.md) records where each module came from.
 
 ## License
 

package/dist/automation/index.cjs

@@ -1404,7 +1404,7 @@ async function applyCoreProperties(zip, next, prior) {
   await ensureContentTypeOverride(zip, CORE_PATH, CORE_TYPE);
   const rels = zip.file("_rels/.rels");
   const relsXml = await rels?.async("string") ?? '<?xml version="1.0" encoding="UTF-8" standalone="yes"?><Relationships xmlns="http://schemas.openxmlformats.org/package/2006/relationships"></Relationships>';
-  if (!relsXml.includes(CORE_REL)) {
+  if (!relsXml.includes(`Type="${CORE_REL}"`)) {
     const ids = [...relsXml.matchAll(/Id="rId(\d+)"/g)].map((match2) => Number(match2[1]));
     const id = `rId${Math.max(0, ...ids) + 1}`;
     zip.file(
@@ -11855,7 +11855,10 @@ function writeDrawing(writer, partName, objects, sourceDrawing) {
   let nextId = Math.max(
     1,
     ...kept.flatMap(
-      (xml) => [...xml.matchAll(/cNvPr[^>]*\sid="(\d+)"/g)].map((m) => Number(m[1]))
+      (xml) => [...xml.matchAll(/cNvPr([^>]*)/g)].flatMap((m) => {
+        const id = /\sid="(\d+)"/.exec(m[1] ?? "");
+        return id ? [Number(id[1])] : [];
+      })
     )
   ) + 1;
   let body = "";
@@ -12841,7 +12844,8 @@ function parseDay(text) {
 function parseDateTimeInput(text, date1904) {
   const time = parseTime(text);
   if (time) return { value: time.fraction, numFmt: time.format };
-  const split = /^(.+?)(?:\s+|T)(\d{1,2}:\d{1,2}(?::\d{1,2})?(?:\s*(?:am|pm|a|p))?)$/i.exec(text);
+  const tail = /(?:\s{1,8}|T)(\d{1,2}:\d{1,2}(?::\d{1,2})?(?:\s{0,8}(?:am|pm|a|p))?)$/i.exec(text);
+  const split = tail && tail.index > 0 ? [tail[0], text.slice(0, tail.index), tail[1]] : null;
   const dayText = split ? split[1] ?? "" : text;
   const day = parseDay(dayText);
   if (!day) return void 0;
@@ -12886,10 +12890,10 @@ function parseNumberInput(text) {
     s = s.slice(lead[0].length);
     if (!sign()) return void 0;
   } else {
-    const trail = /\s*([€£])$/.exec(s);
-    if (trail) {
-      currency = trail[1] ?? "";
-      s = s.slice(0, -trail[0].length);
+    const last = s.slice(-1);
+    if (last === "\u20AC" || last === "\xA3") {
+      currency = last;
+      s = s.slice(0, -1).trimEnd();
     }
   }
   let percent = false;
@@ -25586,7 +25590,7 @@ async function saveXlsx(workbook) {
 
 // xlsx/edit/context.ts
 function sheetAt(workbook, index2) {
-  const sheet = workbook.sheets[index2];
+  const sheet = Number.isInteger(index2) ? workbook.sheets[index2] : void 0;
   if (!sheet) throw new RangeError(`No sheet at index ${index2}`);
   return sheet;
 }
@@ -26731,7 +26735,9 @@ function decodeEntities(text) {
 }
 function parseAttributes(text) {
   const attrs2 = /* @__PURE__ */ new Map();
-  for (const m of text.matchAll(/([\w:-]+)\s*=\s*(?:"([^"]*)"|'([^']*)'|([^\s>]+))/g))
+  for (const m of text.matchAll(
+    /([\w:-]{1,64})\s{0,16}=\s{0,16}(?:"([^"]*)"|'([^']*)'|([^\s>]+))/g
+  ))
     attrs2.set((m[1] ?? "").toLowerCase(), decodeEntities(m[2] ?? m[3] ?? m[4] ?? ""));
   return attrs2;
 }
@@ -26786,9 +26792,56 @@ function stylePatchFromCss(css, inner2) {
   if (numFmt) patch.numFmt = decodeNumberFormat(numFmt);
   return patch;
 }
+function removeComments(text) {
+  let out = "";
+  let from = 0;
+  for (; ; ) {
+    const open = text.indexOf("<!--", from);
+    const close = open < 0 ? -1 : text.indexOf("-->", open + 4);
+    if (close < 0) return out + text.slice(from);
+    out += text.slice(from, open);
+    from = close + 3;
+  }
+}
+function removeTags(text) {
+  let out = "";
+  let i = 0;
+  while (i < text.length) {
+    const ch = text[i];
+    if (ch === "<") {
+      let j = i + 1;
+      while (j < text.length && text[j] !== "<" && text[j] !== ">") j++;
+      if (text[j] === ">") {
+        i = j + 1;
+        continue;
+      }
+    }
+    out += ch;
+    i++;
+  }
+  return out;
+}
+function stripTags(markup) {
+  let previous;
+  let text = markup;
+  do {
+    previous = text;
+    text = removeTags(removeComments(text));
+  } while (text !== previous);
+  return removeAll(text, ["<!--", "-->"]);
+}
+function removeAll(text, tokens2) {
+  let current = text;
+  let previous;
+  do {
+    previous = current;
+    for (const token of tokens2) current = current.split(token).join("");
+  } while (current !== previous);
+  return current;
+}
 function cellText(inner2) {
   return decodeEntities(
-    inner2.replace(/<!--[\s\S]*?-->/g, "").replace(/<br\s*\/?>/gi, "\0").replace(/<[^>]+>/g, "").replace(/\s+/g, " ").replace(/ ?\u0000 ?/g, "\n")
+    stripTags(inner2.replace(/<br\s*\/?>/gi, "\0")).replace(/\s+/g, " ").replace(/ ?\u0000 ?/g, "\n")
   ).replace(/ /g, " ").trim();
 }
 function parseHtmlTable(html, base) {
@@ -26796,7 +26849,9 @@ function parseHtmlTable(html, base) {
   if (table2 === void 0) return void 0;
   const classes = /* @__PURE__ */ new Map();
   for (const block of html.matchAll(/<style\b[^>]*>([\s\S]*?)<\/style>/gi))
-    for (const rule2 of (block[1] ?? "").replace(/<!--|-->/g, "").matchAll(/\.([\w-]+)\s*\{([^}]*)\}/g))
+    for (const rule2 of removeAll(block[1] ?? "", ["<!--", "-->"]).matchAll(
+      /\.([\w-]+)\s*\{([^}]*)\}/g
+    ))
       classes.set(rule2[1] ?? "", rule2[2] ?? "");
   const grid = [];
   const taken = /* @__PURE__ */ new Set();
@@ -26966,7 +27021,15 @@ function linearTrend(values2) {
   const slope = den ? num3 / den : 0;
   return (k) => Number((meanY + slope * (k - meanX)).toPrecision(15));
 }
-var TEXT_NUMBER = /^(.*?)(\d+)(\D*)$/;
+var isDigit3 = (ch) => ch !== void 0 && ch >= "0" && ch <= "9";
+function matchTextNumber(text) {
+  let end = text.length;
+  while (end > 0 && !isDigit3(text[end - 1])) end--;
+  if (end === 0) return null;
+  let start = end;
+  while (start > 0 && isDigit3(text[start - 1])) start--;
+  return [text, text.slice(0, start), text.slice(start, end), text.slice(end)];
+}
 var QUARTER = /^(q|qtr|quarter)(\s?)([1-4])$/i;
 function textSeries(texts) {
   const lists = texts.map(listMatch);
@@ -26988,7 +27051,7 @@ function textSeries(texts) {
     const step = q1 ? mod(Number(q1[3]) - 1 - start, 4) || 1 : 1;
     return (k) => ({ kind: "value", value: `${q0[1]}${q0[2]}${mod(start + step * k, 4) + 1}` });
   }
-  const parts = texts.map((t) => TEXT_NUMBER.exec(t));
+  const parts = texts.map(matchTextNumber);
   const p0 = parts[0];
   if (!p0 || !parts.every((p) => p && p[1] === p0[1] && p[3] === p0[3])) return void 0;
   const numbers3 = parts.map((p) => Number(p?.[2]));
@@ -27585,7 +27648,7 @@ function setAutoFilter(ctx, s, range) {
       const old = sheet.autoFilter;
       if (old) applyFilter(ctx, sheet, { range: old.range });
       if (r) sheet.autoFilter = { range: r };
-      else delete sheet.autoFilter;
+      else Reflect.deleteProperty(sheet, "autoFilter");
     },
     { sheet: s, structural: true }
   );
@@ -28647,9 +28710,10 @@ function updateChart(ctx, s, index2, patch) {
     () => {
       const live = sheet.drawings[index2];
       for (const [key2, value] of Object.entries(structuredClone(patch))) {
-        if (key2 === "kind" || key2 === "partName") continue;
-        if (value === void 0) delete live[key2];
-        else live[key2] = value;
+        if (key2 === "kind" || key2 === "partName" || key2 === "__proto__") continue;
+        if (key2 === "constructor" || key2 === "prototype") continue;
+        if (value === void 0) Reflect.deleteProperty(live, key2);
+        else Reflect.set(live, key2, value);
       }
     },
     { sheet: s }
@@ -29158,7 +29222,7 @@ function setDefaultColumnWidth(ctx, s, width) {
     "format",
     [{ kind: "sheet", sheet: s }],
     () => {
-      if (width === void 0) delete sheet.defaultColWidth;
+      if (width === void 0) Reflect.deleteProperty(sheet, "defaultColWidth");
       else sheet.defaultColWidth = width;
     },
     { sheet: s, structural: true }
@@ -29180,7 +29244,7 @@ function setPageSetup(ctx, s, patch) {
     () => {
       const next = compact2({ ...sheet.pageSetup, ...structuredClone(patch) });
       if (Object.keys(next).length) sheet.pageSetup = next;
-      else delete sheet.pageSetup;
+      else Reflect.deleteProperty(sheet, "pageSetup");
     },
     { sheet: s }
   );
@@ -29196,7 +29260,7 @@ function setPrintOptions(ctx, s, patch) {
       for (const key2 of Object.keys(next))
         if (!next[key2]) delete next[key2];
       if (Object.keys(next).length) sheet.printOptions = next;
-      else delete sheet.printOptions;
+      else Reflect.deleteProperty(sheet, "printOptions");
       sheet.preserved.delete("printOptions");
     },
     { sheet: s }
@@ -29802,7 +29866,7 @@ function setSheetProtection(ctx, s, protection2, password) {
     [{ kind: "sheet", sheet: s }],
     () => {
       if (!protection2) {
-        delete sheet.protection;
+        Reflect.deleteProperty(sheet, "protection");
         return;
       }
       const next = structuredClone(protection2);
@@ -30774,7 +30838,7 @@ function setTabColor(ctx, index2, color2) {
     [{ kind: "sheet", sheet: index2 }],
     () => {
       if (color2) sheet.tabColor = { ...color2 };
-      else delete sheet.tabColor;
+      else Reflect.deleteProperty(sheet, "tabColor");
     },
     { sheet: index2 }
   );

package/dist/automation/index.mjs

@@ -1398,7 +1398,7 @@ async function applyCoreProperties(zip, next, prior) {
   await ensureContentTypeOverride(zip, CORE_PATH, CORE_TYPE);
   const rels = zip.file("_rels/.rels");
   const relsXml = await rels?.async("string") ?? '<?xml version="1.0" encoding="UTF-8" standalone="yes"?><Relationships xmlns="http://schemas.openxmlformats.org/package/2006/relationships"></Relationships>';
-  if (!relsXml.includes(CORE_REL)) {
+  if (!relsXml.includes(`Type="${CORE_REL}"`)) {
     const ids = [...relsXml.matchAll(/Id="rId(\d+)"/g)].map((match2) => Number(match2[1]));
     const id = `rId${Math.max(0, ...ids) + 1}`;
     zip.file(
@@ -11849,7 +11849,10 @@ function writeDrawing(writer, partName, objects, sourceDrawing) {
   let nextId = Math.max(
     1,
     ...kept.flatMap(
-      (xml) => [...xml.matchAll(/cNvPr[^>]*\sid="(\d+)"/g)].map((m) => Number(m[1]))
+      (xml) => [...xml.matchAll(/cNvPr([^>]*)/g)].flatMap((m) => {
+        const id = /\sid="(\d+)"/.exec(m[1] ?? "");
+        return id ? [Number(id[1])] : [];
+      })
     )
   ) + 1;
   let body = "";
@@ -12835,7 +12838,8 @@ function parseDay(text) {
 function parseDateTimeInput(text, date1904) {
   const time = parseTime(text);
   if (time) return { value: time.fraction, numFmt: time.format };
-  const split = /^(.+?)(?:\s+|T)(\d{1,2}:\d{1,2}(?::\d{1,2})?(?:\s*(?:am|pm|a|p))?)$/i.exec(text);
+  const tail = /(?:\s{1,8}|T)(\d{1,2}:\d{1,2}(?::\d{1,2})?(?:\s{0,8}(?:am|pm|a|p))?)$/i.exec(text);
+  const split = tail && tail.index > 0 ? [tail[0], text.slice(0, tail.index), tail[1]] : null;
   const dayText = split ? split[1] ?? "" : text;
   const day = parseDay(dayText);
   if (!day) return void 0;
@@ -12880,10 +12884,10 @@ function parseNumberInput(text) {
     s = s.slice(lead[0].length);
     if (!sign()) return void 0;
   } else {
-    const trail = /\s*([€£])$/.exec(s);
-    if (trail) {
-      currency = trail[1] ?? "";
-      s = s.slice(0, -trail[0].length);
+    const last = s.slice(-1);
+    if (last === "\u20AC" || last === "\xA3") {
+      currency = last;
+      s = s.slice(0, -1).trimEnd();
     }
   }
   let percent = false;
@@ -25580,7 +25584,7 @@ async function saveXlsx(workbook) {
 
 // xlsx/edit/context.ts
 function sheetAt(workbook, index2) {
-  const sheet = workbook.sheets[index2];
+  const sheet = Number.isInteger(index2) ? workbook.sheets[index2] : void 0;
   if (!sheet) throw new RangeError(`No sheet at index ${index2}`);
   return sheet;
 }
@@ -26725,7 +26729,9 @@ function decodeEntities(text) {
 }
 function parseAttributes(text) {
   const attrs2 = /* @__PURE__ */ new Map();
-  for (const m of text.matchAll(/([\w:-]+)\s*=\s*(?:"([^"]*)"|'([^']*)'|([^\s>]+))/g))
+  for (const m of text.matchAll(
+    /([\w:-]{1,64})\s{0,16}=\s{0,16}(?:"([^"]*)"|'([^']*)'|([^\s>]+))/g
+  ))
     attrs2.set((m[1] ?? "").toLowerCase(), decodeEntities(m[2] ?? m[3] ?? m[4] ?? ""));
   return attrs2;
 }
@@ -26780,9 +26786,56 @@ function stylePatchFromCss(css, inner2) {
   if (numFmt) patch.numFmt = decodeNumberFormat(numFmt);
   return patch;
 }
+function removeComments(text) {
+  let out = "";
+  let from = 0;
+  for (; ; ) {
+    const open = text.indexOf("<!--", from);
+    const close = open < 0 ? -1 : text.indexOf("-->", open + 4);
+    if (close < 0) return out + text.slice(from);
+    out += text.slice(from, open);
+    from = close + 3;
+  }
+}
+function removeTags(text) {
+  let out = "";
+  let i = 0;
+  while (i < text.length) {
+    const ch = text[i];
+    if (ch === "<") {
+      let j = i + 1;
+      while (j < text.length && text[j] !== "<" && text[j] !== ">") j++;
+      if (text[j] === ">") {
+        i = j + 1;
+        continue;
+      }
+    }
+    out += ch;
+    i++;
+  }
+  return out;
+}
+function stripTags(markup) {
+  let previous;
+  let text = markup;
+  do {
+    previous = text;
+    text = removeTags(removeComments(text));
+  } while (text !== previous);
+  return removeAll(text, ["<!--", "-->"]);
+}
+function removeAll(text, tokens2) {
+  let current = text;
+  let previous;
+  do {
+    previous = current;
+    for (const token of tokens2) current = current.split(token).join("");
+  } while (current !== previous);
+  return current;
+}
 function cellText(inner2) {
   return decodeEntities(
-    inner2.replace(/<!--[\s\S]*?-->/g, "").replace(/<br\s*\/?>/gi, "\0").replace(/<[^>]+>/g, "").replace(/\s+/g, " ").replace(/ ?\u0000 ?/g, "\n")
+    stripTags(inner2.replace(/<br\s*\/?>/gi, "\0")).replace(/\s+/g, " ").replace(/ ?\u0000 ?/g, "\n")
   ).replace(/ /g, " ").trim();
 }
 function parseHtmlTable(html, base) {
@@ -26790,7 +26843,9 @@ function parseHtmlTable(html, base) {
   if (table2 === void 0) return void 0;
   const classes = /* @__PURE__ */ new Map();
   for (const block of html.matchAll(/<style\b[^>]*>([\s\S]*?)<\/style>/gi))
-    for (const rule2 of (block[1] ?? "").replace(/<!--|-->/g, "").matchAll(/\.([\w-]+)\s*\{([^}]*)\}/g))
+    for (const rule2 of removeAll(block[1] ?? "", ["<!--", "-->"]).matchAll(
+      /\.([\w-]+)\s*\{([^}]*)\}/g
+    ))
       classes.set(rule2[1] ?? "", rule2[2] ?? "");
   const grid = [];
   const taken = /* @__PURE__ */ new Set();
@@ -26960,7 +27015,15 @@ function linearTrend(values2) {
   const slope = den ? num3 / den : 0;
   return (k) => Number((meanY + slope * (k - meanX)).toPrecision(15));
 }
-var TEXT_NUMBER = /^(.*?)(\d+)(\D*)$/;
+var isDigit3 = (ch) => ch !== void 0 && ch >= "0" && ch <= "9";
+function matchTextNumber(text) {
+  let end = text.length;
+  while (end > 0 && !isDigit3(text[end - 1])) end--;
+  if (end === 0) return null;
+  let start = end;
+  while (start > 0 && isDigit3(text[start - 1])) start--;
+  return [text, text.slice(0, start), text.slice(start, end), text.slice(end)];
+}
 var QUARTER = /^(q|qtr|quarter)(\s?)([1-4])$/i;
 function textSeries(texts) {
   const lists = texts.map(listMatch);
@@ -26982,7 +27045,7 @@ function textSeries(texts) {
     const step = q1 ? mod(Number(q1[3]) - 1 - start, 4) || 1 : 1;
     return (k) => ({ kind: "value", value: `${q0[1]}${q0[2]}${mod(start + step * k, 4) + 1}` });
   }
-  const parts = texts.map((t) => TEXT_NUMBER.exec(t));
+  const parts = texts.map(matchTextNumber);
   const p0 = parts[0];
   if (!p0 || !parts.every((p) => p && p[1] === p0[1] && p[3] === p0[3])) return void 0;
   const numbers3 = parts.map((p) => Number(p?.[2]));
@@ -27579,7 +27642,7 @@ function setAutoFilter(ctx, s, range) {
       const old = sheet.autoFilter;
       if (old) applyFilter(ctx, sheet, { range: old.range });
       if (r) sheet.autoFilter = { range: r };
-      else delete sheet.autoFilter;
+      else Reflect.deleteProperty(sheet, "autoFilter");
     },
     { sheet: s, structural: true }
   );
@@ -28641,9 +28704,10 @@ function updateChart(ctx, s, index2, patch) {
     () => {
       const live = sheet.drawings[index2];
       for (const [key2, value] of Object.entries(structuredClone(patch))) {
-        if (key2 === "kind" || key2 === "partName") continue;
-        if (value === void 0) delete live[key2];
-        else live[key2] = value;
+        if (key2 === "kind" || key2 === "partName" || key2 === "__proto__") continue;
+        if (key2 === "constructor" || key2 === "prototype") continue;
+        if (value === void 0) Reflect.deleteProperty(live, key2);
+        else Reflect.set(live, key2, value);
       }
     },
     { sheet: s }
@@ -29152,7 +29216,7 @@ function setDefaultColumnWidth(ctx, s, width) {
     "format",
     [{ kind: "sheet", sheet: s }],
     () => {
-      if (width === void 0) delete sheet.defaultColWidth;
+      if (width === void 0) Reflect.deleteProperty(sheet, "defaultColWidth");
       else sheet.defaultColWidth = width;
     },
     { sheet: s, structural: true }
@@ -29174,7 +29238,7 @@ function setPageSetup(ctx, s, patch) {
     () => {
       const next = compact2({ ...sheet.pageSetup, ...structuredClone(patch) });
       if (Object.keys(next).length) sheet.pageSetup = next;
-      else delete sheet.pageSetup;
+      else Reflect.deleteProperty(sheet, "pageSetup");
     },
     { sheet: s }
   );
@@ -29190,7 +29254,7 @@ function setPrintOptions(ctx, s, patch) {
       for (const key2 of Object.keys(next))
         if (!next[key2]) delete next[key2];
       if (Object.keys(next).length) sheet.printOptions = next;
-      else delete sheet.printOptions;
+      else Reflect.deleteProperty(sheet, "printOptions");
       sheet.preserved.delete("printOptions");
     },
     { sheet: s }
@@ -29796,7 +29860,7 @@ function setSheetProtection(ctx, s, protection2, password) {
     [{ kind: "sheet", sheet: s }],
     () => {
       if (!protection2) {
-        delete sheet.protection;
+        Reflect.deleteProperty(sheet, "protection");
         return;
       }
       const next = structuredClone(protection2);
@@ -30768,7 +30832,7 @@ function setTabColor(ctx, index2, color2) {
     [{ kind: "sheet", sheet: index2 }],
     () => {
       if (color2) sheet.tabColor = { ...color2 };
-      else delete sheet.tabColor;
+      else Reflect.deleteProperty(sheet, "tabColor");
     },
     { sheet: index2 }
   );