@michaleffffff/mcp-trading-server 3.0.29 → 3.0.31

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## 3.0.31 - 2026-03-20
+### Fixed
+- 修复 `normalizeSlippageRatio` 在处理 1% 到 100% 之间数值时的歧义问题。
+- 为 ARB 池验证了 1% 滑点的存入操作（继续验证单参数调用的局限性）。
+
+## [3.0.30] - 2026-03-20
+### Fixed
+- 完全对齐 SDK 流动性逻辑：移除 MCP 手动定义的预言机价格注入（适用于所有 State 0/1/2 池子）。
+- 设置流动性操作默认滑点为 0.1%。
+- 强制所有流动性交易调用 1 个参数的合约签名，以匹配 SDK v1.0.4-beta.4 的行为。
+
 ## [3.0.29] - 2026-03-20
 - **Fix**: Implemented oracle validation bypass for State 0 (Cook) and State 1 (Primed) pools in liquidity management.
   - This allows adding/removing liquidity on newly created pools that haven't initialized their oracle feeds yet.

package/README.md

@@ -98,7 +98,7 @@ If `IS_BETA_MODE` is omitted, MCP now auto-detects beta mode for the two testnet
 - **Direction validation**: when a tool operates on an existing `positionId`, the supplied `direction` must match the live position.
 - **TP/SL semantics**: LONG requires `tpPrice > entryPrice` and `slPrice < entryPrice`; SHORT uses the inverse.
 - **LP safety**: LP preview failures are fail-close and no longer downgrade to `minAmountOut=0`.
-- **LP slippage input**: `manage_liquidity.slippage` only validates numeric format and ratio normalization; MCP no longer imposes its own 5% business cap.
+- **LP slippage input**: `manage_liquidity.slippage` now matches SDK semantics and must be a ratio in `(0, 1]`; MCP still does not impose its old 5% business cap.
 - **LP metadata safety**: `get_pool_metadata(includeLiquidity=true)` now ignores caller-supplied `marketPrice` and derives liquidity depth from a fresh Oracle price only.
 - **LP holdings semantics**: `get_my_lp_holdings` is an inventory view; returned rows are no longer ranked by mixed BASE/QUOTE LP raw balances.
 - **LP token-source safety**: `get_my_lp_holdings` now falls back to live `poolInfo.basePool.poolToken` / `poolInfo.quotePool.poolToken` when market-detail token addresses are stale or mismatched.

package/TOOL_EXAMPLES.md

@@ -304,7 +304,7 @@ Alias-friendly form also works:
 
 LP preview failures now fail closed; the server no longer downgrades to `minAmountOut=0`.
 Oracle-backed LP pricing requires a fresh price snapshot before execution.
-`slippage` here is a ratio, so `0.01 = 1%` and `0.005 = 0.5%`.
+`slippage` here must be a ratio in `(0, 1]`, so `0.01 = 1%` and `0.005 = 0.5%`.
 MCP no longer adds its own 5% LP slippage business cap on top of the provided value.
 When the SDK LP path fails on a non-standard token `allowance()` read, MCP falls back to the direct router path automatically.
 

package/dist/auth/resolveClient.js

