Node.js hook errors with direnv about devbox HOT 7 OPEN

@savil fish 3.7.1 and direnv 2.34.0

from devbox.

FWIW changing the contents of .raw-hooks to use test -v seems to fix the issue, though I'm not 100% sure it's functionally equivalent. Worth noting, test -v was introduced in bash 4.2, so this won't work on versions prior to that.

test -v DEVBOX_COREPACK_ENABLED || corepack enable --install-directory "/Users/evan/Code/devbox-node-repro/.devbox/virtenv/nodejs/corepack-bin/"
test -v DEVBOX_COREPACK_ENABLED || export PATH="/Users/evan/Code/devbox-node-repro/.devbox/virtenv/nodejs/corepack-bin/:$PATH"

from devbox.

Probably worth noting that I’m using fish shell.

from devbox.

@evanleck which version of fish shell are you using?

from devbox.

Hey @evanleck, I've been trying to reproduce this but I can't get the error to replicate with Fish + Direnv. Can you try the following:

1. Update to the latest version of Devbox using devbox version update
2. Delete the contents of your .devbox directory, then regenerate them using devbox shell.

If those don't fix your issue, can you share the contents of the file at devbox-node-repro/.devbox/gen/scripts/.raw-hooks.sh

from devbox.

@Lagoja thanks for looking into the issue.

1. Done, I'm on 0.10.4 now.
2. Done.

Unfortunately, that didn't seem to fix the issue. The contents of .raw-hooks.sh is:

test -z $DEVBOX_COREPACK_ENABLED || corepack enable --install-directory "/Users/evan/Code/devbox-node-repro/.devbox/virtenv/nodejs/corepack-bin/"
test -z $DEVBOX_COREPACK_ENABLED || export PATH="/Users/evan/Code/devbox-node-repro/.devbox/virtenv/nodejs/corepack-bin/:$PATH"

I don't see DEVBOX_COREPACK_ENABLED anywhere in the output of env if that's of any help.

from devbox.

I have exact same issue, with devbox 0.10.9 and zsh 5.9 (NixOS from the current release-24.05 tag). The test -v trick works (thanks a lot for the advice!), and so does wrapping the tests with set +u/set -u.

from devbox.