Salvar negociações importadas no IndexedDB

Oi Renan, tudo bem? Me parece problemático você por o getConnection e criar o DAO dentro de um forEach, isso indica que pra toda negociação você vai criar uma nova conexão e um novo DAO sendo que você pode usar os dois dentro do laço.

Eu colocaria as instâncias dos objetos fora do laço.

no meu caso eu modifiquei o cadastra e o NegociacaoDao para receberem uma array de negociações

NegociacaoService.js

  importa(lista) {
    return Promise.all([
      this.obterSemana(),
      this.obterSemanaAnterior(),
      this.obterSemanaRetrasada(),
    ]).then(negociacoes => negociacoes.flat())
      .then(negociacoes => {
        var n = negociacoes.filter(n => !lista.possui(n))
        this.cadastra(...n)
        return n
      })
      .catch(erro => {
        throw new Error(erro)
      })
  }

  cadastra(...negociacoes) {
    return ConnectionFactory
      .getConnection()
      .then(connection => new NegociacaoDao(connection))
      .then(dao => dao.adiciona(...negociacoes))
      .then(() => 'Negociações adicionadas com sucesso')
      .catch(erro => {
        throw new Error(erro)
      })
  }

NegociacaoDao.js

adiciona(...negociacoes) {
  return new Promise((resolve, reject) => {
    if (!negociacoes.length) {
      return
    }

    let request = this._connection
      .transaction([this._store], 'readwrite')
      .objectStore(this._store)
      .add(negociacoes.pop())

    request.onsuccess = () => {
      if (!negociacoes.length) {
        resolve()
      }
      else {
        this.adiciona(...negociacoes)
      }
    }

    request.onerror = event => {
      console.log(event.target.error)
      reject('Não foi possível adicionar a negociação')
    }
  })
}

É, também é uma alternativa. Eu preferia criar um segundo método que aceita um array e deleta a chamada, por que essa solução me força a passar sempre um array para salvar.