fix(query-graphs): When eliminating suboptimal indirect paths, proper…

…ly check for a direct `@key` edge (#3160)

This PR fixes a bug where when removing suboptimal indirect paths, we
didn't properly check for a direct `@key` edge at the end of the
potential direct path.

<!-- ROUTER-750 -->

.changeset/orange-ears-sip.md

@@ -0,0 +1,5 @@
+---
+"@apollo/query-graphs": patch
+---
+
+When eliminating suboptimal indirect paths during query planning, properly check for a direct `@key` edge at the end of the potential direct path.

query-graphs-js/src/graphPath.ts

@@ -1586,7 +1586,7 @@ function advancePathWithNonCollectingAndTypePreservingTransitions<TTrigger, V ex
           if (prevSubgraphVertex
             && (
               backToPreviousSubgraph
-              || hasValidDirectKeyEdge(toAdvance.graph, prevSubgraphVertex, edge.tail.source, conditionResolver, maxCost) != undefined
+              || hasValidDirectKeyEdge(toAdvance.graph, prevSubgraphVertex, edge.tail.source, conditionResolver, maxCost)
             )
           ) {
             debug.groupEnd(