@superdoc-dev/mcp 0.2.0-next.2 → 0.2.0-next.3

package/dist/index.js

@@ -48744,10 +48744,10 @@ import { access, readFile, writeFile } from "node:fs/promises";
 import { randomBytes } from "node:crypto";
 import { resolve as resolve3, basename as basename2 } from "node:path";
 
-// ../../packages/superdoc/dist/chunks/src-99Lji-cs.es.js
+// ../../packages/superdoc/dist/chunks/src-CYnAk7Cg.es.js
 init_rolldown_runtime_B2q5OVn9_es();
-var exports_src_99Lji_cs_es = {};
-__export(exports_src_99Lji_cs_es, {
+var exports_src_CYnAk7Cg_es = {};
+__export(exports_src_CYnAk7Cg_es, {
   zt: () => defineMark,
   z: () => cM,
   yt: () => removeAwarenessStates,
@@ -115442,7 +115442,7 @@ if (glo[importIdentifier] === true) {
 }
 glo[importIdentifier] = true;
 
-// ../../packages/superdoc/dist/chunks/src-99Lji-cs.es.js
+// ../../packages/superdoc/dist/chunks/src-CYnAk7Cg.es.js
 function isNodeType(node2, name) {
   return node2.type.name === name;
 }
@@ -139610,7 +139610,11 @@ var OPERATION_DEFINITIONS = {
       supportsDryRun: false,
       supportsTrackedMode: false,
       possibleFailureCodes: ["INVALID_TARGET", "NO_OP"],
-      throws: [...T_NOT_FOUND_COMMAND, "INVALID_TARGET"]
+      throws: [
+        ...T_NOT_FOUND_COMMAND,
+        "INVALID_TARGET",
+        "INVALID_INPUT"
+      ]
     }),
     referenceDocPath: "comments/patch.mdx",
     referenceGroup: "comments"
@@ -139676,7 +139680,11 @@ var OPERATION_DEFINITIONS = {
       supportsDryRun: false,
       supportsTrackedMode: false,
       possibleFailureCodes: ["NO_OP"],
-      throws: T_NOT_FOUND_COMMAND
+      throws: [
+        ...T_NOT_FOUND_COMMAND,
+        "INVALID_INPUT",
+        "INVALID_TARGET"
+      ]
     }),
     referenceDocPath: "track-changes/decide.mdx",
     referenceGroup: "trackChanges"
@@ -140668,7 +140676,9 @@ function validateCreateCommentInput(input2) {
       throw new DocumentApiValidationError("INVALID_TARGET", "Cannot combine parentCommentId with target. Replies do not take a target.", { fields: ["parentCommentId", "target"] });
     return;
   }
-  if (hasTarget && !isTextAddress(target))
+  if (!hasTarget)
+    throw new DocumentApiValidationError("INVALID_TARGET", "comments.create requires a target for root comments.", { field: "target" });
+  if (!isTextAddress(target))
     throw new DocumentApiValidationError("INVALID_TARGET", "target must be a text address object.", {
       field: "target",
       value: target
@@ -140692,6 +140702,17 @@ function validatePatchCommentInput(input2) {
       field: "commentId",
       value: commentId
     });
+  const mutationFields = [
+    "text",
+    "target",
+    "status",
+    "isInternal"
+  ];
+  const providedFields = mutationFields.filter((f) => input2[f] !== undefined);
+  if (providedFields.length === 0)
+    throw new DocumentApiValidationError("INVALID_INPUT", "comments.patch requires exactly one mutation field (text, target, status, or isInternal).", { allowedFields: [...mutationFields] });
+  if (providedFields.length > 1)
+    throw new DocumentApiValidationError("INVALID_INPUT", `comments.patch accepts exactly one mutation field per call, got ${providedFields.length}: ${providedFields.join(", ")}.`, { providedFields: [...providedFields] });
   const { status } = input2;
   if (status !== undefined && status !== "resolved")
     throw new DocumentApiValidationError("INVALID_TARGET", `status must be "resolved", got "${String(status)}".`, {
@@ -140715,25 +140736,24 @@ function executeCommentsCreate(adapter, input2, options) {
 }
 function executeCommentsPatch(adapter, input2, options) {
   validatePatchCommentInput(input2);
-  let lastReceipt = { success: true };
   if (input2.text !== undefined)
-    lastReceipt = adapter.edit({
+    return adapter.edit({
       commentId: input2.commentId,
       text: input2.text
     }, options);
   if (input2.target !== undefined)
-    lastReceipt = adapter.move({
+    return adapter.move({
       commentId: input2.commentId,
       target: input2.target
     }, options);
   if (input2.status === "resolved")
-    lastReceipt = adapter.resolve({ commentId: input2.commentId }, options);
+    return adapter.resolve({ commentId: input2.commentId }, options);
   if (input2.isInternal !== undefined)
-    lastReceipt = adapter.setInternal({
+    return adapter.setInternal({
       commentId: input2.commentId,
       isInternal: input2.isInternal
     }, options);
-  return lastReceipt;
+  return { success: true };
 }
 function executeCommentsDelete(adapter, input2, options) {
   return adapter.remove({ commentId: input2.commentId }, options);
@@ -141026,16 +141046,38 @@ function executeTrackChangesList(adapter, input2) {
 function executeTrackChangesGet(adapter, input2) {
   return adapter.get(input2);
 }
-function executeTrackChangesDecide(adapter, input2, options) {
-  const isAll = "scope" in input2.target && input2.target.scope === "all";
+function executeTrackChangesDecide(adapter, rawInput, options) {
+  const raw = rawInput;
+  if (typeof raw !== "object" || raw == null)
+    throw new DocumentApiValidationError("INVALID_INPUT", "trackChanges.decide input must be a non-null object.", { value: raw });
+  const input2 = raw;
+  if (input2.decision !== "accept" && input2.decision !== "reject")
+    throw new DocumentApiValidationError("INVALID_INPUT", `trackChanges.decide decision must be "accept" or "reject", got "${String(input2.decision)}".`, {
+      field: "decision",
+      value: input2.decision
+    });
+  if (typeof input2.target !== "object" || input2.target == null)
+    throw new DocumentApiValidationError("INVALID_TARGET", 'trackChanges.decide target must be an object with { id: string } or { scope: "all" }.', {
+      field: "target",
+      value: input2.target
+    });
+  const target = input2.target;
+  const isAll = target.scope === "all";
+  if (!isAll) {
+    if (typeof target.id !== "string" || target.id.length === 0)
+      throw new DocumentApiValidationError("INVALID_TARGET", 'trackChanges.decide target must have { id: string } or { scope: "all" }.', {
+        field: "target",
+        value: input2.target
+      });
+  }
   if (input2.decision === "accept") {
     if (isAll)
       return adapter.acceptAll({}, options);
-    return adapter.accept({ id: input2.target.id }, options);
+    return adapter.accept({ id: target.id }, options);
   }
   if (isAll)
     return adapter.rejectAll({}, options);
-  return adapter.reject({ id: input2.target.id }, options);
+  return adapter.reject({ id: target.id }, options);
 }
 function buildDispatchTable(api2) {
   return {
@@ -210759,8 +210801,8 @@ function stubFalse() {
   return false;
 }
 var stubFalse_default = stubFalse;
-var freeExports$2 = typeof exports_src_99Lji_cs_es == "object" && exports_src_99Lji_cs_es && !exports_src_99Lji_cs_es.nodeType && exports_src_99Lji_cs_es;
-var freeModule$2 = freeExports$2 && typeof module_src_99Lji_cs_es == "object" && module_src_99Lji_cs_es && !module_src_99Lji_cs_es.nodeType && module_src_99Lji_cs_es;
+var freeExports$2 = typeof exports_src_CYnAk7Cg_es == "object" && exports_src_CYnAk7Cg_es && !exports_src_CYnAk7Cg_es.nodeType && exports_src_CYnAk7Cg_es;
+var freeModule$2 = freeExports$2 && typeof module_src_CYnAk7Cg_es == "object" && module_src_CYnAk7Cg_es && !module_src_CYnAk7Cg_es.nodeType && module_src_CYnAk7Cg_es;
 var Buffer$1 = freeModule$2 && freeModule$2.exports === freeExports$2 ? _root_default.Buffer : undefined;
 var isBuffer_default = (Buffer$1 ? Buffer$1.isBuffer : undefined) || stubFalse_default;
 var argsTag$1 = "[object Arguments]";
@@ -210800,8 +210842,8 @@ function baseUnary(func) {
   };
 }
 var _baseUnary_default = baseUnary;
-var freeExports$1 = typeof exports_src_99Lji_cs_es == "object" && exports_src_99Lji_cs_es && !exports_src_99Lji_cs_es.nodeType && exports_src_99Lji_cs_es;
-var freeModule$1 = freeExports$1 && typeof module_src_99Lji_cs_es == "object" && module_src_99Lji_cs_es && !module_src_99Lji_cs_es.nodeType && module_src_99Lji_cs_es;
+var freeExports$1 = typeof exports_src_CYnAk7Cg_es == "object" && exports_src_CYnAk7Cg_es && !exports_src_CYnAk7Cg_es.nodeType && exports_src_CYnAk7Cg_es;
+var freeModule$1 = freeExports$1 && typeof module_src_CYnAk7Cg_es == "object" && module_src_CYnAk7Cg_es && !module_src_CYnAk7Cg_es.nodeType && module_src_CYnAk7Cg_es;
 var freeProcess = freeModule$1 && freeModule$1.exports === freeExports$1 && _freeGlobal_default.process;
 var _nodeUtil_default = function() {
   try {
@@ -211182,8 +211224,8 @@ Stack.prototype.get = _stackGet_default;
 Stack.prototype.has = _stackHas_default;
 Stack.prototype.set = _stackSet_default;
 var _Stack_default = Stack;
-var freeExports = typeof exports_src_99Lji_cs_es == "object" && exports_src_99Lji_cs_es && !exports_src_99Lji_cs_es.nodeType && exports_src_99Lji_cs_es;
-var freeModule = freeExports && typeof module_src_99Lji_cs_es == "object" && module_src_99Lji_cs_es && !module_src_99Lji_cs_es.nodeType && module_src_99Lji_cs_es;
+var freeExports = typeof exports_src_CYnAk7Cg_es == "object" && exports_src_CYnAk7Cg_es && !exports_src_CYnAk7Cg_es.nodeType && exports_src_CYnAk7Cg_es;
+var freeModule = freeExports && typeof module_src_CYnAk7Cg_es == "object" && module_src_CYnAk7Cg_es && !module_src_CYnAk7Cg_es.nodeType && module_src_CYnAk7Cg_es;
 var Buffer4 = freeModule && freeModule.exports === freeExports ? _root_default.Buffer : undefined;
 var allocUnsafe = Buffer4 ? Buffer4.allocUnsafe : undefined;
 function cloneBuffer(buffer4, isDeep) {
@@ -221201,7 +221243,7 @@ var OPERATION_DEFINITIONS2 = {
       supportsDryRun: false,
       supportsTrackedMode: false,
       possibleFailureCodes: ["INVALID_TARGET", "NO_OP"],
-      throws: [...T_NOT_FOUND_COMMAND2, "INVALID_TARGET"]
+      throws: [...T_NOT_FOUND_COMMAND2, "INVALID_TARGET", "INVALID_INPUT"]
     }),
     referenceDocPath: "comments/patch.mdx",
     referenceGroup: "comments"
@@ -221271,7 +221313,7 @@ var OPERATION_DEFINITIONS2 = {
       supportsDryRun: false,
       supportsTrackedMode: false,
       possibleFailureCodes: ["NO_OP"],
-      throws: T_NOT_FOUND_COMMAND2
+      throws: [...T_NOT_FOUND_COMMAND2, "INVALID_INPUT", "INVALID_TARGET"]
     }),
     referenceDocPath: "track-changes/decide.mdx",
     referenceGroup: "trackChanges"
@@ -222319,7 +222361,12 @@ function validateCreateCommentInput2(input2) {
     }
     return;
   }
-  if (hasTarget && !isTextAddress2(target)) {
+  if (!hasTarget) {
+    throw new DocumentApiValidationError3("INVALID_TARGET", "comments.create requires a target for root comments.", {
+      field: "target"
+    });
+  }
+  if (!isTextAddress2(target)) {
     throw new DocumentApiValidationError3("INVALID_TARGET", "target must be a text address object.", {
       field: "target",
       value: target
@@ -222340,6 +222387,14 @@ function validatePatchCommentInput2(input2) {
       value: commentId
     });
   }
+  const mutationFields = ["text", "target", "status", "isInternal"];
+  const providedFields = mutationFields.filter((f) => input2[f] !== undefined);
+  if (providedFields.length === 0) {
+    throw new DocumentApiValidationError3("INVALID_INPUT", "comments.patch requires exactly one mutation field (text, target, status, or isInternal).", { allowedFields: [...mutationFields] });
+  }
+  if (providedFields.length > 1) {
+    throw new DocumentApiValidationError3("INVALID_INPUT", `comments.patch accepts exactly one mutation field per call, got ${providedFields.length}: ${providedFields.join(", ")}.`, { providedFields: [...providedFields] });
+  }
   const { status } = input2;
   if (status !== undefined && status !== "resolved") {
     throw new DocumentApiValidationError3("INVALID_TARGET", `status must be "resolved", got "${String(status)}".`, {
@@ -222363,20 +222418,19 @@ function executeCommentsCreate2(adapter, input2, options) {
 }
 function executeCommentsPatch2(adapter, input2, options) {
   validatePatchCommentInput2(input2);
-  let lastReceipt = { success: true };
   if (input2.text !== undefined) {
-    lastReceipt = adapter.edit({ commentId: input2.commentId, text: input2.text }, options);
+    return adapter.edit({ commentId: input2.commentId, text: input2.text }, options);
   }
   if (input2.target !== undefined) {
-    lastReceipt = adapter.move({ commentId: input2.commentId, target: input2.target }, options);
+    return adapter.move({ commentId: input2.commentId, target: input2.target }, options);
   }
   if (input2.status === "resolved") {
-    lastReceipt = adapter.resolve({ commentId: input2.commentId }, options);
+    return adapter.resolve({ commentId: input2.commentId }, options);
   }
   if (input2.isInternal !== undefined) {
-    lastReceipt = adapter.setInternal({ commentId: input2.commentId, isInternal: input2.isInternal }, options);
+    return adapter.setInternal({ commentId: input2.commentId, isInternal: input2.isInternal }, options);
   }
-  return lastReceipt;
+  return { success: true };
 }
 function executeCommentsDelete2(adapter, input2, options) {
   return adapter.remove({ commentId: input2.commentId }, options);
@@ -222698,16 +222752,35 @@ function executeTrackChangesList2(adapter, input2) {
 function executeTrackChangesGet2(adapter, input2) {
   return adapter.get(input2);
 }
-function executeTrackChangesDecide2(adapter, input2, options) {
-  const isAll = "scope" in input2.target && input2.target.scope === "all";
+function executeTrackChangesDecide2(adapter, rawInput, options) {
+  const raw = rawInput;
+  if (typeof raw !== "object" || raw == null) {
+    throw new DocumentApiValidationError3("INVALID_INPUT", "trackChanges.decide input must be a non-null object.", {
+      value: raw
+    });
+  }
+  const input2 = raw;
+  if (input2.decision !== "accept" && input2.decision !== "reject") {
+    throw new DocumentApiValidationError3("INVALID_INPUT", `trackChanges.decide decision must be "accept" or "reject", got "${String(input2.decision)}".`, { field: "decision", value: input2.decision });
+  }
+  if (typeof input2.target !== "object" || input2.target == null) {
+    throw new DocumentApiValidationError3("INVALID_TARGET", 'trackChanges.decide target must be an object with { id: string } or { scope: "all" }.', { field: "target", value: input2.target });
+  }
+  const target = input2.target;
+  const isAll = target.scope === "all";
+  if (!isAll) {
+    if (typeof target.id !== "string" || target.id.length === 0) {
+      throw new DocumentApiValidationError3("INVALID_TARGET", 'trackChanges.decide target must have { id: string } or { scope: "all" }.', { field: "target", value: input2.target });
+    }
+  }
   if (input2.decision === "accept") {
     if (isAll)
       return adapter.acceptAll({}, options);
-    return adapter.accept({ id: input2.target.id }, options);
+    return adapter.accept({ id: target.id }, options);
   }
   if (isAll)
     return adapter.rejectAll({}, options);
-  return adapter.reject({ id: input2.target.id }, options);
+  return adapter.reject({ id: target.id }, options);
 }
 
 // ../../packages/document-api/src/invoke/invoke.ts
@@ -261084,7 +261157,10 @@ function registerTrackChangesTools(server, sessions) {
   }, async ({ session_id, id: id2 }) => {
     try {
       const { api: api2 } = sessions.get(session_id);
-      const result = api2.invoke({ operationId: "review.decide", input: { decision: "accept", target: { id: id2 } } });
+      const result = api2.invoke({
+        operationId: "trackChanges.decide",
+        input: { decision: "accept", target: { id: id2 } }
+      });
       return {
         content: [{ type: "text", text: JSON.stringify(result, null, 2) }]
       };
@@ -261106,7 +261182,10 @@ function registerTrackChangesTools(server, sessions) {
   }, async ({ session_id, id: id2 }) => {
     try {
       const { api: api2 } = sessions.get(session_id);
-      const result = api2.invoke({ operationId: "review.decide", input: { decision: "reject", target: { id: id2 } } });
+      const result = api2.invoke({
+        operationId: "trackChanges.decide",
+        input: { decision: "reject", target: { id: id2 } }
+      });
       return {
         content: [{ type: "text", text: JSON.stringify(result, null, 2) }]
       };
@@ -261128,7 +261207,7 @@ function registerTrackChangesTools(server, sessions) {
     try {
       const { api: api2 } = sessions.get(session_id);
       const result = api2.invoke({
-        operationId: "review.decide",
+        operationId: "trackChanges.decide",
         input: { decision: "accept", target: { scope: "all" } }
       });
       return {
@@ -261152,7 +261231,7 @@ function registerTrackChangesTools(server, sessions) {
     try {
       const { api: api2 } = sessions.get(session_id);
       const result = api2.invoke({
-        operationId: "review.decide",
+        operationId: "trackChanges.decide",
         input: { decision: "reject", target: { scope: "all" } }
       });
       return {
@@ -261183,7 +261262,7 @@ function registerCommentTools(server, sessions) {
       const { api: api2 } = sessions.get(session_id);
       const parsed = JSON.parse(target);
       const result = api2.invoke({
-        operationId: "comments.add",
+        operationId: "comments.create",
         input: { text: text4, target: parsed }
       });
       return {
@@ -261234,7 +261313,7 @@ function registerCommentTools(server, sessions) {
     try {
       const { api: api2 } = sessions.get(session_id);
       const result = api2.invoke({
-        operationId: "comments.reply",
+        operationId: "comments.create",
         input: { parentCommentId: comment_id, text: text4 }
       });
       return {
@@ -261259,8 +261338,8 @@ function registerCommentTools(server, sessions) {
     try {
       const { api: api2 } = sessions.get(session_id);
       const result = api2.invoke({
-        operationId: "comments.resolve",
-        input: { commentId: comment_id }
+        operationId: "comments.patch",
+        input: { commentId: comment_id, status: "resolved" }
       });
       return {
         content: [{ type: "text", text: JSON.stringify(result, null, 2) }]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@superdoc-dev/mcp",
-  "version": "0.2.0-next.2",
+  "version": "0.2.0-next.3",
   "type": "module",
   "bin": {
     "superdoc-mcp": "./dist/index.js"