After SERVER-91865, TransactionCoordinator inherits from std::enable_shared_from_this and uses shared_from_this extensively. In that change, we decided to keep the old constructor public, which does allow misuse by constructing a non-shared_ptr TransactionCoordinator. We should consider making it only constructible as a shared_ptr by exposing a factory function or similar.