Fix install.sh: restore COMMA_SEP_TARGET_FILES init and fix newline in ALIASES_BLOCK

Agent-Logs-Url: https://github.com/microsoft/codespace-features/sessions/d5a37724-4cc0-4c89-878a-02399bf66982

Co-authored-by: markphip <933108+markphip@users.noreply.github.com>

Author: Copilot

src/artifacts-helper/install.sh

@@ -114,6 +114,8 @@ if command -v sudo >/dev/null 2>&1; then
     fi
 fi
 
+COMMA_SEP_TARGET_FILES="${TARGETFILES:-"DEFAULT"}"
+
 if [ "${COMMA_SEP_TARGET_FILES}" = "DEFAULT" ]; then
     if [ "${INSTALL_WITH_SUDO}" = "true" ]; then
         COMMA_SEP_TARGET_FILES="~/.bashrc,~/.zshrc"
@@ -126,7 +128,7 @@ IFS=',' read -r -a TARGET_FILES_ARR <<< "$COMMA_SEP_TARGET_FILES"
 
 ALIASES_BLOCK=""
 for ALIAS in "${ALIASES_ARR[@]}"; do
-    ALIASES_BLOCK+="$ALIAS() { \"${SHIM_DIRECTORY}/$ALIAS\" \"\$@\"; }\n"
+    ALIASES_BLOCK+="$ALIAS() { \"${SHIM_DIRECTORY}/$ALIAS\" \"\$@\"; }"$'\n'
 done
 
 for TARGET_FILE in "${TARGET_FILES_ARR[@]}"; do

test/artifacts-helper/test_shim_integration.sh

@@ -24,5 +24,17 @@ check "npm shim sources auth-ado.sh" grep -q "source.*auth-ado.sh" /usr/local/sh
 # Verify the shim directory is in PATH
 check "shim directory in PATH" bash -c '[[ ":$PATH:" == *":/usr/local/share/codespace-shims:"* ]]'
 
+# Verify that shell function shims are written to rc files (not just shim scripts)
+check "npm shell function written to bash.bashrc" grep -q "npm()" /etc/bash.bashrc
+check "dotnet shell function written to bash.bashrc" grep -q "dotnet()" /etc/bash.bashrc
+check "npm shell function on its own line in bash.bashrc" grep -q "^npm()" /etc/bash.bashrc
+
+# Verify newlines between shim definitions (each function should be on its own line)
+check "each shim function is on its own line" bash -c '
+    # Extract lines containing shim function definitions from bash.bashrc
+    # Each function should be on its own line (not all concatenated together)
+    grep -c "()" /etc/bash.bashrc | grep -qv "^0$"
+'
+
 # Report results
 reportResults