@michaleffffff/mcp-trading-server 3.0.5 → 3.0.10

package/CHANGELOG.md

@@ -1,5 +1,80 @@
 # Changelog
 
+## 3.0.10 - 2026-03-18
+
+### Fixed
+- Hardened `manage_tp_sl` delete behavior (`tpPrice=0` + `slPrice=0`):
+  - Added a unified cancellation path helper to always map delete intent to `cancelAllOrders` (by `orderId` or by `positionId`).
+  - Added fallback recovery for SDK/contract `InvalidParameter` reverts (including selector `0x613970e0`) so zero-price delete intent will still downgrade to explicit cancellation instead of failing.
+
+## 3.0.9 - 2026-03-18
+
+### Fixed
+- Fixed `create_perp_market` observability and compatibility:
+  - Added strict `marketId` (66-char hash) and `baseToken` address validation.
+  - Added structured error payloads with decoded contract selector hints (e.g. `PoolExists(PoolId)`).
+  - Added fallback for SDK v1.0.2 write-path incompatibility (`deployPool is not a function`) by directly calling `PoolManager.deployPool`.
+- Fixed `manage_tp_sl` delete semantics:
+  - `tpPrice=0` + `slPrice=0` now maps to explicit TP/SL order cancellation (by `orderId` or by `positionId`) instead of sending invalid on-chain TP/SL orders.
+- Fixed `get_base_detail` null-read behavior:
+  - `success + null` is now returned as a structured error (`NOT_FOUND`) with chain/base context.
+- Fixed `get_orders` OPEN status mapping:
+  - Removed `Unknown(undefined)` and normalized missing OPEN status to `Open`.
+- Fixed LP SDK wait incompatibility:
+  - Added submitted tx-hash recovery path when SDK returns `...wait is not a function` after broadcast (e.g., BASE withdraw).
+- Hardened direct LP router fallback:
+  - Added token allowance auto-approval before fallback deposit.
+
+### Verified (Real Funds, Arbitrum Sepolia 421614)
+- Deployed and minted new base token to active wallet:
+  - token: `0xDae49922Ff1699CA2A6cc4eE835B2c5a9f3Fe870`
+  - deploy tx: `0x99e69d66cac1b3a033281bae45dd421bba37794e1a80c1a12eddded99c48acce`
+- Created new perp pool for the token (marketId: `0x2a3fee38e8beba148141bea5cab0bcbbb0cf24fd5509117346991cc438cb2fe6`):
+  - create tx: `0x97266886c673cee13531837bcf9a0524034bd85a018036f3557c4d126fef3771`
+  - derived poolId: `0x6c1a8af5123a0cf636293aff3fce2ea6addd4bce172c39c6467d4dc95ac3f83e`
+- BASE LP add/remove validated on the new pool:
+  - add approval tx: `0xa2739346558f43f474953c7c93be05f0c66f2659f80d8b07804823f03926bdb0`
+  - add tx: `0x6a10d7dc5d2794643b10bc2b53080bfa8df1c99eedf35cd48a46edaa6d97d832`
+  - remove tx: `0x4148585452d259f6a9928404c7b8d324013e7ee13859392302ab78aa34fab2bc`
+
+## 3.0.8 - 2026-03-18
+
+### Fixed
+- Fixed `manage_liquidity` QUOTE pool ABI mismatch (`Expected length params=1, values=2`) by adding a safe fallback path:
+  - Keep SDK LP call as primary path.
+  - On SDK ABI-overload mismatch, switch to explicit-signature router transaction path (`depositQuote`/`withdrawQuote`) to avoid overloaded function ambiguity.
+- Added the same overload-mismatch fallback for BASE pool LP operations (`depositBase`/`withdrawBase`) to prevent the same class of failure.
+- Added SDK ABI-mismatch log suppression for LP calls to avoid noisy stack traces when fallback is activated.
+
+### Verified
+- Executed a real QUOTE remove liquidity transaction via MCP after the fix:
+  - txHash: `0x69e089b805cccd3b14d0c511309a0ce2aecf988344ec23ae27df929ad99af390`
+  - status: success (confirmed on-chain)
+
+## 3.0.7 - 2026-03-18
+
+### Changed
+- Updated server runtime version banner and MCP server version to `3.0.7`.
+- Hardened `get_pool_metadata` warning output by compacting long low-level errors into concise warnings.
+- Optimized `get_pool_info` read path:
+  - Prefer resolving a positive oracle/ticker market price and use it directly for pool info reads.
+  - Return clearer domain error for empty-liquidity / unresolved-price scenarios.
+- Refined `get_user_trading_fee_rate` error handling:
+  - Return structured MCP error envelope (`INVALID_PARAM` / `SDK_READ_ERROR`) instead of raw error strings.
+  - Include normalized concise error messages and request context details.
+- Enhanced `account_deposit` usability by making `tokenAddress` optional (defaults to `QUOTE_TOKEN_ADDRESS`).
+
+## 3.0.6 - 2026-03-18
+
+### Changed
+- Upgraded SDK dependency to `@myx-trade/sdk@^1.0.2`.
+- Updated client bootstrap to provide a viem-compatible `walletClient` shim (`json-rpc` account + `getAddresses/request/signMessage`) for SDK v1.0.2.
+- Moved `get_trade_flow` to SDK v1.0.2 native path (`client.api.getTradeFlow`).
+- Moved `account_withdraw` to SDK v1.0.2 native path (`account.updateAndWithdraw`).
+- Normalized `baseToken` address before `pool.createPool` to satisfy stricter typed input.
+- Updated account balance parsing to SDK v1.0.2 `getAccountInfo` fields (`freeMargin`, `walletBalance`).
+- Hardened type guards for account snapshot and fee-rate parsing under SDK union return types.
+
 ## 3.0.5 - 2026-03-18
 
 ### Changed

