🐛 Bugfix: When creating a new Agent, the user is not prompted to save it before debugging

🐛 Bugfix: Agent modification of unsaved prompts can bypass information verification
🐛 Bugfix: When an Agent is selected and another Agent is imported, it will prompt that there are modifications
🐛 Bugfix: Creating agents will inherit tool parameters wrongly from previous selected agent

Author: Jasonxia007

frontend/app/[locale]/agents/components/AgentSetupOrchestrator.tsx

@@ -500,15 +500,20 @@ export default function AgentSetupOrchestrator({
 
   const confirmOrRun = useCallback(
     (action: PendingAction) => {
-      if (hasUnsavedChanges) {
+      // In creation mode, always show save confirmation dialog when clicking debug
+      if (isCreatingNewAgent && !isEditingAgent) {
+        setPendingAction(() => action);
+        setConfirmContext("switch");
+        setIsSaveConfirmOpen(true);
+      } else if (hasUnsavedChanges) {
         setPendingAction(() => action);
         setConfirmContext("switch");
         setIsSaveConfirmOpen(true);
       } else {
         void Promise.resolve(action());
       }
     },
-    [hasUnsavedChanges]
+    [hasUnsavedChanges, isCreatingNewAgent, isEditingAgent]
   );
 
   const handleToolConfigDraftSave = useCallback(
@@ -905,6 +910,8 @@ export default function AgentSetupOrchestrator({
     setSelectedTools([]);
     setEnabledToolIds([]);
     setEnabledAgentIds([]);
+    setToolConfigDrafts({});
+    setMainAgentId?.(null);
 
     // Clear business logic model to allow default from global settings
     // The useEffect in PromptManager will set it to the default from localStorage
@@ -916,6 +923,12 @@ export default function AgentSetupOrchestrator({
     setMainAgentModel(null);
     setMainAgentModelId(null);
 
+    try {
+      await onToolsRefresh?.(false);
+    } catch (error) {
+      log.error("Failed to refresh tools in creation mode:", error);
+    }
+
     onEditingStateChange?.(false, null);
   };
 
@@ -1545,7 +1558,8 @@ export default function AgentSetupOrchestrator({
           message.success(
             translationFn("businessLogic.config.error.agentImportSuccess")
           );
-          await refreshAgentList(translationFn);
+          // Don't clear tools when importing to avoid triggering false unsaved changes indicator
+          await refreshAgentList(translationFn, false);
           return true;
         }
 
@@ -1745,9 +1759,26 @@ export default function AgentSetupOrchestrator({
             skipUnsavedCheckRef.current = false;
           }, 0);
           onEditingStateChange?.(false, null);
+        } else {
+          // If deleting another agent that is in enabledAgentIds, remove it and update baseline
+          // to avoid triggering false unsaved changes indicator
+          const deletedId = Number(agentToDelete.id);
+          if (enabledAgentIds.includes(deletedId)) {
+            const updatedEnabledAgentIds = enabledAgentIds.filter(
+              (id) => id !== deletedId
+            );
+            setEnabledAgentIds(updatedEnabledAgentIds);
+            // Update baseline to reflect this change so it doesn't trigger unsaved changes
+            if (baselineRef.current) {
+              baselineRef.current = {
+                ...baselineRef.current,
+                enabledAgentIds: updatedEnabledAgentIds.sort((a, b) => a - b),
+              };
+            }
+          }
         }
-        // Refresh agent list
-        refreshAgentList(t);
+        // Refresh agent list without clearing tools to avoid triggering false unsaved changes indicator
+        refreshAgentList(t, false);
       } else {
         message.error(
           result.message || t("businessLogic.config.error.agentDeleteFailed")
@@ -2144,6 +2175,10 @@ export default function AgentSetupOrchestrator({
             // Only close modal, don't execute discard logic
             setIsSaveConfirmOpen(false);
           }}
+          canSave={localCanSaveAgent}
+          invalidReason={
+            localCanSaveAgent ? undefined : getLocalButtonTitle() || undefined
+          }
         />
         {/* Duplicate import confirmation */}
         <Modal

frontend/app/[locale]/agents/components/SaveConfirmModal.tsx

@@ -12,6 +12,8 @@ interface SaveConfirmModalProps {
   onSave: AsyncOrSyncHandler;
   onClose?: AsyncOrSyncHandler;
   width?: number;
+  canSave?: boolean;
+  invalidReason?: string;
 }
 
 export default function SaveConfirmModal({
@@ -20,6 +22,8 @@ export default function SaveConfirmModal({
   onSave,
   onClose,
   width = 520,
+  canSave = true,
+  invalidReason,
 }: SaveConfirmModalProps) {
   const { t } = useTranslation("common");
 
@@ -46,10 +50,14 @@ export default function SaveConfirmModal({
       centered
       footer={
         <div className="flex justify-end mt-4 gap-3">
-          <Button onClick={handleCancel}>{t("agentConfig.modals.saveConfirm.discard")}</Button>
-          <Button type="primary" onClick={handleSave}>
-            {t("agentConfig.modals.saveConfirm.save")}
+          <Button onClick={handleCancel}>
+            {t("agentConfig.modals.saveConfirm.discard")}
           </Button>
+          {canSave ? (
+            <Button type="primary" onClick={handleSave}>
+              {t("agentConfig.modals.saveConfirm.save")}
+            </Button>
+          ) : null}
         </div>
       }
       width={width}
@@ -61,9 +69,19 @@ export default function SaveConfirmModal({
             style={{ fontSize: "48px" }}
           />
           <div className="ml-3 mt-2">
-            <div className="text-sm leading-6">
-              {t("agentConfig.modals.saveConfirm.content")}
-            </div>
+            {canSave ? (
+              <div className="text-sm leading-6">
+                {t("agentConfig.modals.saveConfirm.content")}
+              </div>
+            ) : (
+              <div className="text-sm leading-6 flex flex-col gap-2">
+                <span>
+                  {t("agentConfig.modals.saveConfirm.invalidContent", {
+                    invalidReason,
+                  })}
+                </span>
+              </div>
+            )}
           </div>
         </div>
       </div>

frontend/app/[locale]/agents/components/tool/ToolConfigModal.tsx

@@ -84,14 +84,15 @@ export default function ToolConfigModal({
         return;
       }
 
-      // In creation mode we do not have an agent ID yet, so use the tool's default params.
+      // In creation mode (no agent ID), always use backend-provided default params
       if (!normalizedAgentId) {
         setCurrentParams(buildDefaultParams());
         return;
       }
 
       setIsLoading(true);
       try {
+        // In edit mode, load tool config for the specific agent
         const result = await searchToolConfig(
           parseInt(tool.id),
           normalizedAgentId

frontend/public/locales/en/common.json

@@ -904,6 +904,7 @@
   "agentConfig.agents.listFetchFailedDebug": "Failed to fetch Agent list:",
   "agentConfig.modals.saveConfirm.title": "Unsaved changes",
   "agentConfig.modals.saveConfirm.content": "You have unsaved changes. Would you like to save them now?",
+  "agentConfig.modals.saveConfirm.invalidContent": "Current configuration cannot be saved: {{invalidReason}}. Please modify and try again.",
   "agentConfig.modals.saveConfirm.discard": "Discard",
   "agentConfig.modals.saveConfirm.save": "Save",
 

frontend/public/locales/zh/common.json

@@ -904,6 +904,7 @@
   "agentConfig.agents.listFetchFailedDebug": "获取 Agent 列表失败:",
   "agentConfig.modals.saveConfirm.title": "未保存的更改",
   "agentConfig.modals.saveConfirm.content": "当前Agent有未保存的更改。是否现在保存？",
+  "agentConfig.modals.saveConfirm.invalidContent": "当前配置无法保存：{{invalidReason}}。请修改后重试。",
   "agentConfig.modals.saveConfirm.discard": "放弃更改",
   "agentConfig.modals.saveConfirm.save": "保存",