[Bug]: syntax error in hooks.sh when using fish

[jasononeil]

Current Behavior (bug)
Since updating to 0.8.6 this morning, I'm now getting:

/Users/jason/workspace/local-ops/.devbox/gen/scripts/.hooks.sh:15: syntax error: unexpected end of file

I'm using Fish as my shell.

This is consistent across all my projects, and has been replicated by my co-worker @jay-aye-see-kay's mac also, though he had to delete his .devbox directory in the project to start seeing the error.

This prevents any commands from starting, for example:

> devbox run setup
/Users/jason/workspace/cultureamp/std-demographics-next/.devbox/gen/scripts/.hooks.sh: line 15: syntax error: unexpected end of file
Error: error running script "setup" in Devbox: exit status 2

Expected Behavior (fix)

There shouldn't be a syntax error when running devbox commands on a fish shell.

Notably, @jay-aye-see-kay pointed out the file is a .sh file with no shebang, but contains fish syntax.

If I run ./devbox/gen/scripts/.hooks.sh I get the error, but if I source it with . ./devbox/gen/scripts/.hooks.sh it works as expected.

Additional context

Please include the output of devbox version -v

Version:     0.8.6
Platform:    darwin_amd64
Commit:      af58be9b50c1a3d19bc21485bcbb77eb8ed8076a
Commit Time: 2024-01-18T18:36:03Z
Go Version:  go1.21.5
Launcher:    0.2.1

A copy of your devbox.json file:

{
  "packages": ["flake:nixpkgs#nodejs_18", "flake:nixpkgs#nodejs_18.pkgs.pnpm"],
  "env": {
    "STD_DEMOGRAPHICS_PORT": "4267"
  },
  "shell": {
    "scripts": {
      "setup": "pnpm install"
    }
  }
}

Fish version: 3.6.1

[jay-aye-see-kay]

Just saw version 0.8.7 was release with a fix. That's working for me thank you! 🙇

[jasononeil]

Yep this was fixed in #1741

Thanks!

[phanirithvij]

I still have it wth 0.8.7, I actually only got this issue after updating to 0.8.7 yesterday.

~/.local/share/devbox/global/default/.devbox/gen/scripts/.hooks.sh (line 2): Missing end to balance this if statement
if [ -n "$__DEVBOX_INIT_HOOK_754bd1fdfe79eaf22b29d5091b2f5218a24516e27e49d4a4e3793baf5286faff" ]; then
^^
from sourcing file ~/.local/share/devbox/global/default/.devbox/gen/scripts/.hooks.sh
        called on line 152 of file -
from sourcing file -
        called on line 7 of file ~/.config/fish/config.fish
from sourcing file ~/.config/fish/config.fish
        called during startup
.: Error while reading file '/home/rithvij/.local/share/devbox/global/default/.devbox/gen/scripts/.hooks.sh'
~ ⚹ (nix-shell-env) ⚬ devbox version                                                                                                                                                    08:54:41
0.8.7
~ ⚹ (nix-shell-env) ⚬ devbox version update                                                                                                                                             08:54:54
Version:     0.8.7
Platform:    linux_amd64
Commit:      525bb17ed665bb3066fbd3fb9bccec0aa961ecb7
Commit Time: 2024-01-23T19:15:31Z
Go Version:  go1.21.5
Launcher:    0.2.1
Success: already at Devbox version 0.8.7
~ ⚹ (nix-shell-env) ⚬cat ~/.config/fish/config.fish
if status is-interactive
    # Commands to run in interactive sessions can go here
end

set -U fish_greeting

devbox global shellenv --init-hook | source

I ran refresh-global, devbox global shellenv --recompute | source many times

[ssoriche]

I too was getting the Missing end to balance this if statement error with devbox 0.8.7 and fish. I was able to revert to 0.8.5 by adding set -gx DEVBOX_USE_VERSION 0.8.5 to my ~/.config/fish/config.fish.

[evanleck]

Yeah, just updated to 0.8.7 and am getting this error as well.

[jay-aye-see-kay]

This error disappeared for me on 0.8.7 but is back on 0.9-pre

❯ export DEVBOX_USE_VERSION=0.9.0-pre
❯ devbox version update
❯ cd my_repo
/Users/jack/my_repo/.devbox/gen/scripts/.hooks.fish.sh:10: syntax error: unexpected end of file
❯ cat /Users/jack/my_repo/.devbox/gen/scripts/.hooks.fish.sh
if set -q __DEVBOX_INIT_HOOK_d33728c8509c4b79ffd3d480e90fd6cade20b6364cb3667ac7814ebf3eb42e9b; then
    exit 0
end

export __DEVBOX_INIT_HOOK_d33728c8509c4b79ffd3d480e90fd6cade20b6364cb3667ac7814ebf3eb42e9b=true



set -e __DEVBOX_INIT_HOOK_d33728c8509c4b79ffd3d480e90fd6cade20b6364cb3667ac7814ebf3eb42e9b

I've tried removing .devbox/ and re-installing it with 0.9-pre to be sure but the issue's still there

[jay-aye-see-kay]

Still getting this error in the 0.9.0 release

[adamdicarlo0]

@jay-aye-see-kay Shot in the dark, but... does rm -rf .devbox and restarting devbox shell fix it?

[jay-aye-see-kay]

@jay-aye-see-kay Shot in the dark, but... does rm -rf .devbox and restarting devbox shell fix it?

Unfortunately not