fix: address code review feedback - WidgetHost lifecycle, ReportRenderer null safety, page-renderer pattern extraction

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

components/renderers/ReportRenderer.tsx

@@ -181,7 +181,7 @@ function SummaryReport({
                   {col.label ?? col.field} ({col.aggregate})
                 </Text>
                 <Text className="text-sm font-semibold text-primary">
-                  {computeAggregate(rows, col.field, col.aggregate!).toLocaleString()}
+                  {computeAggregate(rows, col.field, col.aggregate ?? "count").toLocaleString()}
                 </Text>
               </View>
             ))}

components/renderers/WidgetHost.tsx

@@ -34,7 +34,7 @@ export function WidgetHost({ type, props = {}, fallback }: WidgetHostProps) {
   const mountedRef = useRef(false);
 
   useEffect(() => {
-    if (!entry) return;
+    if (!getWidget(type)) return;
     if (!mountedRef.current) {
       mountedRef.current = true;
       emitWidgetLifecycle({
@@ -50,7 +50,7 @@ export function WidgetHost({ type, props = {}, fallback }: WidgetHostProps) {
         timestamp: Date.now(),
       });
     };
-  }, [entry, type]);
+  }, [type]);
 
   if (!entry) {
     if (fallback) return <>{fallback}</>;

lib/page-renderer.ts

@@ -87,6 +87,12 @@ export function validatePageSchema(
   return raw as PageSchema;
 }
 
+/* ------------------------------------------------------------------ */
+/*  Constants                                                          */
+/* ------------------------------------------------------------------ */
+
+const VARIABLE_PATTERN = /^\{\{(.+)\}\}$/;
+
 /* ------------------------------------------------------------------ */
 /*  Resolution                                                         */
 /* ------------------------------------------------------------------ */
@@ -101,12 +107,15 @@ function resolveProps(
   if (!props) return {};
   const resolved: Record<string, unknown> = {};
   for (const [key, value] of Object.entries(props)) {
-    if (typeof value === "string" && value.startsWith("{{") && value.endsWith("}}")) {
-      const varName = value.slice(2, -2).trim();
-      resolved[key] = variables[varName] ?? value;
-    } else {
-      resolved[key] = value;
+    if (typeof value === "string") {
+      const match = VARIABLE_PATTERN.exec(value);
+      if (match) {
+        const varName = match[1].trim();
+        resolved[key] = variables[varName] ?? value;
+        continue;
+      }
     }
+    resolved[key] = value;
   }
   return resolved;
 }