Refactor swapper provider URL handling

Centralize and simplify provider URL generation and adjust ProxyProvider construction.

- Import SwapperProvider in config.
- Add get_swap_provider_url(provider) -> String to build provider-specific endpoint: "{API_BASE_URL}/v1/swaps/providers/{provider.id()}".
- Remove use of API_BASE_URL_ENV and env lookup in get_swap_proxy_url; it now uses API_BASE_URL directly.
- Update ProxyProvider: rename new_with_path to new_with_provider (signature adjusted), initialize the RPC client with get_swap_provider_url(provider) instead of composing a proxy path, and update new_okx to call the new constructor without a hardcoded path.

This refactor removes per-path env handling and consolidates provider endpoint formation for clarity and consistency.

Author: gemcoder21

core/crates/swapper/src/config.rs

@@ -1,16 +1,18 @@
-use crate::{SwapperSlippage, SwapperSlippageMode};
+use crate::{SwapperProvider, SwapperSlippage, SwapperSlippageMode};
 use primitives::Chain;
 
 pub const DEFAULT_SLIPPAGE_BPS: u32 = 100;
 pub const DEFAULT_SWAP_FEE_BPS: u32 = 50;
 pub const DEFAULT_CHAINFLIP_FEE_BPS: u32 = 45;
 
 pub const API_BASE_URL: &str = "https://api.gemwallet.com";
-pub const API_BASE_URL_ENV: &str = "GEM_API_BASE_URL";
+
+pub fn get_swap_provider_url(provider: SwapperProvider) -> String {
+    format!("{API_BASE_URL}/v1/swaps/providers/{}", provider.id())
+}
 
 pub fn get_swap_proxy_url(path: &str) -> String {
-    let base_url = std::env::var(API_BASE_URL_ENV).unwrap_or_else(|_| API_BASE_URL.to_string());
-    format!("{}/proxy/swap/{path}", base_url.trim_end_matches('/'))
+    format!("{API_BASE_URL}/proxy/swap/{path}")
 }
 
 #[derive(Debug, Clone, PartialEq)]

core/crates/swapper/src/proxy/provider.rs

@@ -7,7 +7,7 @@ use crate::{
     FetchQuoteData, ProviderData, ProviderType, Quote, QuoteRequest, Route, SwapResult, Swapper, SwapperError, SwapperProvider, SwapperProviderMode, SwapperQuoteData,
     alien::{RpcClient, RpcProvider},
     approval::{DEFAULT_EVM_SWAP_GAS_LIMIT, check_approval_erc20, get_swap_gas_limit_with_approval},
-    config::get_swap_proxy_url,
+    config::get_swap_provider_url,
     cross_chain::VaultAddresses,
     fees::{DEFAULT_AGGREGATOR_FEE_BPS, DEFAULT_SWAP_FEE_BPS},
     models::SwapperChainAsset,
@@ -97,16 +97,14 @@ where
 }
 
 impl ProxyProvider<RpcClient> {
-    fn new_with_path(provider: SwapperProvider, path: &str, assets: Vec<SwapperChainAsset>, rpc_provider: Arc<dyn RpcProvider>) -> Self {
-        let base_url = get_swap_proxy_url(&format!("swapper/{path}"));
-        let client = ProxyClient::new(RpcClient::new(base_url, rpc_provider.clone()));
+    fn new_with_provider(provider: SwapperProvider, assets: Vec<SwapperChainAsset>, rpc_provider: Arc<dyn RpcProvider>) -> Self {
+        let client = ProxyClient::new(RpcClient::new(get_swap_provider_url(provider), rpc_provider.clone()));
         Self::new_with_client(provider, client, assets, rpc_provider)
     }
 
     pub fn new_okx(rpc_provider: Arc<dyn RpcProvider>) -> Self {
-        Self::new_with_path(
+        Self::new_with_provider(
             SwapperProvider::Okx,
-            "okx",
             vec![
                 SwapperChainAsset::All(Chain::Solana),
                 SwapperChainAsset::All(Chain::Ethereum),