@ggui-ai/ui-visual-tester 0.17.0 → 0.18.0

package/dist/fixture-runtime.js

@@ -82,6 +82,12 @@ const CLICKABLE_SELECTOR = 'button, [role="button"], input[type="submit"], input
 // parent <details> — a DOM change that says nothing about the action).
 /** Per-control wait in the fallback pass — enough for a synchronous dispatch + a render. */
 const FALLBACK_CLICK_WAIT_MS = 600;
+/**
+ * #1040: how many times ONE control whose click revealed new controls (a wizard's Next, a tab) may be
+ * clicked again while the action's own control is still not visible. Bounds a walk through a
+ * multi-step form; a toggle that keeps revealing and hiding a panel cannot spin the probe.
+ */
+const MAX_NAVIGATOR_RECLICKS = 8;
 /**
  * ONE deadline covers both passes for an action: max(the caller's wait, this
  * floor). A few loosely-matching decoys can no longer run an action for
@@ -99,9 +105,13 @@ function norm(text) {
  * DOM order. A generated component that wires `useAction('toggleItem')` to an
  * unnamed checkbox is found by pass 2 and judged by what its click dispatches.
  */
+/** The one disabled predicate: the collector's and the walk's reveal signal must not drift apart. */
+function isControlDisabled(el) {
+    return el.hasAttribute('disabled') || el.getAttribute('aria-disabled') === 'true';
+}
 function collectActionCandidates(root, name, label) {
     const every = Array.from(root.querySelectorAll(CLICKABLE_SELECTOR)).filter((el) => el instanceof HTMLElement);
-    const isDisabled = (el) => el.hasAttribute('disabled') || el.getAttribute('aria-disabled') === 'true';
+    const isDisabled = isControlDisabled;
     const lcName = name.toLowerCase();
     const lcLabel = label.toLowerCase();
     const nameKey = norm(name);
@@ -140,7 +150,12 @@ function collectActionCandidates(root, name, label) {
     }
     return { named: [...byValue, ...byText], fallback: rest, disabled, namedDisabled, allClickables: every };
 }
