feat(db): no auto migration)

Author: shamikkarkhanis

Makefile

@@ -114,20 +114,6 @@ benchmark:
 	echo "Running benchmarks and saving to $$log_file..."; \
 	go test -bench=. -benchmem -run=^$$ ./tests/benchmarks/... | tee $$log_file
 
-# Database migrations (requires DATABASE_URL)
-migrate-up:
-	migrate -path $(MIGRATIONS_PATH) -database "$(DATABASE_URL)" up
-
-migrate-down:
-	migrate -path $(MIGRATIONS_PATH) -database "$(DATABASE_URL)" down 1
-
-migrate-version:
-	migrate -path $(MIGRATIONS_PATH) -database "$(DATABASE_URL)" version
-
-migrate-create:
-	@if [ -z "$(name)" ]; then echo "usage: make migrate-create name=add_users_index"; exit 1; fi
-	migrate create -ext sql -dir $(MIGRATIONS_PATH) -seq $(name)
-
 # Linting
 lint:
 	golangci-lint run ./...

cmd/server/main.go

@@ -52,13 +52,6 @@ func main() {
 
 	ctx := context.Background()
 
-	// Apply database migrations before serving traffic.
-	if err := database.RunMigrations(ctx, cfg.Database.URL, cfg.Database.MigrationsPath); err != nil {
-		slog.Error("failed to run database migrations", "error", err, "path", cfg.Database.MigrationsPath)
-		os.Exit(1)
-	}
-	slog.Info("database migrations applied", "path", cfg.Database.MigrationsPath)
-
 	// Connect to database
 	pool, err := database.NewPool(ctx, cfg.Database.URL)
 	if err != nil {

internal/database/migrations_integration_test.go

@@ -18,17 +18,20 @@ import (
 )
 
 func TestMigrationsApplyAndRollback(t *testing.T) {
-	pool := testutils.SetupEmptyTestDB(t)
-	defer pool.Close()
-
-	ctx := context.Background()
 	migrationsDir := repoRoot(t, 2)
 	migrationsDir = filepath.Join(migrationsDir, "migrations")
 
 	upFiles, downFiles := migrationFiles(t, migrationsDir)
-	require.NotEmpty(t, upFiles, "no migration files found in %s", migrationsDir)
+	if len(upFiles) == 0 {
+		t.Skipf("no migration files found in %s", migrationsDir)
+	}
 	require.Equal(t, len(upFiles), len(downFiles), "up/down migration file count mismatch")
 
+	pool := testutils.SetupEmptyTestDB(t)
+	defer pool.Close()
+
+	ctx := context.Background()
+
 	beforeState := snapshotPublicSchema(t, ctx, pool)
 
 	for i, path := range upFiles {

migrations/.gitkeep

@@ -0,0 +1 @@
+