From 8e92d8a3b9060e8d787bcaa08c86b3594678f61c Mon Sep 17 00:00:00 2001
From: Classic298 <27028174+Classic298@users.noreply.github.com>
Date: Wed, 23 Sep 2026 00:08:39 +0200
Subject: [PATCH] refac: chat variable property value parsing

Property values keep their raw text unless they parse into a plain JSON value, matching the existing fallback for invalid JSON.
---
 backend/open_webui/utils/chat_variables.py | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/backend/open_webui/utils/chat_variables.py b/backend/open_webui/utils/chat_variables.py
index c1776f333f..3d423cac09 100644
--- a/backend/open_webui/utils/chat_variables.py
+++ b/backend/open_webui/utils/chat_variables.py
@@ -10,6 +10,7 @@ CHAT_VARIABLE_ANY_RE = re.compile(r'{{\s*chat\.variables\.([^\s|}]+)(?:\s*\|\s*(
 USER_VARIABLE_ANY_RE = re.compile(r'{{\s*user\.variables\.([^\s|}]+)(?:\s*\|\s*([^}]*))?\s*}}')
 MAX_VARIABLE_VALUE_LENGTH = 20_000
 MAX_VARIABLES_JSON_LENGTH = 100_000
+MAX_JSON_VALUE_DEPTH = 32
 
 
 class ChatVariablesError(ValueError):
@@ -58,15 +59,27 @@ def split_properties(value: str, delimiter: str) -> list[str]:
     return result
 
 
+def is_nested_too_deep(value: Any) -> bool:
+    stack = [(value, 1)]
+    while stack:
+        item, depth = stack.pop()
+        if isinstance(item, list):
+            if depth > MAX_JSON_VALUE_DEPTH:
+                return True
+            stack.extend((child, depth + 1) for child in item)
+    return False
+
+
 def parse_json_value(value: str) -> Any:
     if value.startswith('"') and value.endswith('"'):
         return value[1:-1]
 
     if re.match(r'^[\[{]', value):
         try:
-            return JSONCodec.loads(value)
-        except JSONCodec.JSONDecodeError:
+            parsed = JSONCodec.loads(value)
+        except (JSONCodec.JSONDecodeError, RecursionError):
             return value
+        return value if is_nested_too_deep(parsed) else parsed
 
     return value