feat: update Dockerfile and application to support database URL as build argument

- Added support for passing DATABASE_URL as a build argument in the Dockerfile.
- Updated the image build process to include database dependency and build-time arguments.
- Refactored TodoApp component to remove direct database URL prop, simplifying its interface.
- Enhanced API response types for better type safety and clarity in handling responses.

Author: claudfuen

apps/infra/index.ts

@@ -1,7 +1,7 @@
 import * as aws from "@pulumi/aws";
 import * as awsx from "@pulumi/awsx";
-import * as pulumi from "@pulumi/pulumi";
 import * as command from "@pulumi/command";
+import * as pulumi from "@pulumi/pulumi";
 import * as random from "@pulumi/random";
 
 // ==========================================
@@ -317,11 +317,16 @@ const repo = new awsx.ecr.Repository("pathfinder-repo", {
   },
 });
 
-// Always build Docker image locally and push to ECR
+// Build Docker image with database dependency and build-time args
 const imageUri = new awsx.ecr.Image("pathfinder-image", {
   repositoryUrl: repo.url,
   context: "../web",
   platform: "linux/amd64", // Required for AWS Fargate
+  args: {
+    DATABASE_URL: pulumi.interpolate`postgresql://${db.username}:${db.password}@${db.endpoint}/${db.dbName}`,
+  },
+}, {
+  dependsOn: [db] // Ensure database is fully created before building image
 }).imageUri;
 
 // Fargate Service with init container for migrations

apps/web/Dockerfile

@@ -10,8 +10,9 @@ RUN bun install --frozen-lockfile
 FROM oven/bun:1.2.18-alpine AS builder
 WORKDIR /app
 
-# Skip environment validation during build (DATABASE_URL is runtime-only)
-ENV SKIP_ENV_VALIDATION=true
+# Accept DATABASE_URL as build argument
+ARG DATABASE_URL
+ENV DATABASE_URL=$DATABASE_URL
 
 # Copy dependencies from previous stage
 COPY --from=deps /app/node_modules ./node_modules

apps/web/package.json

@@ -27,6 +27,7 @@
     "pg": "^8.16.3",
     "react": "^19.0.0",
     "react-dom": "^19.0.0",
+    "tsx": "^4.20.3",
     "zod": "^3.25.76"
   },
   "devDependencies": {

apps/web/src/app/components/TodoApp.tsx

@@ -5,11 +5,10 @@ import type { Todo } from "@/types/api";
 import { useState } from "react";
 
 type TodoAppProps = {
-  databaseUrl: string;
   initialTodos: Todo[];
 };
 
-export default function TodoApp({ databaseUrl, initialTodos }: TodoAppProps) {
+export default function TodoApp({ initialTodos }: TodoAppProps) {
   const [todos, setTodos] = useState<Todo[]>(initialTodos);
   const [newTodo, setNewTodo] = useState("");
   const [loading, setLoading] = useState(false);

apps/web/src/app/page.tsx

@@ -10,12 +10,11 @@ export default async function Home() {
     .from(todos)
     .orderBy(desc(todos.createdAt));
 
+  console.log("initialTodos", initialTodos);
+
   return (
     <div>
-      <TodoApp 
-        databaseUrl={env.DATABASE_URL} 
-        initialTodos={initialTodos}
-      />
+      <TodoApp initialTodos={initialTodos} />
       <div
         style={{
           position: "fixed",

apps/web/src/db/index.ts

@@ -6,7 +6,7 @@ import * as schema from "./schema";
 // Create a connection pool using validated environment variables
 const pool = new Pool({
   connectionString: env.DATABASE_URL,
-  ssl: env.NODE_ENV === "production" ? { rejectUnauthorized: false } : false,
+  // ssl: env.NODE_ENV === "production" ? { rejectUnauthorized: false } : false,
 });
 
 // Create the database instance

apps/web/src/env.ts

@@ -13,8 +13,9 @@ export const env = createEnv({
     // NEXT_PUBLIC_*: z.string(),
   },
   // For Next.js >= 13.4.4, you only need to destructure client variables:
-  experimental__runtimeEnv: {
-    // No client env vars currently
+  runtimeEnv: {
+    DATABASE_URL: process.env.DATABASE_URL,
+    NODE_ENV: process.env.NODE_ENV,
   },
   // Allow skipping validation for development scenarios
   skipValidation: !!process.env.SKIP_ENV_VALIDATION,

apps/web/src/types/api.ts

@@ -5,15 +5,21 @@ import type { InferSelectModel } from "drizzle-orm";
 export type Todo = InferSelectModel<typeof todos>;
 
 // API Response types
-export interface ApiResponse<T = any> {
+export interface ApiResponse<T = unknown> {
   success: boolean;
   data?: T;
   error?: string;
   message?: string;
 }
 
-export interface TodosResponse extends ApiResponse<Todo[]> {}
-export interface TodoResponse extends ApiResponse<Todo> {}
-export interface DeleteResponse extends ApiResponse {
+export interface TodosResponse extends ApiResponse<Todo[]> {
+  data: Todo[];
+}
+
+export interface TodoResponse extends ApiResponse<Todo> {
+  data: Todo;
+}
+
+export interface DeleteResponse extends ApiResponse<never> {
   message: string;
 } 

apps/web/test-db.js

@@ -0,0 +1,24 @@
+const { Pool } = require('pg');
+
+async function testConnection() {
+  const pool = new Pool({
+    connectionString: process.env.DATABASE_URL,
+  });
+  
+  try {
+    const client = await pool.connect();
+    console.log('✅ Database connection successful');
+    
+    const result = await client.query('SELECT * FROM todos LIMIT 1');
+    console.log('✅ Query successful:', result.rowCount, 'rows');
+    
+    client.release();
+    await pool.end();
+  } catch (error) {
+    console.error('❌ Database error:', error.message);
+    console.error('Connection string:', process.env.DATABASE_URL);
+    process.exit(1);
+  }
+}
+
+testConnection();