From f85dbec397caa2096b3fe1a4733b3a7e39f2b7c6 Mon Sep 17 00:00:00 2001 From: Danilo Leone Date: Thu, 25 Jun 2026 00:06:39 -0300 Subject: [PATCH] refactor(journey): remove case orfao snooze-conversation-node (EVO-1920) O case 'snooze-conversation-node' era inalcancavel: nenhum no do editor de jornadas no FE, trigger ou fixture de jornada salva emite esse type. O case 'defer-conversation-node' (unico type canonico do FE) ja invoca o mesmo executor (executeSnoozeConversationNode), entao a funcionalidade de snooze e preservada. Comentario atualizado documentando a consolidacao. --- .../workflows/journey-execution.workflow.ts | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/src/modules/temporal/workflows/journey-execution.workflow.ts b/src/modules/temporal/workflows/journey-execution.workflow.ts index 126877f..ab786ca 100644 --- a/src/modules/temporal/workflows/journey-execution.workflow.ts +++ b/src/modules/temporal/workflows/journey-execution.workflow.ts @@ -928,19 +928,13 @@ export async function JourneyExecutionWorkflow( }); break; - case 'snooze-conversation-node': - nodeResult = - await actionNodeActivities.executeSnoozeConversationNode({ - nodeId: currentNode.id, - conversationId: - input.triggerEvent?.properties?.conversation_id || '', - sessionId: input.sessionId, - nodeData: currentNode.data, - }); - break; - case 'defer-conversation-node': - // Defer aliases snooze — same CRM effect ('snoozed'). + // `defer-conversation-node` is the single canonical node type the FE + // journey editor emits for "snooze/defer". The legacy + // `snooze-conversation-node` case was removed (EVO-1920): no FE node, + // trigger, or saved-journey fixture ever produced that type, so it was + // unreachable. The underlying CRM effect is still "snoozed", which is + // why the executor keeps the name `executeSnoozeConversationNode`. nodeResult = await actionNodeActivities.executeSnoozeConversationNode({ nodeId: currentNode.id,