feat: add theme hot-reload functionality #4879
Open
+190
−66
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Is this feature still being worked on? Hot reloading the theme would be a really nice functionality for theme switching! |
malhashemi
force-pushed
the
feature/theme-hot-reload
branch
from
653a64b to
6216894
Compare
Contributor
Author
|
@rekram1-node would you please review this PR and let me know if you have any changes. |
Comment on lines
+332
to
+335
| { | ||
| display: "/reload-theme", | ||
| description: "reload theme from config", | ||
| onSelect: () => command.trigger("theme.reload"), |
Collaborator
There was a problem hiding this comment.
Instead of slash command can we do command panel?
Like reload themes or something?
Also you are using a watcher is it possible to 'hot reload' them since you are filewatching?
Collaborator
|
/review |
Collaborator
|
If im not looking at something that u need, feel free to dm on discord or x I get like 200 github notifications a day so Im doing my best to go through them but stuff will always sneak through |
Contributor
|
lgtm |
malhashemi
force-pushed
the
feature/theme-hot-reload
branch
from
ddd6482 to
2c96b0c
Compare
- Add /reload-theme command for manual theme reloading - Add file watchers for config and theme file changes - Add SIGWINCH handler to re-query terminal colors on resize - Add SIGUSR1/SIGUSR2 signal handlers for external automation - Make system theme background transparent for terminal opacity - Move getTerminalBackgroundColor to theme context Fixes anomalyco#815
malhashemi
force-pushed
the
feature/theme-hot-reload
branch
from
2c96b0c to
8723897
Compare
This was referenced Jan 5, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Adds the ability to reload themes without restarting opencode. Provides multiple reload triggers to support different workflows:
/reload-themecommandAlso makes "system" theme background transparent to support terminal opacity.
Changes
Architecture
flowchart LR subgraph Triggers A[Command] B[File change] C[Resize] D[Signal] end A & B & C & D --> E[reloadTheme] E --> F[Re-query colors] F --> G[Update store] G --> H[UI re-renders]Key Decisions
Why move
getTerminalBackgroundColorto theme.tsx?Needed by both initial detection (app.tsx) and reload (theme.tsx). Moving it eliminates duplication.
Why
fs.watch()instead of@parcel/watcher?Simpler API, no extra dependencies, sufficient for watching 2 files.
Why transparent background for "system" theme?
Allows terminal opacity/blur to show through while keeping panel/menu backgrounds solid.
Why SIGWINCH handler?
Event-driven color re-detection without polling. Only active for "system" theme.
Testing
Checklist
Fixes #815