audit echo vs log across all scripts — outcomes always visible, verbose for per-item loops

Plugin/unraid/Partnership/containers.sh

@@ -43,7 +43,7 @@ wait_for_container_healthy() {
 
         case "$status" in
             healthy|true)
-                log "  $name ready ✅"
+                echo "  $name ready ✅"
                 return 0
                 ;;
             *)
@@ -159,7 +159,7 @@ deploy_container_from_xml() {
        timeout 120 ssh -i "$ssh_key" -o ConnectTimeout="$SSH_TIMEOUT" root@"$remote_ip" \
         "bash '$remote_script' 2>&1; rc=\$?; rm -f '$remote_script'; exit \$rc" 2>/dev/null | \
         grep -q "deployed:${name}"; then
-        log "  $name deployed ✅"
+        echo "  $name deployed ✅"
         rm -f "$tmp_script"
         return 0
     else
@@ -253,7 +253,7 @@ cleanup_deployed_stack_on_remote() {
             "docker stop '$cname' >/dev/null 2>&1
              docker rm   '$cname' >/dev/null 2>&1 && echo removed" 2>/dev/null | \
             grep -q removed && \
-            log "  $cname removed from $MIRROR ✅" || \
+            echo "  $cname removed from $MIRROR ✅" || \
             log "  $cname not found on $MIRROR — skipping"
 
         while IFS= read -r path; do
@@ -261,7 +261,7 @@ cleanup_deployed_stack_on_remote() {
             timeout "$SSH_TIMEOUT" ssh -i "$ssh_key" \
                 -o ConnectTimeout="$SSH_TIMEOUT" root@"$remote_ip" \
                 "rm -rf '$path' && echo removed" 2>/dev/null | grep -q removed && \
-                log "  Appdata removed on $MIRROR: $path ✅" || \
+                echo "  Appdata removed on $MIRROR: $path ✅" || \
                 warn "  Failed to remove appdata on $MIRROR: $path"
         done <<< "$appdata_paths"
     done
@@ -331,14 +331,14 @@ cleanup_deployed_stack_locally() {
             timeout "${DOCKER_TIMEOUT:-30}" docker stop "$cname" >/dev/null 2>&1 || true
             _PM_TRAP_STOPPED+=("$cname")
             timeout "${DOCKER_TIMEOUT:-30}" docker rm   "$cname" >/dev/null 2>&1 && \
-                log "  $cname removed ✅" || warn "  $cname rm failed"
+                echo "  $cname removed ✅" || warn "  $cname rm failed"
         else
             log "  $cname not found locally — skipping"
         fi
 
         while IFS= read -r path; do
             [[ -z "$path" ]] && continue
-            rm -rf "$path" && log "  Appdata removed: $path ✅" || warn "  Failed to remove: $path"
+            rm -rf "$path" && echo "  Appdata removed: $path ✅" || warn "  Failed to remove: $path"
         done <<< "$appdata_paths"
     done
 }
@@ -372,7 +372,7 @@ reconfigure_webui() {
         -o ConnectTimeout="$SSH_TIMEOUT" root@"$remote_ip" \
         "sed -i 's|<WebUI>.*</WebUI>|<WebUI>http://${target_ip}:${port}/</WebUI>|g' '$template'" \
         2>/dev/null && \
-        log "$container → http://${target_ip}:${port}/ ✅" || {
+        echo "$container → http://${target_ip}:${port}/ ✅" || {
         error "Failed to reconfigure $container WebUI on $label"
         return 1
     }
@@ -408,7 +408,7 @@ reconfigure_local_webuis() {
 
         sed -i "s|<WebUI>.*</WebUI>|<WebUI>http://${target_ip}:${port}/</WebUI>|g" \
             "$template" 2>/dev/null && \
-            log "$container → http://${target_ip}:${port}/ ✅" || \
+            echo "$container → http://${target_ip}:${port}/ ✅" || \
             { error "Failed to reconfigure $container"; (( failures++ )); }
     done
     return $failures

Plugin/unraid/System_Essentials/mover_stop.sh

@@ -135,7 +135,7 @@ MOVER_PID=$(platform_get_mover_pid)
 MOVER_START=$(ps -o lstart= -p "$MOVER_PID" 2>/dev/null | xargs)
 MOVER_ELAPSED=$(ps -o etimes= -p "$MOVER_PID" 2>/dev/null | tr -d ' ')
 warn "Mover is running (PID $MOVER_PID) — stopping in ${MOVER_STOP_TIMEOUT}s"
-log "$ICON_TIME Mover started: ${MOVER_START:-unknown} — running for $(format_duration "${MOVER_ELAPSED:-0}")"
+echo "$ICON_TIME Mover started: ${MOVER_START:-unknown} — running for $(format_duration "${MOVER_ELAPSED:-0}")"
 
 # ── Warn users via wall ───────────────────────────────────────────────────────────────────────
 if [[ "$DRY_RUN" == false ]]; then

Plugin/unraid/System_Essentials/php_fpm_max_children.sh

@@ -196,7 +196,7 @@ if ! sed -i "s/^pm\.max_children.*/pm.max_children = $PHP_MAX_CHILDREN/" "$PHP_C
     exit 1
 fi
 
-log "Config updated"
+echo "Config updated"
 
 # ── Restart PHP-FPM ──────────────────────────────────────────────────────────────────────────
 log "Restarting PHP-FPM..."
@@ -223,7 +223,7 @@ if [[ "${APPLIED_VAL:-0}" -ne "$PHP_MAX_CHILDREN" ]]; then
     warn "Config reads pm.max_children = ${APPLIED_VAL:-unknown} — expected $PHP_MAX_CHILDREN"
     warn "Check $PHP_CONF manually"
 else
-    log "Verified: pm.max_children = $APPLIED_VAL ✅"
+    echo "Verified: pm.max_children = $APPLIED_VAL ✅"
 fi
 
 FPM_WORKERS=$(pgrep -fc "php-fpm" 2>/dev/null || echo "?")

Plugin/unraid/System_Essentials/unraid_api_key_renew.sh

@@ -155,7 +155,7 @@ PUSHSCRIPT
         if timeout 10 ssh -i "$SSH_KEY" -o ConnectTimeout=10 -o BatchMode=yes \
             -o StrictHostKeyChecking=no "root@${partner_ip}" \
             "bash '${remote}'; rc=\$?; rm -f '${remote}'; exit \$rc" 2>/dev/null | grep -q ok; then
-            log "Key pushed to $partner_host ✅"
+            echo "Key pushed to $partner_host ✅"
         else
             warn "Key push to $partner_host failed — they can create their own copy"
         fi

Plugin/unraid/Tools/recreate_shares.sh

@@ -154,7 +154,7 @@ if ! mountpoint -q /mnt/user; then
         "Recreate Shares" "warning"
     exit 1
 fi
-log "Array is started — /mnt/user is mounted ✅"
+echo "Array is started — /mnt/user is mounted ✅"
 
 # Check share cfg directory exists and has files
 if [[ ! -d "$SHARE_CFG_DIR" ]]; then
@@ -222,7 +222,7 @@ for cfg in "${CFG_FILES[@]}"; do
                 warn "DRY RUN — would create: $DISK_PATH"
                 (( DIRS_CREATED++ ))
             elif mkdir -p "$DISK_PATH"; then
-                log "Created: $DISK_PATH ✅"
+                echo "Created: $DISK_PATH ✅"
                 (( DIRS_CREATED++ ))
             else
                 error "Failed to create: $DISK_PATH"
@@ -242,7 +242,7 @@ for cfg in "${CFG_FILES[@]}"; do
             warn "DRY RUN — would place marker: $MARKER_PATH"
             CREATED+=("$SHARE_NAME")
         elif touch "$MARKER_PATH" 2>/dev/null; then
-            log "Marker placed: $MARKER_PATH ✅"
+            echo "Marker placed: $MARKER_PATH ✅"
             CREATED+=("$SHARE_NAME")
         else
             warn "$SHARE_NAME — could not place .recovery marker"