Fix re-inviting previously removed team members

Reuse the existing Removed team_users record instead of inserting a
duplicate, which violated the team_id+email unique constraint.

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

Author: simonhamp

app/Http/Controllers/TeamUserController.php

@@ -58,11 +58,28 @@ public function invite(InviteTeamUserRequest $request): RedirectResponse
                 ->with('show_add_seats', true);
         }
 
-        $member = $team->users()->create([
-            'email' => $email,
-            'invitation_token' => bin2hex(random_bytes(32)),
-            'invited_at' => now(),
-        ]);
+        // Reuse existing removed record if re-inviting a previously removed member
+        $existingRemoved = $team->users()
+            ->where('email', $email)
+            ->where('status', TeamUserStatus::Removed)
+            ->first();
+
+        if ($existingRemoved) {
+            $existingRemoved->update([
+                'status' => TeamUserStatus::Pending,
+                'user_id' => null,
+                'invitation_token' => bin2hex(random_bytes(32)),
+                'invited_at' => now(),
+                'accepted_at' => null,
+            ]);
+            $member = $existingRemoved;
+        } else {
+            $member = $team->users()->create([
+                'email' => $email,
+                'invitation_token' => bin2hex(random_bytes(32)),
+                'invited_at' => now(),
+            ]);
+        }
 
         Notification::route('mail', $email)
             ->notify(new TeamInvitation($member));

tests/Feature/TeamManagementTest.php

@@ -156,6 +156,38 @@ public function test_cannot_invite_duplicate_email(): void
         $response->assertSessionHas('error');
     }
 
+    public function test_can_reinvite_previously_removed_member(): void
+    {
+        Notification::fake();
+
+        [$owner, $team] = $this->createTeamWithOwner();
+
+        // Create a removed team user (previously invited then cancelled/removed)
+        $removed = TeamUser::factory()->create([
+            'team_id' => $team->id,
+            'email' => 'member@example.com',
+            'status' => TeamUserStatus::Removed,
+            'user_id' => null,
+        ]);
+
+        $response = $this->actingAs($owner)
+            ->post(route('customer.team.invite'), ['email' => 'member@example.com']);
+
+        $response->assertSessionHas('success');
+
+        $this->assertDatabaseHas('team_users', [
+            'id' => $removed->id,
+            'team_id' => $team->id,
+            'email' => 'member@example.com',
+            'status' => TeamUserStatus::Pending->value,
+        ]);
+
+        // Should reuse the existing record, not create a new one
+        $this->assertEquals(1, TeamUser::where('team_id', $team->id)->where('email', 'member@example.com')->count());
+
+        Notification::assertSentOnDemand(TeamInvitation::class);
+    }
+
     public function test_cannot_invite_when_team_suspended(): void
     {
         Notification::fake();