@ggui-ai/ui-gen 0.23.0 → 0.25.0

package/dist/classifier/index.d.ts

@@ -1,5 +1,5 @@
-import { S as StreamEventKind, C as Classification, A as AxisVector, R as RiskTier, F as FetchShape, a as AxisSource, L as LayoutShape, b as RealtimeShape, c as RenderShape, d as StateShape, T as ToolingShape, W as WriteTrigger, e as WriteShape } from '../axes-CzLEMDeB.js';
-export { f as AxisProvenance } from '../axes-CzLEMDeB.js';
+import { b as StreamEventKind, C as Classification, A as AxisVector, R as RiskTier, F as FetchShape, c as AxisSource, L as LayoutShape, d as RealtimeShape, a as RenderShape, S as StateShape, T as ToolingShape, e as WriteTrigger, W as WriteShape } from '../axes-CKVCzR_x.js';
+export { f as AxisProvenance } from '../axes-CKVCzR_x.js';
 import * as _ggui_ai_protocol from '@ggui-ai/protocol';
 import { ClientCapabilitiesSpec } from '@ggui-ai/protocol';
 

package/dist/compose.d.ts

@@ -1,5 +1,5 @@
-import { C as Classification } from './axes-CzLEMDeB.js';
-import { C as ComposedHarness } from './types-BOvHNG7K.js';
+import { C as Classification } from './axes-CKVCzR_x.js';
+import { C as ComposedHarness } from './types-Dh1s4K9j.js';
 
 declare function compose(classification: Classification): ComposedHarness;
 

package/dist/{create-ui-generator-D0kbWwln.d.ts → create-ui-generator-DFnnArrv.d.ts}

@@ -2,11 +2,13 @@ import { SingleComponentParams } from './harness/runtime.js';
 import { ModelRef } from '@ggui-ai/protocol';
 import { GadgetCatalogAdapter } from '@ggui-ai/gadgets';
 import { GeneratorTier, UiGenerator } from '@ggui-ai/mcp-server-core';
+import { VisualEvalAgentSpec } from './adapters/generation-dispatch.js';
+import { R as RuntimeRenderProbeConfig } from './adapter-Cg_rkpGq.js';
 import { GenerationResult } from './harness/result-types.js';
 import { D as DesignMode } from './design-mode-DLufhrNp.js';
-import { Q as QualityConfig } from './types-public-Ow-tNtFE.js';
+import { Q as QualityConfig } from './types-public-BJTLpzWn.js';
 import { EvaluationConfig } from './evaluation/types.js';
-import { A as AgentConfig } from './llm-router-DwwUK1bz.js';
+import { A as AgentConfig } from './llm-router-nfO7wPec.js';
 
 /**
  * `createUiGenerator` — full-harness UI generation factory.
@@ -44,16 +46,39 @@ import { A as AgentConfig } from './llm-router-DwwUK1bz.js';
 
 interface CreateUiGeneratorOptions {
     /**
-     * Wire `DEFAULT_RUNTIME_RENDER_CHECK` into the harness's check leg.
-     * When `true`, every coding turn ends with a happy-dom runtime-render
-     * probe that catches contract-wiring bugs (missing `useAction()`,
-     * wrong stream channel name, etc.).
+     * Wire the runtime-render check into the harness's check leg — one
+     * instance per generator, built from `runtimeRenderProbe` (the default
+     * bounds when it is absent). When `true`, the runtime-render probe runs
+     * ONCE at the exit decision,
+     * after the coding turns, in an isolated subprocess: it mounts the
+     * compiled component against the contract and catches crash-class and
+     * contract-wiring bugs (a render that throws, a missing `useAction()`,
+     * a wrong stream channel name). It is never on the per-turn hot path.
      *
-     * Default: `false` for OSS — keeps cold-start light. The probe pulls
-     * happy-dom + @testing-library on first use (~700-1500ms cold). Bench
-     * enables it via dispatch's own default for stricter quality gating.
+     * Without an `evaluation` config the round is observe + one repair turn
+     * (ggui#1380): a recoverable render crash buys exactly one repair turn
+     * and one re-probe, then the generation serves what it has; a probe that
+     * timed out, could not run, or had nothing to probe never blocks and
+     * never repairs. The probe's status and timing ride the result's
+     * metadata (`runtimeProbe`, `evalMs`). With an `evaluation` config the
+     * probe is the evaluation round's exit gate instead.
+     *
+     * Default: `false` — keeps cold-start light. The probe pulls happy-dom
+     * + @testing-library on first use (~700-1500ms cold). A serving
+     * deployment that wants the probe's verdict on every card sets it.
      */
     readonly enableRuntimeRender?: boolean;
