The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 4, 2026 0:35am

PR Summary

Medium Risk
Medium risk because it changes how deployment status/needs-redeployment are sourced and updated across UI, hooks, and registry hydration; regressions would show up as incorrect deploy state or stale deployment UI until queries refresh.

Overview
Removes deployment state from Zustand by deleting deploymentStatuses/needsRedeployment fields, related types/actions, and helper APIs (revertToDeployedState, getAllWorkflowsWithValues, etc.), making React Query the source of truth for deployment info.

Updates UI and Copilot flows to read deployment status from React Query (useDeploymentInfo / getQueryClient().getQueryData(deploymentKeys.info)) and to rely on query invalidation after deploy-related tool calls rather than mutating Zustand.

Cleans up API/state payloads by dropping deploymentStatuses from workflow/checkpoint/deployment revert state normalization, and refactors deploy/undeploy/activate mutations to invalidate caches in onSettled.

Removes the custom-tool-cache utility and rewires custom tool lookups (and tests) to read directly from the React Query cache via getQueryClient().

^{Reviewed by Cursor Bugbot for commit 501b7c1. Configure here.}

Greptile Summary

This PR cleanly removes deploymentStatuses and needsRedeployment from Zustand stores (both WorkflowRegistry and WorkflowState) and makes React Query the single source of truth for deployment status. The approach is architecturally sound: deployment info is seeded into the React Query cache during workflow load (loadWorkflowState in the registry store), and all reads are delegated to useDeploymentInfo / getQueryClient().getQueryData(). Dead code (getBlockWithValues, getAllWorkflowsWithValues, revertToDeployedState, the custom-tool-cache utility) is removed and all call sites updated.

Key changes:

• deploymentStatuses and needsRedeployment removed from WorkflowState and WorkflowRegistryState; React Query cache is seeded on workflow load instead
• useDeployWorkflow and useUndeployWorkflow mutation callbacks consolidated from onSuccess/onError into a single onSettled (fires on both success and error) — aligning with project conventions
• getWorkflowWithValues stripped of isDeployed/deployedAt fields; the sole remaining caller (panel.tsx export handler) does not need them
• All API routes, socket operations, test fixtures, and persistence utilities updated to drop the now-removed fields
• Two smaller mutations (useUpdateDeploymentVersion, useUpdatePublicApi) were not migrated to onSettled and still use the old onSuccess/onError split — minor inconsistency worth addressing for uniformity

Confidence Score: 5/5

Safe to merge — all remaining findings are P2 style suggestions with no impact on runtime correctness.

The refactor is thorough: all references to the removed Zustand deployment state have been cleaned up across stores, API routes, socket operations, and tests. The React Query seeding pattern in loadWorkflowState is correct, and the onSettled migration for the core deploy/undeploy mutations follows project conventions. The two mutations not yet migrated to onSettled (useUpdateDeploymentVersion, useUpdatePublicApi) are functionally correct — the only concern is stylistic inconsistency within the file.

apps/sim/hooks/queries/deployments.ts — minor inconsistency between the newly migrated mutations and two older ones still using onSuccess/onError.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant C as Component
    participant RQ as React Query Cache
    participant ZS as Zustand Store
    participant API as API Route

    Note over C,API: Before this PR — dual source of truth
    C->>ZS: read deploymentStatuses / needsRedeployment
    C->>RQ: read useDeploymentInfo
    API-->>ZS: write on deploy/undeploy
    API-->>RQ: invalidate on deploy/undeploy

    Note over C,API: After this PR — React Query is single source of truth
    C->>RQ: useDeploymentInfo(workflowId)
    RQ-->>C: isDeployed, deployedAt, needsRedeployment

    C->>API: POST /api/workflows/id/deploy
    API-->>RQ: onSettled → invalidateDeploymentQueries()
    RQ-->>C: fresh deployment info

    ZS->>RQ: loadWorkflowState() seeds cache via setQueryData()

_{Reviews (3): Last reviewed commit: "improvement(stores): remove deployment s..." | Re-trigger Greptile}

@greptile

@cursor review

@greptile

@cursor review