fix: improve Laravel version compatibility and simplify middleware configuration

Author: relliv

config/artisan-command-palette-ui.php

@@ -14,7 +14,7 @@
     'route_prefix' => 'artisan-command-palette',
 
     // Middleware for the command palette routes
-    'middleware' => ['web', 'auth'],
+    'middleware' => [],
 
     // Commands to exclude from the palette
     'excluded_commands' => [

routes/web.php

@@ -5,7 +5,7 @@
 
 Route::group([
     'prefix' => config('artisan-command-palette-ui.route_prefix', 'artisan-command-palette'),
-    'middleware' => config('artisan-command-palette-ui.middleware', ['web', 'auth']),
+    'middleware' => config('artisan-command-palette-ui.middleware', ['auth']),
 ], function () {
     Route::get('/', [ArtisanCommandController::class, 'index'])->name('artisan-command-palette.index');
     Route::get('/commands', [ArtisanCommandController::class, 'listCommands'])->name('artisan-command-palette.commands');

src/ArtisanCommandPaletteUIServiceProvider.php

@@ -16,8 +16,17 @@ public function boot()
         // Load views
         $this->loadViewsFrom(__DIR__ . '/../resources/views', 'artisan-command-palette-ui');
 
-        // Load routes
-        $this->loadRoutesFrom(__DIR__ . '/../routes/web.php');
+        // Register routes in a way compatible with all Laravel versions
+        if (version_compare($this->app->version(), '12.0.0', '>=')) {
+            // Laravel 12+ route registration
+            $this->app->booted(function () {
+                $router = $this->app['router'];
+                require __DIR__ . '/../routes/web.php';
+            });
+        } else {
+            // Legacy route registration
+            $this->loadRoutesFrom(__DIR__ . '/../routes/web.php');
+        }
 
         // Load assets
         $this->publishes([
@@ -44,7 +53,8 @@ public function register()
     {
         // Merge config
         $this->mergeConfigFrom(
-            __DIR__ . '/../config/artisan-command-palette-ui.php', 'artisan-command-palette-ui'
+            __DIR__ . '/../config/artisan-command-palette-ui.php',
+            'artisan-command-palette-ui'
         );
     }
 }

src/Http/Controllers/ArtisanCommandController.php

@@ -3,7 +3,6 @@
 namespace LaravelReady\ArtisanCommandPaletteUI\Http\Controllers;
 
 use Illuminate\Http\Request;
-use Illuminate\Http\JsonResponse;
 use Illuminate\Routing\Controller;
 use Illuminate\Support\Facades\App;
 use Illuminate\Support\Facades\Artisan;
@@ -34,14 +33,14 @@ protected function getCommandGroups()
         $commandGroups = Config::get('artisan-command-palette-ui.command_groups', []);
         $envRestrictions = Config::get('artisan-command-palette-ui.environment_restricted_groups', []);
         $currentEnv = App::environment();
-        
+
         // Apply environment restrictions
         foreach ($envRestrictions as $group => $allowedEnvs) {
             if (!in_array($currentEnv, $allowedEnvs) && isset($commandGroups[$group])) {
                 $commandGroups[$group] = [];
             }
         }
-        
+
         return $commandGroups;
     }
 
@@ -60,13 +59,13 @@ public function listCommands()
         // Return both predefined command groups and all available commands
         $commandGroups = $this->getCommandGroups();
         $allCommands = $this->getAllCommands();
-        
+
         return Response::json([
             'groups' => $commandGroups,
             'all' => $allCommands,
         ]);
     }
-    
+
     /**
      * Get all available commands excluding the ones in the excluded list.
      *
@@ -112,7 +111,7 @@ protected function getAllCommands()
     public function executeCommand(Request $request)
     {
         $command = $request->input('command');
-        
+
         if (empty($command)) {
             return Response::json([
                 'success' => false,
@@ -126,8 +125,9 @@ public function executeCommand(Request $request)
         $commandName = array_shift($parts);
         $arguments = $parts;
 
-        // Check if command exists
-        if (!Artisan::has($commandName)) {
+        // Check if command exists - compatible with Laravel 8-12
+        $commands = Artisan::all();
+        if (!array_key_exists($commandName, $commands)) {
             return Response::json([
                 'success' => false,
                 'message' => 'Error executing command',
@@ -147,15 +147,13 @@ public function executeCommand(Request $request)
 
         // Execute the command
         try {
-            ob_start();
-            $exitCode = Artisan::call($commandName, $this->parseArguments($arguments));
-            $output = ob_get_clean();
+            Artisan::call($command);
+            $output = Artisan::output();
 
             return Response::json([
                 'success' => true,
                 'message' => 'Command executed successfully',
                 'output' => $output,
-                'exit_code' => $exitCode
             ]);
         } catch (\Exception $e) {
             return Response::json([
@@ -175,7 +173,7 @@ public function executeCommand(Request $request)
     protected function parseArguments(array $arguments)
     {
         $result = [];
-        
+
         foreach ($arguments as $argument) {
             if (strpos($argument, '=') !== false) {
                 list($key, $value) = explode('=', $argument, 2);
@@ -186,7 +184,7 @@ protected function parseArguments(array $arguments)
                 $result[] = $argument;
             }
         }
-        
+
         return $result;
     }
 }