From 881af6344080f69bfba4f8fbaf2e442ff94c82de Mon Sep 17 00:00:00 2001
From: Flartiny <1335348298@qq.com>
Date: Sun, 22 Mar 2026 14:29:09 +0800
Subject: [PATCH 1/2] feat: allow copy config from existing configs

---
 .../i18n/locales/en-US/features/config.json   |   3 +
 .../i18n/locales/ru-RU/features/config.json   |   5 +-
 .../i18n/locales/zh-CN/features/config.json   |   3 +
 dashboard/src/views/ConfigPage.vue            | 104 ++++++++++++++++--
 4 files changed, 107 insertions(+), 8 deletions(-)

diff --git a/dashboard/src/i18n/locales/en-US/features/config.json b/dashboard/src/i18n/locales/en-US/features/config.json
index 4b726ae3c8..f0b19156b3 100644
--- a/dashboard/src/i18n/locales/en-US/features/config.json
+++ b/dashboard/src/i18n/locales/en-US/features/config.json
@@ -77,13 +77,16 @@
     "title": "Configuration Management",
     "description": "AstrBot supports separate configuration files for different bots. The `default` configuration is used by default.",
     "newConfig": "New Configuration",
+    "copyConfig": "Copy Configuration",
     "editConfig": "Edit Configuration",
     "manageConfigs": "Manage Configurations...",
     "configName": "Name",
     "fillConfigName": "Enter configuration name",
     "confirmDelete": "Are you sure you want to delete the configuration \"{name}\"? This action cannot be undone.",
     "pleaseEnterName": "Please enter a configuration name",
+    "nameExists": "Configuration name already exists. Please use another name.",
     "createFailed": "Failed to create new configuration",
+    "copyFailed": "Failed to copy configuration",
     "deleteFailed": "Failed to delete configuration",
     "updateFailed": "Failed to update configuration"
   },
diff --git a/dashboard/src/i18n/locales/ru-RU/features/config.json b/dashboard/src/i18n/locales/ru-RU/features/config.json
index 7c456ab9d8..555b321d30 100644
--- a/dashboard/src/i18n/locales/ru-RU/features/config.json
+++ b/dashboard/src/i18n/locales/ru-RU/features/config.json
@@ -77,13 +77,16 @@
         "title": "Управление конфигурациями",
         "description": "AstrBot поддерживает несколько конфигураций для разных ботов. По умолчанию используется «default».",
         "newConfig": "Новая конфигурация",
+        "copyConfig": "Копировать конфигурацию",
         "editConfig": "Изменить конфигурацию",
         "manageConfigs": "Управление файлами...",
         "configName": "Имя",
         "fillConfigName": "Введите имя конфигурации",
         "confirmDelete": "Вы уверены, что хотите удалить конфигурацию «{name}»? Это действие необратимо.",
         "pleaseEnterName": "Пожалуйста, введите имя",
+        "nameExists": "Имя конфигурации уже существует. Используйте другое имя.",
         "createFailed": "Ошибка создания конфигурации",
+        "copyFailed": "Ошибка копирования конфигурации",
         "deleteFailed": "Ошибка удаления",
         "updateFailed": "Ошибка обновления"
     },
@@ -126,4 +129,4 @@
             "cancel": "Отмена"
         }
     }
-}
\ No newline at end of file
+}
diff --git a/dashboard/src/i18n/locales/zh-CN/features/config.json b/dashboard/src/i18n/locales/zh-CN/features/config.json
index e7cd90408b..3575d05b73 100644
--- a/dashboard/src/i18n/locales/zh-CN/features/config.json
+++ b/dashboard/src/i18n/locales/zh-CN/features/config.json
@@ -77,13 +77,16 @@
     "title": "配置文件管理",
     "description": "AstrBot 支持针对不同机器人分别设置配置文件。默认会使用 `default` 配置。",
     "newConfig": "新建配置文件",
+    "copyConfig": "复制配置文件",
     "editConfig": "编辑配置文件",
     "manageConfigs": "管理配置文件...",
     "configName": "名称",
     "fillConfigName": "填写配置文件名称",
     "confirmDelete": "确定要删除配置文件 \"{name}\" 吗?此操作不可恢复。",
     "pleaseEnterName": "请填写配置名称",
+    "nameExists": "配置文件名称已存在，请使用其他名称",
     "createFailed": "新配置文件创建失败",
+    "copyFailed": "复制配置文件失败",
     "deleteFailed": "删除配置文件失败",
     "updateFailed": "更新配置文件失败"
   },
