Deploy: drop JSON-as-text blocks (final JSON-in-UI fix)

vivek/backend/combined_prompt.py

@@ -219,12 +219,27 @@ def _chart_has_data(chart: dict[str, Any]) -> bool:
 _NUMERIC_ARRAY_RE = re.compile(r"^\s*\[\s*-?\d+(\.\d+)?(\s*,\s*-?\d+(\.\d+)?)*\s*\]\s*$")
 
 
+def _looks_like_json(s: str) -> bool:
+    try:
+        return isinstance(json.loads(s), (dict, list))
+    except Exception:
+        return s.strip()[:1] in "{["  # truncated JSON-ish
+
+
 def _block_is_renderable(b: dict[str, Any]) -> bool:
     t = str(b.get("type") or "").lower()
     if t == "text":
         content = str(b.get("content") or "").strip()
-        # Drop bare numeric arrays (e.g. tic-tac-toe win lines "[0,1,2]") — that's not prose.
-        return bool(content) and not _NUMERIC_ARRAY_RE.match(content)
+        if not content or _NUMERIC_ARRAY_RE.match(content):
+            return False  # bare numeric arrays (e.g. tic-tac-toe "[0,1,2]") are not prose
+        # Drop text whose content is actually JSON (a block/layout the model stuffed in) —
+        # it must never render as raw text in the UI.
+        if content[:1] in "{[" and (
+            re.search(r'"(?:type|layout|items|tone|label|series|chart)"\s*:', content)
+            or _looks_like_json(content)
+        ):
+            return False
+        return True
     if t == "kpi_row":
         items = b.get("items")
         return isinstance(items, list) and any(

vivek/frontend-vue/dist/assets/{ActionRow-BcmZQovL.js → ActionRow-ByJHOpPb.js}

@@ -1 +1 @@
-import{G as e,M as t,P as n,f as r,h as i,i as a,jt as o,u as s,v as c}from"./runtime-core.esm-bundler-olIhRSij.js";import{tr as l}from"./index-1ugYrWkO.js";var u={class:`flex flex-wrap gap-2`},d=c({__name:`ActionRow`,props:{block:{}},emits:[`action`],setup(c,{emit:d}){let f=d;function p(e){let t=String(e.label||e.intent||``).trim();t&&f(`action`,t)}return(d,f)=>(t(),r(`div`,u,[(t(!0),r(a,null,n(c.block.buttons||[],(n,r)=>(t(),s(l,{key:r,type:`button`,variant:`outline`,size:`sm`,title:n.intent?`Ask: ${n.label||n.intent}`:void 0,onClick:e=>p(n)},{default:e(()=>[i(o(n.label||`Action`),1)]),_:2},1032,[`title`,`onClick`]))),128))]))}});export{d as default};
+import{G as e,M as t,P as n,f as r,h as i,i as a,jt as o,u as s,v as c}from"./runtime-core.esm-bundler-olIhRSij.js";import{tr as l}from"./index-0cd4ouHe.js";var u={class:`flex flex-wrap gap-2`},d=c({__name:`ActionRow`,props:{block:{}},emits:[`action`],setup(c,{emit:d}){let f=d;function p(e){let t=String(e.label||e.intent||``).trim();t&&f(`action`,t)}return(d,f)=>(t(),r(`div`,u,[(t(!0),r(a,null,n(c.block.buttons||[],(n,r)=>(t(),s(l,{key:r,type:`button`,variant:`outline`,size:`sm`,title:n.intent?`Ask: ${n.label||n.intent}`:void 0,onClick:e=>p(n)},{default:e(()=>[i(o(n.label||`Action`),1)]),_:2},1032,[`title`,`onClick`]))),128))]))}});export{d as default};

vivek/frontend-vue/dist/index.html

@@ -5,7 +5,7 @@
     <link rel="icon" type="image/svg+xml" href="/favicon.svg" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <title>frontend-vue</title>
-    <script type="module" crossorigin src="/assets/index-1ugYrWkO.js"></script>
+    <script type="module" crossorigin src="/assets/index-0cd4ouHe.js"></script>
     <link rel="modulepreload" crossorigin href="/assets/runtime-core.esm-bundler-olIhRSij.js">
     <link rel="stylesheet" crossorigin href="/assets/index-yy7aehFb.css">
   </head>

vivek/frontend-vue/src/components/WidgetRegistryRenderer.vue

@@ -84,15 +84,33 @@ function salvageBlocks(s: string): Block[] {
   return out
 }
 
+// A text block's content that is actually JSON (a block/layout the model stuffed into
+// content) — must NEVER render as raw text. Detect and drop.
+const NUMERIC_ARRAY_RE = /^\s*\[\s*-?\d+(\.\d+)?(\s*,\s*-?\d+(\.\d+)?)*\s*\]\s*$/
+function textContentIsJson(content: string): boolean {
+  const t = String(content || '').trim()
+  if (!(t.startsWith('{') || t.startsWith('['))) return false
+  // looks like a widget block/array, or parses as a JSON object/array
+  if (/"(?:type|layout|items|tone|label|series|chart|value)"\s*:/.test(t)) return true
+  try {
+    const o = JSON.parse(t)
+    return o && typeof o === 'object'
+  } catch {
+    return /^[{[]/.test(t) // truncated JSON-ish → still drop
+  }
+}
+
 function finalizeBlocks(layout: unknown[]): Block[] {
   const normalized = layout.map((b) => normalizeWidgetBlock(b)) as Block[]
-  // Drop bare numeric-array text blocks (e.g. tic-tac-toe win lines "[0,1,2]") — not real content.
-  const NUMERIC_ARRAY_RE = /^\s*\[\s*-?\d+(\.\d+)?(\s*,\s*-?\d+(\.\d+)?)*\s*\]\s*$/
   return normalized.filter((b) => {
     const type = String(b.type || '').toLowerCase()
     // Never show an empty chart card — drop charts with no renderable data outright.
     if (type === 'chart' && !chartHasRenderableData((b as { chart?: any }).chart || {})) return false
-    if (type === 'text' && NUMERIC_ARRAY_RE.test(String((b as { content?: string }).content ?? ''))) return false
+    if (type === 'text') {
+      const c = String((b as { content?: string }).content ?? '')
+      // Drop numeric-array junk AND any text block whose content is raw JSON.
+      if (NUMERIC_ARRAY_RE.test(c) || textContentIsJson(c)) return false
+    }
     return true
   })
 }