Support attr readers for fields with names conflicting with Object and Kernel methods

lib/graphql/client/schema/object_type.rb

@@ -27,6 +27,9 @@ class WithDefinition
           include ObjectType
 
           EMPTY_SET = Set.new.freeze
+          BASE_RUBY_METHODS =
+            (Object.methods + Kernel.methods).uniq.filter { |m| m.to_s =~ /^[a-z_][a-z_0-9?]*$/ }
+            .map(&:to_s).to_set.freeze
 
           attr_reader :klass, :defined_fields, :definition
 
@@ -58,6 +61,9 @@ def initialize(klass, defined_fields, definition, spreads)
               name = ActiveSupport::Inflector.underscore(attr)
               @klass::READERS[:"#{name}"] ||= attr
               @klass::PREDICATES[:"#{name}?"] ||= attr
+
+              @klass.undef_method(name) if BASE_RUBY_METHODS.include?(name) && @klass.method_defined?(name)
+              @klass.undef_method("#{name}?") if BASE_RUBY_METHODS.include?("#{name}?") && @klass.method_defined?("#{name}?")
             end
           end
 

test/test_client.rb

@@ -838,4 +838,38 @@ def test_spread_definition_defined_by_other_client
       }
     GRAPHQL
   end
+
+  def test_field_names_that_match_ruby_built_in_methods
+    query_type = Class.new(GraphQL::Schema::Object) do
+      graphql_name "Query"
+      field :things, String, null: false
+      def things
+        "things"
+      end
+
+      field :method, String, null: false
+      def method
+        "method"
+      end
+
+      field :equal, String, null: false
+      def equal
+        "equal"
+      end
+    end
+
+    schema = Class.new(GraphQL::Schema) do
+      query query_type
+    end
+
+    @client = ::GraphQL::Client.new(schema: schema, execute: schema)
+
+    Temp.const_set :Query, @client.parse("{ method things equal }")
+
+    result = @client.query(Temp::Query)
+    assert_equal "method", result.data.method
+    assert_equal "things", result.data.things
+    assert_equal "equal", result.data.equal
+    assert_predicate result.data, :equal?
+  end
 end