pkg/models/user_group.go (3)

16-16: MenuData JSON 序列化策略与现有字段不一致，建议补充 omitempty 减少冗余输出

同一结构体内其他字段普遍使用了 json:",omitempty"，而 menu_data 未加该选项。空字符串会始终下发到前端，占用带宽且易引起语义歧义。

可以考虑如下调整：

-    MenuData    string    `gorm:"type:text" json:"menu_data"`
+    MenuData    string    `gorm:"type:text" json:"menu_data,omitempty"`

---

16-16: 持久化类型为 text 可能不够稳妥，建议评估 datatypes.JSON 或更大容量类型

• text 在 MySQL/MariaDB 上最大约 64KB；若菜单配置较大（历史版本、注释、图标等堆叠），存在溢出风险。
• 为兼容多数据库方言与更清晰的语义，可考虑使用 gorm.io/datatypes.JSON；或至少在 MySQL 上改为 longtext。

可选方案 A（推荐，跨方言更健壮）：

- MenuData    string    `gorm:"type:text" json:"menu_data,omitempty"`
+ // 如需跨方言可靠的 JSON 存储与查询能力，考虑直接用 JSON 类型
+ MenuData    datatypes.JSON `json:"menu_data,omitempty"`

可选方案 B（仅 MySQL/MariaDB 提升容量）：

- MenuData    string    `gorm:"type:text" json:"menu_data,omitempty"`
+ MenuData    string    `gorm:"type:longtext" json:"menu_data,omitempty"`

---

16-16: 接口层建议对 MenuData 做 JSON 校验与限流/限长，避免无效或过大入库

该字段为前端提交的 JSON 字符串，建议在 controller/service 层：

• 校验 JSON 结构与大小（例如限制 256KB/1MB，按需设定）。
• 拒绝无效 JSON，避免脏数据。
• 对历史版本（若有）做留存数量控制，防止无限增长。

如需，我可以补一段保存前的校验逻辑示例。

ui/src/types/menu.ts (3)

6-8: 用类型区分不同 eventType 的必填/互斥字段，提升类型安全

目前 url 与 customEvent 均为可选，运行期才会发现缺失。建议使用可判别联合，静态约束字段的存在性与互斥关系。

参考改造示例（如不影响现有调用）：

-export interface MenuItem {
-    key: string;
-    title: string;
-    icon?: string;
-    url?: string;
-    eventType?: 'url' | 'custom';
-    customEvent?: string;
-    order?: number;
-    children?: MenuItem[];
-    show?: string; // 修改这里，只保留字符串类型
-}
+type BaseMenuItem = {
+  key: string;
+  title: string;
+  icon?: string;
+  order?: number;
+  children?: MenuItem[];
+  show?: string;
+}
+
+type UrlMenuItem = BaseMenuItem & {
+  eventType: 'url';
+  url: string;
+  customEvent?: never;
+}
+
+type CustomEventMenuItem = BaseMenuItem & {
+  eventType: 'custom';
+  customEvent: string;
+  url?: never;
+}
+
+type GroupMenuItem = BaseMenuItem & {
+  eventType?: undefined; // 纯分组节点
+  url?: never;
+  customEvent?: never;
+}
+
+export type MenuItem = UrlMenuItem | CustomEventMenuItem | GroupMenuItem;

---

10-10: 为 show 表达式定义类型别名并补充注释，约定语法，降低使用歧义

当前 show?: string 缺少语法约束（如可用变量/函数、返回值语义）。建议引入别名并在注释中描述语法约定，便于全局复用和 IDE 提示。

-    show?: string; // 修改这里，只保留字符串类型
+    // show 表达式，返回 truthy 即显示。建议：限定可用变量（如 userRole、clusterStatus 等）与函数。
+    show?: VisibilityExpr;

文件头部新增：

export type VisibilityExpr = string; // 统一的可见性表达式类型别名

---

2-2: 命名建议：如该结构常在 JSX 中透传，考虑用 id 替代 key

