openbroker 1.0.52 → 1.0.53

package/CHANGELOG.md

@@ -2,10 +2,15 @@
 
 All notable changes to Open Broker will be documented in this file.
 
+## [1.0.53] - 2026-03-09
+
+### Fixed
+- **HIP-3 Isolated Margin: Use Max Leverage**: The 3x leverage cap was too conservative for isolated margin — at 3x a $83 CL position needs $28 margin, which gets rejected even on funded accounts. Now uses the asset's `maxLeverage` (e.g., 20x for CL, 25x for GOLD) to minimize margin requirement. At 20x isolated, the same position only needs ~$4 margin.
+
 ## [1.0.52] - 2026-03-09
 
 ### Fixed
-- **HIP-3 Trading: Isolated Margin**: HIP-3 perps require isolated margin mode (per Hyperliquid docs), but orders were sent without setting it — causing "Insufficient margin to place order" rejections. Now automatically sets isolated margin (3x or asset max, whichever is lower) on first order for each HIP-3 asset. Affects all trading commands: `buy`, `sell`, `market`, `limit`, `trigger`, `tpsl`, `bracket`, `chase`, `twap`, `scale`.
+- **HIP-3 Trading: Isolated Margin**: HIP-3 perps require isolated margin mode (per Hyperliquid docs), but orders were sent without setting it — causing "Insufficient margin to place order" rejections. Now automatically sets isolated margin on first order for each HIP-3 asset. Affects all trading commands: `buy`, `sell`, `market`, `limit`, `trigger`, `tpsl`, `bracket`, `chase`, `twap`, `scale`.
 
 ## [1.0.51] - 2026-03-09
 

package/SKILL.md

@@ -4,7 +4,7 @@ description: Hyperliquid trading plugin with background position monitoring. Exe
 license: MIT
 compatibility: Requires Node.js 22+, network access to api.hyperliquid.xyz
 homepage: https://www.npmjs.com/package/openbroker
-metadata: {"author": "monemetrics", "version": "1.0.52", "openclaw": {"requires": {"bins": ["openbroker"], "env": ["HYPERLIQUID_PRIVATE_KEY"]}, "primaryEnv": "HYPERLIQUID_PRIVATE_KEY", "install": [{"id": "node", "kind": "node", "package": "openbroker", "bins": ["openbroker"], "label": "Install openbroker (npm)"}]}}
+metadata: {"author": "monemetrics", "version": "1.0.53", "openclaw": {"requires": {"bins": ["openbroker"], "env": ["HYPERLIQUID_PRIVATE_KEY"]}, "primaryEnv": "HYPERLIQUID_PRIVATE_KEY", "install": [{"id": "node", "kind": "node", "package": "openbroker", "bins": ["openbroker"], "label": "Install openbroker (npm)"}]}}
 allowed-tools: ob_account ob_positions ob_funding ob_markets ob_search ob_spot ob_fills ob_orders ob_order_status ob_fees ob_candles ob_funding_history ob_trades ob_rate_limit ob_funding_scan ob_buy ob_sell ob_limit ob_trigger ob_tpsl ob_cancel ob_twap ob_bracket ob_chase ob_watcher_status Bash(openbroker:*)
 ---
 

package/openclaw.plugin.json

