✨ Enhance client management by adding PKCE option and refactoring client retrieval

- Introduced a boolean parameter to `askForClient` method to allow retrieval of all clients.
- Updated client creation process to include an option for requiring PKCE for Authorization Code grants.
- Refactored related code for improved clarity and functionality.

src/Cli/Commands/Command.php

@@ -28,13 +28,22 @@ abstract class Command extends \Symfony\Component\Console\Command\Command implem
     /**
      * @param ClimateOutput $output
      * @param ClimateOutput|ArgvInput $input
+     * @param bool $allClients
      * @return Client | null
      */
-    protected function askForClient(ClimateOutput $output, ClimateOutput|ArgvInput $input): ?Client
+    protected function askForClient(
-    {
+        ClimateOutput $output,
+        ClimateOutput|ArgvInput $input,
+        bool $allClients = false
+    ): ?Client {
+        if ($allClients) {
+            /** @var Collection<Client> $clients */
+            $clients = Client::all(['id', 'name']);
+        } else {
             /** @var Collection<Client> $clients */
             $clients = Client::whereJsonContains('grant_types', 'authorization_code')
                 ->get(['id', 'name']);
+        }
 
         if ($clients->isEmpty()) {
             $output->error('No OAuth clients available.');

src/Cli/Commands/OAuth/CreateClient.php

@@ -40,19 +40,22 @@ class CreateClient extends \Siteworxpro\App\Cli\Commands\Command
             $grantsEnum[] = ClientGrantAlias::from($grant);
         }
 
+        $isExternal = false;
+        if (in_array('authorization_code', $grants)) {
             $question = $this->helper->ask(
                 $input,
                 $output,
                 new \Symfony\Component\Console\Question\ConfirmationQuestion(
-                'External Client (Require PKCE)? (y/N): ',
+                    'Require PKCE for Authorization Code grant? (y/N): ',
                     false,
                     '/^(y|yes)/i'
                 )
             );
 
+            $isExternal = $question === 'y' || $question === true;
+        }
 
-
+        $command = new CreateClientCommand($clientName, $grantsEnum, $clientDescription, $isExternal);
-        $command = new CreateClientCommand($clientName, $grantsEnum, $clientDescription);
         try {
             /** @var Client $client */
             $client = CommandBus::handle($command);

src/Cli/Commands/OAuth/DeleteClient.php

@@ -14,7 +14,7 @@ class DeleteClient extends Command
 {
     public function __invoke(ClimateOutput|ArgvInput $input, $output): int
     {
-        $client = $this->askForClient($output, $input);
+        $client = $this->askForClient($output, $input, true);
 
         if ($client === null) {
             $output->red('No client selected, aborting.');

src/CommandBus/Commands/CreateClient.php

@@ -10,6 +10,7 @@ readonly class CreateClient extends Command
      * @param string $clientName
      * @param array<ClientGrant | mixed> $clientGrants
      * @param string $clientDescription
+     * @param bool $isExternal
      */
     public function __construct(
         private string $clientName,

src/EventListeners/AccessTokenIssuedListener.php

@@ -5,16 +5,17 @@ declare(strict_types=1);
 namespace Siteworxpro\App\EventListeners;
 
 use Siteworxpro\App\Attributes\Events\ListensFor;
+use Siteworxpro\App\Events\AccessToken\Issued as IssuedEvent;
 use Siteworxpro\App\Models\AuditLog;
 use Siteworxpro\App\Models\Enums\AuditLogAction;
 
-#[ListensFor(\Siteworxpro\App\Events\AccessToken\Issued::class)]
+#[ListensFor(IssuedEvent::class)]
 class AccessTokenIssuedListener extends Listener
 {
     /**
      * Handle the event.
      *
-     * @param string | \Siteworxpro\App\Events\AccessToken\Issued $event
+     * @param string | IssuedEvent $event
      * @param array $payload
      * @return AuditLog|null
      */
@@ -24,7 +25,7 @@ class AccessTokenIssuedListener extends Listener
             $event = $payload[0] ?? null;
         }
 
-        if (!$event instanceof \Siteworxpro\App\Events\AccessToken\Issued) {
+        if (!$event instanceof IssuedEvent) {
             return null;
         }