@superdoc/mcp 1.7.1-next.4 → 1.7.1-next.6

package/dist/index.js

@@ -31600,7 +31600,7 @@ var init_host = __esm(() => {
 });
 
 // ../../packages/sdk/langs/node/src/runtime/sdk-version.generated.ts
-var SDK_VERSION = "2.20.1-next.4";
+var SDK_VERSION = "2.20.1-next.6";
 
 // ../../packages/sdk/langs/node/src/runtime/embedded-platform.ts
 import { accessSync, chmodSync, constants, existsSync, statSync } from "node:fs";
@@ -138920,6 +138920,9 @@ function validateRunContents(xml, options) {
   }
   if (runs === "")
     return { ok: true };
+  if (/^[\t\n\r ]+$/.test(runs)) {
+    return { ok: false, reason: "target runs contain only XML whitespace; clearing this representation is not supported" };
+  }
   const replayXml = stripAllowedNoteReferences(runs, options);
   const withoutCarriers = replayXml !== null && options.baseRunXml ? stripRetainedCarrier(replayXml, options.baseRunXml, options.allowNamespaceDeclarationDifference === true) : replayXml;
   const withoutBookmarks = withoutCarriers !== null && /<(?:[\w.-]+:)?bookmark(?:Start|End)\b/.test(withoutCarriers) ? preservedBookmarkRunTokens(withoutCarriers)?.strippedXml ?? null : withoutCarriers;
@@ -388423,15 +388426,27 @@ function projectTrackedChanges(context) {
       });
     }
   }
