fix(federation): fixed a mismatch after merging PR #5743

apollo-federation/src/query_plan/fetch_dependency_graph.rs

@@ -2362,6 +2362,14 @@ impl FetchDependencyGraphNode {
                 .cloned()
                 .collect::<Vec<_>>()
         };
+        let variable_usages = {
+            let mut list = variable_definitions
+                .iter()
+                .map(|var_def| var_def.name.clone())
+                .collect::<Vec<_>>();
+            list.sort();
+            list
+        };
 
         let mut operation = if self.is_entity_fetch {
             operation_for_entities_fetch(
@@ -2387,16 +2395,6 @@ impl FetchDependencyGraphNode {
             operation.reuse_fragments(fragments)?;
         }
 
-        let variable_usages = {
-            let mut list = operation
-                .variables
-                .iter()
-                .map(|variable| variable.name.clone())
-                .collect::<Vec<_>>();
-            list.sort();
-            list
-        };
-
         let operation_document = operation.try_into()?;
 
         let node = super::PlanNode::Fetch(Box::new(super::FetchNode {

apollo-federation/tests/query_plan/build_query_plan_tests/handles_operations_with_directives.rs

@@ -148,6 +148,9 @@ fn test_if_directives_at_the_operation_level_are_passed_down_to_subgraph_queries
         }
       }
     "#);
+    // This checks a regression where the `variable_usages` included the `representations` variable.
+    assert_eq!(b_fetch_nodes[0].variable_usages.len(), 0);
+    assert_eq!(b_fetch_nodes[1].variable_usages.len(), 0);
 }
 
 #[test]