rubocop 1.8.0 → 1.11.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (246) hide show
  1. checksums.yaml +4 -4
  2. data/README.md +14 -13
  3. data/assets/output.html.erb +1 -1
  4. data/config/default.yml +89 -22
  5. data/config/obsoletion.yml +4 -0
  6. data/lib/rubocop.rb +9 -0
  7. data/lib/rubocop/cli/command/auto_genenerate_config.rb +5 -4
  8. data/lib/rubocop/cli/command/execute_runner.rb +1 -1
  9. data/lib/rubocop/cli/command/suggest_extensions.rb +1 -1
  10. data/lib/rubocop/config.rb +5 -2
  11. data/lib/rubocop/config_loader.rb +7 -14
  12. data/lib/rubocop/config_store.rb +12 -1
  13. data/lib/rubocop/cop/base.rb +2 -1
  14. data/lib/rubocop/cop/bundler/duplicated_gem.rb +2 -1
  15. data/lib/rubocop/cop/bundler/gem_comment.rb +1 -0
  16. data/lib/rubocop/cop/bundler/insecure_protocol_source.rb +1 -0
  17. data/lib/rubocop/cop/bundler/ordered_gems.rb +1 -0
  18. data/lib/rubocop/cop/exclude_limit.rb +26 -0
  19. data/lib/rubocop/cop/gemspec/date_assignment.rb +57 -0
  20. data/lib/rubocop/cop/gemspec/duplicated_assignment.rb +2 -0
  21. data/lib/rubocop/cop/gemspec/ordered_dependencies.rb +1 -0
  22. data/lib/rubocop/cop/gemspec/required_ruby_version.rb +2 -0
  23. data/lib/rubocop/cop/gemspec/ruby_version_globals_usage.rb +2 -0
  24. data/lib/rubocop/cop/generator.rb +3 -5
  25. data/lib/rubocop/cop/internal_affairs.rb +6 -1
  26. data/lib/rubocop/cop/internal_affairs/empty_line_between_expect_offense_and_correction.rb +68 -0
  27. data/lib/rubocop/cop/internal_affairs/example_description.rb +90 -0
  28. data/lib/rubocop/cop/internal_affairs/method_name_equal.rb +1 -0
  29. data/lib/rubocop/cop/internal_affairs/node_destructuring.rb +2 -0
  30. data/lib/rubocop/cop/internal_affairs/node_matcher_directive.rb +151 -0
  31. data/lib/rubocop/cop/internal_affairs/node_type_predicate.rb +1 -0
  32. data/lib/rubocop/cop/internal_affairs/offense_location_keyword.rb +2 -0
  33. data/lib/rubocop/cop/internal_affairs/redundant_described_class_as_subject.rb +62 -0
  34. data/lib/rubocop/cop/internal_affairs/redundant_let_rubocop_config_new.rb +65 -0
  35. data/lib/rubocop/cop/internal_affairs/redundant_location_argument.rb +1 -0
  36. data/lib/rubocop/cop/internal_affairs/redundant_message_argument.rb +3 -0
  37. data/lib/rubocop/cop/internal_affairs/style_detected_api_use.rb +4 -0
  38. data/lib/rubocop/cop/internal_affairs/useless_message_assertion.rb +2 -0
  39. data/lib/rubocop/cop/layout/block_alignment.rb +1 -0
  40. data/lib/rubocop/cop/layout/class_structure.rb +8 -2
  41. data/lib/rubocop/cop/layout/empty_line_between_defs.rb +37 -17
  42. data/lib/rubocop/cop/layout/extra_spacing.rb +2 -2
  43. data/lib/rubocop/cop/layout/first_argument_indentation.rb +22 -3
  44. data/lib/rubocop/cop/layout/indentation_width.rb +1 -0
  45. data/lib/rubocop/cop/layout/line_length.rb +2 -1
  46. data/lib/rubocop/cop/layout/multiline_assignment_layout.rb +26 -0
  47. data/lib/rubocop/cop/layout/space_before_block_braces.rb +1 -1
  48. data/lib/rubocop/cop/layout/space_before_brackets.rb +9 -4
  49. data/lib/rubocop/cop/layout/space_inside_block_braces.rb +1 -1
  50. data/lib/rubocop/cop/lint/big_decimal_new.rb +1 -0
  51. data/lib/rubocop/cop/lint/boolean_symbol.rb +1 -0
  52. data/lib/rubocop/cop/lint/constant_definition_in_block.rb +2 -0
  53. data/lib/rubocop/cop/lint/constant_resolution.rb +1 -0
  54. data/lib/rubocop/cop/lint/debugger.rb +60 -14
  55. data/lib/rubocop/cop/lint/deprecated_constants.rb +5 -0
  56. data/lib/rubocop/cop/lint/deprecated_open_ssl_constant.rb +14 -4
  57. data/lib/rubocop/cop/lint/duplicate_branch.rb +1 -1
  58. data/lib/rubocop/cop/lint/duplicate_methods.rb +3 -0
  59. data/lib/rubocop/cop/lint/duplicate_require.rb +3 -2
  60. data/lib/rubocop/cop/lint/each_with_object_argument.rb +1 -0
  61. data/lib/rubocop/cop/lint/else_layout.rb +1 -1
  62. data/lib/rubocop/cop/lint/erb_new_arguments.rb +1 -0
  63. data/lib/rubocop/cop/lint/format_parameter_mismatch.rb +1 -0
  64. data/lib/rubocop/cop/lint/hash_compare_by_identity.rb +1 -0
  65. data/lib/rubocop/cop/lint/ineffective_access_modifier.rb +1 -0
  66. data/lib/rubocop/cop/lint/inherit_exception.rb +1 -0
  67. data/lib/rubocop/cop/lint/multiple_comparison.rb +5 -4
  68. data/lib/rubocop/cop/lint/nested_method_definition.rb +3 -0
  69. data/lib/rubocop/cop/lint/next_without_accumulator.rb +1 -0
  70. data/lib/rubocop/cop/lint/non_deterministic_require_order.rb +7 -0
  71. data/lib/rubocop/cop/lint/non_local_exit_from_iterator.rb +3 -0
  72. data/lib/rubocop/cop/lint/number_conversion.rb +43 -6
  73. data/lib/rubocop/cop/lint/numbered_parameter_assignment.rb +47 -0
  74. data/lib/rubocop/cop/lint/or_assignment_to_constant.rb +39 -0
  75. data/lib/rubocop/cop/lint/raise_exception.rb +2 -0
  76. data/lib/rubocop/cop/lint/rand_one.rb +1 -0
  77. data/lib/rubocop/cop/lint/redundant_dir_glob_sort.rb +5 -3
  78. data/lib/rubocop/cop/lint/redundant_require_statement.rb +1 -0
  79. data/lib/rubocop/cop/lint/redundant_safe_navigation.rb +1 -0
  80. data/lib/rubocop/cop/lint/redundant_splat_expansion.rb +7 -3
  81. data/lib/rubocop/cop/lint/redundant_string_coercion.rb +1 -0
  82. data/lib/rubocop/cop/lint/redundant_with_index.rb +1 -0
  83. data/lib/rubocop/cop/lint/redundant_with_object.rb +1 -0
  84. data/lib/rubocop/cop/lint/safe_navigation_chain.rb +1 -0
  85. data/lib/rubocop/cop/lint/safe_navigation_with_empty.rb +1 -0
  86. data/lib/rubocop/cop/lint/send_with_mixin_argument.rb +1 -0
  87. data/lib/rubocop/cop/lint/shadowed_argument.rb +1 -0
  88. data/lib/rubocop/cop/lint/shadowing_outer_local_variable.rb +1 -0
  89. data/lib/rubocop/cop/lint/struct_new_override.rb +1 -0
  90. data/lib/rubocop/cop/lint/symbol_conversion.rb +103 -0
  91. data/lib/rubocop/cop/lint/to_enum_arguments.rb +3 -0
  92. data/lib/rubocop/cop/lint/triple_quotes.rb +71 -0
  93. data/lib/rubocop/cop/lint/unified_integer.rb +1 -0
  94. data/lib/rubocop/cop/lint/unmodified_reduce_accumulator.rb +5 -0
  95. data/lib/rubocop/cop/lint/unreachable_code.rb +1 -0
  96. data/lib/rubocop/cop/lint/unreachable_loop.rb +1 -0
  97. data/lib/rubocop/cop/lint/unused_method_argument.rb +1 -0
  98. data/lib/rubocop/cop/lint/uri_escape_unescape.rb +1 -0
  99. data/lib/rubocop/cop/lint/useless_access_modifier.rb +4 -0
  100. data/lib/rubocop/cop/lint/useless_setter_call.rb +1 -0
  101. data/lib/rubocop/cop/lint/useless_times.rb +3 -0
  102. data/lib/rubocop/cop/message_annotator.rb +4 -1
  103. data/lib/rubocop/cop/metrics/block_nesting.rb +2 -2
  104. data/lib/rubocop/cop/metrics/module_length.rb +1 -0
  105. data/lib/rubocop/cop/metrics/parameter_lists.rb +6 -2
  106. data/lib/rubocop/cop/metrics/utils/code_length_calculator.rb +6 -4
  107. data/lib/rubocop/cop/metrics/utils/repeated_attribute_discount.rb +2 -0
  108. data/lib/rubocop/cop/mixin/check_line_breakable.rb +5 -0
  109. data/lib/rubocop/cop/mixin/code_length.rb +3 -1
  110. data/lib/rubocop/cop/mixin/comments_help.rb +0 -1
  111. data/lib/rubocop/cop/mixin/configurable_max.rb +1 -0
  112. data/lib/rubocop/cop/mixin/def_node.rb +1 -0
  113. data/lib/rubocop/cop/mixin/empty_lines_around_body.rb +3 -0
  114. data/lib/rubocop/cop/mixin/empty_parameter.rb +1 -0
  115. data/lib/rubocop/cop/mixin/enforce_superclass.rb +2 -0
  116. data/lib/rubocop/cop/mixin/hash_transform_method.rb +1 -0
  117. data/lib/rubocop/cop/mixin/method_complexity.rb +4 -1
  118. data/lib/rubocop/cop/mixin/negative_conditional.rb +3 -0
  119. data/lib/rubocop/cop/mixin/preferred_delimiters.rb +3 -3
  120. data/lib/rubocop/cop/mixin/rational_literal.rb +1 -0
  121. data/lib/rubocop/cop/mixin/safe_assignment.rb +5 -0
  122. data/lib/rubocop/cop/mixin/uncommunicative_name.rb +5 -1
  123. data/lib/rubocop/cop/mixin/visibility_help.rb +1 -0
  124. data/lib/rubocop/cop/naming/binary_operator_parameter_name.rb +1 -0
  125. data/lib/rubocop/cop/naming/constant_name.rb +2 -0
  126. data/lib/rubocop/cop/naming/memoized_instance_variable_name.rb +2 -0
  127. data/lib/rubocop/cop/naming/method_name.rb +3 -0
  128. data/lib/rubocop/cop/naming/predicate_name.rb +1 -0
  129. data/lib/rubocop/cop/naming/rescued_exceptions_variable_name.rb +38 -5
  130. data/lib/rubocop/cop/naming/variable_number.rb +1 -1
  131. data/lib/rubocop/cop/registry.rb +1 -1
  132. data/lib/rubocop/cop/security/eval.rb +1 -0
  133. data/lib/rubocop/cop/security/json_load.rb +1 -0
  134. data/lib/rubocop/cop/security/marshal_load.rb +1 -0
  135. data/lib/rubocop/cop/security/open.rb +1 -0
  136. data/lib/rubocop/cop/security/yaml_load.rb +1 -0
  137. data/lib/rubocop/cop/severity.rb +3 -3
  138. data/lib/rubocop/cop/style/access_modifier_declarations.rb +1 -0
  139. data/lib/rubocop/cop/style/alias.rb +1 -0
  140. data/lib/rubocop/cop/style/arguments_forwarding.rb +3 -0
  141. data/lib/rubocop/cop/style/array_coercion.rb +2 -0
  142. data/lib/rubocop/cop/style/array_join.rb +1 -0
  143. data/lib/rubocop/cop/style/ascii_comments.rb +1 -1
  144. data/lib/rubocop/cop/style/attr.rb +1 -0
  145. data/lib/rubocop/cop/style/case_equality.rb +2 -1
  146. data/lib/rubocop/cop/style/class_equality_comparison.rb +1 -0
  147. data/lib/rubocop/cop/style/collection_compact.rb +2 -0
  148. data/lib/rubocop/cop/style/colon_method_call.rb +1 -0
  149. data/lib/rubocop/cop/style/command_literal.rb +1 -1
  150. data/lib/rubocop/cop/style/conditional_assignment.rb +2 -0
  151. data/lib/rubocop/cop/style/constant_visibility.rb +28 -0
  152. data/lib/rubocop/cop/style/date_time.rb +3 -0
  153. data/lib/rubocop/cop/style/dir.rb +1 -0
  154. data/lib/rubocop/cop/style/disable_cops_within_source_code_directive.rb +49 -9
  155. data/lib/rubocop/cop/style/documentation.rb +5 -0
  156. data/lib/rubocop/cop/style/documentation_method.rb +1 -0
  157. data/lib/rubocop/cop/style/double_negation.rb +3 -2
  158. data/lib/rubocop/cop/style/each_for_simple_loop.rb +1 -0
  159. data/lib/rubocop/cop/style/each_with_object.rb +1 -0
  160. data/lib/rubocop/cop/style/empty_literal.rb +9 -0
  161. data/lib/rubocop/cop/style/endless_method.rb +1 -0
  162. data/lib/rubocop/cop/style/eval_with_location.rb +140 -49
  163. data/lib/rubocop/cop/style/even_odd.rb +1 -0
  164. data/lib/rubocop/cop/style/expand_path_arguments.rb +3 -0
  165. data/lib/rubocop/cop/style/explicit_block_argument.rb +12 -1
  166. data/lib/rubocop/cop/style/exponential_notation.rb +6 -7
  167. data/lib/rubocop/cop/style/float_division.rb +7 -0
  168. data/lib/rubocop/cop/style/format_string.rb +2 -0
  169. data/lib/rubocop/cop/style/format_string_token.rb +19 -2
  170. data/lib/rubocop/cop/style/global_std_stream.rb +1 -0
  171. data/lib/rubocop/cop/style/hash_conversion.rb +105 -0
  172. data/lib/rubocop/cop/style/hash_each_methods.rb +1 -0
  173. data/lib/rubocop/cop/style/hash_except.rb +1 -0
  174. data/lib/rubocop/cop/style/hash_like_case.rb +1 -0
  175. data/lib/rubocop/cop/style/hash_transform_keys.rb +4 -0
  176. data/lib/rubocop/cop/style/hash_transform_values.rb +4 -0
  177. data/lib/rubocop/cop/style/if_inside_else.rb +14 -7
  178. data/lib/rubocop/cop/style/if_with_boolean_literal_branches.rb +122 -0
  179. data/lib/rubocop/cop/style/implicit_runtime_error.rb +1 -0
  180. data/lib/rubocop/cop/style/inverse_methods.rb +2 -0
  181. data/lib/rubocop/cop/style/min_max.rb +1 -0
  182. data/lib/rubocop/cop/style/mixin_usage.rb +2 -0
  183. data/lib/rubocop/cop/style/module_function.rb +5 -0
  184. data/lib/rubocop/cop/style/multiple_comparison.rb +21 -2
  185. data/lib/rubocop/cop/style/mutable_constant.rb +3 -0
  186. data/lib/rubocop/cop/style/negated_if_else_condition.rb +1 -0
  187. data/lib/rubocop/cop/style/nil_comparison.rb +6 -0
  188. data/lib/rubocop/cop/style/nil_lambda.rb +1 -0
  189. data/lib/rubocop/cop/style/non_nil_check.rb +30 -13
  190. data/lib/rubocop/cop/style/numeric_literals.rb +6 -9
  191. data/lib/rubocop/cop/style/numeric_predicate.rb +4 -1
  192. data/lib/rubocop/cop/style/option_hash.rb +1 -0
  193. data/lib/rubocop/cop/style/or_assignment.rb +2 -0
  194. data/lib/rubocop/cop/style/parallel_assignment.rb +6 -0
  195. data/lib/rubocop/cop/style/parentheses_around_condition.rb +1 -0
  196. data/lib/rubocop/cop/style/proc.rb +1 -0
  197. data/lib/rubocop/cop/style/random_with_offset.rb +5 -0
  198. data/lib/rubocop/cop/style/redundant_assignment.rb +1 -0
  199. data/lib/rubocop/cop/style/redundant_begin.rb +7 -1
  200. data/lib/rubocop/cop/style/redundant_conditional.rb +2 -0
  201. data/lib/rubocop/cop/style/redundant_exception.rb +2 -0
  202. data/lib/rubocop/cop/style/redundant_fetch_block.rb +2 -0
  203. data/lib/rubocop/cop/style/redundant_file_extension_in_require.rb +1 -0
  204. data/lib/rubocop/cop/style/redundant_freeze.rb +1 -0
  205. data/lib/rubocop/cop/style/redundant_parentheses.rb +13 -0
  206. data/lib/rubocop/cop/style/redundant_self_assignment.rb +2 -0
  207. data/lib/rubocop/cop/style/redundant_sort.rb +1 -0
  208. data/lib/rubocop/cop/style/redundant_sort_by.rb +1 -0
  209. data/lib/rubocop/cop/style/regexp_literal.rb +1 -1
  210. data/lib/rubocop/cop/style/rescue_standard_error.rb +2 -0
  211. data/lib/rubocop/cop/style/return_nil.rb +6 -0
  212. data/lib/rubocop/cop/style/safe_navigation.rb +2 -0
  213. data/lib/rubocop/cop/style/sample.rb +1 -0
  214. data/lib/rubocop/cop/style/signal_exception.rb +3 -0
  215. data/lib/rubocop/cop/style/single_argument_dig.rb +1 -0
  216. data/lib/rubocop/cop/style/single_line_methods.rb +5 -2
  217. data/lib/rubocop/cop/style/slicing_with_range.rb +1 -0
  218. data/lib/rubocop/cop/style/sole_nested_conditional.rb +28 -4
  219. data/lib/rubocop/cop/style/special_global_vars.rb +3 -3
  220. data/lib/rubocop/cop/style/stderr_puts.rb +1 -0
  221. data/lib/rubocop/cop/style/string_concatenation.rb +2 -1
  222. data/lib/rubocop/cop/style/string_hash_keys.rb +2 -0
  223. data/lib/rubocop/cop/style/strip.rb +1 -0
  224. data/lib/rubocop/cop/style/struct_inheritance.rb +1 -0
  225. data/lib/rubocop/cop/style/symbol_proc.rb +25 -1
  226. data/lib/rubocop/cop/style/ternary_parentheses.rb +2 -1
  227. data/lib/rubocop/cop/style/trailing_body_on_method_definition.rb +1 -0
  228. data/lib/rubocop/cop/style/trailing_method_end_statement.rb +1 -1
  229. data/lib/rubocop/cop/style/trivial_accessors.rb +1 -0
  230. data/lib/rubocop/cop/style/unless_logical_operators.rb +99 -0
  231. data/lib/rubocop/cop/style/unpack_first.rb +1 -0
  232. data/lib/rubocop/cop/style/while_until_modifier.rb +2 -4
  233. data/lib/rubocop/cop/style/yoda_condition.rb +1 -0
  234. data/lib/rubocop/cop/style/zero_length_predicate.rb +5 -0
  235. data/lib/rubocop/formatter/git_hub_actions_formatter.rb +1 -0
  236. data/lib/rubocop/formatter/offense_count_formatter.rb +1 -1
  237. data/lib/rubocop/formatter/simple_text_formatter.rb +2 -1
  238. data/lib/rubocop/formatter/worst_offenders_formatter.rb +1 -1
  239. data/lib/rubocop/magic_comment.rb +30 -1
  240. data/lib/rubocop/name_similarity.rb +1 -1
  241. data/lib/rubocop/options.rb +1 -1
  242. data/lib/rubocop/rspec/expect_offense.rb +5 -2
  243. data/lib/rubocop/runner.rb +1 -0
  244. data/lib/rubocop/target_ruby.rb +21 -13
  245. data/lib/rubocop/version.rb +2 -2
  246. metadata +21 -7