diff --git a/dashboard/src/views/ConfigPage.vue b/dashboard/src/views/ConfigPage.vue
index 0ecc2a41d3..d310e82abb 100644
--- a/dashboard/src/views/ConfigPage.vue
+++ b/dashboard/src/views/ConfigPage.vue
@@ -126,11 +126,15 @@
         <!-- Config List -->
         <v-list lines="two">
           <v-list-item v-for="config in configInfoList" :key="config.id" :title="config.name">
-            <template v-slot:append v-if="config.id !== 'default'">
+            <template v-slot:append>
               <div class="d-flex align-center" style="gap: 8px;">
+                <v-btn icon="mdi-content-copy" size="small" variant="text" color="primary"
+                  @click="startCopyConfig(config)"></v-btn>
                 <v-btn icon="mdi-pencil" size="small" variant="text" color="warning"
+                  v-if="config.id !== 'default'"
                   @click="startEditConfig(config)"></v-btn>
                 <v-btn icon="mdi-delete" size="small" variant="text" color="error"
+                  v-if="config.id !== 'default'"
                   @click="confirmDeleteConfig(config)"></v-btn>
               </div>
             </template>
@@ -141,7 +145,9 @@
         <v-divider v-if="showConfigForm" class="my-6"></v-divider>
 
         <div v-if="showConfigForm">
-          <h3 class="mb-4">{{ isEditingConfig ? tm('configManagement.editConfig') : tm('configManagement.newConfig') }}</h3>
+          <h3 class="mb-4">
+            {{ isEditingConfig ? tm('configManagement.editConfig') : (isCopyingConfig ? tm('configManagement.copyConfig') : tm('configManagement.newConfig')) }}
+          </h3>
 
           <h4>{{ tm('configManagement.configName') }}</h4>
 
@@ -151,7 +157,7 @@
           <div class="d-flex justify-end mt-4" style="gap: 8px;">
             <v-btn variant="text" @click="cancelConfigForm">{{ tm('buttons.cancel') }}</v-btn>
             <v-btn color="primary" @click="saveConfigForm"
-              :disabled="!configFormData.name">
+              :disabled="!configFormData.name || (isCopyingConfig && !copySourceConfigId)">
               {{ isEditingConfig ? tm('buttons.update') : tm('buttons.create') }}
             </v-btn>
           </div>
