Correção do Markdown na integração N8N e tratamento de mensagens do EvolutionBot#1470
Closed
oriondesign2015 wants to merge 4 commits intoEvolutionAPI:developfrom
oriondesign2015:develop
Closed
Correção do Markdown na integração N8N e tratamento de mensagens do EvolutionBot#1470oriondesign2015 wants to merge 4 commits intoEvolutionAPI:developfrom oriondesign2015:develop
oriondesign2015 wants to merge 4 commits intoEvolutionAPI:developfrom
oriondesign2015:develop
Conversation
## Descrição Esta PR implementa melhorias no processamento de mensagens do serviço N8n, especificamente no método `sendMessageWhatsApp`. As alterações visam melhorar o tratamento de links e formatação markdown nas mensagens. ## Mudanças - Ajustada a desestruturação do objeto `match` para capturar corretamente o texto completo do link markdown - Modificado o tratamento de links que não são mídia para preservar a formatação original - Mantida a expressão regular para detecção de links markdown ## Impacto - Mensagens com links markdown agora são processadas de forma mais consistente - A formatação original dos links é preservada quando não são tratados como mídia - Melhor tratamento de casos onde o link não corresponde a um tipo de mídia suportado
Contributor
Reviewer's GuideThis PR enhances message formatting by refining markdown link processing in N8nService—improving regex matching, destructuring, and preserving original link formatting for non-media—and introduces logic in EvolutionBotService to intelligently strip superfluous external single quotes from returned messages without altering legitimate internal apostrophes or quotes. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
## Descrição Esta PR corrige um problema no tratamento de mensagens retornadas na integração do EvolutionBot, onde aspas extras estavam sendo incluídas no início e fim das mensagens quando acompanhado de Markdown. ## Solução Foi implementada uma lógica inteligente para tratar as aspas nas mensagens: - Verifica se a mensagem inteira está entre aspas simples - Se estiver, verifica se há aspas simples dentro do conteúdo - Remove as aspas externas apenas se não houver aspas/apóstrofos dentro do conteúdo Isso garante que: - Mensagens com aspas extras são corrigidas: `'texto'` -> `texto` - Mensagens com aspas/apóstrofos legítimos são preservadas: `'texto com "aspas" e don't'` -> `'texto com "aspas" e don't'` ## Impacto - Melhora a formatação das mensagens enviadas pelo bot - Preserva aspas e apóstrofos que fazem parte legítima do conteúdo - Mantém a compatibilidade com o formato de mensagens do n8n
Contributor
Author
|
@sourcery-ai review |
Contributor
There was a problem hiding this comment.
Hey @oriondesign2015 - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
src/api/integrations/chatbot/evolutionBot/services/evolutionBot.service.ts
Outdated
Show resolved
Hide resolved
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
Contributor
|
top my friend apply npm run lint please |
|
up... ### uma imagem valeu mais que mil palavras |
Corrige problemas de formatação identificados pelo Prettier no arquivo evolutionBot.service.ts que acabei deixando e alertado pelo git, removendo espaços em branco desnecessários e ajustando a indentação do código.
Collaborator
|
implementado no novo formato |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Descrição
Esta PR implementa melhorias no processamento de mensagens do serviço N8N e corrige um problema no tratamento de mensagens retornadas na integração do EvolutionBot. As alterações visam melhorar o tratamento de links e formatação markdown nas mensagens, além de resolver a questão de aspas extras que estavam sendo incluídas no início e fim das mensagens.
Mudanças
matchpara capturar corretamente o texto completo do link markdown.Impacto
'texto'->texto.'texto com "aspas" e don't'->'texto com "aspas" e don't'.Prints Antes:
Tentativa de envio de imagem com markdown na integração do N8N:

Tentativa de envio de imagem com markdown na integração Evolution Bot:

Prints Depois:
Tentativa de envio de imagem com markdown na integração do N8N:

Tentativa de envio de imagem com markdown na integração Evolution Bot:

Summary by Sourcery
Improve markdown link handling in N8N integration and strip extraneous quotes in EvolutionBot message responses.
Bug Fixes:
Enhancements: