-
Notifications
You must be signed in to change notification settings - Fork 1.4k
/
default_required_true.rb
43 lines (40 loc) · 1.35 KB
/
default_required_true.rb
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
# frozen_string_literal: true
require_relative "./base_cop"
module GraphQL
module Rubocop
module GraphQL
# Identify (and auto-correct) any argument configuration which duplicates
# the default `required: true` property.
#
# `required: true` is default because required arguments can always be converted
# to optional arguments (`required: false`) without a breaking change. (The opposite change, from `required: false`
# to `required: true`, change.)
#
# @example
# # Both of these define `id: ID!` in GraphQL:
#
# # bad
# argument :id, ID, required: true
#
# # good
# argument :id, ID
#
class DefaultRequiredTrue < BaseCop
MSG = "`required: true` is the default and can be removed."
def_node_matcher :argument_config_with_required_true?, <<-Pattern
(
send {nil? _} :argument ... (hash <$(pair (sym :required) (true)) ...>)
)
Pattern
def on_send(node)
argument_config_with_required_true?(node) do |required_config|
add_offense(required_config) do |corrector|
cleaned_node_source = source_without_keyword_argument(node, required_config)
corrector.replace(node, cleaned_node_source)
end
end
end
end
end
end
end