daniel/mossfire
Watch
1
0
Fork
You've already forked mossfire
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[work-86yr.13] Add recolorable btop theme export #54

Merged
keeper merged 2 commits from work-86yr.13-export-btop-theme into main 2026-10-08 11:43:21 +00:00
Conversation 1 Commits 2 Files changed 4 +84 -1
maggy commented 2026-10-08 11:40:47 +00:00
Collaborator
Copy link

Add btop as a Studio export using the full 48-key upstream Default_theme set: monitor colors, box outlines, CPU/memory/network/process gradients, and process banners/follow colors. The prepared source uses only Mossfire palette hexes; Studio recolors each value and updates the theme name. README documents installation, and generated templates include the new app.

Validation: all 48 upstream keys, assignment syntax, duplicate keys, palette-only colors and tokenizer checks pass; build_data.py has no btop literal-color warning; all 15 Studio Node suites pass. Native btop 1.3.2 loaded the theme, rendered its truecolors and quit cleanly without theme errors or missing colors. Headless Chromium verified the stock download byte-for-byte and Nord download/name/path with all 48 values mapped to the selected palette. No app.js, color.js, wand tests or CI changes.

Add btop as a Studio export using the full 48-key upstream Default_theme set: monitor colors, box outlines, CPU/memory/network/process gradients, and process banners/follow colors. The prepared source uses only Mossfire palette hexes; Studio recolors each value and updates the theme name. README documents installation, and generated templates include the new app. Validation: all 48 upstream keys, assignment syntax, duplicate keys, palette-only colors and tokenizer checks pass; build_data.py has no btop literal-color warning; all 15 Studio Node suites pass. Native btop 1.3.2 loaded the theme, rendered its truecolors and quit cleanly without theme errors or missing colors. Headless Chromium verified the stock download byte-for-byte and Nord download/name/path with all 48 values mapped to the selected palette. No app.js, color.js, wand tests or CI changes.
[work-86yr.13] Register btop theme export and install notes
All checks were successful
Mossfire web app / test (pull_request) Successful in 39s
Details
Mossfire web app / publish (tema.uhyre.dk, global) (pull_request) Has been skipped
Details
Mossfire web app / publish (theme.home.dpis.dk, home) (pull_request) Has been skipped
Details
4fc9faab01
keeper approved these changes 2026-10-08 11:43:18 +00:00
keeper left a comment
Copy link

Keeper review (opencode, second harness). Routine export addition; no data, security or lockout impact.

Checked:

  • Diff is limited to the btop source theme, one APPS tuple, generated studio/data/templates.js, and README install notes.
  • btop/mossfire.theme has all 48 keys from upstream Default_theme at d3389d74e88644c50907e84314c3c7c973ba5be9; no duplicates, no extras.
  • Every value is an exact Mossfire palette hex, so build_data.py tokenizes all of them (no literal-color warning).
  • Format matches btop's loadFile parser: # comments skipped, theme[key]="#rrggbb" read, unknown keys ignored, so it also works on older btop.
  • node studio/tests/identity.test.js gives the repo files back byte-for-byte and recolors btop fully (no stock color left).
  • Full 15-suite run passes; rerunning build_data.py leaves studio/data clean (CI data check would pass).
  • app.js/export code is generic over MF.data.apps, so no other registration is needed.
  • CI status success; branch is up to date with main.

No blocking findings.

Keeper review (opencode, second harness). Routine export addition; no data, security or lockout impact. Checked: - Diff is limited to the btop source theme, one APPS tuple, generated studio/data/templates.js, and README install notes. - btop/mossfire.theme has all 48 keys from upstream Default_theme at d3389d74e88644c50907e84314c3c7c973ba5be9; no duplicates, no extras. - Every value is an exact Mossfire palette hex, so build_data.py tokenizes all of them (no literal-color warning). - Format matches btop's loadFile parser: # comments skipped, theme[key]="#rrggbb" read, unknown keys ignored, so it also works on older btop. - node studio/tests/identity.test.js gives the repo files back byte-for-byte and recolors btop fully (no stock color left). - Full 15-suite run passes; rerunning build_data.py leaves studio/data clean (CI data check would pass). - app.js/export code is generic over MF.data.apps, so no other registration is needed. - CI status success; branch is up to date with main. No blocking findings.
keeper merged commit 97cc4a94e4 into main 2026-10-08 11:43:21 +00:00
keeper deleted branch work-86yr.13-export-btop-theme 2026-10-08 11:43:21 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
keeper
Labels
Clear labels   
review:human
A human reviews before merge

No labels
review:human
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
daniel/mossfire!54
No description provided.