fix(auth/admin): atomic user+customer signup; Rule::in for country validation (comma-safe)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

app/Actions/Fortify/CreateNewUser.php

@@ -4,6 +4,7 @@ namespace App\Actions\Fortify;
 
 use App\Models\Customer;
 use App\Models\User;
+use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Hash;
 use Illuminate\Support\Facades\Validator;
 use Illuminate\Validation\Rule;
@@ -39,23 +40,25 @@ class CreateNewUser implements CreatesNewUsers
             'password' => $this->passwordRules(),
         ])->validate();
 
-        $user = User::create([
+        // Create the user and its linked customer atomically — a public signup is
-            'name' => $input['name'],
+        // a customer, and the portal (Billing::purchase, Settings, …) needs one.
-            'email' => $input['email'],
+        // Either both exist or neither, so a failure never orphans a user.
-            'password' => Hash::make($input['password']),
+        return DB::transaction(function () use ($input) {
-        ]);
+            $user = User::create([
+                'name' => $input['name'],
+                'email' => $input['email'],
+                'password' => Hash::make($input['password']),
+            ]);
 
-        // A public signup is a customer — create and link the record now so the
+            Customer::query()->create([
-        // portal (Billing::purchase, Settings, …) has a customer to work with
+                'user_id' => $user->id,
-        // before they buy a package.
+                'name' => $input['name'],
-        Customer::query()->create([
+                'email' => $input['email'],
-            'user_id' => $user->id,
+                'locale' => app()->getLocale(),
-            'name' => $input['name'],
+                'status' => 'active',
-            'email' => $input['email'],
+            ]);
-            'locale' => app()->getLocale(),
-            'status' => 'active',
-        ]);
 
-        return $user;
+            return $user;
+        });
     }
 }

app/Livewire/Admin/Datacenters.php

@@ -3,6 +3,7 @@
 namespace App\Livewire\Admin;
 
 use App\Models\Datacenter;
+use Illuminate\Validation\Rule;
 use Livewire\Attributes\Layout;
 use Livewire\Attributes\Validate;
 use Livewire\Component;
@@ -32,7 +33,7 @@ class Datacenters extends Component
         $data = $this->validate([
             'code' => 'required|string|max:8|alpha_dash|unique:datacenters,code',
             'name' => 'required|string|max:255',
-            'location' => 'nullable|in:'.implode(',', array_keys((array) config('countries'))),
+            'location' => ['nullable', Rule::in(array_keys((array) config('countries')))],
         ]);
 
         Datacenter::create([

app/Livewire/Admin/EditDatacenter.php

@@ -3,6 +3,7 @@
 namespace App\Livewire\Admin;
 
 use App\Models\Datacenter;
+use Illuminate\Validation\Rule;
 use Livewire\Attributes\Validate;
 use LivewireUI\Modal\ModalComponent;
 
@@ -51,7 +52,7 @@ class EditDatacenter extends ModalComponent
         }
         $data = $this->validate([
             'name' => 'required|string|max:255',
-            'location' => 'nullable|in:'.implode(',', $allowed),
+            'location' => ['nullable', Rule::in($allowed)], // array form — a legacy value may contain commas
         ]);
 
         Datacenter::query()->where('uuid', $this->uuid)->update([