@@ -343,6 +349,7 @@ export default {
       configManageDialog: false,
       showConfigForm: false,
       isEditingConfig: false,
+      isCopyingConfig: false,
       config_data_has_changed: false,
       config_data_str: "",
       config_data: {
@@ -371,6 +378,7 @@ export default {
         name: '',
       },
       editingConfigId: null,
+      copySourceConfigId: '',
 
       // 测试聊天
       testChatDrawer: false,
@@ -568,8 +576,9 @@ export default {
       }
     },
     createNewConfig() {
+      const configName = this.normalizeConfigName(this.configFormData.name);
       axios.post('/api/config/abconf/new', {
-        name: this.configFormData.name
+        name: configName
       }).then((res) => {
         if (res.data.status === "ok") {
           this.save_message = res.data.message;
@@ -589,6 +598,24 @@ export default {
         this.save_message_success = "error";
       });
     },
+    normalizeConfigName(name) {
+      return (name || '').trim();
+    },
+    hasDuplicateConfigName(name, excludeId = null) {
+      const normalizedName = this.normalizeConfigName(name);
+      if (!normalizedName) {
+        return false;
+      }
+      return this.configInfoList.some((config) => {
+        if (!config || !config.name) {
+          return false;
+        }
+        if (excludeId && config.id === excludeId) {
+          return false;
+        }
+        return this.normalizeConfigName(config.name) === normalizedName;
+      });
+    },
     async onConfigSelect(value) {
       if (value === '_%manage%_') {
         this.configManageDialog = true;
@@ -641,42 +668,104 @@ export default {
     startCreateConfig() {
       this.showConfigForm = true;
       this.isEditingConfig = false;
+      this.isCopyingConfig = false;
       this.configFormData = {
         name: '',
       };
       this.editingConfigId = null;
+      this.copySourceConfigId = '';
     },
     startEditConfig(config) {
       this.showConfigForm = true;
       this.isEditingConfig = true;
+      this.isCopyingConfig = false;
       this.editingConfigId = config.id;
 
       this.configFormData = {
         name: config.name || '',
       };
+      this.copySourceConfigId = '';
+    },
+    startCopyConfig(config) {
+      this.showConfigForm = true;
+      this.isEditingConfig = false;
+      this.isCopyingConfig = true;
+      this.editingConfigId = null;
+      this.copySourceConfigId = config.id;
+      this.configFormData = {
+        name: `${config.name}-copy`,
+      };
     },
     cancelConfigForm() {
       this.showConfigForm = false;
       this.isEditingConfig = false;
+      this.isCopyingConfig = false;
       this.editingConfigId = null;
       this.configFormData = {
         name: '',
       };
+      this.copySourceConfigId = '';
     },
     saveConfigForm() {
-      if (!this.configFormData.name) {
+      const normalizedName = this.normalizeConfigName(this.configFormData.name);
+      if (!normalizedName) {
         this.save_message = this.tm('configManagement.pleaseEnterName');
         this.save_message_snack = true;
         this.save_message_success = "error";
         return;
       }
-
+      const excludeId = this.isEditingConfig ? this.editingConfigId : null;
+      if (this.hasDuplicateConfigName(normalizedName, excludeId)) {
+        this.save_message = this.tm('configManagement.nameExists');
+        this.save_message_snack = true;
+        this.save_message_success = "error";
+        return;
+      }
+      this.configFormData.name = normalizedName;
       if (this.isEditingConfig) {
         this.updateConfigInfo();
+      } else if (this.isCopyingConfig) {
+        this.copyConfig();
       } else {
         this.createNewConfig();
       }
     },
+    copyConfig() {
+      const configName = this.normalizeConfigName(this.configFormData.name);
+      axios.get('/api/config/abconf', {
+        params: { id: this.copySourceConfigId }
+      }).then((res) => {
+        const sourceConfig = res.data?.data?.config;
+        if (!sourceConfig) {
+          this.save_message = this.tm('configManagement.copyFailed');
+          this.save_message_snack = true;
+          this.save_message_success = "error";
+          return;
+        }
+        return axios.post('/api/config/abconf/new', {
+          name: configName,
+          config: sourceConfig
+        });
+      }).then((res) => {
+        if (!res) return;
+        if (res.data.status === "ok") {
+          this.save_message = res.data.message;
+          this.save_message_snack = true;
+          this.save_message_success = "success";
+          this.getConfigInfoList(res.data.data.conf_id);
+          this.cancelConfigForm();
+        } else {
+          this.save_message = res.data.message;
+          this.save_message_snack = true;
+          this.save_message_success = "error";
+        }
+      }).catch((err) => {
+        console.error(err);
+        this.save_message = err?.response?.data?.message || this.tm('configManagement.copyFailed');
+        this.save_message_snack = true;
+        this.save_message_success = "error";
+      });
+    },
     async confirmDeleteConfig(config) {
       const message = this.tm('configManagement.confirmDelete').replace('{name}', config.name);
       if (await askForConfirmationDialog(message, this.confirmDialog)) {
@@ -707,9 +796,10 @@ export default {
       });
     },
     updateConfigInfo() {
+      const configName = this.normalizeConfigName(this.configFormData.name);
       axios.post('/api/config/abconf/update', {
         id: this.editingConfigId,
-        name: this.configFormData.name
+        name: configName
       }).then((res) => {
         if (res.data.status === "ok") {
           this.save_message = res.data.message;

From 51108f19b3ad6eb32693ce2cadefcbde8d7f9352 Mon Sep 17 00:00:00 2001
From: Flartiny <1335348298@qq.com>
Date: Sun, 22 Mar 2026 15:00:30 +0800
Subject: [PATCH 2/2] fix: issues mentioned by reviewer bot

- duplicated logic for initializing/resetting the config
- check whitespace-only names
---
 dashboard/src/views/ConfigPage.vue | 88 ++++++++++++++----------------
 1 file changed, 41 insertions(+), 47 deletions(-)

diff --git a/dashboard/src/views/ConfigPage.vue b/dashboard/src/views/ConfigPage.vue
index d310e82abb..cae432bf1c 100644
--- a/dashboard/src/views/ConfigPage.vue
+++ b/dashboard/src/views/ConfigPage.vue
@@ -145,9 +145,7 @@
         <v-divider v-if="showConfigForm" class="my-6"></v-divider>
 
         <div v-if="showConfigForm">
-          <h3 class="mb-4">
-            {{ isEditingConfig ? tm('configManagement.editConfig') : (isCopyingConfig ? tm('configManagement.copyConfig') : tm('configManagement.newConfig')) }}
-          </h3>
+          <h3 class="mb-4">{{ configFormTitle }}</h3>
 
           <h4>{{ tm('configManagement.configName') }}</h4>
 
@@ -157,7 +155,7 @@
           <div class="d-flex justify-end mt-4" style="gap: 8px;">
             <v-btn variant="text" @click="cancelConfigForm">{{ tm('buttons.cancel') }}</v-btn>
             <v-btn color="primary" @click="saveConfigForm"
-              :disabled="!configFormData.name || (isCopyingConfig && !copySourceConfigId)">
+              :disabled="isConfigFormSaveDisabled">
               {{ isEditingConfig ? tm('buttons.update') : tm('buttons.create') }}
             </v-btn>
           </div>
@@ -303,6 +301,19 @@ export default {
     selectedConfigInfo() {
       return this.configInfoList.find(info => info.id === this.selectedConfigID) || {};
     },
+    configFormTitle() {
+      if (this.isEditingConfig) {
+        return this.tm('configManagement.editConfig');
+      }
+      if (this.isCopyingConfig) {
+        return this.tm('configManagement.copyConfig');
+      }
+      return this.tm('configManagement.newConfig');
+    },
+    isConfigFormSaveDisabled() {
+      const isNameEmpty = !this.normalizeConfigName(this.configFormData.name);
+      return isNameEmpty || (this.isCopyingConfig && !this.copySourceConfigId);
+    },
     configSelectItems() {
       const items = [...this.configInfoList];
       items.push({
@@ -575,8 +586,7 @@ export default {
         this.save_message_snack = true;
       }
     },
-    createNewConfig() {
-      const configName = this.normalizeConfigName(this.configFormData.name);
+    createNewConfig(configName) {
       axios.post('/api/config/abconf/new', {
         name: configName
       }).then((res) => {
@@ -599,7 +609,7 @@ export default {
       });
     },
     normalizeConfigName(name) {
-      return (name || '').trim();
+      return typeof name === 'string' ? name.trim() : '';
     },
     hasDuplicateConfigName(name, excludeId = null) {
       const normalizedName = this.normalizeConfigName(name);
@@ -665,46 +675,32 @@ export default {
         }
       }
     },
+    setConfigFormState({ mode = 'create', config = null, visible = true } = {}) {
+      this.showConfigForm = visible;
+      this.isEditingConfig = mode === 'edit';
+      this.isCopyingConfig = mode === 'copy';
+      this.editingConfigId = this.isEditingConfig && config ? config.id : null;
+      this.copySourceConfigId = this.isCopyingConfig && config ? config.id : '';
+
+      let name = '';
+      if (this.isEditingConfig && config) {
+        name = config.name || '';
+      } else if (this.isCopyingConfig && config) {
+        name = `${config.name || ''}-copy`;
+      }
+      this.configFormData = { name };
+    },
     startCreateConfig() {
-      this.showConfigForm = true;
-      this.isEditingConfig = false;
-      this.isCopyingConfig = false;
-      this.configFormData = {
-        name: '',
-      };
-      this.editingConfigId = null;
-      this.copySourceConfigId = '';
+      this.setConfigFormState({ mode: 'create' });
     },
     startEditConfig(config) {
-      this.showConfigForm = true;
-      this.isEditingConfig = true;
-      this.isCopyingConfig = false;
-      this.editingConfigId = config.id;
-
-      this.configFormData = {
-        name: config.name || '',
-      };
-      this.copySourceConfigId = '';
+      this.setConfigFormState({ mode: 'edit', config });
     },
     startCopyConfig(config) {
-      this.showConfigForm = true;
-      this.isEditingConfig = false;
-      this.isCopyingConfig = true;
-      this.editingConfigId = null;
-      this.copySourceConfigId = config.id;
-      this.configFormData = {
-        name: `${config.name}-copy`,
-      };
+      this.setConfigFormState({ mode: 'copy', config });
     },
     cancelConfigForm() {
-      this.showConfigForm = false;
-      this.isEditingConfig = false;
-      this.isCopyingConfig = false;
-      this.editingConfigId = null;
-      this.configFormData = {
-        name: '',
-      };
-      this.copySourceConfigId = '';
+      this.setConfigFormState({ visible: false });
     },
     saveConfigForm() {
       const normalizedName = this.normalizeConfigName(this.configFormData.name);
@@ -723,15 +719,14 @@ export default {
       }
       this.configFormData.name = normalizedName;
       if (this.isEditingConfig) {
-        this.updateConfigInfo();
+        this.updateConfigInfo(normalizedName);
       } else if (this.isCopyingConfig) {
-        this.copyConfig();
+        this.copyConfig(normalizedName);
       } else {
-        this.createNewConfig();
+        this.createNewConfig(normalizedName);
       }
     },
-    copyConfig() {
-      const configName = this.normalizeConfigName(this.configFormData.name);
+    copyConfig(configName) {
       axios.get('/api/config/abconf', {
         params: { id: this.copySourceConfigId }
       }).then((res) => {
@@ -795,8 +790,7 @@ export default {
         this.save_message_success = "error";
       });
     },
-    updateConfigInfo() {
-      const configName = this.normalizeConfigName(this.configFormData.name);
+    updateConfigInfo(configName) {
       axios.post('/api/config/abconf/update', {
         id: this.editingConfigId,
         name: configName