fix typing

Author: icecrasher321

apps/sim/lib/logs/execution/logger.ts

@@ -230,6 +230,7 @@ export class ExecutionLogger implements IExecutionLoggerService {
     traceSpans?: TraceSpan[]
     workflowInput?: any
     isResume?: boolean // If true, merge with existing data instead of replacing
+    level?: 'info' | 'error' // Optional override for log level (used in cost-only fallback)
   }): Promise<WorkflowExecutionLog> {
     const {
       executionId,
@@ -240,6 +241,7 @@ export class ExecutionLogger implements IExecutionLoggerService {
       traceSpans,
       workflowInput,
       isResume,
+      level: levelOverride,
     } = params
 
     logger.debug(`Completing workflow execution ${executionId}`, { isResume })
@@ -256,6 +258,7 @@ export class ExecutionLogger implements IExecutionLoggerService {
     }
 
     // Determine if workflow failed by checking trace spans for errors
+    // Use the override if provided (for cost-only fallback scenarios)
     const hasErrors = traceSpans?.some((span: any) => {
       const checkSpanForErrors = (s: any): boolean => {
         if (s.status === 'error') return true
@@ -267,7 +270,7 @@ export class ExecutionLogger implements IExecutionLoggerService {
       return checkSpanForErrors(span)
     })
 
-    const level = hasErrors ? 'error' : 'info'
+    const level = levelOverride ?? (hasErrors ? 'error' : 'info')
 
     // Extract files from trace spans, final output, and workflow input
     const executionFiles = this.extractFilesFromExecution(traceSpans, finalOutput, workflowInput)

apps/sim/lib/logs/execution/logging-session.ts

@@ -29,7 +29,7 @@ export interface SessionCompleteParams {
   endedAt?: string
   totalDurationMs?: number
   finalOutput?: any
-  traceSpans?: any[]
+  traceSpans?: TraceSpan[]
   workflowInput?: any
 }
 
@@ -336,6 +336,7 @@ export class LoggingSession {
         endedAt: params.endedAt,
         totalDurationMs: params.totalDurationMs,
         errorMessage: 'Trace spans too large to store',
+        isError: false,
       })
     }
   }
@@ -349,6 +350,7 @@ export class LoggingSession {
         endedAt: params?.endedAt,
         totalDurationMs: params?.totalDurationMs,
         errorMessage: params?.error?.message || 'Execution failed, trace spans too large to store',
+        isError: true,
       })
     }
   }
@@ -358,6 +360,7 @@ export class LoggingSession {
     endedAt?: string
     totalDurationMs?: number
     errorMessage: string
+    isError: boolean
   }): Promise<void> {
     logger.warn(
       `[${this.requestId || 'unknown'}] Logging completion failed for execution ${this.executionId} - attempting cost-only fallback`
@@ -386,6 +389,7 @@ export class LoggingSession {
         finalOutput: { _fallback: true, error: params.errorMessage },
         traceSpans: [],
         isResume: this.isResume,
+        level: params.isError ? 'error' : 'info',
       })
 
       logger.info(