+  const capturedFormattingKeys = new Set;
   for (const partUri of sortedParts(catalog.formattingByPart)) {
     const fragments = catalog.formattingByPart.get(partUri) ?? [];
     for (const f of [...fragments].sort((a, b) => a.byteStart - b.byteStart)) {
+      if (capturedFormattingKeys.has(f.entityKey))
+        continue;
+      capturedFormattingKeys.add(f.entityKey);
       const author = f.author?.name ?? "";
+      const group = [...catalog.formattingByKey.get(f.entityKey) ?? [f]].sort((a, b) => a.partUri.localeCompare(b.partUri) || a.byteStart - b.byteStart);
+      const contentSignature = group.length === 1 ? digest3(`subtype:${f.subtype}#author:${author}#date:${f.date}#before:${f.beforeInnerXml}#after:${f.afterInnerXml}`) : digest3(JSON.stringify(group.map((member) => [
+        member.subtype,
+        member.author?.name ?? "",
+        member.date,
+        member.beforeInnerXml,
+        member.afterInnerXml
+      ])));
       entries.push({
         key: f.entityKey,
         stableKey: f.entityKey,
         semanticKey: `format:${f.subtype}|${author}`,
-        contentSignature: digest3(`subtype:${f.subtype}#author:${author}#date:${f.date}#before:${f.beforeInnerXml}#after:${f.afterInnerXml}`),
+        contentSignature,
         ordinal: ordinal++,
         label: `formatting revision ${f.subtype}`
       });
@@ -389244,6 +389259,29 @@ function requiredPartBytes(context, partUri) {
     throw new SDCompareError("SOURCE_NOT_COMPLETE", `compare capture could not read required part ${partUri}.`);
   return bytes;
 }
+function assertInspectableTablePlacement(context) {
+  const partUris = new Set(listSDDocumentSessionInternalStories(context.session).map((story) => story.partUri));
+  for (const partUri of partUris) {
+    const parents = [];
+    let inlineContainer;
+    walkXmlElements(new TextDecoder().decode(requiredPartBytes(context, partUri)), {
+      onOpenTag({ tag }) {
+        const parent = parents.at(-1);
+        if (tag.namespaceUri === WORDPROCESSINGML_NAMESPACE8 && tag.localName === "tbl" && parent?.namespaceUri === WORDPROCESSINGML_NAMESPACE8 && (parent.localName === "p" || parent.localName === "r"))
+          inlineContainer = parent.localName === "p" ? "paragraph" : "run";
+        if (!tag.isSelfClosing)
+          parents.push(tag);
+      },
+      onCloseTag({ tag }) {
+        if (!tag.isSelfClosing)
+          parents.pop();
+      }
+    });
+    if (inlineContainer) {
+      throw new SDCompareError("INVALID_INPUT", `compare capture found an uninspectable WordprocessingML table inside a ${inlineContainer} in ${partUri}. ` + "Tables must be block content; no comparison snapshot was produced.");
+    }
+  }
+}
 function contentTypeRegistrationForPart(context, partUri) {
   const registry2 = parseContentTypes(new TextDecoder().decode(requiredPartBytes(context, CONTENT_TYPES_URI)));
   if (registry2.parseError)
@@ -389530,6 +389568,7 @@ function captureCompareSnapshotSync(context, options = {}) {
   return projectCompareSnapshot(context, "source-complete", options);
 }
 function projectCompareSnapshot(context, posture, options) {
+  assertInspectableTablePlacement(context);
   const body = projectBody(context);
   const slots = projectSlots(context);
   const { stories, noteParts } = projectStories(context, slots);
@@ -391527,6 +391566,13 @@ function buildCommentApplyPlan(base3, target) {
 }
 function buildExactCommentApplyPlan(base3, target, baseSource, targetSource) {
   const fallback = buildCommentApplyPlan(base3, target);
+  if (fallback.steps.filter((step3) => step3.kind === "create").length > 1) {
+    return {
+      steps: [],
+      partRewrites: [],
+      unsafeReasons: ["multiple comment creations require revision-stable anchor replay"]
+    };
+  }
   const baseById = new Map(base3.map((record3) => [record3.commentId, record3]));
   const targetById = new Map(target.map((record3) => [record3.commentId, record3]));
   const topologyCompatible = base3.length === target.length && base3.every((record3) => {
@@ -395279,7 +395325,11 @@ class V2DiffAdapter {
       base: payload.base.definitions,
       target: payload.target.definitions
     }), canonicalBodyOperations);
-    const trackedTablePropertyPlan = tablesChanged && reviewMarkup ? buildTrackedTablePropertyReplayPlan(liveBaseMainDocument, payloadMainDocument.target) : null;
+    const tablePropertyPlan = tablesChanged ? buildTrackedTablePropertyReplayPlan(liveBaseMainDocument, payloadMainDocument.target) : null;
+    if (tablePropertyPlan && this.context.isCollaborativeSession()) {
+      throw diffValidationError("CAPABILITY_UNSUPPORTED", "diff.apply: table property replay is unavailable in a joined collaboration session; use a standalone session for this input. No mutation was performed.", { changedFamilies: ["tables"], unsupportedReason: "collaborative-table-property-replay-unsupported" });
+    }
+    const trackedTablePropertyPlan = reviewMarkup ? tablePropertyPlan : null;
     let trackedTablePlan = tablesChanged && reviewMarkup && !trackedTablePropertyPlan ? buildTrackedTableRowReplayPlan({
       base: liveBaseMainDocument,
       target: payloadMainDocument.target,
@@ -395750,6 +395800,7 @@ function buildCompareApplyEligibility(context, payload, plans, styleChangeMode)
   }
   const tablesChanged = payload.familyPolicy.some((policy) => policy.family === "tables" && policy.applyRequired);
   let wholeMainDirectReplay = false;
+  let tablePropertyReplayRequiresStandalone = false;
   if (tablesChanged) {
     const bodyDeferredToTableReplay = payload.familyPolicy.some((policy) => policy.family === "body" && policy.applyRequired && policy.disposition === "deferred");
     const mainDocument = payload.mainDocument ?? { base: EMPTY_MAIN_DOCUMENT_SOURCE, target: EMPTY_MAIN_DOCUMENT_SOURCE };
@@ -395781,7 +395832,13 @@ function buildCompareApplyEligibility(context, payload, plans, styleChangeMode)
       bodyOperations: trackedBodyOperations,
       allowTopologyReplacement: true
     }) : null;
-    if (trackedTablePlan.unsafeReasons.length === 0 && trackedTablePlan.steps.length > 0) {
+    const trackedPropertyPlan = buildTrackedTablePropertyReplayPlan(mainDocument.base, mainDocument.target);
+    tablePropertyReplayRequiresStandalone = trackedPropertyPlan !== null && context.isCollaborativeSession();
+    if (trackedPropertyPlan) {
+      tracked.add("tables");
+      if (bodyDeferredToTableReplay)
+        tracked.add("body");
+    } else if (trackedTablePlan.unsafeReasons.length === 0 && trackedTablePlan.steps.length > 0) {
       tracked.add("tables");
       if (bodyDeferredToTableReplay)
         tracked.add("body");
@@ -395802,6 +395859,15 @@ function buildCompareApplyEligibility(context, payload, plans, styleChangeMode)
   if (metadataPartPlan.rewrites.length > 0)
     direct.add("package-graph");
   const hazards = { direct: [], tracked: [] };
+  if (tablePropertyReplayRequiresStandalone) {
+    const hazard = {
+      code: "family-apply-lane-unavailable",
+      message: "Table property replay is unavailable in a joined collaboration session; use a standalone session for this input.",
+      families: ["tables"]
+    };
+    hazards.direct.push(hazard);
+    hazards.tracked.push(hazard);
+  }
   if (metadataPartPlan.unsafeReasons.length > 0) {
     hazards.direct.push({
       code: "family-apply-lane-unavailable",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@superdoc/mcp",
-  "version": "1.7.1-next.4",
+  "version": "1.7.1-next.6",
   "description": "Model Context Protocol server for SuperDoc, giving an AI agent tools to read and edit .docx files.",
   "type": "module",
   "license": "AGPL-3.0",
@@ -24,13 +24,13 @@
   ],
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.26.0",
-    "@superdoc/sdk": "2.20.1-next.4",
+    "@superdoc/sdk": "2.20.1-next.6",
     "debug": "^4.4.3",
     "zod": "^4.3.6"
   },
   "devDependencies": {
-    "superdoc": "2.23.1-next.4",
-    "@superdoc/docx-engine": "0.22.1-next.4",
+    "superdoc": "2.23.1-next.6",
+    "@superdoc/docx-engine": "0.22.1-next.6",
     "@superdoc/document-api": "0.1.0-alpha.0",
     "@types/bun": "^1.3.8",
     "@types/node": "22.19.2",