fix: loga o erro real dos services de envio (em vez de descartar silenciosamente) - #4
Open
FranciscoGoulartJr wants to merge 1 commit into
Conversation
…ces de envio
Os 4 services de envio (send-image, send-audio, send-video, send-message)
tinham catch(e) que nunca usava "e" - qualquer falha na chamada a Z-API
(400, 401, rede) virava so um spinner.fail() generico, sem nenhum detalhe
do motivo real. Adiciona console.error(e.response?.data || e.message) nos
4, o mesmo padrao que axios ja expoe no objeto de erro.
De quebra, send-message.js tinha um bug de escopo: "const spinner" era
declarado dentro do try{}, entao o catch{} que tentava chamar
spinner.fail(...) lançava "spinner is not defined" - mascarando o erro
original com um ReferenceError toda vez que a chamada falhava. Movido
para fora do try (mesmo padrao ja usado nos outros 3 services). Tambem
corrigido o texto do spinner ("Enviando áudio para a API" -> "Enviando
mensagem para a API", copy-paste de send-audio.js).
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
1 participant
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.
O que mudou
Os 4 services de envio (
send-image.js,send-audio.js,send-video.js,send-message.js) tinhamcatch (e) { spinner.fail('...') }sem usareem nenhum lugar — qualquer erro real da chamada à API (400, 401, timeout de rede) virava só uma mensagem genérica no spinner, sem nenhum detalhe do motivo. Adicioneiconsole.error(e.response?.data || e.message)nos 4, aproveitando o shape padrão de erro do axios.De quebra, encontrei um bug de escopo em
send-message.js:const spinnerera declarado dentro do blocotry {}, mas ocatch {}tentava usarspinner.fail(...)— comoconsté block-scoped, isso lançaReferenceError: spinner is not definedtoda vez que a chamada falha, mascarando completamente o erro original. Corrigi movendo a declaração pra fora dotry, no mesmo padrão já usado nos outros 3 services. Aproveitei para corrigir também o texto do spinner nesse arquivo ("Enviando áudio para a API"→"Enviando mensagem para a API", resquício de copy-paste desend-audio.js).Por que
Achei isso estudando os repositórios da org pra melhorar nossa própria integração (somos usuários da Z-API). Como o padrão se repetia idêntico nos 4 arquivos e o bug de escopo é objetivamente incorreto (não é preferência de estilo), pareceu valer a pena mandar de volta pra quem mais usa esse repo como referência de integração.
Teste
Revisão manual do diff — mudança mínima e isolada, sem alterar o fluxo de sucesso em nenhum dos 4 arquivos.