@michaleffffff/mcp-trading-server 3.0.0 → 3.0.1

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 3.0.1 - 2026-03-17
+
+### Added
+- `extractErrorMessage` utility for cleaner error reporting across all tools.
+
+### Changed
+- `manage_liquidity` now performs strict SDK code validation before finalizing transactions.
+- `cancel_all_orders` improved to handle comma-separated and JSON-array strings for `orderIds`.
+- Integrated meaningful error extraction into `marketInfo`, `manageLiquidity`, and `updateOrderTpSl`.
+
 ## 3.0.0 - 2026-03-17
 
 ### Breaking Changes

package/README.md

@@ -161,6 +161,7 @@ The server will run using `stdio` transport and can be connected by any MCP-comp
 - **Price**: `"price"` is human by default and will be converted to **30-decimal** format internally. Use `raw:` to pass a 30-decimal integer directly.
 - **Slippage**: `slippagePct` uses 4-decimal raw units: `100` = `1%`, `50` = `0.5%`, `1000` = `10%`.
 - **Note**: Some account-transfer tools (e.g. `account_deposit`, `account_withdraw`, `check_approval`) use **raw integer strings**.
+- **`adjust_margin` units**: human amount is supported (e.g. `"1"` means 1 quote token). For exact atomic amount, use `raw:` (e.g. `"raw:1"`).
 
 ## P0 Reliability Contract (AI-friendly)
 
@@ -294,8 +295,9 @@ For practical tool-by-tool examples, see: `TOOL_EXAMPLES.md`
 * **close_position**: Close an open position.
 * **close_all_positions**: Emergency: close ALL open positions in a pool at once.
 * **cancel_order**: Cancel an open order by its order ID.
-* **cancel_all_orders**: Cancel multiple open orders by order IDs (use `get_open_orders` first to get IDs).
+* **cancel_all_orders**: Cancel open orders by IDs (supports array, JSON-array string, comma string, or single ID; use `get_open_orders` first).
 * **set_tp_sl**: Set take profit and stop loss prices for an open position.
+* **update_order_tp_sl**: Update TP/SL fields for an existing order. Accepts boolean-like values for `useOrderCollateral` / `isTpSlOrder`.
 * **adjust_margin**: Adjust the margin (collateral) of an open position.
 * **get_user_trading_fee_rate**: Query current maker/taker fee rates by asset class and risk tier.
 * **get_network_fee**: Query estimated network fee requirements for a market.
@@ -306,7 +308,7 @@ For practical tool-by-tool examples, see: `TOOL_EXAMPLES.md`
 * **get_oracle_price**: Get the current EVM oracle price for a specific pool.
 * **get_kline_latest_bar**: Get only the latest bar for an interval.
 * **get_all_tickers**: Get all ticker snapshots in one request.
-* **get_market_list**, **get_market_list_raw**, **get_kline**, **get_pool_info**...
+* **get_market_list**, **get_market_list_raw**, **get_kline**, **get_pool_info**... (`get_pool_info` auto-retries with oracle/ticker price on divide-by-zero reads)
 * **get_pool_symbol_all**, **get_base_detail**, **get_pool_level_config**...
 
 ### Account & Portfolio
@@ -321,7 +323,7 @@ For practical tool-by-tool examples, see: `TOOL_EXAMPLES.md`
 - If one section fails (wallet or trading-account), the tool may return a **partial** snapshot with `meta.partial=true` and section-level `error` details.
 
 ### Liquidity Provision (LP)
-* **manage_liquidity**: Add or withdraw liquidity from a BASE or QUOTE pool.
+* **manage_liquidity**: Add or withdraw liquidity from a BASE or QUOTE pool (aliases: `add/remove/increase/decrease` are supported).
 * **create_perp_market**: Create a new perpetual trading pair.
 
 ---
@@ -371,8 +373,9 @@ Use this query/mutation loop for limit-order management:
 
 1. Place order with `execute_trade` (`orderType=LIMIT`).
 2. Query active orders with `get_open_orders`.
-3. Check historical fills/cancellations with `get_order_history`.
-4. Cancel one order via `cancel_order` or batch via `cancel_all_orders`.
+3. If still pending, you can cancel/update order-level fields; if filled into a position, prefer `set_tp_sl` for position TP/SL.
+4. Check historical fills/cancellations with `get_order_history`.
+5. Cancel one order via `cancel_order` or batch via `cancel_all_orders`.
 
 Minimal query examples:
 

