@aihubspot/agent-trade-mcp 0.1.6 → 0.1.8

package/README.md

@@ -39,10 +39,10 @@ MCP clients that support the current protocol also receive the same envelope in
 
 Use these tools for requests that would otherwise return a broad market payload:
 
-- `market_search_symbols`: bounded symbol search and metadata.
+- `market_search_symbols`: bounded symbol search and metadata. Use this for every MCP symbol lookup; it returns matching rows in `data.value.items`.
 - `market_get_ticker_summary`: watchlist, gainers, losers, and quote-volume leaders.
 - `market_get_depth_summary`: best bid/ask, spread, and up to 20 levels per side.
 - `market_get_trades_summary`: price range, buy/sell statistics, and up to 50 recent trades.
-- `market_get_klines_summary`: period change, high/low, latest candle, and up to 100 candles.
+- `market_get_klines_summary`: period change, high/low, latest candle, and up to 100 candles. It defaults to `60min`; formal intervals are `1min`, `5min`, `15min`, `30min`, `60min`, `1day`, `1week`, and `1month`.
 
-The raw symbol, ticker, depth, trades, and kline tools remain available only when an exact full payload is explicitly needed. Raw trades and klines are capped at 100 rows by this client.
+The complete raw symbols endpoint is intentionally CLI-only because its payload is too large for Agent contexts. Raw ticker, depth, trades, and kline tools remain available only when an exact payload is explicitly needed. Raw trades are capped at 100 rows; raw klines support the upstream maximum of 300 rows.

package/dist/index.js

@@ -656,29 +656,82 @@ function optionalClientOrderId(input) {
 }
 
 // ../core/src/tools/asset-tools.ts
-var accountTypes = ["1", "2", "3", "4", "5"];
+var ACCOUNT_TYPES = ["1", "2", "3", "4", "5"];
+var ACCOUNT_TYPE_NAMES = {
+  "1": "Spot",
+  "2": "Isolated Margin",
+  "3": "Cross Margin",
+  "4": "C2C",
+  "5": "Derivatives"
+};
+var accountTypeSchema = {
+  type: "string",
+  oneOf: [
+    { const: "1", title: "Spot (1)" },
+    { const: "2", title: "Isolated Margin (2; symbol required)" },
+    { const: "3", title: "Cross Margin (3)" },
+    { const: "4", title: "C2C (4)" },
+    { const: "5", title: "Derivatives (5)" }
+  ],
+  description: "Account type: 1=Spot, 2=Isolated Margin (requires symbol), 3=Cross Margin, 4=C2C, 5=Derivatives. Do not use 2 for Derivatives."
+};
+var spotDerivativeAccounts = ["EXCHANGE", "FUTURE"];
 function accountType(value, name) {
   const result2 = requiredString(value, name);
-  if (!accountTypes.includes(result2)) throw new AiHubError("AI_HUB_INVALID_ARGUMENT", `${name} must be an account type from 1 to 5.`);
+  if (!ACCOUNT_TYPES.includes(result2)) throw new AiHubError("AI_HUB_INVALID_ARGUMENT", `${name} must be an account type: 1=Spot, 2=Isolated Margin, 3=Cross Margin, 4=C2C, or 5=Derivatives.`);
   return result2;
 }
