Esse código ta errado, o Promise.all recebe um Array de Promise, para ficar certo seria algo tipo:
Promise.all(data.map(async (item) => item.doSomething()))
Aqui o map vai gerar o array que o Promise.all precisa usando uma função assíncrona
Sim, foi proposital.
Minha pergunta foi:
Feitos os devidos ajustes no meu código de exemplo, será que o map ainda se sairia melhor?
Creio que não tenha diferença, contanto que as Promises estejam sendo guardadas para depois aguardar o fim delas no Promise.all
Do jeito que ta ele vai rodar lento já que as tarefas dentro do for..of provavelmente não são executadas de forma assíncrona