fix(remove workflow.state usage): no more usage of deprecated state column in any routes

[icecrasher321]

Description

Update to only use normalized tables in chat, schedule, workflow in workflow, webhooks.

Type of change

• Bug fix (non-breaking change which fixes an issue)
• Code refactoring (no functional changes)

How Has This Been Tested?

Must test all the functionality listed above.

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added tests that prove my fix is effective or that my feature works
• All tests pass locally and in CI (bun run test)
• My changes generate no new warnings
• Any dependent changes have been merged and published in downstream modules
• I have updated version numbers as needed (if needed)
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Security Considerations:

• My changes do not introduce any new security vulnerabilities
• I have considered the security implications of my changes

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
sim	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 1, 2025 1:44am

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
docs	⬜️ Skipped (Inspect)			Jul 1, 2025 1:44am

[delve-auditor]

✅ No security or compliance issues detected. Reviewed everything up to 7eadb47.

Security Overview

• 🔎 Scanned files: 13 changed file(s)

Detected Code Changes

Change Type

Relevant files

Refactor

► chat/utils.ts     Remove usage of deprecated state column ► schedules/execute/route.ts     Use normalized tables for workflow data ► webhooks/trigger/route.ts     Load workflow data from normalized tables ► workflows/[id]/deploy/route.ts     Use normalized tables for workflow comparison ► workflows/[id]/execute/route.ts     Switch to normalized tables for workflow data ► webhooks/utils.ts     Use normalized tables for execution ► workflows/registry/store.ts     Update state handling for normalized data

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

[greptile-apps]

PR Summary

Major architectural change to replace the deprecated workflow.state JSON column with normalized database tables for workflow data storage and retrieval.

• Introduced new normalized tables in apps/sim/db/schema.ts: workflowBlocks, workflowEdges, and workflowSubflows for better data integrity and query performance
• Added loadWorkflowFromNormalizedTables helper function used across chat, webhooks, and schedule execution routes
• Updated webhook trigger routes in apps/sim/app/api/webhooks/trigger/[path]/route.ts to construct state from normalized data
• Enhanced schedule execution in apps/sim/app/api/schedules/execute/route.ts with improved error handling for missing normalized data
• Enforced stricter deployment requirements in apps/sim/app/api/chat/utils.ts where only deployed workflows can be executed

_{7 files reviewed, 4 comments
Edit PR Review Bot Settings | Greptile}