@michaleffffff/mcp-trading-server 3.0.0 → 3.0.2

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # Changelog
 
+## 3.0.2 - 2026-03-17
+
+### Added
+- Integrated `verifyTradeOutcome` in `execute_trade` for post-transaction state validation.
+- Automatic `tradingFee` calculation in `execute_trade` using `getUserTradingFeeRate`.
+- `parseUserUnits` / `parseUserPrice30` utilities for more consistent human/raw input handling.
+
+### Changed
+- `execute_trade` now surfaces `preflight` normalization details in the successful response.
+- `adjust_margin` now supports displaying `__normalized` data.
+- Refined unit resolution in `account_deposit` and `account_withdraw`.
+
+## 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)
 
@@ -290,12 +291,13 @@ For practical tool-by-tool examples, see: `TOOL_EXAMPLES.md`
 
 ### Trading Operations
 * **open_position_simple**: High-level open position helper (recommended). Computes price/size/tradingFee internally.
-* **execute_trade**: Execute a new trade or add to an existing position.
+* **execute_trade**: Execute a new trade or add to an existing position (includes parameter preflight, supports `human:`/`raw:` amount prefixes, auto-computes `tradingFee` when omitted).
 * **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,15 @@ 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`.
+
+## Unit Safety (P0)
+
+- Monetary fields in major mutation tools support `human:` and `raw:` prefixes.
+- Default behavior is human-readable units (e.g. `1` means `1.0` token), while `raw:<int>` enforces exact raw units.
+- `execute_trade` and `adjust_margin` responses include normalized raw values for auditability.
 
 Minimal query examples:
 

package/TOOL_EXAMPLES.md

@@ -80,11 +80,14 @@ It includes:
     "slippagePct": "100",
     "executionFeeToken": "0xQUOTE_TOKEN",
     "leverage": 5,
-    "tradingFee": "raw:100000",
     "marketId": "0xMARKET_ID"
   }
 }
 ```
