graphql-stitching 1.0.4 → 1.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ccc67e1b52f1a0688d25958d1955580c7f88a8115dc6b6b6905f785da94089db
-  data.tar.gz: 8e0925fb1ad0584bce11672be24868dc261cf680f8f88acdff87a7ca3819d9f8
+  metadata.gz: 2d90338dfde5c26515a55afdee7f5bd936a4563b2b8f83e1ffc1a4420c56a5cf
+  data.tar.gz: d728c5dfabb5e0250c38a6718ac7b535cfa7a183872dc60fa95a9b91b777a64a
 SHA512:
-  metadata.gz: 9213dbd90d0fc0caf10a09b9ba68bfbfe072c3cce9d51c47623b0d5edabb9bfe1eac849fb3135bade04863c2ca9b62c42c2f67b469f1fc268f039fab15f17372
-  data.tar.gz: c62c51fda4c9210ba71c9be894e7a31fe3346c0e4d49ac57632efa32b8db9809174b6d0fb048eddc1437584c57d7f2bd748e4fab3f1e1440c2d5c6f3647657dd
+  metadata.gz: cc5ff912f354e08b9f204028d5e6eed85a81bf283bd5dc13bb7ee9b69b83d0bb39ba83f9fbaac66be1363e86ce3f6bd1b3f7cc4466ef83998a751e3dbccf9013
+  data.tar.gz: 055e1c09f88e8f893a3ebb0aa3d7558156ec5c4a33b704cca69499efee4fd02e3a70e6efedeca1332f92392c7093ea8024ba9aefb159ce77b4f11510cd8d7a1c

data/docs/composer.md

@@ -12,6 +12,7 @@ composer = GraphQL::Stitching::Composer.new(
   mutation_name: "Mutation",
   description_merger: ->(values_by_location, info) { values_by_location.values.join("\n") },
   deprecation_merger: ->(values_by_location, info) { values_by_location.values.first },
+  default_value_merger: ->(values_by_location, info) { values_by_location.values.first },
   directive_kwarg_merger: ->(values_by_location, info) { values_by_location.values.last },
   root_field_location_selector: ->(locations, info) { locations.last },
 )
@@ -27,6 +28,8 @@ Constructor arguments:
 
 - **`deprecation_merger:`** _optional_, a [value merger function](#value-merger-functions) for merging element deprecation strings from across locations.
 
+- **`default_value_merger:`** _optional_, a [value merger function](#value-merger-functions) for merging argument default values from across locations.
+
 - **`directive_kwarg_merger:`** _optional_, a [value merger function](#value-merger-functions) for merging directive keyword arguments from across locations.
 
 - **`root_field_location_selector:`** _optional_, selects a default routing location for root fields with multiple locations. Use this to prioritize sending root fields to their primary data sources (only applies while routing the root operation scope). This handler receives an array of possible locations and an info object with field information, and should return the prioritized location. The last location is used by default.

data/docs/executor.md

@@ -50,8 +50,8 @@ The raw result will contain many irregularities from the stitching process, howe
   "data" => {
     "product" => {
       "upc" => "1",
-      "_STITCH_upc" => "1",
-      "_STITCH_typename" => "Product",
+      "_export_upc" => "1",
+      "_export_typename" => "Product",
       "name" => "iPhone",
       "price" => nil,
     }

data/lib/graphql/stitching/composer.rb

@@ -5,9 +5,13 @@ module GraphQL
     class Composer
       class ComposerError < StitchingError; end
       class ValidationError < ComposerError; end
+      class ReferenceType < GraphQL::Schema::Object
+        field(:f, String) do
+          argument(:a, String)
+        end
+      end
 
-      attr_reader :query_name, :mutation_name, :candidate_types_by_name_and_location, :schema_directives
-
+      NO_DEFAULT_VALUE = ReferenceType.get_field("f").get_argument("a").default_value
       BASIC_VALUE_MERGER = ->(values_by_location, _info) { values_by_location.values.find { !_1.nil? } }
       BASIC_ROOT_FIELD_LOCATION_SELECTOR = ->(locations, _info) { locations.last }
 
@@ -16,6 +20,8 @@ module GraphQL
         "ValidateBoundaries",
       ].freeze
 
+      attr_reader :query_name, :mutation_name, :candidate_types_by_name_and_location, :schema_directives
+
       def initialize(
         query_name: "Query",
         mutation_name: "Mutation",
@@ -195,7 +201,7 @@ module GraphQL
           description(builder.merge_descriptions(directive_name, directives_by_location))
           repeatable(directives_by_location.values.any?(&:repeatable?))
           locations(*directives_by_location.values.flat_map(&:locations).uniq)
-          builder.build_merged_arguments(directive_name, directives_by_location, self)
+          builder.build_merged_arguments(directive_name, directives_by_location, self, directive_name: directive_name)
         end
       end
 
@@ -343,7 +349,7 @@ module GraphQL
         end
       end
 
-      def build_merged_arguments(type_name, members_by_location, owner, field_name: nil)
+      def build_merged_arguments(type_name, members_by_location, owner, field_name: nil, directive_name: nil)
         # "argument_name" => "location" => argument
         args_by_name_location = members_by_location.each_with_object({}) do |(location, member_candidate), memo|
           member_candidate.arguments.each do |argument_name, argument|
@@ -369,7 +375,7 @@ module GraphQL
 
           kwargs = {}
           default_values_by_location = arguments_by_location.each_with_object({}) do |(location, argument), memo|
-            next if argument.default_value.class == Object # << pass on NOT_CONFIGURED (todo: improve this check)
+            next if argument.default_value == NO_DEFAULT_VALUE
             memo[location] = argument.default_value
           end
 
@@ -378,7 +384,8 @@ module GraphQL
               type_name: type_name,
               field_name: field_name,
               argument_name: argument_name,
-            })
+              directive_name: directive_name,
+            }.tap(&:compact!))
           end
 
           type = merge_value_types(type_name, value_types, argument_name: argument_name, field_name: field_name)