+    /**
+     * How the runtime-render probe runs its isolated worker (ggui#1380). A
+     * serving deployment sets the probe's wall-clock bound, worker heap and
+     * concurrency here; the defaults are the evaluation lane's (30 s, 512 MB,
+     * no concurrency cap). The generator builds ONE check from it and hands
+     * that instance to every generation it dispatches — the concurrency cap
+     * lives on the instance, so it bounds the workers this generator has
+     * live at once. Only read when `enableRuntimeRender` is on. Never part
+     * of the prompt or the identity.
+     */
+    readonly runtimeRenderProbe?: RuntimeRenderProbeConfig;
     /** Maximum coding attempts per generation pass. */
     readonly maxAttempts?: number;
     /** Maximum evaluation rounds. */
@@ -71,7 +96,14 @@ interface CreateUiGeneratorOptions {
      * stylesheet to render under). Forwarded verbatim; never part of the
      * prompt or the identity. Default: none — the round runs as today.
      */
-    readonly visualEvaluation?: SingleComponentParams['visualEvaluation'];
+    readonly visualEvaluation?: SingleComponentParams["visualEvaluation"];
+    /**
+     * ggui#1250 — the in-loop VISUAL judge as its own agent (a vision provider, a model, and that
+     * provider's key when the caller resolves keys itself). Absent (default): the judge is the
+     * evaluation agent's provider, which on a lane without a vision path records the score half as
+     * skipped (#1248). Forwarded verbatim to the dispatch; never part of the prompt or the identity.
+     */
+    readonly visualEvalAgent?: VisualEvalAgentSpec;
     /** Quality config controlling eval tiers + improvement behavior. */
     readonly qualityConfig?: QualityConfig;
     /**
@@ -152,7 +184,7 @@ interface CreateUiGeneratorOptions {
      * request. Absent (default) is a no-op — retries still happen and
      * still log via `console.warn`, just without a caller-side hook.
      */
-    readonly onRetry?: AgentConfig['onRetry'];
+    readonly onRetry?: AgentConfig["onRetry"];
     /**
      * Generation observer. Invoked once per successful `generate()` with
      * the FULL harness result — source, compiled code, token counters,

package/dist/{dispatch-wxaIMwZt.d.ts → dispatch-CStl9eAw.d.ts}

@@ -1,6 +1,6 @@
-import { DataContract } from '@ggui-ai/protocol';
-import { C as Classification } from './axes-CzLEMDeB.js';
-import { E as EvalIssue } from './types-public-Ow-tNtFE.js';
+import { DataContract, JsonObject } from '@ggui-ai/protocol';
+import { C as Classification } from './axes-CKVCzR_x.js';
+import { E as EvalIssue } from './types-public-BJTLpzWn.js';
 import { D as DesignMode, C as CanvasClass } from './design-mode-DLufhrNp.js';
 
 interface RunAxisChecksInput {
@@ -12,6 +12,8 @@ interface RunAxisChecksInput {
     designMode?: DesignMode;
     /** The canvas the source is judged for — see `AxisCheckInput.canvas` (ggui#1117); absent = no rendering context. */
     canvas?: CanvasClass;
+    /** The props the source was rendered with — see `AxisCheckInput.fixtureProps` (ggui#1285); absent = the echo check stands down. */
+    fixtureProps?: JsonObject;
 }
 declare function runAxisChecks(classification: Classification, input: RunAxisChecksInput): EvalIssue[];
 

package/dist/evaluation/axis-checks/index.d.ts

@@ -1,11 +1,12 @@
-export { R as RunAxisChecksInput, r as runAxisChecks } from '../../dispatch-wxaIMwZt.js';
+export { R as RunAxisChecksInput, r as runAxisChecks } from '../../dispatch-CStl9eAw.js';
 export { REGISTRY } from './registry.js';
-import { A as AxisCheck } from '../../types-public-Ow-tNtFE.js';
-export { a as AxisCheckInput, m as matches } from '../../types-public-Ow-tNtFE.js';
+import { A as AxisCheck } from '../../types-public-BJTLpzWn.js';
+export { a as AxisCheckInput, m as matches } from '../../types-public-BJTLpzWn.js';
 import '@ggui-ai/protocol';
-import '../../axes-CzLEMDeB.js';
+import '../../axes-CKVCzR_x.js';
 import '../../design-mode-DLufhrNp.js';
-import '../../types-BOvHNG7K.js';
+import '@ggui-ai/mcp-server-core';
+import '../../types-Dh1s4K9j.js';
 
 declare const EXTRA_CHECKS: readonly AxisCheck[];
 

package/dist/evaluation/axis-checks/index.js

@@ -569,6 +569,81 @@ function runActionLabelDropped(input) {
     )
   );
 }
