@ggui-ai/ui-gen 0.12.0 → 0.14.0

package/dist/check/index.d.ts

@@ -1,5 +1,5 @@
 import { DataContract, JsonObject, PropsSpec, ActionSpec, StreamSpec } from '@ggui-ai/protocol';
-import { b as EvalResult, E as EvalIssue } from '../types-public-DIpjC-OA.js';
+import { b as EvalResult, E as EvalIssue } from '../types-public-CkNWEov9.js';
 export { j as jsonSchemaTypeToTs } from '../json-schema-ts-CbB78b3n.js';
 import '../axes-CzLEMDeB.js';
 import '../types-BOvHNG7K.js';

package/dist/check/index.js

@@ -4,6 +4,7 @@ import path from 'path';
 import { createRequire } from 'module';
 import { Linter } from 'eslint';
 import { listContractGadgets, HOOK_NAME_RE } from '@ggui-ai/protocol';
+import { consumedTokenManifest } from '@ggui-ai/design/themes';
 
 // src/check/extract-wire-calls.ts
 var HOOK_TO_KIND = {
@@ -1014,6 +1015,8 @@ function detectCertainDoubleWiredActions(sourceCode, actionBindings) {
   visit(sf);
   return issues;
 }
+var MANIFEST_TOKENS = new Set(consumedTokenManifest);
+var TOKEN_REF_RE = /var\(\s*(--ggui-[a-zA-Z0-9-]+)/g;
 var HOOK_NAME_FOR = {
   action: "useAction",
   stream: "useStream",
@@ -1129,23 +1132,25 @@ ${errorDetail}`,
         });
       }
     }
+    const literalFallbackMatch = line.match(
+      /var\(\s*(--ggui-[a-zA-Z0-9-]+)\s*,\s*([^)]+)\)/
+    );
+    const literalFallback = literalFallbackMatch !== null && !literalFallbackMatch[2].trim().startsWith("var(") ? { token: literalFallbackMatch[1], literal: literalFallbackMatch[2].trim() } : null;
+    if (literalFallback !== null && !assetEscapedLines.has(lineNum)) {
+      issues.push({
+        tier: 0,
+        result: "fail",
+        category: "tokens",
+        subcategory: "token-fallback",
+        severity: "critical",
+        description: `Literal fallback "${literalFallback.literal}" inside a token reference \u2014 the runtime injects every token, and the fallback paints the wrong value exactly when the theme matters most.`,
+        fix: `Drop the fallback: write var(${literalFallback.token}) bare. If the token might not exist, that is a vocabulary problem to fix by choosing a token from the design-system docs, never by a literal.`,
+        line: lineNum
+      });
+    }
     const hexMatch = line.match(/#[0-9a-fA-F]{3,8}\b/);
     if (hexMatch && !assetEscapedLines.has(lineNum)) {
-      const fallbackVarMatch = line.match(
-        /var\((--ggui-[a-zA-Z0-9-]+)\s*,[^)]*#[0-9a-fA-F]{3,8}/
-      );
-      if (fallbackVarMatch) {
-        issues.push({
-          tier: 0,
-          result: "fail",
-          category: "tokens",
-          subcategory: "token-fallback",
-          severity: "critical",
-          description: `Literal fallback "${hexMatch[0]}" inside a token reference \u2014 the runtime injects every token, and the fallback paints the wrong brand exactly when the theme matters most.`,
-          fix: `Drop the fallback: write var(${fallbackVarMatch[1]}) bare. If the token might not exist, that is a vocabulary problem to fix by choosing a token from the design-system docs, never by a literal.`,
-          line: lineNum
-        });
-      } else if (!line.includes("// fallback")) {
+      if (literalFallback !== null && literalFallback.literal.includes(hexMatch[0])) ; else if (!line.includes("// fallback")) {
         issues.push({
           tier: 0,
           result: "fail",
@@ -1171,6 +1176,23 @@ ${errorDetail}`,
         line: lineNum
       });
     }
+    const offManifest = /* @__PURE__ */ new Set();
+    for (const m of line.matchAll(TOKEN_REF_RE)) {
+      const name = m[1];
+      if (!MANIFEST_TOKENS.has(name)) offManifest.add(name);
+    }
+    for (const name of offManifest) {
+      issues.push({
+        tier: 0,
+        result: "fail",
+        category: "tokens",
+        subcategory: "off-manifest-token",
+        severity: "critical",
+        description: `Token "${name}" is not in the design system's token manifest \u2014 no theme defines it, so it renders unset.`,
+        fix: `Choose a token that exists: the closed vocabulary is the manifest the design-system docs list (colors: var(--ggui-color-primary-500), var(--ggui-color-onSurface), \u2026; spacing: var(--ggui-spacing-4); type: var(--ggui-font-size-sm)). Never invent a name and never add a literal fallback.`,
+        line: lineNum
+      });
+    }
     const rawSpacingMatch = line.match(
       /\b(gap|padding|paddingX|paddingY|margin|radius)\s*=\s*["'][\d.]+(?:px|rem|em)["']/
     );
@@ -1332,6 +1354,24 @@ ${errorDetail}`,
       ...detectCertainDoubleWiredActions(sourceCode, actionBindings)
     );
   }
+  if (/\buseRender\s*\(/.test(sourceCode)) {
+    const suppressionRe = /(?<![\w-])disabled\s*=\s*\{([^}]*)\}/gs;
+    for (const m of sourceCode.matchAll(suppressionRe)) {
+      const expr = m[1];
+      if (!/\bisConnected\b/.test(expr)) continue;
+      const attrLine = sourceCode.slice(0, m.index ?? 0).split("\n").length;
+      issues.push({
+        tier: 0,
+        result: "fail",
+        category: "interactivity",
+        subcategory: "connection-suppression",
+        severity: "critical",
+        description: `\`disabled\` is bound to the connection state (\`${expr.trim().replace(/\s+/g, " ")}\`) \u2014 that strips the click handler, and the click is how the runtime detects that the host can relay actions again. The control stays dead after recovery.`,
+        fix: "Leave the control enabled \u2014 never bind `disabled` to the connection state. The runtime presents the dead zone itself (a standing notice, then a per-gesture cue) and the attempt is how it detects recovery; a suppressed control stays dead after the host recovers.",
+        line: attrLine
+      });
+    }
+  }
   const propsInterfaceMatch = sourceCode.match(/interface Props\s*\{([^}]+)\}/);
   if (propsInterfaceMatch) {
     const propsBody = propsInterfaceMatch[1];