skip existing defaults public call

Author: TomasVotruba

rules-tests/Configs/Rector/Closure/FromServicePublicToDefaultsPublicRector/Fixture/with_existing_public_defaults.php.inc

@@ -0,0 +1,32 @@
+<?php
+
+namespace Rector\Symfony\Tests\Configs\Rector\Closure\ServiceTagsToDefaultsAutoconfigureRector\Fixture;
+
+use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
+
+return static function (ContainerConfigurator $containerConfigurator): void {
+    $services = $containerConfigurator->services();
+
+    $services->defaults()->public();
+
+    $services->set(NextCommand::class)
+        ->public();
+};
+
+?>
+-----
+<?php
+
+namespace Rector\Symfony\Tests\Configs\Rector\Closure\ServiceTagsToDefaultsAutoconfigureRector\Fixture;
+
+use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
+
+return static function (ContainerConfigurator $containerConfigurator): void {
+    $services = $containerConfigurator->services();
+
+    $services->defaults()->public();
+
+    $services->set(NextCommand::class);
+};
+
+?>

rules/Configs/Rector/Closure/FromServicePublicToDefaultsPublicRector.php

@@ -98,6 +98,11 @@ public function refactor(Node $node): ?Node
                 return null;
             }
 
+            // not the ->defaults()->public()
+            if ($this->isDefaultCall($node)) {
+                return null;
+            }
+
             $hasChanged = true;
 
             return $node->var;
@@ -113,4 +118,17 @@ public function refactor(Node $node): ?Node
 
         return $node;
     }
+
+    public function isDefaultCall(MethodCall $methodCall): bool
+    {
+        $currentMethodCall = $methodCall;
+        while ($currentMethodCall instanceof MethodCall) {
+            if ($this->isName($currentMethodCall->name, 'defaults')) {
+                return true;
+            }
+            $currentMethodCall = $currentMethodCall->var;
+        }
+
+        return false;
+    }
 }