@@ -430,7 +437,7 @@ module GraphQL
               enum_value: enum_value,
               directive_name: directive_name,
               kwarg_name: kwarg_name,
-            }.compact!)
+            }.tap(&:compact!))
           end
 
           owner.directive(directive_class, **kwargs)
@@ -438,7 +445,7 @@ module GraphQL
       end
 
       def merge_value_types(type_name, type_candidates, field_name: nil, argument_name: nil)
-        path = [type_name, field_name, argument_name].compact.join(".")
+        path = [type_name, field_name, argument_name].tap(&:compact!).join(".")
         alt_structures = type_candidates.map { Util.flatten_type_structure(_1) }
         basis_structure = alt_structures.shift
 
@@ -475,7 +482,7 @@ module GraphQL
           field_name: field_name,
           argument_name: argument_name,
           enum_value: enum_value,
-        }.compact!)
+        }.tap(&:compact!))
       end
 
       def merge_deprecations(type_name, members_by_location, field_name: nil, argument_name: nil, enum_value: nil)
@@ -485,7 +492,7 @@ module GraphQL
           field_name: field_name,
           argument_name: argument_name,
           enum_value: enum_value,
-        }.compact!)
+        }.tap(&:compact!))
       end
 
       def extract_boundaries(type_name, types_by_location)

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

@@ -16,7 +16,7 @@ module GraphQL
 
           if op.if_type
             # operations planned around unused fragment conditions should not trigger requests
-            origin_set.select! { _1[SelectionHint.typename_node.alias] == op.if_type }
+            origin_set.select! { _1[ExportSelection.typename_node.alias] == op.if_type }
           end
 
           memo[op] = origin_set if origin_set.any?
@@ -94,9 +94,9 @@ module GraphQL
       end
 
       def build_key(key, origin_obj, federation: false)
-        key_value = JSON.generate(origin_obj[SelectionHint.key(key)])
+        key_value = JSON.generate(origin_obj[ExportSelection.key(key)])
         if federation
-          "{ __typename: \"#{origin_obj[SelectionHint.typename_node.alias]}\", #{key}: #{key_value} }"
+          "{ __typename: \"#{origin_obj[ExportSelection.typename_node.alias]}\", #{key}: #{key_value} }"
         else
           key_value
         end

data/lib/graphql/stitching/{selection_hint.rb → export_selection.rb}

@@ -4,18 +4,18 @@ module GraphQL
   module Stitching
     # Builds hidden selection fields added by stitiching code,
     # used to request operational data about resolved objects.
-    class SelectionHint
-      HINT_PREFIX = "_STITCH_"
+    class ExportSelection
+      EXPORT_PREFIX = "_export_"
 
       class << self
         def key?(name)
           return false unless name
 
-          name.start_with?(HINT_PREFIX)
+          name.start_with?(EXPORT_PREFIX)
         end
 
         def key(name)
-          "#{HINT_PREFIX}#{name}"
+          "#{EXPORT_PREFIX}#{name}"
         end
 
         def key_node(field_name)

data/lib/graphql/stitching/planner.rb

@@ -42,7 +42,7 @@ module GraphQL
       #      Adjoining selections not available here get split off into new entrypoints (C).
       # B.3) Collect all variable definitions used within the filtered selection.
       #      These specify which request variables to pass along with each step.
-      # B.4) Add a `__typename` hint to abstracts and types that implement fragments.
+      # B.4) Add a `__typename` export to abstracts and types that implement fragments.
       #      This provides resolved type information used during execution.
       #
       # C) Delegate adjoining selections to new entrypoint locations.
@@ -199,7 +199,9 @@ module GraphQL
         input_selections.each do |node|
           case node
           when GraphQL::Language::Nodes::Field
-            if node.name == TYPENAME
+            if node.alias&.start_with?(ExportSelection::EXPORT_PREFIX)
+              raise StitchingError, %(Alias "#{node.alias}" is not allowed because "#{ExportSelection::EXPORT_PREFIX}" is a reserved prefix.)
+            elsif node.name == TYPENAME
               locale_selections << node
               next
             end
