feat(rector): add RectorApplyRunner and fix --no-dry-run obsolete flag

Rector removed the --no-dry-run option. Fix:
- Add RectorApplyRunner: runs rector process without --dry-run (apply mode)
- Register RectorApplyRunner in bin/kcode
- Fix FormatCommand: use rector-apply runner (applies changes, not dry-run)
- Fix RectorCommand: use rector-apply when --fix/--apply flag passed

kcode format now correctly applies cs-fixer + rector changes without error.

Author: walmir-silva

bin/kcode

@@ -68,6 +68,7 @@ declare(strict_types=1);
         $devkit->addRunner(new \KaririCode\Devkit\Runner\PhpStanRunner($executor, $context));
         $devkit->addRunner(new \KaririCode\Devkit\Runner\CsFixerRunner($executor, $context));
         $devkit->addRunner(new \KaririCode\Devkit\Runner\RectorRunner($executor, $context));
+        $devkit->addRunner(new \KaririCode\Devkit\Runner\RectorApplyRunner($executor, $context));
         $devkit->addRunner(new \KaririCode\Devkit\Runner\PsalmRunner($executor, $context));
         $devkit->addRunner(new \KaririCode\Devkit\Runner\ComposerAuditRunner($executor, $context));
     }

build/kcode.phar

@@ -50,9 +50,9 @@ public function execute(Devkit $devkit, array $arguments): int
         }
 
         // Step 2: Rector apply (--no-dry-run overrides runner default)
-        if ($devkit->isToolAvailable('rector')) {
+        if ($devkit->isToolAvailable('rector-apply')) {
             $this->line("\033[1m▸ Running rector process…\033[0m");
-            $result = $devkit->run('rector', ['--no-dry-run', ...$arguments]);
+            $result = $devkit->run('rector-apply', $arguments);
             $this->line($result->output());
 
             if ($result->success) {

src/Command/FormatCommand.php

@@ -36,10 +36,10 @@ public function execute(Devkit $devkit, array $arguments): int
 
         $passthrough = $this->passthrough($arguments, ['--fix', '--apply']);
 
-        // RectorRunner defaults to --dry-run for safety.
-        // When applying, --no-dry-run overrides it (Rector: last flag wins).
+        // RectorRunner defaults to --dry-run (preview only).
+        // RectorApplyRunner runs without --dry-run (applies changes).
         $result = $apply
-            ? $devkit->run('rector', ['--no-dry-run', ...$passthrough])
+            ? $devkit->run('rector-apply', $passthrough)
             : $devkit->run('rector', $passthrough);
 
         $this->line($result->output());

src/Command/RectorCommand.php

@@ -0,0 +1,41 @@
+<?php
+
+declare(strict_types=1);
+
+namespace KaririCode\Devkit\Runner;
+
+use Override;
+
+/**
+ * Runs Rector without --dry-run, applying all changes.
+ *
+ * Used by FormatCommand and RectorCommand (with --fix) to actually write
+ * the changes to disk, as opposed to RectorRunner which only previews.
+ *
+ * @since 1.0.0
+ */
+final class RectorApplyRunner extends AbstractToolRunner
+{
+    #[Override]
+    public function toolName(): string
+    {
+        return 'rector-apply';
+    }
+
+    #[Override]
+    protected function vendorBin(): string
+    {
+        return 'vendor/bin/rector';
+    }
+
+    #[Override]
+    protected function defaultArguments(): array
+    {
+        return [
+            'process',
+            '--config',
+            $this->context->configPath('rector.php'),
+            '--ansi',
+        ];
+    }
+}