@ggui-ai/ui-gen 0.16.0 → 0.18.0

package/dist/check/index.d.ts

@@ -1,6 +1,6 @@
 import { DataContract, JsonObject, PropsSpec, ActionSpec, StreamSpec } from '@ggui-ai/protocol';
-import { D as DesignMode } from '../design-mode-BHEE7flj.js';
-import { b as EvalResult, E as EvalIssue } from '../types-public-D24JLJgl.js';
+import { D as DesignMode } from '../design-mode-DLufhrNp.js';
+import { b as EvalResult, E as EvalIssue } from '../types-public-DVSssdc-.js';
 export { j as jsonSchemaTypeToTs } from '../json-schema-ts-CbB78b3n.js';
 import '../axes-CzLEMDeB.js';
 import '../types-BOvHNG7K.js';

package/dist/check/index.js

@@ -1049,6 +1049,63 @@ function isGadgetExportImported(source, pkg, exportName) {
   }
   return false;
 }
+function detectComponentNeverReturns(sourceCode) {
+  const sf = ts4.createSourceFile(
+    "source.tsx",
+    sourceCode,
+    ts4.ScriptTarget.Latest,
+    /* setParentNodes */
+    true,
+    ts4.ScriptKind.TSX
+  );
+  const isFunctionLike = (n) => ts4.isFunctionDeclaration(n) || ts4.isFunctionExpression(n) || ts4.isArrowFunction(n) || ts4.isMethodDeclaration(n) || ts4.isClassDeclaration(n) || ts4.isClassExpression(n);
+  function resolveTopLevel(name) {
+    for (const st of sf.statements) {
+      if (ts4.isFunctionDeclaration(st) && st.name?.text === name) return st;
+      if (ts4.isVariableStatement(st)) {
+        for (const decl of st.declarationList.declarations) {
+          if (ts4.isIdentifier(decl.name) && decl.name.text === name && decl.initializer !== void 0 && (ts4.isArrowFunction(decl.initializer) || ts4.isFunctionExpression(decl.initializer))) {
+            return decl.initializer;
+          }
+        }
+      }
+    }
+    return void 0;
+  }
+  let component;
+  for (const st of sf.statements) {
+    if (ts4.isFunctionDeclaration(st) && st.modifiers?.some((m) => m.kind === ts4.SyntaxKind.DefaultKeyword)) {
+      component = st;
+      break;
+    }
+    if (ts4.isExportAssignment(st) && st.isExportEquals !== true) {
+      const expr = st.expression;
+      if (ts4.isArrowFunction(expr) || ts4.isFunctionExpression(expr)) component = expr;
+      else if (ts4.isIdentifier(expr)) component = resolveTopLevel(expr.text);
+      break;
+    }
+  }
+  if (component === void 0) return null;
+  const body = component.body;
+  if (body === void 0 || !ts4.isBlock(body)) return null;
+  let returnsValue = false;
+  let jsxStatementLine;
+  const visit = (node) => {
+    if (returnsValue) return;
+    if (ts4.isReturnStatement(node)) {
+      if (node.expression !== void 0) returnsValue = true;
+      return;
+    }
+    if (jsxStatementLine === void 0 && ts4.isExpressionStatement(node) && (ts4.isJsxElement(node.expression) || ts4.isJsxFragment(node.expression) || ts4.isJsxSelfClosingElement(node.expression))) {
+      jsxStatementLine = sf.getLineAndCharacterOfPosition(node.getStart(sf)).line + 1;
+    }
+    node.forEachChild((child) => {
+      if (!isFunctionLike(child)) visit(child);
+    });
+  };
+  for (const st of body.statements) if (!isFunctionLike(st)) visit(st);
+  return returnsValue ? null : { jsxStatementLine };
+}
 async function runTier0Checks(sourceCode, contract, buildErrors, gadgetTypes, designMode = DEFAULT_DESIGN_MODE) {
   const issues = [];
   const lines = sourceCode.split("\n");
@@ -1062,6 +1119,7 @@ async function runTier0Checks(sourceCode, contract, buildErrors, gadgetTypes, de
       tier: 0,
       result: "fail",
       category: "compile",
+      subcategory: "build-error",
       severity: "critical",
       description: `Component failed to compile:
 ${errorDetail}`,
@@ -1134,6 +1192,7 @@ ${errorDetail}`,
           tier: 0,
           result: "fail",
           category: "imports",
+          subcategory: "disallowed-package",
           severity: "critical",
           description: `Import from "${pkg}" is not allowed`,
           fix: `Only import from: ${describeAllowedImports()}`,
@@ -1466,6 +1525,19 @@ ${errorDetail}`,
       fix: "Add export default function Component(props: Props) { ... }"
     });
   }
+  const neverReturns = detectComponentNeverReturns(sourceCode);
+  if (neverReturns !== null) {
+    issues.push({
+      tier: 0,
+      result: "fail",
+      category: "compile",
+      subcategory: "renders-nothing",
+      severity: "critical",
+      description: neverReturns.jsxStatementLine === void 0 ? "The component never returns its tree \u2014 it renders a blank frame" : `The component never returns its tree: the JSX at line ${neverReturns.jsxStatementLine} sits at statement position, so it is built and thrown away and the frame renders blank`,
+      fix: "Wrap the tree in the return the boilerplate gave you: `return (` before the root element and `);` after it.",
+      ...neverReturns.jsxStatementLine === void 0 ? {} : { line: neverReturns.jsxStatementLine }
+    });
+  }
   if (contract) {
     try {
       const contractIssues = validateAllContracts(sourceCode, contract);