Restructure to standard Go project layout

- Move main.go → cmd/bot/main.go (entry point per Go conventions)
- Update Makefile, Dockerfile, CI workflow to build from ./cmd/bot/
- Remove stale root binaries (arbitrage_bot, bot, bot-linux)
- Delete orphaned configs/ directory (unreferenced)
- Add /logs.bak/ to .gitignore
- Remove dead methods from internal/market/manager.go
- Fix README commands (build-all-platforms, make cover)

Author: MSC72m

.github/workflows/ci.yml

@@ -58,7 +58,7 @@ jobs:
               -X triangular-arbitrage-bot/internal/version.Version=${GITHUB_REF_NAME} \
               -X triangular-arbitrage-bot/internal/version.Commit=${{ steps.vars.outputs.sha_short }} \
               -X triangular-arbitrage-bot/internal/version.BuildTime=${{ steps.vars.outputs.build_time }}" \
-            ./
+            ./cmd/bot/
 
   docker:
     if: github.ref == 'refs/heads/main'

.gitignore

@@ -28,6 +28,7 @@ coverage.html
 # Logs
 *.log
 /logs/
+/logs.bak/
 
 # Generated
 CLAUDE.md

Dockerfile

@@ -13,7 +13,7 @@ ARG VERSION=dev
 ARG COMMIT=unknown
 ARG BUILD_TIME=unknown
 
-RUN CGO_ENABLED=0 go build -trimpath -ldflags="-s -w -X triangular-arbitrage-bot/internal/version.Version=${VERSION} -X triangular-arbitrage-bot/internal/version.Commit=${COMMIT} -X triangular-arbitrage-bot/internal/version.BuildTime=${BUILD_TIME}" -o /build/bot .
+RUN CGO_ENABLED=0 go build -trimpath -ldflags="-s -w -X triangular-arbitrage-bot/internal/version.Version=${VERSION} -X triangular-arbitrage-bot/internal/version.Commit=${COMMIT} -X triangular-arbitrage-bot/internal/version.BuildTime=${BUILD_TIME}" -o /build/bot ./cmd/bot/
 
 FROM alpine:3.21
 

Makefile

@@ -19,7 +19,7 @@ all: fmt vet lint test build
 
 build:
 	@mkdir -p $(BUILD_DIR)
-	go build $(GO_FLAGS) $(LD_FLAGS) -o $(BUILD_DIR)/$(BINARY) .
+	go build $(GO_FLAGS) $(LD_FLAGS) -o $(BUILD_DIR)/$(BINARY) ./cmd/bot/
 
 run: build
 	./$(BUILD_DIR)/$(BINARY) -config config.json
@@ -76,11 +76,11 @@ clean:
 # Cross-compilation
 build-linux:
 	@mkdir -p $(BUILD_DIR)
-	GOOS=linux GOARCH=amd64 go build $(GO_FLAGS) $(LD_FLAGS) -o $(BUILD_DIR)/$(BINARY)-linux-amd64 .
+	GOOS=linux GOARCH=amd64 go build $(GO_FLAGS) $(LD_FLAGS) -o $(BUILD_DIR)/$(BINARY)-linux-amd64 ./cmd/bot/
 
 build-darwin:
 	@mkdir -p $(BUILD_DIR)
-	GOOS=darwin GOARCH=arm64 go build $(GO_FLAGS) $(LD_FLAGS) -o $(BUILD_DIR)/$(BINARY)-darwin-arm64 .
+	GOOS=darwin GOARCH=arm64 go build $(GO_FLAGS) $(LD_FLAGS) -o $(BUILD_DIR)/$(BINARY)-darwin-arm64 ./cmd/bot/
 
 build-all-platforms: build-linux build-darwin
 	@echo "Cross-platform builds complete"

README.md

@@ -81,14 +81,14 @@ make run
 ### Cross-platform builds
 
 ```bash
-make build-all   # builds for linux/amd64, windows/amd64, darwin/amd64, darwin/arm64
+make build-all-platforms   # builds for linux/amd64 and darwin/arm64
 ```
 
 ## Testing
 
 ```bash
-make test                    # run all tests
-make test-coverage           # run with coverage report
+make test                    # run all tests with race detector
+make cover                   # run with coverage report (HTML)
 go test -race ./...          # run with race detector
 ```
 

main.go cmd/bot/main.gomain.go renamed to cmd/bot/main.go

@@ -3,8 +3,6 @@ package market
 import (
 	"encoding/json"
 	"fmt"
-	"strconv"
-	"strings"
 	"time"
 
 	"triangular-arbitrage-bot/internal/config"
@@ -155,46 +153,3 @@ func (m *Manager) GetOrderBookREST(market string) (*models.OrderBook, error) {
 
 	return OrderBook, nil
 }
-
-// GetBestPrice returns the best price for a given market and side
-// For buys, we check bids[0] (highest bid)
-// For sells, we check asks[0] (lowest ask)
-func (m *Manager) GetBestPrice(market string, side string) (float64, error) {
-	OrderBook, exists := m.marketDepths.Load(market)
-	if !exists {
-		return 0, fmt.Errorf("no order book data for market: %s", market)
-	}
-
-	switch strings.ToLower(side) {
-	case "buy":
-		// For buys, we want the highest bid (bids[0])
-		if len(OrderBook.Bids) > 0 {
-			price, err := strconv.ParseFloat(OrderBook.Bids[0].Price, 64)
-			if err != nil {
-				return 0, fmt.Errorf("failed to parse bid price: %w", err)
-			}
-			return price, nil
-		}
-	case "sell":
-		// For sells, we want the lowest ask (asks[0])
-		if len(OrderBook.Asks) > 0 {
-			price, err := strconv.ParseFloat(OrderBook.Asks[0].Price, 64)
-			if err != nil {
-				return 0, fmt.Errorf("failed to parse ask price: %w", err)
-			}
-			return price, nil
-		}
-	}
-
-	return 0, fmt.Errorf("no %s orders available for market: %s", side, market)
-}
-
-// GetMarketDepth returns the order book depth for a market
-func (m *Manager) GetMarketDepth(market string) (*models.OrderBook, bool) {
-	return m.marketDepths.Load(market)
-}
-
-// GetAvailableMarkets returns all markets with available order book data
-func (m *Manager) GetAvailableMarkets() []string {
-	return m.marketDepths.GetAvailableMarkets()
-}