following shell logic, the whole:

// lines 187-191

if [ "$code" -eq "0" ] ; then
 exit 0
else
  exit 1
fi

is pointless, just replace all with exit $code

also could replace all of those value.xxx with one value = { but that's personal preference...

Oh yup, my bad, will fix those

you know what, now that am looking at it, since you are using [ and not bash's [[ you can even remove all of lines 178 - 192 for:

#!${pkgs.runtimeShell} -e
for var in "$@"; do
    [ -L "$var" ] || { echo "$var is not controlled by Hjem due to a file conflict"; exit 1; }
done
'';

Made some extra changes but forgot to commit them, will update with all this soon

@eclairevoyant can you please remove your requested change? I have implemented it in other commits.

None of my feedback appears to have been addressed yet