When part of the work for an operation requires running a nested operation inside the same server, we should always use DBDirectClient for that. Currently, there are other places in non-test code that call ServiceEntryPointCommon::handleRequest() directly:

• in ClusterSEPTransactionClientBehaviors::handleRequest()
• in UMCTransaction::runCommand()
• in DefaultSEPTransactionClientBehaviors::handleRequest()
• in runCommandInLocalTxn() in sharding_catalog_manager.cpp
• in commitOrAbortTransaction() in sharding_catalog_manager.cpp
• in AsyncWorkScheduler::scheduleRemoteCommand()