improvement(api): replace workflow jobs with execution resources - #6294
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryHigh Risk Overview Polling is unified on the execution resource: HITL resume adds Reviewed by Cursor Bugbot for commit 94e28e7. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThe PR replaces public asynchronous workflow-job receipts with canonical execution resources while preserving legacy polling compatibility.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains; the previous resume-status issue is fixed by resolving active resume queue entries before the existing paused log, and the documentation convention thread was correctly resolved as inapplicable to external-consumer examples.
|
| Filename | Overview |
|---|---|
| apps/sim/lib/workflows/executor/execution-status.ts | Projects claimed and pending resume attempts ahead of stale paused logs, with terminal durable logs retaining precedence. |
| apps/sim/app/api/resume/resume-handler.ts | Dispatches v2 async resumes with deterministic queue identities and returns canonical execution polling URLs while preserving legacy job receipts. |
| apps/sim/app/api/v2/workflows/[id]/executions/[executionId]/resume/route.ts | Adds the contract-bound v2 endpoint for resuming a paused workflow execution. |
| packages/ts-sdk/src/index.ts | Migrates SDK execution and polling behavior from generic jobs to canonical workflow execution resources. |
| packages/python-sdk/simstudio/init.py | Updates Python SDK async receipts and execution-status polling to use execution IDs. |
| apps/docs/openapi-v2-workflows.json | Documents the v2 resume endpoint and its synchronous, asynchronous, validation, and error responses. |
Sequence Diagram
sequenceDiagram
participant Client
participant API as Workflow API
participant Queue as Execution Queue
participant Worker
participant Logs as Durable Execution Logs
Client->>API: POST execute/resume (async)
API->>Queue: Enqueue using execution identity
API-->>Client: 202 executionId + statusUrl
Client->>API: GET execution resource
API->>Queue: Resolve active queued attempt
Queue-->>API: queued/running state
API-->>Client: Execution resource
Worker->>Logs: Persist execution state/output
Client->>API: GET execution resource
API->>Logs: Read durable state
API-->>Client: completed/failed/paused resource
Reviews (6): Last reviewed commit: "fix(api): prefer terminal logs over stal..." | Re-trigger Greptile
3dcbfe2 to
3313651
Compare
|
@cursor review |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 5698672. Configure here.
5698672 to
a4c1152
Compare
|
@cursor review |
|
@cursor review |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit c903ba1. Configure here.

Summary
Type of Change
Testing
Checklist