From dceb8363854802b6d3c2526081fd69458663db99 Mon Sep 17 00:00:00 2001
From: Zhi <zhi@dev.hangman-lab.top>
Date: Thu, 12 Mar 2026 12:47:17 +0000
Subject: [PATCH] fix: prevent deleting project owner

---
 src/pages/ProjectDetailPage.tsx | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/pages/ProjectDetailPage.tsx b/src/pages/ProjectDetailPage.tsx
index e32818d..e3ae02c 100644
--- a/src/pages/ProjectDetailPage.tsx
+++ b/src/pages/ProjectDetailPage.tsx
@@ -59,7 +59,12 @@ export default function ProjectDetailPage() {
     api.get<ProjectMember[]>(`/projects/${id}/members`).then(({ data }) => setMembers(data))
   }
 
-  const removeMember = async (userId: number) => {
+  const removeMember = async (userId: number, role: string) => {
+    // Prevent removing owner
+    if (role === 'admin') {
+      alert('Cannot remove project owner (admin)')
+      return
+    }
     if (!confirm('Remove this member?')) return
     await api.delete(`/projects/${id}/members/${userId}`)
     api.get<ProjectMember[]>(`/projects/${id}/members`).then(({ data }) => setMembers(data))
@@ -131,7 +136,7 @@ export default function ProjectDetailPage() {
             {members.map((m) => (
               <span key={m.id} className="badge" style={{marginRight: 8}}>
                 {`User #${m.user_id} (${m.role})`}
-                <button onClick={(e) => { e.stopPropagation(); removeMember(m.user_id) }} style={{marginLeft: 8, background: 'none', border: 'none', color: 'red', cursor: 'pointer'}}>×</button>
+                <button onClick={(e) => { e.stopPropagation(); removeMember(m.user_id, m.role) }} style={{marginLeft: 8, background: 'none', border: 'none', color: 'red', cursor: 'pointer'}}>×</button>
               </span>
             ))}
           </div>