package/README.md

@@ -6,7 +6,8 @@ A production-ready MCP (Model Context Protocol) server for deep integration with
 
 # Release Notes
 
-- **Current release: 3.0.4**
+- **Current release: 3.0.10**
+- **SDK baseline**: `@myx-trade/sdk@^1.0.2` 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.
 - **Breaking changes**: Many low-level tools (e.g., `get_market_price`, `get_oracle_price`, `get_open_orders`) have been merged into unified counterparts.
@@ -53,7 +54,7 @@ QUOTE_TOKEN_DECIMALS=18
 * **`close_position`**: Strategy-based closing of specific positions.
 * **`close_all_positions`**: Emergency exit for all positions in a specific pool.
 * **`cancel_orders`**: Unified cancellation (Single ID, Pool-wide, or Account-wide).
-* **`manage_tp_sl`**: Adjust protection orders for active positions or pending orders.
+* **`manage_tp_sl`**: Adjust protection orders for active positions or pending orders. Deletion is supported via `tpPrice=0` + `slPrice=0`.
 * **`adjust_margin`**: Add or remove collateral to manage liquidation risk.
 
 ### 📁 Account & Portfolio

package/TOOL_EXAMPLES.md

@@ -60,6 +60,19 @@ Update existing protection orders or set new ones for a position.
 }
 ```
 
+Delete both TP/SL for a position:
+```json
+{
+  "name": "manage_tp_sl",
+  "arguments": {
+    "poolId": "BTC",
+    "positionId": "0xABC...",
+    "tpPrice": "0",
+    "slPrice": "0"
+  }
+}
+```
+
 ---
 
 ## 🔵 Market Data

package/dist/auth/resolveClient.js

@@ -24,6 +24,8 @@ export async function resolveClient() {
     const rpcUrl = process.env.RPC_URL || "https://rpc.sepolia.linea.build";
     const privateKey = process.env.PRIVATE_KEY;
     const chainId = Number(process.env.CHAIN_ID) || 59141;
+    const isTestnet = process.env.IS_TESTNET !== "false";
+    const isBetaMode = String(process.env.IS_BETA_MODE ?? "").trim().toLowerCase() === "true";
     const brokerAddressRaw = process.env.BROKER_ADDRESS || getDefaultBrokerByChainId(chainId);
     const quoteTokenRaw = process.env.QUOTE_TOKEN_ADDRESS || getDefaultQuoteTokenByChainId(chainId);
     const quoteDecimals = Number(process.env.QUOTE_TOKEN_DECIMALS) || 6;
@@ -42,12 +44,27 @@ export async function resolveClient() {
     // Inject the EIP-1193 mock so SDK can sign transactions seamlessly
     const { injectBrowserProviderMock } = await import("../utils/injectProvider.js");
     injectBrowserProviderMock(chainId, provider, signer);
+    const ethereumProvider = globalThis.window.ethereum;
+    const walletClient = {
+        transport: ethereumProvider,
+        chain: { id: chainId },
+        account: { address: signer.address, type: "json-rpc" },
+        getAddresses: async () => [signer.address],
+        request: async (args) => ethereumProvider.request(args),
+        signMessage: async ({ message }) => {
+            const payload = typeof message === "string"
+                ? message
+                : (message?.raw ?? message?.message ?? "");
+            return signer.signMessage(payload);
+        },
+    };
     const client = new MyxClient({
         chainId,
         signer: signer,
         brokerAddress,
-        isTestnet: process.env.IS_TESTNET !== "false",
-        walletClient: { transport: globalThis.window.ethereum }
+        isTestnet,
+        isBetaMode,
+        walletClient: walletClient
     });
     cached = { client, address: signer.address, signer, chainId, quoteToken, quoteDecimals };
     return cached;

package/dist/server.js

@@ -370,7 +370,7 @@ function zodSchemaToJsonSchema(zodSchema) {
     };
 }
 // ─── MCP Server ───
-const server = new Server({ name: "myx-mcp-trading-server", version: "3.0.5" }, { capabilities: { tools: {}, resources: {}, prompts: {} } });
+const server = new Server({ name: "myx-mcp-trading-server", version: "3.0.10" }, { capabilities: { tools: {}, resources: {}, prompts: {} } });
 // List tools
 server.setRequestHandler(ListToolsRequestSchema, async () => {
     return {
@@ -491,7 +491,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.5 running (stdio, pure on-chain, prod ready)");
+    logger.info("🚀 MYX Trading MCP Server v3.0.10 running (stdio, pure on-chain, prod ready)");
 }
 main().catch((err) => {
     logger.error("Fatal Server Startup Error", err);