chore: avoid menu duplicated

Author: LinuxSuRen

console/atest-ui/src/App.vue

@@ -8,6 +8,7 @@ import {
     Guide,
     DataAnalysis, Help, Setting
 } from '@element-plus/icons-vue'
+import * as ElementPlusIcons from '@element-plus/icons-vue'
 import { ref, watch, getCurrentInstance} from 'vue'
 import { API } from './views/net'
 import { Cache } from './views/cache'
@@ -152,17 +153,13 @@ API.GetMenus((menus) => {
           <el-icon><Guide /></el-icon>
           <template #title>{{ t('title.mock' )}}</template>
         </el-menu-item>
-        <el-menu-item index="data" test-id="data-menu">
-          <el-icon><DataAnalysis /></el-icon>
-          <template #title>{{ t('title.data' )}}</template>
-        </el-menu-item>
         <el-menu-item index="store">
           <el-icon><location /></el-icon>
           <template #title>{{ t('title.stores') }}</template>
         </el-menu-item>
         <span v-for="menu in extensionMenus" :key="menu.index" :index="menu.index">
             <el-menu-item :index="menu.index">
-                <el-icon><IconMenu /></el-icon>
+                <el-icon><component :is="ElementPlusIcons[menu.icon]" /></el-icon>
                 <template #title>{{ menu.name }}</template>
             </el-menu-item>
         </span>

console/atest-ui/src/views/Extension.vue

@@ -10,7 +10,6 @@ const loadPlugin = async (): Promise<void> => {
     try {
         API.GetPageOfCSS(props.name, (d) => {
             const style = document.createElement('style');
-            style.type = 'text/css';
             style.textContent = d.message;
             document.head.appendChild(style);
         });
@@ -26,7 +25,10 @@ const loadPlugin = async (): Promise<void> => {
             if (plugin && plugin.mount) {
                 console.log('extension load success');
                 const container = document.getElementById("plugin-container");
-                plugin.mount(container);
+                if (container) {
+                    container.innerHTML = ''; // Clear previous content
+                    plugin.mount(container);
+                }
             }
         });
     } catch (error) {

pkg/server/remote_server.go

@@ -1395,13 +1395,28 @@ func (s *server) GetMenus(ctx context.Context, _ *Empty) (result *MenuList, err
 		return
 	}
 
+	duplicatedMenus := make(map[string]int) // index is key, value is the slice index
 	for _, loader := range loaders {
 		if menus, mErr := loader.GetMenus(); mErr == nil {
 			for _, menu := range menus {
 				if isSystemMenu(menu.Index) {
 					serverLogger.Info("skip due to conflict with system name", "menu", menu)
 					continue
 				}
+
+				// take the bigger version if the menu index is duplicated
+				if existingIndex, exists := duplicatedMenus[menu.Index]; exists {
+					if menu.Version > result.Data[existingIndex].Version {
+						result.Data[existingIndex] = &Menu{
+							Name:  menu.Name,
+							Icon:  menu.Icon,
+							Index: menu.Index,
+						}
+					}
+					continue
+				}
+
+				duplicatedMenus[menu.Index] = len(result.Data)
 				result.Data = append(result.Data, &Menu{
 					Name:  menu.Name,
 					Icon:  menu.Icon,
@@ -1410,12 +1425,16 @@ func (s *server) GetMenus(ctx context.Context, _ *Empty) (result *MenuList, err
 			}
 		}
 	}
+
+	slices.SortFunc(result.Data, func(a, b *Menu) int {
+		return strings.Compare(a.Index, b.Index)
+	})
 	return
 }
 
 func isSystemMenu(index string) bool {
 	switch index {
-	case "testing", "history", "data", "mock", "store", "welcome", "":
+	case "testing", "history", "mock", "store", "welcome", "":
 		return true
 	}
 	return false