fix(preproc-errors): should not charge base execution cost in this case

Author: icecrasher321

apps/sim/lib/execution/preprocessing.ts

@@ -541,6 +541,7 @@ async function logPreprocessingError(params: {
         stackTrace: undefined,
       },
       traceSpans: [],
+      skipCost: true, // Preprocessing errors should not charge - no execution occurred
     })
 
     logger.debug(`[${requestId}] Logged preprocessing error to database`, {

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

@@ -45,6 +45,7 @@ export interface SessionErrorCompleteParams {
     stackTrace?: string
   }
   traceSpans?: TraceSpan[]
+  skipCost?: boolean
 }
 
 export interface SessionCancelledParams {
@@ -342,27 +343,42 @@ export class LoggingSession {
     }
 
     try {
-      const { endedAt, totalDurationMs, error, traceSpans } = params
+      const { endedAt, totalDurationMs, error, traceSpans, skipCost } = params
 
       const endTime = endedAt ? new Date(endedAt) : new Date()
       const durationMs = typeof totalDurationMs === 'number' ? totalDurationMs : 0
       const startTime = new Date(endTime.getTime() - Math.max(1, durationMs))
 
       const hasProvidedSpans = Array.isArray(traceSpans) && traceSpans.length > 0
 
-      const costSummary = hasProvidedSpans
-        ? calculateCostSummary(traceSpans)
-        : {
-            totalCost: BASE_EXECUTION_CHARGE,
+      // When skipCost is true (preprocessing errors), don't charge anything
+      // When we have trace spans, calculate actual cost from them
+      // Otherwise, charge the base execution fee for errors that occurred during execution
+      const costSummary = skipCost
+        ? {
+            totalCost: 0,
             totalInputCost: 0,
             totalOutputCost: 0,
             totalTokens: 0,
             totalPromptTokens: 0,
             totalCompletionTokens: 0,
-            baseExecutionCharge: BASE_EXECUTION_CHARGE,
+            baseExecutionCharge: 0,
             modelCost: 0,
             models: {},
           }
+        : hasProvidedSpans
+          ? calculateCostSummary(traceSpans)
+          : {
+              totalCost: BASE_EXECUTION_CHARGE,
+              totalInputCost: 0,
+              totalOutputCost: 0,
+              totalTokens: 0,
+              totalPromptTokens: 0,
+              totalCompletionTokens: 0,
+              baseExecutionCharge: BASE_EXECUTION_CHARGE,
+              modelCost: 0,
+              models: {},
+            }
 
       const message = error?.message || 'Execution failed before starting blocks'