@mandujs/mcp 0.38.4 → 0.38.6

package/src/tools/slot-validation.ts

@@ -1,13 +1,13 @@
-import type { Tool } from "@modelcontextprotocol/sdk/types.js";
-import {
-  DEFAULT_SLOT_CONSTRAINTS,
-  API_SLOT_CONSTRAINTS,
-  READONLY_SLOT_CONSTRAINTS,
-} from "@mandujs/core";
-
-export const slotValidationToolDefinitions: Tool[] = [
-  {
-    name: "mandu.slot.constraints",
+import type { Tool } from "@modelcontextprotocol/sdk/types.js";
+import {
+  DEFAULT_SLOT_CONSTRAINTS,
+  API_SLOT_CONSTRAINTS,
+  READONLY_SLOT_CONSTRAINTS,
+} from "@mandujs/core";
+
+export const slotValidationToolDefinitions: Tool[] = [
+  {
+    name: "mandu.slot.constraints",
     description:
       "Get recommended slot constraint presets (default, api, readonly).",
     annotations: {
@@ -27,10 +27,10 @@ export const slotValidationToolDefinitions: Tool[] = [
   },
 ];
 
-export function slotValidationTools(projectRoot: string) {
-  void projectRoot;
-  const handlers: Record<string, (args: Record<string, unknown>) => Promise<unknown>> = {
-    "mandu.slot.constraints": async (args: Record<string, unknown>) => {
+export function slotValidationTools(projectRoot: string) {
+  void projectRoot;
+  const handlers: Record<string, (args: Record<string, unknown>) => Promise<unknown>> = {
+    "mandu.slot.constraints": async (args: Record<string, unknown>) => {
       const { preset } = args as { preset?: "default" | "api" | "readonly" };
 
       const presets = {
@@ -84,10 +84,10 @@ Mandu.filling()
         `.trim(),
       };
     },
-  };
-
-  // Backward-compatible aliases
-  handlers["mandu_get_slot_constraints"] = handlers["mandu.slot.constraints"];
-
+  };
+
+  // Backward-compatible aliases
+  handlers["mandu_get_slot_constraints"] = handlers["mandu.slot.constraints"];
+
   return handlers;
 }

package/src/tools/transaction.ts

@@ -6,7 +6,7 @@ import {
   getTransactionStatus,
   hasActiveTransaction,
 } from "@mandujs/core";
-import { acquireLock, releaseLock, checkLock, requireLock } from "../tx-lock.js";
+import { acquireLock, releaseLock, checkLock } from "../tx-lock.js";
 
 export const transactionToolDefinitions: Tool[] = [
   {
@@ -99,15 +99,9 @@ export function transactionTools(projectRoot: string) {
         return { error: lock.error };
       }
 
-      let change: Awaited<ReturnType<typeof beginChange>>;
-      try {
-        change = await beginChange(projectRoot, {
-          message: message || "MCP transaction",
-        });
-      } catch (error) {
-        if (lock.lockId) releaseLock(lock.lockId);
-        return { error: error instanceof Error ? error.message : String(error) };
-      }
+      const change = await beginChange(projectRoot, {
+        message: message || "MCP transaction",
+      });
 
       return {
         success: true,
@@ -128,10 +122,6 @@ export function transactionTools(projectRoot: string) {
           error: "No active transaction to commit",
         };
       }
-      const lockCheck = requireLock(lockId);
-      if (!lockCheck.allowed) {
-        return { error: lockCheck.error };
-      }
 
       const result = await commitChange(projectRoot);
       if (lockId) releaseLock(lockId);
@@ -152,10 +142,6 @@ export function transactionTools(projectRoot: string) {
           error: "No active transaction to rollback. Provide a changeId to rollback a specific change.",
         };
       }
-      const lockCheck = requireLock(lockId);
-      if (!lockCheck.allowed) {
-        return { error: lockCheck.error };
-      }
 
       const result = await rollbackChange(projectRoot, changeId);
       if (lockId) releaseLock(lockId);