@michaleffffff/mcp-trading-server 2.3.5 → 2.3.6

package/dist/server.js

@@ -81,7 +81,7 @@ function zodSchemaToJsonSchema(zodSchema) {
     };
 }
 // ─── MCP Server ───
-const server = new Server({ name: "myx-mcp-trading-server", version: "2.3.5" }, { capabilities: { tools: {}, resources: {}, prompts: {} } });
+const server = new Server({ name: "myx-mcp-trading-server", version: "2.3.6" }, { 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.3.5 running (stdio, pure on-chain, prod ready)");
+    logger.info("🚀 MYX Trading MCP Server v2.3.6 running (stdio, pure on-chain, prod ready)");
 }
 main().catch((err) => {
     logger.error("Fatal Server Startup Error", err);

package/dist/services/poolService.js

@@ -67,5 +67,8 @@ export async function getLpPrice(poolType, poolId) {
     if (poolType === "BASE") {
         return base.getLpPrice(chainId, poolId);
     }
-    return quote.getLpPrice(chainId, poolId);
+    if (poolType === "QUOTE") {
+        return quote.getLpPrice(chainId, poolId);
+    }
+    throw new Error("poolType must be 'BASE' or 'QUOTE'.");
 }

package/dist/tools/accountInfo.js

@@ -40,8 +40,8 @@ export const getTradeFlowTool = {
     name: "get_trade_flow",
     description: "Get account trade flow / transaction history.",
     schema: {
-        page: z.number().optional().describe("Page number (default 1)"),
-        limit: z.number().optional().describe("Results per page (default 20)"),
+        page: z.coerce.number().optional().describe("Page number (default 1)"),
+        limit: z.coerce.number().optional().describe("Results per page (default 20)"),
     },
     handler: async (args) => {
         try {

package/dist/tools/accountTransfer.js

@@ -1,6 +1,7 @@
 import { z } from "zod";
 import { resolveClient, getChainId, getQuoteToken } from "../auth/resolveClient.js";
 import { normalizeAddress } from "../utils/address.js";
+import { booleanLike } from "../utils/schema.js";
 export const accountDepositTool = {
     name: "account_deposit",
     description: "Deposit funds from wallet into the MYX trading account.",
@@ -31,7 +32,7 @@ export const accountWithdrawTool = {
     schema: {
         poolId: z.string().describe("Pool ID to withdraw from"),
         amount: z.string().describe("Amount to withdraw (in token decimals)"),
-        isQuoteToken: z.boolean().optional().describe("Whether to withdraw as quote token (default true)"),
+        isQuoteToken: booleanLike.optional().describe("Whether to withdraw as quote token (default true)"),
     },
     handler: async (args) => {
         try {

package/dist/tools/checkApproval.js

@@ -1,6 +1,7 @@
 import { z } from "zod";
 import { resolveClient, getChainId, getQuoteToken } from "../auth/resolveClient.js";
 import { normalizeAddress } from "../utils/address.js";
+import { booleanLike } from "../utils/schema.js";
 const MAX_UINT256 = "115792089237316195423570985008687907853269984665640564039457584007913129639935";
 export const checkApprovalTool = {
     name: "check_approval",
@@ -8,8 +9,8 @@ export const checkApprovalTool = {
     schema: {
         amount: z.string().describe("Amount to check approval for (in token decimals)"),
         quoteToken: z.string().optional().describe("Token address to check. Uses default if omitted."),
-        autoApprove: z.boolean().optional().describe("If true, automatically approve when needed."),
-        approveMax: z.boolean().optional().describe("If true with autoApprove, approve unlimited MaxUint256 (default false: approve exact amount only)."),
+        autoApprove: booleanLike.optional().describe("If true, automatically approve when needed."),
+        approveMax: booleanLike.optional().describe("If true with autoApprove, approve unlimited MaxUint256 (default false: approve exact amount only)."),
     },
     handler: async (args) => {
         try {

package/dist/tools/closePosition.js

@@ -7,8 +7,10 @@ export const closePositionTool = {
     schema: {
         poolId: z.string().describe("Pool ID"),
         positionId: z.string().describe("Position ID to close"),
-        direction: z.union([z.literal(0), z.literal(1)]).describe("0 = LONG, 1 = SHORT (must match position direction)"),
-        leverage: z.number().describe("Position leverage"),
+        direction: z.coerce.number().int().refine((value) => value === 0 || value === 1, {
+            message: "direction must be 0 (LONG) or 1 (SHORT)",
+        }).describe("0 = LONG, 1 = SHORT (must match position direction)"),
+        leverage: z.coerce.number().describe("Position leverage"),
         size: z.string().describe("Size to close (human-readable or 18-decimal raw units)"),
         price: z.string().describe("Price in human-readable units"),
         slippagePct: z.string().optional().describe("Slippage in bps, default '100'"),

package/dist/tools/executeTrade.js

@@ -6,12 +6,14 @@ export const executeTradeTool = {
     description: "Execute a new trade or add to an existing position.",
     schema: {
         poolId: z.string().describe("Pool ID to trade in"),
-        direction: z.union([z.literal(0), z.literal(1)]).describe("0 = LONG, 1 = SHORT"),
+        direction: z.coerce.number().int().refine((value) => value === 0 || value === 1, {
+            message: "direction must be 0 (LONG) or 1 (SHORT)",
+        }).describe("0 = LONG, 1 = SHORT"),
         collateralAmount: z.string().describe("Collateral in human-readable units (e.g. '100' for 100 USDC)"),
-        leverage: z.number().optional().describe("Leverage multiplier (e.g. 10, default 10)"),
+        leverage: z.coerce.number().optional().describe("Leverage multiplier (e.g. 10, default 10)"),
         price: z.string().optional().describe("Limit price in human-readable units (e.g. '3000'). Required for LIMIT orders, omitted for MARKET orders."),
         quoteToken: z.string().optional().describe("Quote token address. Uses env default if omitted."),
-        quoteDecimals: z.number().optional().describe("Quote token decimals (default 6)."),
+        quoteDecimals: z.coerce.number().optional().describe("Quote token decimals (default 6)."),
         slippagePct: z.string().optional().describe("Slippage in bps, default '100' = 1%"),
         positionId: z.string().optional().describe("Existing position ID to add to. Default '0' = new position."),
         tpPrice: z.string().optional().describe("Take-profit price (human-readable)"),

package/dist/tools/getAccountVipInfo.js

@@ -4,7 +4,7 @@ export const getAccountVipInfoTool = {
     name: "get_account_vip_info",
     description: "Get account VIP/fee-tier information.",
     schema: {
-        chainId: z.number().optional().describe("Optional chainId override"),
+        chainId: z.coerce.number().optional().describe("Optional chainId override"),
     },
     handler: async (args) => {
         try {

package/dist/tools/getBaseDetail.js

@@ -5,7 +5,7 @@ export const getBaseDetailTool = {
     description: "Get base asset detail metrics for a pool.",
     schema: {
         poolId: z.string().describe("Pool ID"),
-        chainId: z.number().optional().describe("Optional chainId override"),
+        chainId: z.coerce.number().optional().describe("Optional chainId override"),
     },
     handler: async (args) => {
         try {

package/dist/tools/getKline.js

@@ -6,7 +6,7 @@ export const getKlineTool = {
     schema: {
         poolId: z.string().describe("Pool ID"),
         interval: z.string().describe("K-line interval: '1m','5m','15m','30m','1h','4h','1d','1w','1M'"),
-        limit: z.number().optional().describe("Number of bars (default 100)"),
+        limit: z.coerce.number().optional().describe("Number of bars (default 100)"),
     },
     handler: async (args) => {
         try {

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().optional().describe("Max results to request (default 1000)."),
+        limit: z.coerce.number().optional().describe("Max results to request (default 1000)."),
     },
     handler: async (args) => {
         try {

package/dist/tools/getNetworkFee.js

@@ -5,7 +5,7 @@ export const getNetworkFeeTool = {
     description: "Estimate network fee requirements for a market.",
     schema: {
         marketId: z.string().describe("Market ID"),
-        chainId: z.number().optional().describe("Optional chainId override"),
+        chainId: z.coerce.number().optional().describe("Optional chainId override"),
     },
     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().describe("Asset class ID"),
-        riskTier: z.number().describe("Risk tier"),
-        chainId: z.number().optional().describe("Optional chainId override"),
+        assetClass: z.coerce.number().describe("Asset class ID"),
+        riskTier: z.coerce.number().describe("Risk tier"),
+        chainId: z.coerce.number().optional().describe("Optional chainId override"),
     },
     handler: async (args) => {
         try {

package/dist/tools/manageLiquidity.js

@@ -5,8 +5,12 @@ export const manageLiquidityTool = {
     name: "manage_liquidity",
     description: "Add or withdraw liquidity from a BASE or QUOTE pool. Amount is in human-readable units (e.g. 2000 for USDC, 0.01 for ETH).",
     schema: {
-        action: z.string().describe("'deposit' or 'withdraw'"),
-        poolType: z.string().describe("'BASE' or 'QUOTE'"),
+        action: z.string().trim().toLowerCase().refine((value) => value === "deposit" || value === "withdraw", {
+            message: "action must be 'deposit' or 'withdraw'.",
+        }).describe("'deposit' or 'withdraw'"),
+        poolType: z.string().trim().toUpperCase().refine((value) => value === "BASE" || value === "QUOTE", {
+            message: "poolType must be 'BASE' or 'QUOTE'.",
+        }).describe("'BASE' or 'QUOTE'"),
         poolId: z.string().describe("Pool ID"),
         amount: z.union([z.number(), z.string()]).describe("Amount in human-readable units (e.g. 2000 for USDC, 0.01 for ETH)"),
         slippage: z.union([z.number(), z.string()]).describe("Slippage tolerance (e.g. 0.01 = 1%)"),
@@ -16,10 +20,6 @@ export const manageLiquidityTool = {
             const { action, poolType, poolId } = args;
             const amount = parseSafeNumber(args.amount, "amount");
             const slippage = parseSafeNumber(args.slippage, "slippage");
-            if (!["deposit", "withdraw"].includes(action))
-                throw new Error("action must be 'deposit' or 'withdraw'.");
-            if (!["BASE", "QUOTE"].includes(poolType))
-                throw new Error("poolType must be 'BASE' or 'QUOTE'.");
             if (amount <= 0)
                 throw new Error("amount must be a positive number.");
             if (slippage < 0)
@@ -46,7 +46,9 @@ export const getLpPriceTool = {
     name: "get_lp_price",
     description: "Get the current internal net asset value (NAV) price of an LP token for a BASE or QUOTE pool. Note: This is NOT the underlying token's external Oracle market price (e.g. WETH's price), but rather the internal exchange rate / net worth of the LP token itself which fluctuates based on pool PnL and fees.",
     schema: {
-        poolType: z.string().describe("'BASE' or 'QUOTE'"),
+        poolType: z.string().trim().toUpperCase().refine((value) => value === "BASE" || value === "QUOTE", {
+            message: "poolType must be 'BASE' or 'QUOTE'.",
+        }).describe("'BASE' or 'QUOTE'"),
         poolId: z.string().describe("Pool ID"),
     },
     handler: async (args) => {

package/dist/tools/orderQueries.js

@@ -27,8 +27,8 @@ export const getOrderHistoryTool = {
     description: "Get historical orders with optional pool filter and pagination.",
     schema: {
         poolId: z.string().optional().describe("Filter by pool ID"),
-        page: z.number().optional().describe("Page number (default 1)"),
-        limit: z.number().optional().describe("Results per page (default 20)"),
+        page: z.coerce.number().optional().describe("Page number (default 1)"),
+        limit: z.coerce.number().optional().describe("Results per page (default 20)"),
     },
     handler: async (args) => {
         try {

package/dist/tools/positionHistory.js

@@ -6,8 +6,8 @@ export const getPositionHistoryTool = {
     description: "Get historical closed positions with optional pool filter and pagination.",
     schema: {
         poolId: z.string().optional().describe("Filter by pool ID"),
-        page: z.number().optional().describe("Page number (default 1)"),
-        limit: z.number().optional().describe("Results per page (default 20)"),
+        page: z.coerce.number().optional().describe("Page number (default 1)"),
+        limit: z.coerce.number().optional().describe("Results per page (default 20)"),
     },
     handler: async (args) => {
         try {

package/dist/tools/searchMarket.js

@@ -6,7 +6,7 @@ export const searchMarketTool = {
     description: "Search for an active market by keyword.",
     schema: {
         keyword: z.string().describe('Search keyword, e.g. "BTC", "ETH"'),
-        limit: z.number().optional().describe("Max results (default 100)"),
+        limit: z.coerce.number().optional().describe("Max results (default 100)"),
     },
     handler: async (args) => {
         try {

package/dist/tools/setTpSl.js

@@ -7,8 +7,10 @@ export const setTpSlTool = {
     schema: {
         poolId: z.string().describe("Pool ID"),
         positionId: z.string().describe("Position ID"),
-        direction: z.union([z.literal(0), z.literal(1)]).describe("0 = LONG, 1 = SHORT (position direction)"),
-        leverage: z.number().describe("Position leverage"),
+        direction: z.coerce.number().int().refine((value) => value === 0 || value === 1, {
+            message: "direction must be 0 (LONG) or 1 (SHORT)",
+        }).describe("0 = LONG, 1 = SHORT (position direction)"),
+        leverage: z.coerce.number().describe("Position leverage"),
         tpPrice: z.string().optional().describe("Take-profit trigger price (human-readable)"),
         tpSize: z.string().optional().describe("TP size (18 decimals). Omit or '0' for full position."),
         slPrice: z.string().optional().describe("Stop-loss trigger price (human-readable)"),

package/dist/tools/updateOrderTpSl.js

@@ -1,6 +1,7 @@
 import { z } from "zod";
 import { resolveClient } from "../auth/resolveClient.js";
 import { updateOrderTpSl } from "../services/tradeService.js";
+import { booleanLike } from "../utils/schema.js";
 export const updateOrderTpSlTool = {
     name: "update_order_tp_sl",
     description: "Update an existing take profit or stop loss order.",
@@ -11,7 +12,7 @@ export const updateOrderTpSlTool = {
         tpSize: z.string().optional().describe("New TP size (18 decimals). Omit or '0' for full position."),
         slPrice: z.string().optional().describe("New stop-loss trigger price (human-readable)"),
         slSize: z.string().optional().describe("New SL size (18 decimals). Omit or '0' for full position."),
-        isTpSlOrder: z.boolean().optional().describe("Whether this is a TP/SL order (default true)"),
+        isTpSlOrder: booleanLike.optional().describe("Whether this is a TP/SL order (default true)"),
         quoteToken: z.string().optional().describe("Quote token address"),
     },
     handler: async (args) => {

package/dist/utils/schema.js

@@ -0,0 +1,18 @@
+import { z } from "zod";
+const trueValues = new Set(["true", "1", "yes", "y", "on"]);
+const falseValues = new Set(["false", "0", "no", "n", "off"]);
+/**
+ * Safe boolean coercion for MCP inputs:
+ * - boolean: true/false
+ * - number: 1/0
+ * - string: true/false/1/0/yes/no/on/off (case-insensitive)
+ */
+export const booleanLike = z.union([
+    z.boolean(),
+    z.number().refine((value) => value === 0 || value === 1, {
+        message: "Expected 0 or 1 for boolean value.",
+    }).transform((value) => value === 1),
+    z.string().trim().toLowerCase().refine((value) => trueValues.has(value) || falseValues.has(value), {
+        message: "Expected boolean-like value: true/false/1/0/yes/no/on/off.",
+    }).transform((value) => trueValues.has(value)),
+]);

package/package.json

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