fix: address PR review comments and CI failures

- Update Go version from 1.24.4 to 1.23.4 in go.mod and CI
- Fix database credentials mismatch in integration tests (postgres -> testuser)
- Update HighPriority factory method to return priority 10 instead of 9
- Update related test assertions to expect priority 10
- Ensure all unit tests pass

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: starbops

.github/workflows/ci.yml

@@ -7,7 +7,7 @@ on:
     branches: [ main, develop ]
 
 env:
-  GO_VERSION: '1.24.4'
+  GO_VERSION: '1.23.4'
 
 # CI Optimization Strategy:
 # - All jobs use consistent caching with Go version in cache key

go.mod

@@ -1,6 +1,6 @@
 module github.com/voidrunnerhq/voidrunner
 
-go 1.24.4
+go 1.23.4
 
 require (
 	github.com/gin-contrib/cors v1.7.6

internal/api/integration/contract_test.go

@@ -46,8 +46,8 @@ func (s *ContractTestSuite) SetupSuite() {
 			Host:     "localhost",
 			Port:     "5432",
 			Database: "voidrunner_test",
-			User:     "postgres",
-			Password: "password",
+			User:     "testuser",
+			Password: "testpassword",
 			SSLMode:  "disable",
 		},
 		JWT: config.JWTConfig{

internal/api/integration_test.go

@@ -46,8 +46,8 @@ func (s *IntegrationTestSuite) SetupSuite() {
 			Host:     "localhost",
 			Port:     "5432",
 			Database: "voidrunner_test",
-			User:     "postgres",
-			Password: "password",
+			User:     "testuser",
+			Password: "testpassword",
 			SSLMode:  "disable",
 		},
 		JWT: config.JWTConfig{

internal/database/integration_test.go

@@ -40,8 +40,8 @@ func setupTestDatabase(t *testing.T) (*Connection, *Repositories) {
 	cfg := &config.DatabaseConfig{
 		Host:     getEnvOrDefault("TEST_DB_HOST", "localhost"),
 		Port:     getEnvOrDefault("TEST_DB_PORT", "5432"),
-		User:     getEnvOrDefault("TEST_DB_USER", "postgres"),
-		Password: getEnvOrDefault("TEST_DB_PASSWORD", ""),
+		User:     getEnvOrDefault("TEST_DB_USER", "testuser"),
+		Password: getEnvOrDefault("TEST_DB_PASSWORD", "testpassword"),
 		Database: getEnvOrDefault("TEST_DB_NAME", "voidrunner_test"),
 		SSLMode:  getEnvOrDefault("TEST_DB_SSL_MODE", "disable"),
 	}
@@ -415,8 +415,8 @@ func setupBenchmarkDatabase(b *testing.B) (*Connection, *Repositories) {
 	cfg := &config.DatabaseConfig{
 		Host:     getEnvOrDefault("TEST_DB_HOST", "localhost"),
 		Port:     getEnvOrDefault("TEST_DB_PORT", "5432"),
-		User:     getEnvOrDefault("TEST_DB_USER", "postgres"),
-		Password: getEnvOrDefault("TEST_DB_PASSWORD", ""),
+		User:     getEnvOrDefault("TEST_DB_USER", "testuser"),
+		Password: getEnvOrDefault("TEST_DB_PASSWORD", "testpassword"),
 		Database: getEnvOrDefault("TEST_DB_NAME", "voidrunner_test"),
 		SSLMode:  getEnvOrDefault("TEST_DB_SSL_MODE", "disable"),
 	}

internal/testutil/error_scenarios_test.go

@@ -282,7 +282,7 @@ func TestFactoryBuilders_ChainedCalls(t *testing.T) {
 		assert.Equal(t, "Test Description", *task.Description)
 		assert.Equal(t, models.ScriptTypePython, task.ScriptType)
 		assert.Equal(t, models.TaskStatusPending, task.Status)
-		assert.Equal(t, 9, task.Priority) // HighPriority overrides previous priority
+		assert.Equal(t, 10, task.Priority) // HighPriority overrides previous priority
 	})
 
 	t.Run("execution factory method chaining", func(t *testing.T) {
@@ -401,7 +401,7 @@ func TestFactoryPriorityMethods(t *testing.T) {
 
 		// Test HighPriority
 		highPriorityTask := factory.HighPriority().Build()
-		assert.Equal(t, 9, highPriorityTask.Priority)
+		assert.Equal(t, 10, highPriorityTask.Priority)
 
 		// Test LowPriority
 		factory = NewTaskFactory(userID)

internal/testutil/example_test.go

@@ -171,7 +171,7 @@ func TestFactories(t *testing.T) {
 	assert.Equal(t, "Test Task", task.Name)
 	assert.Equal(t, models.ScriptTypePython, task.ScriptType)
 	assert.Equal(t, models.TaskStatusCompleted, task.Status)
-	assert.Equal(t, 9, task.Priority) // High priority
+	assert.Equal(t, 10, task.Priority) // High priority
 	assert.Equal(t, 60, task.TimeoutSeconds)
 	assert.Equal(t, "test", task.Metadata["environment"])
 

internal/testutil/factories.go

@@ -193,7 +193,7 @@ func (f *TaskFactory) WithPriority(priority int) *TaskFactory {
 
 // HighPriority sets high priority (8-10)
 func (f *TaskFactory) HighPriority() *TaskFactory {
-	return f.WithPriority(9)
+	return f.WithPriority(10)
 }
 
 // LowPriority sets low priority (1-3)