当对象被直接展开到组件 props 时，key 是 React 保留属性，不会出现在组件 props 内，易引起困惑。若 MenuItem 会被直接作为 props 传递，建议改为 id 或 menuKey。

ui/src/components/IconPicker/index.module.scss (2)

18-40: .selected 写法在 CSS Modules 下是“全局类”，建议改为导出类，避免样式泄漏

&.selected 生成的 selected 不会哈希，需在组件里加原生类名，且可能与全局样式冲突。建议导出一个选择态类并在组件中组合使用。

可修改为：

 .icon-item {
   display: flex;
   align-items: center;
   justify-content: center;
   width: 48px;
   height: 48px;
   border: 1px solid #eee;
   border-radius: 4px;
   cursor: pointer;
   transition: all 0.2s;
   font-size: 24px;

   &:hover {
     background-color: #f0f7ff;
     border-color: #1890ff;
   }

-  &.selected {
-    background-color: #e6f7ff;
-    border-color: #1890ff;
-    color: #1890ff;
-  }
 }
+
+/* 导出的“选中态”类，在 TSX 中组合：className={`${styles['icon-item']} ${selected ? styles.isSelected : ''}`} */ 
+.isSelected {
+  background-color: #e6f7ff;
+  border-color: #1890ff;
+  color: #1890ff;
+}

同时把组件里的类名从字符串 'selected' 切换为 styles.isSelected。

---

8-15: 提升栅格适配性：使用 auto-fill/minmax 或固定列宽，避免 tile 与单元格尺寸错位

当前列宽为 1fr，而 .icon-item 固定 48px，导致单元格宽度和 tile 宽度不一致。两种可选方案：

方案 A（自适应列数，最小 48px）：

-  grid-template-columns: repeat(8, 1fr);
+  grid-template-columns: repeat(auto-fill, minmax(48px, 1fr));

方案 B（固定 8 列且 tile 撑满单元格）：

-  grid-template-columns: repeat(8, 1fr);
+  grid-template-columns: repeat(8, 1fr);

并将 .icon-item 的宽高改为百分比：

-  width: 48px;
-  height: 48px;
+  width: 100%;
+  height: 100%;

二选一即可，取决于是否需要响应式列数。

ui/src/routes/index.tsx (2)

9-9: MenuEditor 建议按需加载，避免首屏包体增大

当前直接静态引入。与后文 lazyLoad 工具保持一致，可懒加载该页面，减小首屏体积。

-import MenuEditor from '@/pages/MenuEditor'
+const MenuEditor = lazy(() => import('@/pages/MenuEditor'));

---

25-25: 将路由元素改为 lazy 包装，统一懒加载写法

复用现有 lazyLoad 辅助函数。

-            <Route path='/MenuEditor' element={<MenuEditor/>}></Route>
+            <Route path='/MenuEditor' element={lazyLoad(MenuEditor)}></Route>

ui/src/utils/iconOptions.ts (2)

1-3: 为 iconOptions 增加显式类型与只读限定，提升可维护性与类型安全

当前是隐式 any[]，建议加上类型声明并标记只读，避免后续被误改。

