.PHONY: help test test-integration test-all rebuild rebuild-force rebuild-app rebuild-app-force rebuild-force-db clean restart up down logs ps test-env-up test-env-down verify-guidelines verify-quick

# Include .env file for environment variables (single source of truth)
# Ignore if .env doesn't exist yet
ifneq (,$(wildcard ./.env))
    include .env
    export
endif

# Default target
help:
	@echo "Available targets:"
	@echo ""
	@echo "Development:"
	@echo "  make up               - Start containers (if already built)"
	@echo "  make rebuild          - Rebuild all containers (database preserved)"
	@echo "  make rebuild-force    - Rebuild all containers, no cache (database preserved)"
	@echo "  make rebuild-app      - Rebuild app container only (fast, database preserved)"
	@echo "  make rebuild-app-force - Rebuild app container only, no cache (database preserved)"
	@echo "  make rebuild-force-db - Rebuild all containers, no cache (DELETES database)"
	@echo "  make restart          - Restart app container (preserves db)"
	@echo "  make down             - Stop all containers (preserves volumes)"
	@echo "  make clean            - Stop and remove all containers and volumes"
	@echo ""
	@echo "Status & Logs:"
	@echo "  make ps           - Show container status"
	@echo "  make logs         - Show container logs (follow mode)"
	@echo ""
	@echo "Testing:"
	@echo "  make test              - Run unit tests (from host, fast)"
	@echo "  make test-integration  - Run integration tests (in containers, matches prod)"
	@echo "  make test-all          - Run all tests (unit + integration)"
	@echo "  make test-env-up       - Start containers with test mode enabled"
	@echo "  make test-env-down     - Stop test environment"
	@echo ""
	@echo "Verification:"
	@echo "  make verify-guidelines - Run comprehensive guidelines check"
	@echo "  make verify-quick      - Run quick guidelines check"

# Run unit tests locally (fast, no containers)
test:
	go test ./internal/... -v -short

# Run integration tests in containers (matches production environment)
test-integration:
	@echo "Building test containers..."
	podman compose --profile tests build
	@echo "Starting application containers..."
	podman compose up -d db app
	@echo "Waiting for services to be healthy..."
	@until podman exec bookhoard_db pg_isready -U postgres > /dev/null 2>&1; do \
		echo "  Database not ready yet..."; \
		sleep 2; \
	done; \
	echo "  ✓ Database is ready"
	@until podman exec bookhoard curl -sf http://localhost:8765/health > /dev/null 2>&1; do \
		echo "  Application not ready yet..."; \
		sleep 2; \
	done; \
	echo "  ✓ Application is ready"
	@echo ""
	@echo "Running integration tests in container..."
	podman compose --profile tests run --rm tests
	@echo ""
	@echo "✅ Integration tests completed!"
	@echo "📝 Containers are still running. Use 'make logs' to view logs or 'make clean' to stop."

# Run all tests
test-all: test test-integration

# Rebuild app container only (preserve DB, with cache)
rebuild-app:
	@echo "Rebuilding app container (database stays running)..."
	podman compose up --build --force-recreate -d app
	@echo "✓ App container rebuilt and restarted"

# Rebuild app container only (preserve DB, no cache)
rebuild-app-force:
	@echo "Force rebuilding app container (database stays running, no cache)..."
	podman compose build --no-cache app
	podman compose up --force-recreate -d app
	@echo "✓ App container rebuilt and restarted"

# Rebuild all containers (preserve DB, with cache)
rebuild:
	@echo "Rebuilding all containers (database preserved)..."
	podman compose up --build --force-recreate -d
	@echo "✓ All containers rebuilt and restarted"

# Rebuild all containers (preserve DB, no cache)
rebuild-force:
	@echo "Force rebuilding all containers (database preserved, no cache)..."
	podman compose build --no-cache
	podman compose up --force-recreate -d
	@echo "✓ All containers rebuilt and restarted"

# Rebuild all containers (remove DB, no cache)
rebuild-force-db:
	@echo "Force rebuilding all containers (database will be DELETED, no cache)..."
	podman compose down -v
	podman compose build --no-cache
	podman compose up --force-recreate -d
	@echo "✓ All containers rebuilt and restarted"

# Stop and remove containers
clean:
	podman compose down -v

# Quick start (if already built)
up:
	podman compose up -d

# Stop all containers (alias for clean)
down:
	podman compose down

# Restart app container (preserves database)
restart:
	@echo "Restarting app container (database stays running)..."
	podman compose restart app
	@echo "✓ App container restarted"

# Show container status
ps:
	podman compose ps

# Show container logs
logs:
	podman compose logs -f

# Start containers with test mode enabled for manual testing
test-env-up:
	@echo "Starting containers with test mode enabled..."
	TEST_MODE=true RATE_LIMIT_ENABLED=false REQUESTS_PER_MINUTE=1000 podman compose up --build --force-recreate -d
	@echo "Waiting for services to be ready..."
	@until podman exec bookhoard_db pg_isready -U postgres > /dev/null 2>&1; do sleep 1; done
	@until podman exec bookhoard curl -sf http://localhost:8765/health > /dev/null 2>&1; do sleep 1; done
	@echo "✓ Test environment is ready!"
	@echo "Application available at http://localhost:8765"

# Stop test environment
test-env-down:
	podman compose down -v

# Verify project guidelines compliance
verify-guidelines:
	@echo "Running comprehensive project guidelines verification..."
	@./scripts/verify-guidelines.sh

verify-quick:
	@echo "Running quick project guidelines verification..."
	@./scripts/verify-quick.sh
