@michaleffffff/mcp-trading-server 3.0.20 → 3.0.24

package/CHANGELOG.md

@@ -1,12 +1,62 @@
 # Changelog
 
-## 3.0.20 - 2026-03-19
+## 3.0.24 - 2026-03-19
+
+### Fixed
+- Hardened LP MCP behavior:
+  - `get_pool_metadata(includeLiquidity=true)` now ignores caller-supplied `marketPrice` and derives liquidity depth from a fresh Oracle price only.
+  - `get_pool_info` MCP-side market price resolution no longer falls back from fresh Oracle to ticker-derived price.
+  - Beta LP router / `PoolManager` fallback mappings were completed for MCP-managed LP and create-market paths.
+  - `get_my_lp_holdings` no longer ranks rows by mixed BASE/QUOTE LP raw balances.
+
+### Changed
+- Synced release metadata and operator docs for `v3.0.24`:
+  - Updated `README.md`, `TOOL_EXAMPLES.md`, and `trading_best_practices` prompt version strings.
+  - Refreshed LP safety docs to reflect Oracle-only liquidity metadata and inventory-only LP holdings semantics.
+
+## 3.0.23 - 2026-03-19
+
+### Changed
+- Upgraded SDK dependency to `@myx-trade/sdk@^1.0.4-beta.4`.
+- Refreshed operator-facing docs and runtime hints to match the new SDK baseline:
+  - `README.md` now references `@myx-trade/sdk@^1.0.4-beta.4`.
+  - `TOOL_EXAMPLES.md`, `execute_trade`, `close_position`, and `mapTimeInForce` now document the current IOC-only `timeInForce` behavior using `v1.0.4-beta.4`.
+  - `trading_best_practices` prompt now aligns with release `v3.0.23`.
+
+### Fixed
+- Hardened LP MCP behavior and synchronized docs:
+  - `get_pool_metadata(includeLiquidity=true)` now ignores caller-supplied `marketPrice` and derives liquidity depth from a fresh Oracle price only.
+  - `get_pool_info` MCP-side price resolution no longer falls back from fresh Oracle to ticker-derived market price.
+  - Beta LP router / `PoolManager` chain mappings were completed for the MCP fallback path.
+  - `get_my_lp_holdings` no longer ranks rows by mixed BASE/QUOTE LP raw balances.
+  - `README.md`, `TOOL_EXAMPLES.md`, and `mcp_config_guide.md` now document these LP-specific MCP semantics.
+
+## 3.0.22 - 2026-03-19
+
+### Changed
+- Upgraded SDK dependency to `@myx-trade/sdk@^1.0.4-beta.1`.
+- Refreshed operator-facing docs and runtime hints to match the new SDK baseline:
+  - `README.md` now references `@myx-trade/sdk@^1.0.4-beta.1`.
+  - `TOOL_EXAMPLES.md`, `execute_trade`, `close_position`, and `mapTimeInForce` now document the current IOC-only `timeInForce` behavior using the new SDK version.
+  - `trading_best_practices` prompt now aligns with release `v3.0.22`.
+
+## 3.0.21 - 2026-03-19
 
 ### Fixed
 - Hardened `get_pool_metadata` funding-rate formatting:
   - `fundingInfo.nextFundingRate` now preserves readable `%/秒` and `%/天` output for negative rates too.
   - Regression coverage now asserts both numeric funding-rate views and display strings.
 
+### Changed
+- Refreshed operator-facing docs and prompts to match the latest trading safety behavior:
+  - `README.md` now documents Oracle-only execution, exact-approval defaults, base-size semantics, TP/SL semantic checks, and LP preview fail-close.
+  - `mcp_config_guide.md` now includes required `BROKER_ADDRESS` configuration and testnet `MYXBroker` references.
+  - `TOOL_EXAMPLES.md` now reflects fresh-Oracle execution, live-direction validation, human-price TP/SL parsing, and trading `slippagePct` conventions.
+  - `trading_best_practices` prompt now aligns with the current MCP safety constraints and testnet broker references.
+- Updated testnet broker defaults:
+  - Arbitrum Sepolia `BROKER_ADDRESS` -> `0x895C4ae2A22bB26851011d733A9355f663a1F939`
+  - Linea Sepolia `BROKER_ADDRESS` -> `0x634EfDC9dC76D7AbF6E49279875a31B02E9891e2`
+
 ## 3.0.19 - 2026-03-19
 
 ### Fixed

