fix(db): repair merged duplicate in waitForDbReadiness

Two implementations had been spliced together: `if (result.healthy) {` was
opened and never closed, followed by a second check that assigned the result
object to `healthy` and tested it for truthiness — which is always true, since
checkDbHealth returns HealthCheckResult rather than a boolean.

Kept the single correct check. src/db/health.test.ts asserts exactly two pool
queries for "resolves after the database becomes reachable", which only holds
with one health call per attempt. The `dbModule` self-import existed solely for
the duplicated call and is now unused, so it goes too.

This file compiles again; contact.ts, migration.test.ts and transactions.ts
remain broken and are tracked as Stage 3 work.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: Jagadeeshftw

src/db/index.ts

@@ -2,7 +2,6 @@ import { drizzle } from "drizzle-orm/node-postgres";
 import { Pool } from "pg";
 import { env } from "../config.js";
 import * as schema from "./schema.js";
-import * as dbModule from "./index.js";
 
 // Pool tuning shared across whichever connection string we end up using.
 // Bounded size plus idle/connection timeouts keep a stuck DB from exhausting the pool.
@@ -146,8 +145,6 @@ export async function waitForDbReadiness(): Promise<void> {
   for (let attempt = 1; attempt <= maxAttempts; attempt++) {
     const result = await checkDbHealth();
     if (result.healthy) {
-    const healthy = await dbModule.checkDbHealth();
-    if (healthy) {
       return;
     }
     if (attempt < maxAttempts) {