feat: add request injection

Author: PanchoDP

.gitignore

@@ -9,3 +9,5 @@ vendor/
 .DS_Store
 composer.lock
 cliff.toml
+.CLAUDE.md
+CLAUDE.md

README.md

@@ -102,6 +102,7 @@ final class MyAction
 Or you can use more than one subfolder like this:.
 
 ```bash
+
 php artisan make:action MyAction Folder1/Folder2
 ```
 This will create a new action class in the `app/Actions/Folder1/Folder2/` directory with the name `MyAction.php`.
@@ -196,6 +197,78 @@ final class MyAction
 
 ```
 
+- `--r` This flag generates a Laravel Request class and injects it into the action method.
+
+For example, if you want to create an action class with Request injection, you can use:
+
+```bash
+php artisan make:action MyAction --r
+```
+This will generate both an Action class and a Request class (`MyActionRequest`), and will result in:
+
+```php
+<?php
+
+declare(strict_types=1);
+
+namespace App\Actions;
+
+use App\Http\Requests\MyActionRequest;
+
+final class MyAction
+{
+    public static function handle(MyActionRequest $request): void
+    {
+        // This is where the action logic will be implemented.
+    }
+}
+```
+
+### Advanced Flag Combinations
+
+You can combine multiple flags to create actions with different features. All possible combinations are supported:
+
+**Two-flag combinations:**
+- `--tr` or `--rt`: Database transactions with Request injection
+- `--ur` or `--ru`: User injection with Request injection  
+- `--tu` or `--ut`: Database transactions with User injection (as shown above)
+
+**Three-flag combinations (all features):**
+- `--tur`, `--tru`, `--utr`, `--urt`, `--rtu`, `--rut`: All features combined
+
+For example:
+```bash
+php artisan make:action CompleteAction --tur
+```
+
+This will generate:
+```php
+<?php
+
+declare(strict_types=1);
+
+namespace App\Actions;
+
+use Illuminate\Support\Facades\DB;
+use App\Models\User;
+use App\Http\Requests\CompleteActionRequest;
+
+final class CompleteAction
+{
+    public static function handle(User $user, CompleteActionRequest $request): void
+    {
+        DB::transaction(function () use ($request) {
+            // Logic to be executed within the transaction
+        });
+    }
+}
+```
+
+**Individual flags are also supported:**
+- `--t`: Only database transactions
+- `--u`: Only user injection  
+- `--r`: Only request injection
+
 
 ## Contributing
 

src/Actions/GenerateRequest.php

@@ -0,0 +1,48 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Panchodp\LaravelAction\Actions;
+
+use Illuminate\Support\Facades\Artisan;
+use InvalidArgumentException;
+use Throwable;
+
+final class GenerateRequest
+{
+    /**
+     * Generate a Laravel Request class using Artisan command
+     *
+     * @param  string  $actionName  The base name for the action (e.g., 'Limpiar')
+     *
+     * @throws InvalidArgumentException
+     */
+    public static function handle(string $actionName): string
+    {
+        if (empty($actionName)) {
+            throw new InvalidArgumentException('Action name cannot be empty.');
+        }
+
+        // Validate action name format
+        if (! preg_match('/^[A-Z][a-zA-Z0-9]*$/', $actionName)) {
+            throw new InvalidArgumentException('Invalid action name format. Must start with uppercase letter and contain only alphanumeric characters.');
+        }
+
+        $requestName = $actionName.'Request';
+
+        try {
+            // Execute Laravel's make:request command
+            $exitCode = Artisan::call('make:request', [
+                'name' => $requestName,
+            ]);
+
+            if ($exitCode !== 0) {
+                throw new InvalidArgumentException("Failed to generate Request class: {$requestName}");
+            }
+
+            return $requestName;
+        } catch (Throwable $e) {
+            throw new InvalidArgumentException('Error generating Request class: '.$e->getMessage());
+        }
+    }
+}

src/Actions/PrepareStub.php

@@ -9,10 +9,11 @@
 
 final class PrepareStub
 {
-    public static function handle(bool $tuFlag, bool $tFlag, bool $uFlag, string $filename, string $namespace): string
+    public static function handle(bool $tFlag, bool $uFlag, bool $rFlag, string $filename, string $namespace): string
     {
         try {
-            $stubFile = $tFlag ? __DIR__.'/../stubs/action_transaction.stub' : __DIR__.'/../stubs/action.stub';
+            // Select appropriate stub based on flags
+            $stubFile = self::selectStubFile($tFlag, $rFlag);
 
             // Security: Validate stub file exists before reading
             if (! file_exists($stubFile) || ! is_readable($stubFile)) {
@@ -25,6 +26,13 @@ public static function handle(bool $tuFlag, bool $tFlag, bool $uFlag, string $fi
                 throw new RuntimeException("Failed to read stub file: {$stubFile}");
             }
 
+            // Handle Request class injection
+            if ($rFlag) {
+                $requestClass = $filename.'Request';
+                $safeRequestClass = self::sanitizeForTemplate($requestClass);
+                $stub = str_replace('{{ request_class }}', $safeRequestClass, $stub);
+            }
+
             if ($uFlag) {
                 $stub = str_replace('{{ import_model }}', 'use App\Models\User;', $stub);
                 $stub = str_replace('{{ user }}', 'User $user,', $stub);
@@ -53,6 +61,28 @@ public static function handle(bool $tuFlag, bool $tFlag, bool $uFlag, string $fi
 
     }
 
+    /**
+     * Select the appropriate stub file based on flags
+     */
+    private static function selectStubFile(bool $tFlag, bool $rFlag): string
+    {
+        $baseDir = __DIR__.'/../stubs/';
+
+        if ($tFlag && $rFlag) {
+            return $baseDir.'action_transaction_request.stub';
+        }
+
+        if ($rFlag) {
+            return $baseDir.'action_request.stub';
+        }
+
+        if ($tFlag) {
+            return $baseDir.'action_transaction.stub';
+        }
+
+        return $baseDir.'action.stub';
+    }
+
     /**
      * Sanitize input for template usage to prevent template injection.
      *

src/Console/MakeActionCommand.php

@@ -7,6 +7,7 @@
 use Illuminate\Console\Command;
 use Illuminate\Support\Facades\File;
 use Panchodp\LaravelAction\Actions\CreateDirectory;
+use Panchodp\LaravelAction\Actions\GenerateRequest;
 use Panchodp\LaravelAction\Actions\ObtainNamespace;
 use Panchodp\LaravelAction\Actions\PreparePath;
 use Panchodp\LaravelAction\Actions\PrepareStub;
@@ -20,9 +21,20 @@ final class MakeActionCommand extends Command
 {
     protected $signature = 'make:action {name} {subfolder?} 
     {--t : Make a DB transaction action } 
-    {--tu : Make a DB transaction action } 
-    {--ut : Make a DB transaction action } 
-    {--u : Make a User injection}';
+    {--u : Make a User injection}
+    {--r : Generate a Request class and inject it into the action}
+    {--tu : Make a DB transaction action with User injection}
+    {--ut : Make a DB transaction action with User injection}
+    {--tr : Make a DB transaction action with Request injection}
+    {--rt : Make a DB transaction action with Request injection}
+    {--ur : Make a User injection with Request injection}
+    {--ru : Make a User injection with Request injection}
+    {--tur : Make a DB transaction action with User and Request injection}
+    {--tru : Make a DB transaction action with User and Request injection}
+    {--utr : Make a DB transaction action with User and Request injection}
+    {--urt : Make a DB transaction action with User and Request injection}
+    {--rtu : Make a DB transaction action with User and Request injection}
+    {--rut : Make a DB transaction action with User and Request injection}';
 
     protected $description = 'Create a new action class';
 
@@ -37,6 +49,12 @@ public function handle(): int
             $input = $this->processInputs();
             $config = $this->validateAndPrepareConfig($input);
             $this->createDirectoryStructure($config);
+
+            // Generate Request class if --r flag is present
+            if ($config['rFlag'] ?? false) {
+                $this->generateRequestFile($config);
+            }
+
             $this->generateActionFile($config);
             $this->displaySuccessMessages($config);
 
@@ -50,6 +68,7 @@ public function handle(): int
 
     /**
      * Process and sanitize command inputs.
+     *
      * @return array<string, mixed>
      */
     private function processInputs(): array
@@ -60,28 +79,38 @@ private function processInputs(): array
         $subfolder = $this->argument('subfolder');
         $subfolder = is_string($subfolder) ? mb_trim($subfolder, '/\\') : '';
 
-        $tuFlag = (bool) ($this->option('tu') || $this->option('ut'));
-        $tFlag = (bool) ($this->option('t') || $tuFlag);
-        $uFlag = (bool) ($this->option('u') || $tuFlag);
+        // Check for combination flags first
+        $turFlag = (bool) ($this->option('tur') || $this->option('tru') ||
+                          $this->option('utr') || $this->option('urt') ||
+                          $this->option('rtu') || $this->option('rut'));
+
+        $tuFlag = (bool) ($this->option('tu') || $this->option('ut') || $turFlag);
+        $trFlag = (bool) ($this->option('tr') || $this->option('rt') || $turFlag);
+        $urFlag = (bool) ($this->option('ur') || $this->option('ru') || $turFlag);
+
+        // Individual flags or derived from combinations
+        $tFlag = (bool) ($this->option('t') || $tuFlag || $trFlag);
+        $uFlag = (bool) ($this->option('u') || $tuFlag || $urFlag);
+        $rFlag = (bool) ($this->option('r') || $trFlag || $urFlag);
 
         return [
             'name' => $name,
             'subfolder' => $subfolder,
-            'tuFlag' => $tuFlag,
             'tFlag' => $tFlag,
             'uFlag' => $uFlag,
+            'rFlag' => $rFlag,
         ];
     }
 
     /**
-     * @param array<string, mixed> $input
+     * @param  array<string, mixed>  $input
      * @return array<string, mixed>
      */
     private function validateAndPrepareConfig(array $input): array
     {
         $name = is_string($input['name']) ? $input['name'] : '';
         $subfolder = is_string($input['subfolder']) ? $input['subfolder'] : '';
-        
+
         ValidateName::handle($name);
 
         // Security: Validate configuration values
@@ -111,7 +140,7 @@ private function validateAndPrepareConfig(array $input): array
     }
 
     /**
-     * @param array<string, mixed> $config
+     * @param  array<string, mixed>  $config
      */
     private function createDirectoryStructure(array $config): void
     {
@@ -121,27 +150,38 @@ private function createDirectoryStructure(array $config): void
 
         $path = is_string($config['path']) ? $config['path'] : '';
         $relativePath = is_string($config['relative_path']) ? $config['relative_path'] : '';
-        
+
         CreateDirectory::handle($path, $permissions);
         $this->info("Directory {$relativePath} created successfully...");
     }
 
     /**
-     * @param array<string, mixed> $config
+     * @param  array<string, mixed>  $config
+     */
+    private function generateRequestFile(array $config): void
+    {
+        $filename = is_string($config['filename']) ? $config['filename'] : '';
+
+        $requestName = GenerateRequest::handle($filename);
+        $this->info("Request {$requestName} created successfully...");
+    }
+
+    /**
+     * @param  array<string, mixed>  $config
      */
     private function generateActionFile(array $config): void
     {
-        $tuFlag = is_bool($config['tuFlag']) ? $config['tuFlag'] : false;
         $tFlag = is_bool($config['tFlag']) ? $config['tFlag'] : false;
         $uFlag = is_bool($config['uFlag']) ? $config['uFlag'] : false;
+        $rFlag = is_bool($config['rFlag']) ? $config['rFlag'] : false;
         $filename = is_string($config['filename']) ? $config['filename'] : '';
         $namespace = is_string($config['namespace']) ? $config['namespace'] : '';
         $path = is_string($config['path']) ? $config['path'] : '';
-        
+
         $stub = PrepareStub::handle(
-            $tuFlag,
             $tFlag,
             $uFlag,
+            $rFlag,
             $filename,
             $namespace
         );
@@ -150,15 +190,28 @@ private function generateActionFile(array $config): void
     }
 
     /**
-     * @param array<string, mixed> $config
+     * @param  array<string, mixed>  $config
      */
     private function displaySuccessMessages(array $config): void
     {
         $tFlag = is_bool($config['tFlag']) ? $config['tFlag'] : false;
+        $uFlag = is_bool($config['uFlag']) ? $config['uFlag'] : false;
+        $rFlag = is_bool($config['rFlag']) ? $config['rFlag'] : false;
         $filename = is_string($config['filename']) ? $config['filename'] : '';
         $relativePath = is_string($config['relative_path']) ? $config['relative_path'] : '';
-        
-        $transaction = $tFlag ? ' with DB transaction' : '.';
-        $this->info("Action {$filename} created successfully at app/{$relativePath} folder".$transaction);
+
+        $features = [];
+        if ($tFlag) {
+            $features[] = 'DB transaction';
+        }
+        if ($uFlag) {
+            $features[] = 'User injection';
+        }
+        if ($rFlag) {
+            $features[] = 'Request injection';
+        }
+
+        $featuresText = empty($features) ? '.' : ' with '.implode(', ', $features).'.';
+        $this->info("Action {$filename} created successfully at app/{$relativePath} folder{$featuresText}");
     }
 }