refactor(Execute) remove unused Lazy code; mark @api private

Author: rmosolgo

lib/graphql/execution/directive_checks.rb

@@ -3,6 +3,7 @@ module GraphQL
   module Execution
     # Boolean checks for how an AST node's directives should
     # influence its execution
+    # @api private
     module DirectiveChecks
       SKIP = "skip"
       INCLUDE = "include"

lib/graphql/execution/execute.rb

@@ -2,6 +2,7 @@
 module GraphQL
   module Execution
     # A valid execution strategy
+    # @api private
     class Execute
       PROPAGATE_NULL = :__graphql_propagate_null__
 

lib/graphql/execution/field_result.rb

@@ -2,6 +2,7 @@
 module GraphQL
   module Execution
     # This is one key-value pair in a GraphQL response.
+    # @api private
     class FieldResult
       # @return [Any, Lazy] the GraphQL-ready response value, or a {Lazy} instance
       attr_reader :value

lib/graphql/execution/lazy.rb

@@ -10,6 +10,7 @@ module Execution
     # This is an itty-bitty promise-like object, with key differences:
     # - It has only two states, not-resolved and resolved
     # - It has no error-catching functionality
+    # @api private
     class Lazy
       # Traverse `val`, lazily resolving any values along the way
       # @param val [Object] A data structure containing mixed plain values and `Lazy` instances
@@ -19,28 +20,17 @@ def self.resolve(val)
       end
 
       # Create a {Lazy} which will get its inner value by calling the block
-      # @param target [Object]
-      # @param method_name [Symbol]
       # @param get_value_func [Proc] a block to get the inner value (later)
-      def initialize(target = nil, method_name = nil, &get_value_func)
-        if block_given?
-          @get_value_func = get_value_func
-        else
-          @target = target
-          @method_name = method_name
-        end
+      def initialize(&get_value_func)
+        @get_value_func = get_value_func
         @resolved = false
       end
 
       # @return [Object] The wrapped value, calling the lazy block if necessary
       def value
         if !@resolved
           @resolved = true
-          if @get_value_func
-            @value = @get_value_func.call
-          else
-            @value = @target.public_send(@method_name)
-          end
+          @value = @get_value_func.call
         end
         @value
       rescue GraphQL::ExecutionError => err

lib/graphql/execution/lazy/lazy_method_map.rb

@@ -5,6 +5,8 @@ class Lazy
       # {GraphQL::Schema} uses this to match returned values to lazy resolution methods.
       # Methods may be registered for classes, they apply to its subclasses also.
       # The result of this lookup is cached for future resolutions.
+      # @api private
+      # @see {Schema#lazy?} looks up values from this map
       class LazyMethodMap
         def initialize
           @storage = Hash.new do |h, value_class|

lib/graphql/execution/lazy/resolve.rb

@@ -3,6 +3,7 @@ module GraphQL
   module Execution
     class Lazy
       # Helpers for dealing with data structures containing {Lazy} instances
+      # @api private
       module Resolve
         # Mutate `value`, replacing {Lazy} instances in place with their resolved values
         # @return [void]

lib/graphql/execution/selection_result.rb

@@ -2,6 +2,7 @@
 module GraphQL
   module Execution
     # A set of key-value pairs suitable for a GraphQL response.
+    # @api private
     class SelectionResult
       def initialize
         @storage = {}

lib/graphql/execution/typecast.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 module GraphQL
   module Execution
+    # @api private
     module Typecast
       # @return [Boolean]
       def self.subtype?(parent_type, child_type)