@@ -1,7 +1,7 @@
 {
   "id": "openbroker",
   "name": "OpenBroker — Hyperliquid Trading",
-  "version": "1.0.52",
+  "version": "1.0.53",
   "description": "Trade on Hyperliquid DEX with background position monitoring",
   "configSchema": {
     "type": "object",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openbroker",
-  "version": "1.0.52",
+  "version": "1.0.53",
   "description": "Hyperliquid trading CLI - execute orders, manage positions, and run trading strategies",
   "type": "module",
   "bin": {

package/scripts/core/client.ts

@@ -1065,12 +1065,41 @@ export class HyperliquidClient {
     return data;
   }
 
-  async getUserState(user?: string): Promise<ClearinghouseState> {
-    this.log('Fetching clearinghouseState for:', user ?? this.address);
-    const response = await this.info.clearinghouseState({ user: user ?? this.address });
+  async getUserState(user?: string, dex?: string): Promise<ClearinghouseState> {
+    this.log('Fetching clearinghouseState for:', user ?? this.address, dex ? `dex: ${dex}` : '');
+    const params: { user: string; dex?: string } = { user: user ?? this.address };
+    if (dex !== undefined) params.dex = dex;
+    const response = await this.info.clearinghouseState(params as any);
     return response as ClearinghouseState;
   }
 
+  /**
+   * Get user state across all dexes (main + HIP-3).
+   * Returns the main state with HIP-3 positions merged into assetPositions.
+   */
+  async getUserStateAll(user?: string): Promise<ClearinghouseState> {
+    await this.getMetaAndAssetCtxs(); // Ensure HIP-3 dex list is loaded
+
+    const mainState = await this.getUserState(user);
+    const dexs = await this.getPerpDexs();
+
+    for (let i = 1; i < dexs.length; i++) {
+      const dex = dexs[i];
+      if (!dex) continue;
+
+      try {
+        const dexState = await this.getUserState(user, dex.name);
+        if (dexState.assetPositions?.length > 0) {
+          mainState.assetPositions.push(...dexState.assetPositions);
+        }
+      } catch (err) {
+        this.log(`Failed to fetch state for dex ${dex.name}:`, err instanceof Error ? err.message : String(err));
+      }
+    }
+
+    return mainState;
+  }
+
   async getOpenOrders(user?: string): Promise<OpenOrder[]> {
     this.log('Fetching openOrders for:', user ?? this.address);
     const response = await this.info.openOrders({ user: user ?? this.address });
@@ -1080,37 +1109,51 @@ export class HyperliquidClient {
   // ============ Trading ============
 
   /**
-   * HIP-3 perps require isolated margin mode. Automatically sets isolated margin
-   * with default leverage (3x) on first order for each HIP-3 asset this session.
+   * HIP-3 perps have independent margin per dex. Before ordering:
+   * 1. Set isolated margin mode (required for HIP-3)
+   * 2. Transfer USDC from main perp to the HIP-3 dex (each dex has its own balance)
    */
-  private async ensureHip3Isolated(coin: string): Promise<void> {
+  private async ensureHip3Ready(coin: string, notional: number): Promise<void> {
     if (!this.isHip3(coin)) return;
-    if (this.hip3IsolatedSet.has(coin)) return;
 
     const dexInfo = this.coinDexMap.get(coin);
-    const maxLev = this.getHip3MaxLeverage(coin);
-    const leverage = Math.min(3, maxLev || 3);
+    if (!dexInfo?.dexName) return;
+
+    // Set isolated margin on first order per asset
+    if (!this.hip3IsolatedSet.has(coin)) {
+      const maxLev = this.hip3MaxLeverageMap.get(coin) ?? 10;
+      this.log(`HIP-3 asset ${coin} (dex: ${dexInfo.dexName}) — setting isolated margin at ${maxLev}x`);
+      try {
+        await this.updateLeverage(coin, maxLev, false); // false = isolated
+        this.hip3IsolatedSet.add(coin);
+      } catch (err) {
+        this.log(`Failed to set isolated margin for ${coin}:`, err instanceof Error ? err.message : String(err));
+        this.hip3IsolatedSet.add(coin);
+      }
+    }
 
-    this.log(`HIP-3 asset ${coin} (dex: ${dexInfo?.dexName}) — setting isolated margin at ${leverage}x`);
+    // Transfer USDC to the HIP-3 dex to cover margin
+    const maxLev = this.hip3MaxLeverageMap.get(coin) ?? 10;
+    const requiredMargin = notional / maxLev;
+    // Add 20% buffer for fees and slippage
+    const transferAmount = Math.ceil(requiredMargin * 1.2 * 100) / 100;
+
+    this.log(`HIP-3 margin transfer: ${transferAmount} USDC from main → ${dexInfo.dexName} (notional: ${notional}, leverage: ${maxLev}x)`);
     try {
-      await this.updateLeverage(coin, leverage, false); // false = isolated
-      this.hip3IsolatedSet.add(coin);
+      await this.exchange.sendAsset({
+        destination: this.address as `0x${string}`,
+        sourceDex: '',            // main perp dex
+        destinationDex: dexInfo.dexName,
+        token: 'USDC:0x6d1e7cde53ba9467b783cb7c530ce054',
+        amount: String(transferAmount),
+      });
+      this.log(`Transferred ${transferAmount} USDC to ${dexInfo.dexName} dex`);
     } catch (err) {
-      // Log but don't block — might already be set
-      this.log(`Failed to set isolated margin for ${coin}:`, err instanceof Error ? err.message : String(err));
-      this.hip3IsolatedSet.add(coin); // Don't retry every order
+      // Log but don't block — dex may already have sufficient balance
+      this.log(`Margin transfer to ${dexInfo.dexName} failed:`, err instanceof Error ? err.message : String(err));
     }
   }
 
-  /**
-   * Get maxLeverage for a HIP-3 asset from cached metadata.
-   */
-  private getHip3MaxLeverage(coin: string): number | null {
-    // Already loaded via getMetaAndAssetCtxs → loadHip3Assets
-    // Check if we cached it during loadHip3Assets
-    return this.hip3MaxLeverageMap.get(coin) ?? null;
-  }
-
   async order(
     coin: string,
     isBuy: boolean,
@@ -1123,8 +1166,8 @@ export class HyperliquidClient {
     this.requireTrading();
     await this.getMetaAndAssetCtxs();
 
-    // HIP-3 perps require isolated margin mode
-    await this.ensureHip3Isolated(coin);
+    // HIP-3 perps: set isolated margin + transfer USDC to dex
+    await this.ensureHip3Ready(coin, size * price);
 
     const assetIndex = this.getAssetIndex(coin);
     const szDecimals = this.getSzDecimals(coin);
@@ -1244,8 +1287,8 @@ export class HyperliquidClient {
     this.requireTrading();
     await this.getMetaAndAssetCtxs();
 
-    // HIP-3 perps require isolated margin mode
-    await this.ensureHip3Isolated(coin);
+    // HIP-3 perps: set isolated margin + transfer USDC to dex
+    await this.ensureHip3Ready(coin, size * limitPrice);
 
     const assetIndex = this.getAssetIndex(coin);
     const szDecimals = this.getSzDecimals(coin);

package/scripts/info/account.ts

@@ -30,7 +30,7 @@ async function main() {
   console.log('');
 
   try {
-    const state = await client.getUserState();
+    const state = await client.getUserStateAll();
 
     const margin = state.crossMarginSummary;
     const accountValue = parseFloat(margin.accountValue);

package/scripts/info/positions.ts

@@ -14,7 +14,7 @@ async function main() {
 
   try {
     const [state, mids, fundingHistory] = await Promise.all([
-      client.getUserState(),
+      client.getUserStateAll(),
       client.getAllMids(),
       client.getUserFunding(),
     ]);

package/scripts/plugin/tools.ts

@@ -34,7 +34,7 @@ export function createTools(watcher: PositionWatcher | null): PluginTool[] {
       async execute(_id, params) {
         const { getClient } = await import('../core/client.js');
         const client = getClient();
-        const state = await client.getUserState();
+        const state = await client.getUserStateAll();
 
         const result: Record<string, unknown> = {
           address: client.address,
@@ -85,7 +85,7 @@ export function createTools(watcher: PositionWatcher | null): PluginTool[] {
       async execute(_id, params) {
         const { getClient } = await import('../core/client.js');
         const client = getClient();
-        const state = await client.getUserState();
+        const state = await client.getUserStateAll();
 
         let positions = state.assetPositions
           .filter(ap => parseFloat(ap.position.szi) !== 0)
@@ -969,7 +969,7 @@ export function createTools(watcher: PositionWatcher | null): PluginTool[] {
         const coin = normalizeCoin(params.coin as string);
 
         // Get current position
-        const state = await client.getUserState();
+        const state = await client.getUserStateAll();
         const position = state.assetPositions.find(
           ap => ap.position.coin === coin && parseFloat(ap.position.szi) !== 0,
         );

package/scripts/plugin/watcher.ts

@@ -117,7 +117,7 @@ export class PositionWatcher implements PluginService {
     try {
       const { getClient } = await import('../core/client.js');
       const client = getClient();
-      const state = await client.getUserState(this.accountAddress);
+      const state = await client.getUserStateAll(this.accountAddress);
 
       const snapshot = this.buildSnapshot(state);