🚀 Eliminate KuCoin Hardcoded Dependency - Enable 66+ Exchange Ecosystem - #15
Merged
Merged
Conversation
- Created setup.py with dynamic version generation from git commit - Fixed version format from invalid '2026.02.21-e713ca2' to compliant '2026.2.21.dev0+commit' - Added robust error handling for git operations - Successfully builds both source distribution (.tar.gz) and wheel (.whl) - Resolves CI/CD pipeline packaging failures - Supports console script entry point for 'powertrader' command
- Remove hardcoded 'robinhood' default from pt_hub.py primary_exchange - Add handling for empty primary_exchange to prevent credential errors - Set US region default exchanges to disabled instead of enabling robinhood - Prevents CI failures when GitHub secrets/credentials aren't configured - Allows graceful operation without requiring immediate exchange setup - Resolves integration test failures in CI/CD pipeline
- Move credential loading from import-time to runtime in pt_trader.py - Replace SystemExit(1) with graceful error handling - Add POWERTRADER_ENV=test check to skip credentials in CI - Credentials now loaded only when CryptoAPITrading class is instantiated - Prevents build failures when GitHub secrets/credentials aren't configured - Allows basic import testing without requiring exchange credentials This follows proper CI/CD practices where builds shouldn't need external credentials
- Make KuCoin client imports optional in pt_thinker.py and pt_trainer.py - Skip KuCoin initialization in POWERTRADER_ENV=test environments - Add proper error handling for when market client is unavailable - Move pt_thinker initialization from import-time to main block - Prevents CI/CD failures from pkg_resources dependency issues in kucoin-python Key changes: - Conditional KuCoin imports with graceful fallback - Import-time network calls moved to runtime only - Compatible with Python 3.13+ where pkg_resources was deprecated - Maintains full functionality in production environments
…tion - Add DataProvider abstraction layer to decouple core modules from specific exchanges - Create comprehensive exchange configuration with 66+ platforms across: * Centralized exchanges (28): Tier 1/2 + regional (Rain, Yellow Card, Quidax, VALR) * DeFi/DEX platforms (18): Ethereum, Layer2, lending, staking protocols * Derivatives (7): perpetuals, options, synthetics * Specialized platforms (8): prediction markets, P2P, NFT/gaming * Cross-chain infrastructure (5): bridges and aggregators - Update pt_thinker.py and pt_trainer.py to use provider interface - Enable regional exchange optimization (US/EU/Asia/Africa-MENA priorities) - Implement intelligent fallback strategies across entire ecosystem - Restore user choice in exchange selection vs KuCoin lock-in Resolves architectural issue where sophisticated multi-exchange system was bypassed by hardcoded dependencies in core prediction modules.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
🎯 Problem Solved
Issue: PowerTrader had hardcoded KuCoin dependencies in core modules (
pt_thinker.pyandpt_trainer.py) despite having a sophisticated 65+ exchange ecosystem available. This forced all users through KuCoin regardless of their preferences or regional availability.🏗️ Architectural Solution
Created a DataProvider abstraction layer that eliminates exchange lock-in and enables user choice across the complete cryptocurrency ecosystem.
✅ Key Changes
pt_data_provider.py): Clean interface that decouples prediction logic from specific exchangespt_thinker.pyandpt_trainer.pynow use provider interface instead of hardcoded KuCoin callsdata_provider_config.json): 66 exchanges across all categories🌍 Complete Exchange Ecosystem (66 Platforms)
🎯 Regional Optimization
🚀 Impact
✅ User Freedom: No more forced KuCoin dependency
✅ Regional Compliance: Optimized exchange selection by geography
✅ Intelligent Fallbacks: Automatic failover across entire ecosystem
✅ Performance: Regional routing for better latency and fees
✅ Future-Proof: Easy to add new exchanges via configuration
🧪 Testing
Result: PowerTrader now properly utilizes its full 66-exchange infrastructure instead of being locked to a single provider! 🎉