Add post hooks for workflow deletion

[thomtrp]

Delete all workflow sub objects when workflow is deleted. Other sub objects cannot be deleted otherwise.

We do not listen to deletion events so I am not adding them. Those post hooks should be deleted Q1 once we properly handle cascade for soft deletion

[greptile-apps]

PR Summary

Adds post-query hooks to handle cascading deletion of workflow sub-entities (versions, runs, event listeners) when workflows are deleted, serving as a temporary solution until proper soft deletion cascade is implemented.

• Critical bug: cleanWorkflowsSubEntities in /packages/twenty-server/src/modules/workflow/common/workspace-services/workflow-common.workspace-service.ts uses incorrect entity type WorkflowVersionWorkspaceEntity for workflowRun and eventListener repositories
• Performance issue: map() is used incorrectly for async operations without awaiting results in cleanWorkflowsSubEntities
• Missing error handling for delete operations in cleanWorkflowsSubEntities
• No transaction wrapping for the cascade deletion operations, risking partial deletions

_{4 file(s) reviewed, 7 comment(s)
Edit PR Review Bot Settings | Greptile}

[Weiko]

LGTM

[Weiko]

I guess if you really want to fire-and-forget, you don't need map here but foreach instead. This won't guarantee the execution of your promises though I'd suggest using Promise.all()