fix: format number value from string first

[fengmk2]

closes X-Profiler/all-in-one#7 #50

[Copilot]

Pull request overview

This PR addresses an issue where numeric values stored as strings in the database were causing errors when methods like .toFixed() were called on them. The fix adds explicit Number() conversions before using these string values in arithmetic operations or calling number methods on them.

Changes:

• Added format parameter to getAverageMetric method in metric.js to convert string values to numbers before processing
• Added format: value => Number(value) functions to various metric key configurations in service files
• Added explicit Number() conversions in controller files before calling .toFixed() on numeric values

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
app/service/metric.js	Added format parameter to getAverageMetric method and applied it to metric values before processing
app/service/system.js	Added format functions to osCpuTrend, loadTrend, and httpResponseTrend metric keys
app/service/process.js	Added format functions to all cpuTrend metric keys (cpu_now, cpu_15, cpu_30, cpu_60)
app/controller/system.js	Added Number() conversions for used_cpu, used_memory_percent, load values, and http_rt before using toFixed()
app/controller/process.js	Added Number() conversion for cpu_60 before calling toFixed()

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.