fix(cluster): validate container IDs cluster-wide across all nodes

- Query /cluster/resources via pvesh to check all VMs/CTs on ALL nodes
- Check /etc/pve/nodes/*/qemu-server and /etc/pve/nodes/*/lxc dirs
- Handles pmxcfs sync delays that caused sporadic ID conflicts
- Remove duplicate validate_container_id/get_valid_container_id definitions
- Add max_attempts safeguard to prevent infinite loops

CHANGELOG.md

@@ -403,24 +403,12 @@ Exercise vigilance regarding copycat or coat-tailing sites that seek to exploit
 
 ## 2026-02-14
 
-### 🚀 Updated Scripts
-
-  - #### ✨ New Features
-
-    - core: overwriteable app version [@CrazyWolf13](https://github.com/CrazyWolf13) ([#11753](https://github.com/community-scripts/ProxmoxVE/pull/11753))
-
 ### 💾 Core
 
   - #### ✨ New Features
 
-    - core: validate container IDs cluster-wide across all nodes [@MickLesk](https://github.com/MickLesk) ([#11906](https://github.com/community-scripts/ProxmoxVE/pull/11906))
-    - core: improve error reporting with structured error strings and better categorization + output formatting [@MickLesk](https://github.com/MickLesk) ([#11907](https://github.com/community-scripts/ProxmoxVE/pull/11907))
     - core: unified logging system with combined logs [@MickLesk](https://github.com/MickLesk) ([#11761](https://github.com/community-scripts/ProxmoxVE/pull/11761))
 
-### 🧰 Tools
-
-  - lxc-updater: add patchmon aware [@failure101](https://github.com/failure101) ([#11905](https://github.com/community-scripts/ProxmoxVE/pull/11905))
-
 ### ❔ Uncategorized
 
   - Disable UniFi script - APT packages no longer available [@Copilot](https://github.com/Copilot) ([#11898](https://github.com/community-scripts/ProxmoxVE/pull/11898))

ct/ollama.sh

@@ -9,7 +9,7 @@ APP="Ollama"
 var_tags="${var_tags:-ai}"
 var_cpu="${var_cpu:-4}"
 var_ram="${var_ram:-4096}"
-var_disk="${var_disk:-40}"
+var_disk="${var_disk:-35}"
 var_os="${var_os:-ubuntu}"
 var_version="${var_version:-24.04}"
 var_gpu="${var_gpu:-yes}"

ct/openwebui.sh

@@ -9,7 +9,7 @@ APP="Open WebUI"
 var_tags="${var_tags:-ai;interface}"
 var_cpu="${var_cpu:-4}"
 var_ram="${var_ram:-8192}"
-var_disk="${var_disk:-50}"
+var_disk="${var_disk:-25}"
 var_os="${var_os:-debian}"
 var_version="${var_version:-13}"
 var_unprivileged="${var_unprivileged:-1}"

frontend/public/json/github-versions.json

@@ -1,5 +1,5 @@
 {
-  "generated": "2026-02-14T18:07:29Z",
+  "generated": "2026-02-14T12:08:41Z",
   "versions": [
     {
       "slug": "2fauth",
@@ -116,9 +116,9 @@
     {
       "slug": "bentopdf",
       "repo": "alam00000/bentopdf",
-      "version": "v2.2.1",
+      "version": "v2.2.0",
       "pinned": false,
-      "date": "2026-02-14T16:33:47Z"
+      "date": "2026-02-09T07:07:40Z"
     },
     {
       "slug": "beszel",
@@ -354,9 +354,9 @@
     {
       "slug": "firefly",
       "repo": "firefly-iii/firefly-iii",
-      "version": "v6.4.20",
+      "version": "v6.4.19",
       "pinned": false,
-      "date": "2026-02-14T12:39:02Z"
+      "date": "2026-02-14T11:55:40Z"
     },
     {
       "slug": "fladder",
@@ -578,9 +578,9 @@
     {
       "slug": "jackett",
       "repo": "Jackett/Jackett",
-      "version": "v0.24.1113",
+      "version": "v0.24.1109",
       "pinned": false,
-      "date": "2026-02-14T17:46:58Z"
+      "date": "2026-02-14T05:54:26Z"
     },
     {
       "slug": "jellystat",
@@ -1411,9 +1411,9 @@
     {
       "slug": "tandoor",
       "repo": "TandoorRecipes/recipes",
-      "version": "2.5.3",
+      "version": "2.5.1",
       "pinned": false,
-      "date": "2026-02-14T12:42:14Z"
+      "date": "2026-02-13T15:57:27Z"
     },
     {
       "slug": "tasmoadmin",

frontend/public/json/ollama.json

@@ -21,7 +21,7 @@
       "resources": {
         "cpu": 4,
         "ram": 4096,
-        "hdd": 40,
+        "hdd": 35,
         "os": "Ubuntu",
         "version": "24.04"
       }

frontend/public/json/openwebui.json

@@ -21,7 +21,7 @@
       "resources": {
         "cpu": 4,
         "ram": 8192,
-        "hdd": 50,
+        "hdd": 25,
         "os": "debian",
         "version": "13"
       }

misc/api.func

@@ -237,21 +237,16 @@ explain_exit_code() {
 # json_escape()
 #
 # - Escapes a string for safe JSON embedding
-# - Strips ANSI escape sequences and non-printable control characters
 # - Handles backslashes, quotes, newlines, tabs, and carriage returns
 # ------------------------------------------------------------------------------
 json_escape() {
   local s="$1"
-  # Strip ANSI escape sequences (color codes etc.)
-  s=$(printf '%s' "$s" | sed 's/\x1b\[[0-9;]*[a-zA-Z]//g')
   s=${s//\\/\\\\}
   s=${s//"/\\"/}
   s=${s//$'\n'/\\n}
   s=${s//$'\r'/}
   s=${s//$'\t'/\\t}
-  # Remove any remaining control characters (0x00-0x1F except those already handled)
-  s=$(printf '%s' "$s" | tr -d '\000-\010\013\014\016-\037')
-  printf '%s' "$s"
+  echo "$s"
 }
 
 # ------------------------------------------------------------------------------
@@ -288,33 +283,7 @@ get_error_text() {
   fi
 
   if [[ -n "$logfile" && -s "$logfile" ]]; then
-    tail -n 20 "$logfile" 2>/dev/null | sed 's/\r$//' | sed 's/\x1b\[[0-9;]*[a-zA-Z]//g'
-  fi
-}
-
-# ------------------------------------------------------------------------------
-# build_error_string()
-#
-# - Builds a structured error string for telemetry reporting
-# - Format: "exit_code=<N> | <explanation>\n---\n<last 20 log lines>"
-# - If no log lines available, returns just the explanation
-# - Arguments:
-#   * $1: exit_code (numeric)
-#   * $2: log_text (optional, output from get_error_text)
-# - Returns structured error string via stdout
-# ------------------------------------------------------------------------------
-build_error_string() {
-  local exit_code="${1:-1}"
-  local log_text="${2:-}"
-  local explanation
-  explanation=$(explain_exit_code "$exit_code")
-
-  if [[ -n "$log_text" ]]; then
-    # Structured format: header + separator + log lines
-    printf 'exit_code=%s | %s\n---\n%s' "$exit_code" "$explanation" "$log_text"
-  else
-    # No log available - just the explanation with exit code
-    printf 'exit_code=%s | %s' "$exit_code" "$explanation"
+    tail -n 20 "$logfile" 2>/dev/null | sed 's/\r$//'
   fi
 }
 
@@ -696,12 +665,13 @@ post_update_to_api() {
     else
       exit_code=1
     fi
-    # Get log lines and build structured error string
     local error_text=""
     error_text=$(get_error_text)
-    local full_error
-    full_error=$(build_error_string "$exit_code" "$error_text")
-    error=$(json_escape "$full_error")
+    if [[ -n "$error_text" ]]; then
+      error=$(json_escape "$error_text")
+    else
+      error=$(json_escape "$(explain_exit_code "$exit_code")")
+    fi
     short_error=$(json_escape "$(explain_exit_code "$exit_code")")
     error_category=$(categorize_error "$exit_code")
     [[ -z "$error" ]] && error="Unknown error"
@@ -844,52 +814,31 @@ EOF
 categorize_error() {
   local code="$1"
   case "$code" in
-  # Network errors (curl/wget)
-  6 | 7 | 22 | 35) echo "network" ;;
+  # Network errors
+  6 | 7 | 22 | 28 | 35) echo "network" ;;
 
-  # Timeout errors
-  28 | 124 | 211) echo "timeout" ;;
+  # Storage errors
+  214 | 217 | 219) echo "storage" ;;
 
-  # Storage errors (Proxmox storage)
-  214 | 217 | 219 | 224) echo "storage" ;;
-
-  # Dependency/Package errors (APT, DPKG, pip, commands)
-  100 | 101 | 102 | 127 | 160 | 161 | 162 | 255) echo "dependency" ;;
+  # Dependency/Package errors
+  100 | 101 | 102 | 127 | 160 | 161 | 162) echo "dependency" ;;
 
   # Permission errors
   126 | 152) echo "permission" ;;
 
-  # Configuration errors (Proxmox config, invalid args)
-  128 | 203 | 204 | 205 | 206 | 207 | 208) echo "config" ;;
+  # Timeout errors
+  124 | 28 | 211) echo "timeout" ;;
 
-  # Proxmox container/template errors
-  200 | 209 | 210 | 212 | 213 | 215 | 216 | 218 | 220 | 221 | 222 | 223 | 225 | 231) echo "proxmox" ;;
-
-  # Service/Systemd errors
-  150 | 151 | 153 | 154) echo "service" ;;
-
-  # Database errors (PostgreSQL, MySQL, MongoDB)
-  170 | 171 | 172 | 173 | 180 | 181 | 182 | 183 | 190 | 191 | 192 | 193) echo "database" ;;
-
-  # Node.js / JavaScript runtime errors
-  243 | 245 | 246 | 247 | 248 | 249) echo "runtime" ;;
-
-  # Python environment errors
-  # (already covered: 160-162 under dependency)
+  # Configuration errors
+  203 | 204 | 205 | 206 | 207 | 208) echo "config" ;;
 
   # Aborted by user
   130) echo "aborted" ;;
 
-  # Resource errors (OOM, SIGKILL, SIGABRT)
-  134 | 137) echo "resource" ;;
+  # Resource errors (OOM, etc)
+  137 | 134) echo "resource" ;;
 
-  # Signal/Process errors (SIGTERM, SIGPIPE, SIGSEGV)
-  139 | 141 | 143) echo "signal" ;;
-
-  # Shell errors (general error, syntax error)  
-  1 | 2) echo "shell" ;;
-
-  # Default - truly unknown
+  # Default
   *) echo "unknown" ;;
   esac
 }
