yuluo-yx opened a new pull request, #4399: URL: https://github.com/apache/rocketmq-dashboard/pull/4399
<!-- Make sure the base branch is `master`: that is the RocketMQ Studio trunk. --> ### Which Issue(s) This PR Fixes - Fixes #3148 - Replaces #3149, closed by the branch switch in #4379. ### Brief Description This is a resubmission of #3149 against `master`, following the branch switch notice in #4379. The fork branch is unchanged; `master` is now the RocketMQ Studio trunk. ## What changed - add a read-only portfolio review to both cluster and business alert pages - review the complete domain rule inventory rather than only the current page - identify enabled rules with identical evaluation signatures, duplicate names, enabled rules without notification channels, deleted instance references, and disabled-only scopes - keep distinct thresholds separate to avoid false conflict reports - provide issue and rule views with state, severity, issue, and text filters - export the currently filtered rule review to CSV - add Chinese/English copy and focused pure-function/component tests ## Design boundaries - no rule mutations are performed - no findings are inferred from runtime samples - global rules without an instance are valid - only enabled rules participate in exact duplicate evaluation detection - equivalent duration spellings such as `1m` and `60s` are normalized Fixes #3148 ## 行为与兼容性 用结构化作用域字段消除展示分隔符造成的身份碰撞；按后端语法归一化复合时长，使用大整数避免精度丢失。 ### How Did You Test This Change? ## 验证 - 相关前端测试：13 项通过。 - ESLint、TypeScript/Vite 生产构建、合并差异检查通过。 - 已合入最新 `rocketmq-studio`，使用普通推送保留原提交历史。 - 未执行远端工作流、浏览器 E2E 或真实 Broker 测试。 验证日期：2026-09-14。 ### Checklist - [x] One coherent change; unrelated modifications are not bundled in - [x] Commit subject follows Conventional Commits (`feat:` / `fix:` / `refactor:` / `chore:` / `docs:` / `perf:`) - [x] Tests added or updated for non-trivial changes, test methods named `...Test` - [x] New UI text has both Chinese and English entries under `web/src/i18n/`, or the change does not add UI text - [x] Architecture constraints stay green (`mvn test` runs the ArchUnit checks), or the original verification scope is stated above - [x] New source files carry the ASF license header, or no new source files were added - [x] Documentation touched where behaviour changed (README / `docs/` / in-app help), or no documentation change was required -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
