From 57084f2caa494003e6c372da717a32844b7b2fb0 Mon Sep 17 00:00:00 2001
From: Atef Bellaaj <slalom.bellaaj@external.daimlertruck.com>
Date: Wed, 18 Jun 2025 18:09:55 +0200
Subject: [PATCH] feat: enforce at least one owner requirement for permission
 updates and add corresponding localization messages

---
 .../Agents/Sharing/ManagePermissionsDialog.tsx | 18 +++++++++++++++++-
 client/src/locales/de/translation.json         |  3 ++-
 client/src/locales/en/translation.json         |  5 +++--
 3 files changed, 22 insertions(+), 4 deletions(-)

diff --git a/client/src/components/SidePanel/Agents/Sharing/ManagePermissionsDialog.tsx b/client/src/components/SidePanel/Agents/Sharing/ManagePermissionsDialog.tsx
index a4fb2c1036..c70b22d229 100644
--- a/client/src/components/SidePanel/Agents/Sharing/ManagePermissionsDialog.tsx
+++ b/client/src/components/SidePanel/Agents/Sharing/ManagePermissionsDialog.tsx
@@ -165,6 +165,11 @@ export default function ManagePermissionsDialog({
   const totalShares = managedShares.length + (managedIsPublic ? 1 : 0);
   const originalTotalShares = currentShares.length + (isPublic ? 1 : 0);
 
+  // Check if there's at least one owner (user, group, or public with owner role)
+  const hasAtLeastOneOwner =
+    managedShares.some((share) => share.accessRoleId === ACCESS_ROLE_IDS.AGENT_OWNER) ||
+    (managedIsPublic && managedPublicRole === ACCESS_ROLE_IDS.AGENT_OWNER);
+
   return (
     <OGDialog open={isModalOpen} onOpenChange={setIsModalOpen}>
       <OGDialogTrigger asChild>
@@ -281,7 +286,12 @@ export default function ManagePermissionsDialog({
             </OGDialogClose>
             <Button
               onClick={handleSaveChanges}
-              disabled={updatePermissionsMutation.isLoading || !hasChanges || isLoadingPermissions}
+              disabled={
+                updatePermissionsMutation.isLoading ||
+                !hasChanges ||
+                isLoadingPermissions ||
+                !hasAtLeastOneOwner
+              }
               className="min-w-[120px]"
             >
               {updatePermissionsMutation.isLoading ? (
@@ -300,6 +310,12 @@ export default function ManagePermissionsDialog({
               * {localize('com_ui_unsaved_changes')}
             </div>
           )}
+
+          {!hasAtLeastOneOwner && hasChanges && (
+            <div className="text-xs text-red-600 dark:text-red-400">
+              * {localize('com_ui_at_least_one_owner_required')}
+            </div>
+          )}
         </div>
       </OGDialogContent>
     </OGDialog>
diff --git a/client/src/locales/de/translation.json b/client/src/locales/de/translation.json
index bac1bbdee3..17bbe3a76f 100644
--- a/client/src/locales/de/translation.json
+++ b/client/src/locales/de/translation.json
@@ -963,5 +963,6 @@
   "com_ui_unsaved_changes": "Sie haben ungespeicherte Änderungen",
   "com_ui_share_with_everyone": "Mit allen teilen",
   "com_ui_make_agent_available_all_users": "Diesen Agenten für alle LibreChat-Benutzer verfügbar machen",
-  "com_ui_public_access_level": "Öffentliche Zugriffsebene"
+  "com_ui_public_access_level": "Öffentliche Zugriffsebene",
+  "com_ui_at_least_one_owner_required": "Mindestens ein Besitzer ist erforderlich"
 }
diff --git a/client/src/locales/en/translation.json b/client/src/locales/en/translation.json
index 7b39abe709..ca1bf25620 100644
--- a/client/src/locales/en/translation.json
+++ b/client/src/locales/en/translation.json
@@ -1095,5 +1095,6 @@
   "com_ui_unsaved_changes": "You have unsaved changes",
   "com_ui_share_with_everyone": "Share with everyone",
   "com_ui_make_agent_available_all_users": "Make this agent available to all LibreChat users",
-  "com_ui_public_access_level": "Public access level"
-}
\ No newline at end of file
+  "com_ui_public_access_level": "Public access level",
+  "com_ui_at_least_one_owner_required": "At least one owner is required"
+}