@@ -3,8 +3,8 @@ import { JsonRpcProvider, Wallet } from "ethers";
 import { normalizeAddress } from "../utils/address.js";
 let cached = null;
 const BETA_BROKERS_BY_CHAIN = {
-    421614: [],
-    59141: [],
+    421614: ["0x895c4ae2a22bb26851011d733a9355f663a1f939"],
+    59141: ["0x634efdc9dc76d7abf6e49279875a31b02e9891e2"],
 };
 function getDefaultBrokerByChainId(chainId) {
     // Testnet mappings

package/dist/prompts/tradingGuide.js

@@ -12,7 +12,7 @@ export const tradingGuidePrompt = {
                     content: {
                         type: "text",
                         text: `
-# MYX Trading MCP Best Practices (v3.0.27)
+# MYX Trading MCP Best Practices (v3.0.31)
 
 You are an expert crypto trader using the MYX Protocol. To ensure successful execution and safe handling of user funds, follow these patterns:
 

package/dist/server.js

@@ -461,7 +461,7 @@ function zodSchemaToJsonSchema(zodSchema) {
     };
 }
 // ─── MCP Server ───
-const server = new Server({ name: "myx-mcp-trading-server", version: "3.0.29" }, { capabilities: { tools: {}, resources: {}, prompts: {} } });
+const server = new Server({ name: "myx-mcp-trading-server", version: "3.0.31" }, { capabilities: { tools: {}, resources: {}, prompts: {} } });
 // List tools
 server.setRequestHandler(ListToolsRequestSchema, async () => {
     return {

package/dist/services/marketService.js

@@ -31,6 +31,17 @@ function normalizeMarketState(value) {
     const casted = Number(value);
     return Number.isFinite(casted) ? casted : null;
 }
+function getMarketStatePriority(state) {
+    if (state === 2)
+        return 0;
+    if (state === 1)
+        return 1;
+    if (state === 0)
+        return 2;
+    if (state === null)
+        return 3;
+    return 4;
+}
 function normalizePoolId(row) {
     const poolId = row?.poolId ?? row?.pool_id ?? "";
     return String(poolId);
@@ -163,17 +174,23 @@ export async function searchMarket(client, keyword, limit = 1000, chainIdOverrid
             dedupedByPoolId.set(poolId, row);
         }
     }
-    const activeMarkets = Array.from(dedupedByPoolId.values())
-        .filter((row) => {
-        const state = normalizeMarketState(row?.state ?? row?.poolState);
-        return state === null || state === 2;
+    const orderedMarkets = Array.from(dedupedByPoolId.values())
+        .sort((left, right) => {
+        const leftState = normalizeMarketState(left?.state ?? left?.poolState);
+        const rightState = normalizeMarketState(right?.state ?? right?.poolState);
+        const byState = getMarketStatePriority(leftState) - getMarketStatePriority(rightState);
+        if (byState !== 0)
+            return byState;
+        const leftSymbol = String(left?.baseQuoteSymbol ?? left?.symbolName ?? left?.poolId ?? "");
+        const rightSymbol = String(right?.baseQuoteSymbol ?? right?.symbolName ?? right?.poolId ?? "");
+        return leftSymbol.localeCompare(rightSymbol);
     })
         .slice(0, requestedLimit);
     // Get tickers for these pools to get price and change24h
     let tickers = [];
-    if (activeMarkets.length > 0) {
+    if (orderedMarkets.length > 0) {
         try {
-            const poolIds = activeMarkets.map((market) => normalizePoolId(market));
+            const poolIds = orderedMarkets.map((market) => normalizePoolId(market));
             const tickerRes = await client.markets.getTickerList({ chainId, poolIds });
             tickers = Array.isArray(tickerRes) ? tickerRes : (tickerRes?.data || []);
         }
@@ -181,7 +198,7 @@ export async function searchMarket(client, keyword, limit = 1000, chainIdOverrid
             console.error("Failed to fetch tickers:", e);
         }
     }
-    return activeMarkets.map((market) => {
+    return orderedMarkets.map((market) => {
         const poolId = normalizePoolId(market);
         const state = normalizeMarketState(market?.state ?? market?.poolState);
         const ticker = tickers.find((t) => String(t.poolId).toLowerCase() === poolId.toLowerCase());
@@ -279,9 +296,8 @@ export async function resolvePool(client, poolId, keyword, chainIdOverride) {
             };
             const rows = collect(poolListRes.data ?? poolListRes);
             const searchKey = (kw || pid || "").toUpperCase();
-            const match = rows.find((row) => {
-                if (Number(row?.state) !== 2)
-                    return false;
+            const match = rows
+                .filter((row) => {
                 const base = String(row?.baseSymbol ?? "").toUpperCase();
                 const pair = String(row?.baseQuoteSymbol ?? "").toUpperCase();
                 const id = String(row?.poolId ?? row?.pool_id ?? "").toUpperCase();
@@ -292,7 +308,12 @@ export async function resolvePool(client, poolId, keyword, chainIdOverride) {
                     id === searchKey ||
                     baseToken === searchKey ||
                     quoteToken === searchKey;
-            });
+            })
+                .sort((left, right) => {
+                const leftState = normalizeMarketState(left?.state ?? left?.poolState);
+                const rightState = normalizeMarketState(right?.state ?? right?.poolState);
+                return getMarketStatePriority(leftState) - getMarketStatePriority(rightState);
+            })[0];
             if (match) {
                 return String(match.poolId ?? match.pool_id);
             }

package/dist/services/poolService.js

@@ -8,7 +8,7 @@ import { logger } from "../utils/logger.js";
 import { assertOracleFreshness, getFreshOraclePrice } from "./marketService.js";
 const LP_DECIMALS = 18;
 const POOL_MANAGER_BY_CHAIN = {
-    421614: "0xf268D9FeD3Bd56fd9aBdb4FeEb993338613678A8",
+    421614: "0xB131655F326E82753b0e76c1ce853E257524f3a4",
     59141: "0x85e869d98216221807A06636541Ec93C9c0a4B0c",
     97: "0x4F917ef137b573D9790b87e3cF6dfb698cF00c9c",
     56: "0x13F2130c2F3bfd612BBCBF35FB9E467dd32bAF3A",
@@ -25,14 +25,14 @@ const POOL_MANAGER_ABI = [
 ];
 const LP_ROUTER_BY_CHAIN = {
     421614: {
-        router: "0x5E16C0699de2Ee5D69a620AB38AE2c81A29a58df",
-        basePool: "0xDb41D850e9b30fc0417F488D1aB64fdc1d0DdFdc",
-        quotePool: "0xF0784c1023C051Bf1D5b8470A896c536039B06B9",
+        router: "0x0fb875c10fe2fF981e467765E3daaE4355b180D0",
+        basePool: "0xeC0b3C76cC1C47f9B29313c706c1F6FD8D1C023f",
+        quotePool: "0x608Bfd6B8Df0807aA6d1500B76a3eAb3C9DFd728",
     },
     59141: {
-        router: "0xf1395643D77DD839E4b8d3Bb122fd0275Cf07631",
-        basePool: "0x31b093BBADbb2A3Bcd7E433E8Ed5bcdD76fFAFFe",
-        quotePool: "0xFF8A6797E2063A7a4cd1468eB7cC0db9798C753f",
+        router: "0xc5331ab0159379E6CfCC1f09b63360D0B9715D74",
+        basePool: "0x8440a78E65F07D8013dd5B0640E0E713c8fd9893",
+        quotePool: "0x50F1F7A772672FE0637cF89AD3aFc093584eD040",
     },
     97: {
         router: "0xe9F2E58562aD1D50AfB1eD92EAa6D367A6D0e552",
@@ -191,11 +191,11 @@ function getLpAddresses(chainId) {
 function normalizeSlippageRatio(slippage) {
     if (!Number.isFinite(slippage) || slippage < 0)
         return 0;
-    if (slippage > 1 && slippage <= 100)
-        return slippage / 100;
-    if (slippage > 1)
-        return 1;
-    return slippage;
+    if (slippage > 100)
+        return slippage / 10000; // assume bps e.g. 200 = 2%
+    if (slippage >= 1)
+        return slippage / 100; // assume percent e.g. 1.5 = 1.5%
+    return slippage; // assume ratio e.g. 0.01 = 1%
 }
 function applyMinOutBySlippage(amountOut, slippage) {
     if (amountOut <= 0n)
@@ -327,6 +327,7 @@ async function executeLiquidityTxViaRouter(params) {
     const amountOut = await previewAmountOutForLiquidity(signer, chainId, poolId, poolType, action, amountIn, oraclePayload.referencePrice30);
     const minAmountOut = applyMinOutBySlippage(amountOut, slippage);
     const routerContract = new Contract(addresses.router, ROUTER_ABI, signer);
+    const oraclePriceTuples = oraclePayload.prices;
     const txOverrides = {};
     if (oraclePayload.value > 0n) {
         txOverrides.value = oraclePayload.value;
@@ -340,21 +341,16 @@ async function executeLiquidityTxViaRouter(params) {
             address,
             [],
         ];
-        if (oraclePayload.prices.length > 0) {
-            tx = await routerContract["depositQuote((bytes32,uint8,uint64,bytes)[],(bytes32,uint256,uint256,address,(uint256,uint256,uint8,uint256)[]))"](oraclePayload.prices, paramsTuple, txOverrides);
-        }
-        else {
-            tx = await routerContract["depositQuote((bytes32,uint256,uint256,address,(uint256,uint256,uint8,uint256)[]))"](paramsTuple, txOverrides);
-        }
+        tx = await routerContract["depositQuote((bytes32,uint8,uint64,bytes)[],(bytes32,uint256,uint256,address,(uint256,uint256,uint8,uint256)[]))"](oraclePriceTuples, paramsTuple, txOverrides);
     }
     else if (poolType === "QUOTE" && action === "withdraw") {
-        const paramsTuple = [poolId, amountIn, minAmountOut, address];
-        if (oraclePayload.prices.length > 0) {
-            tx = await routerContract["withdrawQuote((bytes32,uint8,uint64,bytes)[],(bytes32,uint256,uint256,address))"](oraclePayload.prices, paramsTuple, txOverrides);
-        }
-        else {
-            tx = await routerContract["withdrawQuote((bytes32,uint256,uint256,address))"](paramsTuple, txOverrides);
-        }
+        const paramsTuple = [
+            poolId,
+            amountIn,
+            minAmountOut,
+            address,
+        ];
+        tx = await routerContract["withdrawQuote((bytes32,uint8,uint64,bytes)[],(bytes32,uint256,uint256,address))"](oraclePriceTuples, paramsTuple, txOverrides);
     }
     else if (poolType === "BASE" && action === "deposit") {
         const paramsTuple = [
@@ -364,21 +360,16 @@ async function executeLiquidityTxViaRouter(params) {
             address,
             [],
         ];
-        if (oraclePayload.prices.length > 0) {
-            tx = await routerContract["depositBase((bytes32,uint8,uint64,bytes)[],(bytes32,uint256,uint256,address,(uint256,uint256,uint8,uint256)[]))"](oraclePayload.prices, paramsTuple, txOverrides);
-        }
-        else {
-            tx = await routerContract["depositBase((bytes32,uint256,uint256,address,(uint256,uint256,uint8,uint256)[]))"](paramsTuple, txOverrides);
-        }
+        tx = await routerContract["depositBase((bytes32,uint8,uint64,bytes)[],(bytes32,uint256,uint256,address,(uint256,uint256,uint8,uint256)[]))"](oraclePriceTuples, paramsTuple, txOverrides);
     }
     else {
-        const paramsTuple = [poolId, amountIn, minAmountOut, address];
-        if (oraclePayload.prices.length > 0) {
-            tx = await routerContract["withdrawBase((bytes32,uint8,uint64,bytes)[],(bytes32,uint256,uint256,address))"](oraclePayload.prices, paramsTuple, txOverrides);
-        }
-        else {
-            tx = await routerContract["withdrawBase((bytes32,uint256,uint256,address))"](paramsTuple, txOverrides);
-        }
+        const paramsTuple = [
+            poolId,
+            amountIn,
+            minAmountOut,
+            address,
+        ];
+        tx = await routerContract["withdrawBase((bytes32,uint8,uint64,bytes)[],(bytes32,uint256,uint256,address))"](oraclePriceTuples, paramsTuple, txOverrides);
     }
     const txHash = String(tx?.hash ?? "").trim();
     if (!txHash || !txHash.startsWith("0x")) {

package/dist/tools/getMyLpHoldings.js

@@ -228,7 +228,8 @@ export const getMyLpHoldingsTool = {
                 let quoteCandidates = collectAddressCandidates(quotePoolToken);
                 let baseResolved = await resolveLpBalanceFromCandidates(provider, address, baseCandidates);
                 let quoteResolved = await resolveLpBalanceFromCandidates(provider, address, quoteCandidates);
-                const needPoolInfoEnrichment = (BigInt(baseResolved.balanceRaw) === 0n && BigInt(quoteResolved.balanceRaw) === 0n) ||
+                const needPoolInfoEnrichment = BigInt(baseResolved.balanceRaw) === 0n ||
+                    BigInt(quoteResolved.balanceRaw) === 0n ||
                     !baseResolved.tokenAddress ||
                     !quoteResolved.tokenAddress;
                 if (needPoolInfoEnrichment) {

package/dist/tools/manageLiquidity.js

@@ -69,7 +69,7 @@ export const manageLiquidityTool = {
         poolType: z.enum(["BASE", "QUOTE"]).describe("'BASE' or 'QUOTE'"),
         poolId: z.string().describe("Pool ID or Base Token Address"),
         amount: z.coerce.string().describe("Amount in human-readable units string"),
-        slippage: z.coerce.number().min(0).describe("LP slippage ratio (e.g. 0.01 = 1%)"),
+        slippage: z.coerce.number().gt(0).max(1).describe("LP slippage ratio in (0, 1], e.g. 0.01 = 1%"),
         chainId: z.coerce.number().int().positive().optional().describe("Optional chainId override"),
     },
     handler: async (args) => {

package/dist/utils/slippage.js

@@ -46,8 +46,8 @@ export function normalizeSlippagePct4dpFlexible(value, label = "slippagePct") {
 }
 export function normalizeLpSlippageRatio(value, label = "slippage") {
     const numeric = Number(value);
-    if (!Number.isFinite(numeric) || numeric < 0) {
-        throw new Error(`${label} must be a finite number >= 0.`);
+    if (!Number.isFinite(numeric) || numeric <= 0 || numeric > 1) {
+        throw new Error(`${label} must be a finite ratio in (0, 1].`);
     }
-    return numeric > 1 && numeric <= 100 ? numeric / 100 : numeric;
+    return numeric;
 }

package/package.json

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