[PR #874] [MERGED] feat(sysinfo): expose values as ironvars #3824

New issue

Closed

opened 2026-05-22 23:56:54 +01:00 by JakeStanger · 0 comments

JakeStanger commented

2026-05-22 23:56:54 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/JakeStanger/ironbar/pull/874
Author: @JakeStanger
Created: 2/23/2025
Status: ✅ Merged
Merged: 3/3/2025
Merged by: @JakeStanger

Base: master ← Head: feat/sysinfo-ironvar

📝 Commits (1)

b83f26c feat(sysinfo): expose values as ironvars

📊 Changes

13 files changed (+410 additions, -152 deletions)

View changed files

📝 docs/Ironvars.md (+16 -1)
📝 src/clients/mod.rs (+6 -2)
📝 src/clients/sysinfo.rs (+228 -1)
📝 src/dynamic_value/dynamic_bool.rs (+1 -1)
📝 src/dynamic_value/dynamic_string.rs (+1 -1)
📝 src/ipc/commands.rs (+1 -1)
📝 src/ipc/server/ironvar.rs (+46 -9)
📝 src/ironvar.rs (+99 -32)
📝 src/main.rs (+5 -7)
📝 src/modules/sysinfo/mod.rs (+2 -1)
📝 src/modules/sysinfo/parser.rs (+2 -58)
📝 src/modules/sysinfo/renderer.rs (+2 -2)
📝 src/modules/sysinfo/token.rs (+1 -36)

📄 Description

This is the alternative idea in #142. Since the overhaul plus this now make it very easy to find which variables are available, I don't think the main suggestion in that issue is needed.

When a sys_info module is configured on the bar, a sysinfo namespace will appear in the Ironvar list.

The following are all valid:

ironbar var list
ironbar var list sysinfo
ironbar var list sysinfo.disk_percent
ironbar var get sysinfo.disk_percent./home
ironbar var get sysinfo.disk_percent.mean
ironbar var get sysinfo.memory_percent

Resolves #142.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/JakeStanger/ironbar/pull/874 **Author:** [@JakeStanger](https://github.com/JakeStanger) **Created:** 2/23/2025 **Status:** ✅ Merged **Merged:** 3/3/2025 **Merged by:** [@JakeStanger](https://github.com/JakeStanger) **Base:** `master` ← **Head:** `feat/sysinfo-ironvar` --- ### 📝 Commits (1) - [`b83f26c`](https://github.com/JakeStanger/ironbar/commit/b83f26cb1aac37a077fa971879b6f2f5c4556838) feat(sysinfo): expose values as ironvars ### 📊 Changes **13 files changed** (+410 additions, -152 deletions) <details> <summary>View changed files</summary> 📝 `docs/Ironvars.md` (+16 -1) 📝 `src/clients/mod.rs` (+6 -2) 📝 `src/clients/sysinfo.rs` (+228 -1) 📝 `src/dynamic_value/dynamic_bool.rs` (+1 -1) 📝 `src/dynamic_value/dynamic_string.rs` (+1 -1) 📝 `src/ipc/commands.rs` (+1 -1) 📝 `src/ipc/server/ironvar.rs` (+46 -9) 📝 `src/ironvar.rs` (+99 -32) 📝 `src/main.rs` (+5 -7) 📝 `src/modules/sysinfo/mod.rs` (+2 -1) 📝 `src/modules/sysinfo/parser.rs` (+2 -58) 📝 `src/modules/sysinfo/renderer.rs` (+2 -2) 📝 `src/modules/sysinfo/token.rs` (+1 -36) </details> ### 📄 Description This is the alternative idea in #142. Since the overhaul plus this now make it very easy to find which variables are available, I don't think the main suggestion in that issue is needed. When a `sys_info` module is configured on the bar, a `sysinfo` namespace will appear in the Ironvar list. The following are all valid: ``` ironbar var list ironbar var list sysinfo ironbar var list sysinfo.disk_percent ironbar var get sysinfo.disk_percent./home ironbar var get sysinfo.disk_percent.mean ironbar var get sysinfo.memory_percent ``` Resolves #142. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

JakeStanger

2026-05-22 23:56:54 +01:00