fix(logging): scope completeWithCancellation and completeWithPause reads by workflowId

Both SELECT queries that check execution status before writing a
terminal result were only filtering on executionId. Adds workflowId
to the WHERE clause so all seven reads and writes in LoggingSession
consistently scope by (workflowId, executionId).

Author: waleedlatif1

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

@@ -671,7 +671,12 @@ export class LoggingSession {
       const currentLog = await db
         .select({ status: workflowExecutionLogs.status })
         .from(workflowExecutionLogs)
-        .where(eq(workflowExecutionLogs.executionId, this.executionId))
+        .where(
+          and(
+            eq(workflowExecutionLogs.workflowId, this.workflowId),
+            eq(workflowExecutionLogs.executionId, this.executionId)
+          )
+        )
         .limit(1)
         .then((rows) => rows[0])
 
@@ -770,7 +775,12 @@ export class LoggingSession {
       const currentLog = await db
         .select({ status: workflowExecutionLogs.status })
         .from(workflowExecutionLogs)
-        .where(eq(workflowExecutionLogs.executionId, this.executionId))
+        .where(
+          and(
+            eq(workflowExecutionLogs.workflowId, this.workflowId),
+            eq(workflowExecutionLogs.executionId, this.executionId)
+          )
+        )
         .limit(1)
         .then((rows) => rows[0])