Fix null safety in Horizon gate, GitHub callback route, plugin name handling

- Handle null user in HorizonServiceProvider gate to prevent crash on unauthenticated Horizon requests
- Fix undefined route 'customer.licenses' to 'customer.licenses.list' in GitHub OAuth callback
- Add null safety to Plugin::routeParams() for plugins without a name set
- Check for duplicate plugin name before updating in PluginSyncService to avoid unique constraint violation

Author: shanerbaner82

app/Http/Controllers/GitHubIntegrationController.php

@@ -57,7 +57,7 @@ public function handleCallback(): RedirectResponse
                 'trace' => $e->getTraceAsString(),
             ]);
 
-            $route = Auth::check() ? 'customer.licenses' : 'customer.login';
+            $route = Auth::check() ? 'customer.licenses.list' : 'customer.login';
 
             return to_route($route)
                 ->with('error', 'GitHub authentication failed. Please try again.');

app/Models/Plugin.php

@@ -47,9 +47,12 @@ public static function findByVendorPackageOrFail(string $vendor, string $package
      */
     public function routeParams(): array
     {
-        [$vendor, $package] = explode('/', $this->name);
+        $parts = explode('/', $this->name ?? '');
 
-        return ['vendor' => $vendor, 'package' => $package];
+        return [
+            'vendor' => $parts[0] ?? '',
+            'package' => $parts[1] ?? '',
+        ];
     }
 
     protected $guarded = [];

app/Providers/HorizonServiceProvider.php

@@ -28,7 +28,7 @@ public function boot(): void
     protected function gate(): void
     {
         Gate::define('viewHorizon', function ($user = null) {
-            return $user->isAdmin();
+            return $user?->isAdmin() ?? false;
         });
     }
 }

app/Services/PluginSyncService.php

@@ -65,7 +65,18 @@ public function sync(Plugin $plugin): bool
 
         if ($composerData) {
             if (isset($composerData['name']) && ! $plugin->name) {
-                $updateData['name'] = $composerData['name'];
+                $existing = Plugin::where('name', $composerData['name'])
+                    ->where('id', '!=', $plugin->id)
+                    ->exists();
+
+                if (! $existing) {
+                    $updateData['name'] = $composerData['name'];
+                } else {
+                    Log::warning('[PluginSync] Plugin name already taken', [
+                        'plugin_id' => $plugin->id,
+                        'name' => $composerData['name'],
+                    ]);
+                }
             }
 
             if (isset($composerData['description'])) {