Revert "feat: make pool_address optional, add swap/lp executor docs"

This reverts commit 870c9ad.

Author: fengtality

hummingbot_api_client/routers/executors.py

@@ -3,16 +3,7 @@
 
 
 class ExecutorsRouter(BaseRouter):
-    """Executors router for managing trading executors and position holds.
-
-    Supported executor types:
-    - order_executor: Simple order execution with retry logic (MARKET, LIMIT, LIMIT_MAKER, LIMIT_CHASER)
-    - position_executor: Directional positions with stop-loss and take-profit
-    - dca_executor: Dollar-cost averaging over time
-    - grid_executor: Grid trading in ranging markets
-    - swap_executor: DEX token swaps via Gateway (Jupiter, Raydium)
-    - lp_executor: CLMM liquidity positions on DEXs (Meteora, Raydium)
-    """
+    """Executors router for managing trading executors and position holds."""
 
     # Executor CRUD Operations
     async def create_executor(
@@ -24,64 +15,17 @@ async def create_executor(
         Create a new executor with the given configuration.
 
         Args:
-            executor_config: Executor configuration dictionary containing:
-                - type: Executor type (order_executor, position_executor, dca_executor,
-                        grid_executor, swap_executor, lp_executor)
-                - Other fields depend on executor type
-
-        Common executor configs:
-
-        swap_executor (DEX swaps):
-            {
-                "type": "swap_executor",
-                "connector_name": "jupiter/router",
-                "trading_pair": "TOKEN-SOL",
-                "side": 1,  # 1=BUY, 2=SELL
-                "amount": "0.5"  # or "$10" for USD amount
-            }
-
-        lp_executor (CLMM LP positions):
-            {
-                "type": "lp_executor",
-                "connector_name": "meteora/clmm",
-                "trading_pair": "TOKEN-SOL",
-                "side": 0,  # 0=double-sided, 1=base only, 2=quote only
-                "lower_price": 0.001,
-                "upper_price": 0.002,
-                "base_amount": 1000,  # or quote_amount
-            }
-
+            executor_config: Executor configuration dictionary
             account_name: Account to run the executor on (optional)
 
         Returns:
             Created executor information
 
         Example:
-            # Create a DCA executor
             executor = await client.executors.create_executor(
-                {"type": "dca_executor", "trading_pair": "BTC-USDT", ...},
+                {"type": "dca", "trading_pair": "BTC-USDT", ...},
                 account_name="master_account"
             )
-
-            # Create a swap executor
-            executor = await client.executors.create_executor({
-                "type": "swap_executor",
-                "connector_name": "jupiter/router",
-                "trading_pair": "WIF-SOL",
-                "side": 1,
-                "amount": "$5"
-            })
-
-            # Create an LP executor
-            executor = await client.executors.create_executor({
-                "type": "lp_executor",
-                "connector_name": "meteora/clmm",
-                "trading_pair": "WIF-SOL",
-                "side": 0,
-                "lower_price": 0.5,
-                "upper_price": 1.5,
-                "base_amount": 100
-            })
         """
         body = {"executor_config": executor_config}
         if account_name is not None:

hummingbot_api_client/routers/gateway_clmm.py

@@ -248,29 +248,22 @@ async def get_positions_owned(
         self,
         connector: str,
         network: str,
-        pool_address: Optional[str] = None,
+        pool_address: str,
         wallet_address: Optional[str] = None
     ) -> List[Dict[str, Any]]:
         """
-        Get all liquidity positions owned by a wallet.
+        Get all liquidity positions owned by a wallet for a specific pool.
 
         Args:
             connector: CLMM connector (e.g., 'meteora')
             network: Network ID in format 'chain-network' (e.g., 'solana-mainnet-beta')
-            pool_address: Pool contract address (optional - if not provided, returns all positions)
+            pool_address: Pool contract address
             wallet_address: Wallet address (uses default if not provided)
 
         Returns:
-            List of position information
+            List of position information for the specified pool
 
         Example:
-            # Get all positions for a wallet
-            positions = await client.gateway_clmm.get_positions_owned(
-                connector='meteora',
-                network='solana-mainnet-beta'
-            )
-
-            # Get positions for a specific pool
             positions = await client.gateway_clmm.get_positions_owned(
                 connector='meteora',
                 network='solana-mainnet-beta',
@@ -281,10 +274,9 @@ async def get_positions_owned(
         """
         request_data = {
             "connector": connector,
-            "network": network
+            "network": network,
+            "pool_address": pool_address
         }
-        if pool_address:
-            request_data["pool_address"] = pool_address
         if wallet_address:
             request_data["wallet_address"] = wallet_address