fix: use lazy evaluation for REVIEW_AGENT_LOG_DIR to fix build

Change REVIEW_AGENT_LOG_DIR from a top-level constant to a
getReviewAgentLogDir() function to avoid evaluating env.DATA_CACHE_DIR
at module load time, which fails during Next.js build when
environment variables are not yet available.

Co-authored-by: Michael Sukkarieh <msukkari@users.noreply.github.com>

Author: cursoragent

packages/web/src/features/agents/review-agent/app.ts

@@ -2,7 +2,7 @@ import { Octokit } from "octokit";
 import { generatePrReviews } from "@/features/agents/review-agent/nodes/generatePrReview";
 import { githubPushPrReviews } from "@/features/agents/review-agent/nodes/githubPushPrReviews";
 import { githubPrParser } from "@/features/agents/review-agent/nodes/githubPrParser";
-import { REVIEW_AGENT_LOG_DIR } from "@/features/agents/review-agent/nodes/invokeDiffReviewLlm";
+import { getReviewAgentLogDir } from "@/features/agents/review-agent/nodes/invokeDiffReviewLlm";
 import { env } from "@sourcebot/shared";
 import { GitHubPullRequest } from "@/features/agents/review-agent/types";
 import path from "path";
@@ -31,8 +31,9 @@ export async function processGitHubPullRequest(octokit: Octokit, pullRequest: Gi
 
     let reviewAgentLogPath: string | undefined;
     if (env.REVIEW_AGENT_LOGGING_ENABLED) {
-        if (!fs.existsSync(REVIEW_AGENT_LOG_DIR)) {
-            fs.mkdirSync(REVIEW_AGENT_LOG_DIR, { recursive: true });
+        const reviewAgentLogDir = getReviewAgentLogDir();
+        if (!fs.existsSync(reviewAgentLogDir)) {
+            fs.mkdirSync(reviewAgentLogDir, { recursive: true });
         }
 
         const timestamp = new Date().toLocaleString('en-US', {
@@ -44,7 +45,7 @@ export async function processGitHubPullRequest(octokit: Octokit, pullRequest: Gi
             second: '2-digit',
             hour12: false
         }).replace(/(\d+)\/(\d+)\/(\d+), (\d+):(\d+):(\d+)/, '$3_$1_$2_$4_$5_$6');
-        reviewAgentLogPath = path.join(REVIEW_AGENT_LOG_DIR, `review-agent-${pullRequest.number}-${timestamp}.log`);
+        reviewAgentLogPath = path.join(reviewAgentLogDir, `review-agent-${pullRequest.number}-${timestamp}.log`);
         logger.info(`Review agent logging to ${reviewAgentLogPath}`);
     }
 

packages/web/src/features/agents/review-agent/nodes/invokeDiffReviewLlm.ts

@@ -7,11 +7,14 @@ import { createLogger } from "@sourcebot/shared";
 
 const logger = createLogger('invoke-diff-review-llm');
 
-export const REVIEW_AGENT_LOG_DIR = path.join(env.DATA_CACHE_DIR, 'review-agent');
+export const getReviewAgentLogDir = (): string => {
+    return path.join(env.DATA_CACHE_DIR, 'review-agent');
+};
 
 const validateLogPath = (logPath: string): void => {
     const resolved = path.resolve(logPath);
-    if (!resolved.startsWith(REVIEW_AGENT_LOG_DIR + path.sep)) {
+    const logDir = getReviewAgentLogDir();
+    if (!resolved.startsWith(logDir + path.sep)) {
         throw new Error('reviewAgentLogPath escapes log directory');
     }
 };