fix(coder/modules/claude-code): move set -euo pipefail after sourcing .bashrc

[matifali]

Moves the strict error handling flags after sourcing user bashrc files to prevent failures from unbound variables in user dotfiles. This is especially important when users load dotfiles from git.

Fixes agentapi server startup failures caused by BASHRCSOURCED and other unbound variables in /etc/bashrc or user-specific bashrc files.

[35C4n0r]

@matifali @DevelopmentCats
I recently(while building the opencode module) found out that this dosen't work in a shell script

if [ -f "$HOME/.bashrc" ]; then
  source "$HOME"/.bashrc // This line 
fi

This line in the bashrc prevents sourcing the bash script if we are in non-interactive mode

# If not running interactively, don't do anything
case $- in
    *i*) ;;
      *) return;;
esac

[DevelopmentCats]

@matifali @DevelopmentCats I recently(while building the opencode module) found out that this dosen't work in a shell script

if [ -f "$HOME/.bashrc" ]; then
  source "$HOME"/.bashrc // This line 
fi

This line in the bashrc prevents sourcing the bash script if we are in non-interactive mode

# If not running interactively, don't do anything
case $- in
    *i*) ;;
      *) return;;
esac

Wow you are totally right about this, Great Catch!

So basically right now those lines are essentially doing nothing haha.

[matifali]

I guess then it's good to merge?

[35C4n0r]

Yup, The other modules that are working right now, are due to the other exports that we have along sourcing of the bash file.

[matifali]

Yup, The other modules that are working right now, are due to the other exports that we have along sourcing of the bash file.

@35C4n0r, please feel free to go through all of them to fix this.