package/README.md

@@ -6,10 +6,11 @@ A production-ready MCP (Model Context Protocol) server for deep integration with
 
 # Release Notes
 
-- **Current release: 3.0.20**
-- **SDK baseline**: `@myx-trade/sdk@^1.0.2` compatibility completed.
+- **Current release: 3.0.24**
+- **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.
+- **Safety refresh**: Docs and prompt guidance now reflect Oracle-only execution, exact-approval defaults, notional-based fee checks, TP/SL semantic validation, and LP preview fail-close behavior.
 - **Breaking changes**: Many low-level tools (e.g., `get_market_price`, `get_oracle_price`, `get_open_orders`) have been merged into unified counterparts.
 
 ---
@@ -31,13 +32,20 @@ Copy `.env.example` to `.env` and configure your trading wallet:
 
 ```bash
 PRIVATE_KEY=0x...
-RPC_URL=https://bsc-dataseed.bnbchain.org
-CHAIN_ID=56
+RPC_URL=https://your-testnet-or-mainnet-rpc
+CHAIN_ID=...
 BROKER_ADDRESS=0x...
 QUOTE_TOKEN_ADDRESS=0x...
-QUOTE_TOKEN_DECIMALS=18
+QUOTE_TOKEN_DECIMALS=...
 ```
 
+## Testnet `MYXBroker` Reference
+
+- Arbitrum test: `0x895C4ae2A22bB26851011d733A9355f663a1F939`
+- Linea test: `0x634EfDC9dC76D7AbF6E49279875a31B02E9891e2`
+
+Use the broker that matches your active RPC and chain configuration.
+
 ---
 
 # Core Tools Reference
@@ -47,7 +55,7 @@ QUOTE_TOKEN_DECIMALS=18
 * **`list_pools`**: List all tradable assets on the current chain.
 * **`search_tools`**: Discover the right MCP tool by keyword, legacy tool name, or intent phrase.
 * **`get_price`**: Fetch real-time prices (Impact Market Price or Oracle Price).
-* **`get_pool_metadata`**: Comprehensive metrics (Fees, Open Interest, Liquidity Depth).
+* **`get_pool_metadata`**: Comprehensive metrics (Fees, Open Interest, Liquidity Depth via fresh Oracle price).
 * **`get_kline`**: Fetch candlestick data for technical analysis.
 
 ### ⚔️ Trading Operations
@@ -77,6 +85,19 @@ QUOTE_TOKEN_DECIMALS=18
 
 ---
 
+# Safety Defaults
+
+- **Oracle-only execution**: trading paths now require a fresh Oracle price and reject stale / missing execution prices.
+- **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.
+- **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 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.
+
+---
+
 # Tool Discovery
 
 When a client or LLM is unsure which tool to call:
@@ -108,9 +129,10 @@ Use these conventions when generating tool arguments:
 - `status`: `OPEN` / `HISTORY` / `ALL` are canonical; lowercase is tolerated
 - `poolType`: `BASE` / `QUOTE` are canonical; lowercase is tolerated
 - `orderType`: `MARKET` / `LIMIT` / `STOP` / `CONDITIONAL`
-- `timeInForce`: SDK `v1.0.2` currently supports `IOC` only, so use `0` or `"IOC"`
+- `timeInForce`: SDK `v1.0.4-beta.4` currently supports `IOC` only, so use `0` or `"IOC"`
 - `size`: base token quantity, not USD notional; expected order value is usually `collateralAmount * leverage`
 - `executionFeeToken`: must be a real token address; zero address is rejected. Use the pool `quoteToken`
