graphql-stitching 1.7.1 → 1.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 43ca2665bc1d0e0a87eead760ee4f7b1caddf324861b66414bc011958901eeff
-  data.tar.gz: 18a14928e703744d29b57665c51ecdb6a17b1d9d2b7bc58fe8727a37b55fbcd5
+  metadata.gz: 10934c3d783de823cba55bfe33b361f7ffb6ead4a113a415199f660db97790be
+  data.tar.gz: b310d93e9493319363924fd4eaca3a828e110479a91906394b35df5cb2b7d2d7
 SHA512:
-  metadata.gz: b5ee7817b699447b35c937eb3df872433f8c318310debab6fea1577513171260a39e900c64cf37b4bfcb6f4fde4b509b5216ddf95879dd53a9878a3924016b5d
-  data.tar.gz: 0ddda7a4df33d5b6946a3a6283a4388d507c18d7ff758db1b2f09179546d6be87b6e78390fa6f623442e0f6e79a91aac69b0922542c4134673175377c47cfc0f
+  metadata.gz: d998d78a25d271bd41b70df55f6d9e5442e2a5ce38c05c30ed281cc34a28e88aa41a0f923f07e306e1c472143e4a164ad109f1297a8b0af0c621769e38b92f30
+  data.tar.gz: 9aee659c12c236428062e2621d189b6a79518e31ed8fed60c995a27fb18fe746479c554edcb0bc9e091ed39e412185c63da0223d7acee21b304bd6707c630298

data/lib/graphql/stitching/executor/root_source.rb

@@ -16,7 +16,7 @@ module GraphQL::Stitching
           @executor.request.operation_name,
           @executor.request.operation_directives,
         )
-        query_variables = @executor.request.variables.slice(*op.variables.keys)
+        query_variables = @executor.request.variables.slice(*op.variables.each_key)
         result = @executor.request.supergraph.execute_at_location(op.location, query_document, query_variables, @executor.request)
         @executor.query_count += 1
 

data/lib/graphql/stitching/executor/shaper.rb

@@ -32,7 +32,7 @@ module GraphQL::Stitching
             field_name = node.alias || node.name
 
             if @request.query.get_field(parent_type, node.name).introspection?
-              if node.name == TYPENAME && parent_type == @root_type
+              if node.name == TYPENAME && parent_type == @root_type && node != TypeResolver::TYPENAME_EXPORT_NODE
                 raw_object[field_name] = @root_type.graphql_name
               end
               next

data/lib/graphql/stitching/executor/type_resolver_source.rb

@@ -165,7 +165,7 @@ module GraphQL::Stitching
         if pathed_errors_by_op_index_and_object_id.any?
           pathed_errors_by_op_index_and_object_id.each do |op_index, pathed_errors_by_object_id|
             repath_errors!(pathed_errors_by_object_id, ops.dig(op_index, "path"))
-            errors_result.concat(pathed_errors_by_object_id.values)
+            errors_result.push(*pathed_errors_by_object_id.each_value)
           end
         end
 

data/lib/graphql/stitching/planner.rb

@@ -213,7 +213,7 @@ module GraphQL
         input_selections.each do |node|
           case node
           when GraphQL::Language::Nodes::Field
-            if node.alias&.start_with?(TypeResolver::EXPORT_PREFIX)
+            if node.alias&.start_with?(TypeResolver::EXPORT_PREFIX) && node != TypeResolver::TYPENAME_EXPORT_NODE
               raise StitchingError, %(Alias "#{node.alias}" is not allowed because "#{TypeResolver::EXPORT_PREFIX}" is a reserved prefix.)
             elsif node.name == TYPENAME
               locale_selections << node
@@ -284,7 +284,7 @@ module GraphQL
           remote_selections_by_location = delegate_remote_selections(parent_type, remote_selections)
 
           # D) Create paths routing to new entrypoint locations via resolver queries.
-          routes = @supergraph.route_type_to_locations(parent_type.graphql_name, current_location, remote_selections_by_location.keys)
+          routes = @supergraph.route_type_to_locations(parent_type.graphql_name, current_location, remote_selections_by_location.each_key)
 
           # E) Translate resolver pathways into new entrypoints.
           routes.each_value do |route|

data/lib/graphql/stitching/request.rb

@@ -147,6 +147,11 @@ module GraphQL
         @query.static_errors
       end
 
+      # @return [Boolean] is the request valid?
+      def valid?
+        validate.empty?
+      end
+
       # Gets and sets the query plan for the request. Assigned query plans may pull from a cache,
       # which is useful for redundant GraphQL documents (commonly sent by frontend clients).
       # ```ruby

data/lib/graphql/stitching/supergraph/from_definition.rb

@@ -21,11 +21,8 @@ module GraphQL::Stitching
         field_map = {}
         resolver_map = {}
         possible_locations = {}
-        visibility_profiles = if (visibility_def = schema.directives[GraphQL::Stitching.visibility_directive])
-          visibility_def.get_argument("profiles").default_value
-        else
-          []
-        end
+        visibility_definition = schema.directives[GraphQL::Stitching.visibility_directive]
+        visibility_profiles = visibility_definition&.get_argument("profiles")&.default_value || EMPTY_ARRAY
 
         schema.types.each do |type_name, type|
           next if type.introspection?
@@ -75,7 +72,7 @@ module GraphQL::Stitching
           end
         end
 
-        executables = possible_locations.keys.each_with_object({}) do |location, memo|
+        executables = possible_locations.each_key.each_with_object({}) do |location, memo|
           executable = executables[location] || executables[location.to_sym]
           if validate_executable!(location, executable)
             memo[location] = executable

data/lib/graphql/stitching/supergraph.rb

@@ -38,7 +38,7 @@ module GraphQL
         @locations_by_type_and_field = @memoized_introspection_types.each_with_object(fields) do |(type_name, type), memo|
           next unless type.kind.fields?
 
-          memo[type_name] = type.fields.keys.each_with_object({}) do |field_name, m|
+          memo[type_name] = type.fields.each_key.each_with_object({}) do |field_name, m|
             m[field_name] = [SUPERGRAPH_LOCATION]
           end
         end.freeze
@@ -75,7 +75,7 @@ module GraphQL
       end
 
       def locations
-        @executables.keys.reject { _1 == SUPERGRAPH_LOCATION }
+        @executables.each_key.reject { _1 == SUPERGRAPH_LOCATION }
       end
 
       def memoized_schema_fields(type_name)
@@ -130,7 +130,7 @@ module GraphQL
       # "Type" => ["location1", "location2", ...]
       def locations_by_type
         @locations_by_type ||= @locations_by_type_and_field.each_with_object({}) do |(type_name, fields), memo|
-          memo[type_name] = fields.values.flatten.uniq
+          memo[type_name] = fields.values.tap(&:flatten!).tap(&:uniq!)
         end
       end
 

data/lib/graphql/stitching/version.rb

@@ -2,6 +2,6 @@
 
 module GraphQL
   module Stitching
-    VERSION = "1.7.1"
+    VERSION = "1.7.2"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: graphql-stitching
 version: !ruby/object:Gem::Version
-  version: 1.7.1
+  version: 1.7.2
 platform: ruby
 authors:
 - Greg MacWilliam
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-05-10 00:00:00.000000000 Z
+date: 2025-05-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: graphql