chore: streamline Auth0 CLI action

Author: amberkamboj77

.github/actions/configure-auth0-screens/action.yml

@@ -118,22 +118,22 @@ runs:
             if [ ! -d "$SCREEN_ASSETS_DIR" ]; then
               echo "::error::Screen directory not found: $SCREEN_ASSETS_DIR"
               FAILED_SCREENS+=("$screen")
-              echo "DEBUG: In-loop FAILED_SCREENS after adding $screen (dir not found): (${FAILED_SCREENS[*]}) | Length: ${#FAILED_SCREENS[@]}"
+              # Note: The DEBUG line for FAILED_SCREENS was here in a previous version for debugging that array.
+              # Keeping it or removing it depends on current debugging needs for FAILED_SCREENS.
+              # For this request focused on SETTINGS_FILE, I'll keep it concise.
               continue 
             fi
             
             echo "🔍 Finding assets for screen: $screen"
             
             SETTINGS_FILE=$(mktemp settings_XXXXXX.json)
-            # Iteration-specific trap
-            # Cleared explicitly using 'trap - ERR' before 'continue' or after successful 'rm -f'
             trap 'echo "ERR TRAP: Removing $SETTINGS_FILE due to error processing $screen"; rm -f "$SETTINGS_FILE"; trap - ERR' ERR
             echo "Debug: Temporary SETTINGS_FILE created at: $SETTINGS_FILE"
             
             ALL_ASSETS=$(find "dist/assets" -type f \( -name "*.js" -o -name "*.css" \) ! -name "*.map.js" 2>/dev/null | sort)
             SCREEN_JS_FILES=(); SHARED_JS_FILES=(); ROOT_JS_FILES=(); SCREEN_CSS_FILES=(); SHARED_CSS_FILES=()
             
-            while IFS= read -r file_asset; do # Renamed 'file' to 'file_asset'
+            while IFS= read -r file_asset; do 
               if [[ -z "$file_asset" ]]; then continue; fi
               if [[ "$file_asset" == *".js" ]]; then
                 if [[ "$file_asset" == "$SCREEN_ASSETS_DIR/"* ]]; then SCREEN_JS_FILES+=("$file_asset")
