packages/bruno-app/src/components/Devtools/Performance/StyledWrapper.js (1)

119-348: New performance header and processes-table styling look consistent

The new header, selector, and table styles align with the markup in Performance/index.js and fit the existing theming approach. Only minor nit: the hard-coded badge/CPU colors (#ef4444, #f59e0b, etc.) could be moved into the theme later for easier theming, but that’s optional.

packages/bruno-app/src/components/Devtools/Performance/index.js (1)

65-83: Remove or wire up unused trend / IconChartLine in SystemResourceCard

SystemResourceCard accepts a trend prop and renders an <IconChartLine />, but:

• IconChartLine is not imported.
• None of the current call sites pass trend, so this code is effectively dead but will break as soon as someone starts using that prop.

I’d either import IconChartLine and start using trend, or simplify the card by dropping the trend prop and the extra markup until it’s needed.

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

packages/bruno-app/src/components/Devtools/Performance/index.js (1)

57-73: Consider hoisting pure utility functions outside the component.

formatBytes and formatUptime have no dependencies on component state or props. Moving them outside Performance avoids recreating them on each render.

+const formatBytes = (bytes) => {
+  if (bytes === 0) return '0 Bytes';
+  const k = 1024;
+  const sizes = ['Bytes', 'KB', 'MB', 'GB'];
+  const i = Math.floor(Math.log(bytes) / Math.log(k));
+  return parseFloat((bytes / Math.pow(k, i)).toFixed(2)) + ' ' + sizes[i];
+};
+
+const formatUptime = (seconds) => {
+  const hours = Math.floor(seconds / 3600);
+  const minutes = Math.floor((seconds % 3600) / 60);
+  const secs = Math.floor(seconds % 60);
+
+  if (hours > 0) return `${hours}h ${minutes}m ${secs}s`;
+  if (minutes > 0) return `${minutes}m ${secs}s`;
+  return `${secs}s`;
+};
+
 const Performance = () => {
   // ...remove formatBytes and formatUptime from here

packages/bruno-electron/src/app/system-monitor.js (1)

71-87: Use const instead of var for processes

processes is never reassigned, so const communicates intent better and avoids accidental mutation; everything else in this block looks consistent and well‑structured.

-      const uptime = (currentTime - this.startTime) / 1000;
-      var processes = metrics.map((metric) => ({
+      const uptime = (currentTime - this.startTime) / 1000;
+      const processes = metrics.map((metric) => ({
         pid: metric.pid,
         title: metric.title,
         memory: metric.memory.workingSetSize * 1024,
         cpu: metric.cpu.percentCPUUsage,
         type: metric.type || 'unknown',
         creationTime: metric.creationTime
       }));

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro