@michaleffffff/mcp-trading-server 2.9.0 → 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.9.0" }, { 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.9.0 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/closePosition.js

@@ -19,18 +19,26 @@ export const closePositionTool = {
         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().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 { 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");

package/dist/tools/executeTrade.js

@@ -22,7 +22,7 @@ export const executeTradeTool = {
         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().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.union([z.string(), z.number()]).optional().describe("Take Profit size. Use '0' to disable."),
@@ -35,11 +35,21 @@ export const executeTradeTool = {
     handler: async (args) => {
         try {
             const { client, address, signer } = await resolveClient();
+            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");

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/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.",
@@ -15,9 +15,7 @@ export const setTpSlTool = {
         executionFeeToken: z.string().describe("Address of token to pay gas/execution fees."),
         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().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%).`),
         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/package.json

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