fix(internal): skip advisory lock integration tests in CI without database

Author: brandonkachen

packages/internal/src/db/__tests__/advisory-lock.integration.test.ts

@@ -54,8 +54,14 @@ const DEFAULT_TEST_DATABASE_URL =
   'postgresql://postgres:postgres@127.0.0.1:5432/testdb'
 const TEST_DATABASE_URL = process.env.DATABASE_URL || DEFAULT_TEST_DATABASE_URL
 
-// Skip tests if DATABASE_URL is not configured (e.g., in unit test runs)
-const SKIP_INTEGRATION_TESTS = !process.env.DATABASE_URL && !process.env.RUN_INTEGRATION_TESTS
+// Skip tests if:
+// 1. Running in GitHub Actions CI (which doesn't have a real Postgres for packages/internal)
+// 2. DATABASE_URL is not configured locally and RUN_INTEGRATION_TESTS is not set
+// Note: CI injects DATABASE_URL as a secret, but there's no actual database running
+// for packages/internal integration tests (only packages/billing has Postgres in CI)
+const SKIP_INTEGRATION_TESTS = 
+  process.env.CODEBUFF_GITHUB_ACTIONS === 'true' ||
+  (!process.env.DATABASE_URL && !process.env.RUN_INTEGRATION_TESTS)
 
 // Create test database connection
 let testClient: ReturnType<typeof postgres> | null = null