slidemind 0.3.16 → 0.3.18

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "slidemind",
-  "version": "0.3.16",
+  "version": "0.3.18",
   "description": "Connect Claude Code to the slidemind PowerPoint add-in — build and edit native, on-brand PowerPoint decks against the active corporate design, from the terminal.",
   "author": {
     "name": "statworx",

package/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "slidemind",
-  "version": "0.3.16",
+  "version": "0.3.18",
   "description": "Connect Codex to the slidemind PowerPoint add-in — build and edit native, on-brand PowerPoint decks against the active corporate design. Requires the slidemind CLI (npm install -g slidemind); the skills explain the install when it is missing.",
   "author": {
     "name": "statworx",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "slidemind",
-  "version": "0.3.16",
+  "version": "0.3.18",
   "description": "Connect Claude Code, Codex & opencode to the slidemind PowerPoint add-in — let your coding agent build and edit native, on-brand PowerPoint decks against your active corporate design, from the terminal.",
   "type": "module",
   "bin": {

package/skills/create-slides/SKILL.md

@@ -652,7 +652,9 @@ From here:
    the gate will open). Two things come back, and both bind:
    - **Measured lines** under each slide — `[fill:warn]` (a stretched region whose organisms pool
      at its top, with the region and the dead band) and `[inventory:warn]` (connectors listed in
-     `asset-plan.md` but not drawn, an empty inventory, a fetched asset no slide references) —
+     `asset-plan.md` but not drawn — counted per route kind too, so five `curved` rows drawn as
+     elbows warn even when the total matches; an empty inventory, a fetched asset no slide
+     references) —
      and a closing `DRAFT CHECK: N measured defects` line. Fix every one in the JSX and re-run;
      the review gate refuses to open while they stand for an unchanged slide. A fill band is
      fixed by distributing the organisms (`grid-template-rows` with one track per organism,

package/src/bridge/build-and-push.mjs

@@ -621,7 +621,10 @@ export async function viewDraft(
 
   let captured;
   try {
-    captured = await captureDeckLocally({ deckDir, bundleDir, onProgress, only });
+    // Converted slides never reach the local engine — it would lint each one
+    // as an unregistered layout and the noise would ride the translate gate.
+    const capturable = all.filter((f) => f.endsWith(".jsx") && !conversions[f]);
+    captured = await captureDeckLocally({ deckDir, bundleDir, onProgress, only: capturable });
   } catch (err) {
     throw new Error(
       `local draft capture failed (${err instanceof Error ? err.message : String(err)}). ` +

package/src/builders/build.mjs

@@ -38,6 +38,7 @@ import { loadChartPreview, loadConnectorPreview } from "../bridge/capture.mjs";
 import { FAVICON_LINK } from "./gate-server.mjs";
 import { lintSlidesOrThrow } from "./lint.mjs";
 import { resolveSkillForDeck } from "./resolve-skill.mjs";
+import { loadConversions } from "../lib/conversions.mjs";
 import { slideOrderWarning, sortSlideFilenames } from "../lib/slide-order.mjs";
 
 const __filename = fileURLToPath(import.meta.url);
@@ -87,6 +88,29 @@ function inlineLayout(file) {
   );
 }
 
+// Preview stand-ins for the ConvertedHost layouts converted slides invoke. The
+// host's name carries its canvas (`ConvertedHost<cx>x<cy>`, EMU), so the stub
+// sizes itself the way the minted host does (EMU / 9525, 3 decimals) instead of
+// assuming 16:9 — a 4:3 source previews at 960x720, not stretched to 1280.
+// Real geometry still comes from the server render.
+const HOST_NAME_RE = /<(ConvertedHost(\d+)x(\d+))/;
+const emuToCssPx = (emu) => `${Math.round((Number(emu) / 9525) * 1000) / 1000}px`;
+
+function convertedHostStubs(slideFiles, conversions) {
+  const hosts = new Map();
+  for (const file of slideFiles) {
+    if (!conversions[path.basename(file)]) continue;
+    const m = HOST_NAME_RE.exec(fs.readFileSync(file, "utf8"));
+    if (m) hosts.set(m[1], { w: emuToCssPx(m[2]), h: emuToCssPx(m[3]) });
+  }
+  return [...hosts].map(
+    ([name, { w, h }]) =>
+      `<script type="text/jsx-template" data-kind="layout" data-name="${name}" data-binding="${name}">\n` +
+      `export default function ${name}({ background, children }) { return <div style={{ position: "relative", width: "${w}", height: "${h}", overflow: "hidden", background: background || "#FFFFFF" }}>{children}</div>; }\n` +
+      `</script>`,
+  );
+}
+
 function layoutBindingNameForStem(stem) {
   return "Slide" + stem.split("-").map((p) => (
     p ? p[0].toUpperCase() + p.slice(1) : ""
@@ -354,7 +378,15 @@ export async function buildDeck(deckDir, opts = {}) {
       fs.readFileSync(file, "utf8"),
     ]),
   );
-  await lintSlidesOrThrow({ deckDir, slideFiles, layoutSources });
+  // Converted slides (`slidemind adopt`) invoke a synthetic ConvertedHost
+  // layout the server mints per source canvas; no bundle carries it, so the
+  // local engine would fail every one of them as an unregistered layout. They
+  // are linted server-side on render — skip them here, lint the authored rest.
+  const conversions = loadConversions(deckDir, { includeSource: false });
+  const authoredFiles = slideFiles.filter((f) => !conversions[path.basename(f)]);
+  if (authoredFiles.length > 0) {
+    await lintSlidesOrThrow({ deckDir, slideFiles: authoredFiles, layoutSources });
+  }
 
   // The <Chart> stand-in runtime rides in the same bundle cache (see
   // loadChartPreview) — resolved after lint, which already proved the bundle
@@ -364,7 +396,10 @@ export async function buildDeck(deckDir, opts = {}) {
 
   copyDesignSystem(deckDir, skill);
 
-  const layoutsBlock = layoutFiles.map(inlineLayout).join("\n");
+  const layoutsBlock = layoutFiles
+    .map(inlineLayout)
+    .concat(convertedHostStubs(slideFiles, conversions))
+    .join("\n");
   const slidesBlock = slideFiles
     .map((file, i) => inlineSlide(file, i))
     .join("\n");

package/src/builders/draft-check.mjs

@@ -30,6 +30,20 @@ const NONE_RE = /^\(?\s*(none|n\/a|no\b)/i;
 
 const pad2 = (n) => String(n).padStart(2, "0");
 
+// Route kind per connector row / directive — part of the mockup contract: a
+// loop the concept draws as curves that lands as elbows is a different figure,
+// and the count alone (5 = 5) never fires. Port of inventory.ts (gotcha 84).
+const ROUTE_KINDS = ["straight", "elbow", "curved"];
+const emptyRoutes = () => ({ straight: 0, elbow: 0, curved: 0 });
+function routeKindOf(text) {
+  for (const token of text.split(",")) {
+    const t = token.trim().toLowerCase();
+    if (ROUTE_KINDS.includes(t)) return t;
+  }
+  return null;
+}
+const describeRoutes = (r) => ROUTE_KINDS.map((k) => `${r[k]} ${k}`).join(", ");
+
 /** Seed the inventory form once (never overwrites a filled one). `images`: [{slideNumber,label}]. */
 export function seedAssetPlan(deckDir, images) {
   const file = path.join(deckDir, ASSET_PLAN_FILE);
@@ -91,7 +105,7 @@ function parseAssetPlan(text) {
     const line = raw.trim();
     const heading = SLIDE_HEADING_RE.exec(line);
     if (heading) {
-      current = { filled: false, connectors: 0, organisms: 0, icons: 0, iconsResolved: 0, verdict: null, ticked: 0, unticked: 0 };
+      current = { filled: false, connectors: 0, routes: emptyRoutes(), organisms: 0, icons: 0, iconsResolved: 0, verdict: null, ticked: 0, unticked: 0 };
       out.set(Number(heading[1]), current);
       section = "";
       continue;
@@ -119,7 +133,12 @@ function parseAssetPlan(text) {
       if (inline && !NONE_RE.test(inline)) {
         if (section === "organisms" || section === "regions") current.filled = true;
         if (section === "connectors") {
-          current.connectors += inline.split(";").filter((x) => x.trim()).length;
+          for (const row of inline.split(";")) {
+            if (!row.trim()) continue;
+            current.connectors += 1;
+            const kind = routeKindOf(row);
+            if (kind) current.routes[kind] += 1;
+          }
         }
       }
       continue;
@@ -129,7 +148,11 @@ function parseAssetPlan(text) {
     if (!body || NONE_RE.test(body)) continue;
     if (section === "organisms" || section === "regions") current.filled = true;
     if (section === "organisms") current.organisms += 1;
-    if (section === "connectors") current.connectors += 1;
+    if (section === "connectors") {
+      current.connectors += 1;
+      const kind = routeKindOf(body);
+      if (kind) current.routes[kind] += 1;
+    }
     if (section === "icons") {
       current.icons += 1;
       // Resolved = a fetched file appended, or an explicit "(no result)".
@@ -167,6 +190,19 @@ function countConnectors(jsx) {
   return attached + markers;
 }
 
+// Route kind per `data-connector` directive; a missing data-route is straight.
+// Imported native connectors carry their own preset and are not authored
+// against the mockup, so they are left out of the kind tally.
+function countRoutes(jsx) {
+  const routes = emptyRoutes();
+  for (const tag of jsx.match(/<[^<>]*\bdata-connector\b[^<>]*>/g) ?? []) {
+    if (/\bdata-native-connector\b/.test(tag)) continue;
+    const kind = /\bdata-route\s*=\s*["']([a-z]+)["']/.exec(tag)?.[1] ?? "straight";
+    if (ROUTE_KINDS.includes(kind)) routes[kind] += 1;
+  }
+  return routes;
+}
+
 /**
  * `[inventory:warn]` lines for the deck: per authored slide (inventory empty,
  * connectors listed > drawn) plus fetched assets no slide references. Empty
@@ -212,6 +248,14 @@ export function inventoryWarnings(deckDir, jsxFiles = null) {
         `[inventory:warn] ${jsxPath} — asset-plan.md lists ${inv.connectors} connector${inv.connectors === 1 ? "" : "s"} for this slide, the JSX draws ${drawn} (data-connector / marker-end). Draw the missing ones per workflow/authoring.md → Arrow contract; a mockup connector is never dropped.`,
       );
     }
+    const drawnRoutes = countRoutes(jsx);
+    for (const kind of ROUTE_KINDS) {
+      if (inv.routes[kind] > drawnRoutes[kind]) {
+        lines.push(
+          `[inventory:warn] ${jsxPath} — asset-plan.md lists ${inv.routes[kind]} ${kind} connector${inv.routes[kind] === 1 ? "" : "s"} for this slide, the JSX draws ${drawnRoutes[kind]} (${describeRoutes(drawnRoutes)}). The mockup's route shape is the contract: keep data-route="${kind}" and fix the endpoint sides or node positions instead of changing the route kind.`,
+        );
+      }
+    }
   }
   // Fetched assets nobody references: the mockup staged them, the deck lost them.
   const referenced = allJsx

package/src/lib/adopt.mjs

@@ -30,7 +30,14 @@ import { decodeSourceTags, mintSlideGuid, readSlideTags } from "../pptx/tags.mjs
 import { foldPickedSlides } from "../pptx/slide-swap.mjs";
 import { readDeckBundleRef, writeDeckBundleRef } from "../bridge/bundle.mjs";
 import { loadSlideGuids, saveSlideGuids, slideGuidEntry } from "./slide-guids.mjs";
-import { hasSourcedChart, writeConversionRecord } from "./conversions.mjs";
+import { writeConversionRecord } from "./conversions.mjs";
+
+// Converted JSX needs a fresh baseline against the imported PPTX. Source tags
+// alone cannot recover the original conversion evidence or preserved natives.
+export function needsConversion(decoded) {
+  return !decoded || /<ConvertedHost\d+x\d+\b/.test(decoded.jsx) ||
+    /^\s*\/\/\s*SLIDEMIND_DIALECT:\s*converted\b/m.test(decoded.jsx);
+}
 
 export const LIVE_DECK_UNSUPPORTED =
   "conversion of live presentations requires the add-in path in v1 — " +
@@ -174,21 +181,13 @@ export function adoptDeckFromPptx({ pptxPath, deckDir, converted = null, source
   const native = [];
   let bundle = null;
 
-  // Fold in ONE pass: foldPickedSlides unzips and prunes the whole source
-  // package per call, so folding slide by slide redid that work for the entire
-  // deck on every native slide. Three kinds of pick share the pass — a slide
-  // that stays native, a CONVERTED slide whose preserved natives (OLE /
-  // SmartArt / unsupported chart kinds) need their verbatim source package kept
-  // for the render-time splice (design D4/D6), and a converted slide carrying
-  // the deck's own chart as `<Chart data-sm-source-id>`, which the render
-  // patches against that same package (native-charts 5.4a).
+  // Fold every converted slide in one pass. The package supplies preserved
+  // natives AND the original envelope (layouts, passive third-party tags,
+  // relationships). Tagged converted JSX alone cannot restore either safely.
   const picks = [];
   decodedByIndex.forEach((decoded, index) => {
-    if (decoded) return;
-    const conv = convertedSlides.get(index + 1);
-    if (!conv || conv.preserved?.length > 0 || hasSourcedChart(conv.jsx)) {
-      picks.push(index + 1);
-    }
+    if (!needsConversion(decoded)) return;
+    picks.push(index + 1);
   });
   const foldByPick = new Map(
     (picks.length
@@ -229,7 +228,7 @@ export function adoptDeckFromPptx({ pptxPath, deckDir, converted = null, source
   }
 
   decodedByIndex.forEach((decoded, index) => {
-    if (decoded) {
+    if (decoded && !needsConversion(decoded)) {
       const stem = stemFor(decoded.meta.name, index, taken);
       let jsx = decoded.jsx;
       for (const asset of decoded.assets) {
@@ -251,11 +250,14 @@ export function adoptDeckFromPptx({ pptxPath, deckDir, converted = null, source
     if (conv) {
       // Deterministically converted: editable jsx in the converted dialect,
       // plus the identity/evidence record push threads back to the server.
-      const stem = stemFor(conv.name ?? `${pad2(index + 1)}-converted`, index, taken);
+      const stem = stemFor(decoded?.meta.name ?? conv.name ?? `${pad2(index + 1)}-converted`, index, taken);
       let jsx = conv.jsx;
       for (const [from, to] of convAssetRenames) jsx = jsx.split(from).join(to);
       fs.writeFileSync(path.join(slidesDir, `${stem}.jsx`), jsx, "utf8");
-      const guid = conv.guid && !seenGuids.has(conv.guid) ? conv.guid : mintSlideGuid();
+      const candidateGuid = decoded?.guid ?? conv.guid;
+      const owns = !decoded || ownerIndexByGuid.get(decoded.guid)?.index === index;
+      const guid = owns && candidateGuid && !seenGuids.has(candidateGuid)
+        ? candidateGuid : mintSlideGuid();
       seenGuids.add(guid);
       guids[`slides/${stem}.jsx`] = slideGuidEntry(guid, jsx);
       writeConversionRecord(
@@ -277,6 +279,7 @@ export function adoptDeckFromPptx({ pptxPath, deckDir, converted = null, source
         },
         fold?.bytes ?? null,
       );
+      bundle ??= decoded?.meta.bundle ?? null;
       convertedFiles.push(`${stem}.jsx`);
       return;
     }
@@ -329,7 +332,10 @@ export function adoptDeckFromPptx({ pptxPath, deckDir, converted = null, source
 }
 
 /**
- * Deterministically convert a FILE deck's untagged slides through the add-in's
+ * Convert foreign slides and re-convert embedded converted slides against the
+ * CURRENT imported PPTX. This recovers a fresh baseline/evidence pair without
+ * pretending the original conversion was reviewed. Ordinary authored JSX is
+ * restored verbatim. Requests go through the add-in's
  * `POST /api/cli/brands/:brandId/convert-pptx` route. Returns the route result
  * (`{ slides, assets, skipped, warnings }` + the `brandId` it ran against) to
  * hand to {@link adoptDeckFromPptx}, or null when conversion is not available
@@ -349,7 +355,7 @@ export async function convertUntaggedSlides({
   const { buffer, decodedByIndex } = source ?? readSourceDeck(pptxPath);
   const slides = [];
   decodedByIndex.forEach((decoded, i) => {
-    if (!decoded) slides.push(i + 1);
+    if (needsConversion(decoded)) slides.push(i + 1);
   });
   if (slides.length === 0) return null;
 
@@ -372,7 +378,7 @@ export async function convertUntaggedSlides({
     if (brands.length !== 1) {
       note(
         brands.length === 0
-          ? "no corporate design available to convert against — untagged slides stay native"
+          ? "no corporate design available to convert against — slides requiring conversion stay native"
           : "several corporate designs available — pass `--brand <presFP>` to convert the untagged slides",
       );
       return null;
@@ -383,7 +389,7 @@ export async function convertUntaggedSlides({
     // capability probe — do it here, so a whole .pptx is never shipped to a
     // backend that will just 404 the route.
     if (backendURL && !(await client.backendSupportsConvertPptx(backendURL))) {
-      note("this add-in does not convert foreign slides — untagged slides stay native");
+      note("this add-in does not convert foreign slides — slides requiring conversion stay native");
       return null;
     }
   }
@@ -396,15 +402,15 @@ export async function convertUntaggedSlides({
       backendURL,
     });
     if (result?.ok === false) {
-      note("the add-in declined to convert this file — untagged slides stay native");
+      note("the add-in declined to convert this file — slides requiring conversion stay native");
       return null;
     }
     return { ...result, brandId: resolved };
   } catch (err) {
     note(
       err?.code === "convert_route_unavailable"
-        ? "this add-in does not convert foreign slides — untagged slides stay native"
-        : `slide conversion unavailable (${err?.message ?? err}) — untagged slides stay native`,
+        ? "this add-in does not convert foreign slides — slides requiring conversion stay native"
+        : `slide conversion unavailable (${err?.message ?? err}) — slides requiring conversion stay native`,
     );
     return null;
   }

package/src/lib/reconcile.mjs

@@ -166,8 +166,34 @@ export async function collectLiveReconcileInputs({
     const before = Object.keys(exported).length;
     const ack = await exportRound(exportSlides, pending);
     deck ??= ack?.deck ?? null;
+    // PowerPoint web (package fallback on) may hand back the SAVED file's bytes
+    // (`provenance.source` graph/saved) when the host export fails. Saved bytes
+    // cannot show unsaved manual edits, so projecting them would read "no
+    // drift" for a slide the user is editing right now — treat them as not
+    // exported, same failure mode as a slide that never arrived.
+    const stale = [];
     for (const [guid, slide] of Object.entries(ack?.slides ?? {})) {
-      if (slide?.base64 && slide?.tags) exported[guid] = slide;
+      if (!slide?.base64 || !slide?.tags) continue;
+      if (slide.provenance && slide.provenance.source !== "host") stale.push(guid);
+      else exported[guid] = slide;
+    }
+    // `failed` names slides neither the host nor the stored file could supply
+    // (web fallback); they never come back, so they fail closed at once.
+    const failed = Object.keys(ack?.failed ?? {}).filter(
+      (guid) => !exported[guid] && requested.includes(guid),
+    );
+    const unverifiable = [...new Set([...failed, ...stale])];
+    if (unverifiable.length > 0) {
+      const err = new Error(
+        `the pane could not export ${unverifiable.length} live slide(s) from the ` +
+          `open presentation (${failed.length} unavailable, ${stale.length} only as ` +
+          `the saved file), so their manual edits cannot be checked. Save the ` +
+          `presentation and retry, or re-run with --force to push anyway and ` +
+          `discard any manual edits.`,
+      );
+      err.code = "export_incomplete";
+      err.failed = ack?.failed ?? {};
+      throw err;
     }
     const next = (Array.isArray(ack?.truncated) ? ack.truncated : []).filter(
       (guid) => !exported[guid] && requested.includes(guid),

package/src/pptx/app-props.mjs

@@ -20,7 +20,6 @@ import { unzip, zip, readText } from "./zip.mjs";
 import { slideOrderPaths, decodeXmlEntities } from "./ooxml.mjs";
 import { slideText } from "./read-local.mjs";
 import { stripHiddenOleGraphicFrames } from "./hidden-ole.mjs";
-import { stripRiskyImageExtensions } from "./image-extensions.mjs";
 
 const APP_PROPS = "docProps/app.xml";
 
@@ -61,9 +60,6 @@ export function normalizePptxAppProperties(buffer) {
   if (stripHiddenOleGraphicFrames(entries)) {
     changed = true;
   }
-  if (stripRiskyImageExtensions(entries)) {
-    changed = true;
-  }
 
   return {
     bytes: changed ? zip(entries) : buffer,

package/src/pptx/slide-swap.mjs

@@ -30,7 +30,6 @@ import {
   resolvePartPath,
 } from "./ooxml.mjs";
 import { stripHiddenOleGraphicFrames } from "./hidden-ole.mjs";
-import { stripRiskyImageExtensions } from "./image-extensions.mjs";
 import {
   slideHasForeignTags,
   slideReferencesEmbeddedWorkbook,
@@ -112,7 +111,6 @@ export function foldPickedSlides(buffer, picks, { sourceName = "reused" } = {})
           part === lowest ? picked : part === picked ? lowest : part,
         ) ?? false,
     });
-    const riskyImagesStripped = stripRiskyImageExtensions(copy);
 
     // Prune to the lowest slide's dependency closure + trimmed skeleton.
     const { entries: pruned, changed } = pruneToLowest(copy);
@@ -123,7 +121,6 @@ export function foldPickedSlides(buffer, picks, { sourceName = "reused" } = {})
       !swapped &&
       !notesStripped &&
       !hiddenOleStripped &&
-      !riskyImagesStripped &&
       !changed
     ) {
       return { selection: n, slideId, bytes: buffer };

package/src/pptx/image-extensions.mjs

@@ -1,190 +0,0 @@
-import { readText } from "./zip.mjs";
-import { relsFor, relsPathFor, resolvePartPath } from "./ooxml.mjs";
-
-const CONTENT_TYPES = "[Content_Types].xml";
-
-export function stripRiskyImageExtensions(files) {
-  let changed = false;
-  const removedTargets = new Set();
-  const candidates = Object.keys(files).filter((part) =>
-    /^ppt\/(?:slides|slideMasters|slideLayouts)\/[^/]+\.xml$/.test(part),
-  );
-
-  for (const part of candidates) {
-    const before = readText(files, part);
-    if (!before) continue;
-
-    const removedIds = new Set();
-    const relsById = new Map(relsFor(files, part).map((rel) => [rel.id, rel]));
-    const partDir = part.includes("/") ? part.slice(0, part.lastIndexOf("/")) : "";
-    let after = before.replace(
-      /<a:ext\b[\s\S]*?<\/a:ext>/g,
-      (ext) => {
-        if (!/<(?:asvg:svgBlip|a14:imgLayer)\b/.test(ext)) return ext;
-        for (const id of relationshipIdsIn(ext)) removedIds.add(id);
-        return "";
-      },
-    );
-    after = after.replace(/<a:extLst\b[^>]*>\s*<\/a:extLst>/g, "");
-    after = after.replace(
-      /<p:pic\b[\s\S]*?<\/p:pic>/g,
-      (pic) => {
-        if (!/<a:blip\b(?![^>]*\br:(?:embed|link)=)[^>]*>\s*<\/a:blip>/.test(pic)) {
-          return pic;
-        }
-        for (const id of relationshipIdsIn(pic)) removedIds.add(id);
-        return "";
-      },
-    );
-    after = after.replace(
-      /<p:pic\b[\s\S]*?<\/p:pic>/g,
-      (pic) => {
-        for (const id of relationshipIdsIn(pic)) {
-          const rel = relsById.get(id);
-          if (
-            rel &&
-            rel.targetMode !== "External" &&
-            /\.svg(?:$|[?#])/i.test(rel.target ?? "")
-          ) {
-            removedIds.add(id);
-            return "";
-          }
-        }
-        return pic;
-      },
-    );
-    const relsPath = relsPathFor(part);
-    const relsXml = readText(files, relsPath);
-    const remainingIds = relationshipIdsIn(after);
-    if (relsXml) {
-      for (const rel of relsById.values()) {
-        if (
-          rel.targetMode !== "External" &&
-          !remainingIds.has(rel.id) &&
-          /\.(?:svg|wdp)(?:$|[?#])/i.test(rel.target ?? "")
-        ) {
-          removedIds.add(rel.id);
-        }
-      }
-    }
-
-    if (after === before && removedIds.size === 0) continue;
-
-    if (after !== before) {
-      files[part] = new TextEncoder().encode(after);
-      changed = true;
-    }
-    if (!relsXml) continue;
-
-    let nextRels = relsXml;
-    for (const id of removedIds) {
-      if (remainingIds.has(id)) continue;
-      nextRels = removeRelationshipById(nextRels, id);
-      const rel = relsById.get(id);
-      if (rel && rel.targetMode !== "External") {
-        removedTargets.add(resolvePartPath(partDir, rel.target));
-      }
-    }
-    if (nextRels !== relsXml) {
-      files[relsPath] = new TextEncoder().encode(nextRels);
-    }
-  }
-
-  const deleted = deleteUnreferencedTargets(files, removedTargets);
-  if (deleted.size > 0) {
-    removeContentTypeOverrides(files, deleted);
-    changed = true;
-  }
-  if (removeUnusedMediaDefaults(files, ["svg", "wdp"])) {
-    changed = true;
-  }
-
-  return changed;
-}
-
-function deleteUnreferencedTargets(files, removedTargets) {
-  const remainingRefs = referencedInternalTargets(files);
-  const deleted = new Set();
-  for (const part of removedTargets) {
-    if (!part || remainingRefs.has(part) || files[part] === undefined) continue;
-    delete files[part];
-    const sidecar = relsPathFor(part);
-    if (files[sidecar] !== undefined) delete files[sidecar];
-    deleted.add(part);
-  }
-  return deleted;
-}
-
-function referencedInternalTargets(files) {
-  const refs = new Set();
-  for (const relsPath of Object.keys(files)) {
-    if (!relsPath.endsWith(".rels")) continue;
-    const dir = relsPath.replace(/\/?_rels\/[^/]+$/, "");
-    const owner =
-      dir === ""
-        ? relsPath.replace(/^_rels\/(.*)\.rels$/, "$1")
-        : `${dir}/${relsPath.replace(/^.*_rels\/(.*)\.rels$/, "$1")}`;
-    for (const rel of relsFor(files, owner)) {
-      if (rel.targetMode === "External") continue;
-      refs.add(resolvePartPath(dir, rel.target));
-    }
-  }
-  return refs;
-}
-
-function removeContentTypeOverrides(files, deletedParts) {
-  const before = readText(files, CONTENT_TYPES);
-  if (!before) return;
-  let after = before;
-  for (const part of deletedParts) {
-    after = after.replace(
-      new RegExp(
-        `<Override\\b(?=[^>]*\\bPartName="/${escapeRe(part)}")[^>]*?/>`,
-        "g",
-      ),
-      "",
-    );
-  }
-  if (after !== before) files[CONTENT_TYPES] = new TextEncoder().encode(after);
-}
-
-function removeUnusedMediaDefaults(files, exts) {
-  const before = readText(files, CONTENT_TYPES);
-  if (!before) return false;
-  let after = before;
-  for (const ext of exts) {
-    const hasPart = Object.keys(files).some((part) =>
-      part.toLowerCase().endsWith(`.${ext}`),
-    );
-    if (hasPart) continue;
-    after = after.replace(
-      new RegExp(`<Default\\b(?=[^>]*\\bExtension="${escapeRe(ext)}")[^>]*?/>`, "g"),
-      "",
-    );
-  }
-  if (after !== before) {
-    files[CONTENT_TYPES] = new TextEncoder().encode(after);
-    return true;
-  }
-  return false;
-}
-
-function relationshipIdsIn(xml) {
-  const ids = new Set();
-  for (const m of String(xml).matchAll(/\br:(?:id|embed|link)\s*=\s*"([^"]+)"/g)) {
-    ids.add(m[1]);
-  }
-  return ids;
-}
-
-function removeRelationshipById(xml, id) {
-  return xml.replace(
-    new RegExp(
-      `<Relationship\\b(?=[^>]*\\bId="${escapeRe(id)}")[^>]*?(?:/>|>[\\s\\S]*?</Relationship>)`,
-      "g",
-    ),
-    "",
-  );
-}
-
-const escapeRe = (s) => s.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");