flash-trade-mcp 0.3.0 → 0.3.2

package/dist/index.js

@@ -21063,7 +21063,7 @@ var require_fallback = __commonJS((exports, module) => {
 
 // node_modules/bufferutil/index.js
 var require_bufferutil = __commonJS((exports, module) => {
-  var __dirname = "/Users/kalebrupe/Desktop/flash-trade-MCP/mcp/node_modules/bufferutil";
+  var __dirname = "/home/runner/work/flash-trade-MCP/flash-trade-MCP/mcp/node_modules/bufferutil";
   try {
     module.exports = require_node_gyp_build2()(__dirname);
   } catch (e) {
@@ -21479,7 +21479,7 @@ var require_fallback2 = __commonJS((exports, module) => {
 
 // node_modules/utf-8-validate/index.js
 var require_utf_8_validate = __commonJS((exports, module) => {
-  var __dirname = "/Users/kalebrupe/Desktop/flash-trade-MCP/mcp/node_modules/utf-8-validate";
+  var __dirname = "/home/runner/work/flash-trade-MCP/flash-trade-MCP/mcp/node_modules/utf-8-validate";
   try {
     module.exports = require_node_gyp_build2()(__dirname);
   } catch (e) {
@@ -49053,6 +49053,15 @@ function loadConfig() {
   if (!apiBaseUrl) {
     throw new Error("FLASH_API_URL environment variable is required");
   }
+  let parsed;
+  try {
+    parsed = new URL(apiBaseUrl);
+  } catch {
+    throw new Error(`FLASH_API_URL is not a valid URL: ${apiBaseUrl}`);
+  }
+  if (parsed.protocol !== "https:") {
+    console.error(`[flash-trade-mcp] WARNING: FLASH_API_URL uses ${parsed.protocol} — HTTPS is strongly recommended for production`);
+  }
   return {
     apiBaseUrl: apiBaseUrl.replace(/\/$/, ""),
     timeoutMs: parseInt(process.env.FLASH_API_TIMEOUT ?? "30000", 10),
@@ -49069,8 +49078,8 @@ class FlashApiError extends Error {
     super(`Flash API error [${statusCode}] ${endpoint}: ${message}`);
     this.statusCode = statusCode;
     this.endpoint = endpoint;
-    this.responseBody = responseBody;
     this.name = "FlashApiError";
+    this.responseBody = responseBody?.slice(0, 500);
   }
 }
 
@@ -49137,51 +49146,51 @@ class FlashApiClient {
     return this.get("/markets");
   }
   async getMarket(pubkey) {
-    return this.get(`/markets/${pubkey}`);
+    return this.get(`/markets/${encodeURIComponent(pubkey)}`);
   }
   async getPools() {
     return this.get("/pools");
   }
   async getPool(pubkey) {
-    return this.get(`/pools/${pubkey}`);
+    return this.get(`/pools/${encodeURIComponent(pubkey)}`);
   }
   async getCustodies() {
     return this.get("/custodies");
   }
   async getCustody(pubkey) {
-    return this.get(`/custodies/${pubkey}`);
+    return this.get(`/custodies/${encodeURIComponent(pubkey)}`);
   }
   async getPrices() {
     return this.get("/prices");
   }
   async getPrice(symbol2) {
-    return this.get(`/prices/${symbol2}`);
+    return this.get(`/prices/${encodeURIComponent(symbol2)}`);
   }
   async getPositions(owner) {
-    const query = owner ? `?owner=${owner}` : "";
+    const query = owner ? `?owner=${encodeURIComponent(owner)}` : "";
     return this.get(`/positions${query}`);
   }
   async getPosition(pubkey) {
-    return this.get(`/positions/${pubkey}`);
+    return this.get(`/positions/${encodeURIComponent(pubkey)}`);
   }
   async getOwnerPositions(owner, includePnlInLeverage = false) {
-    return this.get(`/positions/owner/${owner}?includePnlInLeverageDisplay=${includePnlInLeverage}`);
+    return this.get(`/positions/owner/${encodeURIComponent(owner)}?includePnlInLeverageDisplay=${includePnlInLeverage}`);
   }
   async getOrders(owner) {
-    const query = owner ? `?owner=${owner}` : "";
+    const query = owner ? `?owner=${encodeURIComponent(owner)}` : "";
     return this.get(`/orders${query}`);
   }
   async getOrder(pubkey) {
-    return this.get(`/orders/${pubkey}`);
+    return this.get(`/orders/${encodeURIComponent(pubkey)}`);
   }
   async getOwnerOrders(owner) {
-    return this.get(`/orders/owner/${owner}`);
+    return this.get(`/orders/owner/${encodeURIComponent(owner)}`);
   }
   async getPoolData() {
     return this.get("/pool-data");
   }
   async getPoolSnapshot(poolPubkey) {
-    return this.get(`/pool-data/${poolPubkey}`);
+    return this.get(`/pool-data/${encodeURIComponent(poolPubkey)}`);
   }
   async openPosition(req) {
     return this.post("/transaction-builder/open-position", req);
@@ -49198,6 +49207,18 @@ class FlashApiClient {
   async reversePosition(req) {
     return this.post("/transaction-builder/reverse-position", req);
   }
+  async placeTriggerOrder(req) {
+    return this.post("/transaction-builder/place-trigger-order", req);
+  }
+  async editTriggerOrder(req) {
+    return this.post("/transaction-builder/edit-trigger-order", req);
+  }
+  async cancelTriggerOrder(req) {
+    return this.post("/transaction-builder/cancel-trigger-order", req);
+  }
+  async cancelAllTriggerOrders(req) {
+    return this.post("/transaction-builder/cancel-all-trigger-orders", req);
+  }
   async previewLimitOrderFees(req) {
     return this.post("/preview/limit-order-fees", req);
   }
@@ -49220,8 +49241,9 @@ function registerHealthTools(server, client) {
     const health = await client.getHealth();
     const lines = [`Status: ${health.status}`];
     for (const [key, val] of Object.entries(health)) {
-      if (key !== "status")
-        lines.push(`${key}: ${val}`);
+      if (key !== "status") {
+        lines.push(`${key}: ${typeof val === "object" ? JSON.stringify(val) : val}`);
+      }
     }
     return { content: [{ type: "text", text: lines.join(`
 `) }] };
@@ -49307,7 +49329,7 @@ function registerMarketTools(server, client) {
   });
   server.registerTool("get_market", {
     description: "Get detailed information about a specific market by its on-chain account pubkey. Returns full market configuration including permissions and collective position data.",
-    inputSchema: { pubkey: exports_external.string().describe("Solana pubkey of the market account") }
+    inputSchema: { pubkey: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Solana pubkey of the market account") }
   }, async ({ pubkey }) => {
     const market = await client.getMarket(pubkey);
     return { content: [{ type: "text", text: JSON.stringify(market, null, 2) }] };
@@ -49320,11 +49342,25 @@ function registerPoolTools(server, client) {
     description: "List all liquidity pools on Flash Trade. Pools hold the collateral that backs perpetual positions. Returns pool addresses and configuration."
   }, async () => {
     const pools = await client.getPools();
-    return { content: [{ type: "text", text: JSON.stringify(pools, null, 2) }] };
+    const lines = [
+      `${pools.length} pools available:
+`,
+      "Pool Name      | Custodies | Pubkey",
+      "---------------|-----------|----------------------------------------------"
+    ];
+    for (const p of pools) {
+      const name = (p.account?.name ?? "Unknown").padEnd(15);
+      const custodyCount = String(p.account?.custodies?.length ?? 0).padEnd(10);
+      lines.push(`${name}| ${custodyCount}| ${p.pubkey}`);
+    }
+    lines.push(`
+Use get_pool with a pubkey for full details. Use get_pool_data for AUM and utilization metrics.`);
+    return { content: [{ type: "text", text: lines.join(`
+`) }] };
   });
   server.registerTool("get_pool", {
     description: "Get detailed information about a specific pool by its on-chain account pubkey.",
-    inputSchema: { pubkey: exports_external.string().describe("Solana pubkey of the pool account") }
+    inputSchema: { pubkey: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Solana pubkey of the pool account") }
   }, async ({ pubkey }) => {
     const pool = await client.getPool(pubkey);
     return { content: [{ type: "text", text: JSON.stringify(pool, null, 2) }] };
@@ -49341,7 +49377,7 @@ function registerCustodyTools(server, client) {
   });
   server.registerTool("get_custody", {
     description: "Get detailed custody information for a specific custody account. Includes utilization, fees, and limits.",
-    inputSchema: { pubkey: exports_external.string().describe("Solana pubkey of the custody account") }
+    inputSchema: { pubkey: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Solana pubkey of the custody account") }
   }, async ({ pubkey }) => {
     const custody = await client.getCustody(pubkey);
     return { content: [{ type: "text", text: JSON.stringify(custody, null, 2) }] };
@@ -49366,18 +49402,18 @@ function registerPriceTools(server, client) {
   });
   server.registerTool("get_price", {
     description: 'Get the current oracle price for a specific asset symbol (e.g., "SOL", "BTC", "ETH"). Case-insensitive. Returns price in USD with timestamp.',
-    inputSchema: { symbol: exports_external.string().describe('Asset symbol, e.g. "SOL", "BTC", "ETH"') }
+    inputSchema: { symbol: exports_external.string().max(16).describe('Asset symbol, e.g. "SOL", "BTC", "ETH"') }
   }, async ({ symbol: symbol2 }) => {
     const data = await client.getPrice(symbol2);
     const usd = formatPrice(data.price, data.exponent);
-    return {
-      content: [{
-        type: "text",
-        text: `${symbol2.toUpperCase()}: $${usd}
-Raw: ${data.price} (exp: ${data.exponent})
-Timestamp: ${data.timestamp}`
-      }]
-    };
+    const lines = [
+      `${symbol2.toUpperCase()}: $${usd}`,
+      `Raw: ${data.price} (exp: ${data.exponent})`
+    ];
+    if (data.timestamp)
+      lines.push(`Timestamp: ${data.timestamp}`);
+    return { content: [{ type: "text", text: lines.join(`
+`) }] };
   });
 }
 
@@ -49400,7 +49436,7 @@ function registerPositionTools(server, client) {
   server.registerTool("get_positions", {
     description: "List perpetual positions, optionally filtered by wallet owner. Without an owner filter, returns ALL open positions (may be large). With owner, returns enriched positions with computed PnL, leverage, and liquidation price.",
     inputSchema: {
-      owner: exports_external.string().optional().describe("Wallet pubkey to filter by. When provided, returns enriched positions with PnL, leverage, and liquidation price.")
+      owner: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).optional().describe("Wallet pubkey to filter by. When provided, returns enriched positions with PnL, leverage, and liquidation price.")
     }
   }, async ({ owner }) => {
     if (owner) {
@@ -49420,7 +49456,7 @@ ${text}` }] };
   });
   server.registerTool("get_position", {
     description: "Get a single position by its on-chain account pubkey. Returns raw position data. For enriched data (PnL, leverage, liq price), use get_positions with the owner filter instead.",
-    inputSchema: { pubkey: exports_external.string().describe("Position account pubkey") }
+    inputSchema: { pubkey: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Position account pubkey") }
   }, async ({ pubkey }) => {
     const position = await client.getPosition(pubkey);
     return { content: [{ type: "text", text: JSON.stringify(position, null, 2) }] };
@@ -49446,7 +49482,7 @@ function registerOrderTools(server, client) {
   server.registerTool("get_orders", {
     description: "List open orders (limit orders, take-profit, stop-loss), optionally filtered by wallet owner. When owner is provided, returns enriched order data with computed trigger prices and sizes.",
     inputSchema: {
-      owner: exports_external.string().optional().describe("Wallet pubkey to filter by. When provided, returns enriched orders.")
+      owner: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).optional().describe("Wallet pubkey to filter by. When provided, returns enriched orders.")
     }
   }, async ({ owner }) => {
     if (owner) {
@@ -49466,7 +49502,7 @@ ${text}` }] };
   });
   server.registerTool("get_order", {
     description: "Get a single order account by its on-chain pubkey.",
-    inputSchema: { pubkey: exports_external.string().describe("Order account pubkey") }
+    inputSchema: { pubkey: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Order account pubkey") }
   }, async ({ pubkey }) => {
     const order = await client.getOrder(pubkey);
     return { content: [{ type: "text", text: JSON.stringify(order, null, 2) }] };
@@ -49478,15 +49514,33 @@ function registerPoolDataTools(server, client) {
   server.registerTool("get_pool_data", {
     description: "Get computed pool metrics including AUM (assets under management), LP token stats, custody ratios, and utilization. Data is cached and refreshed every 15 seconds. Provide a pool_pubkey for a specific pool, or omit for all pools.",
     inputSchema: {
-      pool_pubkey: exports_external.string().optional().describe("Specific pool pubkey. If omitted, returns all pool snapshots.")
+      pool_pubkey: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).optional().describe("Specific pool pubkey. If omitted, returns all pool snapshots.")
     }
   }, async ({ pool_pubkey }) => {
     if (pool_pubkey) {
       const snapshot = await client.getPoolSnapshot(pool_pubkey);
       return { content: [{ type: "text", text: JSON.stringify(snapshot, null, 2) }] };
     }
-    const data = await client.getPoolData();
-    return { content: [{ type: "text", text: JSON.stringify(data, null, 2) }] };
+    const raw = await client.getPoolData();
+    const pools = raw.pools ?? raw;
+    const lines = [
+      `${pools.length} pools:
+`,
+      "Pool           | AUM          | LP Price | Stable% | Custodies",
+      "---------------|--------------|----------|---------|----------"
+    ];
+    for (const p of pools) {
+      const name = (p.poolName ?? "Unknown").padEnd(15);
+      const aum = `$${p.lpStats?.totalPoolValueUsd ?? "?"}`.padEnd(14);
+      const lp = `$${p.lpStats?.lpPrice ?? "?"}`.padEnd(9);
+      const stable = `${p.lpStats?.stableCoinPercentage ?? "?"}%`.padEnd(8);
+      const custodies = (p.custodyStats ?? []).map((c) => c.symbol).join(", ");
+      lines.push(`${name}| ${aum}| ${lp}| ${stable}| ${custodies}`);
+    }
+    lines.push(`
+Use get_pool_data with pool_pubkey for full custody stats.`);
+    return { content: [{ type: "text", text: lines.join(`
+`) }] };
   });
 }
 
@@ -49536,17 +49590,17 @@ function registerOpenPositionTool(server, client) {
   server.registerTool("open_position", {
     description: "Build a transaction to open a new perpetual position on Flash Trade. Returns a preview (entry price, fees, leverage, liquidation price) AND an unsigned transaction. The transaction must be signed by the user's wallet and submitted to Solana separately. IMPORTANT: Always present the preview to the user before they sign. This tool does NOT execute the trade. Supports both MARKET and LIMIT orders, with optional take-profit and stop-loss. COLLATERAL WARNING: Limit orders, take-profit, and stop-loss require >$10 collateral AFTER entry fees. A $10 position will have fees deducted, dropping collateral below $10 and preventing TP/SL/limit orders. Use at least $11-12 input_amount when planning to set TP/SL.",
     inputSchema: {
-      input_token_symbol: exports_external.string().describe('Token to pay with: "USDC", "SOL", etc.'),
-      output_token_symbol: exports_external.string().describe('Market to trade: "SOL", "BTC", "ETH", etc.'),
-      input_amount: exports_external.string().describe('Amount of input token, e.g. "100.0"'),
-      leverage: exports_external.string().describe('Leverage multiplier, e.g. "5.0"'),
+      input_token_symbol: exports_external.string().max(16).describe('Token to pay with: "USDC", "SOL", etc.'),
+      output_token_symbol: exports_external.string().max(16).describe('Market to trade: "SOL", "BTC", "ETH", etc.'),
+      input_amount: exports_external.string().max(32).describe('Amount of input token, e.g. "100.0"'),
+      leverage: exports_external.string().max(8).describe('Leverage multiplier, e.g. "5.0"'),
       trade_type: exports_external.enum(["LONG", "SHORT"]).describe("Trade direction"),
-      owner: exports_external.string().describe("Wallet pubkey (required to build the transaction)"),
+      owner: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Wallet pubkey (required to build the transaction)"),
       order_type: exports_external.enum(["MARKET", "LIMIT"]).optional().describe("Default: MARKET"),
-      limit_price: exports_external.string().optional().describe("Required for LIMIT orders, UI format price"),
-      slippage_percentage: exports_external.string().optional().describe('Default: "0.5" (0.5%)'),
-      take_profit: exports_external.string().optional().describe("TP trigger price in UI format"),
-      stop_loss: exports_external.string().optional().describe("SL trigger price in UI format"),
+      limit_price: exports_external.string().max(32).optional().describe("Required for LIMIT orders, UI format price"),
+      slippage_percentage: exports_external.string().max(8).optional().describe('Default: "0.5" (0.5%)'),
+      take_profit: exports_external.string().max(32).optional().describe("TP trigger price in UI format"),
+      stop_loss: exports_external.string().max(32).optional().describe("SL trigger price in UI format"),
       degen_mode: exports_external.boolean().optional().describe("Enable degen mode (higher leverage limits)")
     }
   }, async (params) => {
@@ -49564,12 +49618,21 @@ function registerOpenPositionTool(server, client) {
       stopLoss: params.stop_loss,
       degenMode: params.degen_mode
     });
-    const text = formatOpenPreview({
+    let text = formatOpenPreview({
       outputTokenSymbol: params.output_token_symbol,
       tradeType: params.trade_type,
       inputTokenSymbol: params.input_token_symbol,
       inputAmountUi: params.input_amount
     }, res);
+    if (params.take_profit || params.stop_loss) {
+      const collateral = parseFloat(res.youPayUsdUi || "0");
+      const fee = parseFloat(res.entryFee || "0");
+      if (collateral - fee < 10) {
+        text += `
+
+WARNING: Collateral after fees is below $10. Take-profit and stop-loss orders require >$10 collateral and will FAIL on-chain. Use at least $11-12 input_amount.`;
+      }
+    }
     return { content: [{ type: "text", text }] };
   });
 }
@@ -49609,11 +49672,11 @@ function registerClosePositionTool(server, client) {
   server.registerTool("close_position", {
     description: "Build a transaction to close (fully or partially) an existing perpetual position. Returns a preview with PnL, fees, and receive amount, plus an unsigned transaction. For a full close, set input_usd to the position's full size. For a partial close, use a smaller amount. The transaction must be signed and submitted separately.",
     inputSchema: {
-      position_key: exports_external.string().describe("Position account pubkey to close"),
-      input_usd: exports_external.string().describe('USD amount to close, e.g. "500.00" for full or "250.00" for partial'),
-      withdraw_token_symbol: exports_external.string().describe('Token to receive: "USDC", "SOL", etc.'),
+      position_key: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Position account pubkey to close"),
+      input_usd: exports_external.string().max(32).describe('USD amount to close, e.g. "500.00" for full or "250.00" for partial'),
+      withdraw_token_symbol: exports_external.string().max(16).describe('Token to receive: "USDC", "SOL", etc.'),
       keep_leverage_same: exports_external.boolean().optional().describe("Keep leverage constant during partial close"),
-      slippage_percentage: exports_external.string().optional().describe('Default: "0.5" (0.5%)')
+      slippage_percentage: exports_external.string().max(8).optional().describe('Default: "0.5" (0.5%)')
     }
   }, async (params) => {
     const res = await client.closePosition({
@@ -49672,11 +49735,11 @@ function registerCollateralTools(server, client) {
   server.registerTool("add_collateral", {
     description: "Build a transaction to add collateral to an existing position. This reduces leverage and moves the liquidation price further from the current price (safer). Returns a preview and unsigned transaction.",
     inputSchema: {
-      position_key: exports_external.string().describe("Position account pubkey"),
-      deposit_amount: exports_external.string().describe("Amount to deposit in UI format"),
-      deposit_token_symbol: exports_external.string().describe('Token to deposit: "USDC", "SOL", etc.'),
-      owner: exports_external.string().describe("Wallet pubkey"),
-      slippage_percentage: exports_external.string().optional().describe('Default: "0.5" (0.5%)')
+      position_key: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Position account pubkey"),
+      deposit_amount: exports_external.string().max(32).describe("Amount to deposit in UI format"),
+      deposit_token_symbol: exports_external.string().max(16).describe('Token to deposit: "USDC", "SOL", etc.'),
+      owner: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Wallet pubkey"),
+      slippage_percentage: exports_external.string().max(8).optional().describe('Default: "0.5" (0.5%)')
     }
   }, async (params) => {
     const res = await client.addCollateral({
@@ -49691,11 +49754,11 @@ function registerCollateralTools(server, client) {
   server.registerTool("remove_collateral", {
     description: "Build a transaction to remove collateral from an existing position. This increases leverage and moves the liquidation price closer (riskier). WARNING: Removing too much collateral can lead to liquidation. Returns a preview and unsigned transaction.",
     inputSchema: {
-      position_key: exports_external.string().describe("Position account pubkey"),
-      withdraw_amount_usd: exports_external.string().describe("USD amount to withdraw"),
-      withdraw_token_symbol: exports_external.string().describe('Token to receive: "USDC", "SOL", etc.'),
-      owner: exports_external.string().describe("Wallet pubkey"),
-      slippage_percentage: exports_external.string().optional().describe('Default: "0.5" (0.5%)')
+      position_key: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Position account pubkey"),
+      withdraw_amount_usd: exports_external.string().max(32).describe("USD amount to withdraw"),
+      withdraw_token_symbol: exports_external.string().max(16).describe('Token to receive: "USDC", "SOL", etc.'),
+      owner: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Wallet pubkey"),
+      slippage_percentage: exports_external.string().max(8).optional().describe('Default: "0.5" (0.5%)')
     }
   }, async (params) => {
     const res = await client.removeCollateral({
@@ -49743,9 +49806,9 @@ function registerReversePositionTool(server, client) {
   server.registerTool("reverse_position", {
     description: "Build a transaction to reverse a position (close current + open opposite direction). For example, close a LONG and open a SHORT with the same collateral. Returns combined close+open preview and a single unsigned transaction.",
     inputSchema: {
-      position_key: exports_external.string().describe("Position account pubkey to reverse"),
-      owner: exports_external.string().describe("Wallet pubkey"),
-      slippage_percentage: exports_external.string().optional().describe('Default: "0.5" (0.5%)'),
+      position_key: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Position account pubkey to reverse"),
+      owner: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Wallet pubkey"),
+      slippage_percentage: exports_external.string().max(8).optional().describe('Default: "0.5" (0.5%)'),
       degen_mode: exports_external.boolean().optional().describe("Enable degen mode for the new position")
     }
   }, async (params) => {
@@ -49764,11 +49827,11 @@ function registerPreviewTools(server, client) {
   server.registerTool("preview_limit_order_fees", {
     description: "Preview the entry price, fees, liquidation price, and borrow rate for a limit order BEFORE placing it. Use this to evaluate a trade before committing. No transaction is built.",
     inputSchema: {
-      market_symbol: exports_external.string().describe('Market symbol, e.g. "SOL", "BTC", "ETH"'),
-      input_amount: exports_external.string().describe("Collateral amount in UI format"),
-      output_amount: exports_external.string().describe("Position size in target token"),
+      market_symbol: exports_external.string().max(16).describe('Market symbol, e.g. "SOL", "BTC", "ETH"'),
+      input_amount: exports_external.string().max(32).describe("Collateral amount in UI format"),
+      output_amount: exports_external.string().max(32).describe("Position size in target token"),
       side: exports_external.enum(["LONG", "SHORT"]).describe("Trade direction"),
-      limit_price: exports_external.string().optional().describe("Limit price; uses live price if omitted"),
+      limit_price: exports_external.string().max(32).optional().describe("Limit price; uses live price if omitted"),
       trading_fee_discount_percent: exports_external.number().optional().describe("Fee discount from FAF staking (0-100)")
     }
   }, async (params) => {
@@ -49796,8 +49859,8 @@ WARNING: ${res.err}`);
   server.registerTool("preview_exit_fee", {
     description: "Preview the exit fee and exit price for closing a specific amount of a position. Use this to estimate close costs before calling close_position. No transaction is built.",
     inputSchema: {
-      position_key: exports_external.string().describe("Position account pubkey"),
-      close_amount_usd: exports_external.string().describe("USD amount to close")
+      position_key: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Position account pubkey"),
+      close_amount_usd: exports_external.string().max(32).describe("USD amount to close")
     }
   }, async (params) => {
     const res = await client.previewExitFee({
@@ -49819,14 +49882,14 @@ WARNING: ${res.err}`);
     description: 'Calculate take-profit or stop-loss prices and projected PnL. Three modes: "forward" (trigger price → PnL), "reverse_pnl" (target PnL → trigger price), "reverse_roi" (target ROI% → trigger price). Works for existing positions (by pubkey) or hypothetical orders (provide market_symbol, entry_price, size, collateral, side).',
     inputSchema: {
       mode: exports_external.enum(["forward", "reverse_pnl", "reverse_roi"]).describe("Calculation mode"),
-      position_key: exports_external.string().optional().describe("Position pubkey (for existing positions)"),
-      market_symbol: exports_external.string().optional().describe("Market symbol (for hypothetical orders)"),
-      entry_price: exports_external.string().optional().describe("Entry price (for hypothetical orders)"),
-      size_usd: exports_external.string().optional().describe("Position size USD (for hypothetical orders)"),
-      collateral_usd: exports_external.string().optional().describe("Collateral USD (for hypothetical orders)"),
+      position_key: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).optional().describe("Position pubkey (for existing positions)"),
+      market_symbol: exports_external.string().max(16).optional().describe("Market symbol (for hypothetical orders)"),
+      entry_price: exports_external.string().max(32).optional().describe("Entry price (for hypothetical orders)"),
+      size_usd: exports_external.string().max(32).optional().describe("Position size USD (for hypothetical orders)"),
+      collateral_usd: exports_external.string().max(32).optional().describe("Collateral USD (for hypothetical orders)"),
       side: exports_external.enum(["LONG", "SHORT"]).optional().describe("Side (for hypothetical orders)"),
-      trigger_price: exports_external.string().optional().describe('Trigger price (required for "forward" mode)'),
-      target_pnl_usd: exports_external.string().optional().describe('Target PnL USD (required for "reverse_pnl" mode)'),
+      trigger_price: exports_external.string().max(32).optional().describe('Trigger price (required for "forward" mode)'),
+      target_pnl_usd: exports_external.string().max(32).optional().describe('Target PnL USD (required for "reverse_pnl" mode)'),
       target_roi_percent: exports_external.number().optional().describe('Target ROI% (required for "reverse_roi" mode)')
     }
   }, async (params) => {
@@ -49858,8 +49921,8 @@ WARNING: ${res.err}`);
   server.registerTool("preview_margin", {
     description: "Preview the effect of adding or removing margin (collateral) on a position. Shows new leverage, new liquidation price, and max adjustable amount. Use this before calling add_collateral or remove_collateral. No transaction is built.",
     inputSchema: {
-      position_key: exports_external.string().describe("Position account pubkey"),
-      margin_delta_usd: exports_external.string().describe("Amount in USD to add or remove"),
+      position_key: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Position account pubkey"),
+      margin_delta_usd: exports_external.string().max(32).describe("Amount in USD to add or remove"),
       action: exports_external.enum(["ADD", "REMOVE"]).describe("ADD to reduce leverage, REMOVE to increase leverage")
     }
   }, async (params) => {
@@ -56796,15 +56859,19 @@ var $VersionedTransaction = VersionedTransaction;
 
 // src/tools/sign-and-send.ts
 import fs from "node:fs";
+
+// src/sanitize.ts
 function sanitizeError(e) {
   const msg = e instanceof Error ? e.message : String(e);
-  return msg.replace(/\[[\d,\s]{20,}\]/g, "[REDACTED]").replace(/[0-9a-fA-F]{40,}/g, "[REDACTED]").replace(/[123456789ABCDEFGHJKLMNPQRSTUVWXYZabcdefghijkmnopqrstuvwxyz]{40,}/g, "[REDACTED]");
+  return msg.replace(/\[[\d,\s]{20,}\]/g, "[REDACTED]").replace(/[0-9a-fA-F]{40,}/g, "[REDACTED]").replace(/[123456789ABCDEFGHJKLMNPQRSTUVWXYZabcdefghijkmnopqrstuvwxyz]{40,}/g, "[REDACTED]").replace(/[A-Za-z0-9+/]{40,}={0,2}/g, "[REDACTED]");
 }
+
+// src/tools/sign-and-send.ts
 function registerSignAndSendTool(server) {
   server.registerTool("sign_and_send", {
     description: "Sign and submit a base64-encoded unsigned Solana transaction using the locally configured keypair. " + "Call this AFTER a transaction tool (open_position, close_position, add_collateral, remove_collateral, reverse_position) " + "returns a transactionBase64 string AND the user has reviewed and approved the preview. " + "The keypair is read from KEYPAIR_PATH (default: ~/.config/solana/id.json). " + "Returns the confirmed transaction signature and a Solscan link. " + "IMPORTANT: Always show the transaction preview to the user and get their approval BEFORE calling this tool. " + "This tool signs with the local keypair and submits to Solana mainnet — the action is IRREVERSIBLE. " + "NOTE: This tool never exposes private key material in its output.",
     inputSchema: {
-      transaction_base64: exports_external.string().describe("The base64-encoded unsigned transaction returned by a transaction tool")
+      transaction_base64: exports_external.string().max(1e4).describe("The base64-encoded unsigned transaction returned by a transaction tool")
     }
   }, async (params) => {
     const rpcUrl = process.env.SOLANA_RPC_URL ?? "https://api.mainnet-beta.solana.com";
@@ -56880,6 +56947,152 @@ Signature: ${signature2}` }],
   });
 }
 
+// src/tools/trigger-orders.ts
+function registerTriggerOrderTools(server, client) {
+  server.registerTool("place_trigger_order", {
+    description: "Place a take-profit (TP) or stop-loss (SL) trigger order on an existing position. The trigger order will automatically close part or all of the position when the price hits the trigger level. Requires an open position for the given market/side. Returns an unsigned transaction. Up to 5 trigger orders per position.",
+    inputSchema: {
+      market_symbol: exports_external.string().max(16).describe('Market symbol, e.g. "SOL", "BTC", "ETH"'),
+      collateral_symbol: exports_external.string().max(16).describe('Collateral token, e.g. "USDC", "JITOSOL"'),
+      side: exports_external.enum(["LONG", "SHORT"]).describe("Position side"),
+      trigger_price: exports_external.string().max(32).describe('Trigger price in UI format, e.g. "160.00"'),
+      size_amount: exports_external.string().max(32).describe('Size in target token to close when triggered, e.g. "0.5"'),
+      is_stop_loss: exports_external.boolean().describe("true = stop-loss, false = take-profit"),
+      owner: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Wallet pubkey (must own the position)")
+    }
+  }, async (params) => {
+    const res = await client.placeTriggerOrder({
+      marketSymbol: params.market_symbol,
+      collateralSymbol: params.collateral_symbol,
+      side: params.side,
+      triggerPriceUi: params.trigger_price,
+      sizeAmountUi: params.size_amount,
+      isStopLoss: params.is_stop_loss,
+      owner: params.owner
+    });
+    const lines = [
+      `=== Place ${params.is_stop_loss ? "Stop-Loss" : "Take-Profit"} Order ===`,
+      `Market: ${params.market_symbol} ${params.side}`,
+      `Trigger: $${params.trigger_price}`,
+      `Size: ${params.size_amount} ${params.market_symbol}`
+    ];
+    if (res.err)
+      lines.push(`
+WARNING: ${res.err}`);
+    if (res.transactionBase64) {
+      lines.push(`
+Transaction (base64, unsigned — sign with wallet):`);
+      lines.push(res.transactionBase64);
+    }
+    return { content: [{ type: "text", text: lines.join(`
+`) }] };
+  });
+  server.registerTool("edit_trigger_order", {
+    description: "Edit an existing take-profit or stop-loss trigger order. Change the trigger price, size, or order type without canceling and re-placing. Requires the order_id (0-7) from get_orders. Returns an unsigned transaction.",
+    inputSchema: {
+      market_symbol: exports_external.string().max(16).describe('Market symbol, e.g. "SOL", "BTC", "ETH"'),
+      collateral_symbol: exports_external.string().max(16).describe('Collateral token, e.g. "USDC", "JITOSOL"'),
+      side: exports_external.enum(["LONG", "SHORT"]).describe("Position side"),
+      order_id: exports_external.number().describe("Index of the trigger order to edit (0-7)"),
+      trigger_price: exports_external.string().max(32).describe("New trigger price in UI format"),
+      size_amount: exports_external.string().max(32).describe("New size in target token"),
+      is_stop_loss: exports_external.boolean().describe("true = stop-loss, false = take-profit"),
+      owner: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Wallet pubkey (must be original order owner)")
+    }
+  }, async (params) => {
+    const res = await client.editTriggerOrder({
+      marketSymbol: params.market_symbol,
+      collateralSymbol: params.collateral_symbol,
+      side: params.side,
+      orderId: params.order_id,
+      triggerPriceUi: params.trigger_price,
+      sizeAmountUi: params.size_amount,
+      isStopLoss: params.is_stop_loss,
+      owner: params.owner
+    });
+    const lines = [
+      `=== Edit ${params.is_stop_loss ? "Stop-Loss" : "Take-Profit"} Order #${params.order_id} ===`,
+      `Market: ${params.market_symbol} ${params.side}`,
+      `New Trigger: $${params.trigger_price}`,
+      `New Size: ${params.size_amount} ${params.market_symbol}`
+    ];
+    if (res.err)
+      lines.push(`
+WARNING: ${res.err}`);
+    if (res.transactionBase64) {
+      lines.push(`
+Transaction (base64, unsigned — sign with wallet):`);
+      lines.push(res.transactionBase64);
+    }
+    return { content: [{ type: "text", text: lines.join(`
+`) }] };
+  });
+  server.registerTool("cancel_trigger_order", {
+    description: "Cancel a single take-profit or stop-loss trigger order. Requires the order_id (0-7) from get_orders. Returns an unsigned transaction.",
+    inputSchema: {
+      market_symbol: exports_external.string().max(16).describe('Market symbol, e.g. "SOL", "BTC", "ETH"'),
+      collateral_symbol: exports_external.string().max(16).describe('Collateral token, e.g. "USDC", "JITOSOL"'),
+      side: exports_external.enum(["LONG", "SHORT"]).describe("Position side"),
+      order_id: exports_external.number().describe("Index of the trigger order to cancel (0-7)"),
+      is_stop_loss: exports_external.boolean().describe("true = stop-loss, false = take-profit"),
+      owner: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Wallet pubkey (must own the order)")
+    }
+  }, async (params) => {
+    const res = await client.cancelTriggerOrder({
+      marketSymbol: params.market_symbol,
+      collateralSymbol: params.collateral_symbol,
+      side: params.side,
+      orderId: params.order_id,
+      isStopLoss: params.is_stop_loss,
+      owner: params.owner
+    });
+    const lines = [
+      `=== Cancel ${params.is_stop_loss ? "Stop-Loss" : "Take-Profit"} Order #${params.order_id} ===`,
+      `Market: ${params.market_symbol} ${params.side}`
+    ];
+    if (res.err)
+      lines.push(`
+WARNING: ${res.err}`);
+    if (res.transactionBase64) {
+      lines.push(`
+Transaction (base64, unsigned — sign with wallet):`);
+      lines.push(res.transactionBase64);
+    }
+    return { content: [{ type: "text", text: lines.join(`
+`) }] };
+  });
+  server.registerTool("cancel_all_trigger_orders", {
+    description: "Cancel ALL take-profit and stop-loss trigger orders for a market+side. Removes all TP and SL orders in one transaction. Returns an unsigned transaction.",
+    inputSchema: {
+      market_symbol: exports_external.string().max(16).describe('Market symbol, e.g. "SOL", "BTC", "ETH"'),
+      collateral_symbol: exports_external.string().max(16).describe('Collateral token, e.g. "USDC", "JITOSOL"'),
+      side: exports_external.enum(["LONG", "SHORT"]).describe("Position side"),
+      owner: exports_external.string().regex(/^[1-9A-HJ-NP-Za-km-z]{32,44}$/).describe("Wallet pubkey (must own the orders)")
+    }
+  }, async (params) => {
+    const res = await client.cancelAllTriggerOrders({
+      marketSymbol: params.market_symbol,
+      collateralSymbol: params.collateral_symbol,
+      side: params.side,
+      owner: params.owner
+    });
+    const lines = [
+      `=== Cancel All Trigger Orders ===`,
+      `Market: ${params.market_symbol} ${params.side}`
+    ];
+    if (res.err)
+      lines.push(`
+WARNING: ${res.err}`);
+    if (res.transactionBase64) {
+      lines.push(`
+Transaction (base64, unsigned — sign with wallet):`);
+      lines.push(res.transactionBase64);
+    }
+    return { content: [{ type: "text", text: lines.join(`
+`) }] };
+  });
+}
+
 // src/tools/index.ts
 function registerReadTools(server, client) {
   registerHealthTools(server, client);
@@ -56896,6 +57109,7 @@ function registerTransactionTools(server, client) {
   registerClosePositionTool(server, client);
   registerCollateralTools(server, client);
   registerReversePositionTool(server, client);
+  registerTriggerOrderTools(server, client);
   registerSignAndSendTool(server);
 }
 
@@ -56960,11 +57174,11 @@ function registerResources(server, client) {
 
 // src/index.ts
 process.on("uncaughtException", (err) => {
-  console.error("[flash-trade-mcp] Uncaught exception:", err);
+  console.error("[flash-trade-mcp] Uncaught exception:", sanitizeError(err));
   process.exit(1);
 });
 process.on("unhandledRejection", (reason) => {
-  console.error("[flash-trade-mcp] Unhandled rejection:", reason);
+  console.error("[flash-trade-mcp] Unhandled rejection:", sanitizeError(reason));
   process.exit(1);
 });
 try {
@@ -56972,7 +57186,7 @@ try {
   const client = new FlashApiClient(config2);
   const server = new McpServer({
     name: "flash-trade",
-    version: "0.2.2"
+    version: "0.3.2"
   }, {
     capabilities: {
       tools: {},
@@ -56986,6 +57200,6 @@ try {
   const transport = new StdioServerTransport;
   await server.connect(transport);
 } catch (err) {
-  console.error("[flash-trade-mcp] Fatal startup error:", err);
+  console.error("[flash-trade-mcp] Fatal startup error:", sanitizeError(err));
   process.exit(1);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "flash-trade-mcp",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "description": "MCP server for Flash Trade — perpetual futures DEX on Solana. Provides AI agents with typed tools for trading, position management, and market data.",
   "module": "src/index.ts",
   "main": "dist/index.js",