Skip to content

Conversation

@zktaiga
Copy link
Contributor

@zktaiga zktaiga commented Nov 9, 2025

Description

Initialize direnv in bling.sh for bash and zsh. direnv is already shipped in bluefin-cli.Brewfile but was non-functional without shell initialization.

direnv by default has built-in safeguards, so you'll still need to run direnv allow wherever necessary.

A small note: direnv is initialized before bash-preexec to avoid PROMPT_COMMAND conflicts. There's an edge case in bash-preexec where if direnv modifies PROMPT_COMMAND after bash-preexec sources, it breaks atuin's history capture. By initializing direnv first, we avoid this issue entirely. See:

Type of Change

  • Bug fix/new feature (direnv was installed but not configured OOTB)

Testing Done

  • Tested on running system: Yes (manually sourced modified bling.sh, verified direnv and atuin both work)

@zktaiga zktaiga requested a review from Zeglius as a code owner November 9, 2025 09:41
@zktaiga zktaiga force-pushed the init-direnv branch 3 times, most recently from a2d87f2 to a60b13d Compare November 9, 2025 09:49
@castrojo castrojo enabled auto-merge November 14, 2025 15:11
@KyleGospo KyleGospo disabled auto-merge November 21, 2025 01:29
@KyleGospo KyleGospo merged commit d76246a into ublue-os:main Nov 21, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants