@guiie/buda-mcp 1.5.1 → 1.5.2

package/src/tools/compare_markets.ts

@@ -2,6 +2,7 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { z } from "zod";
 import { BudaClient, BudaApiError } from "../client.js";
 import { MemoryCache, CACHE_TTL } from "../cache.js";
+import { validateCurrency } from "../validation.js";
 import type { AllTickersResponse } from "../types.js";
 
 export const toolSchema = {
@@ -24,6 +25,82 @@ export const toolSchema = {
   },
 };
 
+export async function handleCompareMarkets(
+  args: { base_currency: string },
+  client: BudaClient,
+  cache: MemoryCache,
+): Promise<{ content: Array<{ type: "text"; text: string }>; isError?: boolean }> {
+  const { base_currency } = args;
+
+  const currencyError = validateCurrency(base_currency);
+  if (currencyError) {
+    return {
+      content: [{ type: "text", text: JSON.stringify({ error: currencyError, code: "INVALID_CURRENCY" }) }],
+      isError: true,
+    };
+  }
+
+  try {
+    const base = base_currency.toUpperCase();
+    const data = await cache.getOrFetch<AllTickersResponse>(
+      "tickers:all",
+      CACHE_TTL.TICKER,
+      () => client.get<AllTickersResponse>("/tickers"),
+    );
+
+    const matching = data.tickers.filter((t) => {
+      const [tickerBase] = t.market_id.split("-");
+      return tickerBase === base;
+    });
+
+    if (matching.length === 0) {
+      return {
+        content: [
+          {
+            type: "text",
+            text: JSON.stringify({
+              error: `No markets found for base currency '${base}'.`,
+              code: "NOT_FOUND",
+            }),
+          },
+        ],
+        isError: true,
+      };
+    }
+
+    const result = {
+      base_currency: base,
+      markets: matching.map((t) => ({
+        market_id: t.market_id,
+        last_price: parseFloat(t.last_price[0]),
+        last_price_currency: t.last_price[1],
+        best_bid: t.max_bid ? parseFloat(t.max_bid[0]) : null,
+        best_ask: t.min_ask ? parseFloat(t.min_ask[0]) : null,
+        volume_24h: t.volume ? parseFloat(t.volume[0]) : null,
+        price_change_24h: t.price_variation_24h
+          ? parseFloat((parseFloat(t.price_variation_24h) * 100).toFixed(4))
+          : null,
+        price_change_7d: t.price_variation_7d
+          ? parseFloat((parseFloat(t.price_variation_7d) * 100).toFixed(4))
+          : null,
+      })),
+    };
+
+    return {
+      content: [{ type: "text", text: JSON.stringify(result, null, 2) }],
+    };
+  } catch (err) {
+    const msg =
+      err instanceof BudaApiError
+        ? { error: err.message, code: err.status }
+        : { error: String(err), code: "UNKNOWN" };
+    return {
+      content: [{ type: "text", text: JSON.stringify(msg) }],
+      isError: true,
+    };
+  }
+}
+
 export function register(server: McpServer, client: BudaClient, cache: MemoryCache): void {
   server.tool(
     toolSchema.name,
@@ -35,66 +112,6 @@ export function register(server: McpServer, client: BudaClient, cache: MemoryCac
           "Base currency to compare across all available markets (e.g. 'BTC', 'ETH', 'XRP').",
         ),
     },
-    async ({ base_currency }) => {
-      try {
-        const base = base_currency.toUpperCase();
-        const data = await cache.getOrFetch<AllTickersResponse>(
-          "tickers:all",
-          CACHE_TTL.TICKER,
-          () => client.get<AllTickersResponse>("/tickers"),
-        );
-
-        const matching = data.tickers.filter((t) => {
-          const [tickerBase] = t.market_id.split("-");
-          return tickerBase === base;
-        });
-
-        if (matching.length === 0) {
-          return {
-            content: [
-              {
-                type: "text",
-                text: JSON.stringify({
-                  error: `No markets found for base currency '${base}'.`,
-                  code: "NOT_FOUND",
-                }),
-              },
-            ],
-            isError: true,
-          };
-        }
-
-        const result = {
-          base_currency: base,
-          markets: matching.map((t) => ({
-            market_id: t.market_id,
-            last_price: parseFloat(t.last_price[0]),
-            last_price_currency: t.last_price[1],
-            best_bid: t.max_bid ? parseFloat(t.max_bid[0]) : null,
-            best_ask: t.min_ask ? parseFloat(t.min_ask[0]) : null,
-            volume_24h: t.volume ? parseFloat(t.volume[0]) : null,
-            price_change_24h: t.price_variation_24h
-              ? parseFloat((parseFloat(t.price_variation_24h) * 100).toFixed(4))
-              : null,
-            price_change_7d: t.price_variation_7d
-              ? parseFloat((parseFloat(t.price_variation_7d) * 100).toFixed(4))
-              : null,
-          })),
-        };
-
-        return {
-          content: [{ type: "text", text: JSON.stringify(result, null, 2) }],
-        };
-      } catch (err) {
-        const msg =
-          err instanceof BudaApiError
-            ? { error: err.message, code: err.status, path: err.path }
-            : { error: String(err), code: "UNKNOWN" };
-        return {
-          content: [{ type: "text", text: JSON.stringify(msg) }],
-          isError: true,
-        };
-      }
-    },
+    (args) => handleCompareMarkets(args, client, cache),
   );
 }

