update the direnv hook

[amtoine]

i wanted to install and run direnv with Nushell, came accross this hook and thought i could update it a bit 😋

changelog

• make config.nu possible to source => it will add the hook to the list of hooks, without overwriting
• will run the hook on environment change thank to $env.config.hooks.env_change.PWD instead of $env.config.hooks.pre_prompt => i think this is the most controversial change, we can discuss that of course
• checks if direnv is in the PATH before running the rest of the hook
• uses a oneliner with default to load the environment
• uses directly a closure instead of string
• the comments were out of date, so i removed them

i tested this before and after and i think this works just as before 😌

[fdncred]

Just guessing here but it seems that one reason to run a hook in pre-prompt is to allow something from the direnv to update the prompt. Maybe you can do that too in the env_change hook. Not sure.

[amtoine]

Just guessing here but it seems that one reason to run a hook in pre-prompt is to allow something from the direnv to update the prompt. Maybe you can do that too in the env_change hook. Not sure.

not sure 🤔

one thing we could do is define a constant in that script that one could source and add to either the env_change or the pre_prompt?

[amtoine]

@fdncred
let me know if 6c01cf9 makes this any better 😌

[fdncred]

Seems like it explain two ways to do it, so it looks better to me. I don't use direnv so I can't test it.

[amtoine]

@fdncred
i just ran, from nu -n,

$env.config.hooks.env_change.PWD = []
$env.config.hooks.env_change.PWD = (
    $env.config.hooks.env_change.PWD | append (source ~/documents/repos/github.com/amtoine/nu_scripts/hooks/direnv/config.nu)
)

went into a directory with a .envrc and direnv kicked in 😉