improvement(deployments): bugfixes for run-block, airtable + external sub management#5680
improvement(deployments): bugfixes for run-block, airtable + external sub management#5680icecrasher321 wants to merge 7 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
…ment-bugs # Conflicts: # apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/deploy/components/deploy-modal/deploy-modal.tsx
PR SummaryHigh Risk Overview Execution pins to admitted versions: MCP tool calls require an active deployment version row, pass Smaller changes: Airtable Reviewed by Cursor Bugbot for commit 7ffd781. Configure here. |
|
bugbot run |
Greptile SummaryThis PR makes deployment status follow the real deployment lifecycle. The main changes are:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (2): Last reviewed commit: "address comments" | Re-trigger Greptile |
|
bugbot run |
|
bugbot run |
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 7ffd781. Configure here.
Summary
Make deployment status truthful end-to-end: deployments now move through an explicit state machine (preparing → activating → active / failed / superseded) backed by an outbox with inline-first execution, so a version is only reported live once its webhooks, schedules, and external subscriptions actually exist. Webhook registration is rebuilt on stable, generation-fenced rows with create-before-delete cutover — a failed deploy can no longer take down the currently live version, leave ghost registrations that wedge future deploys, or silently swallow provider errors (now surfaced in the deploy modal, versions list, and API responses).
Type of Change
Testing
Tested manually
Checklist