@@ -80,6 +80,7 @@ module RuboCop
80
80
 
81
81
  RESTRICT_ON_SEND = %i[new].freeze
82
82
 
83
+ # @!method erb_new_with_non_keyword_arguments(node)
83
84
  def_node_matcher :erb_new_with_non_keyword_arguments, <<~PATTERN
84
85
  (send
85
86
  (const {nil? cbase} :ERB) :new $...)
@@ -92,6 +92,7 @@ module RuboCop
92
92
  end
93
93
  end
94
94
 
95
+ # @!method called_on_string?(node)
95
96
  def_node_matcher :called_on_string?, <<~PATTERN
96
97
  {(send {nil? const_type?} _ (str _) ...)
97
98
  (send (str ...) ...)}
@@ -24,6 +24,7 @@ module RuboCop
24
24
 
25
25
  MSG = 'Use `Hash#compare_by_identity` instead of using `object_id` for keys.'
26
26
 
27
+ # @!method id_as_hash_key?(node)
27
28
  def_node_matcher :id_as_hash_key?, <<~PATTERN
28
29
  (send _ {:key? :has_key? :fetch :[] :[]=} (send _ :object_id) ...)
29
30
  PATTERN
@@ -53,6 +53,7 @@ module RuboCop
53
53
  ALTERNATIVE_PROTECTED = '`protected` inside a `class << self` ' \
