[Fix rubocop#8497] Fix Style/IfUnlessModifier to handle if-end condit…

…ion in method argument list

CHANGELOG.md

@@ -11,6 +11,7 @@
 * [#8480](https://github.com/rubocop-hq/rubocop/issues/8480): Tweak callback list of `Lint/MissingSuper`. ([@marcandre][])
 * [#8481](https://github.com/rubocop-hq/rubocop/pull/8481): Fix autocorrect for elements with newlines in `Style/SymbolArray` and `Style/WordArray`. ([@biinari][])
 * [#8475](https://github.com/rubocop-hq/rubocop/issues/8475): Fix a false positive for `Style/HashAsLastArrayItem` when there are duplicate hashes in the array. ([@wcmonty][])
+* [#8497](https://github.com/rubocop-hq/rubocop/issues/8497): Fix `Style/IfUnlessModifier` to add parentheses when converting if-end condition inside a parenthesized method argument list. ([@dsavochkin][])
 
 ### Changes
 

lib/rubocop/cop/mixin/statement_modifier.rb

@@ -72,7 +72,7 @@ def parenthesize?(node)
         return true if parent.assignment? || parent.operator_keyword?
         return true if %i[array pair].include?(parent.type)
 
-        node.parent.send_type? && !node.parent.parenthesized?
+        node.parent.send_type?
       end
 
       def max_line_length

spec/rubocop/cop/style/if_unless_modifier_spec.rb

@@ -457,7 +457,7 @@ def f
   end
 
   context 'if-end is argument to a parenthesized method call' do
-    it "doesn't add redundant parentheses" do
+    it 'adds parentheses because otherwise it would cause SyntaxError' do
       expect_offense(<<~RUBY)
         puts("string", if a
                        ^^ Favor modifier `if` usage when having a single-line body. [...]
@@ -466,7 +466,7 @@ def f
       RUBY
 
       expect_correction(<<~RUBY)
-        puts("string", 1 if a)
+        puts("string", (1 if a))
       RUBY
     end
   end