+- `slippagePct`: trading tools use 4-decimal raw units where `100 = 1.00%` and `50 = 0.50%`
 - Human units: `"100"` means 100 USDC or 100 token units depending on field
 - Raw units: `"raw:1000000"` means exact on-chain integer units
 
@@ -136,6 +158,8 @@ Examples:
 }
 ```
 
+For `get_pool_metadata(includeLiquidity=true)`, do not rely on a custom `marketPrice` override. MCP now ignores that field and uses a fresh Oracle price automatically.
+
 ```json
 {
   "name": "open_position_simple",

package/TOOL_EXAMPLES.md

@@ -1,4 +1,4 @@
-# MYX MCP Tool Examples Handbook (v3.0.20)
+# MYX MCP Tool Examples Handbook (v3.0.24)
 
 This guide provides practical MCP payload examples for the current unified toolset.
 All examples use the MCP format:
@@ -7,6 +7,13 @@ All examples use the MCP format:
 { "name": "tool_name", "arguments": { "...": "..." } }
 ```
 
+## Environment Notes
+
+- Common testnet brokers:
+  - Arbitrum test: `0x895C4ae2A22bB26851011d733A9355f663a1F939`
+  - Linea test: `0x634EfDC9dC76D7AbF6E49279875a31B02E9891e2`
+- Keep `RPC_URL`, `CHAIN_ID`, and `BROKER_ADDRESS` on the same network.
+
 ---
 
 ## Discovery First
@@ -75,6 +82,8 @@ Recommended high-level entry tool.
 `marketId` is optional on `open_position_simple`. If supplied, it is validated against the market resolved from `poolId` or `keyword`.
 `size` is always the base-asset quantity, not the USD notional. For example, a 500 USD order at price 1200 implies `size ≈ 0.416666...`.
 `collateralAmount` remains required on `open_position_simple`; if omitted, MCP now returns an actionable suggestion instead of a generic parse error.
+If `price` is omitted, the tool uses a fresh Oracle price only; stale or unresolved execution prices are rejected.
+Auto-computed `tradingFee` follows notional semantics rather than raw collateral-only estimation.
 
 Raw-units example:
 
@@ -111,8 +120,9 @@ Low-level increase-order tool when you want full control.
 }
 ```
 
-`timeInForce` should be `0` (or `"IOC"` in string form) for SDK `v1.0.2`.
+`timeInForce` should be `0` (or `"IOC"` in string form) for SDK `v1.0.4-beta.4`.
 `executionFeeToken` must be a real token address; do not pass the zero address. Use the pool `quoteToken`.
+If `positionId` is supplied on increase flows, `direction` must remain consistent with the live position.
 
 ### `close_position`
 Close or reduce a position. Use `ALL` for a full close.
@@ -137,6 +147,8 @@ Close or reduce a position. Use `ALL` for a full close.
 }
 ```
 
+`direction` must match the live position. MCP now validates live direction before sending the close request.
+
 ### `manage_tp_sl`
 Create or update TP/SL on an open position.
 
@@ -154,6 +166,9 @@ Create or update TP/SL on an open position.
 }
 ```
 
+Plain integer prices such as `"2800"` are treated as human prices, not implicit raw 30-decimal values.
+For LONG positions, use `tpPrice > entryPrice` and `slPrice < entryPrice`. For SHORT positions, use the inverse.
+
 Delete both TP/SL orders:
 
 ```json
@@ -234,6 +249,7 @@ Unified pool detail, config, and liquidity info.
 ```
 
 `get_pool_metadata` returns raw values in `poolInfo` and precision-safe human-readable values in `poolInfoFormatted`, including readable funding epoch timestamps, funding-rate `%/秒` and `%/天`, and IO notional-at-entry views.
+When `includeLiquidity=true`, MCP derives liquidity depth from a fresh Oracle price automatically. Any caller-supplied `marketPrice` is now treated as deprecated and ignored.
 
 ### `get_kline`
 Read chart data. Use `limit: 1` for the latest bar.
@@ -284,6 +300,9 @@ 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.
+
 ### `get_lp_price`
 Read LP NAV price for BASE or QUOTE side.
 
