improvement(processing): fix type safety in execution error logging

Replace `as any` cast in non-SSE error path with proper `buildTraceSpans()`
transformation, matching the SSE error path. Remove redundant `as any` cast
in preprocessing.ts where the types already align.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: waleedlatif1

apps/sim/app/api/workflows/[id]/execute/route.ts

@@ -702,10 +702,14 @@ export async function POST(req: NextRequest, { params }: { params: Promise<{ id:
         // fire-and-forget call. The `completed` guard in LoggingSession prevents
         // double-writes, so this is a no-op — but we avoid awaiting it to reduce
         // error-response latency.
+        const { traceSpans, totalDuration } = executionResult
+          ? buildTraceSpans(executionResult)
+          : { traceSpans: [], totalDuration: 0 }
+
         void loggingSession.safeCompleteWithError({
-          totalDurationMs: executionResult?.metadata?.duration,
+          totalDurationMs: totalDuration || executionResult?.metadata?.duration,
           error: { message: errorMessage },
-          traceSpans: executionResult?.logs as any,
+          traceSpans,
         })
 
         return NextResponse.json(

apps/sim/lib/execution/preprocessing.ts

@@ -480,7 +480,7 @@ async function logPreprocessingError(params: {
 
   try {
     const session =
-      loggingSession || new LoggingSession(workflowId, executionId, triggerType as any, requestId)
+      loggingSession || new LoggingSession(workflowId, executionId, triggerType, requestId)
 
     await session.safeStart({
       userId,