Style/RedundantBegin autocorrect breaks rhs begin-end

[tsugimoto]

Auto-correcting Style/RedundantBegin with RuboCop v0.93 breaks code if the value of subject begin-end block is used as a method call parameter or as rhs of a binop.

No offense if value of subject begin-end block is rhs of an assignment, e.g. @var ||= begin ...

# frozen_string_literal: true

puts begin
  'foo'
  'bar'
end

def f
  'hoge' || begin
    'fuga'
    'piyo'
  end
end

puts f

---

Expected behavior

These code should not offense nor auto-correct.

Actual behavior

Auto-correction breaks code.

Steps to reproduce the problem

$ ruby main.rb 
bar
hoge
$ rubocop --cache false --auto-correct main.rb --only Style/RedundantBegin
Inspecting 1 file
C

Offenses:

main.rb:3:6: C: [Corrected] Style/RedundantBegin: Redundant begin block detected.
puts begin
     ^^^^^
main.rb:9:13: C: [Corrected] Style/RedundantBegin: Redundant begin block detected.
  'hoge' || begin
            ^^^^^

1 file inspected, 2 offenses detected, 2 offenses corrected
$ ruby main.rb 

piyo
$ 

RuboCop version

$ rubocop -V
0.93.0 (using Parser 2.7.2.0, rubocop-ast 0.7.1, running on ruby 2.6.3 x86_64-darwin18)

[tsugimoto]

Technically, single line begin-end blocks may or may not break, but they actually look some kind of redundant...

# frozen_string_literal: true

puts begin
  'foo'
end

def f
  'hoge' && begin
    'fuga'
  end
end

puts f