@h30s is attempting to deploy a commit to the Sim Team on Vercel.

A member of the Team first needs to authorize it.

Greptile Overview

Greptile Summary

Fixed a race condition in pause/resume flow where resume requests could fail with "Paused execution not found" if they arrived before the paused state was fully persisted to the database.

Key Changes:

• Wrapped the database insert/update in persistPauseResult within a transaction (line 125-161)
• Moved processQueuedResumes call after transaction commits to ensure row visibility
• Added unit tests verifying transaction usage and call sequencing

How it works:
The transaction ensures atomic persistence of paused executions. When a concurrent resume request arrives via enqueueOrStartResume, it uses SELECT ... FOR UPDATE which will wait for the transaction to commit, preventing the "not found" error. After the transaction commits and the row is visible, queued resumes are processed.

Confidence Score: 5/5

• This PR is safe to merge with minimal risk - it's a focused bug fix with proper transaction handling
• The implementation correctly addresses the race condition by wrapping the database operation in a transaction, ensuring atomic visibility. The fix is minimal, focused, and follows the existing transaction patterns in the codebase. Unit tests verify the correct behavior.
• No files require special attention

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant WF as Workflow Executor
    participant PM as PauseResumeManager
    participant DB as Database
    participant API as Resume API
    participant Queue as processQueuedResumes

    Note over WF,Queue: Race Condition Scenario (Fixed)

    WF->>PM: persistPauseResult()
    activate PM
    
    PM->>DB: BEGIN TRANSACTION
    activate DB
    
    PM->>DB: INSERT/UPDATE paused_executions
    Note over PM,DB: Row locked in transaction
    
    par Concurrent Resume Request
        API->>PM: enqueueOrStartResume()
        activate PM
        PM->>DB: SELECT ... FOR UPDATE
        Note over PM,DB: Waits for transaction commit
    end
    
    DB-->>PM: INSERT committed
    PM->>DB: COMMIT TRANSACTION
    deactivate DB
    
    Note over PM: Transaction committed,<br/>row now visible
    
    DB-->>PM: paused execution found
    PM->>DB: Insert/update resume queue
    PM-->>API: Resume started/queued
    deactivate PM
    
    PM->>Queue: processQueuedResumes(executionId)
    activate Queue
    Queue->>DB: SELECT pending resumes
    Queue->>PM: startResumeExecution()
    deactivate Queue
    
    deactivate PM

@waleedlatif1 Ready from my Side!