package/TOOL_EXAMPLES.md

@@ -140,6 +140,15 @@ It includes:
   }
 }
 ```
+Single-ID string is also accepted:
+```json
+{
+  "name": "cancel_all_orders",
+  "arguments": {
+    "orderIds": "0xORDER_ID_1"
+  }
+}
+```
 
 ### `set_tp_sl`
 ```json
@@ -180,6 +189,28 @@ It includes:
   }
 }
 ```
+String booleans are also accepted:
+```json
+{
+  "name": "update_order_tp_sl",
+  "arguments": {
+    "orderId": "0xORDER_ID",
+    "marketId": "0xMARKET_ID",
+    "size": 0.01,
+    "price": 65000,
+    "tpPrice": 70000,
+    "tpSize": 0.005,
+    "slPrice": 60000,
+    "slSize": 0.005,
+    "useOrderCollateral": "true",
+    "isTpSlOrder": "true",
+    "quoteToken": "0xQUOTE_TOKEN"
+  }
+}
+```
+Note:
+- This tool is most reliable for existing TP/SL orders or open positions.
+- If an unfilled LIMIT/STOP order returns `Failed to update order`, wait for fill and use `set_tp_sl`.
 
 ### `adjust_margin`
 ```json
@@ -188,7 +219,18 @@ It includes:
   "arguments": {
     "poolId": "0xPOOL_ID",
     "positionId": "0xPOSITION_ID",
-    "adjustAmount": "1000000"
+    "adjustAmount": "1"
+  }
+}
+```
+Raw precision mode:
+```json
+{
+  "name": "adjust_margin",
+  "arguments": {
+    "poolId": "0xPOOL_ID",
+    "positionId": "0xPOSITION_ID",
+    "adjustAmount": "raw:1000000"
   }
 }
 ```
@@ -295,6 +337,7 @@ Note: this tool may internally fallback to SDK `api.getMarketList()` / `api.getP
   }
 }
 ```
+Note: the server auto-retries with oracle/ticker market price when direct on-chain read hits divide-by-zero.
 
 ### `get_liquidity_info`
 ```json
@@ -502,6 +545,20 @@ If one section fails, check `data.meta.partial` and section-level `error` fields
   }
 }
 ```
+Alias action example (`remove` == `withdraw`):
+```json
+{
+  "name": "manage_liquidity",
+  "arguments": {
+    "action": "remove",
+    "poolType": "BASE",
+    "poolId": "0xPOOL_ID",
+    "amount": 1.5,
+    "slippage": 0.01
+  }
+}
+```
+If operation fails, check `error.message` for concrete reasons (for example `Insufficient Balance`) instead of generic `undefined`.
 
 ### `get_lp_price`
 ```json

package/dist/prompts/tradingGuide.js

