@michaleffffff/mcp-trading-server 2.8.2 → 2.9.2

package/dist/server.js

@@ -81,7 +81,7 @@ function zodSchemaToJsonSchema(zodSchema) {
     };
 }
 // ─── MCP Server ───
-const server = new Server({ name: "myx-mcp-trading-server", version: "2.8.2" }, { capabilities: { tools: {}, resources: {}, prompts: {} } });
+const server = new Server({ name: "myx-mcp-trading-server", version: "2.9.2" }, { capabilities: { tools: {}, resources: {}, prompts: {} } });
 // List tools
 server.setRequestHandler(ListToolsRequestSchema, async () => {
     return {
@@ -181,7 +181,7 @@ server.setRequestHandler(GetPromptRequestSchema, async (request) => {
 async function main() {
     const transport = new StdioServerTransport();
     await server.connect(transport);
-    logger.info("🚀 MYX Trading MCP Server v2.8.2 running (stdio, pure on-chain, prod ready)");
+    logger.info("🚀 MYX Trading MCP Server v2.9.2 running (stdio, pure on-chain, prod ready)");
 }
 main().catch((err) => {
     logger.error("Fatal Server Startup Error", err);

package/dist/services/tradeService.js

@@ -13,16 +13,24 @@ function resolveDirection(direction) {
 /**
  * 自动推断开启订单的触发类型 (Limit/Stop)
  */
-function resolveTriggerType(orderType, direction, triggerType) {
+function resolveTriggerType(orderType, direction, isDecrease = false, triggerType) {
     if (triggerType !== undefined && triggerType !== null && triggerType !== 0) {
         return triggerType;
     }
-    // LIMIT LONG: LTE(2), LIMIT SHORT: GTE(1)
+    // Opening: LIMIT LONG -> LTE(2), LIMIT SHORT -> GTE(1)
+    // Closing: LIMIT LONG -> GTE(1), LIMIT SHORT -> LTE(2)
     if (orderType === OrderType.LIMIT) {
+        if (isDecrease) {
+            return direction === 0 ? 1 : 2;
+        }
         return direction === 0 ? 2 : 1;
     }
-    // STOP LONG: GTE(1), STOP SHORT: LTE(2)
+    // Opening: STOP LONG -> GTE(1), STOP SHORT -> LTE(2)
+    // Closing: STOP LONG -> LTE(2), STOP SHORT -> GTE(1)
     if (orderType === OrderType.STOP) {
+        if (isDecrease) {
+            return direction === 0 ? 2 : 1;
+        }
         return direction === 0 ? 1 : 2;
     }
     return 0; // MARKET order typically uses 0
@@ -138,7 +146,7 @@ export async function openPosition(client, address, args) {
         poolId: args.poolId,
         positionId: args.positionId,
         orderType: args.orderType,
-        triggerType: resolveTriggerType(args.orderType, args.direction, args.triggerType),
+        triggerType: resolveTriggerType(args.orderType, args.direction, false, args.triggerType),
         direction: dir,
         collateralAmount: collateralRaw,
         size: sizeRaw,
@@ -184,7 +192,7 @@ export async function closePosition(client, address, args) {
         poolId: args.poolId,
         positionId: args.positionId,
         orderType: args.orderType,
-        triggerType: resolveTriggerType(args.orderType, args.direction, args.triggerType),
+        triggerType: resolveTriggerType(args.orderType, args.direction, true, args.triggerType),
         direction: dir,
         collateralAmount: ensureUnits(args.collateralAmount, quoteDecimals, "collateralAmount"),
         size: ensureUnits(args.size, baseDecimals, "size"),

package/dist/tools/accountTransfer.js

@@ -6,7 +6,7 @@ export const accountDepositTool = {
     name: "account_deposit",
     description: "Deposit funds from wallet into the MYX trading account.",
     schema: {
-        amount: z.string().regex(/^\d+$/).describe("Amount to deposit (token raw units)"),
+        amount: z.union([z.string(), z.number()]).describe("Amount to deposit (human-readable or raw units)"),
         tokenAddress: z.string().describe("Token address"),
     },
     handler: async (args) => {
@@ -14,8 +14,13 @@ export const accountDepositTool = {
             const { client, signer } = await resolveClient();
             const chainId = getChainId();
             const tokenAddress = normalizeAddress(args.tokenAddress, "tokenAddress");
+            // For deposit, we default to quote decimals (6) as it's the most common use case.
+            // 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 raw = await client.account.deposit({
-                amount: args.amount,
+                amount,
                 tokenAddress,
                 chainId,
             });
@@ -32,17 +37,22 @@ export const accountWithdrawTool = {
     description: "Withdraw funds from MYX trading account back to wallet.",
     schema: {
         poolId: z.string().describe("Pool ID to withdraw from"),
-        amount: z.string().regex(/^\d+$/).describe("Amount to withdraw (token raw units)"),
-        isQuoteToken: z.boolean().describe("Whether to withdraw as quote token"),
+        amount: z.union([z.string(), z.number()]).describe("Amount to withdraw (human-readable or raw units)"),
+        isQuoteToken: z.coerce.boolean().describe("Whether to withdraw as quote token"),
     },
     handler: async (args) => {
         try {
             const { client, address, signer } = await resolveClient();
             const chainId = getChainId();
+            const { ensureUnits } = await import("../utils/units.js");
+            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 raw = await client.account.withdraw({
                 chainId,
                 receiver: address,
-                amount: args.amount,
+                amount,
                 poolId: args.poolId,
                 isQuoteToken: args.isQuoteToken,
             });

package/dist/tools/closePosition.js

@@ -2,32 +2,45 @@ import { z } from "zod";
 import { resolveClient } from "../auth/resolveClient.js";
 import { closePosition as closePos } from "../services/tradeService.js";
 import { finalizeMutationResult } from "../utils/mutationResult.js";
-import { isValidSlippagePct4dp, SLIPPAGE_PCT_4DP_DESC } from "../utils/slippage.js";
+import { SLIPPAGE_PCT_4DP_DESC } from "../utils/slippage.js";
 import { verifyTradeOutcome } from "../utils/verification.js";
+import { mapDirection, mapOrderType, mapTriggerType } from "../utils/mappings.js";
 export const closePositionTool = {
     name: "close_position",
     description: "Create a decrease order using SDK-native parameters.",
     schema: {
         poolId: z.string().describe("Pool ID"),
         positionId: z.string().describe("Position ID to close"),
-        orderType: z.coerce.number().int().min(0).max(3).describe("OrderType enum value"),
-        triggerType: z.coerce.number().int().min(0).max(2).describe("TriggerType enum value"),
-        direction: z.coerce.number().pipe(z.union([z.literal(0), z.literal(1)])).describe("0 = LONG, 1 = SHORT"),
-        collateralAmount: z.union([z.string(), z.number()]).describe("Collateral amount (human-readable or raw units)"),
-        size: z.union([z.string(), z.number()]).describe("Position size (human-readable or raw units)"),
-        price: z.union([z.string(), z.number()]).describe("Price (human-readable or 30-dec raw units)"),
-        timeInForce: z.coerce.number().int().describe("TimeInForce enum value"),
+        orderType: z.union([z.number(), z.string()]).describe("Order type: 0/MARKET or 1/LIMIT"),
+        triggerType: z.union([z.number(), z.string()]).optional().describe("Trigger type: 0/NONE, 1/GTE, 2/LTE"),
+        direction: z.union([z.number(), z.string()]).describe("Position direction: 0/LONG or 1/SHORT"),
+        collateralAmount: z.union([z.string(), z.number()]).describe("Collateral amount (human or raw units)"),
+        size: z.union([z.string(), z.number()]).describe("Position size (human or raw units)"),
+        price: z.union([z.string(), z.number()]).describe("Price (human or 30-dec raw units)"),
+        timeInForce: z.coerce.number().int().describe("TimeInForce: 0=GTC, 1=IOC, 2=FOK"),
         postOnly: z.coerce.boolean().describe("Post-only flag"),
-        slippagePct: z.coerce.string().refine(isValidSlippagePct4dp, {
-            message: "slippagePct must be an integer in [0, 10000] with 4-decimal precision (1 = 0.01%).",
-        }).describe(SLIPPAGE_PCT_4DP_DESC),
+        slippagePct: z.coerce.string().default("50").describe(SLIPPAGE_PCT_4DP_DESC),
         executionFeeToken: z.string().describe("Execution fee token address"),
         leverage: z.coerce.number().describe("Leverage"),
     },
     handler: async (args) => {
         try {
             const { client, address, signer } = await resolveClient();
-            const raw = await closePos(client, address, args);
+            const { chainId } = await resolveClient();
+            const poolId = args.poolId;
+            // Fetch pool detail to get quoteToken for execution fee
+            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 mappedArgs = {
+                ...args,
+                direction: mapDirection(args.direction),
+                orderType: mapOrderType(args.orderType),
+                triggerType: args.triggerType !== undefined ? mapTriggerType(args.triggerType) : undefined,
+                executionFeeToken: poolData.quoteToken || args.executionFeeToken
+            };
+            const raw = await closePos(client, address, mappedArgs);
             const data = await finalizeMutationResult(raw, signer, "close_position");
             const txHash = data.confirmation?.txHash;
             let verification = null;

package/dist/tools/executeTrade.js

@@ -2,8 +2,9 @@ import { z } from "zod";
 import { resolveClient } from "../auth/resolveClient.js";
 import { openPosition } from "../services/tradeService.js";
 import { finalizeMutationResult } from "../utils/mutationResult.js";
-import { isValidSlippagePct4dp, SLIPPAGE_PCT_4DP_DESC } from "../utils/slippage.js";
+import { SLIPPAGE_PCT_4DP_DESC } from "../utils/slippage.js";
 import { verifyTradeOutcome } from "../utils/verification.js";
+import { mapDirection, mapOrderType, mapTriggerType } from "../utils/mappings.js";
 const POSITION_ID_RE = /^$|^0x[0-9a-fA-F]{64}$/;
 export const executeTradeTool = {
     name: "execute_trade",
@@ -13,30 +14,44 @@ export const executeTradeTool = {
         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."),
-        orderType: z.coerce.number().int().min(0).max(3).describe("0=Market, 1=Limit, 2=Stop, 3=StopLimit"),
-        triggerType: z.coerce.number().int().min(0).max(2).describe("0=None (Market), 1=Price, 2=Trailing"),
-        direction: z.coerce.number().pipe(z.union([z.literal(0), z.literal(1)])).describe("0 = LONG, 1 = SHORT"),
-        collateralAmount: z.coerce.string().describe("Collateral. e.g. '100' or 'raw:100000000' (6 decimals for USDC)."),
-        size: z.coerce.string().describe("Notional size in base tokens. e.g. '0.5' BTC or 'raw:50000000'."),
-        price: z.coerce.string().describe("Execution or Limit price. e.g. '65000' or 'raw:65000000000000000000000000000000000' (30 decimals)."),
+        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."),
+        collateralAmount: z.union([z.string(), z.number()]).describe("Collateral. e.g. '100' or 'raw:100000000' (6 decimals for USDC)."),
+        size: z.union([z.string(), z.number()]).describe("Notional size in base tokens. e.g. '0.5' BTC or 'raw:50000000'."),
+        price: z.union([z.string(), z.number()]).describe("Execution or Limit price. e.g. '65000' or 'raw:...'"),
         timeInForce: z.coerce.number().int().describe("0=GTC, 1=IOC, 2=FOK"),
         postOnly: z.coerce.boolean().describe("If true, order only executes as Maker."),
-        slippagePct: z.coerce.string().refine(isValidSlippagePct4dp, {
-            message: "slippagePct must be an integer in [0, 10000] with 4-decimal precision (1 = 0.01%).",
-        }).describe(`${SLIPPAGE_PCT_4DP_DESC}. Standard is 100 (1%).`),
+        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."),
-        tpSize: z.coerce.string().optional().describe("Take Profit size. Use '0' to disable."),
-        tpPrice: z.coerce.string().optional().describe("Take Profit trigger price."),
-        slSize: z.coerce.string().optional().describe("Stop Loss size. Use '0' to disable."),
-        slPrice: z.coerce.string().optional().describe("Stop Loss trigger price."),
-        tradingFee: z.coerce.string().describe("Estimated fee in raw units. Fetch via get_user_trading_fee_rate."),
+        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."),
         marketId: z.string().describe("Specific Market Config Hash. Fetch via get_market_list."),
     },
     handler: async (args) => {
         try {
             const { client, address, signer } = await resolveClient();
-            const raw = await openPosition(client, address, args);
+            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 poolData = poolResponse?.data || (poolResponse?.marketId ? poolResponse : null);
+            if (!poolData)
+                throw new Error(`Could not find pool metadata for ID: ${poolId}`);
+            const mappedArgs = {
+                ...args,
+                direction: mapDirection(args.direction),
+                orderType: mapOrderType(args.orderType),
+                triggerType: args.triggerType !== undefined ? mapTriggerType(args.triggerType) : undefined,
+                // Normalize positionId
+                positionId: (args.positionId === "0" || !args.positionId) ? "" : args.positionId,
+                // Enforce executionFeeToken as quoteToken
+                executionFeeToken: poolData.quoteToken || args.executionFeeToken
+            };
+            const raw = await openPosition(client, address, mappedArgs);
             const data = await finalizeMutationResult(raw, signer, "execute_trade");
             const txHash = data.confirmation?.txHash;
             let verification = null;

package/dist/tools/getMarketList.js

@@ -5,7 +5,7 @@ export const getMarketListTool = {
     name: "get_market_list",
     description: "Get tradable markets/pools (state=2 Active). Supports configurable result limit; backend may still enforce its own cap.",
     schema: {
-        limit: z.number().int().positive().optional().describe("Max results to request (default 1000)."),
+        limit: z.coerce.number().int().positive().optional().describe("Max results to request (default 1000)."),
     },
     handler: async (args) => {
         try {

package/dist/tools/getPositions.js

@@ -7,13 +7,67 @@ export const getPositionsTool = {
     schema: {},
     handler: async () => {
         try {
-            const { client, address } = await resolveClient();
+            const { client, address, chainId } = await resolveClient();
             const data = await getPositions(client, address);
             const positions = Array.isArray(data) ? data : (Array.isArray(data?.data) ? data.data : []);
-            const enhancedData = positions.map((pos) => ({
-                ...pos,
-                directionDesc: getDirectionDesc(pos.direction)
+            if (positions.length === 0) {
+                return { content: [{ type: "text", text: JSON.stringify({ status: "success", data: [] }) }] };
+            }
+            // 1. Fetch Tickers for all relevant pools
+            const poolIds = [...new Set(positions.map((p) => p.poolId))];
+            const tickersRes = await client.markets.getTickerList({ chainId, poolIds });
+            const tickers = Array.isArray(tickersRes) ? tickersRes : (tickersRes?.data ?? []);
+            // 2. Fetch Pool Level Configs to get MM (Maintenance Margin Rate)
+            const configs = await Promise.all(poolIds.map(async (pid) => {
+                try {
+                    const res = await client.markets.getPoolLevelConfig(pid, chainId);
+                    return { poolId: pid, config: res?.levelConfig || res?.data?.levelConfig };
+                }
+                catch {
+                    return { poolId: pid, config: null };
+                }
             }));
+            const enhancedData = positions.map((pos) => {
+                const ticker = tickers.find((t) => t.poolId === pos.poolId);
+                const currentPrice = Number(ticker?.price || 0);
+                const entryPrice = Number(pos.entryPrice || 0);
+                const size = Number(pos.size || 0);
+                const collateral = Number(pos.collateralAmount || 0);
+                const direction = pos.direction; // 0 = LONG, 1 = SHORT
+                const mm = configs.find(c => c.poolId === pos.poolId)?.config?.maintainCollateralRate || 0.02;
+                // Estimated PnL
+                let estimatedPnl = 0;
+                if (direction === 0) { // LONG
+                    estimatedPnl = (currentPrice - entryPrice) * size;
+                }
+                else { // SHORT
+                    estimatedPnl = (entryPrice - currentPrice) * size;
+                }
+                // ROI
+                const roi = collateral > 0 ? (estimatedPnl / collateral) * 100 : 0;
+                // Liquidation Price
+                // Long: (Entry * Size - Collateral) / (Size * (1 - MM))
+                // Short: (Entry * Size + Collateral) / (Size * (1 + MM))
+                let liqPrice = 0;
+                if (size > 0) {
+                    if (direction === 0) {
+                        liqPrice = (entryPrice * size - collateral) / (size * (1 - mm));
+                    }
+                    else {
+                        liqPrice = (entryPrice * size + collateral) / (size * (1 + mm));
+                    }
+                }
+                if (liqPrice < 0)
+                    liqPrice = 0;
+                return {
+                    ...pos,
+                    directionDesc: getDirectionDesc(pos.direction),
+                    currentPrice: currentPrice.toString(),
+                    estimatedPnl: estimatedPnl.toFixed(4),
+                    roi: roi.toFixed(2) + "%",
+                    liquidationPrice: liqPrice.toFixed(4)
+                };
+            });
             return { content: [{ type: "text", text: JSON.stringify({ status: "success", data: enhancedData }, (_, v) => typeof v === 'bigint' ? v.toString() : v, 2) }] };
         }
         catch (error) {

package/dist/tools/getUserTradingFeeRate.js

@@ -4,9 +4,9 @@ export const getUserTradingFeeRateTool = {
     name: "get_user_trading_fee_rate",
     description: "Get maker/taker fee rates for a given assetClass and riskTier.",
     schema: {
-        assetClass: z.number().int().nonnegative().describe("Asset class ID"),
-        riskTier: z.number().int().nonnegative().describe("Risk tier"),
-        chainId: z.number().int().positive().optional().describe("Optional chainId override"),
+        assetClass: z.coerce.number().int().nonnegative().describe("Asset class ID"),
+        riskTier: z.coerce.number().int().nonnegative().describe("Risk tier"),
+        chainId: z.coerce.number().int().positive().optional().describe("Optional chainId override"),
     },
     handler: async (args) => {
         try {

package/dist/tools/manageLiquidity.js

@@ -9,9 +9,9 @@ export const manageLiquidityTool = {
         action: z.enum(["deposit", "withdraw"]).describe("'deposit' or 'withdraw'"),
         poolType: z.enum(["BASE", "QUOTE"]).describe("'BASE' or 'QUOTE'"),
         poolId: z.string().describe("Pool ID"),
-        amount: z.number().positive().describe("Amount in human-readable units"),
-        slippage: z.number().min(0).describe("LP slippage ratio (e.g. 0.01 = 1%)"),
-        chainId: z.number().int().positive().optional().describe("Optional chainId override"),
+        amount: z.coerce.number().positive().describe("Amount in human-readable units"),
+        slippage: z.coerce.number().min(0).describe("LP slippage ratio (e.g. 0.01 = 1%)"),
+        chainId: z.coerce.number().int().positive().optional().describe("Optional chainId override"),
     },
     handler: async (args) => {
         try {
@@ -29,6 +29,9 @@ export const manageLiquidityTool = {
                     ? await baseDeposit(poolId, amount, slippage, args.chainId)
                     : await baseWithdraw(poolId, amount, slippage, args.chainId);
             }
+            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.`);
+            }
             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) }] };
         }
@@ -43,7 +46,7 @@ export const getLpPriceTool = {
     schema: {
         poolType: z.enum(["BASE", "QUOTE"]).describe("'BASE' or 'QUOTE'"),
         poolId: z.string().describe("Pool ID"),
-        chainId: z.number().int().positive().optional().describe("Optional chainId override"),
+        chainId: z.coerce.number().int().positive().optional().describe("Optional chainId override"),
     },
     handler: async (args) => {
         try {

package/dist/tools/setTpSl.js

@@ -3,7 +3,7 @@ import { resolveClient } from "../auth/resolveClient.js";
 import { setPositionTpSl } from "../services/tradeService.js";
 import { finalizeMutationResult } from "../utils/mutationResult.js";
 import { mapDirection, mapTriggerType } from "../utils/mappings.js";
-import { isValidSlippagePct4dp, SLIPPAGE_PCT_4DP_DESC } from "../utils/slippage.js";
+import { SLIPPAGE_PCT_4DP_DESC } from "../utils/slippage.js";
 export const setTpSlTool = {
     name: "set_tp_sl",
     description: "Create TP/SL order using SDK-native parameters.",
@@ -13,11 +13,9 @@ export const setTpSlTool = {
         direction: z.any().describe("0=LONG, 1=SHORT, or strings like 'BUY'/'SELL'/'LONG'/'SHORT'."),
         leverage: z.coerce.number().describe("Leverage multiplier, e.g., 10."),
         executionFeeToken: z.string().describe("Address of token to pay gas/execution fees."),
-        tpTriggerType: z.union([z.number(), z.string()]).optional().describe("0=None, 1=Price (Market), 2=Trailing."),
-        slTriggerType: z.union([z.number(), z.string()]).optional().describe("0=None, 1=Price (Market), 2=Trailing."),
-        slippagePct: z.coerce.string().refine(isValidSlippagePct4dp, {
-            message: "slippagePct must be an integer in [0, 10000] with 4-decimal precision (1 = 0.01%).",
-        }).describe(`${SLIPPAGE_PCT_4DP_DESC}. Standard is 100 (1%).`),
+        tpTriggerType: z.union([z.number(), z.string()]).optional().describe("0/NONE, 1/GTE, 2/LTE. e.g. 'GTE'."),
+        slTriggerType: z.union([z.number(), z.string()]).optional().describe("0/NONE, 1/GTE, 2/LTE. e.g. 'LTE'."),
+        slippagePct: z.coerce.string().default("50").describe(`${SLIPPAGE_PCT_4DP_DESC}. Default is 50 (0.5%).`),
         tpPrice: z.coerce.string().optional().describe("Take Profit trigger price. e.g. '2.5' or 'raw:...' (30 decimals)."),
         tpSize: z.coerce.string().optional().describe("TP size in base tokens. Use '0' to disable."),
         slPrice: z.coerce.string().optional().describe("Stop Loss trigger price. e.g. '2.1' or 'raw:...' (30 decimals)."),
@@ -26,12 +24,20 @@ export const setTpSlTool = {
     handler: async (args) => {
         try {
             const { client, address, signer } = await resolveClient();
+            const { chainId } = await resolveClient();
+            const poolId = args.poolId;
+            // Fetch pool detail to get quoteToken for execution fee
+            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}`);
             // Map inputs
             const mappedArgs = {
                 ...args,
                 direction: mapDirection(args.direction),
                 tpTriggerType: args.tpTriggerType !== undefined ? mapTriggerType(args.tpTriggerType) : undefined,
                 slTriggerType: args.slTriggerType !== undefined ? mapTriggerType(args.slTriggerType) : undefined,
+                executionFeeToken: poolData.quoteToken || args.executionFeeToken
             };
             const raw = await setPositionTpSl(client, address, mappedArgs);
             const data = await finalizeMutationResult(raw, signer, "set_tp_sl");

package/dist/utils/units.js

@@ -20,7 +20,7 @@ function normalizeDecimal(input) {
     return `${sign}${intPart || "0"}.${fracPart}`;
 }
 export function ensureUnits(value, decimals, label = "value") {
-    const str = String(value).trim();
+    let str = String(value).trim();
     if (!str)
         throw new Error(`${label} is required.`);
     if (RAW_PREFIX_RE.test(str)) {
@@ -31,6 +31,14 @@ export function ensureUnits(value, decimals, label = "value") {
     }
     if (!DECIMAL_RE.test(str))
         throw new Error(`${label} must be a numeric string.`);
+    // Truncate decimals if they exceed the allowed precision to prevent parseUnits throwing
+    if (str.includes(".")) {
+        const parts = str.split(".");
+        if (parts[1].length > decimals) {
+            console.warn(`[ensureUnits] Truncating ${label} precision: ${str} -> ${parts[0]}.${parts[1].slice(0, decimals)}`);
+            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)) {

package/package.json

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