@@ -257,10 +259,10 @@ module GraphQL
           end
         end
 
-        # B.4) Add a `__typename` hint to abstracts and types that implement
+        # B.4) Add a `__typename` export to abstracts and types that implement
         # fragments so that resolved type information is available during execution.
         if requires_typename
-          locale_selections << SelectionHint.typename_node
+          locale_selections << ExportSelection.typename_node
         end
 
         if remote_selections
@@ -274,18 +276,18 @@ module GraphQL
           routes.each_value do |route|
             route.reduce(locale_selections) do |parent_selections, boundary|
               # E.1) Add the key of each boundary query into the prior location's selection set.
-              foreign_key = SelectionHint.key(boundary.key)
+              foreign_key = ExportSelection.key(boundary.key)
               has_key = false
               has_typename = false
 
               parent_selections.each do |node|
                 next unless node.is_a?(GraphQL::Language::Nodes::Field)
                 has_key ||= node.alias == foreign_key
-                has_typename ||= node.alias == SelectionHint.typename_node.alias
+                has_typename ||= node.alias == ExportSelection.typename_node.alias
               end
 
-              parent_selections << SelectionHint.key_node(boundary.key) unless has_key
-              parent_selections << SelectionHint.typename_node unless has_typename
+              parent_selections << ExportSelection.key_node(boundary.key) unless has_key
+              parent_selections << ExportSelection.typename_node unless has_typename
 
               # E.2) Add a planner step for each new entrypoint location.
               add_step(

data/lib/graphql/stitching/request.rb

@@ -69,7 +69,7 @@ module GraphQL
 
       def prepare!
         operation.variables.each do |v|
-          @variables[v.name] ||= v.default_value
+          @variables[v.name] = v.default_value if @variables[v.name].nil?
         end
 
         if @may_contain_runtime_directives

data/lib/graphql/stitching/shaper.rb

@@ -4,7 +4,7 @@
 module GraphQL
   module Stitching
     # Shapes the final results payload to the request selection and schema definition.
-    # This eliminates unrequested selection hints and applies null bubbling.
+    # This eliminates unrequested export selections and applies null bubbling.
     class Shaper
       def initialize(supergraph:, request:)
         @supergraph = supergraph
@@ -21,8 +21,8 @@ module GraphQL
       def resolve_object_scope(raw_object, parent_type, selections, typename = nil)
         return nil if raw_object.nil?
 
-        typename ||= raw_object[SelectionHint.typename_node.alias]
-        raw_object.reject! { |key, _v| SelectionHint.key?(key) }
+        typename ||= raw_object[ExportSelection.typename_node.alias]
+        raw_object.reject! { |key, _v| ExportSelection.key?(key) }
 
         selections.each do |node|
           case node

data/lib/graphql/stitching/skip_include.rb

@@ -35,7 +35,7 @@ module GraphQL
           end
 
           if filtered_selections.none?
-            filtered_selections << SelectionHint.typename_node
+            filtered_selections << ExportSelection.typename_node
           end
 
           if changed

data/lib/graphql/stitching/version.rb

@@ -2,6 +2,6 @@
 
 module GraphQL
   module Stitching
-    VERSION = "1.0.4"
+    VERSION = "1.0.5"
   end
 end

data/lib/graphql/stitching.rb

@@ -28,12 +28,12 @@ require_relative "stitching/boundary"
 require_relative "stitching/client"
 require_relative "stitching/composer"
 require_relative "stitching/executor"
+require_relative "stitching/export_selection"
 require_relative "stitching/http_executable"
 require_relative "stitching/plan"
 require_relative "stitching/planner_step"
 require_relative "stitching/planner"
 require_relative "stitching/request"
-require_relative "stitching/selection_hint"
 require_relative "stitching/shaper"
 require_relative "stitching/skip_include"
 require_relative "stitching/util"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: graphql-stitching
 version: !ruby/object:Gem::Version
-  version: 1.0.4
+  version: 1.0.5
 platform: ruby
 authors:
 - Greg MacWilliam
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-11-03 00:00:00.000000000 Z
+date: 2023-11-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: graphql
@@ -106,12 +106,12 @@ files:
 - lib/graphql/stitching/executor.rb
 - lib/graphql/stitching/executor/boundary_source.rb
 - lib/graphql/stitching/executor/root_source.rb
+- lib/graphql/stitching/export_selection.rb
 - lib/graphql/stitching/http_executable.rb
 - lib/graphql/stitching/plan.rb
 - lib/graphql/stitching/planner.rb
 - lib/graphql/stitching/planner_step.rb
 - lib/graphql/stitching/request.rb
-- lib/graphql/stitching/selection_hint.rb
 - lib/graphql/stitching/shaper.rb
 - lib/graphql/stitching/skip_include.rb
 - lib/graphql/stitching/supergraph.rb