feat: add dry-run option to copy-from-vendor command for previews

Author: dermatz

.github/workflows/functional-tests.yml

@@ -162,18 +162,11 @@ jobs:
           echo "Test help command:"
           bin/magento mageforge:theme:copy-from-vendor --help
 
-          echo "Test with dry-run (no actual copying):"
-          bin/magento mageforge:theme:copy-from-vendor Magento/blank hyva-themes/magento2-default-theme::src/etc/view.xml --dry-run || echo "Expected: may fail if file doesn't exist"
-
-          echo "Test aliases:"
-          bin/magento theme:copy --help
-
-      - name: Test Inspector Status
-        working-directory: magento2
-        run: |
-          echo "=== Inspector Tests ==="
-          bin/magento mageforge:theme:inspector status
-
+          echo "Test with dry-run (preview only, no actual copying):"
+          # Use a file that actually exists in vendor (composer.json from any installed package)
+          bin/magento mageforge:theme:copy-from-vendor vendor/composer/composer/composer.json Magento/blank --dry-run
+          
+          echo "Test alias:"
       - name: Test Inspector Functionality
         working-directory: magento2
         run: |

src/Console/Command/Theme/CopyFromVendorCommand.php

@@ -15,6 +15,7 @@
 use OpenForgeProject\MageForge\Service\VendorFileMapper;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 
 use function Laravel\Prompts\search;
@@ -34,15 +35,17 @@ protected function configure(): void
     {
         $this->setName('mageforge:theme:copy-from-vendor')
              ->setDescription('Copy a file from vendor/ to a specific theme with correct path resolution')
-             ->setAliases(['m:t:cfv'])
+             ->setAliases(['theme:copy'])
              ->addArgument('file', InputArgument::REQUIRED, 'Path to the source file (vendor/...)')
-             ->addArgument('theme', InputArgument::OPTIONAL, 'Target theme code (e.g. Magento/luma)');
+             ->addArgument('theme', InputArgument::OPTIONAL, 'Target theme code (e.g. Magento/luma)')
+             ->addOption('dry-run', null, InputOption::VALUE_NONE, 'Preview the copy operation without performing it');
     }
 
     protected function executeCommand(InputInterface $input, OutputInterface $output): int
     {
         try {
             $sourceFileArg = $input->getArgument('file');
+            $isDryRun = $input->getOption('dry-run');
             $absoluteSourcePath = $this->getAbsoluteSourcePath($sourceFileArg);
 
             // Update sourceFileArg if it was normalized to relative path
@@ -57,6 +60,11 @@ protected function executeCommand(InputInterface $input, OutputInterface $output
             $destinationPath = $this->vendorFileMapper->mapToThemePath($sourceFile, $themePath);
             $absoluteDestPath = $this->getAbsoluteDestPath($destinationPath, $rootPath);
 
+            if ($isDryRun) {
+                $this->showDryRunPreview($sourceFile, $absoluteDestPath, $rootPath);
+                return Cli::RETURN_SUCCESS;
+            }
+
             if (!$this->confirmCopy($sourceFile, $absoluteDestPath, $rootPath)) {
                 return Cli::RETURN_SUCCESS;
             }
@@ -175,6 +183,29 @@ private function performCopy(string $absoluteSourcePath, string $absoluteDestPat
         copy($absoluteSourcePath, $absoluteDestPath);
     }
 
+    private function showDryRunPreview(string $sourceFile, string $absoluteDestPath, string $rootPath): void
+    {
+        $destinationDisplay = str_starts_with($absoluteDestPath, $rootPath . '/')
+            ? substr($absoluteDestPath, strlen($rootPath) + 1)
+            : $absoluteDestPath;
+
+        $this->io->section('Dry Run - Copy Preview');
+        $this->io->text([
+            "Source: <info>$sourceFile</info>",
+            "Target: <info>$destinationDisplay</info>",
+            "Absolute Target: <comment>$absoluteDestPath</comment>"
+        ]);
+        $this->io->newLine();
+
+        if (file_exists($absoluteDestPath)) {
+            $this->io->warning("File already exists at destination and would be overwritten!");
+        } else {
+            $this->io->info("File would be created at destination.");
+        }
+
+        $this->io->note("No files were modified (dry-run mode).");
+    }
+
     private function fixPromptEnvironment(): void
     {
         if (getenv('DDEV_PROJECT')) {