Fix protected or private method access for WP Cli classes

Author: ogorzalka

composer.json

@@ -40,7 +40,8 @@
         "nwidart/laravel-modules": "^12.0@dev",
         "symfony/process": "^7.3.4",
         "illuminate/config": "^12.34",
-        "laravel/prompts": "^0.3.7@dev"
+        "laravel/prompts": "^0.3.7@dev",
+        "spatie/invade": "dev-main"
     },
     "autoload": {
         "psr-4": {

src/WpCli/Application/Services/WpCliService.php

@@ -61,7 +61,15 @@ private function registerWithWpCli(string $name, string|array $className, string
         try {
             // If it's an array (callable), validate the class exists
             if (is_array($className)) {
-                if (!class_exists($className[0])) {
+                // Handle anonymous classes (like our invade wrapper)
+                if (is_object($className[0])) {
+                    // For anonymous classes, we can't validate with class_exists
+                    // but we can check if the method exists
+                    if (!method_exists($className[0], $className[1])) {
+                        error_log("WP CLI command method {$className[1]} does not exist on anonymous class");
+                        return;
+                    }
+                } elseif (!class_exists($className[0])) {
                     error_log("WP CLI command class {$className[0]} does not exist");
                     return;
                 }

src/WpCli/Infrastructure/Services/WpCliDiscovery.php

@@ -257,26 +257,26 @@ private function processSubcommands(ReflectionClass $reflectionClass, string $cl
             $args
         );
 
-        // Then register private/protected methods with Command attributes as individual subcommands
+        // Then register methods with Command attributes as individual subcommands
         foreach ($reflectionClass->getMethods() as $method) {
             $commandAttributes = $method->getAttributes(Command::class);
 
             if ($commandAttributes === []) {
                 continue;
             }
 
-            // Skip if method is public (already handled by base class registration)
-            if ($method->isPublic()) {
-                continue;
-            }
-
             /** @var Command $commandAttribute */
             $commandAttribute = $commandAttributes[0]->newInstance();
             $subcommandName = $commandAttribute->getSubcommandName($method->getName());
             $fullCommandName = "{$baseCommandName} {$subcommandName}";
 
             // Create a callable array for the subcommand
-            $callable = [$className, $method->getName()];
+            // For private/protected methods, we need to use a wrapper with invade
+            if ($method->isPrivate() || $method->isProtected()) {
+                $callable = $this->createInvadeWrapper($className, $method->getName());
+            } else {
+                $callable = [$className, $method->getName()];
+            }
 
             // Register subcommand with WP CLI using method-level arguments
             if (\defined('WP_CLI') && WP_CLI) {
@@ -329,6 +329,34 @@ private function collectMethodArguments(ReflectionMethod $reflectionMethod): arr
         return $this->collectWpCliArguments($reflectionMethod);
     }
 
+    /**
+     * Create an invade wrapper for calling private/protected methods.
+     *
+     * @param string $className The class name
+     * @param string $methodName The method name
+     * @return array A callable array that uses invade
+     */
+    private function createInvadeWrapper(string $className, string $methodName): array
+    {
+        return [new class($className, $methodName) {
+            private string $className;
+            private string $methodName;
+
+            public function __construct(string $className, string $methodName)
+            {
+                $this->className = $className;
+                $this->methodName = $methodName;
+            }
+
+            public function __invoke(array $args, array $assocArgs): mixed
+            {
+                $instance = app($this->className);
+                $invadedInstance = invade($instance);
+                return $invadedInstance->{$this->methodName}($args, $assocArgs);
+            }
+        }, '__invoke'];
+    }
+
     /**
      * Collect WP CLI arguments from reflection attributes.
      *