Skip to content

refactor(journey): snooze-conversation-node orphan case (EVO-1920)#95

Open
daniloleonecarneiro wants to merge 1 commit into
developfrom
danilocarneiro/evo-1920-jornadas-registry-case-snooze-conversation-node-orfao-fe-so
Open

refactor(journey): snooze-conversation-node orphan case (EVO-1920)#95
daniloleonecarneiro wants to merge 1 commit into
developfrom
danilocarneiro/evo-1920-jornadas-registry-case-snooze-conversation-node-orfao-fe-so

Conversation

@daniloleonecarneiro

Copy link
Copy Markdown

Summary

Remove o case órfão snooze-conversation-node do journey-execution.workflow.ts.

Escolha + justificativa

O workflow tinha dois cases (snooze-conversation-node e defer-conversation-node) apontando para o mesmo executor executeSnoozeConversationNode. Confirmei que:

  • O FE (editor de jornadas) registra apenas defer-conversation-node (pasta actions/defer-conversation); não existesnooze-conversation-node no palette/manifest da jornada.
  • As únicas referências a snooze-conversation-node no FE estão no subsistema de Automação (automationService.ts, color map em lib/utils.ts) — domínio separado, que não emite eventos de workflow de jornada para o evo-flow.
  • No evo-flow, snooze-conversation-node aparecia no case do workflow — nenhum trigger, fixture JSON, jornada salva ou teste depende dele.
  • O spec de cobertura (journey-execution.coverage.spec.ts) valida 1 case por nó do palette e tem defer-conversation-node; não havia teste para snooze-conversation-node (nunca foi nó do palette).

Logo o case era inalcançável e redundante. Removi-o e mantive defer-conversation-node (o type canônico do FE), que já invoca o mesmo executor — sem perda de funcionalidade. O efeito CRM continua sendo "snoozed", por isso o executor preserva o nome executeSnoozeConversationNode. Comentário atualizado documentando a consolidação.

Critério de aceite

Um único tipo de nó (defer-conversation-node) consistente entre FE e backend. ✓

Test plan

  • tsc -p tsconfig.json --noEmit → OK (exit 0)
  • jest journey-execution.coverage.spec.ts → 26/26 passando (inclui assert do case defer-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.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @daniloleonecarneiro, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@daniloleonecarneiro daniloleonecarneiro self-assigned this Jun 25, 2026
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.

1 participant