@michaleffffff/mcp-trading-server 2.8.2 → 2.9.0

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.0" }, { 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.0 running (stdio, pure on-chain, prod ready)");
 }
 main().catch((err) => {
     logger.error("Fatal Server Startup Error", err);

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,37 @@ 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().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 mappedArgs = {
+                ...args,
+                direction: mapDirection(args.direction),
+                orderType: mapOrderType(args.orderType),
+                triggerType: args.triggerType !== undefined ? mapTriggerType(args.triggerType) : undefined,
+            };
+            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,34 @@ 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().describe(`${SLIPPAGE_PCT_4DP_DESC}. Standard is 100 (1%).`),
         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 mappedArgs = {
+                ...args,
+                direction: mapDirection(args.direction),
+                orderType: mapOrderType(args.orderType),
+                triggerType: args.triggerType !== undefined ? mapTriggerType(args.triggerType) : undefined,
+            };
+            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/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/setTpSl.js

@@ -13,8 +13,8 @@ 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."),
+        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%).`),

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.0",
   "type": "module",
   "bin": {
     "myx-mcp": "dist/server.js"