fix(ui): improve error output formatting and remove redundant log paths

- error_handler: Use msg_info/msg_ok/msg_warn for container cleanup
  instead of raw echo with manual ANSI codes
- error_handler: Add ❓ icon before 'Remove broken container?' prompt
- error_handler: Indent log output with TAB for visual consistency
- build.func: Use msg_custom for installation log path display
- build.func: Use msg_info → msg_ok for container removal flow
- build.func: Use msg_warn for 'kept for debugging' message
- core.func/vm-core.func: Remove redundant container-internal log
  path display (📋 View full log) since combined log on host is
  the canonical location shown after failure

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,7 +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'
+    tail -n 20 "$logfile" 2>/dev/null | sed 's/\r$//'
   fi
 }
 

misc/build.func

@@ -277,9 +277,8 @@ install_ssh_keys_into_ct() {
 # ------------------------------------------------------------------------------
 # validate_container_id()
 #
-# - Validates if a container ID is available for use (CLUSTER-WIDE)
-# - Checks cluster resources via pvesh for VMs/CTs on ALL nodes
-# - Falls back to local config file check if pvesh unavailable
+# - Validates if a container ID is available for use
+# - Checks if ID is already used by VM or LXC container
 # - Checks if ID is used in LVM logical volumes
 # - Returns 0 if ID is available, 1 if already in use
 # ------------------------------------------------------------------------------
@@ -291,35 +290,11 @@ validate_container_id() {
     return 1
   fi
 
-  # CLUSTER-WIDE CHECK: Query all VMs/CTs across all nodes
-  # This catches IDs used on other nodes in the cluster
-  # NOTE: Works on single-node too - Proxmox always has internal cluster structure
-  # Falls back gracefully if pvesh unavailable or returns empty
-  if command -v pvesh &>/dev/null; then
-    local cluster_ids
-    cluster_ids=$(pvesh get /cluster/resources --type vm --output-format json 2>/dev/null | 
-      grep -oP '"vmid":\s*\K[0-9]+' 2>/dev/null || true)
-    if [[ -n "$cluster_ids" ]] && echo "$cluster_ids" | grep -qw "$ctid"; then
-      return 1
-    fi
-  fi
-
-  # LOCAL FALLBACK: Check if config file exists for VM or LXC
-  # This handles edge cases where pvesh might not return all info
+  # Check if config file exists for VM or LXC
   if [[ -f "/etc/pve/qemu-server/${ctid}.conf" ]] || [[ -f "/etc/pve/lxc/${ctid}.conf" ]]; then
     return 1
   fi
 
-  # Check ALL nodes in cluster for config files (handles pmxcfs sync delays)
-  # NOTE: On single-node, /etc/pve/nodes/ contains just the one node - still works
-  if [[ -d "/etc/pve/nodes" ]]; then
-    for node_dir in /etc/pve/nodes/*/; do
-      if [[ -f "${node_dir}qemu-server/${ctid}.conf" ]] || [[ -f "${node_dir}lxc/${ctid}.conf" ]]; then
-        return 1
-      fi
-    done
-  fi
-
   # Check if ID is used in LVM logical volumes
   if lvs --noheadings -o lv_name 2>/dev/null | grep -qE "(^|[-_])${ctid}($|[-_])"; then
     return 1
@@ -331,30 +306,63 @@ validate_container_id() {
 # ------------------------------------------------------------------------------
 # get_valid_container_id()
 #
-# - Returns a valid, unused container ID (CLUSTER-AWARE)
-# - Uses pvesh /cluster/nextid as starting point (already cluster-aware)
+# - Returns a valid, unused container ID
 # - If provided ID is valid, returns it
-# - Otherwise increments until a free one is found across entire cluster
+# - Otherwise increments from suggested ID until a free one is found
 # - Calls validate_container_id() to check availability
 # ------------------------------------------------------------------------------
 get_valid_container_id() {
-  local suggested_id="${1:-$(pvesh get /cluster/nextid 2>/dev/null || echo 100)}"
-
-  # Ensure we have a valid starting ID
-  if ! [[ "$suggested_id" =~ ^[0-9]+$ ]]; then
-    suggested_id=$(pvesh get /cluster/nextid 2>/dev/null || echo 100)
-  fi
-
-  local max_attempts=1000
-  local attempts=0
+  local suggested_id="${1:-$(pvesh get /cluster/nextid)}"
+
+  while ! validate_container_id "$suggested_id"; do
+    suggested_id=$((suggested_id + 1))
+  done
+
+  echo "$suggested_id"
+}
+
+# ------------------------------------------------------------------------------
+# validate_container_id()
+#
+# - Validates if a container ID is available for use
+# - Checks if ID is already used by VM or LXC container
+# - Checks if ID is used in LVM logical volumes
+# - Returns 0 if ID is available, 1 if already in use
+# ------------------------------------------------------------------------------
+validate_container_id() {
+  local ctid="$1"
+
+  # Check if ID is numeric
+  if ! [[ "$ctid" =~ ^[0-9]+$ ]]; then
+    return 1
+  fi
+
+  # Check if config file exists for VM or LXC
+  if [[ -f "/etc/pve/qemu-server/${ctid}.conf" ]] || [[ -f "/etc/pve/lxc/${ctid}.conf" ]]; then
+    return 1
+  fi
+
+  # Check if ID is used in LVM logical volumes
+  if lvs --noheadings -o lv_name 2>/dev/null | grep -qE "(^|[-_])${ctid}($|[-_])"; then
+    return 1
+  fi
+
+  return 0
+}
+
+# ------------------------------------------------------------------------------
+# get_valid_container_id()
+#
+# - Returns a valid, unused container ID
+# - If provided ID is valid, returns it
+# - Otherwise increments from suggested ID until a free one is found
+# - Calls validate_container_id() to check availability
+# ------------------------------------------------------------------------------
+get_valid_container_id() {
+  local suggested_id="${1:-$(pvesh get /cluster/nextid)}"
 
   while ! validate_container_id "$suggested_id"; do
     suggested_id=$((suggested_id + 1))
-    attempts=$((attempts + 1))
-    if [[ $attempts -ge $max_attempts ]]; then
-      msg_error "Could not find available container ID after $max_attempts attempts"
-      exit 1
-    fi
   done
 
   echo "$suggested_id"

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:-}"
 

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