I’d double-check the disposeBag comments.

I started to reply to this, then the whole thing ended up being quite long so I ended up just publishing it as it’s own piece.

Please check out: Swift 5.1 and Combine Memory Management: A Problem?

I’d be interested in your thoughts.