Skip to content

feat: Allow overriding default bat themes using environment variables#7140

Merged
DOsinga merged 1 commit intoblock:mainfrom
ekoeppen:main
Feb 11, 2026
Merged

feat: Allow overriding default bat themes using environment variables#7140
DOsinga merged 1 commit intoblock:mainfrom
ekoeppen:main

Conversation

@ekoeppen
Copy link
Contributor

@ekoeppen ekoeppen commented Feb 11, 2026

Summary

Replaces the hardcoded theme names for the light and dark themes with the values from GOOSE_CLI_DARK_THEME and GOOSE_CLI_LIGHT_THEME, if unset, fall back to the current hardcoded values.

Type of Change

  • Feature
  • Bug fix
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Other (specify below)

AI Assistance

  • This PR was created or reviewed with AI assistance

Testing

Manual testing

Related Issues

Relates to #7037
Discussion: LINK (if any)

Screenshots/Demos (for UX changes)

Before: (GitHub):

image

After: (Solarized (light)):

image

@ekoeppen ekoeppen requested a review from a team as a code owner February 11, 2026 05:35
DOsinga
DOsinga approved these changes Feb 11, 2026
View reviewed changes
Copy link
Collaborator

@DOsinga DOsinga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's get it in, if you can use config = Config::global();

crates/goose-cli/src/session/output.rs Outdated
Theme::Dark => "zenburn",
Theme::Ansi => "base16",
Theme::Light => {
std::env::var("GOOSE_CLI_LIGHT_THEME").unwrap_or_else(|_| "GitHub".to_string())
Copy link
Collaborator

@DOsinga DOsinga Feb 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we use the settings module instead? it will read environment variables, but also what's in config.yaml. no need to add it to the configure flow, but for people that just want to fix this that way, mioght be nice

@dianed-square
Copy link
Contributor

dianed-square commented Feb 11, 2026

@ekoeppen Can you please remove/hide the doc changes for now so that the public docs don't get ahead of the release? We'll add them when the feature is available in a released version. Thanks!

blackgirlbytes
blackgirlbytes requested changes Feb 11, 2026
View reviewed changes
Copy link
Contributor

@blackgirlbytes blackgirlbytes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for being proactive and adding the docs. could you actually remove them for now? my team will add them back in once the feature is released. appreciate you!

@ekoeppen
Copy link
Contributor Author

ekoeppen commented Feb 11, 2026

thanks for being proactive and adding the docs. could you actually remove them for now? my team will add them back in once the feature is released. appreciate you!

Sure, no problem :)

@ekoeppen
feat: Allow overriding default bat themes using environment variables
1ca014d 
Signed-off-by: Eckhart Köppen <eck@40hz.org>
@DOsinga DOsinga merged commit 1aacd56 into block:main Feb 11, 2026
19 checks passed
tlongwell-block added a commit that referenced this pull request Feb 11, 2026
@tlongwell-block
Merge remote-tracking branch 'origin/main' into tui_gem
3776f63 
* origin/main: (107 commits)
  feat: Allow overriding default bat themes using environment variables (#7140)
  Make the system prompt smaller (#6991)
  Pre release script (#7145)
  Spelling (#7137)
  feat(mcp): upgrade rmcp to 0.15.0 and advertise MCP Apps UI extension capability (#6927)
  fix: ensure assistant messages with tool_calls include content field (#7076)
  fix(canonical): handle gcp_vertex_ai model mapping correctly (#6836)
  Group dependencies in root Cargo.toml (#6948)
  refactor: updated elevenLabs API module and `remove button` UX (#6781)
  fix: we were missing content from langfuse traces (#7135)
  docs: update username in authors.yml (#7132)
  fix extension selector syncing issues (#7133)
  fix(acp): per-session Agent for model isolation and load_session restore (#7115)
  fix(claude-code): defensive coding improvements for model switching (#7131)
  feat(claude-code): dynamic model listing and mid-session model switching (#7120)
  Inline worklet source (#7128)
  [docs] One shot prompting is dead - Blog Post (#7113)
  fix: correct spelling of Debbie O'Brien's name in authors.yml (#7127)
  docs: GCP Vertex AI org policy filtering & update OnboardingProviderSetup component (#7125)
  feat: replace subagent and skills with unified summon extension (#6964)
  ...

# Conflicts:
#	Cargo.lock
#	Cargo.toml
zanesq added a commit to Abhijay007/goose that referenced this pull request Feb 11, 2026
@zanesq
Merge remote-tracking branch 'upstream/main' into feat/recipeProvider
8733a30 
* upstream/main: (109 commits)
  [docs] Skills Marketplace UI Improvements (block#7158)
  More no-window flags (block#7122)
  feat: Allow overriding default bat themes using environment variables (block#7140)
  Make the system prompt smaller (block#6991)
  Pre release script (block#7145)
  Spelling (block#7137)
  feat(mcp): upgrade rmcp to 0.15.0 and advertise MCP Apps UI extension capability (block#6927)
  fix: ensure assistant messages with tool_calls include content field (block#7076)
  fix(canonical): handle gcp_vertex_ai model mapping correctly (block#6836)
  Group dependencies in root Cargo.toml (block#6948)
  refactor: updated elevenLabs API module and `remove button` UX (block#6781)
  fix: we were missing content from langfuse traces (block#7135)
  docs: update username in authors.yml (block#7132)
  fix extension selector syncing issues (block#7133)
  fix(acp): per-session Agent for model isolation and load_session restore (block#7115)
  fix(claude-code): defensive coding improvements for model switching (block#7131)
  feat(claude-code): dynamic model listing and mid-session model switching (block#7120)
  Inline worklet source (block#7128)
  [docs] One shot prompting is dead - Blog Post (block#7113)
  fix: correct spelling of Debbie O'Brien's name in authors.yml (block#7127)
  ...
jh-block added a commit that referenced this pull request Feb 12, 2026
@jh-block
Merge remote-tracking branch 'origin/main' into local-models-candle
7f7a449 
* origin/main: (33 commits)
  fix: replace panic with proper error handling in get_tokenizer (#7175)
  Lifei/smoke test for developer (#7174)
  fix text editor view broken (#7167)
  docs: White label guide (#6857)
  Add PATH detection back to developer extension (#7161)
  docs: pin version in ci/cd (#7168)
  Desktop: - No Custom Headers field for custom OpenAI-compatible providers  (#6681)
  feat: edit model and extensions of a recipe from GUI (#6804)
  feat: MCP support for agentic CLI providers (#6972)
  docs: keyring fallback to secrets.yaml (#7165)
  feat: load provider/model specified inside the recipe config (#6884)
  fix ask-ai bot hitting tool call limits (#7162)
  fix flatpak icon (#7154)
  [docs] Skills Marketplace UI Improvements (#7158)
  More no-window flags (#7122)
  feat: Allow overriding default bat themes using environment variables (#7140)
  Make the system prompt smaller (#6991)
  Pre release script (#7145)
  Spelling (#7137)
  feat(mcp): upgrade rmcp to 0.15.0 and advertise MCP Apps UI extension capability (#6927)
  ...
@ekoeppen ekoeppen mentioned this pull request Feb 12, 2026
Closed
1 task
@github-actions github-actions bot mentioned this pull request Feb 12, 2026
Open
@BrewTestBot BrewTestBot mentioned this pull request Feb 12, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@blackgirlbytes blackgirlbytes blackgirlbytes requested changes

@DOsinga DOsinga DOsinga approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ekoeppen @dianed-square @DOsinga @blackgirlbytes