Resolvi um novo bug crítico no @nodejs core com apenas UMA LINHA de código🤯 (e 30 linhas de testes 🥳)
🧵/1
#javascript #opensource #nodejs #js #streams #nodejsstreams #javascriptstreams
🧵/1
#javascript #opensource #nodejs #js #streams #nodejsstreams #javascriptstreams
Ontem saiu o Node.js v18.15 com uma pancada de coisas legais e esse bug eu havia corrigido no 19 e finalmente ele veio para a versão LTS
Pra mim, essa é a prova de que quantidade de código não tem nada a ver com produção ou impacto
/2
Pra mim, essa é a prova de que quantidade de código não tem nada a ver com produção ou impacto
/2
Qual era o problema?
As Node.js Streams são parte do Node.js desde o início e muita coisa interna do Node.js as usa para controlar eventos e processar dados sob demanda
/3
As Node.js Streams são parte do Node.js desde o início e muita coisa interna do Node.js as usa para controlar eventos e processar dados sob demanda
/3
Você pode usar as classes Readable, Writable e etc, para processar pentabytes de dados em JavaScript se precisar
O problema, é que elas trabalham com callbacks, então para ir para a forma mais moderna, há algum tempo é possível usar async generators.
/4
O problema, é que elas trabalham com callbacks, então para ir para a forma mais moderna, há algum tempo é possível usar async generators.
/4
Algo como
async function *(stream) {
for await (const chunk of stream) {
// manipular pedaço de dados
}
}
E isso funcionava nas versões anteriores, mas por alguma razão no Node.js 19 (experimental) isso parou de funcionar 😬
/5
async function *(stream) {
for await (const chunk of stream) {
// manipular pedaço de dados
}
}
E isso funcionava nas versões anteriores, mas por alguma razão no Node.js 19 (experimental) isso parou de funcionar 😬
/5
Consequentemente, isso quebrou todos os meus projetos 🥵
E se eu não tivesse resolvido, enquanto experimental, poderia ter quebrado centenas de milhares de pacotes que usam Node.js streams 😰😱
/6
E se eu não tivesse resolvido, enquanto experimental, poderia ter quebrado centenas de milhares de pacotes que usam Node.js streams 😰😱
/6
Gastei alguns bons dias discutindo e depurando o projeto Node.js para entender o que poderia fazer até que finalmente entendi
/7
/7
O negócio era que para fazer com que essas async generators funcionem, o modulo stream precisa ser carregado na inicialização da aplicação
Se você tentasse importar o método async pipeline, usado para controlar o fluxo de dados, como 'node:stream/promises'
/8
Se você tentasse importar o método async pipeline, usado para controlar o fluxo de dados, como 'node:stream/promises'
/8
Ele na verdade, importava tudo que tinha dentro de "promises" e não carregava o módulo "stream"
Então, a linha única de código que fiz para solucionar o problema, foi no momento que carregasse "promises", também trouxesse o "stream"
/9
Então, a linha única de código que fiz para solucionar o problema, foi no momento que carregasse "promises", também trouxesse o "stream"
/9
Uma linha de código que me tomou semanas para encontrar a solução e que sem ela poderia ter causado o CAOS 😂
/10
/10
Só que não acaba ai, você precisa implementar testes para validar que o bug não existe mais, e aí gastei mais horas ainda para descobrir como faria esse teste
/11
/11
Isso porque, a suite de testes do Node.js já carrega o modulo stream na inicialização, então precisei burlar o linter e dar uma "hackeada" na parada para funcionar
Dá uma olhada no PR: bit.ly/bug-nodejs-str…
/12
Dá uma olhada no PR: bit.ly/bug-nodejs-str…
/12
Ah e vem estudar JavaScript avançado comigo!
Liberamos hoje a possibilidade de parcelamento no BOLETO.
O treinamento conta com quase 40 horas de aulas gravadas, mais de 30 projetos, desafios e suporte exclusivo
Bora? bit.ly/jsexpert-erick…
/13
Liberamos hoje a possibilidade de parcelamento no BOLETO.
O treinamento conta com quase 40 horas de aulas gravadas, mais de 30 projetos, desafios e suporte exclusivo
Bora? bit.ly/jsexpert-erick…
/13
• • •
Missing some Tweet in this thread? You can try to
force a refresh
