Still working on getting the remote user to work.

Author: metcalfc

src/atuin/devcontainer-feature.json

@@ -1,7 +1,7 @@
 {
   "name": "Atuin CLI",
   "id": "atuin",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "description": "A feature to install Atuin",
   "options": {
     "completion": {

src/atuin/install.sh

@@ -6,7 +6,10 @@ echo "Activating feature 'Atuin'"
 COMPLETION=${COMPLETION:-true}
 echo "Atuin completion files installed: $COMPLETION"
 
-DIRECTORY=${HOME:-$_REMOTE_USER_HOME}
+# feature scripts run as root but we want to install atuin for the container
+# or remote user
+# https://containers.dev/implementors/features/#user-env-var
+DIRECTORY="$_REMOTE_USER_HOME"
 
 export DEBIAN_FRONTEND=noninteractive
 apt_get_update()

src/atuin/scripts/postAttachCommand.sh

@@ -3,7 +3,7 @@ set -e
 
 # check a file for the given string if it exists do nothing. If it doesn't exist, add the string to the file.
 checkFileForString() {
-    if ! grep -q "$1" "$2"; then
+    if ! grep -sq "$1" "$2"; then
         echo "$1" >> "$2"
     fi
 }