@michaleffffff/mcp-trading-server 3.0.29 → 3.1.0

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## 3.1.0 - 2026-03-20
+### Changed
+- 发布 `3.1.0`，统一版本元数据到新的主次版本号。
+- 运行环境现在只区分测试环境与正式环境，不再走 beta host / beta 地址分支。
+- 修复测试环境下的市场发现链路，`list_pools` 恢复返回当前链池列表，`get_all_tickers` 在主接口不可用时继续走 fallback。
+
+## 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

@@ -6,7 +6,7 @@ A production-ready MCP (Model Context Protocol) server for deep integration with
 
 # Release Notes
 
-- **Current release: 3.0.24**
+- **Current release: 3.1.0**
 - **SDK baseline**: `@myx-trade/sdk@^1.0.4-beta.4` compatibility completed.
 - **Refinement**: Consolidated 40+ specialized tools into ~26 high-level unified tools.
 - **Improved UX**: Enhanced AI parameter parsing, automated unit conversion, and structured error reporting.
@@ -37,7 +37,6 @@ CHAIN_ID=...
 BROKER_ADDRESS=0x...
 QUOTE_TOKEN_ADDRESS=0x...
 QUOTE_TOKEN_DECIMALS=...
-IS_BETA_MODE=true
 ```
 
 ## Testnet `MYXBroker` Reference
@@ -46,7 +45,6 @@ IS_BETA_MODE=true
 - Linea test: `0x634EfDC9dC76D7AbF6E49279875a31B02E9891e2`
 
 Use the broker that matches your active RPC and chain configuration.
-If `IS_BETA_MODE` is omitted, MCP now auto-detects beta mode for the two testnet brokers above.
 
 ---
 
@@ -91,14 +89,13 @@ If `IS_BETA_MODE` is omitted, MCP now auto-detects beta mode for the two testnet
 
 - **Explicit execution price**: `open_position_simple` no longer auto-fills a fresh Oracle price for `MARKET` orders; provide `price` explicitly when opening a position.
 - **SDK-delegated funding delta**: MCP no longer performs its own increase-order margin/deposit reconciliation. New increase orders delegate collateral shortfall handling to SDK `createIncreaseOrder`.
-- **Beta broker compatibility**: current Arbitrum Sepolia / Linea Sepolia test brokers auto-enable beta mode when `IS_BETA_MODE` is unset.
 - **Exact approvals by default**: local fallback flows now prefer exact approvals instead of implicit unlimited approval.
 - **Size semantics**: `size` always means base-asset quantity, not USD notional.
 - **Pre-check semantics**: `check_account_ready` now uses SDK `getAvailableMarginBalance`. When that read fails, the response marks `summary.degraded=true` and includes diagnostics instead of silently trusting stale `freeMargin`.
 - **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

@@ -2,10 +2,6 @@ import { MyxClient } from "@myx-trade/sdk";
 import { JsonRpcProvider, Wallet } from "ethers";
 import { normalizeAddress } from "../utils/address.js";
 let cached = null;
-const BETA_BROKERS_BY_CHAIN = {
-    421614: [],
-    59141: [],
-};
 function getDefaultBrokerByChainId(chainId) {
     // Testnet mappings
     if (chainId === 421614)
@@ -22,14 +18,6 @@ function getDefaultQuoteTokenByChainId(chainId) {
         return "0xD984fd34f91F92DA0586e1bE82E262fF27DC431b"; // Linea Sepolia
     return "0xD984fd34f91F92DA0586e1bE82E262fF27DC431b";
 }
-function resolveIsBetaMode(chainId, brokerAddressRaw) {
-    const explicit = String(process.env.IS_BETA_MODE ?? "").trim().toLowerCase();
-    if (explicit === "true")
-        return true;
-    if (explicit === "false")
-        return false;
-    return Boolean(BETA_BROKERS_BY_CHAIN[chainId]?.includes(String(brokerAddressRaw ?? "").trim().toLowerCase()));
-}
 export async function resolveClient() {
     if (cached)
         return cached;
@@ -50,7 +38,6 @@ export async function resolveClient() {
         throw new Error("QUOTE_TOKEN_ADDRESS env var is required.");
     const brokerAddress = normalizeAddress(brokerAddressRaw, "BROKER_ADDRESS");
     const quoteToken = normalizeAddress(quoteTokenRaw, "QUOTE_TOKEN_ADDRESS");
-    const isBetaMode = resolveIsBetaMode(chainId, brokerAddress);
     const provider = new JsonRpcProvider(rpcUrl);
     const signer = new Wallet(privateKey, provider);
     // Inject the EIP-1193 mock so SDK can sign transactions seamlessly
@@ -75,7 +62,7 @@ export async function resolveClient() {
         signer: signer,
         brokerAddress,
         isTestnet,
-        isBetaMode,
+        isBetaMode: false,
         walletClient: walletClient
     });
     cached = { client, address: signer.address, signer, chainId, quoteToken, quoteDecimals };

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.1.0)
 
 You are an expert crypto trader using the MYX Protocol. To ensure successful execution and safe handling of user funds, follow these patterns:
 
@@ -36,7 +36,6 @@ You are an expert crypto trader using the MYX Protocol. To ensure successful exe
 - **Execution Price**: \`open_position_simple\` no longer auto-fills a fresh Oracle price for \`MARKET\`; provide \`price\` explicitly when you want MCP to compute size / fee previews.
 - **Funding Delta Ownership**: MCP no longer performs its own increase-order margin/deposit reconciliation. SDK \`createIncreaseOrder\` owns deposit-delta handling for new increase orders.
 - **Pre-check Diagnostics**: \`check_account_ready\` now reports SDK \`availableMarginBalance\` first. If that read degrades, inspect \`summary.degraded\` and \`diagnostics.availableMarginError\` before trusting fallback account fields.
-- **Beta Broker Mode**: Current Arbitrum Sepolia / Linea Sepolia test brokers auto-enable beta mode when \`IS_BETA_MODE\` is omitted.
 - **Approval Safety**: Local fallback flows prefer exact approval sizing. Do not assume unlimited approvals are necessary.
 - **Position Semantics**: \`size\` is BASE quantity, not USD notional. If a \`positionId\` is supplied, \`direction\` must match the live position.
 - **TP/SL Semantics**: LONG should use \`tpPrice > entryPrice\` and \`slPrice < entryPrice\`; SHORT uses the inverse. Plain integer strings like \`"65000"\` are treated as human prices, not implicit raw 30-decimal values.
@@ -49,7 +48,6 @@ You are an expert crypto trader using the MYX Protocol. To ensure successful exe
 - Arbitrum test: \`0x895C4ae2A22bB26851011d733A9355f663a1F939\`
 - Linea test: \`0x634EfDC9dC76D7AbF6E49279875a31B02E9891e2\`
 - Always keep \`RPC_URL\`, \`CHAIN_ID\`, and \`BROKER_ADDRESS\` on the same network.
-- If \`IS_BETA_MODE\` is unset, MCP auto-detects beta mode for those two testnet brokers.
 
 Current Session:
 - Wallet: ${address}

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.1.0" }, { capabilities: { tools: {}, resources: {}, prompts: {} } });
 // List tools
 server.setRequestHandler(ListToolsRequestSchema, async () => {
     return {
@@ -582,7 +582,7 @@ server.setRequestHandler(GetPromptRequestSchema, async (request) => {
 async function main() {
     const transport = new StdioServerTransport();
     await server.connect(transport);
-    logger.info("🚀 MYX Trading MCP Server v3.0.24 running (stdio, pure on-chain, prod ready)");
+    logger.info("🚀 MYX Trading MCP Server v3.1.0 running (stdio, pure on-chain, prod ready)");
 }
 main().catch((err) => {
     logger.error("Fatal Server Startup Error", err);

package/dist/services/marketService.js

@@ -31,10 +31,27 @@ 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);
 }
+function matchesChainId(row, chainId) {
+    const raw = row?.chainId ?? row?.chain_id;
+    if (raw === undefined || raw === null || raw === "")
+        return true;
+    return Number(raw) === chainId;
+}
 function matchesKeyword(row, keywordUpper) {
     const poolId = normalizePoolId(row);
     const haystack = [
@@ -56,7 +73,7 @@ async function fetchApiMarketRows(client, chainId) {
     // Preferred path: documented markets.searchMarket
     try {
         const searchRes = await client.markets.searchMarket({ chainId, keyword: "", limit: 2000 });
-        const searchRows = extractMarketRows(searchRes, chainId).filter((row) => normalizePoolId(row));
+        const searchRows = extractMarketRows(searchRes, chainId).filter((row) => normalizePoolId(row) && matchesChainId(row, chainId));
         if (searchRows.length > 0)
             return searchRows;
     }
@@ -65,7 +82,7 @@ async function fetchApiMarketRows(client, chainId) {
     // Secondary path: documented markets.getPoolSymbolAll
     try {
         const symbolsRes = await client.markets.getPoolSymbolAll();
-        const symbolRows = collectRows(symbolsRes?.data ?? symbolsRes).filter((row) => normalizePoolId(row));
+        const symbolRows = collectRows(symbolsRes?.data ?? symbolsRes).filter((row) => normalizePoolId(row) && matchesChainId(row, chainId));
         if (symbolRows.length > 0)
             return symbolRows;
     }
@@ -73,12 +90,12 @@ async function fetchApiMarketRows(client, chainId) {
     }
     // Legacy fallback: internal api namespace (for backward compatibility)
     const marketListRes = await client.api?.getMarketList?.().catch(() => null);
-    const marketRows = extractMarketRows(marketListRes, chainId);
+    const marketRows = extractMarketRows(marketListRes, chainId).filter((row) => matchesChainId(row, chainId));
     const marketRowsWithPoolId = marketRows.filter((row) => normalizePoolId(row));
     if (marketRowsWithPoolId.length > 0)
         return marketRowsWithPoolId;
     const poolListRes = await client.api?.getPoolList?.().catch(() => null);
-    return collectRows(poolListRes?.data ?? poolListRes).filter((row) => normalizePoolId(row));
+    return collectRows(poolListRes?.data ?? poolListRes).filter((row) => normalizePoolId(row) && matchesChainId(row, chainId));
 }
 export async function getMarketPrice(client, poolId, chainIdOverride) {
     const chainId = chainIdOverride ?? getChainId();
@@ -163,17 +180,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 +204,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());
@@ -213,7 +236,7 @@ export async function getPoolList(client, chainIdOverride) {
     const chainId = chainIdOverride ?? getChainId();
     try {
         const searchRes = await client.markets.searchMarket({ chainId, keyword: "", limit: 2000 });
-        const rows = extractMarketRows(searchRes, chainId).filter((row) => normalizePoolId(row));
+        const rows = extractMarketRows(searchRes, chainId).filter((row) => normalizePoolId(row) && matchesChainId(row, chainId));
         if (rows.length > 0)
             return rows;
     }
@@ -221,13 +244,14 @@ export async function getPoolList(client, chainIdOverride) {
     }
     try {
         const symbolsRes = await client.markets.getPoolSymbolAll();
-        const rows = collectRows(symbolsRes?.data ?? symbolsRes).filter((row) => normalizePoolId(row));
+        const rows = collectRows(symbolsRes?.data ?? symbolsRes).filter((row) => normalizePoolId(row) && matchesChainId(row, chainId));
         if (rows.length > 0)
             return rows;
     }
     catch {
     }
-    return client.api?.getPoolList?.();
+    const poolListRes = await client.api?.getPoolList?.();
+    return collectRows(poolListRes?.data ?? poolListRes).filter((row) => normalizePoolId(row) && matchesChainId(row, chainId));
 }
 /**
  * 获取池子分级配置
@@ -279,9 +303,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 +315,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,31 +8,25 @@ 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",
 };
-const POOL_MANAGER_BY_CHAIN_BETA = {
-    421614: "0x05314a21Fc97B74f168730153b2B63A870D25dE5",
-    59141: "0xcf51a6895864c6D8E507fC31EF16b9011287c5f4",
-    97: "0x9E84a999e15CCdb2F64a5AF10939c25769dF6b07",
-    56: "0x9E84a999e15CCdb2F64a5AF10939c25769dF6b07",
-};
 const POOL_MANAGER_ABI = [
     "function deployPool((bytes32 marketId,address baseToken))",
     "function getMarketPool(bytes32 marketId,address asset) view returns ((bytes32 marketId,bytes32 poolId,address baseToken,address quoteToken,uint8 riskTier,uint8 state,address basePoolToken,address quotePoolToken,uint16 maxPriceDeviation,bool compoundEnabled,uint64 windowCapUsd,address poolVault,address tradingVault))",
 ];
 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",
@@ -45,28 +39,6 @@ const LP_ROUTER_BY_CHAIN = {
         quotePool: "0xB1E6df749A602892FafB27bb39Fd4F044527121E",
     },
 };
-const LP_ROUTER_BY_CHAIN_BETA = {
-    421614: {
-        router: "0xfb790ECE13Cd9e296b4a06ABF38D10431360c236",
-        basePool: "0x1F767BEa83EDe5E0C18904f439C579f52c2c0F1b",
-        quotePool: "0x50ad7da312c58c6689bEF028954a366cb5b8ee13",
-    },
-    59141: {
-        router: "0x0AE31989318565620c03d35889a0B2536b8fba5C",
-        basePool: "0xc9e3826c42183207B418116A16827C1605132c51",
-        quotePool: "0x9A9934D3b22103dE822d94A22A3177d728FE0e5a",
-    },
-    97: {
-        router: "0x0F4C6f18Fb136DD1eBd6Da3C5d86a86597CF79a3",
-        basePool: "0x51B62554a76197d5DF2D5dC4D57FF54d40775938",
-        quotePool: "0x783Ed065a12e1C1D33c2a8d6408385C1843D3084",
-    },
-    56: {
-        router: "0x0F4C6f18Fb136DD1eBd6Da3C5d86a86597CF79a3",
-        basePool: "0x51B62554a76197d5DF2D5dC4D57FF54d40775938",
-        quotePool: "0x783Ed065a12e1C1D33c2a8d6408385C1843D3084",
-    },
-};
 const PREVIEW_POOL_ABI = [
     "function previewLpAmountOut(bytes32,uint256,uint256) view returns (uint256)",
     "function previewQuoteAmountOut(bytes32,uint256,uint256) view returns (uint256)",
@@ -165,24 +137,19 @@ async function withMutedSdkAbiMismatchLogs(runner) {
         console.error = original;
     }
 }
-function isBetaModeEnabled() {
-    return String(process.env.IS_BETA_MODE ?? "").trim().toLowerCase() === "true";
-}
 function getPoolManagerAddress(chainId) {
     const envAddress = String(process.env.POOL_MANAGER_ADDRESS ?? "").trim();
     if (envAddress) {
         return normalizeAddress(envAddress, "POOL_MANAGER_ADDRESS");
     }
-    const source = isBetaModeEnabled() ? POOL_MANAGER_BY_CHAIN_BETA : POOL_MANAGER_BY_CHAIN;
-    const mapped = source[chainId];
+    const mapped = POOL_MANAGER_BY_CHAIN[chainId];
     if (!mapped) {
-        throw new Error(`Pool manager address is not configured for chainId=${chainId} (beta=${isBetaModeEnabled()}). Set POOL_MANAGER_ADDRESS env var.`);
+        throw new Error(`Pool manager address is not configured for chainId=${chainId}. Set POOL_MANAGER_ADDRESS env var.`);
     }
     return mapped;
 }
 function getLpAddresses(chainId) {
-    const source = isBetaModeEnabled() ? LP_ROUTER_BY_CHAIN_BETA : LP_ROUTER_BY_CHAIN;
-    const matched = source[chainId];
+    const matched = LP_ROUTER_BY_CHAIN[chainId];
     if (!matched) {
         throw new Error(`Liquidity router config not found for chainId=${chainId}.`);
     }
@@ -191,11 +158,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 +294,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 +308,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 +327,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/getAllTickers.js

@@ -15,7 +15,7 @@ export const getAllTickersTool = {
                 // Fallback for networks/environments where getAllTickers endpoint is unavailable.
                 const chainId = getChainId();
                 const poolList = await getPoolList(client, chainId);
-                const pools = Array.isArray(poolList?.data) ? poolList.data : (Array.isArray(poolList) ? poolList : []);
+                const pools = Array.isArray(poolList) ? poolList : (Array.isArray(poolList?.data) ? poolList.data : []);
                 const poolIds = pools.map((p) => p?.poolId ?? p?.pool_id).filter((id) => !!id);
                 if (poolIds.length === 0) {
                     throw new Error("Failed to fetch all tickers and no pools were available for fallback query.");

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/listPools.js

@@ -21,7 +21,8 @@ export const listPoolsTool = {
                 getPoolList(client),
                 client.markets.getPoolSymbolAll().catch(() => ({ data: [] }))
             ]);
-            const poolsRaw = collectRows(poolListRes?.data ?? poolListRes);
+            const poolsSource = Array.isArray(poolListRes) ? poolListRes : poolListRes?.data ?? poolListRes;
+            const poolsRaw = collectRows(poolsSource);
             const symbolsRaw = collectRows(symbolsRes?.data ?? symbolsRes);
             const symbolMap = new Map(symbolsRaw
                 .filter((row) => row?.poolId || row?.pool_id)

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