more code optimizations.

Author: gmac

lib/graphql/stitching/planner.rb

@@ -11,7 +11,7 @@ class Planner
       ROOT_INDEX = 0
 
       class ScopePartition
-        attr_accessor :location, :selections
+        attr_reader :location, :selections
 
         def initialize(location:, selections:)
           @location = location

lib/graphql/stitching/supergraph.rb

@@ -187,7 +187,17 @@ def route_type_to_locations(type_name, start_location, goal_locations)
 
       private
 
-      PathNode = Struct.new(:location, :key, :cost, :resolver, keyword_init: true)
+      class PathNode
+        attr_reader :location, :key, :resolver
+        attr_accessor :cost
+      
+        def initialize(location:, key:, resolver: nil, cost: 0)
+          @location = location
+          @key = key
+          @resolver = resolver
+          @cost = cost
+        end
+      end
 
       # tunes A* search to favor paths with fewest joining locations, ie:
       # favor longer paths through target locations over shorter paths with additional locations.
@@ -196,7 +206,7 @@ def route_type_to_locations_via_search(type_name, start_location, goal_locations
         costs = {}
 
         paths = possible_keys_for_type_and_location(type_name, start_location).map do |possible_key|
-          [PathNode.new(location: start_location, key: possible_key, cost: 0)]
+          [PathNode.new(location: start_location, key: possible_key)]
         end
 
         while paths.any?

lib/graphql/stitching/util.rb

@@ -4,12 +4,29 @@ module GraphQL
   module Stitching
     # General utilities to aid with stitching.
     class Util
-      TypeStructure = Struct.new(:list, :null, :name, keyword_init: true) do
-        alias_method :list?, :list
-        alias_method :null?, :null
+      class TypeStructure
+        attr_reader :name
+      
+        def initialize(list:, null:, name:)
+          @list = list
+          @null = null
+          @name = name
+        end
+
+        def list?
+          @list
+        end
+        
+        def null?
+          @null
+        end
 
         def non_null?
-          !null
+          !@null
+        end
+
+        def ==(other)
+          @list == other.list? && @null == other.null? && @name == other.name
         end
       end