Currently, the command request for the find command is parsed inside the Invocation::run and Invocation::explain functions, and stored as a unique pointer in a local variable. During the execution of these functions, the ownership of that unique pointer is passed into CanonicalQuery and then further on, until it (potentially) ends up in a cursor that outlives the invocation.

As a result of the changes in SERVER-86084, the request will be parsed inside the FindCmd::parse and FindCmd::parseForExplain functions and stored in an instance member variable, which will then be used inside Invocation::run and Invocation::explain.

Passing the ownership of that unique pointer will leave it null and introduce the risk of writing code that can result in a crash. We don't want to block SERVER-86084 on this improvement, so we're filing a separate ticket to investigate how we can prevent this risk and refactor the code accordingly.