54
54
  'block'
55
55
 
56
+ # @!method private_class_methods(node)
56
57
  def_node_search :private_class_methods, <<~PATTERN
57
58
  (send nil? :private_class_method $...)
58
59
  PATTERN
@@ -58,6 +58,7 @@ module RuboCop
58
58
 
59
59
  RESTRICT_ON_SEND = %i[new].freeze
60
60
 
61
+ # @!method class_new_call?(node)
61
62
  def_node_matcher :class_new_call?, <<~PATTERN
62
63
  (send
63
64
  (const {cbase nil?} :Class) :new
@@ -11,14 +11,10 @@ module RuboCop
11
11
  # @example
12
12
  #
13
13
  # # bad
14
- #
15
14
  # x < y < z
16
15
  # 10 <= x <= 20
17
16
  #
18
- # @example
19
- #
20
17
  # # good
21
- #
22
18
  # x < y && y < z
23
19
  # 10 <= x && x <= 20
24
20
  class MultipleComparison < Base
@@ -26,14 +22,19 @@ module RuboCop
26
22
 
27
23
  MSG = 'Use the `&&` operator to compare multiple values.'
28
24
  COMPARISON_METHODS = %i[< > <= >=].freeze
25
+ SET_OPERATION_OPERATORS = %i[& | ^].freeze
29
26
  RESTRICT_ON_SEND = COMPARISON_METHODS
30
27
 
28
+ # @!method multiple_compare?(node)
31
29
  def_node_matcher :multiple_compare?, <<~PATTERN
32
30
  (send (send _ {:< :> :<= :>=} $_) {:#{COMPARISON_METHODS.join(' :')}} _)
33
31
  PATTERN
34
32
 
35
33
  def on_send(node)
36
34
  return unless (center = multiple_compare?(node))
35
+ # It allows multiple comparison using `&`, `|`, and `^` set operation operators.
36
+ # e.g. `x >= y & y < z`
37
+ return if center.send_type? && SET_OPERATION_OPERATORS.include?(center.method_name)
37
38
 
38
39
  add_offense(node) do |corrector|
39
40
  new_center = "#{center.source} && #{center.source}"
@@ -81,14 +81,17 @@ module RuboCop
81
81
  class_or_module_or_struct_new_call?(child)
82
82
  end
83
83
 
84
+ # @!method eval_call?(node)
84
85
  def_node_matcher :eval_call?, <<~PATTERN
85
86
  (block (send _ {:instance_eval :class_eval :module_eval} ...) ...)
86
87
  PATTERN
87
88
 
89
+ # @!method exec_call?(node)
88
90
  def_node_matcher :exec_call?, <<~PATTERN
89
91
  (block (send _ {:instance_exec :class_exec :module_exec} ...) ...)
90
92
  PATTERN
91
93
 
94
+ # @!method class_or_module_or_struct_new_call?(node)
92
95
  def_node_matcher :class_or_module_or_struct_new_call?, <<~PATTERN
93
96
  (block (send (const {nil? cbase} {:Class :Module :Struct}) :new ...) ...)
94
97
  PATTERN
@@ -25,6 +25,7 @@ module RuboCop
25
25
  class NextWithoutAccumulator < Base
26
26
  MSG = 'Use `next` with an accumulator argument in a `reduce`.'
27
27
 
28
+ # @!method on_body_of_reduce(node)
28
29
  def_node_matcher :on_body_of_reduce, <<~PATTERN
29
30
  (block (send _recv {:reduce :inject} !sym) _blockargs $(begin ...))
30
31
  PATTERN
@@ -129,32 +129,39 @@ module RuboCop
129
129
  unsorted_dir_glob_pass?(node) || unsorted_dir_each_pass?(node)
130
130
  end
131
131
 
132
+ # @!method unsorted_dir_block?(node)
132
133
  def_node_matcher :unsorted_dir_block?, <<~PATTERN
133
134
  (send (const {nil? cbase} :Dir) :glob ...)
134
135
  PATTERN
135
136
 
137
+ # @!method unsorted_dir_each?(node)
136
138
  def_node_matcher :unsorted_dir_each?, <<~PATTERN
137
139
  (send (send (const {nil? cbase} :Dir) {:[] :glob} ...) :each)
138
140
  PATTERN
139
141
 
142
+ # @!method method_require?(node)
140
143
  def_node_matcher :method_require?, <<~PATTERN
141
144
  (block-pass (send nil? :method (sym :require)))
142
145
  PATTERN
143
146
 
147
+ # @!method unsorted_dir_glob_pass?(node)
144
148
  def_node_matcher :unsorted_dir_glob_pass?, <<~PATTERN
145
149
  (send (const {nil? cbase} :Dir) :glob ...
146
150
  (block-pass (send nil? :method (sym :require))))
147
151
  PATTERN
148
152
 
153
+ # @!method unsorted_dir_each_pass?(node)
149
154
  def_node_matcher :unsorted_dir_each_pass?, <<~PATTERN
150
155
  (send (send (const {nil? cbase} :Dir) {:[] :glob} ...) :each
151
156
  (block-pass (send nil? :method (sym :require))))
152
157
  PATTERN
153
158
 
159
+ # @!method loop_variable(node)
154
160
  def_node_matcher :loop_variable, <<~PATTERN
155
161
  (args (arg $_))
156
162
  PATTERN
157
163
 
164
+ # @!method var_is_required?(node, name)
158
165
  def_node_search :var_is_required?, <<~PATTERN
159
166
  (send nil? :require (lvar %1))
160
167
  PATTERN
@@ -73,7 +73,10 @@ module RuboCop
73
73
  !return_node.children.empty?
74
74
  end
75
75
 
76
+ # @!method chained_send?(node)
76
77
  def_node_matcher :chained_send?, '(send !nil? ...)'
78
+
79
+ # @!method define_method?(node)
77
80
  def_node_matcher :define_method?, <<~PATTERN
78
81
  (send _ {:define_method :define_singleton_method} _)
79
82
  PATTERN
@@ -25,12 +25,18 @@ module RuboCop
25
25
  # '10'.to_i
26
26
  # '10.2'.to_f
27
27
  # '10'.to_c
28
+ # ['1', '2', '3'].map(&:to_i)
29
+ # foo.try(:to_f)
30
+ # bar.send(:to_c)
28
31
  #
29
32
  # # good
30
33
  #
31
34
  # Integer('10', 10)
32
35
  # Float('10.2')
33
36
  # Complex('10')
37
+ # ['1', '2', '3'].map { |i| Integer(i, 10) }
38
+ # foo.try { |i| Float(i) }
39
+ # bar.send { |i| Complex(i) }
34
40
  #
35
41
  # @example IgnoredMethods: [minutes]
36
42
  #
@@ -52,22 +58,35 @@ module RuboCop
52
58
  }.freeze
53
59
  MSG = 'Replace unsafe number conversion with number '\
54
60
  'class parsing, instead of using '\
55
- '%<number_object>s.%<to_method>s, use stricter '\
61
+ '%<current>s, use stricter '\
56
62
  '%<corrected_method>s.'
57
- RESTRICT_ON_SEND = CONVERSION_METHOD_CLASS_MAPPING.keys.freeze
63
+ METHODS = CONVERSION_METHOD_CLASS_MAPPING.keys.map(&:inspect).join(' ')
58
64
 
65
+ # @!method to_method(node)
59
66
  def_node_matcher :to_method, <<~PATTERN
60
- (send $_ ${:to_i :to_f :to_c})
67
+ (send $_ ${#{METHODS}})
68
+ PATTERN
69
+
70
+ # @!method to_method_symbol(node)
71
+ def_node_matcher :to_method_symbol, <<~PATTERN
72
+ {(send _ $_ ${(sym ${#{METHODS}})} ...)
73
+ (send _ $_ ${(block_pass (sym ${#{METHODS}}))} ...)}
61
74
  PATTERN
62
75
 
63
76
  def on_send(node)
77
+ handle_conversion_method(node)
78
+ handle_as_symbol(node)
79
+ end
80
+
81
+ private
82
+
83
+ def handle_conversion_method(node)
64
84
  to_method(node) do |receiver, to_method|
65
85
  next if receiver.nil? || ignore_receiver?(receiver)
66
86
 
67
87
  message = format(
68
88
  MSG,
69
- number_object: receiver.source,
70
- to_method: to_method,
89
+ current: "#{receiver.source}.#{to_method}",
71
90
  corrected_method: correct_method(node, receiver)
72
91
  )
73
92
  add_offense(node, message: message) do |corrector|
@@ -76,13 +95,31 @@ module RuboCop
76
95
  end
77
96
  end
78
97
 
79
- private
98
+ def handle_as_symbol(node)
99
+ to_method_symbol(node) do |receiver, sym_node, to_method|
100
+ next if receiver.nil?
101
+
102
+ message = format(
103
+ MSG,
104
+ current: sym_node.source,
105
+ corrected_method: correct_sym_method(to_method)
106
+ )
107
+ add_offense(node, message: message) do |corrector|
108
+ corrector.replace(sym_node, correct_sym_method(to_method))
109
+ end
110
+ end
111
+ end
80
112
 
81
113
  def correct_method(node, receiver)
82
114
  format(CONVERSION_METHOD_CLASS_MAPPING[node.method_name],
83
115
  number_object: receiver.source)
84
116
  end
85
117
 
118
+ def correct_sym_method(to_method)
119
+ body = format(CONVERSION_METHOD_CLASS_MAPPING[to_method], number_object: 'i')
120
+ "{ |i| #{body} }"
121
+ end
122
+
86
123
  def ignore_receiver?(receiver)
87
124
  if receiver.send_type? && ignored_method?(receiver.method_name)
88
125
  true
@@ -0,0 +1,47 @@
1
+ # frozen_string_literal: true
2
+
3
+ module RuboCop
4
+ module Cop
5
+ module Lint
6
+ # This cop checks for uses of numbered parameter assignment.
7
+ # It emulates the following warning in Ruby 2.7:
8
+ #
9
+ # % ruby -ve '_1 = :value'
10
+ # ruby 2.7.2p137 (2020-10-01 revision 5445e04352) [x86_64-darwin19]
11
+ # -e:1: warning: `_1' is reserved for numbered parameter; consider another name
12
+ #
13
+ # Assiging to numbered parameter (from `_1` to `_9`) cause an error in Ruby 3.0.
14
+ #
15
+ # % ruby -ve '_1 = :value'
16
+ # ruby 3.0.0p0 (2020-12-25 revision 95aff21468) [x86_64-darwin19]
17
+ # -e:1: _1 is reserved for numbered parameter
18
+ #
19
+ # NOTE: The parametered parameters are from `_1` to `_9`. This cop checks `_0`, and over `_10`
20
+ # as well to prevent confusion.
21
+ #
22
+ # @example
23
+ #
24
+ # # bad
25
+ # _1 = :value
26
+ #
27
+ # # good
28
+ # non_numbered_parameter_name = :value
29
+ #
30
+ class NumberedParameterAssignment < Base
31
+ NUM_PARAM_MSG = '`_%<number>s` is reserved for numbered parameter; consider another name.'
32
+ LVAR_MSG = '`_%<number>s` is similar to numbered parameter; consider another name.'
33
+ NUMBERED_PARAMETER_RANGE = (1..9).freeze
34
+
35
+ def on_lvasgn(node)
36
+ lhs, _rhs = *node
37
+ return unless /\A_(\d+)\z/ =~ lhs
38
+
39
+ number = Regexp.last_match(1).to_i
40
+ template = NUMBERED_PARAMETER_RANGE.include?(number) ? NUM_PARAM_MSG : LVAR_MSG
41
+
42
+ add_offense(node, message: format(template, number: number))
43
+ end
44
+ end
45
+ end
46
+ end
47
+ end
@@ -0,0 +1,39 @@
1
+ # frozen_string_literal: true
2
+
3
+ module RuboCop
4
+ module Cop
5
+ module Lint
6
+ # This cop checks for unintended or-assignment to a constant.
7
+ #
8
+ # Constants should always be assigned in the same location. And its value
9
+ # should always be the same. If constants are assigned in multiple
10
+ # locations, the result may vary depending on the order of `require`.
11
+ #
12
+ # Also, if you already have such an implementation, auto-correction may
13
+ # change the result.
14
+ #
15
+ # @example
16
+ #
17
+ # # bad
18
+ # CONST ||= 1
19
+ #
20
+ # # good
21
+ # CONST = 1
22
+ #
23
+ class OrAssignmentToConstant < Base
24
+ extend AutoCorrector
25
+
26
+ MSG = 'Avoid using or-assignment with constants.'
27
+
28
+ def on_or_asgn(node)
29
+ lhs, _rhs = *node
30
+ return unless lhs&.casgn_type?
31
+
32
+ add_offense(node.loc.operator) do |corrector|
33
+ corrector.replace(node.loc.operator, '=')
34
+ end
35
+ end
36
+ end
37
+ end
38
+ end
39
+ end
@@ -33,10 +33,12 @@ module RuboCop
33
33
  MSG = 'Use `StandardError` over `Exception`.'
34
34
  RESTRICT_ON_SEND = %i[raise fail].freeze
35
35
 
36
+ # @!method exception?(node)
36
37
  def_node_matcher :exception?, <<~PATTERN
37
38
  (send nil? {:raise :fail} $(const ${cbase nil?} :Exception) ... )
38
39
  PATTERN
39
40
 
41
+ # @!method exception_new_with_message?(node)
40
42
  def_node_matcher :exception_new_with_message?, <<~PATTERN
41
43
  (send nil? {:raise :fail}
42
44
  (send $(const ${cbase nil?} :Exception) :new ... ))
@@ -25,6 +25,7 @@ module RuboCop
25
25
  'Perhaps you meant `rand(2)` or `rand`?'
26
26
  RESTRICT_ON_SEND = %i[rand].freeze
27
27
 
28
+ # @!method rand_one?(node)
28
29
  def_node_matcher :rand_one?, <<~PATTERN
29
30
  (send {(const {nil? cbase} :Kernel) nil?} :rand {(int {-1 1}) (float {-1.0 1.0})})
30
31
  PATTERN
@@ -34,11 +34,13 @@ module RuboCop
34
34
 
35
35
  def on_send(node)
36
36
  return unless (receiver = node.receiver)
37
- return unless receiver.receiver.const_type? && receiver.receiver.short_name == :Dir
37
+ return unless receiver.receiver&.const_type? && receiver.receiver.short_name == :Dir
38
38
  return unless GLOB_METHODS.include?(receiver.method_name)
39
39
 
40
- add_offense(node.loc.selector) do |corrector|
41
- corrector.remove(node.loc.selector)
40
+ selector = node.loc.selector
41
+
42
+ add_offense(selector) do |corrector|
43
+ corrector.remove(selector)
42
44
  corrector.remove(node.loc.dot)
43
45
  end
44
46
  end
@@ -28,6 +28,7 @@ module RuboCop
28
28
  MSG = 'Remove unnecessary `require` statement.'
29
29
  RESTRICT_ON_SEND = %i[require].freeze
30
30
 
31
+ # @!method unnecessary_require_statement?(node)
31
32
  def_node_matcher :unnecessary_require_statement?, <<~PATTERN
32
33
  (send nil? :require
33
34
  (str {"enumerator" "rational" "complex" "thread"}))
@@ -43,6 +43,7 @@ module RuboCop
43
43
 
44
44
  NIL_SPECIFIC_METHODS = (nil.methods - Object.new.methods).to_set.freeze
45
45
 
46
+ # @!method respond_to_nil_specific_method?(node)
46
47
  def_node_matcher :respond_to_nil_specific_method?, <<~PATTERN
47
48
  (csend _ :respond_to? (sym %NIL_SPECIFIC_METHODS))
48
49
  PATTERN
@@ -79,6 +79,7 @@ module RuboCop
79
79
  PERCENT_CAPITAL_I = '%I'
80
80
  ASSIGNMENT_TYPES = %i[lvasgn ivasgn cvasgn gvasgn].freeze
81
81
 
82
+ # @!method array_new?(node)
82
83
  def_node_matcher :array_new?, <<~PATTERN
83
84
  {
84
85
  $(send (const {nil? cbase} :Array) :new ...)
@@ -86,6 +87,7 @@ module RuboCop
86
87
  }
87
88
  PATTERN
88
89
 
90
+ # @!method literal_expansion(node)
89
91
  def_node_matcher :literal_expansion, <<~PATTERN
90
92
  (splat {$({str dstr int float array} ...) (block $#array_new? ...) $#array_new?} ...)
91
93
  PATTERN
@@ -140,13 +142,15 @@ module RuboCop
140
142
  def replacement_range_and_content(node)
141
143
  variable, = *node
142
144
  loc = node.loc
145
+ expression = loc.expression
143
146
 
144
147
  if array_new?(variable)
145
- [node.parent.loc.expression, variable.source]
148
+ expression = node.parent.loc.expression if node.parent.array_type?
149
+ [expression, variable.source]
146
150
  elsif !variable.array_type?
147
- [loc.expression, "[#{variable.source}]"]
151
+ [expression, "[#{variable.source}]"]
148
152
  elsif redundant_brackets?(node)
149
- [loc.expression, remove_brackets(variable)]
153
+ [expression, remove_brackets(variable)]
150
154
  else
151
155
  [loc.operator, '']
152
156
  end
@@ -25,6 +25,7 @@ module RuboCop
25
25
  MSG_SELF = 'Use `self` instead of `Object#to_s` in ' \
26
26
  'interpolation.'
27
27
 
28
+ # @!method to_s_without_args?(node)
28
29
  def_node_matcher :to_s_without_args?, '(send _ :to_s)'
29
30
 
30
31
  def on_interpolation(begin_node)