@superdoc/mcp 1.7.1-next.5 → 1.7.1-next.7

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.5";
+var SDK_VERSION = "2.20.1-next.7";
 
 // ../../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;
@@ -205791,6 +205794,7 @@ function parsePlainRun(runXml, textStart, byteStart, byteEnd, options = {}) {
     runClose
   });
   const childRe = /<([\w.-]+):([\w.-]+)\b([^>]*)>/g;
+  const pairedNoBreakHyphenCloseRe = /\s*<\/([\w.-]+):noBreakHyphen>/y;
   childRe.lastIndex = runOpen.length;
   let tMatch = null;
   let controlKind = null;
@@ -205835,7 +205839,18 @@ function parsePlainRun(runXml, textStart, byteStart, byteEnd, options = {}) {
       childRe.lastIndex = closeMatch.index + closeMatch[0].length;
       continue;
     }
-    if (isSelfClose && (local === "tab" || local === "br" || local === "cr")) {
+    let supportedControlXml = m[0];
+    let isPairedNoBreakHyphen = false;
+    if (local === "noBreakHyphen" && !isSelfClose) {
+      pairedNoBreakHyphenCloseRe.lastIndex = childRe.lastIndex;
+      const close = pairedNoBreakHyphenCloseRe.exec(without);
+      if (!close || close[1] !== ns)
+        return fail4("invalid-range");
+      childRe.lastIndex = pairedNoBreakHyphenCloseRe.lastIndex;
+      supportedControlXml += close[0];
+      isPairedNoBreakHyphen = true;
+    }
+    if ((isSelfClose || isPairedNoBreakHyphen) && (local === "tab" || local === "br" || local === "cr" || local === "noBreakHyphen")) {
       if (!options.allowReplaceableInlineControls) {
         if (!tMatch && !controlKind) {
           const trailing = matchSingleTrailingTextRun(without, childRe.lastIndex, without.length - runClose.length);
@@ -205853,7 +205868,7 @@ function parsePlainRun(runXml, textStart, byteStart, byteEnd, options = {}) {
                 text: text2,
                 rPr,
                 runOpen,
-                leadingControlXml: m[0],
+                leadingControlXml: supportedControlXml,
                 runClose,
                 ...leadingTransparentXml ? { leadingTransparentXml } : {}
               }
@@ -205867,7 +205882,7 @@ function parsePlainRun(runXml, textStart, byteStart, byteEnd, options = {}) {
       if (tMatch || controlKind)
         return fail4("invalid-range");
       controlKind = local;
-      controlXml = m[0];
+      controlXml = supportedControlXml;
       continue;
     }
     if (local === "drawing" || local === "pict" || local === "object") {
@@ -206014,7 +206029,7 @@ function rewriteRunSpanText(paragraphXml, startOffset, endOffset, newText) {
   if (collected.runs.length === 0) {
     return rewriteEmptyParagraphInsert(paragraphXml, newText, startOffset, endOffset);
   }
-  const boundaryInsert = rewriteCollapsedBoundaryAfterAnchorGapInsert(paragraphXml, collected.runs, startOffset, endOffset, (carrier) => newText.length > 0 ? renderTextRun(newText, carrier.runOpen, carrier.runClose, carrier.rPr) : "");
+  const boundaryInsert = rewriteCollapsedRunBoundaryInsert(paragraphXml, collected.runs, startOffset, endOffset, (carrier) => newText.length > 0 ? renderTextRun(newText, carrier.runOpen, carrier.runClose, carrier.rPr) : "");
   if (boundaryInsert)
     return { ok: true, xml: boundaryInsert };
   const tailInsert = rewriteCollapsedParagraphTailInsert(paragraphXml, collected.runs, startOffset, endOffset, (carrier) => newText.length > 0 ? renderTextRun(newText, carrier.runOpen, carrier.runClose, carrier.rPr) : "");
@@ -206154,7 +206169,7 @@ function rewriteRunSpanTrackedReplace(paragraphXml, startOffset, endOffset, stam
   if (!paragraphMarkFallback.ok)
     return paragraphMarkFallback;
   const carrierRPr = (carrier) => carrier.rPr || paragraphMarkFallback.rPr;
-  const boundaryInsert = rewriteCollapsedBoundaryAfterAnchorGapInsert(paragraphXml, collected.runs, startOffset, endOffset, (carrier) => stamp.newText.length > 0 ? buildInsWrapper(stamp.newText, carrier.runOpen, carrier.runClose, carrierRPr(carrier), {
+  const boundaryInsert = rewriteCollapsedRunBoundaryInsert(paragraphXml, collected.runs, startOffset, endOffset, (carrier) => stamp.newText.length > 0 ? buildInsWrapper(stamp.newText, carrier.runOpen, carrier.runClose, carrierRPr(carrier), {
     rsid: stamp.insRsid,
     wId: stamp.wId,
     author: stamp.author,
@@ -206322,7 +206337,7 @@ function rewriteRunSpanTrackedReplacePreservingMarkers(paragraphXml, span, stamp
 function visibleTextLength(xml) {
   let len = 0;
   const visibleXml = stripElements(stripElements(xml, "del"), "moveFrom");
-  for (const match of visibleXml.matchAll(/<[\w.-]+:t\b[^>]*>([\s\S]*?)<\/[\w.-]+:t>|<[\w.-]+:(tab|br|cr|footnoteReference|endnoteReference)\b[^>]*\/>/g)) {
+  for (const match of visibleXml.matchAll(/<[\w.-]+:t\b[^>]*>([\s\S]*?)<\/[\w.-]+:t>|<[\w.-]+:(tab|br|cr|noBreakHyphen|footnoteReference|endnoteReference)\b[^>]*\/>|<[\w.-]+:noBreakHyphen\b[^>]*>\s*<\/[\w.-]+:noBreakHyphen>/g)) {
     if (match[1] != null) {
       len += decodeXmlText6(match[1]).length;
       continue;
@@ -206655,9 +206670,23 @@ function preserveTextElementAttributes(attributes, text) {
   }
   return `${attributes} xml:space="preserve"`;
 }
-function rewriteCollapsedBoundaryAfterAnchorGapInsert(paragraphXml, runs, startOffset, endOffset, buildContent) {
+function rewriteCollapsedRunBoundaryInsert(paragraphXml, runs, startOffset, endOffset, buildContent) {
   if (startOffset !== endOffset || runs.length === 0)
     return null;
+  const first = runs[0];
+  if (startOffset === 0) {
+    let beforeControl = first.leadingControlXml != null && first.textStart === 1;
+    if (first.kind === "control" && first.textStart === 0) {
+      const parsed = parsePlainRun(first.xml, 0, first.byteStart, first.byteEnd, {
+        allowReplaceableInlineControls: true
+      });
+      beforeControl = parsed.ok && parsed.run.controlKind != null;
+    }
+    if (beforeControl) {
+      const content = buildContent(first);
+      return `${paragraphXml.slice(0, first.byteStart)}${content}${paragraphXml.slice(first.byteStart)}`;
+    }
+  }
   const closeTagMatch = paragraphXml.match(/<\/[\w.-]+:p>\s*$/);
   if (!closeTagMatch || closeTagMatch.index == null)
     return null;
@@ -214994,6 +215023,12 @@ function runTrackedChangeDecideCommitWithLookupContext(lookupContext, input, xml
       const existingPatch = firstReplacePatch(perPart[partUri]);
       const workingBytes = existingPatch && existingPatch.kind === "replaceBytes" ? existingPatch.bytes : baseBytes;
       let workingXml = TD52.decode(workingBytes);
+      let commentFreeMarkerIndex;
+      const currentCommentMarkerIndex = () => {
+        if (/commentRangeStart|commentRangeEnd|commentReference/.test(workingXml))
+          return;
+        return commentFreeMarkerIndex ?? (commentFreeMarkerIndex = buildCommentMarkerIndex(workingXml));
+      };
       const sortedGroups = [...groups].sort((left, right) => {
         const leftOwnership = left.fragments.some((fragment3) => fragment3.nativeSdtOwnership);
         const rightOwnership = right.fragments.some((fragment3) => fragment3.nativeSdtOwnership);
@@ -215088,7 +215123,8 @@ function runTrackedChangeDecideCommitWithLookupContext(lookupContext, input, xml
               structuralHyperlinkRemovalParts.add(partUri);
             }
             const classification = classifyDroppedCommentMarkers(workingXml, result.deletedRange, {
-              preserveAdjacentEndBoundaryWIds
+              preserveAdjacentEndBoundaryWIds,
+              markerIndex: currentCommentMarkerIndex()
             });
             for (const wId of classification.delete) {
               cascadeCommentWIds.add(wId);
@@ -389256,6 +389292,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)
@@ -389542,6 +389601,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);
@@ -391539,6 +391599,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) => {
@@ -395291,7 +395358,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,
@@ -395762,6 +395833,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 };
@@ -395793,7 +395865,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");
@@ -395814,6 +395892,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.5",
+  "version": "1.7.1-next.7",
   "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.5",
+    "@superdoc/sdk": "2.20.1-next.7",
     "debug": "^4.4.3",
     "zod": "^4.3.6"
   },
   "devDependencies": {
-    "superdoc": "2.23.1-next.5",
-    "@superdoc/docx-engine": "0.22.1-next.5",
+    "superdoc": "2.23.1-next.7",
+    "@superdoc/docx-engine": "0.22.1-next.7",
     "@superdoc/document-api": "0.1.0-alpha.0",
     "@types/bun": "^1.3.8",
     "@types/node": "22.19.2",