+Notes:
+- `tradingFee` is optional (auto-computed if omitted).
+- Amount-like fields support `human:` and `raw:` prefixes.
+- `positionId: "0x000...000"` is auto-treated as a new position.
 
 ### `close_position`
 ```json
@@ -140,6 +143,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 +192,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,10 +222,22 @@ 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"
+  }
+}
+```
+Success payload includes normalized raw amount under `data.normalized.adjustAmountRaw`.
 
 ### `check_approval`
 ```json
@@ -295,6 +341,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 +549,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

@@ -26,9 +26,14 @@ You are an expert crypto trader using the MYX Protocol. To ensure successful exe
 ## 2. Parameter Tips
 - **Position IDs**: When opening a NEW position, \`positionId\` MUST be an empty string \`""\`.
 - **Decimals**: Human-readable units (e.g., "0.1" BTC) are default for \`open_position_simple\`. SDK-native tools often require raw units; use the \`raw:\` prefix if you need forced precision.
+- **Unit Prefixes**: For mutation tools, prefer explicit prefixes: \`human:\` for readable amounts, \`raw:\` for exact on-chain units.
 - **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.
+- **execute_trade**: Has built-in preflight normalization and can auto-compute \`tradingFee\` when omitted.
 - **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")) {
@@ -112,6 +103,102 @@ function parseFirstText(result) {
     const firstText = result.content.find((item) => item?.type === "text" && typeof item.text === "string");
     return firstText?.text ?? null;
 }
+function unwrapSchema(schema) {
+    let current = schema;
+    for (let i = 0; i < 8; i++) {
+        const type = current?._def?.type;
+        if (!type)
+            break;
+        if (type === "optional" || type === "nullable" || type === "default" || type === "prefault" || type === "catch") {
+            current = current?._def?.innerType;
+            continue;
+        }
+        if (type === "pipe") {
+            current = current?._def?.out;
+            continue;
+        }
+        break;
+    }
+    return current;
+}
+function coerceBooleanString(input) {
+    if (typeof input !== "string")
+        return input;
+    const normalized = input.trim().toLowerCase();
+    if (normalized === "true" || normalized === "1")
+        return true;
+    if (normalized === "false" || normalized === "0")
+        return false;
+    return input;
+}
+function coerceStringToStringArray(input) {
+    if (Array.isArray(input))
+        return input.map((item) => String(item).trim()).filter(Boolean);
+    if (typeof input !== "string")
+        return input;
+    const text = input.trim();
+    if (!text)
+        return input;
+    if (text.startsWith("[") && text.endsWith("]")) {
+        try {
+            const parsed = JSON.parse(text);
+            if (Array.isArray(parsed)) {
+                return parsed.map((item) => String(item).trim()).filter(Boolean);
+            }
+        }
+        catch {
+        }
+    }
+    if (text.includes(",")) {
+        const split = text.split(",").map((item) => item.trim()).filter(Boolean);
+        if (split.length > 0)
+            return split;
+    }
+    return input;
+}
+function coerceBySchema(value, schema) {
+    const unwrapped = unwrapSchema(schema);
+    const type = unwrapped?._def?.type;
+    if (type === "boolean") {
+        return coerceBooleanString(value);
+    }
+    if (type === "array") {
+        const elementType = unwrapSchema(unwrapped?._def?.element)?._def?.type;
+        if (elementType === "string") {
+            return coerceStringToStringArray(value);
+        }
+    }
+    if (type === "union") {
+        const options = Array.isArray(unwrapped?._def?.options) ? unwrapped._def.options : [];
+        for (const option of options) {
+            const optionType = unwrapSchema(option)?._def?.type;
+            if (optionType === "boolean") {
+                const coerced = coerceBooleanString(value);
+                if (typeof coerced === "boolean")
+                    return coerced;
+            }
+            if (optionType === "array") {
+                const coerced = coerceStringToStringArray(value);
+                if (Array.isArray(coerced))
+                    return coerced;
+            }
+        }
+    }
+    return value;
+}
+function normalizeToolArgsBySchema(rawArgs, schema) {
+    const source = rawArgs && typeof rawArgs === "object" ? { ...rawArgs } : {};
+    const normalized = {};
+    for (const key of Object.keys(source)) {
+        normalized[key] = source[key];
+    }
+    for (const [key, fieldSchema] of Object.entries(schema)) {
+        if (!Object.prototype.hasOwnProperty.call(source, key))
+            continue;
+        normalized[key] = coerceBySchema(source[key], fieldSchema);
+    }
+    return normalized;
+}
 function normalizeToolErrorResult(toolName, result) {
     const rawText = parseFirstText(result);
     if (!rawText) {
@@ -134,14 +221,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);
@@ -217,27 +304,30 @@ function normalizeSdkReadFailure(toolName, result) {
 function zodSchemaToJsonSchema(zodSchema) {
     const toPropSchema = (value) => {
         const def = value?._def;
-        const typeName = def?.typeName;
-        if (typeName === "ZodOptional" || typeName === "ZodNullable") {
+        const zodType = def?.type;
+        if (zodType === "optional" || zodType === "nullable" || zodType === "default" || zodType === "prefault" || zodType === "catch") {
             return toPropSchema(def?.innerType);
         }
-        if (typeName === "ZodString")
+        if (zodType === "pipe") {
+            return toPropSchema(def?.out);
+        }
+        if (zodType === "string")
             return { type: "string" };
-        if (typeName === "ZodNumber")
+        if (zodType === "number")
             return { type: "number" };
-        if (typeName === "ZodBoolean")
+        if (zodType === "boolean")
             return { type: "boolean" };
-        if (typeName === "ZodArray") {
-            return { type: "array", items: toPropSchema(def?.type) };
+        if (zodType === "array") {
+            return { type: "array", items: toPropSchema(def?.element) };
         }
-        if (typeName === "ZodLiteral") {
+        if (zodType === "literal") {
             return { const: def?.value };
         }
-        if (typeName === "ZodUnion") {
+        if (zodType === "union") {
             const options = def?.options || [];
             return { anyOf: options.map((opt) => toPropSchema(opt)) };
         }
-        if (typeName === "ZodObject") {
+        if (zodType === "object") {
             return { type: "object" };
         }
         return { type: "string" };
@@ -246,7 +336,8 @@ function zodSchemaToJsonSchema(zodSchema) {
     const required = [];
     for (const [key, value] of Object.entries(zodSchema)) {
         const desc = value?._def?.description || "";
-        const isOptional = value?.isOptional?.() || value?._def?.typeName === "ZodOptional";
+        const fieldType = value?._def?.type;
+        const isOptional = value?.isOptional?.() || fieldType === "optional" || fieldType === "default";
         const prop = toPropSchema(value);
         if (desc)
             prop.description = desc;
@@ -261,7 +352,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.2" }, { capabilities: { tools: {}, resources: {}, prompts: {} } });
 // List tools
 server.setRequestHandler(ListToolsRequestSchema, async () => {
     return {
@@ -288,8 +379,9 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
     try {
         let validatedArgs = args ?? {};
         if (tool.schema) {
+            const normalizedArgs = normalizeToolArgsBySchema(args ?? {}, tool.schema);
             try {
-                validatedArgs = z.object(tool.schema).strict().parse(args ?? {});
+                validatedArgs = z.object(tool.schema).strict().parse(normalizedArgs);
             }
             catch (validationError) {
                 if (validationError instanceof ZodError) {
@@ -381,7 +473,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.2 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 };
+}
 /**
  * 开仓 / 加仓
  */
@@ -67,7 +136,7 @@ export async function openPosition(client, address, args) {
     }
     const baseDecimals = poolData.baseDecimals || 18;
     const quoteDecimals = poolData.quoteDecimals || 6;
-    const collateralRaw = ensureUnits(args.collateralAmount, quoteDecimals, "collateralAmount");
+    const collateralRaw = ensureUnits(args.collateralAmount, quoteDecimals, "collateralAmount", { allowImplicitRaw: false });
     // --- Pre-flight Check: minOrderSizeInUsd ---
     try {
         const levelRes = await client.markets.getPoolLevelConfig(args.poolId, chainId);
@@ -94,9 +163,9 @@ export async function openPosition(client, address, args) {
             throw e;
         console.warn(`[tradeService] Limit check skipped: ${e.message}`);
     }
-    const sizeRaw = ensureUnits(args.size, baseDecimals, "size");
-    const priceRaw = ensureUnits(args.price, 30, "price");
-    const tradingFeeRaw = ensureUnits(args.tradingFee, quoteDecimals, "tradingFee");
+    const sizeRaw = ensureUnits(args.size, baseDecimals, "size", { allowImplicitRaw: false });
+    const priceRaw = ensureUnits(args.price, 30, "price", { allowImplicitRaw: false });
+    const tradingFeeRaw = ensureUnits(args.tradingFee, quoteDecimals, "tradingFee", { allowImplicitRaw: false });
     // --- Auto-Deposit Logic (Strict) ---
     const allowAutoDeposit = args.autoDeposit !== false;
     console.log(`[tradeService] Checking marginBalance for ${address} in pool ${args.poolId}...`);
@@ -158,13 +227,13 @@ export async function openPosition(client, address, args) {
         leverage: args.leverage,
     };
     if (args.tpSize)
-        orderParams.tpSize = ensureUnits(args.tpSize, baseDecimals, "tpSize");
+        orderParams.tpSize = ensureUnits(args.tpSize, baseDecimals, "tpSize", { allowImplicitRaw: false });
     if (args.tpPrice)
-        orderParams.tpPrice = ensureUnits(args.tpPrice, 30, "tpPrice");
+        orderParams.tpPrice = ensureUnits(args.tpPrice, 30, "tpPrice", { allowImplicitRaw: false });
     if (args.slSize)
-        orderParams.slSize = ensureUnits(args.slSize, baseDecimals, "slSize");
+        orderParams.slSize = ensureUnits(args.slSize, baseDecimals, "slSize", { allowImplicitRaw: false });
     if (args.slPrice)
-        orderParams.slPrice = ensureUnits(args.slPrice, 30, "slPrice");
+        orderParams.slPrice = ensureUnits(args.slPrice, 30, "slPrice", { allowImplicitRaw: false });
     return client.order.createIncreaseOrder(orderParams, tradingFeeRaw, args.marketId);
 }
 /**
@@ -194,9 +263,9 @@ export async function closePosition(client, address, args) {
         orderType: args.orderType,
         triggerType: resolveTriggerType(args.orderType, args.direction, true, args.triggerType),
         direction: dir,
-        collateralAmount: ensureUnits(args.collateralAmount, quoteDecimals, "collateralAmount"),
-        size: ensureUnits(args.size, baseDecimals, "size"),
-        price: ensureUnits(args.price, 30, "price"),
+        collateralAmount: ensureUnits(args.collateralAmount, quoteDecimals, "collateralAmount", { allowImplicitRaw: false }),
+        size: ensureUnits(args.size, baseDecimals, "size", { allowImplicitRaw: false }),
+        price: ensureUnits(args.price, 30, "price", { allowImplicitRaw: false }),
         timeInForce: args.timeInForce,
         postOnly: args.postOnly,
         slippagePct: normalizeSlippagePct4dp(args.slippagePct),
@@ -234,13 +303,13 @@ export async function setPositionTpSl(client, address, args) {
         slippagePct: normalizeSlippagePct4dp(args.slippagePct),
     };
     if (args.tpPrice)
-        params.tpPrice = ensureUnits(args.tpPrice, 30, "tpPrice");
+        params.tpPrice = ensureUnits(args.tpPrice, 30, "tpPrice", { allowImplicitRaw: false });
     if (args.tpSize)
-        params.tpSize = ensureUnits(args.tpSize, baseDecimals, "tpSize");
+        params.tpSize = ensureUnits(args.tpSize, baseDecimals, "tpSize", { allowImplicitRaw: false });
     if (args.slPrice)
-        params.slPrice = ensureUnits(args.slPrice, 30, "slPrice");
+        params.slPrice = ensureUnits(args.slPrice, 30, "slPrice", { allowImplicitRaw: false });
     if (args.slSize)
-        params.slSize = ensureUnits(args.slSize, baseDecimals, "slSize");
+        params.slSize = ensureUnits(args.slSize, baseDecimals, "slSize", { allowImplicitRaw: false });
     return client.order.createPositionTpSlOrder(params);
 }
 /**
@@ -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", { allowImplicitRaw: false });
     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,
@@ -264,7 +349,14 @@ export async function adjustMargin(client, address, args) {
     if (args.poolOracleType !== undefined) {
         params.poolOracleType = Number(args.poolOracleType);
     }
-    return client.position.adjustCollateral(params);
+    const result = await client.position.adjustCollateral(params);
+    if (result && typeof result === "object") {
+        result.__normalized = {
+            adjustAmountRaw: adjustAmount,
+            quoteToken,
+        };
+    }
+    return result;
 }
 /**
  * 平掉所有仓位
@@ -319,16 +411,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", { allowImplicitRaw: false }),
+        tpPrice: ensureUnits(args.tpPrice, 30, "tpPrice", { allowImplicitRaw: false }),
+        slSize: ensureUnits(args.slSize, baseDecimals, "slSize", { allowImplicitRaw: false }),
+        slPrice: ensureUnits(args.slPrice, 30, "slPrice", { allowImplicitRaw: false }),
+        useOrderCollateral: Boolean(args.useOrderCollateral),
         executionFeeToken: quoteToken,
-        size: args.size,
-        price: args.price,
+        size: ensureUnits(args.size, baseDecimals, "size", { allowImplicitRaw: false }),
+        price: ensureUnits(args.price, 30, "price", { allowImplicitRaw: false }),
     };
-    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/accountTransfer.js

@@ -18,7 +18,7 @@ export const accountDepositTool = {
             // ensureUnits handles 'raw:' prefix if absolute precision is needed.
             const { ensureUnits } = await import("../utils/units.js");
             const { getQuoteDecimals } = await import("../auth/resolveClient.js");
-            const amount = ensureUnits(args.amount, getQuoteDecimals(), "amount");
+            const amount = ensureUnits(args.amount, getQuoteDecimals(), "amount", { allowImplicitRaw: false });
             const raw = await client.account.deposit({
                 amount,
                 tokenAddress,
@@ -48,7 +48,7 @@ export const accountWithdrawTool = {
             const { getQuoteDecimals } = await import("../auth/resolveClient.js");
             // Assuming 18 decimals for base and quoteDecimals for quote
             const decimals = args.isQuoteToken ? getQuoteDecimals() : 18;
-            const amount = ensureUnits(args.amount, decimals, "amount");
+            const amount = ensureUnits(args.amount, decimals, "amount", { allowImplicitRaw: false });
             const raw = await client.account.withdraw({
                 chainId,
                 receiver: address,

package/dist/tools/adjustMargin.js

@@ -2,13 +2,14 @@ import { z } from "zod";
 import { resolveClient } from "../auth/resolveClient.js";
 import { adjustMargin as adjustMarginSvc } from "../services/tradeService.js";
 import { finalizeMutationResult } from "../utils/mutationResult.js";
+import { extractErrorMessage } from "../utils/errorMessage.js";
 export const adjustMarginTool = {
     name: "adjust_margin",
     description: "Adjust the margin (collateral) of an open position.",
     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"),
     },
@@ -16,11 +17,16 @@ export const adjustMarginTool = {
         try {
             const { client, address, signer } = await resolveClient();
             const raw = await adjustMarginSvc(client, address, args);
+            const normalized = raw?.__normalized;
+            if (raw && typeof raw === "object" && "__normalized" in raw) {
+                delete raw.__normalized;
+            }
             const data = await finalizeMutationResult(raw, signer, "adjust_margin");
-            return { content: [{ type: "text", text: JSON.stringify({ status: "success", data }, (_, v) => typeof v === 'bigint' ? v.toString() : v, 2) }] };
+            const payload = normalized ? { ...data, normalized } : data;
+            return { content: [{ type: "text", text: JSON.stringify({ status: "success", data: payload }, (_, 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/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/executeTrade.js

@@ -1,11 +1,14 @@
 import { z } from "zod";
-import { resolveClient } from "../auth/resolveClient.js";
+import { resolveClient, getChainId } from "../auth/resolveClient.js";
 import { openPosition } from "../services/tradeService.js";
 import { finalizeMutationResult } from "../utils/mutationResult.js";
-import { SLIPPAGE_PCT_4DP_DESC } from "../utils/slippage.js";
+import { normalizeSlippagePct4dp, SLIPPAGE_PCT_4DP_DESC } from "../utils/slippage.js";
 import { verifyTradeOutcome } from "../utils/verification.js";
 import { mapDirection, mapOrderType, mapTriggerType } from "../utils/mappings.js";
+import { extractErrorMessage } from "../utils/errorMessage.js";
+import { parseUserUnits } from "../utils/units.js";
 const POSITION_ID_RE = /^$|^0x[0-9a-fA-F]{64}$/;
+const ZERO_POSITION_ID_RE = /^0x0{64}$/i;
 export const executeTradeTool = {
     name: "execute_trade",
     description: "Create an increase order using SDK-native parameters.",
@@ -13,7 +16,7 @@ export const executeTradeTool = {
         poolId: z.string().describe("Hex Pool ID, e.g. '0x14a19...'. Get via get_pool_list."),
         positionId: z.string().refine((value) => POSITION_ID_RE.test(value), {
             message: "positionId must be empty string for new position, or a bytes32 hex string.",
-        }).describe("Position ID: Use empty string '' for NEW positions, or valid hex for INCREASING existing ones."),
+        }).describe("Position ID: Use empty string '' for NEW positions, or valid hex for INCREASING existing ones. 0x000..00 is auto-treated as NEW."),
         orderType: z.union([z.number(), z.string()]).describe("Market/Limit/Stop. e.g. 0 or 'MARKET'."),
         triggerType: z.union([z.number(), z.string()]).optional().describe("0=None (Market), 1=GTE, 2=LTE. e.g. 'GTE'."),
         direction: z.union([z.number(), z.string()]).describe("0/LONG/BUY or 1/SHORT/SELL."),
@@ -24,33 +27,105 @@ export const executeTradeTool = {
         postOnly: z.coerce.boolean().describe("If true, order only executes as Maker."),
         slippagePct: z.coerce.string().default("50").describe(`${SLIPPAGE_PCT_4DP_DESC}. Default is 50 (0.5%).`),
         executionFeeToken: z.string().describe("Address of token to pay gas/execution fees (typically USDC)."),
-        leverage: z.coerce.number().describe("Leverage multiplier, e.g., 10 for 10x."),
+        leverage: z.coerce.number().positive().describe("Leverage multiplier, e.g., 10 for 10x."),
         tpSize: z.union([z.string(), z.number()]).optional().describe("Take Profit size. Use '0' to disable."),
         tpPrice: z.union([z.string(), z.number()]).optional().describe("Take Profit trigger price."),
         slSize: z.union([z.string(), z.number()]).optional().describe("Stop Loss size. Use '0' to disable."),
         slPrice: z.union([z.string(), z.number()]).optional().describe("Stop Loss trigger price."),
-        tradingFee: z.union([z.string(), z.number()]).describe("Estimated fee in raw units. Fetch via get_user_trading_fee_rate."),
+        tradingFee: z.union([z.string(), z.number()]).optional().describe("Trading fee in quote token units. Supports human/raw prefix. Optional: auto-computed via get_user_trading_fee_rate."),
+        assetClass: z.coerce.number().int().nonnegative().optional().describe("Optional fee lookup assetClass (default from pool config or 1)."),
+        riskTier: z.coerce.number().int().nonnegative().optional().describe("Optional fee lookup riskTier (default from pool config or 1)."),
         marketId: z.string().describe("Specific Market Config Hash. Fetch via get_market_list."),
     },
     handler: async (args) => {
         try {
             const { client, address, signer } = await resolveClient();
+            const chainId = getChainId();
+            const normalizedPositionId = args.positionId === "0" || !args.positionId || ZERO_POSITION_ID_RE.test(String(args.positionId))
+                ? ""
+                : String(args.positionId);
             const poolId = args.poolId;
             // Fetch pool detail to get quoteToken for execution fee
-            const poolResponse = await client.markets.getMarketDetail({ chainId: await resolveClient().then(r => r.chainId), poolId });
+            const poolResponse = await client.markets.getMarketDetail({ chainId, poolId });
             const poolData = poolResponse?.data || (poolResponse?.marketId ? poolResponse : null);
             if (!poolData)
                 throw new Error(`Could not find pool metadata for ID: ${poolId}`);
+            const baseDecimals = Number(poolData.baseDecimals ?? 18);
+            const quoteDecimals = Number(poolData.quoteDecimals ?? 6);
+            const mappedDirection = mapDirection(args.direction);
+            const mappedOrderType = mapOrderType(args.orderType);
+            const mappedTriggerType = args.triggerType !== undefined ? mapTriggerType(args.triggerType) : undefined;
+            const slippagePctNormalized = normalizeSlippagePct4dp(args.slippagePct);
+            const collateralRaw = parseUserUnits(args.collateralAmount, quoteDecimals, "collateralAmount");
+            const sizeRaw = parseUserUnits(args.size, baseDecimals, "size");
+            const priceRaw = parseUserUnits(args.price, 30, "price");
+            if (BigInt(collateralRaw) <= 0n)
+                throw new Error("collateralAmount must be > 0.");
+            if (BigInt(sizeRaw) <= 0n)
+                throw new Error("size must be > 0.");
+            if (BigInt(priceRaw) <= 0n)
+                throw new Error("price must be > 0.");
+            let tradingFeeRaw = "";
+            let tradingFeeMeta = { source: "user" };
+            const tradingFeeInput = String(args.tradingFee ?? "").trim();
+            if (tradingFeeInput) {
+                tradingFeeRaw = parseUserUnits(tradingFeeInput, quoteDecimals, "tradingFee");
+                if (BigInt(tradingFeeRaw) < 0n)
+                    throw new Error("tradingFee must be >= 0.");
+            }
+            else {
+                let poolAssetClass = 1;
+                let poolRiskTier = 1;
+                try {
+                    const levelRes = await client.markets.getPoolLevelConfig(poolId, chainId);
+                    const levelConfig = levelRes?.levelConfig || levelRes?.data?.levelConfig || {};
+                    if (Number.isFinite(Number(levelConfig.assetClass))) {
+                        poolAssetClass = Number(levelConfig.assetClass);
+                    }
+                    if (Number.isFinite(Number(levelConfig.riskTier))) {
+                        poolRiskTier = Number(levelConfig.riskTier);
+                    }
+                }
+                catch {
+                }
+                const assetClass = Number(args.assetClass ?? poolAssetClass ?? 1);
+                const riskTier = Number(args.riskTier ?? poolRiskTier ?? 1);
+                const feeRes = await client.utils.getUserTradingFeeRate(assetClass, riskTier, chainId);
+                if (Number(feeRes?.code) !== 0 || !feeRes?.data) {
+                    throw new Error(`Failed to compute tradingFee automatically (assetClass=${assetClass}, riskTier=${riskTier}). Provide tradingFee manually.`);
+                }
+                const rateRaw = args.postOnly ? feeRes.data.makerFeeRate : feeRes.data.takerFeeRate;
+                const rateBig = BigInt(String(rateRaw ?? "0"));
+                tradingFeeRaw = ((BigInt(collateralRaw) * rateBig) / 1000000n).toString();
+                tradingFeeMeta = { source: "computed", assetClass, riskTier, feeRate: String(rateRaw ?? "0") };
+            }
             const mappedArgs = {
                 ...args,
-                direction: mapDirection(args.direction),
-                orderType: mapOrderType(args.orderType),
-                triggerType: args.triggerType !== undefined ? mapTriggerType(args.triggerType) : undefined,
+                direction: mappedDirection,
+                orderType: mappedOrderType,
+                triggerType: mappedTriggerType,
                 // Normalize positionId
-                positionId: (args.positionId === "0" || !args.positionId) ? "" : args.positionId,
+                positionId: normalizedPositionId,
                 // Enforce executionFeeToken as quoteToken
-                executionFeeToken: poolData.quoteToken || args.executionFeeToken
+                executionFeeToken: poolData.quoteToken || args.executionFeeToken,
+                collateralAmount: `raw:${collateralRaw}`,
+                size: `raw:${sizeRaw}`,
+                price: `raw:${priceRaw}`,
+                tradingFee: `raw:${tradingFeeRaw}`,
+                slippagePct: slippagePctNormalized,
             };
+            if (args.tpSize !== undefined) {
+                mappedArgs.tpSize = `raw:${parseUserUnits(args.tpSize, baseDecimals, "tpSize")}`;
+            }
+            if (args.tpPrice !== undefined) {
+                mappedArgs.tpPrice = `raw:${parseUserUnits(args.tpPrice, 30, "tpPrice")}`;
+            }
+            if (args.slSize !== undefined) {
+                mappedArgs.slSize = `raw:${parseUserUnits(args.slSize, baseDecimals, "slSize")}`;
+            }
+            if (args.slPrice !== undefined) {
+                mappedArgs.slPrice = `raw:${parseUserUnits(args.slPrice, 30, "slPrice")}`;
+            }
             const raw = await openPosition(client, address, mappedArgs);
             const data = await finalizeMutationResult(raw, signer, "execute_trade");
             const txHash = data.confirmation?.txHash;
@@ -58,11 +133,27 @@ export const executeTradeTool = {
             if (txHash) {
                 verification = await verifyTradeOutcome(client, address, args.poolId, txHash);
             }
-            const payload = { ...data, verification };
+            const payload = {
+                ...data,
+                verification,
+                preflight: {
+                    normalized: {
+                        collateralAmountRaw: collateralRaw,
+                        sizeRaw,
+                        priceRaw30: priceRaw,
+                        tradingFeeRaw,
+                        tpSizeRaw: mappedArgs.tpSize?.replace(/^raw:/i, "") ?? null,
+                        tpPriceRaw30: mappedArgs.tpPrice?.replace(/^raw:/i, "") ?? null,
+                        slSizeRaw: mappedArgs.slSize?.replace(/^raw:/i, "") ?? null,
+                        slPriceRaw30: mappedArgs.slPrice?.replace(/^raw:/i, "") ?? null,
+                    },
+                    tradingFeeMeta,
+                },
+            };
             return { content: [{ type: "text", text: JSON.stringify({ status: "success", data: payload }, (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/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/dist/utils/units.js

@@ -1,6 +1,7 @@
 import { parseUnits } from "ethers";
 const DECIMAL_RE = /^-?\d+(\.\d+)?$/;
 const RAW_PREFIX_RE = /^raw:/i;
+const HUMAN_PREFIX_RE = /^human:/i;
 const INTEGER_RE = /^-?\d+$/;
 function normalizeDecimal(input) {
     let value = input.trim();
@@ -19,10 +20,14 @@ function normalizeDecimal(input) {
     }
     return `${sign}${intPart || "0"}.${fracPart}`;
 }
-export function ensureUnits(value, decimals, label = "value") {
+export function ensureUnits(value, decimals, label = "value", options = {}) {
+    const allowImplicitRaw = options.allowImplicitRaw ?? true;
     let str = String(value).trim();
     if (!str)
         throw new Error(`${label} is required.`);
+    if (HUMAN_PREFIX_RE.test(str)) {
+        str = str.replace(HUMAN_PREFIX_RE, "").trim();
+    }
     if (RAW_PREFIX_RE.test(str)) {
         const raw = str.replace(RAW_PREFIX_RE, "").trim();
         if (!INTEGER_RE.test(raw))
@@ -39,9 +44,8 @@ export function ensureUnits(value, decimals, label = "value") {
             str = `${parts[0]}.${parts[1].slice(0, decimals)}`;
         }
     }
-    // If it's already a very large integer (e.g. > 12 digits or > decimals digits), 
-    // assume it's already in the smallest unit (Wei/Raw).
-    if (!str.includes(".") && (str.length > 12 || str.length > decimals)) {
+    // Legacy compatibility: optionally treat very large integers as already raw.
+    if (allowImplicitRaw && !str.includes(".") && (str.length > 12 || str.length > decimals)) {
         return str;
     }
     try {
@@ -53,9 +57,12 @@ export function ensureUnits(value, decimals, label = "value") {
     }
 }
 export function parseUserUnits(value, decimals, label = "value") {
-    const str = String(value).trim();
+    let str = String(value).trim();
     if (!str)
         throw new Error(`${label} is required.`);
+    if (HUMAN_PREFIX_RE.test(str)) {
+        str = str.replace(HUMAN_PREFIX_RE, "").trim();
+    }
     if (RAW_PREFIX_RE.test(str)) {
         const raw = str.replace(RAW_PREFIX_RE, "").trim();
         if (!INTEGER_RE.test(raw))

package/package.json

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