coder999o opened a new pull request, #6635: URL: https://github.com/apache/rocketmq-dashboard/pull/6635
## What is the purpose of the change Business-key trace lookups can contain different messages, but the diagnostics calculate their combined timestamps as one lifecycle. Two successful 10 ms messages sent a minute apart can consequently appear as a critical 60-second delay. Fixes #6634 ## Brief changelog - Attach lookup context to completed frontend trace results, preserving null results and existing status normalization. - Leave lifecycle latency unavailable for Key results without per-message correlation; retain real node costs, timestamps, failures and retry evidence. - Explain the timing boundary in Chinese and English. Editing the input mode does not reinterpret a displayed result; a new Message-ID lookup restores lifecycle timing. - Add regressions for both stage orders, retained failure evidence and the bilingual page flow. This changes frontend interpretation only, without changing the backend response or database schema. A Key result containing one message also leaves lifecycle timing unavailable because the payload does not establish per-node message identity. ## Verifying this change - `cd web && npm test -- src/utils/messageTraceDiagnostics.test.ts src/api/message.test.ts src/pages/instance/__tests__/MessagePage.test.tsx src/pages/instance/__tests__/MessagePageAsyncState.test.tsx src/services/messageService.test.ts` (81 passed). - `cd web && npx tsc -p tsconfig.app.json --noEmit --composite false --incremental false` (passed). - `cd web && npx eslint src/api/message.ts src/api/message.test.ts src/utils/messageTraceDiagnostics.ts src/utils/messageTraceDiagnostics.test.ts src/pages/instance/message.tsx src/pages/instance/__tests__/MessagePage.test.tsx src/i18n/translations.ts` (passed). - `cd web && npx prettier --check src/api/message.ts src/api/message.test.ts src/utils/messageTraceDiagnostics.ts src/utils/messageTraceDiagnostics.test.ts src/pages/instance/message.tsx src/pages/instance/__tests__/MessagePage.test.tsx src/i18n/translations.ts` (passed). - `git diff --check` (passed). The new behavior was verified failing before the fix. Tests used frontend fixtures and mocked APIs; the full frontend suite and live Broker integration were not run. -- 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]
