zmuxuny opened a new pull request, #6379:
URL: https://github.com/apache/rocketmq-dashboard/pull/6379

   ### Which Issue(s) This PR Fixes
   
   Fixes #6360
   
   ### Brief Description
   
   Audit risk insights currently use display fallbacks as identity: a missing 
cluster becomes the literal string global, and a missing target becomes -. 
Actual clusters/targets with those names therefore collapse into the same 
group. Separate one-off failures can appear as a repeated failing target, and 
independent failure counts are combined.
   
   Use a structured tuple with null for missing identity fields. Keep existing 
normalization and display fallbacks unchanged. The key is internal to 
grouping/table rendering; no API or storage format changes.
   
   ### How Did You Test This Change?
   
   Base: rocketmq-studio at 779b8c0e7aed2396d3cc72743dcd544ea48e9ae6.
   
   - Four new tests fail on original production (11 existing tests pass): 
missing versus literal global, independent repeated groups in both record 
orders, missing versus literal '-' target, and the real AuditRiskInsights 
panel's false repeat signal.
   - `node node_modules/vitest/vitest.mjs run --maxWorkers=1 
src/pages/ops/__tests__/auditRiskInsights.test.ts 
src/pages/ops/__tests__/AuditRiskInsights.test.tsx 
src/pages/ops/__tests__/AuditPage.test.tsx 
src/pages/ops/__tests__/auditPresentation.test.ts`: 4 files / 39 tests passed.
   - Independent review and independent 15-test model/panel rerun passed.
   - Full frontend lint: 0 errors / 10 existing warnings. Changed-file Prettier 
and diff check passed.
   - Full TypeScript/Vite production build and license check passed (133 
components / 140 files).
   - Synthetic audit fixtures only. Full frontend suite/backend tests were not 
run locally; CI remains to be verified.
   
   ### Checklist
   
   - [x] One coherent change and Conventional Commit
   - [x] Model and real-panel regressions added with Test suffix
   - [x] No new UI text, files, dependencies, or configuration
   - [ ] Backend architecture checks (not run locally; frontend-only change)
   - [x] No public API or documented behavior change
   


-- 
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]

Reply via email to