In a code review, Charlie pointed out that in dispatchShardPipeline() we sometimes pass the collation from the request AggregationRequest when doing shard targeting, rather than passing the collation which is the default for the collection (which is computed here).

We looked into it some more and could not find any incorrect behavior, but it might be worth doing a more thorough check, and refactor the code so that we only use one collation.