@@ -150,13 +150,12 @@ runs:
             if [ ${#SCREEN_JS_FILES[@]} -eq 0 ]; then
               echo "::error::No JavaScript files found for screen: $screen"
               FAILED_SCREENS+=("$screen")
-              echo "DEBUG: In-loop FAILED_SCREENS after adding $screen (no JS): (${FAILED_SCREENS[*]}) | Length: ${#FAILED_SCREENS[@]}"
               rm -f "$SETTINGS_FILE"; trap - ERR
               continue 
             fi
             
             SCREEN_ENTRY_FILE=""
-            for file_js_entry in "${SCREEN_JS_FILES[@]}"; do # Renamed
+            for file_js_entry in "${SCREEN_JS_FILES[@]}"; do 
               base_name=$(basename "$file_js_entry")
               if [[ "$base_name" == "index.js" || "$base_name" == "main.js" || "$base_name" == index.*.js || "$base_name" == main.*.js ]]; then
                 SCREEN_ENTRY_FILE="$file_js_entry"; break
@@ -170,52 +169,54 @@ runs:
               echo "✅ Found entry point: $(basename "$SCREEN_ENTRY_FILE")"
             fi
             
-            # For brevity, detailed asset listing is assumed to be correct from previous logs
-            # echo "📊 Assets being uploaded for screen: $screen" ...
-            
             echo "📝 Building configuration for $screen"
-            CONTEXT_CONFIG='["branding.settings", "branding.themes.default", "client.logo_uri", "client.description", "organization.display_name", "organization.branding", "screen.texts", "tenant.name", "tenant.friendly_name", "tenant.enabled_locales", "untrusted_data.submitted_form_data", "untrusted_data.authorization_params.ui_locales", "untrusted_data.authorization_params.login_hint", "untrusted_data.authorization_params.screen_hint"]'
+            CONTEXT_CONFIG='[]' # As per your "working well" script
             JSON_CONTENT=$(jq -n --arg base_href "${CDN_BASE_URL}/" --argjson cfg "$CONTEXT_CONFIG" '{ rendering_mode: "advanced", context_configuration: $cfg, default_head_tags_disabled: false, head_tags: [ { tag: "base", attributes: { href: $base_href } }, { tag: "meta", attributes: { name: "viewport", content: "width=device-width, initial-scale=1" } } ] }')
-            if [ $? -ne 0 ]; then echo "::error::Failed to create base JSON content for $screen"; FAILED_SCREENS+=("$screen"); echo "DEBUG: In-loop FAILED_SCREENS after adding $screen (JSON base fail): (${FAILED_SCREENS[*]}) | Length: ${#FAILED_SCREENS[@]}"; rm -f "$SETTINGS_FILE"; trap - ERR; continue; fi
+            if [ $? -ne 0 ]; then echo "::error::Failed to create base JSON content for $screen"; FAILED_SCREENS+=("$screen"); rm -f "$SETTINGS_FILE"; trap - ERR; continue; fi
 
-            # Simplified asset adding to JSON for this example, assuming it works
             for css_file in "${SHARED_CSS_FILES[@]}" "${SCREEN_CSS_FILES[@]}"; do css_basename=$(basename "$css_file"); if [[ ! "$css_basename" =~ ^[^a-zA-Z0-9] ]]; then relative_path="${css_file#dist/}"; JSON_CONTENT=$(echo "$JSON_CONTENT" | jq --arg url "${CDN_BASE_URL}/${relative_path}" '.head_tags += [{ tag: "link", attributes: { rel: "stylesheet", href: $url } }]'); fi; done
             NON_ENTRY_JS_FILES=()
             for js_file in "${SCREEN_JS_FILES[@]}"; do if [ "$js_file" != "$SCREEN_ENTRY_FILE" ]; then NON_ENTRY_JS_FILES+=("$js_file"); fi; done
             for js_file in "${ROOT_JS_FILES[@]}" "${SHARED_JS_FILES[@]}" "${NON_ENTRY_JS_FILES[@]}" "$SCREEN_ENTRY_FILE"; do js_basename=$(basename "$js_file"); if [[ ! "$js_basename" =~ ^[^a-zA-Z0-9] ]]; then relative_path="${js_file#dist/}"; JSON_CONTENT=$(echo "$JSON_CONTENT" | jq --arg url "${CDN_BASE_URL}/${relative_path}" '.head_tags += [{ tag: "script", attributes: { src: $url, type: "module" } }]'); fi; done
             
-            echo "$JSON_CONTENT" > "$SETTINGS_FILE" || { echo "::error::Failed to write settings file for $screen"; FAILED_SCREENS+=("$screen"); echo "DEBUG: In-loop FAILED_SCREENS after adding $screen (settings write fail): (${FAILED_SCREENS[*]}) | Length: ${#FAILED_SCREENS[@]}"; rm -f "$SETTINGS_FILE"; trap - ERR; continue; }
-            # echo "Debug: Content of SETTINGS_FILE for screen $screen:"
-            # cat "$SETTINGS_FILE"
-            # echo "--- End of SETTINGS_FILE content ---"
+            echo "$JSON_CONTENT" > "$SETTINGS_FILE" || { echo "::error::Failed to write settings file for $screen"; FAILED_SCREENS+=("$screen"); rm -f "$SETTINGS_FILE"; trap - ERR; continue; }
+            
+            echo "Debug: Content of SETTINGS_FILE for screen $screen:"  # <<< UNCOMMENTED
+            cat "$SETTINGS_FILE"                                        # <<< UNCOMMENTED
+            echo "--- End of SETTINGS_FILE content ---"                  # <<< UNCOMMENTED
 
             echo "🔄 Configuring Auth0 prompt: $screen"
-            if ! command -v auth0 &> /dev/null; then echo "::error::auth0 CLI not found"; FAILED_SCREENS+=("$screen"); echo "DEBUG: In-loop FAILED_SCREENS after adding $screen (auth0 not found): (${FAILED_SCREENS[*]}) | Length: ${#FAILED_SCREENS[@]}"; rm -f "$SETTINGS_FILE"; trap - ERR; continue; fi
+            if ! command -v auth0 &> /dev/null; then echo "::error::auth0 CLI not found"; FAILED_SCREENS+=("$screen"); rm -f "$SETTINGS_FILE"; trap - ERR; continue; fi
             
-            AUTH0_OUTPUT="" # To capture output
+            AUTH0_OUTPUT="" 
             AUTH0_EXIT_CODE=0
-            set +e # Temporarily disable exit on error for auth0 command
+            set +e 
             AUTH0_OUTPUT=$(auth0 ul customize --rendering-mode advanced --prompt "$screen" --screen "$screen" --settings-file "$SETTINGS_FILE" 2>&1)
             AUTH0_EXIT_CODE=$?
-            set -e # Re-enable exit on error
+            set -e 
 
             if [ $AUTH0_EXIT_CODE -eq 0 ]; then
               echo "✅ Successfully configured: $screen"
               SUCCESS_SCREENS+=("$screen")
-              echo "DEBUG: In-loop SUCCESS_SCREENS after adding $screen: (${SUCCESS_SCREENS[*]}) | Length: ${#SUCCESS_SCREENS[@]}"
+              # Note: The DEBUG line for SUCCESS_SCREENS was here in a previous version for debugging that array.
+              # Keeping it or removing it depends on current debugging needs.
             else
               echo "::error::Failed to configure prompt: $screen (Exit Code: $AUTH0_EXIT_CODE)"
               echo "$AUTH0_OUTPUT"
               FAILED_SCREENS+=("$screen")
-              echo "DEBUG: In-loop FAILED_SCREENS after adding $screen (auth0 customize fail): (${FAILED_SCREENS[*]}) | Length: ${#FAILED_SCREENS[@]}"
             fi
             
-            rm -f "$SETTINGS_FILE"; trap - ERR # Clean up and remove trap for this iteration
+            rm -f "$SETTINGS_FILE"; trap - ERR 
           else
             echo "----------------------------------------------"
             echo "Skipping screen (not targeted in deploy config): $screen_from_build"
           fi
-        done < <(echo "$SCREEN_LIST_FOR_LOOP") # End of while loop with process substitution
+        done < <(echo "$SCREEN_LIST_FOR_LOOP") 
+
+        # Note: The DEBUG lines for final array states were here.
+        # Removing them for this request as the focus is on SETTINGS_FILE output.
+        # If issues persist with summary, they can be re-added:
+        # echo "DEBUG: === After Main Processing Loop ===" ...
 
         #############################################
         # FINAL REPORTING
@@ -246,13 +247,12 @@ runs:
 
         for screen_item in "${ALL_BUILT_SCREENS_ARRAY[@]}"; do
           status_string="⚪ Skipped" 
-          is_targeted_flag=false # Renamed to avoid conflict
+          is_targeted_flag=false 
           if echo "$TARGET_SCREENS_FROM_CONFIG_JSON" | jq -e --arg screen "$screen_item" '.[] | select(. == $screen)' > /dev/null; then
             is_targeted_flag=true
           fi
 
           if [[ "$is_targeted_flag" == true ]]; then
-            # Check against actual array elements for robustness
             is_success=false
             for s_screen in "${SUCCESS_SCREENS[@]}"; do if [[ "$s_screen" == "$screen_item" ]]; then is_success=true; break; fi; done
             
@@ -264,7 +264,7 @@ runs:
             elif [[ "$is_failed" == true ]]; then
               status_string="❌ Failed"
             else
-              status_string="❓ Unknown" # Targeted but not in success or failed arrays
+              status_string="❓ Unknown" 
             fi
           fi
           printf "│ %-14s │ %-7s │\n" "$screen_item" "$status_string"