@@ -952,9 +901,11 @@ post_tool_to_api() {
     [[ ! "$exit_code" =~ ^[0-9]+$ ]] && exit_code=1
     local error_text=""
     error_text=$(get_error_text)
-    local full_error
-    full_error=$(build_error_string "$exit_code" "$error_text")
-    error=$(json_escape "$full_error")
+    if [[ -n "$error_text" ]]; then
+      error=$(json_escape "$error_text")
+    else
+      error=$(json_escape "$(explain_exit_code "$exit_code")")
+    fi
     error_category=$(categorize_error "$exit_code")
   fi
 
@@ -1017,9 +968,11 @@ post_addon_to_api() {
     [[ ! "$exit_code" =~ ^[0-9]+$ ]] && exit_code=1
     local error_text=""
     error_text=$(get_error_text)
-    local full_error
-    full_error=$(build_error_string "$exit_code" "$error_text")
-    error=$(json_escape "$full_error")
+    if [[ -n "$error_text" ]]; then
+      error=$(json_escape "$error_text")
+    else
+      error=$(json_escape "$(explain_exit_code "$exit_code")")
+    fi
     error_category=$(categorize_error "$exit_code")
   fi
 
@@ -1114,9 +1067,11 @@ post_update_to_api_extended() {
     fi
     local error_text=""
     error_text=$(get_error_text)
-    local full_error
-    full_error=$(build_error_string "$exit_code" "$error_text")
-    error=$(json_escape "$full_error")
+    if [[ -n "$error_text" ]]; then
+      error=$(json_escape "$error_text")
+    else
+      error=$(json_escape "$(explain_exit_code "$exit_code")")
+    fi
     error_category=$(categorize_error "$exit_code")
     [[ -z "$error" ]] && error="Unknown error"
   fi

misc/build.func

@@ -4125,7 +4125,8 @@ EOF'
 
     # Show combined log location
     if [[ -n "$CTID" && -n "${SESSION_ID:-}" ]]; then
-      msg_custom "📋" "${YW}" "Installation log: ${combined_log}"
+      echo ""
+      echo -e "${GN}✔${CL} Installation log: ${BL}${combined_log}${CL}"
     fi
 
     # Dev mode: Keep container or open breakpoint shell
@@ -4148,21 +4149,19 @@ EOF'
       exit $install_exit_code
     fi
 
-    # Prompt user for cleanup with 60s timeout
+    # Prompt user for cleanup with 60s timeout (plain echo - no msg_info to avoid spinner)
     echo ""
-    echo -en "${TAB}❓${TAB}${YW}Remove broken container ${CTID}? (Y/n) [auto-remove in 60s]: ${CL}"
+    echo -en "${YW}Remove broken container ${CTID}? (Y/n) [auto-remove in 60s]: ${CL}"
 
     if read -t 60 -r response; then
       if [[ -z "$response" || "$response" =~ ^[Yy]$ ]]; then
         # Remove container
-        echo ""
-        msg_info "Removing container ${CTID}"
+        echo -e "\n${TAB}${HOLD}${YW}Removing container ${CTID}${CL}"
         pct stop "$CTID" &>/dev/null || true
         pct destroy "$CTID" &>/dev/null || true
-        msg_ok "Container ${CTID} removed"
+        echo -e "${BFR}${CM}${GN}Container ${CTID} removed${CL}"
       elif [[ "$response" =~ ^[Nn]$ ]]; then
-        echo ""
-        msg_warn "Container ${CTID} kept for debugging"
+        echo -e "\n${TAB}${YW}Container ${CTID} kept for debugging${CL}"
 
         # Dev mode: Setup MOTD/SSH for debugging access to broken container
         if [[ "${DEV_MODE_MOTD:-false}" == "true" ]]; then
@@ -4178,11 +4177,11 @@ EOF'
       fi
     else
       # Timeout - auto-remove
-      echo ""
-      msg_info "No response - removing container ${CTID}"
+      echo -e "\n${YW}No response - auto-removing container${CL}"
+      echo -e "${TAB}${HOLD}${YW}Removing container ${CTID}${CL}"
       pct stop "$CTID" &>/dev/null || true
       pct destroy "$CTID" &>/dev/null || true
-      msg_ok "Container ${CTID} removed"
+      echo -e "${BFR}${CM}${GN}Container ${CTID} removed${CL}"
     fi
 
     # Force one final status update attempt after cleanup

misc/core.func

@@ -522,9 +522,15 @@ silent() {
     msg_custom "→" "${YWB}" "${cmd}"
 
     if [[ -s "$logfile" ]]; then
-      echo -e "\n${TAB}--- Last 10 lines of log ---"
+      local log_lines=$(wc -l <"$logfile")
+      echo "--- Last 10 lines of silent log ---"
       tail -n 10 "$logfile"
-      echo -e "${TAB}-----------------------------------\n"
+      echo "-----------------------------------"
+
+      # Show how to view full log if there are more lines
+      if [[ $log_lines -gt 10 ]]; then
+        msg_custom "📋" "${YW}" "View full log (${log_lines} lines): ${logfile}"
+      fi
     fi
 
     exit "$rc"

misc/error_handler.func

@@ -175,9 +175,9 @@ error_handler() {
   fi
 
   if [[ -n "$active_log" && -s "$active_log" ]]; then
-    echo -e "\n${TAB}--- Last 20 lines of log ---"
+    echo "--- Last 20 lines of silent log ---"
     tail -n 20 "$active_log"
-    echo -e "${TAB}-----------------------------------\n"
+    echo "-----------------------------------"
 
     # Detect context: Container (INSTALL_LOG set + /root exists) vs Host (BUILD_LOG)
     if [[ -n "${INSTALL_LOG:-}" && -d /root ]]; then
@@ -204,50 +204,23 @@ error_handler() {
         fi
 
         echo ""
-        if declare -f msg_custom >/dev/null 2>&1; then
-          echo -en "${TAB}❓${TAB}${YW}Remove broken container ${CTID}? (Y/n) [auto-remove in 60s]: ${CL}"
-        else
-          echo -en "${YW}Remove broken container ${CTID}? (Y/n) [auto-remove in 60s]: ${CL}"
-        fi
+        echo -en "${YW}Remove broken container ${CTID}? (Y/n) [auto-remove in 60s]: ${CL}"
 
         if read -t 60 -r response; then
           if [[ -z "$response" || "$response" =~ ^[Yy]$ ]]; then
-            echo ""
-            if declare -f msg_info >/dev/null 2>&1; then
-              msg_info "Removing container ${CTID}"
-            else
-              echo -e "${YW}Removing container ${CTID}${CL}"
-            fi
+            echo -e "\n${YW}Removing container ${CTID}${CL}"
             pct stop "$CTID" &>/dev/null || true
             pct destroy "$CTID" &>/dev/null || true
-            if declare -f msg_ok >/dev/null 2>&1; then
-              msg_ok "Container ${CTID} removed"
-            else
-              echo -e "${GN}✔${CL} Container ${CTID} removed"
-            fi
+            echo -e "${GN}✔${CL} Container ${CTID} removed"
           elif [[ "$response" =~ ^[Nn]$ ]]; then
-            echo ""
-            if declare -f msg_warn >/dev/null 2>&1; then
-              msg_warn "Container ${CTID} kept for debugging"
-            else
-              echo -e "${YW}Container ${CTID} kept for debugging${CL}"
-            fi
+            echo -e "\n${YW}Container ${CTID} kept for debugging${CL}"
           fi
         else
           # Timeout - auto-remove
-          echo ""
-          if declare -f msg_info >/dev/null 2>&1; then
-            msg_info "No response - removing container ${CTID}"
-          else
-            echo -e "${YW}No response - removing container ${CTID}${CL}"
-          fi
+          echo -e "\n${YW}No response - auto-removing container${CL}"
           pct stop "$CTID" &>/dev/null || true
           pct destroy "$CTID" &>/dev/null || true
-          if declare -f msg_ok >/dev/null 2>&1; then
-            msg_ok "Container ${CTID} removed"
-          else
-            echo -e "${GN}✔${CL} Container ${CTID} removed"
-          fi
+          echo -e "${GN}✔${CL} Container ${CTID} removed"
         fi
 
         # Force one final status update attempt after cleanup
@@ -281,10 +254,6 @@ on_exit() {
   # post_to_api was called ("installing" sent) but post_update_to_api was never called
   if [[ "${POST_TO_API_DONE:-}" == "true" && "${POST_UPDATE_DONE:-}" != "true" ]]; then
     if declare -f post_update_to_api >/dev/null 2>&1; then
-      # Ensure log is accessible on host before reporting
-      if declare -f ensure_log_on_host >/dev/null 2>&1; then
-        ensure_log_on_host
-      fi
       if [[ $exit_code -ne 0 ]]; then
         post_update_to_api "failed" "$exit_code"
       else
@@ -304,10 +273,6 @@ on_exit() {
 # - Exits with code 130 (128 + SIGINT=2)
 # ------------------------------------------------------------------------------
 on_interrupt() {
-  # Ensure log is accessible on host before reporting
-  if declare -f ensure_log_on_host >/dev/null 2>&1; then
-    ensure_log_on_host
-  fi
   # Report interruption to telemetry API (prevents stuck "installing" records)
   if declare -f post_update_to_api >/dev/null 2>&1; then
     post_update_to_api "failed" "130"
@@ -329,10 +294,6 @@ on_interrupt() {
 # - Triggered by external process termination
 # ------------------------------------------------------------------------------
 on_terminate() {
-  # Ensure log is accessible on host before reporting
-  if declare -f ensure_log_on_host >/dev/null 2>&1; then
-    ensure_log_on_host
-  fi
   # Report termination to telemetry API (prevents stuck "installing" records)
   if declare -f post_update_to_api >/dev/null 2>&1; then
     post_update_to_api "failed" "143"

misc/tools.func

@@ -1913,7 +1913,7 @@ function fetch_and_deploy_codeberg_release() {
   local app="$1"
   local repo="$2"
   local mode="${3:-tarball}" # tarball | binary | prebuild | singlefile | tag
-  local version="${var_appversion:-${4:-latest}}"
+  local version="${4:-latest}"
   local target="${5:-/opt/$app}"
   local asset_pattern="${6:-}"
 
@@ -2443,7 +2443,7 @@ function fetch_and_deploy_gh_release() {
   local app="$1"
   local repo="$2"
   local mode="${3:-tarball}" # tarball | binary | prebuild | singlefile
-  local version="${var_appversion:-${4:-latest}}"
+  local version="${4:-latest}"
   local target="${5:-/opt/$app}"
   local asset_pattern="${6:-}"
 

misc/vm-core.func

@@ -207,9 +207,15 @@ silent() {
     msg_custom "→" "${YWB}" "${cmd}"
 
     if [[ -s "$logfile" ]]; then
-      echo -e "\n${TAB}--- Last 10 lines of log ---"
+      local log_lines=$(wc -l <"$logfile")
+      echo "--- Last 10 lines of log ---"
       tail -n 10 "$logfile"
-      echo -e "${TAB}----------------------------\n"
+      echo "----------------------------"
+
+      # Show how to view full log if there are more lines
+      if [[ $log_lines -gt 10 ]]; then
+        msg_custom "📋" "${YW}" "View full log (${log_lines} lines): ${logfile}"
+      fi
     fi
 
     exit "$rc"

tools/pve/update-lxcs.sh

@@ -110,11 +110,6 @@ for container in $(pct list | awk '{if(NR>1) print $1}'); do
       container_hostname=$(pct exec "$container" hostname)
       containers_needing_reboot+=("$container ($container_hostname)")
     fi
-    # check if patchmon agent is present in container and run a report if found
-    if pct exec "$container" -- [ -e "/usr/local/bin/patchmon-agent" ]; then
-      echo -e "${BL}[Info]${GN} patchmon-agent found in ${BL} $container ${CL}, triggering report. \n"
-      pct exec "$container" -- "/usr/local/bin/patchmon-agent" "report"
-    fi
   fi
 done
 wait