Conversation
There was a problem hiding this comment.
Pull request overview
This PR introduces a new Resource Monitor applet for the Cinnamon desktop environment that displays real-time CPU, RAM, and Swap usage directly in the panel. The applet reads system metrics from /proc filesystem files for lightweight, dependency-free monitoring with customizable display options.
Changes:
- Added a new system resource monitoring applet with CPU, RAM, and Swap tracking
- Implemented customizable settings to toggle individual metrics on/off
- Provided comprehensive documentation including README and LICENSE files
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 14 comments.
Show a summary per file
| File | Description |
|---|---|
| resource-monitor@kprakesh/applet.js | Main applet implementation with /proc parsing logic and update loop |
| resource-monitor@kprakesh/metadata.json | Applet metadata including name, description, version, and Cinnamon compatibility |
| resource-monitor@kprakesh/settings-schema.json | Configuration schema for toggling CPU, RAM, and Swap display |
| resource-monitor@kprakesh/README.md | Comprehensive documentation covering features, installation, and technical details |
| resource-monitor@kprakesh/LICENSE | GPL-3.0 license text |
| resource-monitor@kprakesh/utilities-system-monitor.png | Icon file (should be renamed to icon.png) |
…solve memory leaks
Best-practices scannerThis is a regex-based check for API usage that can pose security, performance or This check is not perfect will not replace a normal review.Found 5 potential issue(s):
|
Best-practices scannerThis is a regex-based check for API usage that can pose security, performance or This check is not perfect will not replace a normal review.Found 2 potential issue(s): ℹ️ shell_string_spawnscreen-snipper@kprakesh/applet.js:63 Prefer argument vector spawn functions over shell command strings. screen-snipper@kprakesh/applet.js:68 Prefer argument vector spawn functions over shell command strings. Automated pattern check. |
…solve first-cycle CPU bug
Best-practices scannerThis is a regex-based check for API usage that can pose security, performance or This check is not perfect will not replace a normal review.Found 2 potential issue(s): ℹ️ shell_string_spawnscreen-snipper@kprakesh/applet.js:63 Prefer argument vector spawn functions over shell command strings. screen-snipper@kprakesh/applet.js:68 Prefer argument vector spawn functions over shell command strings. Automated pattern check. |
Best-practices scannerThis is a regex-based check for API usage that can pose security, performance or This check is not perfect will not replace a normal review.Found 2 potential issue(s): ℹ️ shell_string_spawnscreen-snipper@kprakesh/applet.js:63 Prefer argument vector spawn functions over shell command strings. screen-snipper@kprakesh/applet.js:68 Prefer argument vector spawn functions over shell command strings. Automated pattern check. |
This PR adds a new applet: Resource Monitor.
Features:
I've tested this on Cinnamon 6.6.6 (Linux Mint 22.3 x86_64) and ensured it follows the standard desktop UI guidelines.