Skip to content

Conversation

@emir-karabeg
Copy link
Collaborator

Summary

Optimizes the workflow redeployment check to prevent unnecessary API calls. Previously, the /api/workflows/[id]/status endpoint was triggered by all state changes, including UI-only interactions like moving blocks or opening/closing tools. This PR introduces a deployment-signature utility to ensure the check is only triggered when deployment-relevant properties of the workflow change, significantly reducing API call frequency.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Other: Optimization

Testing

The changes have been verified through local testing and code review to ensure the deployment-signature accurately reflects deployment-relevant changes and the use-change-detection hook triggers the status check only when necessary. Reviewers should focus on the logic within createDeploymentSignature to confirm all UI-only properties (e.g., position, layout, expanded states, test values) are correctly excluded.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Screenshots/Videos


Open in Cursor Open in Web

@cursor
Copy link

cursor bot commented Dec 19, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@vercel
Copy link

vercel bot commented Dec 19, 2025

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

1 Skipped Deployment
Project Deployment Review Updated (UTC)
docs Skipped Skipped Dec 19, 2025 11:29pm

@waleedlatif1 waleedlatif1 deleted the cursor/workflow-redeployment-check-optimization-6139 branch December 20, 2025 03:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants