fix(logging): make workflowId required in markExecutionAsFailed

Making workflowId optional left a footgun — future callers could silently
omit it and the WHERE clause would degrade to executionId-only, losing the
cross-tenant scoping guarantee. All callers already supply workflowId, so
making it required (with string | undefined for the middle params to keep
call sites unchanged) closes the gap without touching any caller.

Author: waleedlatif1

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

@@ -1090,9 +1090,9 @@ export class LoggingSession {
 
   static async markExecutionAsFailed(
     executionId: string,
-    errorMessage?: string,
-    requestId?: string,
-    workflowId?: string
+    errorMessage: string | undefined,
+    requestId: string | undefined,
+    workflowId: string
   ): Promise<void> {
     try {
       const message = errorMessage || 'Run failed'
@@ -1118,7 +1118,7 @@ export class LoggingSession {
         .where(
           and(
             eq(workflowExecutionLogs.executionId, executionId),
-            workflowId ? eq(workflowExecutionLogs.workflowId, workflowId) : undefined
+            eq(workflowExecutionLogs.workflowId, workflowId)
           )
         )