@superdoc/mcp 1.0.1-next.6 → 1.0.1-next.8

package/dist/index.js

@@ -31314,7 +31314,7 @@ var init_host = __esm(() => {
 });
 
 // ../../packages/sdk/langs/node/src/runtime/sdk-version.generated.ts
-var SDK_VERSION = "2.13.1-next.6";
+var SDK_VERSION = "2.13.1-next.8";
 
 // ../../packages/sdk/langs/node/src/runtime/embedded-platform.ts
 import { chmodSync, existsSync } from "node:fs";
@@ -193399,9 +193399,10 @@ function runInsertInlineControlCommit(input) {
     preconditions,
     patchMaps,
     textRangeShifts: shifts,
-    ...changeMode === "direct" && op.control === "lineBreak" && "runPropertiesXml" in rewritten ? {
-      directLineBreak: {
-        kind: "directLineBreak",
+    ...changeMode === "direct" && (op.control === "tab" || op.control === "lineBreak") && "runPropertiesXml" in rewritten ? {
+      directInlineControl: {
+        kind: "directInlineControl",
+        control: op.control,
         afterParagraphTail: "afterParagraphTail" in rewritten && rewritten.afterParagraphTail === true,
         blockId: op.on.startBlockId,
         startOffset: op.on.startOffset,
@@ -193708,7 +193709,7 @@ function finalizeInlineControlCommit(args) {
     changeMode: op.changeMode ?? "direct",
     targetBlockIds: [op.on.startBlockId],
     atomCarrierContract: SD_DEFAULT_ATOM_CARRIER_CONTRACT,
-    ...args.directLineBreak ? { directLineBreak: args.directLineBreak } : {},
+    ...args.directInlineControl ? { directInlineControl: args.directInlineControl } : {},
     ...args.trackedTextInsert ? { trackedTextInsert: args.trackedTextInsert } : {},
     ...args.trackedTextEdit ? { trackedTextEdit: args.trackedTextEdit } : {}
   };
@@ -369135,6 +369136,7 @@ async function compilePlan(context, input) {
   }
   detectTargetConflicts(context, steps, failures);
   detectUnsupportedRuntimeDrift(input.steps, failures);
+  detectTargetOffsetDriftRisk(steps, failures);
   return { evaluatedRevision, steps, failures };
 }
 function isOrderedAtomicTextProgram(steps) {
@@ -369619,6 +369621,14 @@ function detectUnsupportedRuntimeDrift(steps, failures) {
           message: "multi-step mutation plans cannot use revision-bound ref targets; use selector targets so the engine can re-resolve each step before commit."
         });
       }
+      if (step3.where.by === "target" && step3.where.target.kind === "selection" && step3.where.target.start.kind === "text" && step3.where.target.end.kind === "text" && step3.where.target.start.blockId !== step3.where.target.end.blockId) {
+        failures.push({
+          code: "CAPABILITY_UNSUPPORTED",
+          stepId: step3.id,
+          phase: "compile",
+          message: 'multi-step mutation plans cannot combine a cross-block by:"target" step with other mutating steps; ' + "conflict and offset-drift detection only see that step's first block. Apply it alone, or split the " + "plan so the cross-block step runs in its own mutations.apply call."
+        });
+      }
     }
   }
 }
@@ -369659,6 +369669,33 @@ function detectTargetConflicts(context, steps, failures) {
     }
   }
 }
+function detectTargetOffsetDriftRisk(steps, failures) {
+  const textShiftingOps = new Set(["text.rewrite", "text.delete", "text.insert"]);
+  const byBlock = new Map;
+  for (const step3 of steps) {
+    if (!textShiftingOps.has(step3.op) || step3.source.op === "assert" || step3.source.where?.by !== "target")
+      continue;
+    const resolutions = step3.resolutions ?? (step3.resolution ? [step3.resolution] : []);
+    for (const resolution of resolutions) {
+      const key2 = `${storyKeyForTarget(resolution.target)}::${resolution.target.blockId}`;
+      const list2 = byBlock.get(key2) ?? [];
+      list2.push(step3);
+      byBlock.set(key2, list2);
+    }
+  }
+  for (const blockSteps of byBlock.values()) {
+    if (blockSteps.length <= 1)
+      continue;
+    for (const step3 of blockSteps) {
+      failures.push({
+        code: "CAPABILITY_UNSUPPORTED",
+        stepId: step3.stepId,
+        phase: "compile",
+        message: `step "${step3.stepId}" shares a block with another by:"target" text-shifting step in this plan; explicit ` + "offsets are not re-derived between steps, so an earlier same-block step can shift this one's offsets. " + 'Split same-block edits into separate mutations.apply calls, or use by:"select" for later occurrences in ' + "the same block."
+      });
+    }
+  }
+}
 function conflictRangeForResolution(context, step3, resolution) {
   if (step3.source.op !== "format.apply" || step3.source.args.scope !== "block" && step3.source.args.alignment === undefined) {
     return resolution.range;
@@ -369713,9 +369750,117 @@ async function resolveMutationStep(context, step3, story, changeMode) {
     const compiled = resolveRefMutationStep(context, step3, story, matches[0].ref);
     return applySelectMatchReplacementPolicy(compiled, step3, matches[0].groups);
   }
+  if (step3.where.by === "target") {
+    return resolveTargetWhereMutationStep(context, step3, step3.where, story, changeMode);
+  }
   const ref5 = resolveStepRef(step3);
   return resolveRefMutationStep(context, step3, story, ref5);
 }
+function resolveTargetWhereMutationStep(context, step3, where, story, changeMode) {
+  const target = where.target;
+  if (target.kind !== "selection" || target.start.kind !== "text" || target.end.kind !== "text") {
+    throw new DocumentApiValidationError2("INVALID_TARGET", `v2 ${step3.op} target steps require a text SelectionTarget with text-kind endpoints.`);
+  }
+  const storyLocator = target.story ?? story;
+  if (!isSupportedMutationStoryScope(storyLocator)) {
+    throw new DocumentApiValidationError2("CAPABILITY_UNSUPPORTED", `v2 mutation plan engine supports only the body story; "${storyLocator?.storyType}" story targets are not supported.`);
+  }
+  if (step3.op === "text.rewrite" && step3.args.replacementMode === "regex-template") {
+    throw new DocumentApiValidationError2("CAPABILITY_UNSUPPORTED", 'v2 text.rewrite target steps do not support replacementMode "regex-template"; an explicit target carries no ' + 'capture groups. Use a by:"select" step for regex-template replacements.');
+  }
+  const selectionMutation = new V2SelectionMutationAdapter(context);
+  if (step3.op === "text.insert") {
+    const { text: text4, position: position2 } = readInsertArgs(step3);
+    const point3 = position2 === "before" ? target.start : target.end;
+    const collapsed = {
+      kind: "selection",
+      start: point3,
+      end: point3,
+      ...target.story ? { story: target.story } : {},
+      ...target.coordinateSpace ? { coordinateSpace: target.coordinateSpace } : {}
+    };
+    const request = { kind: "insert", target: collapsed, text: text4, ...story ? { in: story } : {} };
+    const receipt3 = selectionMutation.execute({ kind: "insert", target: collapsed, text: text4, ...story ? { in: story } : {} }, { changeMode, dryRun: true });
+    if (!receipt3.success)
+      throw new DocumentApiValidationError2(receipt3.failure.code, receipt3.failure.message);
+    const resolution2 = {
+      target: receipt3.resolution.target,
+      range: receipt3.resolution.range,
+      text: receipt3.resolution.text
+    };
+    return {
+      stepId: step3.id,
+      op: "text.insert",
+      resolution: resolution2,
+      request,
+      noopEquivalentText: text4.length === 0 ? text4 : undefined,
+      source: step3
+    };
+  }
+  if (step3.op === "format.apply") {
+    const crossBlock = target.start.blockId !== target.end.blockId;
+    if (crossBlock && (step3.args.alignment !== undefined || step3.args.scope === "block")) {
+      throw new DocumentApiValidationError2("CAPABILITY_UNSUPPORTED", "v2 format.apply target steps spanning more than one block support only an inline patch with the default " + '(non-block) scope; split a cross-block alignment or scope:"block" edit into one step per block.');
+    }
+    const placeholderResolution = {
+      target: {
+        kind: "text",
+        blockId: target.start.blockId,
+        range: { start: 0, end: 0 },
+        ...storyLocator ? { story: storyLocator } : {}
+      },
+      range: { from: 0, to: 0 },
+      text: ""
+    };
+    const request = buildFormatCommitRequest(context, step3, placeholderResolution, target, story, storyLocator);
+    if (request.kind !== "format") {
+      throw new DocumentApiValidationError2("INVALID_TARGET", "v2 format.apply target steps require a resolvable target.");
+    }
+    let resolution2;
+    if (request.inline) {
+      const receipt3 = "target" in request ? selectionMutation.execute({ kind: "format", target: request.target, inline: request.inline, ...request.in ? { in: request.in } : {} }, { changeMode, dryRun: true, ...request.skipAtomicObjects ? { skipAtomicObjects: true } : {} }) : selectionMutation.execute({ kind: "format", ref: request.ref, inline: request.inline, ...request.in ? { in: request.in } : {} }, { changeMode, dryRun: true, ...request.skipAtomicObjects ? { skipAtomicObjects: true } : {} });
+      if (!receipt3.success)
+        throw new DocumentApiValidationError2(receipt3.failure.code, receipt3.failure.message);
+      resolution2 = {
+        target: receipt3.resolution.target,
+        range: receipt3.resolution.range,
+        text: receipt3.resolution.text
+      };
+    } else {
+      const blockText = readBlockTargetText(context, target.start.blockId, Number.MAX_SAFE_INTEGER, storyLocator, target.coordinateSpace);
+      resolution2 = {
+        target: { kind: "text", blockId: target.start.blockId, range: { start: 0, end: blockText.length }, ...storyLocator ? { story: storyLocator } : {} },
+        range: { from: 0, to: blockText.length },
+        text: blockText
+      };
+    }
+    return {
+      stepId: step3.id,
+      op: "format.apply",
+      resolution: resolution2,
+      request,
+      source: step3
+    };
+  }
+  const rewriteTemplate = step3.op === "text.delete" ? "" : normalizeReplacement(step3.args.replacement);
+  const probeReceipt = selectionMutation.execute({ kind: "replace", target, text: rewriteTemplate, ...story ? { in: story } : {} }, { changeMode, dryRun: true });
+  if (!probeReceipt.success)
+    throw new DocumentApiValidationError2(probeReceipt.failure.code, probeReceipt.failure.message);
+  const resolution = {
+    target: probeReceipt.resolution.target,
+    range: probeReceipt.resolution.range,
+    text: probeReceipt.resolution.text
+  };
+  const replacementText = step3.op === "text.delete" ? "" : replacementTextForMatch(step3, resolution.text);
+  return {
+    stepId: step3.id,
+    op: step3.op,
+    resolution,
+    request: { kind: "replace", target, text: replacementText, ...story ? { in: story } : {} },
+    ...step3.op === "text.rewrite" ? { noopEquivalentText: replacementText } : {},
+    source: step3
+  };
+}
 function resolveRefMutationStep(context, step3, story, ref5, cache) {
   if (isV2RangeRef(ref5)) {
     return resolveRangeRefMutationStep(context, step3, story, ref5);
@@ -424205,8 +424350,12 @@ function tryMirrorStoryText(prepared, extension2, options, admitted) {
   const validation = admitted ?? validateDirectStoryText(prepared, extension2, options);
   if (!validation.ok)
     return { mirrored: false, failureReason: validation.failureReason };
-  const source = validation.source.kind === "insertInlineControl" ? { changeMode: "direct", on: validation.source.on, newText: `
-` } : validation.source;
+  const source = validation.source.kind === "insertInlineControl" ? {
+    changeMode: "direct",
+    on: validation.source.on,
+    newText: validation.inlineControl?.control === "tab" ? "\t" : `
+`
+  } : validation.source;
   const target = validation.target;
   try {
     target.scope.doc.transact(() => {
@@ -424275,7 +424424,7 @@ function tryMirrorStoryText(prepared, extension2, options, admitted) {
         });
       }
       if (source.newText.length > 0) {
-        if (deleteLength === 0 || validation.lineBreak) {
+        if (deleteLength === 0 || validation.inlineControl) {
           if (!validation.directInsert) {
             throw new Error("direct storyText mirror missing typed insert metadata");
           }
@@ -424285,7 +424434,7 @@ function tryMirrorStoryText(prepared, extension2, options, admitted) {
             text: source.newText,
             txId: prepared.txId,
             directInsert: validation.directInsert,
-            preserveZeroLengthAtOffset: validation.lineBreak?.afterParagraphTail,
+            preserveZeroLengthAtOffset: validation.inlineControl?.afterParagraphTail,
             acceptedTextInsertionSplit: validation.acceptedTextInsertionSplit
           });
         } else {
@@ -424405,10 +424554,10 @@ function cloneTrackedWrapperAtom(source, id2, visibleOffset) {
 }
 function validateDirectStoryText(prepared, extension2, options) {
   const source = prepared.source;
-  const lineBreak = source.kind === "insertInlineControl" ? extension2.directLineBreak : undefined;
+  const inlineControl = source.kind === "insertInlineControl" ? extension2.directInlineControl : undefined;
   if (source.kind === "insertInlineControl") {
-    if (source.control !== "lineBreak" || (source.changeMode ?? "direct") !== "direct" || !lineBreak || lineBreak.kind !== "directLineBreak" || typeof lineBreak.afterParagraphTail !== "boolean" || lineBreak.blockId !== source.on.startBlockId || lineBreak.startOffset !== source.on.startOffset || lineBreak.endOffset !== source.on.endOffset || parsePreparedRunAttributes(lineBreak.runPropertiesXml) === null) {
-      return { ok: false, failureReason: "storyText requires matching direct line-break metadata" };
+    if (source.control !== "tab" && source.control !== "lineBreak" || (source.changeMode ?? "direct") !== "direct" || !inlineControl || inlineControl.kind !== "directInlineControl" || inlineControl.control !== source.control || typeof inlineControl.afterParagraphTail !== "boolean" || inlineControl.blockId !== source.on.startBlockId || inlineControl.startOffset !== source.on.startOffset || inlineControl.endOffset !== source.on.endOffset || parsePreparedRunAttributes(inlineControl.runPropertiesXml) === null) {
+      return { ok: false, failureReason: "storyText requires matching direct inline-control metadata" };
     }
   }
   if (source.kind !== "editText" && source.kind !== "insertInlineControl") {
@@ -424456,15 +424605,15 @@ function validateDirectStoryText(prepared, extension2, options) {
   if (trackedEdit && !trackedEdit.ok) {
     return { ok: false, failureReason: trackedEdit.failureReason };
   }
-  const directInsert = lineBreak ? {
+  const directInsert = inlineControl ? {
     ok: true,
     directInsert: {
       kind: "directTextInsert",
-      blockId: lineBreak.blockId,
-      startOffset: lineBreak.startOffset,
-      text: `
+      blockId: inlineControl.blockId,
+      startOffset: inlineControl.startOffset,
+      text: inlineControl.control === "tab" ? "\t" : `
 `,
-      runPropertiesXml: lineBreak.runPropertiesXml
+      runPropertiesXml: inlineControl.runPropertiesXml
     }
   } : source.kind === "editText" && source.changeMode === "direct" && isPureDirectTextInsert(source) ? validateDirectTextInsert(source, extension2) : null;
   if (directInsert && !directInsert.ok) {
@@ -424502,7 +424651,7 @@ function validateDirectStoryText(prepared, extension2, options) {
     };
   }
   const target = resolveStoryTextMutationTarget(doc2, source.on.startBlockId);
-  if (lineBreak && !lineBreak.afterParagraphTail && hasStoryTextBoundaryInRange(target, source.on.startOffset, source.on.endOffset)) {
+  if (inlineControl?.control === "lineBreak" && !inlineControl.afterParagraphTail && hasStoryTextBoundaryInRange(target, source.on.startOffset, source.on.endOffset)) {
     return { ok: false, failureReason: "line-break boundary placement requires typed collaboration support" };
   }
   let validatedTrackedRefinement;
@@ -424535,7 +424684,7 @@ function validateDirectStoryText(prepared, extension2, options) {
     target,
     ...acceptedTextInsertionSplit ? { acceptedTextInsertionSplit } : {},
     ...directInsert?.ok ? { directInsert: directInsert.directInsert } : {},
-    ...lineBreak ? { lineBreak: { afterParagraphTail: lineBreak.afterParagraphTail } } : {},
+    ...inlineControl ? { inlineControl: { control: inlineControl.control, afterParagraphTail: inlineControl.afterParagraphTail } } : {},
     ...trackedInsert?.ok ? { trackedInsert: trackedInsert.trackedInsert } : {},
     ...trackedInsertContinuation ? { trackedInsertContinuation: true } : {},
     ...validatedTrackedRefinement ? { trackedRefinement: validatedTrackedRefinement } : {},

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@superdoc/mcp",
-  "version": "1.0.1-next.6",
+  "version": "1.0.1-next.8",
   "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.13.1-next.6",
+    "@superdoc/sdk": "2.13.1-next.8",
     "debug": "^4.4.3",
     "zod": "^4.3.6"
   },
   "devDependencies": {
-    "superdoc": "2.16.1-next.6",
-    "@superdoc/docx-engine": "0.15.1-next.5",
+    "superdoc": "2.16.1-next.8",
+    "@superdoc/docx-engine": "0.15.1-next.8",
     "@superdoc/document-api": "0.1.0-alpha.0",
     "@types/bun": "^1.3.8",
     "@types/node": "22.19.2",