-// 所有Font Awesome实心图标选项
-const iconOptions = [
+// 所有Font Awesome实心图标选项
+export type IconOption = { value: string };
+const iconOptions: ReadonlyArray<IconOption> = [

---

1-1006: 发现重复的图标项：fa-wifi

• 当前 ui/src/utils/iconOptions.ts 中存在重复项：fa-wifi
• 建议：
  • 在构建或导出阶段对 iconOptions 做去重，移除重复值
  • 将超大静态列表抽离为独立 JSON 文件，并在 IconPicker 首次打开时按需加载，以降低首屏包体积
  • 或者在运行时对加载的数据做懒加载/分页，避免整个列表常驻主包

[optional_refactors_recommended]

pkg/service/user.go (2)

26-64: 返回 any 导致前后端类型不一致，建议改为 string 并用空串表示缺省

当前返回类型为 any，Controller 无法感知并最终把 null 下发到前端，导致 useUserRole 以 string 处理时潜在运行时问题。建议改为 string，并在未配置时返回空串。

-func (u *userService) GetGroupMenuData(groupNames string) (any, error) {
+func (u *userService) GetGroupMenuData(groupNames string) (string, error) {
 	if groupNames == "" {
-		return nil, nil
+		return "", nil
 	}
 
 	// 按逗号分割，取第一个组名
 	groupNameList := strings.Split(groupNames, ",")
 	firstGroupName := strings.TrimSpace(groupNameList[0])
 
 	if firstGroupName == "" {
-		return nil, nil
+		return "", nil
 	}
 
 	cacheKey := u.formatCacheKey("user:groupmenu:%s", firstGroupName)
 
-	result, err := utils.GetOrSetCache(CacheService().CacheInstance(), cacheKey, 5*time.Minute, func() (any, error) {
+	result, err := utils.GetOrSetCache(CacheService().CacheInstance(), cacheKey, 5*time.Minute, func() (string, error) {
 		params := &dao.Params{}
 		userGroup := &models.UserGroup{}
 		queryFunc := func(db *gorm.DB) *gorm.DB {
 			return db.Select("menu_data").Where("group_name = ?", firstGroupName)
 		}
 
 		item, err := userGroup.GetOne(params, queryFunc)
 		if err != nil {
 			if errors.Is(err, gorm.ErrRecordNotFound) {
-				return nil, nil
+				return "", nil
 			}
-			return nil, err
+			return "", err
 		}
 
 		if item.MenuData == "" {
-			return nil, nil
+			return "", nil
 		}
 
 		return item.MenuData, nil
 	})
 
 	return result, err
 }

如采纳上述改动，请同步更新调用处 Controller 的变量类型。

---

39-41: TTL 常量化建议

5 分钟 TTL 多处硬编码，建议抽到常量或配置，便于统一调整和可测试性。

pkg/controller/admin/user/user_group.go (1)

82-116: 仅更新 menu_data 字段，避免误更新其他字段

当前使用 Save 可能将未预期字段一并写回（取决于 Save 实现）。本处仅需更新 menu_data，建议使用 UpdateColumn 精确更新。

-    // 更新菜单数据
-    userGroup.MenuData = requestData.MenuData
-    err = userGroup.Save(params)
+    // 仅更新菜单字段，避免误改其他字段
+    err = userGroup.UpdateColumn("menu_data", requestData.MenuData)
     if err != nil {
         amis.WriteJsonError(c, err)
         return
     }

ui/src/components/IconPicker/index.tsx (2)

15-15: 非变动数据不需要放入 state，简化为常量并减少不必要的 effect 依赖

iconsPerPage 不会动态改变，没必要放入 React state。简化为常量也能减少 useEffect 的依赖与无效触发。

-  const [iconsPerPage] = useState(64); // 8x8网格
+  const iconsPerPage = 64; // 8x8网格

-  }, [currentPage, iconsPerPage]);
+  }, [currentPage]);

Also applies to: 23-23

---

18-24: 打开弹窗时重置页码到第一页

从 UX 上讲，用户每次打开选择器看到第 1 页更自然。

   useEffect(() => {
     const startIndex = (currentPage - 1) * iconsPerPage;
     const endIndex = startIndex + iconsPerPage;
     setDisplayIcons(iconOptions.slice(startIndex, endIndex));
   }, [currentPage, iconsPerPage]);
 
+  // 打开时重置页码
+  useEffect(() => {
+    if (open) {
+      setCurrentPage(1);
+    }
+  }, [open]);

ui/src/hooks/useCRDStatus.ts (2)

23-28: 提高健壮性：后端字段缺失时避免向 setState 传入 undefined

当前直接解构 response.data.data 并赋值，如果后端某字段缺失，可能传入 undefined 给 useState<boolean> 的 setter（尽管 TS 可能放过，运行时仍有隐患）。建议做布尔化处理。

-                    const status = response.data.data as CRDSupportedStatus;
-                    setIsGatewayAPISupported(status.IsGatewayAPISupported);
-                    setIsOpenKruiseSupported(status.IsOpenKruiseSupported);
-                    setIsIstioSupported(status.IsIstioSupported);
+                    const status = (response.data?.data || {}) as Partial<CRDSupportedStatus>;
+                    setIsGatewayAPISupported(!!status.IsGatewayAPISupported);
+                    setIsOpenKruiseSupported(!!status.IsOpenKruiseSupported);
+                    setIsIstioSupported(!!status.IsIstioSupported);

---

15-35: 避免组件卸载后 setState 引发内存泄漏或控制台警告

为异步请求添加取消标记，防止卸载后仍更新状态。

   useEffect(() => {
-        const fetchCRDSupportedStatus = async () => {
+        let cancelled = false;
+        const fetchCRDSupportedStatus = async () => {
             try {
                 const response = await fetcher({
                     url: '/k8s/crd/status',
                     method: 'get'
                 });
                 
                 if (response.data && typeof response.data === 'object') {
-                    const status = response.data.data as CRDSupportedStatus;
-                    setIsGatewayAPISupported(status.IsGatewayAPISupported);
-                    setIsOpenKruiseSupported(status.IsOpenKruiseSupported);
-                    setIsIstioSupported(status.IsIstioSupported);
+                    if (cancelled) return;
+                    const status = (response.data?.data || {}) as Partial<CRDSupportedStatus>;
+                    setIsGatewayAPISupported(!!status.IsGatewayAPISupported);
+                    setIsOpenKruiseSupported(!!status.IsOpenKruiseSupported);
+                    setIsIstioSupported(!!status.IsIstioSupported);
                 }
             } catch (error) {
                 console.error('Failed to fetch CRD status:', error);
             }
         };
 
         fetchCRDSupportedStatus();
-    }, []);
+        return () => { cancelled = true; };
+    }, []);