+var MIN_ECHO_CHARS = 12;
+function collectStrings(value, out) {
+  if (typeof value === "string") {
+    out.push(value);
+  } else if (Array.isArray(value)) {
+    for (const v of value) collectStrings(v, out);
+  } else if (value !== null && typeof value === "object") {
+    for (const v of Object.values(value)) collectStrings(v, out);
+  }
+}
+function fixtureOnlyStrings(input) {
+  if (input.fixtureProps === void 0) return [];
+  const values = [];
+  collectStrings(input.fixtureProps, values);
+  const stated = `${input.originalPrompt}
+${input.contract === void 0 ? "" : JSON.stringify(input.contract)}`;
+  const seen = /* @__PURE__ */ new Set();
+  const out = [];
+  for (const v of values) {
+    const t = v.trim();
+    if (t.length < MIN_ECHO_CHARS || !t.includes(" ") || seen.has(t)) continue;
+    seen.add(t);
+    if (stated.includes(t)) continue;
+    out.push(t);
+  }
+  return out;
+}
+function runFixtureDefaultEcho(input) {
+  if (input.compiledCode === null) return [];
+  const src = input.sourceCode;
+  const issues = [];
+  for (const value of fixtureOnlyStrings(input)) {
+    if (!src.includes(value)) continue;
+    issues.push(
+      mkIssue(
+        "universal.fixture_default_echo",
+        `A literal in the source equals a fixture value the request never stated: "${value.length > 60 ? `${value.slice(0, 57)}\u2026` : value}". The fixture is judging data; a visitor sees this copy whenever the agent leaves the prop out.`,
+        "Take that copy from the prop it renders and drop the literal \u2014 no default, fallback or placeholder that repeats a fixture value. A default may only carry text the request itself states.",
+        "warn"
+      )
+    );
+  }
+  return issues;
+}
+var FORM_OPEN_RX = /<form\b[^<>]*\bonSubmit\s*=[^<>]*>/g;
+var CONTROL_TAG_RX = /<(?:Button|button|input)\b[^<>]*>/g;
+var SUBMIT_TYPE_RX = /\btype\s*=\s*(?:["']submit["']|\{\s*["']submit["']\s*\})/;
+function submitForms(sourceCode) {
+  const out = [];
+  for (const m of sourceCode.matchAll(FORM_OPEN_RX)) {
+    const start = m.index ?? 0;
+    const end = sourceCode.indexOf("</form>", start + m[0].length);
+    out.push(end === -1 ? sourceCode.slice(start) : sourceCode.slice(start, end + "</form>".length));
+  }
+  return out;
+}
+function runFormSubmitControl(input) {
+  if (input.compiledCode === null) return [];
+  const issues = [];
+  for (const form of submitForms(input.sourceCode)) {
+    const controls = [...form.matchAll(CONTROL_TAG_RX)].map((m) => m[0]);
+    if (controls.length === 0) continue;
+    if (controls.some((tag) => SUBMIT_TYPE_RX.test(tag))) continue;
+    const named = controls.find((tag) => /<Button\b/.test(tag)) ?? controls[0];
+    issues.push(
+      mkIssue(
+        "universal.form_submit_control",
+        `A <form onSubmit> renders ${controls.length} control(s) and none submits it: the design Button defaults to type="button", so clicking ${named.length > 60 ? `${named.slice(0, 57)}\u2026` : named} does nothing \u2014 only Enter in a field submits.`,
+        'Give the control that submits this form type="submit" \u2014 a Button inside a form defaults to type="button" and never submits on click; keep type="button" on the others.',
+        "fail"
+      )
+    );
+  }
+  return issues;
+}
 var UNIVERSAL_CHECKS = [
   {
     id: "universal.icon_name_known",
@@ -606,6 +681,18 @@ var UNIVERSAL_CHECKS = [
     values: ALL_RENDER_VALUES,
     run: runTerminalActionUnguarded
   },
+  {
+    id: "universal.form_submit_control",
+    axis: "render",
+    values: ALL_RENDER_VALUES,
+    run: runFormSubmitControl
+  },
+  {
+    id: "universal.fixture_default_echo",
+    axis: "render",
+    values: ALL_RENDER_VALUES,
+    run: runFixtureDefaultEcho
+  },
   {
     id: "universal.accent_text_ink",
     axis: "render",
@@ -729,7 +816,7 @@ function runMapKeyIsId(input) {
       mkIssue(
         "render.map_key_is_id",
         `Array key uses index variable "${key}" \u2014 reorders and stream merges will break React reconciliation.`,
-        `Replace key={${key}} with key={item.id} (or item.symbol / whatever the entity id field is).`
+        `Replace key={${key}} with key={item.id} \u2014 or, when the items carry no id, with the field that names each one (item.label / item.symbol / item.name).`
       )
     );
   }
@@ -1676,7 +1763,8 @@ function runAxisChecks(classification, input) {
     originalPrompt: input.originalPrompt,
     classification,
     ...input.designMode !== void 0 ? { designMode: input.designMode } : {},
-    ...input.canvas !== void 0 ? { canvas: input.canvas } : {}
+    ...input.canvas !== void 0 ? { canvas: input.canvas } : {},
+    ...input.fixtureProps !== void 0 ? { fixtureProps: input.fixtureProps } : {}
   };
   if (input.compiledCode === null) {
     traceAxisChecksSkipped(facts, "compiledCode null \u2014 no check ran");