Loyal-Young opened a new pull request, #6434:
URL: https://github.com/apache/rocketmq-dashboard/pull/6434

   ### Which Issue(s) This PR Fixes
   
   Fixes #6429
   
   ### Brief Description
   
   An empty or whitespace-only metric sample was coerced to zero by `Number()`. 
Reuse the existing finite-number parser so missing values do not become 
invented measurements in chart points, summaries, sample details, or CSV 
exports. Explicit zero and finite scientific notation remain valid.
   
   ### How Did You Test This Change?
   
   - Focused Vitest suite `src/utils/metricsExplorerDiagnostics.test.ts`: **12 
passed**; both added regressions failed before the fix.
   - Tests exercise mixed valid/blank values and blank-only series across the 
chart, summary, detail and export transformations.
   - ESLint on changed files and full application TypeScript check passed.
   - New test code formatted with Prettier; unrelated baseline formatting was 
excluded from the diff.
   - `git diff --check` passed.
   
   No live metrics datasource or browser end-to-end test was run.
   
   ### Checklist
   
   - [x] One coherent bug fix with regression tests ending in `Test`
   - [x] No new UI strings, files, or API changes
   - [x] Conventional Commit subject
   
   ### AI Assistance
   
   AI assistance was used to trace the sample conversions, draft the patch and 
regression tests, and execute the local checks listed above.
   


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