ui/src/pages/MenuEditor/menuData.ts (3)

924-926: 清理测试/占位文案

“关于333”疑似调试残留，建议改回正式文案“关于”。

-        title: '关于333',
+        title: '关于',

---

125-131: 精简可见性表达式：移除多余的 ==true

expr-eval 中函数已返回布尔值，可直接写 isOpenKruiseSupported() 等，提升可读性。

示例：

-        show: 'isOpenKruiseSupported()==true',
+        show: 'isOpenKruiseSupported()',

类似处：Istio、GatewayAPI、平台设置、Helm 仓库等。

Also applies to: 246-251, 472-477, 745-751, 371-381

---

7-8: 统一图标类名约定（与渲染端一致）

当前数据中多处直接存储完整的 FA 类（如 fa-solid fa-cube），而渲染端（Preview/Picker）有的会再加 fa-solid 前缀，导致重复。建议约定 MenuItem.icon 始终为“完整类名字符串”，渲染时直接使用 className={item.icon}，避免重复与歧义。

如需我统一生成数据内的类名规范（或脚本检测不规范项），请告知。

Also applies to: 15-16, 31-32, 247-249, 368-376, 918-921

ui/src/utils/menuVisibility.ts (2)

22-25: 移除未使用的 evalContext，或补充必要变量

evalContext 仅含 user.role，但表达式并未用到该变量，保持为空对象即可，避免误导。

-            const evalContext = {
-                user: { role: 'user' },
-            };
+            // 无需传递上下文变量，均通过注入函数获得

-            const result = expr.evaluate(evalContext);
+            const result = expr.evaluate({});

Also applies to: 57-59

---

16-18: 类型与实现不一致：show 已限定为 string，分支冗余

MenuItem.show 在类型定义中已限定为字符串（见 types/menu.ts 注释“只保留字符串类型”）。这里保留 boolean 分支会造成心智负担。建议二选一：

• 要兼容旧数据：更新类型为 string | boolean 并标注迁移计划；
• 若不再兼容：去掉 boolean 分支，简化逻辑。

