chore: Clean up path vars (#348)

Author: gewenyu99

src/lib/middleware/config.ts

@@ -64,8 +64,11 @@ export function loadBenchmarkConfig(installDir: string): BenchmarkConfig {
     if (process.env.POSTHOG_WIZARD_BENCHMARK_FILE) {
       config.output.benchmarkPath = process.env.POSTHOG_WIZARD_BENCHMARK_FILE;
     }
-    if (process.env.POSTHOG_WIZARD_LOG_FILE) {
-      config.output.logPath = process.env.POSTHOG_WIZARD_LOG_FILE;
+    if (process.env.POSTHOG_WIZARD_LOG_DIR) {
+      config.output.logPath = path.join(
+        process.env.POSTHOG_WIZARD_LOG_DIR,
+        'posthog-wizard.log',
+      );
     }
 
     // If benchmark output is disabled, disable the jsonWriter plugin
@@ -83,8 +86,11 @@ export function loadBenchmarkConfig(installDir: string): BenchmarkConfig {
     if (process.env.POSTHOG_WIZARD_BENCHMARK_FILE) {
       config.output.benchmarkPath = process.env.POSTHOG_WIZARD_BENCHMARK_FILE;
     }
-    if (process.env.POSTHOG_WIZARD_LOG_FILE) {
-      config.output.logPath = process.env.POSTHOG_WIZARD_LOG_FILE;
+    if (process.env.POSTHOG_WIZARD_LOG_DIR) {
+      config.output.logPath = path.join(
+        process.env.POSTHOG_WIZARD_LOG_DIR,
+        'posthog-wizard.log',
+      );
     }
 
     return config;

src/run.ts

@@ -14,7 +14,7 @@ import { FRAMEWORK_REGISTRY } from './lib/registry';
 import { analytics } from './utils/analytics';
 import { runAgentWizard } from './lib/agent-runner';
 import { EventEmitter } from 'events';
-import { logToFile } from './utils/debug';
+import { logToFile, configureLogFileFromEnvironment } from './utils/debug';
 import { wizardAbort } from './utils/wizard-abort';
 import { readApiKeyFromEnv } from './utils/env-api-key';
 
@@ -38,6 +38,9 @@ type Args = {
 };
 
 export async function runWizard(argv: Args, session?: WizardSession) {
+  // Apply log file env overrides for all modes (CI, benchmark, and interactive).
+  configureLogFileFromEnvironment();
+
   const finalArgs = {
     ...argv,
     ...readEnvironment(),

src/utils/debug.ts

@@ -1,4 +1,5 @@
 import { appendFileSync } from 'fs';
+import path from 'path';
 import { prepareMessage } from './logging';
 import { getUI } from '../ui';
 
@@ -22,6 +23,18 @@ export function configureLogFile(opts: {
   if (opts.enabled !== undefined) logEnabled = opts.enabled;
 }
 
+/**
+ * Configure log path from environment variables.
+ *
+ * Uses POSTHOG_WIZARD_LOG_DIR when set, joined with posthog-wizard.log.
+ */
+export function configureLogFileFromEnvironment(): void {
+  const envLogDir = process.env.POSTHOG_WIZARD_LOG_DIR;
+  if (envLogDir) {
+    configureLogFile({ path: path.join(envLogDir, 'posthog-wizard.log') });
+  }
+}
+
 /**
  * Initialize the log file with a run header.
  * Call this at the start of each wizard run.