Cleanup: remove dead code after P3 refactor

main.go

@@ -165,7 +165,7 @@ func main() {
 			t1 := time.Now()
 
 			// Scan for arbitrage entries using maker fees (limit orders)
-			makerOpps := ScanArbWithFees(store, makerFees)
+			makerOpps := ScanBGHL(store)
 			dashboard.UpdateScan(makerOpps)
 			t2 := time.Now()
 

scanner.go

@@ -4,28 +4,16 @@ import (
 	"sort"
 )
 
-// Exchange names
+// Exchange names — only Bitget and HyperLiquid are trading exchanges
 const (
-	ExBinance     = "Binance"
 	ExHyperLiquid = "HyperLiquid"
 	ExBitget      = "Bitget"
-	ExDydx        = "dYdX"
 )
 
-// Fee rates (%) — taker fees per exchange
-var feeRates = map[string]float64{
-	ExBinance:     0.040,
-	ExHyperLiquid: 0.035,
-	ExBitget:      0.040, // standard taker
-	ExDydx:        0.050, // dYdX v4 standard taker
-}
-
-// Maker fee rates (%) — for limit orders
+// Maker fee rates (%) — for limit orders on trading exchanges
 var makerFees = map[string]float64{
-	ExBinance:     0.020, // standard maker (USDT pairs)
 	ExHyperLiquid: 0.015,
 	ExBitget:      0.020, // standard maker
-	ExDydx:        0.020,
 }
 
 // TickerCoins defines all coins we monitor.
@@ -105,14 +93,3 @@ func ScanBGHL(store *PriceStore) []*ArbOpportunity {
 
 	return results
 }
-
-// ScanArbWithFees checks all coins — REDIRECTED to ScanBGHL for performance (P3-1).
-// Kept for backward compatibility; only BG↔HL is relevant for trading.
-func ScanArbWithFees(store *PriceStore, fees map[string]float64) []*ArbOpportunity {
-	return ScanBGHL(store)
-}
-
-// ScanArb checks all coins using taker fees.
-func ScanArb(store *PriceStore) []*ArbOpportunity {
-	return ScanArbWithFees(store, feeRates)
-}