-const TEXT_LIKE_INPUT_TYPES = new Set(['text', 'email', 'search', 'url', 'tel', 'number', 'password', '']);
+// #1040: date-like types too — a wizard's first step is routinely gated on one ("preferred follow-up
+// date"), and a Next that never enables hides every step behind it from the probe.
+const TEXT_LIKE_INPUT_TYPES = new Set([
+    'text', 'email', 'search', 'url', 'tel', 'number', 'password', '',
+    'date', 'time', 'month', 'week', 'datetime-local',
+]);
 function sampleValueFor(el) {
     const type = el instanceof HTMLInputElement ? el.type : 'textarea';
     switch (type) {
@@ -152,6 +167,16 @@ function sampleValueFor(el) {
             return 'https://example.com';
         case 'tel':
             return '5550100';
+        case 'date':
+            return '2026-01-15';
+        case 'time':
+            return '09:30';
+        case 'month':
+            return '2026-01';
+        case 'week':
+            return '2026-W03';
+        case 'datetime-local':
+            return '2026-01-15T09:30';
         default:
             return 'probe';
     }
@@ -189,7 +214,8 @@ function primeInputs(root) {
         if (el.value.trim().length > 0)
             continue;
         setNativeValue(el, sampleValueFor(el));
-        primed += 1;
+        if (el.value.trim().length > 0)
+            primed += 1;
     }
     for (const el of Array.from(root.querySelectorAll('select'))) {
         if (!(el instanceof HTMLSelectElement) || el.disabled)
@@ -422,7 +448,8 @@ async function run(input) {
             return {
                 status: 'action-not-rendered',
                 diagnostic: `no clickable control in the rendered DOM for actionSpec.${input.actionName} (label "${entry.label}")` +
-                    (first.disabled > 0 ? ` — ${first.disabled} control(s) rendered disabled, none naming the action` : ''),
+                    (first.disabled > 0 ? ` — ${first.disabled} control(s) rendered disabled, none naming the action` : '') +
+                    ` (primed ${primed} input(s))`,
             };
         }
         // One deadline for the whole action. Named controls get the caller's wait
@@ -459,27 +486,71 @@ async function run(input) {
         let namedDisabledSeen = first.namedDisabled;
         let navigated = false;
         let knownSignatures = new Set(first.allClickables.map((el) => signatureOf(container, el)));
+        // #1040: the walk. A wizard's Next is ONE element at ONE DOM path on every step, so "tried once by
+        // path" stopped the probe at step 1 forever, and the terminal control never mounted. A control
+        // whose click revealed controls the probe had not seen is a NAVIGATOR: while the action's own
+        // control is not visible, the probe clicks it again (bounded) before exploring further — the way
+        // a user presses Next until the last step. Inputs a step reveals are primed like the first step's.
+        const navigators = new Map();
+        let lastClickedSignature;
+        // The action's OWN control is keyed by path AND text: a wizard renders Complete at the path Next
+        // occupied on every earlier step, and "tried by path" would read it as the Next already clicked.
+        const namedKey = (el) => `${signatureOf(container, el)}|${(el.textContent ?? '').trim()}`;
+        const triedNamed = new Set();
+        // A step is "revealed" when a control the probe had not seen appears OR a control it had only
+        // seen disabled becomes enabled (a wizard's Back on step 2, a Next a step's own field unlocked).
+        const stateOf = (el) => `${signatureOf(container, el)}|${isControlDisabled(el) ? 'off' : 'on'}`;
+        let knownStates = new Set(first.allClickables.map(stateOf));
         for (;;) {
             if (remaining() === 0) {
                 exhausted = true;
                 break;
             }
-            const live = collectActionCandidates(container, input.actionName, entry.label);
-            namedDisabledSeen = Math.max(namedDisabledSeen, live.namedDisabled);
+            let live = collectActionCandidates(container, input.actionName, entry.label);
             // Disabled controls count: a step that reveals a disabled Complete IS navigation into the gate.
             const liveSignatures = live.allClickables.map((el) => signatureOf(container, el));
+            const liveStates = live.allClickables.map(stateOf);
+            // `navigated` (the reason on an unreachable exit) keeps its meaning: controls the probe had not
+            // seen appeared. `revealed` (the walk) is wider: that, or a control seen only disabled enabling.
             if (clicked > 0 && liveSignatures.some((sig) => !knownSignatures.has(sig)))
                 navigated = true;
+            const revealed = clicked > 0 && liveStates.some((state) => !knownStates.has(state));
             knownSignatures = new Set([...knownSignatures, ...liveSignatures]);
+            knownStates = new Set([...knownStates, ...liveStates]);
+            if (revealed) {
+                if (lastClickedSignature !== undefined && !navigators.has(lastClickedSignature))
+                    navigators.set(lastClickedSignature, 0);
+                // The revealed step may carry its own gate ("name", "date") — give it what the first step got.
+                if (primeInputs(container) > 0) {
+                    await delay(150);
+                    live = collectActionCandidates(container, input.actionName, entry.label);
+                }
+            }
+            namedDisabledSeen = Math.max(namedDisabledSeen, live.namedDisabled);
             const untried = (list) => list.find((el) => el.isConnected && !tried.has(signatureOf(container, el)));
-            const nextNamed = untried(live.named);
-            const next = nextNamed ?? untried(live.fallback);
+            const nextNamed = live.named.find((el) => el.isConnected && !triedNamed.has(namedKey(el)));
+            const navigator = nextNamed === undefined
+                ? live.fallback.find((el) => {
+                    const sig = signatureOf(container, el);
+                    return el.isConnected && (navigators.get(sig) ?? MAX_NAVIGATOR_RECLICKS) < MAX_NAVIGATOR_RECLICKS;
+                })
+                : undefined;
+            // Untried controls first — including the ones the last click revealed — and only when nothing
+            // untried remains, the navigator again: a wizard walks (one extra Back click); an "Add item"
+            // or a tab is not clicked eight more times before what it revealed gets its turn.
+            const next = nextNamed ?? untried(live.fallback) ?? navigator;
             if (next === undefined)
                 break;
             const via = nextNamed !== undefined ? 'named' : 'fallback';
             if (via === 'named')
                 namedAttempted = true;
-            tried.add(signatureOf(container, next));
+            const nextSignature = signatureOf(container, next);
+            if (navigator !== undefined && next === navigator)
+                navigators.set(nextSignature, (navigators.get(nextSignature) ?? 0) + 1);
+            if (via === 'named')
+                triedNamed.add(namedKey(next));
+            tried.add(nextSignature);
+            lastClickedSignature = nextSignature;
             const hit = await attempt(next, via === 'named' ? Math.min(input.waitMs, remaining()) : Math.min(FALLBACK_CLICK_WAIT_MS, remaining()), via);
             if (hit !== null)
                 return hit;
@@ -490,6 +561,7 @@ async function run(input) {
         if (req.requireDom && !bestDom)
             missing.push('DOM change');
         const others = otherActions.size > 0 ? `; other actions dispatched: ${[...otherActions].join(', ')}` : '';
+        const navigatorReclicks = [...navigators.values()].reduce((sum, n) => sum + n, 0);
         // #1040: the named control was seen only disabled and never clicked enabled — the probe could not
         // reach the action. Not measured, with the reason: a click made new controls appear (the gate sits
         // behind navigation the probe walked into but could not finish) or none did (the gate is input the
@@ -500,14 +572,14 @@ async function run(input) {
                 status: 'action-unreachable',
                 reason,
                 diagnostic: `actionSpec.${input.actionName} (label "${entry.label}") rendered disabled on every observation (${namedDisabledSeen} matching control(s)) and was never clicked enabled; ` +
-                    `clicked ${clicked} other control(s) (${tried.size} distinct; primed ${primed} input(s)); new controls appeared after a click=${navigated}; DOM changed=${bestDom}` +
+                    `clicked ${clicked} other control(s) (${tried.size} distinct; primed ${primed} input(s); navigator re-clicks=${navigatorReclicks}); new controls appeared after a click=${navigated}; DOM changed=${bestDom}` +
                     `${exhausted ? '; budget exhausted' : ''}${others}`,
             };
         }
         return {
             status: 'action-no-effect',
             diagnostic: `${req.description}; clicked ${clicked} control(s) (${tried.size} distinct; first collection ${first.named.length} named, ${first.fallback.length} fallback, ${first.disabled} disabled; primed ${primed} input(s)` +
-                `${exhausted ? '; budget exhausted' : ''}); best observed dispatch=${bestDispatch}, domChanged=${bestDom}; ` +
+                `; navigator re-clicks=${navigatorReclicks}${exhausted ? '; budget exhausted' : ''}); best observed dispatch=${bestDispatch}, domChanged=${bestDom}; ` +
                 `missing=${missing.join('+')}${others}`,
         };
     }

package/fixture/fixture-runtime.js

@@ -32820,15 +32820,19 @@
   }
   var CLICKABLE_SELECTOR = 'button, [role="button"], input[type="submit"], input[type="button"], input[type="checkbox"], input[type="radio"], [role="switch"], [role="checkbox"], [role="menuitem"], [role="tab"], [role="option"], [data-action]';
   var FALLBACK_CLICK_WAIT_MS = 600;
+  var MAX_NAVIGATOR_RECLICKS = 8;
   var ACTION_BUDGET_FLOOR_MS = 6e3;
   function norm(text) {
     return text.toLowerCase().replace(/[^a-z0-9]/g, "");
   }
+  function isControlDisabled(el) {
+    return el.hasAttribute("disabled") || el.getAttribute("aria-disabled") === "true";
+  }
   function collectActionCandidates(root, name, label) {
     const every = Array.from(root.querySelectorAll(CLICKABLE_SELECTOR)).filter(
       (el) => el instanceof HTMLElement
     );
-    const isDisabled = (el) => el.hasAttribute("disabled") || el.getAttribute("aria-disabled") === "true";
+    const isDisabled = isControlDisabled;
     const lcName = name.toLowerCase();
     const lcLabel = label.toLowerCase();
     const nameKey = norm(name);
@@ -32860,7 +32864,21 @@
     }
     return { named: [...byValue, ...byText], fallback: rest, disabled, namedDisabled, allClickables: every };
   }
-  var TEXT_LIKE_INPUT_TYPES = /* @__PURE__ */ new Set(["text", "email", "search", "url", "tel", "number", "password", ""]);
+  var TEXT_LIKE_INPUT_TYPES = /* @__PURE__ */ new Set([
+    "text",
+    "email",
+    "search",
+    "url",
+    "tel",
+    "number",
+    "password",
+    "",
+    "date",
+    "time",
+    "month",
+    "week",
+    "datetime-local"
+  ]);
   function sampleValueFor(el) {
     const type = el instanceof HTMLInputElement ? el.type : "textarea";
     switch (type) {
@@ -32872,6 +32890,16 @@
         return "https://example.com";
       case "tel":
         return "5550100";
+      case "date":
+        return "2026-01-15";
+      case "time":
+        return "09:30";
+      case "month":
+        return "2026-01";
+      case "week":
+        return "2026-W03";
+      case "datetime-local":
+        return "2026-01-15T09:30";
       default:
         return "probe";
     }
@@ -32892,7 +32920,7 @@
       if (el instanceof HTMLInputElement && !TEXT_LIKE_INPUT_TYPES.has(el.type)) continue;
       if (el.value.trim().length > 0) continue;
       setNativeValue(el, sampleValueFor(el));
-      primed += 1;
+      if (el.value.trim().length > 0) primed += 1;
     }
     for (const el of Array.from(root.querySelectorAll("select"))) {
       if (!(el instanceof HTMLSelectElement) || el.disabled) continue;
@@ -33084,7 +33112,7 @@
         }
         return {
           status: "action-not-rendered",
-          diagnostic: `no clickable control in the rendered DOM for actionSpec.${input.actionName} (label "${entry.label}")` + (first.disabled > 0 ? ` \u2014 ${first.disabled} control(s) rendered disabled, none naming the action` : "")
+          diagnostic: `no clickable control in the rendered DOM for actionSpec.${input.actionName} (label "${entry.label}")` + (first.disabled > 0 ? ` \u2014 ${first.disabled} control(s) rendered disabled, none naming the action` : "") + ` (primed ${primed} input(s))`
         };
       }
       const deadline = Date.now() + Math.max(input.waitMs, ACTION_BUDGET_FLOOR_MS);
@@ -33109,23 +33137,47 @@
       let namedDisabledSeen = first.namedDisabled;
       let navigated = false;
       let knownSignatures = new Set(first.allClickables.map((el) => signatureOf(container, el)));
+      const navigators = /* @__PURE__ */ new Map();
+      let lastClickedSignature;
+      const namedKey = (el) => `${signatureOf(container, el)}|${(el.textContent ?? "").trim()}`;
+      const triedNamed = /* @__PURE__ */ new Set();
+      const stateOf = (el) => `${signatureOf(container, el)}|${isControlDisabled(el) ? "off" : "on"}`;
+      let knownStates = new Set(first.allClickables.map(stateOf));
       for (; ; ) {
         if (remaining() === 0) {
           exhausted = true;
           break;
         }
-        const live = collectActionCandidates(container, input.actionName, entry.label);
-        namedDisabledSeen = Math.max(namedDisabledSeen, live.namedDisabled);
+        let live = collectActionCandidates(container, input.actionName, entry.label);
         const liveSignatures = live.allClickables.map((el) => signatureOf(container, el));
+        const liveStates = live.allClickables.map(stateOf);
         if (clicked > 0 && liveSignatures.some((sig) => !knownSignatures.has(sig))) navigated = true;
+        const revealed = clicked > 0 && liveStates.some((state) => !knownStates.has(state));
         knownSignatures = /* @__PURE__ */ new Set([...knownSignatures, ...liveSignatures]);
+        knownStates = /* @__PURE__ */ new Set([...knownStates, ...liveStates]);
+        if (revealed) {
+          if (lastClickedSignature !== void 0 && !navigators.has(lastClickedSignature)) navigators.set(lastClickedSignature, 0);
+          if (primeInputs(container) > 0) {
+            await delay(150);
+            live = collectActionCandidates(container, input.actionName, entry.label);
+          }
+        }
+        namedDisabledSeen = Math.max(namedDisabledSeen, live.namedDisabled);
         const untried = (list) => list.find((el) => el.isConnected && !tried.has(signatureOf(container, el)));
-        const nextNamed = untried(live.named);
-        const next = nextNamed ?? untried(live.fallback);
+        const nextNamed = live.named.find((el) => el.isConnected && !triedNamed.has(namedKey(el)));
+        const navigator2 = nextNamed === void 0 ? live.fallback.find((el) => {
+          const sig = signatureOf(container, el);
+          return el.isConnected && (navigators.get(sig) ?? MAX_NAVIGATOR_RECLICKS) < MAX_NAVIGATOR_RECLICKS;
+        }) : void 0;
+        const next = nextNamed ?? untried(live.fallback) ?? navigator2;
         if (next === void 0) break;
         const via = nextNamed !== void 0 ? "named" : "fallback";
         if (via === "named") namedAttempted = true;
-        tried.add(signatureOf(container, next));
+        const nextSignature = signatureOf(container, next);
+        if (navigator2 !== void 0 && next === navigator2) navigators.set(nextSignature, (navigators.get(nextSignature) ?? 0) + 1);
+        if (via === "named") triedNamed.add(namedKey(next));
+        tried.add(nextSignature);
+        lastClickedSignature = nextSignature;
         const hit = await attempt(
           next,
           via === "named" ? Math.min(input.waitMs, remaining()) : Math.min(FALLBACK_CLICK_WAIT_MS, remaining()),
@@ -33137,17 +33189,18 @@
       if (req.requireDispatch && !bestDispatch) missing.push("dispatch");
       if (req.requireDom && !bestDom) missing.push("DOM change");
       const others = otherActions.size > 0 ? `; other actions dispatched: ${[...otherActions].join(", ")}` : "";
+      const navigatorReclicks = [...navigators.values()].reduce((sum, n) => sum + n, 0);
       if (!namedAttempted && namedDisabledSeen > 0) {
         const reason = navigated ? "behind-navigation" : "disabled-after-priming";
         return {
           status: "action-unreachable",
           reason,
-          diagnostic: `actionSpec.${input.actionName} (label "${entry.label}") rendered disabled on every observation (${namedDisabledSeen} matching control(s)) and was never clicked enabled; clicked ${clicked} other control(s) (${tried.size} distinct; primed ${primed} input(s)); new controls appeared after a click=${navigated}; DOM changed=${bestDom}${exhausted ? "; budget exhausted" : ""}${others}`
+          diagnostic: `actionSpec.${input.actionName} (label "${entry.label}") rendered disabled on every observation (${namedDisabledSeen} matching control(s)) and was never clicked enabled; clicked ${clicked} other control(s) (${tried.size} distinct; primed ${primed} input(s); navigator re-clicks=${navigatorReclicks}); new controls appeared after a click=${navigated}; DOM changed=${bestDom}${exhausted ? "; budget exhausted" : ""}${others}`
         };
       }
       return {
         status: "action-no-effect",
-        diagnostic: `${req.description}; clicked ${clicked} control(s) (${tried.size} distinct; first collection ${first.named.length} named, ${first.fallback.length} fallback, ${first.disabled} disabled; primed ${primed} input(s)${exhausted ? "; budget exhausted" : ""}); best observed dispatch=${bestDispatch}, domChanged=${bestDom}; missing=${missing.join("+")}${others}`
+        diagnostic: `${req.description}; clicked ${clicked} control(s) (${tried.size} distinct; first collection ${first.named.length} named, ${first.fallback.length} fallback, ${first.disabled} disabled; primed ${primed} input(s); navigator re-clicks=${navigatorReclicks}${exhausted ? "; budget exhausted" : ""}); best observed dispatch=${bestDispatch}, domChanged=${bestDom}; missing=${missing.join("+")}${others}`
       };
     } finally {
       window.removeEventListener("error", onWindowError);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ggui-ai/ui-visual-tester",
-  "version": "0.17.0",
+  "version": "0.18.0",
   "description": "Behavioral validator for ggui-generated component code. Renders a component in real Chromium via Playwright, clicks each action declared in its contract, and verifies the action actually does something — mutating bound state or dispatching to the agent. Catches buttons wired to no-op handlers that structural validators miss. Playwright is an optional peer dependency, injected by the caller.",
   "keywords": [
     "ggui",
@@ -48,9 +48,9 @@
   "dependencies": {
     "react": "^19.0.0",
     "react-dom": "^19.0.0",
-    "@ggui-ai/design": "0.17.0",
-    "@ggui-ai/protocol": "0.17.0",
-    "@ggui-ai/wire": "0.17.0"
+    "@ggui-ai/design": "0.18.0",
+    "@ggui-ai/wire": "0.18.0",
+    "@ggui-ai/protocol": "0.18.0"
   },
   "devDependencies": {
     "@playwright/test": "^1.60.0",