@aihubspot/agent-trade-mcp 0.1.7 → 0.1.8

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",
@@ -2581,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.7" },
+    { 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 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. 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 () => ({

package/package.json

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