+function accountTypeName(value) {
+  return ACCOUNT_TYPE_NAMES[value];
+}
+function isIsolatedMarginTransfer(fromAccountType, toAccountType) {
+  return fromAccountType === "2" || toAccountType === "2";
+}
 function requiredTransfer(input) {
   const value = strictObject(input, ["fromAccountType", "toAccountType", "symbol", "coinSymbol", "amount"]);
-  return { fromAccountType: accountType(value, "fromAccountType"), toAccountType: accountType(value, "toAccountType"), coinSymbol: requiredString(value, "coinSymbol"), amount: positiveDecimal(value, "amount"), ...optionalString(value, "symbol") ? { symbol: optionalString(value, "symbol") } : {} };
+  const fromAccountType = accountType(value, "fromAccountType");
+  const toAccountType = accountType(value, "toAccountType");
+  if (fromAccountType === toAccountType) throw new AiHubError("AI_HUB_INVALID_ARGUMENT", "fromAccountType and toAccountType must be different.");
+  const symbol = optionalString(value, "symbol");
+  if (isIsolatedMarginTransfer(fromAccountType, toAccountType) && !symbol) {
+    throw new AiHubError("AI_HUB_ISOLATED_MARGIN_SYMBOL_REQUIRED", "symbol is required when fromAccountType or toAccountType is 2 (Isolated Margin). Provide the isolated-margin trading pair, for example ETHUSDT.");
+  }
+  return { fromAccountType, toAccountType, coinSymbol: requiredString(value, "coinSymbol"), amount: positiveDecimal(value, "amount"), ...symbol ? { symbol } : {} };
 }
 function transferQuery(input) {
   const value = strictObject(input, ["fromAccountType", "toAccountType", "symbol", "coinSymbol", "page", "pageSize"]);
-  return { fromAccountType: accountType(value, "fromAccountType"), toAccountType: accountType(value, "toAccountType"), page: optionalPositiveInteger(value, "page", 1), pageSize: optionalPositiveInteger(value, "pageSize", 20, 100), ...optionalString(value, "symbol") ? { symbol: optionalString(value, "symbol") } : {}, ...optionalString(value, "coinSymbol") ? { coinSymbol: optionalString(value, "coinSymbol") } : {} };
+  const fromAccountType = accountType(value, "fromAccountType");
+  const toAccountType = accountType(value, "toAccountType");
+  const symbol = optionalString(value, "symbol");
+  if (isIsolatedMarginTransfer(fromAccountType, toAccountType) && !symbol) {
+    throw new AiHubError("AI_HUB_ISOLATED_MARGIN_SYMBOL_REQUIRED", "symbol is required when fromAccountType or toAccountType is 2 (Isolated Margin). Provide the isolated-margin trading pair, for example ETHUSDT.");
+  }
+  return { fromAccountType, toAccountType, page: optionalPositiveInteger(value, "page", 1), pageSize: optionalPositiveInteger(value, "pageSize", 20, 100), ...symbol ? { symbol } : {}, ...optionalString(value, "coinSymbol") ? { coinSymbol: optionalString(value, "coinSymbol") } : {} };
 }
 function transferAccounts(input) {
   const value = strictObject(input, ["coinSymbol", "amount", "fromAccount", "toAccount"]);
-  return { coinSymbol: requiredString(value, "coinSymbol"), amount: positiveDecimal(value, "amount"), fromAccount: requiredString(value, "fromAccount"), toAccount: requiredString(value, "toAccount") };
+  const fromAccount = requiredString(value, "fromAccount").toUpperCase();
+  const toAccount = requiredString(value, "toAccount").toUpperCase();
+  if (!spotDerivativeAccounts.includes(fromAccount) || !spotDerivativeAccounts.includes(toAccount) || fromAccount === toAccount) {
+    throw new AiHubError("AI_HUB_INVALID_ARGUMENT", "account transfer supports only EXCHANGE -> FUTURE or FUTURE -> EXCHANGE. Use wallet transfer for isolated margin, cross margin, or C2C.");
+  }
+  return { coinSymbol: requiredString(value, "coinSymbol"), amount: positiveDecimal(value, "amount"), fromAccount, toAccount };
 }
 function transferAccountHistory(input) {
   const value = strictObject(input, ["transferId", "coinSymbol", "fromAccount", "toAccount", "startTime", "endTime", "page", "limit"]);
   const transferId = optionalString(value, "transferId");
   if (!transferId && (!optionalString(value, "fromAccount") || !optionalString(value, "toAccount"))) throw new AiHubError("AI_HUB_INVALID_ARGUMENT", "fromAccount and toAccount are required when transferId is omitted.");
-  return { ...transferId ? { transferId } : {}, ...optionalString(value, "coinSymbol") ? { coinSymbol: optionalString(value, "coinSymbol") } : {}, ...optionalString(value, "fromAccount") ? { fromAccount: optionalString(value, "fromAccount") } : {}, ...optionalString(value, "toAccount") ? { toAccount: optionalString(value, "toAccount") } : {}, ...optionalString(value, "startTime") ? { startTime: optionalString(value, "startTime") } : {}, ...optionalString(value, "endTime") ? { endTime: optionalString(value, "endTime") } : {}, page: optionalPositiveInteger(value, "page", 1), limit: optionalPositiveInteger(value, "limit", 20, 100) };
+  const fromAccount = optionalString(value, "fromAccount")?.toUpperCase();
+  const toAccount = optionalString(value, "toAccount")?.toUpperCase();
+  if (!transferId && (!spotDerivativeAccounts.includes(fromAccount) || !spotDerivativeAccounts.includes(toAccount) || fromAccount === toAccount)) {
+    throw new AiHubError("AI_HUB_INVALID_ARGUMENT", "account transfer history supports only EXCHANGE -> FUTURE or FUTURE -> EXCHANGE when transferId is omitted.");
+  }
+  return { ...transferId ? { transferId } : {}, ...optionalString(value, "coinSymbol") ? { coinSymbol: optionalString(value, "coinSymbol") } : {}, ...fromAccount ? { fromAccount } : {}, ...toAccount ? { toAccount } : {}, ...optionalString(value, "startTime") ? { startTime: optionalString(value, "startTime") } : {}, ...optionalString(value, "endTime") ? { endTime: optionalString(value, "endTime") } : {}, page: optionalPositiveInteger(value, "page", 1), limit: optionalPositiveInteger(value, "limit", 20, 100) };
+}
+function withAccountTypeMetadata(response, value) {
+  const metadata = { accountType: value, accountTypeName: accountTypeName(value) };
+  if (response && typeof response === "object" && !Array.isArray(response)) return { ...response, ...metadata };
+  return { ...metadata, response };
 }
 function pagedDates(input) {
   const value = strictObject(input, ["startTime", "endTime", "page", "pageSize"]);
@@ -691,18 +744,21 @@ function coinOnly(input) {
 var assetTools = [
   {
     name: "account_transfer",
-    title: "Transfer Between Accounts",
-    description: "Transfer an asset between documented account types after confirmation.",
+    title: "Transfer Between Spot and Derivatives",
+    description: "Transfer only between Spot and Derivatives after confirmation. Use fromAccount=EXCHANGE and toAccount=FUTURE for Spot to Derivatives; use the reverse for Derivatives to Spot. Do not pass numeric account types to this Tool.",
     cliPath: ["account", "transfer"],
     module: "spot-account",
     access: "signed",
     operation: "write",
     riskLevel: "high",
-    inputSchema: { type: "object", properties: { coinSymbol: { type: "string" }, amount: { type: "string" }, fromAccount: { type: "string" }, toAccount: { type: "string" } }, required: ["coinSymbol", "amount", "fromAccount", "toAccount"], additionalProperties: false },
+    inputSchema: { type: "object", properties: { coinSymbol: { type: "string" }, amount: { type: "string" }, fromAccount: { type: "string", enum: spotDerivativeAccounts, description: "EXCHANGE means Spot; FUTURE means Derivatives." }, toAccount: { type: "string", enum: spotDerivativeAccounts, description: "EXCHANGE means Spot; FUTURE means Derivatives." } }, required: ["coinSymbol", "amount", "fromAccount", "toAccount"], additionalProperties: false },
     errorCodes: writeErrors,
     validate: transferAccounts,
     handler: (input, context) => context.api.signedPost("/sapi/v1/asset/transfer", input, signed(context)),
-    writeSummary: (input) => ({ action: "account_transfer", ...input })
+    writeSummary: (input) => {
+      const value = input;
+      return { action: "spot_derivatives_transfer", coinSymbol: value.coinSymbol, amount: value.amount, fromAccount: value.fromAccount, fromAccountName: value.fromAccount === "EXCHANGE" ? "Spot" : "Derivatives", toAccount: value.toAccount, toAccountName: value.toAccount === "EXCHANGE" ? "Spot" : "Derivatives" };
+    }
   },
   {
     name: "account_get_transfer_history",
@@ -721,28 +777,31 @@ var assetTools = [
   {
     name: "wallet_universal_transfer",
     title: "Universal Asset Transfer",
-    description: "Transfer between account types after confirmation.",
+    description: "Transfer between numeric account types after confirmation: 1=Spot, 2=Isolated Margin, 3=Cross Margin, 4=C2C, 5=Derivatives. Account type 2 is not Derivatives and requires symbol. For a straightforward Spot to Derivatives transfer, prefer account_transfer with EXCHANGE -> FUTURE.",
     cliPath: ["wallet", "transfer"],
     module: "spot-deposit-withdraw",
     access: "signed",
     operation: "write",
     riskLevel: "high",
-    inputSchema: { type: "object", properties: { fromAccountType: { type: "string" }, toAccountType: { type: "string" }, symbol: { type: "string" }, coinSymbol: { type: "string" }, amount: { type: "string" } }, required: ["fromAccountType", "toAccountType", "coinSymbol", "amount"], additionalProperties: false },
+    inputSchema: { type: "object", properties: { fromAccountType: accountTypeSchema, toAccountType: accountTypeSchema, symbol: { type: "string", description: "Required isolated-margin trading pair when either account type is 2, for example ETHUSDT. Not a Derivatives identifier." }, coinSymbol: { type: "string" }, amount: { type: "string" } }, required: ["fromAccountType", "toAccountType", "coinSymbol", "amount"], additionalProperties: false },
     errorCodes: writeErrors,
     validate: requiredTransfer,
     handler: (input, context) => context.api.signedPost("/sapi/v1/asset/universal_transfer", input, signed(context)),
-    writeSummary: (input) => ({ action: "universal_transfer", ...input })
+    writeSummary: (input) => {
+      const value = input;
+      return { action: "universal_transfer", fromAccountType: value.fromAccountType, fromAccountTypeName: accountTypeName(String(value.fromAccountType)), toAccountType: value.toAccountType, toAccountTypeName: accountTypeName(String(value.toAccountType)), coinSymbol: value.coinSymbol, amount: value.amount, symbol: value.symbol ?? null };
+    }
   },
   {
     name: "wallet_get_universal_transfer_history",
     title: "Get Universal Transfer History",
-    description: "Query universal asset transfer history.",
+    description: "Query universal transfer history using account types 1=Spot, 2=Isolated Margin, 3=Cross Margin, 4=C2C, and 5=Derivatives.",
     cliPath: ["wallet", "transfer-history"],
     module: "spot-deposit-withdraw",
     access: "signed",
     operation: "read",
     riskLevel: "low",
-    inputSchema: { type: "object", properties: { fromAccountType: { type: "string" }, toAccountType: { type: "string" }, symbol: { type: "string" }, coinSymbol: { type: "string" }, page: { type: "integer" }, pageSize: { type: "integer" } }, required: ["fromAccountType", "toAccountType"], additionalProperties: false },
+    inputSchema: { type: "object", properties: { fromAccountType: accountTypeSchema, toAccountType: accountTypeSchema, symbol: { type: "string", description: "Isolated-margin trading pair when account type 2 is involved." }, coinSymbol: { type: "string" }, page: { type: "integer" }, pageSize: { type: "integer" } }, required: ["fromAccountType", "toAccountType"], additionalProperties: false },
     errorCodes: signedReadErrors,
     validate: transferQuery,
     handler: (input, context) => context.api.signedPost("/sapi/v1/asset/universal_transfer_query", input, signed(context))
@@ -795,19 +854,22 @@ var assetTools = [
   {
     name: "wallet_get_transferable_assets",
     title: "Get Transferable Assets",
-    description: "Query assets transferable from one account type.",
+    description: "Query transferable assets for one account type. The response echoes the selected account type and its name: 1=Spot, 2=Isolated Margin, 3=Cross Margin, 4=C2C, 5=Derivatives.",
     cliPath: ["wallet", "transferable-assets"],
     module: "spot-deposit-withdraw",
     access: "signed",
     operation: "read",
     riskLevel: "low",
-    inputSchema: { type: "object", properties: { accountType: { type: "string" } }, required: ["accountType"], additionalProperties: false },
+    inputSchema: { type: "object", properties: { accountType: accountTypeSchema }, required: ["accountType"], additionalProperties: false },
     errorCodes: signedReadErrors,
     validate: (input) => {
       const value = strictObject(input, ["accountType"]);
       return { accountType: accountType(value, "accountType") };
     },
-    handler: (input, context) => context.api.signedPost("/sapi/v1/asset/account/by_type", input, signed(context))
+    handler: async (input, context) => {
+      const value = input;
+      return withAccountTypeMetadata(await context.api.signedPost("/sapi/v1/asset/account/by_type", value, signed(context)), value.accountType);
+    }
   },
   {
     name: "wallet_get_exchange_account",
@@ -1106,8 +1168,164 @@ function summarizeKlines(value, symbol, interval) {
   };
 }
 
+// ../core/src/tools/symbol-rules.ts
+var CACHE_TTL_MS = 60 * 60 * 1e3;
+var cache = /* @__PURE__ */ new Map();
+var pendingLoads = /* @__PURE__ */ new Map();
+function cacheKey(context) {
+  return `${context.profile.name}\0${context.profile.configVersion}\0${context.profile.openApiBaseUrl}`;
+}
+function normalizedSymbol(symbol) {
+  return symbol.replaceAll("/", "").trim().toUpperCase();
+}
+function stringValue(value) {
+  if (typeof value === "string" && value.trim()) return value.trim();
+  if (typeof value === "number" && Number.isFinite(value)) return String(value);
+  return void 0;
+}
+function nonNegativeInteger(value) {
+  const result2 = typeof value === "number" ? value : typeof value === "string" ? Number(value) : Number.NaN;
+  return Number.isInteger(result2) && result2 >= 0 ? result2 : void 0;
+}
+function rows(response) {
+  if (Array.isArray(response)) return response.filter((item) => Boolean(item) && typeof item === "object" && !Array.isArray(item));
+  if (!response || typeof response !== "object" || Array.isArray(response)) return [];
+  const root = response;
+  const candidates = [root.symbols, root.data, root.data?.symbols];
+  for (const candidate of candidates) {
+    if (Array.isArray(candidate)) return candidate.filter((item) => Boolean(item) && typeof item === "object" && !Array.isArray(item));
+  }
+  return [];
+}
+function parseRules(response) {
+  const result2 = /* @__PURE__ */ new Map();
+  for (const row of rows(response)) {
+    const symbol = stringValue(row.symbol);
+    if (!symbol) continue;
+    result2.set(normalizedSymbol(symbol), {
+      symbol,
+      quantityPrecision: nonNegativeInteger(row.quantityPrecision),
+      pricePrecision: nonNegativeInteger(row.pricePrecision),
+      limitVolumeMin: stringValue(row.limitVolumeMin),
+      limitAmountMin: stringValue(row.limitAmountMin),
+      limitPriceMin: stringValue(row.limitPriceMin)
+    });
+  }
+  return result2;
+}
+async function loadSnapshot(context) {
+  const key = cacheKey(context);
+  const current = cache.get(key);
+  if (current && current.expiresAt > Date.now()) return current;
+  const pending = pendingLoads.get(key);
+  if (pending) return pending;
+  const load = context.api.symbols().then((response) => {
+    const entry = { response, rules: parseRules(response), expiresAt: Date.now() + CACHE_TTL_MS };
+    cache.set(key, entry);
+    return entry;
+  }).finally(() => pendingLoads.delete(key));
+  pendingLoads.set(key, load);
+  return load;
+}
+async function getCachedSymbols(context) {
+  return (await loadSnapshot(context)).response;
+}
+async function getSymbolRule(context, symbol) {
+  const entry = await loadSnapshot(context);
+  const rule = entry.rules.get(normalizedSymbol(symbol));
+  if (!rule) throw new AiHubError("AI_HUB_SYMBOL_NOT_FOUND", `Symbol "${symbol}" was not returned by the configured tenant OpenAPI symbols endpoint.`);
+  return rule;
+}
+function decimalScale(value) {
+  return value.split(".")[1]?.length ?? 0;
+}
+function decimalParts(value) {
+  const [whole, fraction = ""] = value.split(".");
+  return { digits: BigInt(`${whole}${fraction}`.replace(/^0+(?=\d)/, "") || "0"), scale: fraction.length };
+}
+function compareDecimal(left, right) {
+  const a = decimalParts(left);
+  const b = decimalParts(right);
+  const scale = Math.max(a.scale, b.scale);
+  const leftValue = a.digits * 10n ** BigInt(scale - a.scale);
+  const rightValue = b.digits * 10n ** BigInt(scale - b.scale);
+  return leftValue === rightValue ? 0 : leftValue > rightValue ? 1 : -1;
+}
+function multipliedDecimal(left, right) {
+  const a = decimalParts(left);
+  const b = decimalParts(right);
+  const scale = a.scale + b.scale;
+  const digits = (a.digits * b.digits).toString();
+  if (scale === 0) return digits;
+  const padded = digits.padStart(scale + 1, "0");
+  return `${padded.slice(0, -scale)}.${padded.slice(-scale)}`;
+}
+function assertPrecision(name, value, precision, symbol) {
+  if (precision !== void 0 && decimalScale(value) > precision) {
+    throw new AiHubError("AI_HUB_SYMBOL_PRECISION_INVALID", `${name} for ${symbol} allows at most ${precision} decimal places; received ${value}.`);
+  }
+}
+function assertAtLeast(name, value, minimum, symbol) {
+  if (minimum && compareDecimal(value, minimum) < 0) {
+    throw new AiHubError("AI_HUB_SYMBOL_MINIMUM_NOT_MET", `${name} for ${symbol} must be at least ${minimum}; received ${value}.`);
+  }
+}
+async function preflightSymbolOrder(context, order) {
+  const rule = await getSymbolRule(context, order.symbol);
+  if (order.type === "MARKET" && order.side === "BUY" && order.quoteAmount) {
+    assertPrecision("quoteAmount", order.quoteAmount, rule.pricePrecision, rule.symbol);
+  }
+  if (order.baseQuantity) assertPrecision("baseQuantity", order.baseQuantity, rule.quantityPrecision, rule.symbol);
+  if (order.price) assertPrecision("price", order.price, rule.pricePrecision, rule.symbol);
+  if (order.type === "LIMIT" && order.baseQuantity && order.price) {
+    assertAtLeast("baseQuantity", order.baseQuantity, rule.limitVolumeMin, rule.symbol);
+    assertAtLeast("price", order.price, rule.limitPriceMin, rule.symbol);
+    if (rule.limitAmountMin && compareDecimal(rule.limitAmountMin, "0") > 0) {
+      assertAtLeast("limit order amount", multipliedDecimal(order.baseQuantity, order.price), rule.limitAmountMin, rule.symbol);
+    }
+  }
+  return rule;
+}
+
 // ../core/src/tools/market-tools.ts
 var readErrors = ["AI_HUB_INVALID_ARGUMENT", "AI_HUB_OPENAPI_NETWORK_ERROR", "AI_HUB_OPENAPI_HTTP_ERROR", "AI_HUB_OPENAPI_INVALID_RESPONSE", "AI_HUB_OPENAPI_BUSINESS_ERROR"];
+var KLINE_INTERVALS = ["1min", "5min", "15min", "30min", "60min", "1day", "1week", "1month"];
+var KLINE_INTERVAL_ALIASES = {
+  "1m": "1min",
+  "5m": "5min",
+  "15m": "15min",
+  "30m": "30min",
+  "60m": "60min",
+  "1h": "60min",
+  "1d": "1day",
+  "1w": "1week",
+  "1mo": "1month",
+  "1M": "1month"
+};
+var klineIntervalDescription = "Kline interval. Supported values: 1min, 5min, 15min, 30min, 60min, 1day, 1week, 1month. Use 60min, not 1h.";
+var klineTimezoneDescription = "Optional timezone such as UTC+08 or UTC-09. Defaults to UTC+08. The upstream API applies timezone-specific data only to periods above 60min.";
+function normalizeKlineInterval(value) {
+  const trimmed = value.trim();
+  const normalized = KLINE_INTERVAL_ALIASES[trimmed] ?? trimmed.toLowerCase();
+  if (KLINE_INTERVALS.includes(normalized)) return normalized;
+  throw new AiHubError("AI_HUB_INVALID_ARGUMENT", `interval must be one of: ${KLINE_INTERVALS.join(", ")}. Use 60min, not 1h.`);
+}
+function validateKlineInput(input, options) {
+  const value = strictObject(input, ["symbol", "interval", "startTime", "endTime", "timezone", "limit"]);
+  const startTime = value.startTime === void 0 ? void 0 : optionalInteger(value, "startTime", 0, 0, Number.MAX_SAFE_INTEGER);
+  const endTime = value.endTime === void 0 ? void 0 : optionalInteger(value, "endTime", 0, 0, Number.MAX_SAFE_INTEGER);
+  if (startTime !== void 0 && endTime !== void 0 && startTime > endTime) {
+    throw new AiHubError("AI_HUB_INVALID_ARGUMENT", "startTime must be less than or equal to endTime.");
+  }
+  return {
+    symbol: requiredString(value, "symbol"),
+    interval: value.interval === void 0 && options.summary ? "60min" : normalizeKlineInterval(requiredString(value, "interval")),
+    startTime,
+    endTime,
+    timezone: optionalString(value, "timezone"),
+    limit: optionalInteger(value, "limit", options.summary ? 20 : 50, 1, options.summary ? 100 : 300)
+  };
+}
 var marketTools = [
   {
     name: "market_ping",
@@ -1149,15 +1367,16 @@ var marketTools = [
     access: "public",
     operation: "read",
     riskLevel: "low",
+    mcpVisible: false,
     inputSchema: { type: "object", additionalProperties: false },
     errorCodes: readErrors,
     validate: (input) => strictObject(input, []),
-    handler: (_input, context) => context.api.symbols()
+    handler: (_input, context) => getCachedSymbols(context)
   },
   {
     name: "market_search_symbols",
     title: "Search Spot Symbols",
-    description: "Search the configured tenant's spot symbols and return a bounded metadata result. Use this instead of market_get_symbols unless complete raw metadata is explicitly required.",
+    description: "Search the configured tenant's spot symbols and return a bounded metadata result. In MCP, matching rows are at data.value.items. Use this instead of market_get_symbols unless complete raw metadata is explicitly required.",
     cliPath: ["market", "symbols-search"],
     module: "spot-common",
     access: "public",
@@ -1171,7 +1390,7 @@ var marketTools = [
     },
     handler: async (input, context) => {
       const value = input;
-      return summarizeSymbols(await context.api.symbols(), value);
+      return summarizeSymbols(await getCachedSymbols(context), value);
     }
   },
   {
@@ -1297,38 +1516,32 @@ var marketTools = [
   {
     name: "market_get_klines",
     title: "Get Spot Klines",
-    description: "Get spot candlestick data for one symbol and interval.",
+    description: "Get raw spot candlestick data for one symbol and a supported interval. Use market_get_klines_summary for Agent analysis.",
     cliPath: ["market", "klines"],
     module: "spot-common",
     access: "public",
     operation: "read",
     riskLevel: "low",
-    inputSchema: { type: "object", properties: { symbol: { type: "string" }, interval: { type: "string" }, startTime: { type: "integer" }, endTime: { type: "integer" }, timezone: { type: "string" }, limit: { type: "integer", minimum: 1, maximum: 100 } }, required: ["symbol", "interval"], additionalProperties: false },
+    inputSchema: { type: "object", properties: { symbol: { type: "string", description: "Spot symbol, for example ETHUSDT or ETH/USDT." }, interval: { type: "string", enum: KLINE_INTERVALS, description: klineIntervalDescription }, startTime: { type: "integer", description: "Optional inclusive Unix timestamp in milliseconds." }, endTime: { type: "integer", description: "Optional inclusive Unix timestamp in milliseconds." }, timezone: { type: "string", description: klineTimezoneDescription }, limit: { type: "integer", minimum: 1, maximum: 300, description: "Number of newest-first candles. The upstream maximum is 300." } }, required: ["symbol", "interval"], additionalProperties: false },
     errorCodes: readErrors,
     validate: (input) => {
-      const value = strictObject(input, ["symbol", "interval", "startTime", "endTime", "timezone", "limit"]);
-      const startTime = value.startTime === void 0 ? void 0 : optionalInteger(value, "startTime", 0, 0, Number.MAX_SAFE_INTEGER);
-      const endTime = value.endTime === void 0 ? void 0 : optionalInteger(value, "endTime", 0, 0, Number.MAX_SAFE_INTEGER);
-      return { symbol: requiredString(value, "symbol"), interval: requiredString(value, "interval"), startTime, endTime, timezone: optionalString(value, "timezone"), limit: optionalInteger(value, "limit", 50, 1, 100) };
+      return validateKlineInput(input, { summary: false });
     },
     handler: (input, context) => context.api.klines(input)
   },
   {
     name: "market_get_klines_summary",
     title: "Get Spot Klines Summary",
-    description: "Get a bounded candle analysis with period change, high/low, latest candle, and a caller-limited candle sample. Prefer this to market_get_klines for Agent analysis.",
+    description: "Get a bounded candle analysis with period change, high/low, latest candle, and a caller-limited candle sample. Defaults to 60min. Prefer this to market_get_klines for Agent analysis.",
     cliPath: ["market", "klines-summary"],
     module: "spot-common",
     access: "public",
     operation: "read",
     riskLevel: "low",
-    inputSchema: { type: "object", properties: { symbol: { type: "string" }, interval: { type: "string" }, startTime: { type: "integer" }, endTime: { type: "integer" }, timezone: { type: "string" }, limit: { type: "integer", minimum: 1, maximum: 100 } }, required: ["symbol", "interval"], additionalProperties: false },
+    inputSchema: { type: "object", properties: { symbol: { type: "string", description: "Spot symbol, for example ETHUSDT or ETH/USDT." }, interval: { type: "string", enum: KLINE_INTERVALS, description: `${klineIntervalDescription} Defaults to 60min when omitted.` }, startTime: { type: "integer", description: "Optional inclusive Unix timestamp in milliseconds." }, endTime: { type: "integer", description: "Optional inclusive Unix timestamp in milliseconds." }, timezone: { type: "string", description: klineTimezoneDescription }, limit: { type: "integer", minimum: 1, maximum: 100, description: "Number of candles included in the bounded analysis." } }, required: ["symbol"], additionalProperties: false },
     errorCodes: readErrors,
     validate: (input) => {
-      const value = strictObject(input, ["symbol", "interval", "startTime", "endTime", "timezone", "limit"]);
-      const startTime = value.startTime === void 0 ? void 0 : optionalInteger(value, "startTime", 0, 0, Number.MAX_SAFE_INTEGER);
-      const endTime = value.endTime === void 0 ? void 0 : optionalInteger(value, "endTime", 0, 0, Number.MAX_SAFE_INTEGER);
-      return { symbol: requiredString(value, "symbol"), interval: requiredString(value, "interval"), startTime, endTime, timezone: optionalString(value, "timezone"), limit: optionalInteger(value, "limit", 20, 1, 100) };
+      return validateKlineInput(input, { summary: true });
     },
     handler: async (input, context) => {
       const value = input;
@@ -1337,122 +1550,6 @@ var marketTools = [
   }
 ];
 
-// ../core/src/tools/symbol-rules.ts
-var CACHE_TTL_MS = 5 * 60 * 1e3;
-var cache = /* @__PURE__ */ new Map();
-var pendingLoads = /* @__PURE__ */ new Map();
-function cacheKey(context) {
-  return `${context.profile.name}\0${context.profile.configVersion}\0${context.profile.openApiBaseUrl}`;
-}
-function normalizedSymbol(symbol) {
-  return symbol.replaceAll("/", "").trim().toUpperCase();
-}
-function stringValue(value) {
-  if (typeof value === "string" && value.trim()) return value.trim();
-  if (typeof value === "number" && Number.isFinite(value)) return String(value);
-  return void 0;
-}
-function nonNegativeInteger(value) {
-  const result2 = typeof value === "number" ? value : typeof value === "string" ? Number(value) : Number.NaN;
-  return Number.isInteger(result2) && result2 >= 0 ? result2 : void 0;
-}
-function rows(response) {
-  if (Array.isArray(response)) return response.filter((item) => Boolean(item) && typeof item === "object" && !Array.isArray(item));
-  if (!response || typeof response !== "object" || Array.isArray(response)) return [];
-  const root = response;
-  const candidates = [root.symbols, root.data, root.data?.symbols];
-  for (const candidate of candidates) {
-    if (Array.isArray(candidate)) return candidate.filter((item) => Boolean(item) && typeof item === "object" && !Array.isArray(item));
-  }
-  return [];
-}
-function parseRules(response) {
-  const result2 = /* @__PURE__ */ new Map();
-  for (const row of rows(response)) {
-    const symbol = stringValue(row.symbol);
-    if (!symbol) continue;
-    result2.set(normalizedSymbol(symbol), {
-      symbol,
-      quantityPrecision: nonNegativeInteger(row.quantityPrecision),
-      pricePrecision: nonNegativeInteger(row.pricePrecision),
-      limitVolumeMin: stringValue(row.limitVolumeMin),
-      limitAmountMin: stringValue(row.limitAmountMin),
-      limitPriceMin: stringValue(row.limitPriceMin)
-    });
-  }
-  return result2;
-}
-async function loadRules(context) {
-  const key = cacheKey(context);
-  const current = cache.get(key);
-  if (current && current.expiresAt > Date.now()) return current;
-  const pending = pendingLoads.get(key);
-  if (pending) return pending;
-  const load = context.api.symbols().then((response) => {
-    const entry = { rules: parseRules(response), expiresAt: Date.now() + CACHE_TTL_MS };
-    cache.set(key, entry);
-    return entry;
-  }).finally(() => pendingLoads.delete(key));
-  pendingLoads.set(key, load);
-  return load;
-}
-async function getSymbolRule(context, symbol) {
-  const entry = await loadRules(context);
-  const rule = entry.rules.get(normalizedSymbol(symbol));
-  if (!rule) throw new AiHubError("AI_HUB_SYMBOL_NOT_FOUND", `Symbol "${symbol}" was not returned by the configured tenant OpenAPI symbols endpoint.`);
-  return rule;
-}
-function decimalScale(value) {
-  return value.split(".")[1]?.length ?? 0;
-}
-function decimalParts(value) {
-  const [whole, fraction = ""] = value.split(".");
-  return { digits: BigInt(`${whole}${fraction}`.replace(/^0+(?=\d)/, "") || "0"), scale: fraction.length };
-}
-function compareDecimal(left, right) {
-  const a = decimalParts(left);
-  const b = decimalParts(right);
-  const scale = Math.max(a.scale, b.scale);
-  const leftValue = a.digits * 10n ** BigInt(scale - a.scale);
-  const rightValue = b.digits * 10n ** BigInt(scale - b.scale);
-  return leftValue === rightValue ? 0 : leftValue > rightValue ? 1 : -1;
-}
-function multipliedDecimal(left, right) {
-  const a = decimalParts(left);
-  const b = decimalParts(right);
-  const scale = a.scale + b.scale;
-  const digits = (a.digits * b.digits).toString();
-  if (scale === 0) return digits;
-  const padded = digits.padStart(scale + 1, "0");
-  return `${padded.slice(0, -scale)}.${padded.slice(-scale)}`;
-}
-function assertPrecision(name, value, precision, symbol) {
-  if (precision !== void 0 && decimalScale(value) > precision) {
-    throw new AiHubError("AI_HUB_SYMBOL_PRECISION_INVALID", `${name} for ${symbol} allows at most ${precision} decimal places; received ${value}.`);
-  }
-}
-function assertAtLeast(name, value, minimum, symbol) {
-  if (minimum && compareDecimal(value, minimum) < 0) {
-    throw new AiHubError("AI_HUB_SYMBOL_MINIMUM_NOT_MET", `${name} for ${symbol} must be at least ${minimum}; received ${value}.`);
-  }
-}
-async function preflightSymbolOrder(context, order) {
-  const rule = await getSymbolRule(context, order.symbol);
-  if (order.type === "MARKET" && order.side === "BUY" && order.quoteAmount) {
-    assertPrecision("quoteAmount", order.quoteAmount, rule.pricePrecision, rule.symbol);
-  }
-  if (order.baseQuantity) assertPrecision("baseQuantity", order.baseQuantity, rule.quantityPrecision, rule.symbol);
-  if (order.price) assertPrecision("price", order.price, rule.pricePrecision, rule.symbol);
-  if (order.type === "LIMIT" && order.baseQuantity && order.price) {
-    assertAtLeast("baseQuantity", order.baseQuantity, rule.limitVolumeMin, rule.symbol);
-    assertAtLeast("price", order.price, rule.limitPriceMin, rule.symbol);
-    if (rule.limitAmountMin && compareDecimal(rule.limitAmountMin, "0") > 0) {
-      assertAtLeast("limit order amount", multipliedDecimal(order.baseQuantity, order.price), rule.limitAmountMin, rule.symbol);
-    }
-  }
-  return rule;
-}
-
 // ../core/src/tools/margin-tools.ts
 function validateMarginSemanticOrder(input) {
   const value = strictObject(input, ["symbol", "side", "type", "quoteAmount", "baseQuantity", "price", "newClientOrderId", "isolated"]);
@@ -2526,6 +2623,9 @@ function toMcpTool(tool) {
     }
   };
 }
+function isMcpVisible(tool) {
+  return tool.mcpVisible !== false;
+}
 function prepareToolName(tool) {
   const [prefix, ...rest] = tool.name.split("_");
   return `${prefix ?? "spot"}_prepare_${rest.join("_")}`;
@@ -2543,10 +2643,10 @@ function createServer(profileName, readOnly) {
   const registry = createToolRegistry();
   const writeExecutor = new ToolWriteExecutor(registry);
   const server = new Server(
-    { name: "ai-hub-agent-trade", version: "0.1.6" },
+    { name: "ai-hub-agent-trade", version: "0.1.8" },
     {
       capabilities: { tools: {} },
-      instructions: "Every successful read-tool response provides both JSON text and MCP structuredContent in the envelope { ok: true, data: ... }. All read-tool data is normalized: dataType=array means use data.items and data.count; dataType=object or scalar means use data.value; dataType=null means no value. Inspect data.dataType before formatting and never assume undocumented nested keys. For broad market questions, call market_search_symbols, market_get_ticker_summary, market_get_depth_summary, market_get_trades_summary, or market_get_klines_summary instead of fetching large raw payloads. Prepare/confirm tools return their documented action payload directly in data. For every state-changing action, call only a spot_prepare_* or margin_prepare_* tool first and show its exact summary to the user. Stop and wait for a new, explicit user confirmation message. Only then call confirm_action with that new message verbatim in userConfirmation. Never call prepare and confirm consecutively for one user instruction; never infer confirmation from prior intent, silence, or an Agent-generated message. For spot and margin orders, MARKET BUY always uses quoteAmount (the quote asset to spend); MARKET SELL uses baseQuantity (the base asset to sell). Never reinterpret a requested base quantity as quoteAmount."
+      instructions: "Every successful read-tool response provides both JSON text and MCP structuredContent in the envelope { ok: true, data: ... }. All read-tool data is normalized: dataType=array means use data.items and data.count; dataType=object or scalar means use data.value; dataType=null means no value. Inspect data.dataType before formatting and never assume undocumented nested keys. For symbol lookup, use market_search_symbols. It returns an object whose bounded matches are at data.value.items; the complete raw symbols payload is intentionally unavailable in MCP. For broad market questions, call market_get_ticker_summary, market_get_depth_summary, market_get_trades_summary, or market_get_klines_summary instead of fetching large raw payloads. Account types are fixed: 1=Spot, 2=Isolated Margin, 3=Cross Margin, 4=C2C, 5=Derivatives. Never call account type 2 Derivatives. For Spot to Derivatives, use account_prepare_transfer with fromAccount=EXCHANGE and toAccount=FUTURE. Use wallet_prepare_universal_transfer for margin or C2C; account type 2 requires its isolated-margin trading pair in symbol. Prepare/confirm tools return their documented action payload directly in data. For every state-changing action, call only a spot_prepare_* or margin_prepare_* tool first and show its exact summary to the user. Stop and wait for a new, explicit user confirmation message. Only then call confirm_action with that new message verbatim in userConfirmation. Never call prepare and confirm consecutively for one user instruction; never infer confirmation from prior intent, silence, or an Agent-generated message. For spot and margin orders, MARKET BUY always uses quoteAmount (the quote asset to spend); MARKET SELL uses baseQuantity (the base asset to sell). Never reinterpret a requested base quantity as quoteAmount."
     }
   );
   server.setRequestHandler(ListToolsRequestSchema, async () => ({
@@ -2558,7 +2658,7 @@ function createServer(profileName, readOnly) {
         inputSchema: { type: "object", additionalProperties: false },
         annotations: { readOnlyHint: true, destructiveHint: false, idempotentHint: true, openWorldHint: false }
       },
-      ...registry.list({ readOnly }).flatMap((tool) => tool.operation === "write" ? [toPrepareMcpTool(tool)] : [toMcpTool(tool)]),
+      ...registry.list({ readOnly }).filter(isMcpVisible).flatMap((tool) => tool.operation === "write" ? [toPrepareMcpTool(tool)] : [toMcpTool(tool)]),
       ...readOnly ? [] : [{
         name: CONFIRM_ACTION_TOOL,
         title: "Confirm Prepared Action",
@@ -2577,7 +2677,7 @@ function createServer(profileName, readOnly) {
           data: {
             profile: { name: context.profile.name, host: new URL(context.profile.openApiBaseUrl).host, configVersion: context.profile.configVersion },
             readOnly,
-            capabilities: registry.capabilities(context, { readOnly })
+            capabilities: registry.capabilities(context, { readOnly }).filter((capability) => isMcpVisible(registry.byName(capability.name)))
           }
         });
       }
@@ -2588,10 +2688,14 @@ function createServer(profileName, readOnly) {
         }
         return result({ ok: true, data: await writeExecutor.confirm(input.confirmationId, input.userConfirmation, context) });
       }
-      const preparedTool = registry.list({ readOnly: false }).find((tool) => tool.operation === "write" && prepareToolName(tool) === request.params.name);
+      const preparedTool = registry.list({ readOnly: false }).find((tool2) => tool2.operation === "write" && prepareToolName(tool2) === request.params.name);
       if (preparedTool) {
         return result({ ok: true, data: await writeExecutor.prepare(preparedTool.name, request.params.arguments ?? {}, context) });
       }
+      const tool = registry.byName(request.params.name, { readOnly });
+      if (!isMcpVisible(tool)) {
+        throw new AiHubError("AI_HUB_TOOL_NOT_AVAILABLE", `Tool "${request.params.name}" is not available through MCP. Use market_search_symbols for symbol lookup.`);
+      }
       const data = await registry.execute(request.params.name, request.params.arguments ?? {}, context, { readOnly });
       return toMcpReadResult(formatMcpData(data));
     } catch (error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aihubspot/agent-trade-mcp",
-  "version": "0.1.6",
+  "version": "0.1.8",
   "description": "Local stdio MCP server for AI Hub spot trading tools",
   "license": "UNLICENSED",
   "repository": {