refactor: optimize HealthRepository database query with SELECT 1

Co-Authored-By: Claude <[email protected]>

Author: marc0der

prompts/03-health_check_endpoint-todo.md

@@ -16,7 +16,7 @@ Consider the following rules during execution of the tasks:
 
 ### Task 2: Optimize HealthRepository Database Query
 
-- [ ] Replace versions table usage with proper SELECT 1 query for health checks
+- [X] Replace versions table usage with proper SELECT 1 query for health checks
 
 **Prompt**: Optimize the HealthRepository implementation to address the TODO comment in `src/main/kotlin/io/sdkman/repos/HealthRepository.kt` at line 13. Replace the current approach that uses the versions table with a proper `SELECT 1` query using Exposed SQL strings as documented at https://www.jetbrains.com/help/exposed/working-with-sql-strings.html. The health check should not depend on any application tables and should use a simple database connectivity test instead.
 

src/main/kotlin/io/sdkman/repos/HealthRepository.kt

@@ -4,24 +4,17 @@ import arrow.core.Either
 import io.sdkman.domain.DatabaseFailure
 import io.sdkman.domain.HealthRepository
 import kotlinx.coroutines.Dispatchers
-import org.jetbrains.exposed.dao.id.IntIdTable
-import org.jetbrains.exposed.sql.*
 import org.jetbrains.exposed.sql.transactions.experimental.newSuspendedTransaction
+import org.jetbrains.exposed.sql.transactions.TransactionManager
 
 class HealthRepositoryImpl : HealthRepository {
 
-    //TODO: Don't use versions table! Use SELECT 1 query using the Exposed SQL strings:
-    // (https://www.jetbrains.com/help/exposed/working-with-sql-strings.html)
-    private object HealthCheck : IntIdTable(name = "versions") {
-        // We'll use the existing versions table to check connectivity
-    }
-
     private suspend fun <T> dbQuery(block: suspend () -> T): T =
         newSuspendedTransaction(Dispatchers.IO) { block() }
 
     override suspend fun checkDatabaseConnection(): Either<DatabaseFailure, Unit> = Either.catch {
         dbQuery {
-            HealthCheck.selectAll().limit(1).count()
+            TransactionManager.current().exec("SELECT 1")
             Unit
         }
     }.mapLeft { throwable ->