-
-
Notifications
You must be signed in to change notification settings - Fork 246
fix(solana): handle Helius mixed parsed/base64 responses and Meteora position width errors #555
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
rapcmia
merged 4 commits into
development
from
fix/meteora-struct-error-and-position-width
Nov 17, 2025
Merged
fix(solana): handle Helius mixed parsed/base64 responses and Meteora position width errors #555
rapcmia
merged 4 commits into
development
from
fix/meteora-struct-error-and-position-width
Nov 17, 2025
+107
−31
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sync / Gateway staging -> master for Hummingbot version 2.9.0
…position width errors - Add fallback to base64 encoding when Helius returns mixed parsed/base64 token account data - Fix StructError validation failures in deprecated getBalance() method - Remove balance validation from Meteora openPosition (aligns with PancakeSwap-Sol) - Add proper error handling for InvalidPositionWidth (error 6040) with user guidance - Preserve actual error messages in Meteora openPosition route handler 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Contributor
|
Commit 2b0f924
Test meteora position opening with valid and invalid ranges ✅
Test if
|
rapcmia
approved these changes
Nov 17, 2025
Contributor
|
Merged to development and part of release v2.11.0 |
fengtality
added a commit
that referenced
this pull request
Nov 19, 2025
- Add Collect Fees and Close Position buttons to LiquidityPositionCard - Add confirmation dialogs for all position operations (Add, Collect Fees, Close) - Add Trading API namespace with collectFees and closePosition methods - Replace position card layout with step area chart visualization - Quote liquidity shown in accent color (left of current price) - Base liquidity shown in primary color (right of current price) - Current price marked with vertical reference line - Always display unclaimed fees section (even when 0) - Re-apply PR #555 fix for Meteora InvalidPositionWidth error handling 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR fixes two critical issues with Solana token account handling and Meteora position creation:
Changes
Helius Mixed Response Handling
fetchTokenAccountsBase64()fallback method that uses base64 encoding exclusivelyfetchTokenAccounts()to catch StructError and fall back to base64 encodinggetBalance()method to handle mixed responsesMeteora Position Width Error Handling
Code Quality Improvements
getBalance()method in connector codeTest Plan
🤖 Generated with Claude Code