Como ListaNegociacoes é o modelo que gerencia e conhece as negociações, sugiro que ela também forneça um comportamento de verificar se contém ou não uma negociação.
Assim, ListaNegociacoes teria o método "contem(negociacao)" que encapsularia a lógica com "some()...".
O filtro do NegociacaoController ficaria mais simples:
.then(negociacoes => negociacoes.filter(n => !this._listaNegociacoes.contem(n)))Além de simplificar, também isola a lógica de saber se contém ou não, pois poderia ser chamada em outros lugares abstraindo esse conhecimento.