package/src/tools/dead_mans_switch.ts

@@ -2,6 +2,7 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { z } from "zod";
 import { BudaClient, BudaApiError } from "../client.js";
 import { validateMarketId } from "../validation.js";
+import { logAudit } from "../audit.js";
 import type { OrdersResponse, OrderResponse } from "../types.js";
 
 // ---- Module-level timer state (persists across HTTP requests / tool invocations) ----
@@ -125,6 +126,7 @@ type ScheduleArgs = {
 export async function handleScheduleCancelAll(
   args: ScheduleArgs,
   client: BudaClient,
+  transport: "http" | "stdio" = "stdio",
 ): Promise<{ content: Array<{ type: "text"; text: string }>; isError?: boolean }> {
   const { market_id, ttl_seconds, confirmation_token } = args;
 
@@ -173,6 +175,7 @@ export async function handleScheduleCancelAll(
   const id = market_id.toLowerCase();
   const entry = armTimer(id, ttl_seconds, client);
 
+  logAudit({ ts: new Date().toISOString(), tool: "schedule_cancel_all", transport, args_summary: { market_id: market_id.toUpperCase(), ttl_seconds }, success: true });
   return {
     content: [
       {
@@ -334,7 +337,23 @@ export function register(
         .string()
         .describe("Market ID whose timer should be renewed (e.g. 'BTC-CLP')."),
     },
-    (args) => handleRenewCancelTimer(args, client),
+    (args) => {
+      if (transport === "http") {
+        return Promise.resolve({
+          content: [{
+            type: "text" as const,
+            text: JSON.stringify({
+              error:
+                "renew_cancel_timer is not available on the HTTP transport. " +
+                "Timer state is process-local and only meaningful in stdio mode on a persistent local process.",
+              code: "TRANSPORT_NOT_SUPPORTED",
+            }),
+          }],
+          isError: true,
+        });
+      }
+      return Promise.resolve(handleRenewCancelTimer(args, client));
+    },
   );
 
   server.tool(
@@ -345,6 +364,22 @@ export function register(
         .string()
         .describe("Market ID whose timer should be disarmed (e.g. 'BTC-CLP')."),
     },
-    (args) => handleDisarmCancelTimer(args),
+    (args) => {
+      if (transport === "http") {
+        return Promise.resolve({
+          content: [{
+            type: "text" as const,
+            text: JSON.stringify({
+              error:
+                "disarm_cancel_timer is not available on the HTTP transport. " +
+                "Timer state is process-local and only meaningful in stdio mode on a persistent local process.",
+              code: "TRANSPORT_NOT_SUPPORTED",
+            }),
+          }],
+          isError: true,
+        });
+      }
+      return Promise.resolve(handleDisarmCancelTimer(args));
+    },
   );
 }

package/src/tools/deposits.ts

@@ -104,7 +104,7 @@ export async function handleGetDepositHistory(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],
@@ -189,7 +189,7 @@ export async function handleCreateFiatDeposit(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],

package/src/tools/fees.ts

@@ -69,7 +69,7 @@ export async function handleGetNetworkFees(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],

package/src/tools/lightning.ts

@@ -2,6 +2,7 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { z } from "zod";
 import { BudaClient, BudaApiError } from "../client.js";
 import { flattenAmount } from "../utils.js";
+import { logAudit } from "../audit.js";
 import type { LightningWithdrawalResponse, LightningInvoiceResponse } from "../types.js";
 
 export const lightningWithdrawalToolSchema = {
@@ -69,6 +70,7 @@ type CreateLightningInvoiceArgs = {
 export async function handleLightningWithdrawal(
   args: LightningWithdrawalArgs,
   client: BudaClient,
+  transport: "http" | "stdio" = "stdio",
 ): Promise<{ content: Array<{ type: "text"; text: string }>; isError?: boolean }> {
   const { invoice, confirmation_token } = args;
 
@@ -91,7 +93,7 @@ export async function handleLightningWithdrawal(
     };
   }
 
-  const BOLT11_RE = /^ln(bc|tb|bcrt)\d/i;
+  const BOLT11_RE = /^ln(bc|tb|bcrt)\d*[munp]?1[a-z0-9]{20,}$/i;
   if (!BOLT11_RE.test(invoice)) {
     return {
       content: [{
@@ -117,10 +119,10 @@ export async function handleLightningWithdrawal(
     const amount = flattenAmount(lw.amount);
     const fee = flattenAmount(lw.fee);
 
-    return {
+    const result = {
       content: [
         {
-          type: "text",
+          type: "text" as const,
           text: JSON.stringify(
             {
               id: lw.id,
@@ -138,15 +140,16 @@ export async function handleLightningWithdrawal(
         },
       ],
     };
+    logAudit({ ts: new Date().toISOString(), tool: "lightning_withdrawal", transport, args_summary: {}, success: true });
+    return result;
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
-    return {
-      content: [{ type: "text", text: JSON.stringify(msg) }],
-      isError: true,
-    };
+    const result = { content: [{ type: "text" as const, text: JSON.stringify(msg) }], isError: true as const };
+    logAudit({ ts: new Date().toISOString(), tool: "lightning_withdrawal", transport, args_summary: {}, success: false, error_code: msg.code });
+    return result;
   }
 }
 
@@ -192,7 +195,7 @@ export async function handleCreateLightningInvoice(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],

package/src/tools/market_sentiment.ts

@@ -118,7 +118,7 @@ export async function handleMarketSentiment(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],

package/src/tools/market_summary.ts

@@ -101,7 +101,7 @@ export async function handleMarketSummary(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],

package/src/tools/markets.ts

@@ -68,7 +68,7 @@ export function register(server: McpServer, client: BudaClient, cache: MemoryCac
       } catch (err) {
         const msg =
           err instanceof BudaApiError
-            ? { error: err.message, code: err.status, path: err.path }
+            ? { error: err.message, code: err.status }
             : { error: String(err), code: "UNKNOWN" };
         return {
           content: [{ type: "text", text: JSON.stringify(msg) }],

package/src/tools/order_lookup.ts

@@ -88,7 +88,7 @@ export async function handleGetOrder(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],
@@ -109,7 +109,7 @@ export async function handleGetOrderByClientId(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],

package/src/tools/orderbook.ts

@@ -83,7 +83,7 @@ export function register(server: McpServer, client: BudaClient, cache: MemoryCac
       } catch (err) {
         const msg =
           err instanceof BudaApiError
-            ? { error: err.message, code: err.status, path: err.path }
+            ? { error: err.message, code: err.status }
             : { error: String(err), code: "UNKNOWN" };
         return {
           content: [{ type: "text", text: JSON.stringify(msg) }],

package/src/tools/orders.ts

@@ -135,7 +135,7 @@ export function register(server: McpServer, client: BudaClient): void {
       } catch (err) {
         const msg =
           err instanceof BudaApiError
-            ? { error: err.message, code: err.status, path: err.path }
+            ? { error: err.message, code: err.status }
             : { error: String(err), code: "UNKNOWN" };
         return {
           content: [{ type: "text", text: JSON.stringify(msg) }],

package/src/tools/place_order.ts

@@ -2,6 +2,7 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { z } from "zod";
 import { BudaClient, BudaApiError } from "../client.js";
 import { validateMarketId } from "../validation.js";
+import { logAudit } from "../audit.js";
 import type { OrderResponse } from "../types.js";
 
 export const toolSchema = {
@@ -91,6 +92,7 @@ type PlaceOrderArgs = {
 export async function handlePlaceOrder(
   args: PlaceOrderArgs,
   client: BudaClient,
+  transport: "http" | "stdio" = "stdio",
 ): Promise<{ content: Array<{ type: "text"; text: string }>; isError?: boolean }> {
   const {
     market_id,
@@ -243,18 +245,35 @@ export async function handlePlaceOrder(
       payload,
     );
 
-    return {
-      content: [{ type: "text", text: JSON.stringify(data.order, null, 2) }],
+    const result = {
+      content: [{ type: "text" as const, text: JSON.stringify(data.order, null, 2) }],
     };
+    logAudit({
+      ts: new Date().toISOString(),
+      tool: "place_order",
+      transport,
+      args_summary: { market_id, type, price_type, amount },
+      success: true,
+    });
+    return result;
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
-    return {
-      content: [{ type: "text", text: JSON.stringify(msg) }],
+    const result = {
+      content: [{ type: "text" as const, text: JSON.stringify(msg) }],
       isError: true,
     };
+    logAudit({
+      ts: new Date().toISOString(),
+      tool: "place_order",
+      transport,
+      args_summary: { market_id, type, price_type, amount },
+      success: false,
+      error_code: msg.code,
+    });
+    return result;
   }
 }
 

package/src/tools/price_history.ts

@@ -108,7 +108,7 @@ export function register(server: McpServer, client: BudaClient, _cache: MemoryCa
       } catch (err) {
         const msg =
           err instanceof BudaApiError
-            ? { error: err.message, code: err.status, path: err.path }
+            ? { error: err.message, code: err.status }
             : { error: String(err), code: "UNKNOWN" };
         return {
           content: [{ type: "text", text: JSON.stringify(msg) }],

package/src/tools/quotation.ts

@@ -100,7 +100,7 @@ export async function handleGetRealQuotation(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],

package/src/tools/receive_addresses.ts

@@ -2,6 +2,7 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { z } from "zod";
 import { BudaClient, BudaApiError } from "../client.js";
 import { validateCurrency } from "../validation.js";
+import { logAudit } from "../audit.js";
 import type { ReceiveAddressesResponse, SingleReceiveAddressResponse, ReceiveAddress } from "../types.js";
 
 export const createReceiveAddressToolSchema = {
@@ -114,7 +115,7 @@ export async function handleListReceiveAddresses(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],
@@ -147,7 +148,7 @@ export async function handleGetReceiveAddress(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],
@@ -159,6 +160,7 @@ export async function handleGetReceiveAddress(
 export async function handleCreateReceiveAddress(
   args: { currency: string; confirmation_token: string },
   client: BudaClient,
+  transport: "http" | "stdio" = "stdio",
 ): Promise<{ content: Array<{ type: "text"; text: string }>; isError?: boolean }> {
   const { currency, confirmation_token } = args;
 
@@ -193,18 +195,17 @@ export async function handleCreateReceiveAddress(
       `/currencies/${currency.toUpperCase()}/receive_addresses`,
       {},
     );
-    return {
-      content: [{ type: "text", text: JSON.stringify(normalizeAddress(data.receive_address), null, 2) }],
-    };
+    const result = { content: [{ type: "text" as const, text: JSON.stringify(normalizeAddress(data.receive_address), null, 2) }] };
+    logAudit({ ts: new Date().toISOString(), tool: "create_receive_address", transport, args_summary: { currency }, success: true });
+    return result;
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
-    return {
-      content: [{ type: "text", text: JSON.stringify(msg) }],
-      isError: true,
-    };
+    const result = { content: [{ type: "text" as const, text: JSON.stringify(msg) }], isError: true as const };
+    logAudit({ ts: new Date().toISOString(), tool: "create_receive_address", transport, args_summary: { currency }, success: false, error_code: msg.code });
+    return result;
   }
 }
 

package/src/tools/remittance_recipients.ts

@@ -87,7 +87,7 @@ export async function handleListRemittanceRecipients(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],
@@ -108,7 +108,7 @@ export async function handleGetRemittanceRecipient(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],

package/src/tools/remittances.ts

@@ -3,6 +3,7 @@ import { z } from "zod";
 import { BudaClient, BudaApiError } from "../client.js";
 import { flattenAmount } from "../utils.js";
 import { validateCurrency } from "../validation.js";
+import { logAudit } from "../audit.js";
 import type { RemittancesResponse, SingleRemittanceResponse, Remittance } from "../types.js";
 
 export const listRemittancesToolSchema = {
@@ -152,7 +153,7 @@ export async function handleListRemittances(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],
@@ -173,7 +174,7 @@ export async function handleGetRemittance(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],
@@ -185,6 +186,7 @@ export async function handleGetRemittance(
 export async function handleQuoteRemittance(
   args: { currency: string; amount: number; recipient_id: number; confirmation_token: string },
   client: BudaClient,
+  transport: "http" | "stdio" = "stdio",
 ): Promise<{ content: Array<{ type: "text"; text: string }>; isError?: boolean }> {
   const { currency, amount, recipient_id, confirmation_token } = args;
 
@@ -222,24 +224,24 @@ export async function handleQuoteRemittance(
         recipient_id,
       },
     });
-    return {
-      content: [{ type: "text", text: JSON.stringify(normalizeRemittance(data.remittance), null, 2) }],
-    };
+    const result = { content: [{ type: "text" as const, text: JSON.stringify(normalizeRemittance(data.remittance), null, 2) }] };
+    logAudit({ ts: new Date().toISOString(), tool: "quote_remittance", transport, args_summary: { currency, amount, recipient_id }, success: true });
+    return result;
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
-    return {
-      content: [{ type: "text", text: JSON.stringify(msg) }],
-      isError: true,
-    };
+    const result = { content: [{ type: "text" as const, text: JSON.stringify(msg) }], isError: true as const };
+    logAudit({ ts: new Date().toISOString(), tool: "quote_remittance", transport, args_summary: { currency, amount, recipient_id }, success: false, error_code: msg.code });
+    return result;
   }
 }
 
 export async function handleAcceptRemittanceQuote(
   args: { id: number; confirmation_token: string },
   client: BudaClient,
+  transport: "http" | "stdio" = "stdio",
 ): Promise<{ content: Array<{ type: "text"; text: string }>; isError?: boolean }> {
   const { id, confirmation_token } = args;
 
@@ -264,18 +266,17 @@ export async function handleAcceptRemittanceQuote(
     const data = await client.put<SingleRemittanceResponse>(`/remittances/${id}`, {
       remittance: { state: "confirming" },
     });
-    return {
-      content: [{ type: "text", text: JSON.stringify(normalizeRemittance(data.remittance), null, 2) }],
-    };
+    const result = { content: [{ type: "text" as const, text: JSON.stringify(normalizeRemittance(data.remittance), null, 2) }] };
+    logAudit({ ts: new Date().toISOString(), tool: "accept_remittance_quote", transport, args_summary: { remittance_id: id }, success: true });
+    return result;
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
-    return {
-      content: [{ type: "text", text: JSON.stringify(msg) }],
-      isError: true,
-    };
+    const result = { content: [{ type: "text" as const, text: JSON.stringify(msg) }], isError: true as const };
+    logAudit({ ts: new Date().toISOString(), tool: "accept_remittance_quote", transport, args_summary: { remittance_id: id }, success: false, error_code: msg.code });
+    return result;
   }
 }
 

package/src/tools/simulate_order.ts

@@ -148,7 +148,7 @@ export async function handleSimulateOrder(
   } catch (err) {
     const msg =
       err instanceof BudaApiError
-        ? { error: err.message, code: err.status, path: err.path }
+        ? { error: err.message, code: err.status }
         : { error: String(err), code: "UNKNOWN" };
     return {
       content: [{ type: "text", text: JSON.stringify(msg) }],

package/src/tools/spread.ts

@@ -86,7 +86,7 @@ export function register(server: McpServer, client: BudaClient, cache: MemoryCac
       } catch (err) {
         const msg =
           err instanceof BudaApiError
-            ? { error: err.message, code: err.status, path: err.path }
+            ? { error: err.message, code: err.status }
             : { error: String(err), code: "UNKNOWN" };
         return {
           content: [{ type: "text", text: JSON.stringify(msg) }],