Skip to content

Network-first pools management and default networks feature#76

Open
fengtality wants to merge 11 commits intomainfrom
feat/network-pools-management
Open

Network-first pools management and default networks feature#76
fengtality wants to merge 11 commits intomainfrom
feat/network-pools-management

Conversation

@fengtality
Copy link
Copy Markdown
Contributor

@fengtality fengtality commented Apr 28, 2026

Summary

  • Add Orca pool listing to /lp menu with full feature parity with Meteora
  • Fix pool pair display truncation (quote asset was being cut off)
  • Fix chart not showing: add missing context parameter to fetch_liquidity_bins
  • Fix Y-axis range for OHLCV chart (was starting from 0, squishing candles)
  • Show full pool address for easy copy/paste instead of truncated version

Related PRs

Test plan

  • Manual testing of Orca pool listing
  • Verify chart displays with proper Y-axis range
  • Verify full address is displayed and copyable

🤖 Generated with Claude Code

fengtality and others added 7 commits April 28, 2026 13:59
Telegram Handlers:
- Update show_connector_pools() to use get_network_pools()
- Update prompt_remove_pool() to use get_network_pools()
- Update remove_pool() to use delete_network_pool()
- Update handle_pool_input() to use add_network_pool()
- Get chain from connector data for network_id construction

MCP Tools:
- Update pools list action to use network_id parameter
- Update pools add action to use add_network_pool()
- Add pools delete action using delete_network_pool()
- Update error messages to guide users on network_id format

These changes align with hummingbot-api updates that organize pools
by network (chain-network format) instead of by connector.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Changed from connector-first to network-first selection
- Now shows list of networks (like tokens menu)
- Pools displayed with pagination and grid layout
- Pool details show connector, type, fee, and address
- Add pool now uses format: connector,pool_type,address
- Uses network_id directly instead of building from connector chain

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Adding and removing pools takes effect immediately without
requiring a gateway restart.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Added get_default_networks() helper to fetch default_networks from
  solana-mainnet-beta and ethereum-mainnet configs
- Tokens menu now shows only default networks with "All Networks" button
- Pools menu now shows only default networks with "All Networks" button
- Networks detail view shows default status with toggle button
- Users can add/remove networks from defaults via the toggle button

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Networks that are in default_networks now show ✓ prefix
- Updated message to show default count and explain checkmark meaning

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Abbreviate long base token names with ellipsis
- Always show full quote token symbol
- Fix truncation issue where quote asset was cut off

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
fengtality and others added 4 commits April 28, 2026 17:42
- Add Orca to /lp explore pools menu
- Support both Meteora and Orca connectors in pool listing
- Store connector in context for pool selection
- Add connector field to pool dicts for proper handling
- Update headers to show connector name (Meteora/Orca)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add missing context parameter to fetch_liquidity_bins function
- Pass context to all fetch_liquidity_bins calls in pools.py and geckoterminal.py
- Show full pool address in code block for easy copy/paste instead of truncated

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The Y-axis was starting from 0, squishing candles at the top when prices
were high (e.g., 50). Now calculates range from OHLCV lows/highs with 5%
padding for proper candle visibility.

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