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

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.8";
+var SDK_VERSION = "2.13.1-next.9";
 
 // ../../packages/sdk/langs/node/src/runtime/embedded-platform.ts
 import { chmodSync, existsSync } from "node:fs";
@@ -81329,7 +81329,6 @@ When formatting newly created content, use the right source:
 - **Do NOT hardcode formatting values.** Always read from blocks data and replicate.
 - **Do NOT copy heading/title formatting onto body paragraphs.** Read from body text blocks (alignment "justify" or "left"), not title blocks.
 - **Pass structured objects, not JSON-encoded strings.** Fields like \`at\`, \`target\`, and \`inline\` expect objects, not serialized JSON strings.
-- **Only pass \`dryRun\` when the action's schema explicitly lists it.** Do not assume every action accepts it. Prefer a real call over a preview for destructive actions unless dryRun is documented for that action.
 - **If blocks still report \`underline: true\` after you explicitly removed it, treat it as a style inheritance artifact.** Do not retry formatting to fix it.
 - **On "Unknown field" errors, drop the unrecognized field and retry.** Use the narrowest working call shape rather than guessing alternative field names.
 - **Table styling crosses two tools.** Borders / shading / cnf flags / spacing are on \`superdoc_table\`; cell-text alignment and font color/weight are on \`superdoc_format\` (paragraph- and run-level). A "style the whole table" pass calls both. See the Tables: cross-tool workflows section for the full recipe.
@@ -81815,7 +81814,6 @@ When formatting newly created content, use the right source:
 - **Do NOT hardcode formatting values.** Always read from blocks data and replicate.
 - **Do NOT copy heading/title formatting onto body paragraphs.** Read from body text blocks (alignment "justify" or "left"), not title blocks.
 - **Pass structured objects, not JSON-encoded strings.** Fields like \`at\`, \`target\`, and \`inline\` expect objects, not serialized JSON strings.
-- **Only pass \`dryRun\` when the action's schema explicitly lists it.** Do not assume every action accepts it. Prefer a real call over a preview for destructive actions unless dryRun is documented for that action.
 - **If blocks still report \`underline: true\` after you explicitly removed it, treat it as a style inheritance artifact.** Do not retry formatting to fix it.
 - **On "Unknown field" errors, drop the unrecognized field and retry.** Use the narrowest working call shape rather than guessing alternative field names.
 - **Table styling crosses two tools.** Borders / shading / cnf flags / spacing are on \`superdoc_table\`; cell-text alignment and font color/weight are on \`superdoc_format\` (paragraph- and run-level). A "style the whole table" pass calls both. See the Tables: cross-tool workflows section for the full recipe.
@@ -90178,17 +90176,14 @@ async function runInsertTableRow(doc2, args) {
     };
     if (args.changeMode)
       insertParams.changeMode = args.changeMode;
-    if (args.dryRun === true)
-      insertParams.dryRun = true;
     const insertResult = await insertFn(insertParams);
     const executed = [
       {
         operationId: "doc.tables.insertRow",
-        result: insertResult,
-        rationale: args.dryRun === true ? "Preview only; no document mutation applied." : undefined
+        result: insertResult
       }
     ];
-    if (args.dryRun !== true && args.cellTexts && args.cellTexts.length > 0) {
+    if (args.cellTexts && args.cellTexts.length > 0) {
       const insertedRowIndex = position === "below" ? rowIndex + 1 : rowIndex;
       const cells = args.cellTexts.map((cell, columnIndex) => cellTextFromSpec(cell, insertedRowIndex, columnIndex));
       const applied = await applyTableCellTexts(doc2, table.nodeId, table.ordinal, cells, args.changeMode);
@@ -90217,17 +90212,17 @@ async function runInsertTableRow(doc2, args) {
     const postIdentity = await readDocumentIdentity(doc2);
     const postTable = await getTableShape2(doc2, table.nodeId);
     const verification = [
-      revisionVerification(preIdentity.revision, postIdentity.revision, args.dryRun !== true)
+      revisionVerification(preIdentity.revision, postIdentity.revision, true)
     ];
     if (postTable) {
       verification.push({
         check: {
           kind: "table-shape",
           nodeId: table.nodeId,
-          rows: args.dryRun === true ? table.rows : table.rows + 1,
+          rows: table.rows + 1,
           columns: table.columns
         },
-        passed: postTable.rows === (args.dryRun === true ? table.rows : table.rows + 1) && postTable.columns === table.columns
+        passed: postTable.rows === table.rows + 1 && postTable.columns === table.columns
       });
     }
     return {
@@ -92297,8 +92292,7 @@ async function dispatchSuperdocPerformAction(doc2, args) {
         rowIndex: rowIndex != null && Number.isInteger(rowIndex) && rowIndex >= 0 ? rowIndex : undefined,
         position,
         cellTexts: cellTexts ?? undefined,
-        changeMode: parseChangeMode4(args.changeMode),
-        dryRun: args.dryRun === true
+        changeMode: parseChangeMode4(args.changeMode)
       });
     }
     case "insert_table_column": {
@@ -92964,10 +92958,10 @@ var init_actions = __esm(() => {
       trackedOverrideReason: "The V2 TOC adapter guarantees native tracked creation."
     },
     insert_table_row: {
-      hint: "tableOrdinal?, rowIndex?, position?: before|after|above|below, cellTexts? (one per column; each a plain string OR a STYLED {runs, marks?} cell to reproduce the table's cell formatting), changeMode?, dryRun? — to match existing cell styling, inspect the table with includeTableCellRuns:true first, then style each new cell HERE. PLACEMENT: when the user does not say WHERE the row goes, append it after the LAST existing row (rowIndex = last row's index, position: below); never pick a middle position on your own",
+      hint: "tableOrdinal?, rowIndex?, position?: before|after|above|below, cellTexts? (one per column; each a plain string OR a STYLED {runs, marks?} cell to reproduce the table's cell formatting), changeMode? — to match existing cell styling, inspect the table with includeTableCellRuns:true first, then style each new cell HERE. PLACEMENT: when the user does not say WHERE the row goes, append it after the LAST existing row (rowIndex = last row's index, position: below); never pick a middle position on your own",
       group: "Table edits",
       groupOrder: 3,
-      args: ["tableOrdinal", "rowIndex", "position", "cellTexts", "changeMode", "dryRun"],
+      args: ["tableOrdinal", "rowIndex", "position", "cellTexts", "changeMode"],
       operations: ["tables.insertRow"]
     },
     insert_table_column: {
@@ -93472,7 +93466,6 @@ var init_catalog = __esm(() => {
     rowIndex: { type: "number" },
     columnIndex: { type: "number" },
     position: { type: "string" },
-    dryRun: { type: "boolean" },
     headerText: { type: "string" },
     separatorText: { type: "string" },
     anchorText: { type: "string" },
@@ -94405,7 +94398,7 @@ ACTIONS (superdoc_perform_action with flat args)
 - move_table: tableOrdinal? (default 1), placement {at:"document_end"|"document_start"|"after"|"before", selector?}. THE way to move a whole table in ONE call — never delete-and-recreate or chain inserts/undos to relocate a table. changeMode:"tracked" records the move for review.
 - delete_table: tableOrdinal? (default 1), changeMode?. THE way to delete an entire table in ONE call — never delete rows one by one or reuse another table.
 - style_table: tableOrdinal? (default 1), accentColor? (header fill hex). ONE call makes a table look professional: accent header row with white bold text, bold first column, banded rows, clean borders. Use after create_table or on any existing table. changeMode:"tracked" records the formatting for review.
-- insert_table_row: tableOrdinal (1-based), rowIndex (0-based; omit to append), position:"above"|"below", optional cellTexts. dryRun:true for preview.
+- insert_table_row: tableOrdinal (1-based), rowIndex (0-based; omit to append), position:"above"|"below", optional cellTexts.
 - insert_table_column: tableOrdinal, columnIndex (0-based; omit to append right), position:"left"|"right", optional headerText.
 - delete_table_row / delete_table_column: rowIndex or columnIndex required, tableOrdinal optional.
 - split_table: tableOrdinal, rowIndex (>=1), optional separatorText.
@@ -94434,7 +94427,7 @@ OPERATING RULES
 - For anchored rewrites or multi-term edits, prefer textSearch selectors over copied nodeIds. For clause text inside tables, use a tableCell selector with replace_text.
 - For tab-indented headings, replace only the visible text with replace_text (rewrite_block can delete the tab node).
 - For BULK or PATTERN transforms (every percentage, every date), use ONE replace_text call with multiple edits. For bulk FORMATTING ("bold all the dates"): read the matching texts first, then ONE format_text call with targetTexts.
-- For preview-only requests ("show what it would look like", "don't save"), pass dryRun:true and make NO other mutating call; describe the preview from the receipt.
+- For preview-only requests ("show what it would look like", "don't save"), make NO mutating call — read the document with superdoc_inspect and describe the change in your answer. If they want to SEE it in the document before it is final, make the edit with changeMode:"tracked" so it lands as a reviewable suggestion they can accept or reject — but only on an action that records tracked changes; for an action marked Direct edit, describe the change instead of making it.
 - Pure count questions: superdoc_inspect countsOnly:true, then stop. Use includeDomains to keep snapshots small.
 - Do not include doc or sessionId in tool args. Never rely on benchmark routing, eval metadata, or fixture names.
 - If the runtime truly cannot express the request, say what is missing instead of faking success.
@@ -94507,9 +94500,16 @@ function validateAgentToolArgs(toolName, args) {
   const knownKeys = new Set(Object.keys(properties));
   const unknownKeys = Object.keys(args).filter((key) => !knownKeys.has(key));
   if (unknownKeys.length > 0) {
-    throw new SuperDocCliError(`Unknown argument(s) for ${toolName}: ${unknownKeys.join(", ")}`, {
+    const retiredKeys = unknownKeys.filter((key) => Object.hasOwn(RETIRED_ACTION_ARGS, key));
+    const message = retiredKeys.length > 0 ? retiredKeys.map((key) => RETIRED_ACTION_ARGS[key]).join(" ") : `Unknown argument(s) for ${toolName}: ${unknownKeys.join(", ")}`;
+    throw new SuperDocCliError(message, {
       code: "INVALID_ARGUMENT",
-      details: { toolName, unknownKeys, knownKeys: [...knownKeys] }
+      details: {
+        toolName,
+        unknownKeys,
+        knownKeys: [...knownKeys],
+        ...retiredKeys.length > 0 ? { retiredKeys } : {}
+      }
     });
   }
   const missingKeys = required2.filter((key) => args[key] == null);
@@ -94873,7 +94873,7 @@ async function coreDispatch(documentHandle, toolName, args, invokeOptions) {
     details: { toolName, preset: "core" }
   });
 }
-var MODULE_DIR2, PROMPT_DIR_CANDIDATES, _actionsOnlyPromptCache = null, _mcpPromptCache = null, RECEIPT_LIST_CAP = 8, EXECUTE_CODE_SPEC, SENTENCE_BOUNDARY, ENFORCEMENT_SIDE_DOORS, POLICY_OPTION_KEYS, corePreset;
+var MODULE_DIR2, PROMPT_DIR_CANDIDATES, _actionsOnlyPromptCache = null, _mcpPromptCache = null, RETIRED_ACTION_ARGS, RECEIPT_LIST_CAP = 8, EXECUTE_CODE_SPEC, SENTENCE_BOUNDARY, ENFORCEMENT_SIDE_DOORS, POLICY_OPTION_KEYS, corePreset;
 var init_core3 = __esm(() => {
   init_errors4();
   init_debug_trace();
@@ -94883,6 +94883,9 @@ var init_core3 = __esm(() => {
   init_embedded_prompts_generated();
   MODULE_DIR2 = path5.dirname(fileURLToPath4(import.meta.url));
   PROMPT_DIR_CANDIDATES = [path5.resolve(MODULE_DIR2, "..", "prompts"), path5.resolve(MODULE_DIR2, "prompts")];
+  RETIRED_ACTION_ARGS = {
+    dryRun: "dryRun is not supported by superdoc_perform_action: no action previews a mutation. To make an edit reviewable before it is final, pass changeMode: 'tracked' on an action that records tracked changes; to take one back, use the undo_changes action."
+  };
   EXECUTE_CODE_SPEC = {
     operationId: "doc.executeCode",
     commandTokens: ["execute", "code"],
@@ -328904,9 +328907,10 @@ function rawProperties(prXml) {
 }
 function extractText3(xml) {
   let out = "";
-  const re = /<w:t\b[^>]*>([\s\S]*?)<\/w:t>/g;
+  const re = /<w:t\b[^>]*?(?:\/>|>([\s\S]*?)<\/w:t>)/g;
   for (let match = re.exec(xml);match; match = re.exec(xml)) {
-    out += decodeXmlEntities(match[1]);
+    if (match[1] !== undefined)
+      out += decodeXmlEntities(match[1]);
   }
   return out;
 }

package/package.json

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