@@ -29,6 +29,9 @@ You are an expert crypto trader using the MYX Protocol. To ensure successful exe
 - **Slippage**: Default is 100 (1%). For volatile meme tokens, consider 200-300 (2-3%).
 - **Fees**: Use \`get_user_trading_fee_rate\` to estimate fees before large trades.
 - **Balances**: Use \`get_account\` to clearly separate wallet balance vs trading-account balance (pass \`poolId\` for trading-account metrics).
+- **Adjust Margin**: \`adjust_margin.adjustAmount\` supports human amount (e.g. "1"), and also supports exact raw amount via \`raw:\` prefix.
+- **TP/SL Updates**: \`update_order_tp_sl\` accepts boolean-like values for \`useOrderCollateral\` and \`isTpSlOrder\`, but pending LIMIT/STOP orders can still be rejected by protocol rules; after fill, prefer \`set_tp_sl\`.
+- **Pool Reads**: \`get_pool_info\` auto-retries with oracle/ticker price when direct on-chain read returns divide-by-zero.
 - **Examples**: Follow \`TOOL_EXAMPLES.md\` payload patterns when building tool arguments.
 
 ## 3. Self-Healing

package/dist/server.js

@@ -11,6 +11,7 @@ import * as baseResources from "./resources/index.js";
 import * as basePrompts from "./prompts/index.js";
 import { logger } from "./utils/logger.js";
 import { MCPError, ErrorCode } from "./utils/errors.js";
+import { extractErrorMessage, isMeaningfulErrorMessage } from "./utils/errorMessage.js";
 // --- Process Logic Protection ---
 // Catch unhandled promise rejections
 process.on('unhandledRejection', (reason, promise) => {
@@ -36,16 +37,6 @@ const allPrompts = Object.values(basePrompts);
 function safeJsonStringify(value) {
     return JSON.stringify(value, (_, item) => (typeof item === "bigint" ? item.toString() : item), 2);
 }
-function extractErrorMessage(raw) {
-    if (raw instanceof Error)
-        return raw.message;
-    if (typeof raw === "string")
-        return raw;
-    if (raw && typeof raw === "object" && "message" in raw && typeof raw.message === "string") {
-        return String(raw.message);
-    }
-    return String(raw ?? "Unknown error");
-}
 function inferToolErrorCode(message) {
     const lower = message.toLowerCase();
     if (lower.includes("required") || lower.includes("invalid") || lower.includes("must be") || lower.includes("unexpected") || lower.includes("unrecognized")) {
@@ -134,14 +125,14 @@ function normalizeToolErrorResult(toolName, result) {
         parsed.status === "error" &&
         parsed.error &&
         typeof parsed.error.code === "string" &&
-        typeof parsed.error.message === "string") {
+        isMeaningfulErrorMessage(parsed.error.message)) {
         return result;
     }
-    const plainMessage = typeof parsed?.error?.message === "string"
+    const plainMessage = extractErrorMessage(typeof parsed?.error?.message === "string"
         ? parsed.error.message
         : typeof parsed?.message === "string"
             ? parsed.message
-            : rawText.replace(/^Error:\s*/i, "").trim();
+            : rawText.replace(/^Error:\s*/i, "").trim(), `Tool "${toolName}" failed.`);
     const code = typeof parsed?.error?.code === "string"
         ? parsed.error.code
         : inferToolErrorCode(plainMessage);
@@ -261,7 +252,7 @@ function zodSchemaToJsonSchema(zodSchema) {
     };
 }
 // ─── MCP Server ───
-const server = new Server({ name: "myx-mcp-trading-server", version: "3.0.0" }, { capabilities: { tools: {}, resources: {}, prompts: {} } });
+const server = new Server({ name: "myx-mcp-trading-server", version: "3.0.1" }, { capabilities: { tools: {}, resources: {}, prompts: {} } });
 // List tools
 server.setRequestHandler(ListToolsRequestSchema, async () => {
     return {
@@ -381,7 +372,7 @@ server.setRequestHandler(GetPromptRequestSchema, async (request) => {
 async function main() {
     const transport = new StdioServerTransport();
     await server.connect(transport);
-    logger.info("🚀 MYX Trading MCP Server v3.0.0 running (stdio, pure on-chain, prod ready)");
+    logger.info("🚀 MYX Trading MCP Server v3.0.1 running (stdio, pure on-chain, prod ready)");
 }
 main().catch((err) => {
     logger.error("Fatal Server Startup Error", err);

package/dist/services/poolService.js

@@ -1,5 +1,24 @@
 import { pool, quote, base } from "@myx-trade/sdk";
 import { getChainId, resolveClient } from "../auth/resolveClient.js";
+import { extractErrorMessage } from "../utils/errorMessage.js";
+import { ensureUnits } from "../utils/units.js";
+function isDivideByZeroError(message) {
+    const lower = message.toLowerCase();
+    return (lower.includes("divide_by_zero") ||
+        lower.includes("division by zero") ||
+        lower.includes("panic code 0x12") ||
+        lower.includes("panic code: 0x12") ||
+        lower.includes("0x4e487b71"));
+}
+function toPositiveBigint(input) {
+    try {
+        const value = BigInt(String(input ?? "").trim());
+        return value > 0n ? value : null;
+    }
+    catch {
+        return null;
+    }
+}
 /**
  * 创建合约市场池子
  */
@@ -11,9 +30,64 @@ export async function createPool(baseToken, marketId) {
 /**
  * 获取池子信息
  */
-export async function getPoolInfo(poolId, chainIdOverride) {
+export async function getPoolInfo(poolId, chainIdOverride, clientOverride) {
     const chainId = chainIdOverride ?? getChainId();
-    return pool.getPoolInfo(chainId, poolId);
+    let needOracleFallback = false;
+    try {
+        const direct = await pool.getPoolInfo(chainId, poolId);
+        if (direct)
+            return direct;
+        needOracleFallback = true;
+    }
+    catch (error) {
+        const message = extractErrorMessage(error);
+        if (!isDivideByZeroError(message)) {
+            throw new Error(`get_pool_info failed: ${message}`);
+        }
+        needOracleFallback = true;
+    }
+    if (!needOracleFallback)
+        return undefined;
+    const client = clientOverride ?? (await resolveClient()).client;
+    if (!client?.utils?.getOraclePrice) {
+        throw new Error("get_pool_info failed and oracle fallback is unavailable (client.utils.getOraclePrice missing).");
+    }
+    let oracleRaw = 0n;
+    try {
+        const oracle = await client.utils.getOraclePrice(poolId, chainId);
+        const byValue = toPositiveBigint(oracle?.value);
+        const byPrice = toPositiveBigint(oracle?.price);
+        oracleRaw = byValue ?? byPrice ?? 0n;
+    }
+    catch (error) {
+        throw new Error(`get_pool_info fallback failed to fetch oracle price: ${extractErrorMessage(error)}`);
+    }
+    if (oracleRaw <= 0n) {
+        try {
+            const tickerRes = await client.markets.getTickerList({ chainId, poolIds: [poolId] });
+            const row = Array.isArray(tickerRes) ? tickerRes[0] : tickerRes?.data?.[0];
+            if (row?.price) {
+                const tickerRaw = ensureUnits(row.price, 30, "ticker price");
+                const byTicker = toPositiveBigint(tickerRaw);
+                oracleRaw = byTicker ?? 0n;
+            }
+        }
+        catch {
+        }
+    }
+    if (oracleRaw <= 0n) {
+        throw new Error("get_pool_info fallback requires a positive oracle/ticker price, but both resolved to 0.");
+    }
+    try {
+        const retried = await pool.getPoolInfo(chainId, poolId, oracleRaw);
+        if (!retried) {
+            throw new Error(`Pool info for ${poolId} returned undefined after oracle-price retry.`);
+        }
+        return retried;
+    }
+    catch (error) {
+        throw new Error(`get_pool_info failed after oracle-price retry: ${extractErrorMessage(error)}`);
+    }
 }
 /**
  * 获取池子详情

package/dist/services/tradeService.js

@@ -1,9 +1,10 @@
 import { Direction, OrderType, TriggerType } from "@myx-trade/sdk";
-import { getChainId, getQuoteToken } from "../auth/resolveClient.js";
+import { getChainId, getQuoteToken, getQuoteDecimals } from "../auth/resolveClient.js";
 import { ensureUnits } from "../utils/units.js";
 import { normalizeAddress } from "../utils/address.js";
 import { normalizeSlippagePct4dp } from "../utils/slippage.js";
 import { finalizeMutationResult } from "../utils/mutationResult.js";
+import { extractErrorMessage } from "../utils/errorMessage.js";
 function resolveDirection(direction) {
     if (direction !== 0 && direction !== 1) {
         throw new Error("direction must be 0 (LONG) or 1 (SHORT).");
@@ -51,6 +52,74 @@ function resolveTpSlTriggerType(isTp, direction, triggerType) {
         return direction === 0 ? 2 : 1;
     }
 }
+function collectRows(input) {
+    if (Array.isArray(input))
+        return input.flatMap(collectRows);
+    if (!input || typeof input !== "object")
+        return [];
+    if (input.poolId || input.pool_id || input.marketId || input.market_id)
+        return [input];
+    return Object.values(input).flatMap(collectRows);
+}
+function parseDecimals(value, fallback) {
+    const parsed = Number(value);
+    if (Number.isFinite(parsed) && parsed >= 0) {
+        return Math.floor(parsed);
+    }
+    return fallback;
+}
+function normalizeIdentifier(value) {
+    return String(value ?? "").trim().toLowerCase();
+}
+async function resolveDecimalsForUpdateOrder(client, chainId, marketId, poolIdHint) {
+    let baseDecimals = 18;
+    let quoteDecimals = getQuoteDecimals();
+    let resolvedPoolId = String(poolIdHint ?? "").trim();
+    const hydrateFromPoolDetail = async (poolId) => {
+        if (!poolId)
+            return;
+        const detailRes = await client.markets.getMarketDetail({ chainId, poolId });
+        const detail = detailRes?.data || (detailRes?.marketId ? detailRes : null);
+        if (!detail)
+            return;
+        baseDecimals = parseDecimals(detail.baseDecimals, baseDecimals);
+        quoteDecimals = parseDecimals(detail.quoteDecimals, quoteDecimals);
+        resolvedPoolId = String(detail.poolId ?? poolId ?? resolvedPoolId);
+    };
+    if (resolvedPoolId) {
+        try {
+            await hydrateFromPoolDetail(resolvedPoolId);
+            return { baseDecimals, quoteDecimals, poolId: resolvedPoolId };
+        }
+        catch {
+            resolvedPoolId = "";
+        }
+    }
+    try {
+        const marketListRes = await client.api.getMarketList();
+        const rows = collectRows(marketListRes?.data ?? marketListRes);
+        const targetMarketId = normalizeIdentifier(marketId);
+        const row = rows.find((item) => normalizeIdentifier(item?.marketId ?? item?.market_id) === targetMarketId);
+        if (row) {
+            baseDecimals = parseDecimals(row?.baseDecimals ?? row?.base_decimals, baseDecimals);
+            quoteDecimals = parseDecimals(row?.quoteDecimals ?? row?.quote_decimals, quoteDecimals);
+            const fromRowPoolId = String(row?.poolId ?? row?.pool_id ?? "").trim();
+            if (fromRowPoolId) {
+                resolvedPoolId = fromRowPoolId;
+            }
+        }
+    }
+    catch {
+    }
+    if (resolvedPoolId) {
+        try {
+            await hydrateFromPoolDetail(resolvedPoolId);
+        }
+        catch {
+        }
+    }
+    return { baseDecimals, quoteDecimals, poolId: resolvedPoolId || undefined };
+}
 /**
  * 开仓 / 加仓
  */
@@ -249,9 +318,25 @@ export async function setPositionTpSl(client, address, args) {
 export async function adjustMargin(client, address, args) {
     const chainId = getChainId();
     const quoteToken = normalizeAddress(args.quoteToken || getQuoteToken(), "quoteToken");
-    const adjustAmount = String(args.adjustAmount ?? "").trim();
+    const adjustAmountInput = String(args.adjustAmount ?? "").trim();
+    if (!adjustAmountInput) {
+        throw new Error("adjustAmount is required.");
+    }
+    let quoteDecimals = getQuoteDecimals();
+    try {
+        const detailRes = await client.markets.getMarketDetail({ chainId, poolId: args.poolId });
+        const detail = detailRes?.data || (detailRes?.marketId ? detailRes : null);
+        const parsed = Number(detail?.quoteDecimals);
+        if (Number.isFinite(parsed) && parsed >= 0) {
+            quoteDecimals = parsed;
+        }
+    }
+    catch {
+        // Fallback to env quote decimals if market detail is unavailable.
+    }
+    const adjustAmount = ensureUnits(adjustAmountInput, quoteDecimals, "adjustAmount");
     if (!/^-?\d+$/.test(adjustAmount)) {
-        throw new Error("adjustAmount must be an integer string in quote token raw units.");
+        throw new Error("adjustAmount must resolve to an integer string (raw units).");
     }
     const params = {
         poolId: args.poolId,
@@ -319,16 +404,36 @@ export async function closeAllPositions(client, address) {
 export async function updateOrderTpSl(client, address, args) {
     const chainId = getChainId();
     const quoteToken = normalizeAddress(args.quoteToken, "quoteToken");
+    const marketId = String(args.marketId ?? "").trim();
+    const isTpSlOrder = typeof args.isTpSlOrder === "boolean" ? args.isTpSlOrder : true;
+    const { baseDecimals } = await resolveDecimalsForUpdateOrder(client, chainId, marketId, args.poolId);
     const params = {
         orderId: args.orderId,
-        tpSize: args.tpSize,
-        tpPrice: args.tpPrice,
-        slSize: args.slSize,
-        slPrice: args.slPrice,
-        useOrderCollateral: args.useOrderCollateral,
+        tpSize: ensureUnits(args.tpSize, baseDecimals, "tpSize"),
+        tpPrice: ensureUnits(args.tpPrice, 30, "tpPrice"),
+        slSize: ensureUnits(args.slSize, baseDecimals, "slSize"),
+        slPrice: ensureUnits(args.slPrice, 30, "slPrice"),
+        useOrderCollateral: Boolean(args.useOrderCollateral),
         executionFeeToken: quoteToken,
-        size: args.size,
-        price: args.price,
+        size: ensureUnits(args.size, baseDecimals, "size"),
+        price: ensureUnits(args.price, 30, "price"),
     };
-    return client.order.updateOrderTpSl(params, quoteToken, chainId, address, args.marketId, args.isTpSlOrder);
+    try {
+        const result = await client.order.updateOrderTpSl(params, quoteToken, chainId, address, marketId, isTpSlOrder);
+        if (Number(result?.code) === 0) {
+            return result;
+        }
+        const message = extractErrorMessage(result, "Failed to update order");
+        if (/failed to update order/i.test(message)) {
+            throw new Error(`Failed to update TP/SL for order ${args.orderId}. If this is a pending LIMIT/STOP order, wait for fill and then use set_tp_sl on the position.`);
+        }
+        throw new Error(`update_order_tp_sl failed: ${message}`);
+    }
+    catch (error) {
+        const message = extractErrorMessage(error, "Failed to update order");
+        if (/failed to update order/i.test(message)) {
+            throw new Error(`Failed to update TP/SL for order ${args.orderId}. If this is a pending LIMIT/STOP order, wait for fill and then use set_tp_sl on the position.`);
+        }
+        throw new Error(message);
+    }
 }

package/dist/tools/adjustMargin.js

@@ -8,7 +8,7 @@ export const adjustMarginTool = {
     schema: {
         poolId: z.string().describe("Pool ID"),
         positionId: z.string().describe("Position ID"),
-        adjustAmount: z.coerce.string().regex(/^-?\d+$/).describe("Quote token raw units. Positive = add, negative = remove."),
+        adjustAmount: z.union([z.string(), z.number()]).describe("Adjust amount. Human units are supported (e.g. '1' = 1 USDC). Use 'raw:<int>' for exact raw units."),
         quoteToken: z.string().optional().describe("Quote token address"),
         poolOracleType: z.coerce.number().optional().describe("Oracle type: 1 for Chainlink, 2 for Pyth"),
     },

package/dist/tools/cancelAllOrders.js

@@ -1,17 +1,49 @@
 import { z } from "zod";
 import { resolveClient, getChainId } from "../auth/resolveClient.js";
 import { finalizeMutationResult } from "../utils/mutationResult.js";
+function normalizeOrderIds(input) {
+    if (Array.isArray(input)) {
+        return input.map((id) => String(id).trim()).filter(Boolean);
+    }
+    if (typeof input === "string") {
+        const text = input.trim();
+        if (!text)
+            return [];
+        // Support toolchains that serialize arrays as JSON strings.
+        if (text.startsWith("[") && text.endsWith("]")) {
+            try {
+                const parsed = JSON.parse(text);
+                if (Array.isArray(parsed)) {
+                    return parsed.map((id) => String(id).trim()).filter(Boolean);
+                }
+            }
+            catch {
+                // Fallback to comma/single parsing below.
+            }
+        }
+        if (text.includes(",")) {
+            return text.split(",").map((id) => id.trim()).filter(Boolean);
+        }
+        return [text];
+    }
+    return [];
+}
 export const cancelAllOrdersTool = {
     name: "cancel_all_orders",
-    description: "Cancel multiple open orders by orderIds.",
+    description: "Cancel open orders by orderIds. Accepts array, JSON-array string, comma-separated string, or single orderId.",
     schema: {
-        orderIds: z.array(z.string()).min(1).describe("Order IDs to cancel."),
+        orderIds: z
+            .union([z.array(z.string()).min(1), z.string().min(1)])
+            .describe("Order IDs to cancel. Supports array or string (single/comma/JSON-array)."),
     },
     handler: async (args) => {
         try {
             const { client, signer } = await resolveClient();
             const chainId = getChainId();
-            const orderIds = args.orderIds;
+            const orderIds = normalizeOrderIds(args.orderIds);
+            if (orderIds.length === 0) {
+                throw new Error("orderIds is required and must include at least one non-empty order id.");
+            }
             const raw = await client.order.cancelAllOrders(orderIds, chainId);
             const result = await finalizeMutationResult(raw, signer, "cancel_all_orders");
             return {

package/dist/tools/manageLiquidity.js

@@ -3,6 +3,7 @@ import { quoteDeposit, quoteWithdraw, baseDeposit, baseWithdraw, getLpPrice, } f
 import { resolveClient } from "../auth/resolveClient.js";
 import { resolvePool } from "../services/marketService.js";
 import { finalizeMutationResult } from "../utils/mutationResult.js";
+import { extractErrorMessage } from "../utils/errorMessage.js";
 export const manageLiquidityTool = {
     name: "manage_liquidity",
     description: "Add or withdraw liquidity from a BASE or QUOTE pool.",
@@ -40,11 +41,14 @@ export const manageLiquidityTool = {
             if (!raw) {
                 throw new Error(`SDK returned an empty result for liquidity ${action}. This usually occurs if the pool is not in an Active state (state: 2) or if there is a contract-level restriction. Please check pool_info.`);
             }
+            if (raw && typeof raw === "object" && "code" in raw && Number(raw.code) !== 0) {
+                throw new Error(`Liquidity ${action} failed: ${extractErrorMessage(raw)}`);
+            }
             const data = await finalizeMutationResult(raw, signer, "manage_liquidity");
             return { content: [{ type: "text", text: JSON.stringify({ status: "success", data }, (_, v) => typeof v === 'bigint' ? v.toString() : v, 2) }] };
         }
         catch (error) {
-            return { content: [{ type: "text", text: `Error: ${error.message}` }], isError: true };
+            return { content: [{ type: "text", text: `Error: ${extractErrorMessage(error)}` }], isError: true };
         }
     },
 };
@@ -62,7 +66,7 @@ export const getLpPriceTool = {
             return { content: [{ type: "text", text: JSON.stringify({ status: "success", data }, (k, v) => typeof v === 'bigint' ? v.toString() : v, 2) }] };
         }
         catch (error) {
-            return { content: [{ type: "text", text: `Error: ${error.message}` }], isError: true };
+            return { content: [{ type: "text", text: `Error: ${extractErrorMessage(error)}` }], isError: true };
         }
     },
 };

package/dist/tools/marketInfo.js

@@ -2,6 +2,7 @@ import { z } from "zod";
 import { resolveClient, getChainId } from "../auth/resolveClient.js";
 import { getMarketDetail, resolvePool } from "../services/marketService.js";
 import { getPoolInfo, getLiquidityInfo } from "../services/poolService.js";
+import { extractErrorMessage } from "../utils/errorMessage.js";
 export const getMarketDetailTool = {
     name: "get_market_detail",
     description: "Get detailed information for a specific trading pool (fee rates, open interest, etc.). Supports PoolId, Token Address, or Keywords.",
@@ -20,7 +21,7 @@ export const getMarketDetailTool = {
             return { content: [{ type: "text", text: JSON.stringify({ status: "success", data }, (_, v) => typeof v === 'bigint' ? v.toString() : v, 2) }] };
         }
         catch (error) {
-            return { content: [{ type: "text", text: `Error: ${error.message}` }], isError: true };
+            return { content: [{ type: "text", text: `Error: ${extractErrorMessage(error)}` }], isError: true };
         }
     },
 };
@@ -36,13 +37,13 @@ export const getPoolInfoTool = {
             const { client } = await resolveClient();
             const poolId = await resolvePool(client, args.poolId);
             const chainId = args.chainId ?? getChainId();
-            const data = await getPoolInfo(poolId, chainId);
+            const data = await getPoolInfo(poolId, chainId, client);
             if (!data)
                 throw new Error(`Pool info for ${poolId} returned undefined. The pool may not exist on chainId ${chainId}.`);
             return { content: [{ type: "text", text: JSON.stringify({ status: "success", data }, (_, v) => typeof v === 'bigint' ? v.toString() : v, 2) }] };
         }
         catch (error) {
-            return { content: [{ type: "text", text: `Error: ${error.message}` }], isError: true };
+            return { content: [{ type: "text", text: `Error: ${extractErrorMessage(error)}` }], isError: true };
         }
     },
 };
@@ -65,7 +66,7 @@ export const getLiquidityInfoTool = {
             return { content: [{ type: "text", text: JSON.stringify({ status: "success", data }, (_, v) => typeof v === 'bigint' ? v.toString() : v, 2) }] };
         }
         catch (error) {
-            return { content: [{ type: "text", text: `Error: ${error.message}` }], isError: true };
+            return { content: [{ type: "text", text: `Error: ${extractErrorMessage(error)}` }], isError: true };
         }
     },
 };

package/dist/tools/updateOrderTpSl.js

@@ -2,20 +2,22 @@ import { z } from "zod";
 import { resolveClient } from "../auth/resolveClient.js";
 import { updateOrderTpSl } from "../services/tradeService.js";
 import { finalizeMutationResult } from "../utils/mutationResult.js";
+import { extractErrorMessage } from "../utils/errorMessage.js";
 export const updateOrderTpSlTool = {
     name: "update_order_tp_sl",
     description: "Update an existing take profit or stop loss order.",
     schema: {
         orderId: z.string().describe("The ID of the order to update"),
         marketId: z.string().describe("The market ID (config hash) for the order"),
-        size: z.string().describe("Order size (raw or human-readable)"),
-        price: z.string().describe("Order price (raw or human-readable, 30 decimals)"),
-        tpPrice: z.string().describe("TP price (raw or human-readable, 30 decimals)"),
-        tpSize: z.string().describe("TP size (raw or human-readable)"),
-        slPrice: z.string().describe("SL price (raw or human-readable, 30 decimals)"),
-        slSize: z.string().describe("SL size (raw or human-readable)"),
-        useOrderCollateral: z.boolean().describe("Whether to use order collateral"),
-        isTpSlOrder: z.boolean().optional().describe("Whether this is a TP/SL order"),
+        poolId: z.string().optional().describe("Optional poolId hint for decimal resolution."),
+        size: z.union([z.string(), z.number()]).describe("Order size (raw or human-readable)"),
+        price: z.union([z.string(), z.number()]).describe("Order price (raw or human-readable, 30 decimals)"),
+        tpPrice: z.union([z.string(), z.number()]).describe("TP price (raw or human-readable, 30 decimals)"),
+        tpSize: z.union([z.string(), z.number()]).describe("TP size (raw or human-readable)"),
+        slPrice: z.union([z.string(), z.number()]).describe("SL price (raw or human-readable, 30 decimals)"),
+        slSize: z.union([z.string(), z.number()]).describe("SL size (raw or human-readable)"),
+        useOrderCollateral: z.coerce.boolean().describe("Whether to use order collateral"),
+        isTpSlOrder: z.coerce.boolean().optional().describe("Whether this is a TP/SL order"),
         quoteToken: z.string().describe("Quote token address"),
     },
     handler: async (args) => {
@@ -26,7 +28,7 @@ export const updateOrderTpSlTool = {
             return { content: [{ type: "text", text: JSON.stringify({ status: "success", data }, (_, v) => typeof v === 'bigint' ? v.toString() : v, 2) }] };
         }
         catch (error) {
-            return { content: [{ type: "text", text: `Error: ${error.message}` }], isError: true };
+            return { content: [{ type: "text", text: `Error: ${extractErrorMessage(error)}` }], isError: true };
         }
     },
 };

package/dist/utils/errorMessage.js

@@ -0,0 +1,69 @@
+const USELESS_MESSAGE_SET = new Set([
+    "",
+    "undefined",
+    "null",
+    "[object object]",
+    "{}",
+]);
+function cleanMessage(input) {
+    if (typeof input !== "string")
+        return null;
+    const text = input.trim();
+    if (!text)
+        return null;
+    if (USELESS_MESSAGE_SET.has(text.toLowerCase()))
+        return null;
+    return text;
+}
+function isRecord(value) {
+    return !!value && typeof value === "object";
+}
+function safeStringify(value) {
+    try {
+        return cleanMessage(JSON.stringify(value, (_, item) => (typeof item === "bigint" ? item.toString() : item)));
+    }
+    catch {
+        return cleanMessage(String(value ?? ""));
+    }
+}
+export function isMeaningfulErrorMessage(message) {
+    return cleanMessage(message) !== null;
+}
+export function extractErrorMessage(error, fallback = "Unknown error") {
+    const visited = new Set();
+    const read = (value, depth) => {
+        if (depth > 6 || value === null || value === undefined)
+            return null;
+        if (typeof value === "string")
+            return cleanMessage(value);
+        if (typeof value === "number" || typeof value === "boolean" || typeof value === "bigint") {
+            return cleanMessage(String(value));
+        }
+        if (value instanceof Error) {
+            const message = cleanMessage(value.message);
+            if (message)
+                return message;
+            return read(value.cause, depth + 1);
+        }
+        if (!isRecord(value)) {
+            return cleanMessage(String(value));
+        }
+        if (visited.has(value))
+            return null;
+        visited.add(value);
+        const directKeys = ["message", "reason", "shortMessage", "msg", "detail", "error_description"];
+        for (const key of directKeys) {
+            const message = read(value[key], depth + 1);
+            if (message)
+                return message;
+        }
+        const nestedKeys = ["error", "data", "cause", "response", "info"];
+        for (const key of nestedKeys) {
+            const message = read(value[key], depth + 1);
+            if (message)
+                return message;
+        }
+        return safeStringify(value);
+    };
+    return read(error, 0) ?? fallback;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@michaleffffff/mcp-trading-server",
-  "version": "3.0.0",
+  "version": "3.0.1",
   "type": "module",
   "bin": {
     "myx-mcp": "dist/server.js"