我可以按你的意向提交相应重构补丁。

ui/src/pages/MenuEditor/Preview.tsx (2)

34-40: 移除未使用的 _menuData 字段，保持上下文语义纯净

visibilityContext 中的 _menuData 未被 shouldShowMenuItem 使用，易引起误解。

     const visibilityContext = {
         userRole,
-        _menuData,
         isGatewayAPISupported,
         isOpenKruiseSupported,
         isIstioSupported
     };

---

108-114: 可提升交互的一点建议：用 Tree 的 onSelect 统一处理点击

当前通过包裹 span 的 onClick 处理，容易与 Tree 的选择态脱节。可以考虑改为 onSelect={(keys) => handleClick(keys[0] as string)}，让整行节点点击都一致触发。

这是可选优化，不影响功能。

ui/src/pages/MenuEditor/CustomEventTags.tsx (1)

91-94: 事件处理逻辑可能存在浏览器兼容性问题

使用 tagName.toLowerCase() 检查 SVG 元素来阻止事件冒泡的方式可能不够可靠，因为关闭按钮的实现细节可能会变化。

建议使用更可靠的方式来区分点击事件：

-                        onClick={(e) => {
-                            // 防止点击关闭按钮时触发 tag 的点击事件
-                            if ((e.target as HTMLElement).tagName.toLowerCase() !== 'svg') {
-                                onChange?.(tag.value);
-                            }
+                        onClick={(e) => {
+                            // 使用 currentTarget 和 target 的比较更可靠
+                            if (e.target === e.currentTarget) {
+                                onChange?.(tag.value);
+                            }

ui/src/components/Sidebar/index.tsx (2)

45-46: 正则表达式可能无法匹配所有有效的路径格式

当前的正则表达式 /loadJsonPage\("([^"]+)"\)/ 只能匹配双引号格式，如果将来 customEvent 使用单引号或模板字符串，将无法正确提取路径。

建议使用更灵活的正则表达式：

-                    const match = customEvent.match(/loadJsonPage\("([^"]+)"\)/);
+                    const match = customEvent.match(/loadJsonPage\(["'`]([^"'`]+)["'`]\)/);

---

104-111: useMemo 依赖项中包含 navigate 可能导致不必要的重新计算

navigate 函数在每次组件重新渲染时都会创建新的引用，这会导致 menuItems 频繁重新计算。

建议将 navigate 从依赖项中移除，因为它的功能不会改变：

     const menuItems = useMemo(() => {
         const menuDataToUse = getMenuData();
         return convertMenuItems(menuDataToUse);
     }, [
-        navigate,
         userRole,
         menuData,
         isGatewayAPISupported,
         isOpenKruiseSupported,
         isIstioSupported
     ]);

pkg/controller/admin/menu/menu.go (1)

117-122: 缺少输入验证

strconv.Atoi 转换失败时会返回错误，但没有验证 ID 是否为正数。

建议添加 ID 有效性检查：

     id, err := strconv.Atoi(idStr)
     if err != nil {
         amis.WriteJsonError(c, err)
         return
     }
+    if id <= 0 {
+        amis.WriteJsonError(c, fmt.Errorf("invalid ID: %d", id))
+        return
+    }

pkg/models/menu.go (1)

79-81: 方法接收者命名不一致

Save 方法的接收者命名为 c，而其他方法都使用 m，这种不一致可能会造成困惑。

-func (c *Menu) Save(params *dao.Params, queryFuncs ...func(*gorm.DB) *gorm.DB) error {
-    return dao.GenericSave(params, c, queryFuncs...)
+func (m *Menu) Save(params *dao.Params, queryFuncs ...func(*gorm.DB) *gorm.DB) error {
+    return dao.GenericSave(params, m, queryFuncs...)
 }

ui/public/pages/admin/user/user_group.json (4)

5-10: 信息提示内容可读性与一致性微调（避免内联重复 alert 类、统一括号风格）

• 当前在 Alert 组件内部再次包裹 <div class='alert alert-info'>，样式重复，容易导致不一致展示。
• 标题中使用了全角括号“（ESC 关闭）”，而页面其他处使用半角括号“(ESC 关闭)”。建议统一。

可考虑直接让 Alert 渲染纯内容，移除内层 div 包裹；同时开启 showIcon 增强可读性。

       "type": "alert",
       "level": "info",
-      "body": "<div class='alert alert-info'><p><strong>菜单配置说明：</strong></p><ul><li><strong>自定义菜单：</strong>如果用户组设置了菜单数据，系统将使用该用户组的自定义菜单配置，覆盖默认菜单</li><li><strong>默认菜单：</strong>如果用户组未设置菜单数据，系统将使用默认的菜单配置</li><li><strong>多用户组冲突：</strong>当用户归属多个用户组且出现菜单冲突时，按照用户第一个用户组的菜单配置为准</li><li><strong>菜单优先级：</strong>用户组自定义菜单 > 系统默认菜单</li></ul></div>",
+      "showIcon": true,
+      "body": "<p><strong>菜单配置说明：</strong></p><ul><li><strong>自定义菜单：</strong>如果用户组设置了菜单数据，系统将使用该用户组的自定义菜单配置，覆盖默认菜单</li><li><strong>默认菜单：</strong>如果用户组未设置菜单数据，系统将使用默认的菜单配置</li><li><strong>多用户组冲突：</strong>当用户归属多个用户组且出现菜单冲突时，按照用户第一个用户组的菜单配置为准</li><li><strong>菜单优先级：</strong>用户组自定义菜单 > 系统默认菜单</li></ul>",
       "className": "mb-3"

---

195-209: 将通用编辑器替换为 JSON 专用控件，确保类型安全与校验体验

当前使用 type: editor + language: json。如果后端返回的是对象（而非字符串），Monaco 可能收到非字符串初始值，存在兼容风险。建议改为 amis 的 input-json 组件，内置格式化与校验，更贴合场景；同时对初始值使用 json 过滤器，保证为字符串。

-                    {
-                      "type": "editor",
-                      "name": "menu_data",
-                      "size": "xl",
-                      "required": true,
-                      "allowFullscreen": true,
-                      "placeholder": "请粘贴菜单JSON配置",
-                      "language": "json",
-                      "value": "${menu_data}",
-                      "options": {
-                        "wordWrap": "on",
-                        "scrollbar": {
-                          "vertical": "auto"
-                        }
-                      }
-                    },
+                    {
+                      "type": "input-json",
+                      "name": "menu_data",
+                      "height": 420,
+                      "required": true,
+                      "placeholder": "请粘贴菜单 JSON 配置",
+                      "value": "${menu_data|json}",
+                      "validateOnChange": true
+                    },

---

218-231: 建议补充失败态提示，提升可观测性与可用性

保存失败时目前无反馈。补充 submitFail 事件，toast 提示具体错误，有助于定位 JSON 校验/后端报错。

                   "onEvent": {
                     "submitSucc": {
                       "actions": [
                         {
                           "actionType": "reload",
                           "componentId": "groupCRUD"
                         },
                         {
                           "actionType": "closeDialog"
                         }
                       ]
                     }
+                  ,
+                    "submitFail": {
+                      "actions": [
+                        {
+                          "actionType": "toast",
+                          "args": {
+                            "msgType": "error",
+                            "msg": "保存失败：${lastError || '请检查 JSON 是否为合法格式，或联系管理员查看后端日志'}"
+                          }
+                        }
+                      ]
+                    }
                   }

---

183-191: 安全细节：target=_blank 打开新标签的反制措施

使用 "blank": true 可能仅设置 target=_blank。为防止反向标签劫持（reverse tabnabbing），通常需要 rel="noopener noreferrer"。请确认 amis 的 url 动作是否默认添加 rel 属性；若未添加，建议升级 amis 或采用受控链接组件以携带 rel。

是否需要我帮忙检索 amis 文档并确认该属性的默认行为？

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled
• Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.