Skip to content

feat: make pool_address optional, add swap/lp executor docs#16

Open
fengtality wants to merge 5 commits intomainfrom
feature/swap-lp-executor-support
Open

feat: make pool_address optional, add swap/lp executor docs#16
fengtality wants to merge 5 commits intomainfrom
feature/swap-lp-executor-support

Conversation

@fengtality
Copy link
Copy Markdown
Contributor

@fengtality fengtality commented Mar 12, 2026

Summary

  • Make pool_address optional in CLMM methods
  • Add documentation for swap_executor and lp_executor usage
  • Add get_bot_lp_history() method for LP position history

Key Changes

  • routers/executors.py: Add swap_executor and lp_executor examples and documentation
  • routers/gateway_clmm.py: Make pool_address optional where applicable
  • routers/bot_orchestration.py: Add get_bot_lp_history() for LP position history

Related PRs

QA Note: These 4 PRs should be tested together as a coordinated release.

Repo PR Description
hummingbot #8150 Move retry logic from executors to connectors
hummingbot-api #142 Add SwapExecutor with connector-level retry
hummingbot-api-client #16 Add swap/lp executor client support (this PR)
gateway #620 Fix fee calculation, add error parsing

Test plan

  • Verify swap_executor can be created via client
  • Verify lp_executor can be created via client
  • Verify get_bot_lp_history() returns LP position data

🤖 Generated with Claude Code

fengtality and others added 3 commits March 12, 2026 18:46
- get_positions_owned: pool_address is now optional (fetches all positions if not provided)
- executors: added documentation for swap_executor and lp_executor types
- executors: added example configs for DEX swaps and LP positions

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- get_positions_owned: pool_address is now optional (fetches all positions if not provided)
- executors: added documentation for swap_executor and lp_executor types
- executors: added example configs for DEX swaps and LP positions

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Adds client support for the /bot-orchestration/{bot_name}/lphistory endpoint
which returns LP-specific data including position updates, fees collected,
and liquidity additions/removals.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- swap_executor: add required network field to examples
- lp_executor: add network and pool_address fields to examples

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant