fix: make toolLogs optional

Author: amishne

report-app/src/app/pages/report-viewer/report-viewer.html

@@ -411,7 +411,7 @@ <h4>Debugging Tools</h4>
                   (click)="downloadDebuggingZip(result)">
                   Download ZIP for debugging
                 </button>
-                @if (result.toolLogs.length > 0) {
+                @if (result.toolLogs && result.toolLogs.length > 0) {
                   <expansion-panel>
                     <expansion-panel-header>Tool Logs</expansion-panel-header>
                     <ul class="tool-logs-list">

runner/codegen/llm-runner.ts

@@ -148,7 +148,7 @@ interface BaseLlmGenerateResponse {
   /** Reasoning messages from the LLM. */
   reasoning: string;
   /** Tool requests and responses. */
-  toolLogs: ToolLogEntry[];
+  toolLogs?: ToolLogEntry[];
 }
 
 /** File generation response from the LLM. */

runner/orchestration/generate.ts

@@ -350,7 +350,7 @@ async function startEvaluationTask(
       progress
     );
 
-    const toolLogs = initialResponse.toolLogs;
+    const toolLogs = initialResponse.toolLogs ?? [];
 
     if (!initialResponse) {
       progress.log(
@@ -688,7 +688,7 @@ async function installChrome(): Promise<void> {
 
   try {
     await chromeInstallPromise;
-  } catch {}
+  } catch {} // Ignore errors here, as it might be already installed.
 }
 
 /**

runner/shared-interfaces.ts

@@ -106,7 +106,7 @@ export interface LlmResponse {
   /** Reasoning messages from the LLM for generating this response. */
   reasoning: string;
   /** Tool requests logs (e.g. MCP requests and responses). */
-  toolLogs: ToolLogEntry[];
+  toolLogs?: ToolLogEntry[];
 }
 
 /** Error response from an LLM API. */
@@ -409,7 +409,7 @@ export interface AssessmentResult {
   /** The number of repair attempts made after the axe initial failures. */
   axeRepairAttempts: number;
   /** Tool requests logs (e.g. MCP requests and responses). */
-  toolLogs: ToolLogEntry[];
+  toolLogs?: ToolLogEntry[];
 }
 
 /**