@@ -312,6 +331,8 @@ Read current LP balances across pools.
 }
 ```
 
+`get_my_lp_holdings` is an inventory view. Results are now stably ordered by symbol / pool id instead of summing BASE LP raw units with QUOTE LP raw units into a misleading mixed-unit ranking.
+
 ---
 
 ## Account And Monitoring
@@ -391,3 +412,5 @@ Read open positions, history, or both.
 4. Canonical enums are still preferred:
    `OPEN|HISTORY|ALL`, `BASE|QUOTE`, `LONG|SHORT`, `MARKET|LIMIT|STOP`.
 5. The server tolerates common lowercase and alias forms for better AI compatibility.
+6. Trading `slippagePct` uses 4-decimal raw units, so `100 = 1.00%` and `50 = 0.50%`.
+7. High-risk execution paths prefer fresh Oracle pricing, exact approval sizing, and fail-close behavior on missing previews or invalid units.

package/dist/auth/resolveClient.js

@@ -5,10 +5,10 @@ let cached = null;
 function getDefaultBrokerByChainId(chainId) {
     // Testnet mappings
     if (chainId === 421614)
-        return "0x8f3153C18f698166f5D6124d8ba5B567F5f120f9"; // Arbitrum Sepolia
+        return "0x895C4ae2A22bB26851011d733A9355f663a1F939"; // Arbitrum Sepolia
     if (chainId === 59141)
-        return "0x0FB08D3A1Ea6bE515fe78D3e0CaEb6990b468Cf3"; // Linea Sepolia
-    return "0x8f3153C18f698166f5D6124d8ba5B567F5f120f9";
+        return "0x634EfDC9dC76D7AbF6E49279875a31B02E9891e2"; // Linea Sepolia
+    return "0x895C4ae2A22bB26851011d733A9355f663a1F939";
 }
 function getDefaultQuoteTokenByChainId(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.12)
+# MYX Trading MCP Best Practices (v3.0.24)
 
 You are an expert crypto trader using the MYX Protocol. To ensure successful execution and safe handling of user funds, follow these patterns:
 
@@ -28,21 +28,29 @@ You are an expert crypto trader using the MYX Protocol. To ensure successful exe
 - **Consolidated Tools**: Many legacy tools have been merged. Always use the high-level versions (e.g., \`get_price\` instead of \`get_market_price\`).
 - **Discovery**: \`search_tools\` understands legacy names like \`get_open_orders\` and intent phrases like \`add base lp\`.
 - **Unit Prefixes**: Prefer \`human:\` for readable amounts (e.g., "100" USDC) and \`raw:\` for exact on-chain units.
-- **Slippage**: Default is 100 (1%). For volatile tokens, consider 200-300 (2-3%).
+- **Slippage**: Trading tools use 4-decimal raw units where \`100 = 1.00%\` and \`50 = 0.50%\`. Keep it tight unless the market is genuinely illiquid.
 - **Fees**: Use \`get_pool_metadata\` to view current fee tiers and pool configuration.
+- **Liquidity Metadata**: When calling \`get_pool_metadata(includeLiquidity=true)\`, MCP uses a fresh Oracle price automatically and ignores caller-supplied \`marketPrice\`.
 - **LP Strategy**: Use \`get_my_lp_holdings\` to monitor liquidity positions. Naming follows \`mBASE.QUOTE\` (e.g., \`mBTC.USDC\`).
 - **Enum Tolerance**: The server tolerates common lowercase or alias inputs such as \`open\`, \`base\`, \`buy\`, and \`add\`, but canonical forms are still preferred in documentation.
+- **Oracle Safety**: Execution flows now require a fresh Oracle price. Do not fall back to stale ticker or user-supplied execution prices when the Oracle is unavailable.
+- **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.
+- **LP Safety**: LP execution requires a fresh price snapshot and preview success; do not continue after preview failure.
+- **LP Read Semantics**: Treat \`get_my_lp_holdings\` as an inventory listing, not a portfolio ranking by economic value.
+
+## 3. Testnet Broker Reference
+- Arbitrum test: \`0x895C4ae2A22bB26851011d733A9355f663a1F939\`
+- Linea test: \`0x634EfDC9dC76D7AbF6E49279875a31B02E9891e2\`
+- Always keep \`RPC_URL\`, \`CHAIN_ID\`, and \`BROKER_ADDRESS\` on the same network.
 
 Current Session:
 - Wallet: ${address}
 - Chain ID: ${chainId}
 
-## 3. Self-Healing
+## 4. Self-Healing
 If a transaction reverts with a hex code, the server will attempt to decode it (e.g., "AccountInsufficientFreeAmount"). Error payloads now include structured \`code/hint/action\` fields; use them to provide concrete next steps.
-
-Current Session:
-- Wallet: ${address}
-- Chain ID: ${chainId}
 `
                     }
                 }

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.20" }, { capabilities: { tools: {}, resources: {}, prompts: {} } });
+const server = new Server({ name: "myx-mcp-trading-server", version: "3.0.24" }, { 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.20 running (stdio, pure on-chain, prod ready)");
+    logger.info("🚀 MYX Trading MCP Server v3.0.24 running (stdio, pure on-chain, prod ready)");
 }
 main().catch((err) => {
     logger.error("Fatal Server Startup Error", err);

package/dist/services/marketService.js

@@ -1,5 +1,6 @@
 import { getChainId } from "../auth/resolveClient.js";
 import { getMarketStateDesc } from "../utils/mappings.js";
+export const DEFAULT_ORACLE_MAX_AGE_SEC = Number(process.env.ORACLE_MAX_AGE_SEC ?? 90);
 function collectRows(input) {
     if (Array.isArray(input))
         return input.flatMap(collectRows);
@@ -92,6 +93,46 @@ export async function getOraclePrice(client, poolId, chainIdOverride) {
     const chainId = chainIdOverride ?? getChainId();
     return client.utils.getOraclePrice(poolId, chainId);
 }
+function parseOraclePublishTime(value, poolId) {
+    const text = String(value ?? "").trim();
+    if (!/^\d+$/.test(text)) {
+        throw new Error(`Oracle publishTime missing or invalid for poolId=${poolId}.`);
+    }
+    const parsed = BigInt(text);
+    if (parsed <= 0n) {
+        throw new Error(`Oracle publishTime must be positive for poolId=${poolId}.`);
+    }
+    return parsed;
+}
+export function assertOracleFreshness(publishTimeValue, poolId, maxAgeSec = DEFAULT_ORACLE_MAX_AGE_SEC) {
+    if (!Number.isFinite(maxAgeSec) || maxAgeSec <= 0) {
+        throw new Error(`Invalid oracle max age configuration: ${maxAgeSec}`);
+    }
+    const publishTime = parseOraclePublishTime(publishTimeValue, poolId);
+    const nowSec = BigInt(Math.floor(Date.now() / 1000));
+    const maxAge = BigInt(Math.floor(maxAgeSec));
+    if (publishTime > nowSec + 5n) {
+        throw new Error(`Oracle publishTime is in the future for poolId=${poolId}.`);
+    }
+    const age = nowSec - publishTime;
+    if (age > maxAge) {
+        throw new Error(`Oracle price expired for poolId=${poolId}: age=${age.toString()}s exceeds maxAge=${maxAge.toString()}s.`);
+    }
+    return publishTime;
+}
+export async function getFreshOraclePrice(client, poolId, chainIdOverride, maxAgeSec = DEFAULT_ORACLE_MAX_AGE_SEC) {
+    const oracle = await getOraclePrice(client, poolId, chainIdOverride);
+    const price = String(oracle?.price ?? "").trim();
+    if (!price) {
+        throw new Error(`Oracle price missing for poolId=${poolId}.`);
+    }
+    const publishTime = assertOracleFreshness(oracle?.publishTime, poolId, maxAgeSec);
+    return {
+        ...oracle,
+        price,
+        publishTime: publishTime.toString(),
+    };
+}
 export async function searchMarket(client, keyword, limit = 1000, chainIdOverride) {
     const chainId = chainIdOverride ?? getChainId();
     const normalizedKeyword = String(keyword ?? "").trim();

package/dist/services/poolService.js

@@ -1,13 +1,23 @@
 import { pool, quote, base } from "@myx-trade/sdk";
 import { getChainId, resolveClient } from "../auth/resolveClient.js";
 import { extractErrorMessage } from "../utils/errorMessage.js";
-import { ensureUnits } from "../utils/units.js";
+import { ensureUnits, parseUserUnits } from "../utils/units.js";
 import { normalizeAddress } from "../utils/address.js";
-import { Contract, parseUnits, MaxUint256 } from "ethers";
+import { Contract } from "ethers";
 import { logger } from "../utils/logger.js";
+import { assertOracleFreshness, getFreshOraclePrice } from "./marketService.js";
 const LP_DECIMALS = 18;
 const POOL_MANAGER_BY_CHAIN = {
     421614: "0xf268D9FeD3Bd56fd9aBdb4FeEb993338613678A8",
+    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))",
@@ -51,6 +61,11 @@ const LP_ROUTER_BY_CHAIN_BETA = {
         basePool: "0x51B62554a76197d5DF2D5dC4D57FF54d40775938",
         quotePool: "0x783Ed065a12e1C1D33c2a8d6408385C1843D3084",
     },
+    56: {
+        router: "0x0F4C6f18Fb136DD1eBd6Da3C5d86a86597CF79a3",
+        basePool: "0x51B62554a76197d5DF2D5dC4D57FF54d40775938",
+        quotePool: "0x783Ed065a12e1C1D33c2a8d6408385C1843D3084",
+    },
 };
 const PREVIEW_POOL_ABI = [
     "function previewLpAmountOut(bytes32,uint256,uint256) view returns (uint256)",
@@ -150,9 +165,10 @@ function getPoolManagerAddress(chainId) {
     if (envAddress) {
         return normalizeAddress(envAddress, "POOL_MANAGER_ADDRESS");
     }
-    const mapped = POOL_MANAGER_BY_CHAIN[chainId];
+    const source = isBetaModeEnabled() ? POOL_MANAGER_BY_CHAIN_BETA : POOL_MANAGER_BY_CHAIN;
+    const mapped = source[chainId];
     if (!mapped) {
-        throw new Error(`Pool manager address is not configured for chainId=${chainId}. Set POOL_MANAGER_ADDRESS env var.`);
+        throw new Error(`Pool manager address is not configured for chainId=${chainId} (beta=${isBetaModeEnabled()}). Set POOL_MANAGER_ADDRESS env var.`);
     }
     return mapped;
 }
@@ -202,10 +218,13 @@ async function buildOraclePricePayload(client, chainId, poolId, fallbackOracleTy
     if (!vaa || !vaa.startsWith("0x")) {
         throw new Error(`Oracle VAA unavailable for pool ${poolId}.`);
     }
-    const publishTime = toPositiveBigint(oracle?.publishTime) ?? BigInt(Math.floor(Date.now() / 1000));
+    const publishTime = assertOracleFreshness(oracle?.publishTime, poolId);
     const oracleType = Number.isFinite(Number(oracle?.oracleType)) ? Number(oracle.oracleType) : fallbackOracleType;
     const value = toPositiveBigint(oracle?.value) ?? 0n;
     const referencePrice30 = BigInt(ensureUnits(String(oracle?.price ?? "0"), 30, "oracle price"));
+    if (referencePrice30 <= 0n) {
+        throw new Error(`Oracle price must be positive for pool ${poolId}.`);
+    }
     return {
         prices: [[poolId, oracleType, publishTime, vaa]],
         value,
@@ -229,8 +248,7 @@ async function previewAmountOutForLiquidity(signer, chainId, poolId, poolType, a
         return toPositiveBigint(out) ?? 0n;
     }
     catch (error) {
-        logger.warn("LP preview call failed; fallback to minAmountOut=0.", extractErrorMessage(error));
-        return 0n;
+        throw new Error(`LP preview failed: ${extractErrorMessage(error)}`);
     }
 }
 async function executeLiquidityTxViaRouter(params) {
@@ -244,7 +262,7 @@ async function executeLiquidityTxViaRouter(params) {
     if (!Number.isFinite(decimals) || decimals < 0) {
         throw new Error(`Invalid decimals while preparing ${poolType} ${action} transaction.`);
     }
-    const amountIn = parseUnits(String(amount), decimals);
+    const amountIn = BigInt(parseUserUnits(String(amount), decimals, "amount"));
     if (amountIn <= 0n) {
         throw new Error(`Liquidity ${poolType.toLowerCase()} ${action} amount must be > 0.`);
     }
@@ -255,7 +273,7 @@ async function executeLiquidityTxViaRouter(params) {
         const allowance = toPositiveBigint(await tokenContract.allowance(address, addresses.router)) ?? 0n;
         if (allowance < amountIn) {
             logger.info(`[LP fallback] allowance insufficient for ${poolType} deposit, approving router. required=${amountIn.toString()}, current=${allowance.toString()}`);
-            const approveTx = await tokenContract.approve(addresses.router, MaxUint256);
+            const approveTx = await tokenContract.approve(addresses.router, amountIn);
             approvalTxHash = String(approveTx?.hash ?? "").trim() || null;
             const approveReceipt = await approveTx?.wait?.();
             if (approveReceipt && approveReceipt.status !== 1) {
@@ -350,28 +368,14 @@ async function resolvePositiveMarketPrice30(client, poolId, chainId) {
     if (!client)
         return null;
     try {
-        const oracle = await client.utils?.getOraclePrice?.(poolId, chainId);
-        const byValue = toPositiveBigint(oracle?.value);
-        const byPrice = toPositiveBigint(oracle?.price);
-        if (byValue)
-            return byValue;
+        const oracle = await getFreshOraclePrice(client, poolId, chainId);
+        const oraclePrice30 = ensureUnits(String(oracle?.price ?? "").trim(), 30, "oracle price");
+        const byPrice = toPositiveBigint(oraclePrice30);
         if (byPrice)
             return byPrice;
     }
     catch {
     }
-    try {
-        const tickerRes = await client.markets?.getTickerList?.({ chainId, poolIds: [poolId] });
-        const row = Array.isArray(tickerRes) ? tickerRes[0] : tickerRes?.data?.[0];
-        if (row?.price) {
-            const tickerRaw = ensureUnits(row.price, 30, "ticker price");
-            const byTicker = toPositiveBigint(tickerRaw);
-            if (byTicker)
-                return byTicker;
-        }
-    }
-    catch {
-    }
     return null;
 }
 /**
@@ -472,89 +476,46 @@ export async function getPoolDetail(poolId, chainIdOverride) {
 /**
  * 获取流动性信息
  */
-export async function getLiquidityInfo(client, poolId, marketPrice, chainIdOverride) {
+export async function getLiquidityInfo(client, poolId, chainIdOverride) {
     const chainId = chainIdOverride ?? getChainId();
-    return client.utils.getLiquidityInfo({ chainId, poolId, marketPrice });
+    const oracle = await getFreshOraclePrice(client, poolId, chainId);
+    const marketPrice = ensureUnits(String(oracle?.price ?? "").trim(), 30, "oracle price");
+    const liquidityInfo = await client.utils.getLiquidityInfo({ chainId, poolId, marketPrice });
+    return {
+        liquidityInfo,
+        marketPrice,
+        marketPriceSource: "oracle",
+        oraclePublishTime: String(oracle?.publishTime ?? ""),
+        oracleType: oracle?.oracleType ?? null,
+    };
 }
 /**
  * Quote 池 deposit
  */
 export async function quoteDeposit(poolId, amount, slippage, chainIdOverride) {
     const chainId = chainIdOverride ?? getChainId();
-    const txHashBefore = readLastMockTxHash();
-    try {
-        return await withMutedSdkAbiMismatchLogs(() => quote.deposit({ chainId, poolId, amount, slippage }));
-    }
-    catch (error) {
-        const message = extractErrorMessage(error);
-        const recovered = recoverSdkSubmittedTxHash(txHashBefore, message, { poolType: "QUOTE", action: "deposit" });
-        if (recovered)
-            return recovered;
-        if (!isAbiLengthMismatchError(message))
-            throw error;
-        logger.warn("quote.deposit hit SDK ABI mismatch; switching to explicit router path.");
-        return executeLiquidityTxViaRouter({ chainId, poolId, poolType: "QUOTE", action: "deposit", amount, slippage });
-    }
+    return executeLiquidityTxViaRouter({ chainId, poolId, poolType: "QUOTE", action: "deposit", amount, slippage });
 }
 /**
  * Quote 池 withdraw
  */
 export async function quoteWithdraw(poolId, amount, slippage, chainIdOverride) {
     const chainId = chainIdOverride ?? getChainId();
-    const txHashBefore = readLastMockTxHash();
-    try {
-        return await withMutedSdkAbiMismatchLogs(() => quote.withdraw({ chainId, poolId, amount, slippage }));
-    }
-    catch (error) {
-        const message = extractErrorMessage(error);
-        const recovered = recoverSdkSubmittedTxHash(txHashBefore, message, { poolType: "QUOTE", action: "withdraw" });
-        if (recovered)
-            return recovered;
-        if (!isAbiLengthMismatchError(message))
-            throw error;
-        logger.warn("quote.withdraw hit SDK ABI mismatch; switching to explicit router path.");
-        return executeLiquidityTxViaRouter({ chainId, poolId, poolType: "QUOTE", action: "withdraw", amount, slippage });
-    }
+    return executeLiquidityTxViaRouter({ chainId, poolId, poolType: "QUOTE", action: "withdraw", amount, slippage });
 }
 /**
  * Base 池 deposit
  */
 export async function baseDeposit(poolId, amount, slippage, chainIdOverride) {
     const chainId = chainIdOverride ?? getChainId();
-    const txHashBefore = readLastMockTxHash();
-    try {
-        return await withMutedSdkAbiMismatchLogs(() => base.deposit({ chainId, poolId, amount, slippage }));
-    }
-    catch (error) {
-        const message = extractErrorMessage(error);
-        const recovered = recoverSdkSubmittedTxHash(txHashBefore, message, { poolType: "BASE", action: "deposit" });
-        if (recovered)
-            return recovered;
-        if (!isAbiLengthMismatchError(message))
-            throw error;
-        logger.warn("base.deposit hit SDK ABI mismatch; switching to explicit router path.");
-        return executeLiquidityTxViaRouter({ chainId, poolId, poolType: "BASE", action: "deposit", amount, slippage });
-    }
+    return executeLiquidityTxViaRouter({ chainId, poolId, poolType: "BASE", action: "deposit", amount, slippage });
 }
 /**
  * Base 池 withdraw
  */
 export async function baseWithdraw(poolId, amount, slippage, chainIdOverride) {
     const chainId = chainIdOverride ?? getChainId();
-    const txHashBefore = readLastMockTxHash();
-    try {
-        return await withMutedSdkAbiMismatchLogs(() => base.withdraw({ chainId, poolId, amount, slippage }));
-    }
-    catch (error) {
-        const message = extractErrorMessage(error);
-        const recovered = recoverSdkSubmittedTxHash(txHashBefore, message, { poolType: "BASE", action: "withdraw" });
-        if (recovered)
-            return recovered;
-        if (!isAbiLengthMismatchError(message))
-            throw error;
-        logger.warn("base.withdraw hit SDK ABI mismatch; switching to explicit router path.");
-        return executeLiquidityTxViaRouter({ chainId, poolId, poolType: "BASE", action: "withdraw", amount, slippage });
-    }
+    return executeLiquidityTxViaRouter({ chainId, poolId, poolType: "BASE", action: "withdraw", amount, slippage });
 }
 /**
  * 获取 LP 价格