From 478c753a5065845ea627cc0e6acf34b5d630eb9b Mon Sep 17 00:00:00 2001 From: Paul Rastoin <45004772+prastoin@users.noreply.github.com> Date: Wed, 10 Dec 2025 15:23:22 +0100 Subject: [PATCH] [`Validate build and run`] Early return if workspace migration has no actions (#16467) # Introduction Early returning in `validateBuildAndRun` as the runner even if passed an empty workspace migration actions array be invalidating dependency related flat entity maps. When we will refactor the runner to consume the generic flat entity maps tools such as `addFlatEntityToFlatEntityAndRelatedEntityMapsThroughMutationOrThrow` we will be able to remove the dependency cache invalidation --- .../workspace-migration-validate-build-and-run-service.ts | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/packages/twenty-server/src/engine/workspace-manager/workspace-migration-v2/services/workspace-migration-validate-build-and-run-service.ts b/packages/twenty-server/src/engine/workspace-manager/workspace-migration-v2/services/workspace-migration-validate-build-and-run-service.ts index 571ec62dc03..620eb4db46c 100644 --- a/packages/twenty-server/src/engine/workspace-manager/workspace-migration-v2/services/workspace-migration-validate-build-and-run-service.ts +++ b/packages/twenty-server/src/engine/workspace-manager/workspace-migration-v2/services/workspace-migration-validate-build-and-run-service.ts @@ -189,6 +189,13 @@ export class WorkspaceMigrationValidateBuildAndRunService { return validateAndBuildResult; } + // Note: This should be removed once we've refactored the runner optimistic rendering + // As with the current implementation passing an empty workspace migration might result + // in dependency flat entity maps invalidation + if (validateAndBuildResult.workspaceMigration.actions.length === 0) { + return; + } + await this.workspaceMigrationRunnerV2Service .run(validateAndBuildResult.workspaceMigration) .catch((error) => {