@enricai/barnacle 1.10.1 → 1.11.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/scraper/deep-locator-click.d.ts +5 -5
- package/dist/scraper/flow-runner.d.ts +92 -62
- package/dist/scraper/flow-runner.d.ts.map +1 -1
- package/dist/scraper/flow-runner.js +291 -157
- package/dist/scraper/flow-runner.js.map +1 -1
- package/dist/scraper/phantom-click.d.ts +12 -16
- package/dist/scraper/phantom-click.d.ts.map +1 -1
- package/dist/scraper/phantom-click.js +11 -14
- package/dist/scraper/phantom-click.js.map +1 -1
- package/dist/scripts/recon-generate.js +2 -2
- package/package.json +1 -1
|
@@ -23,6 +23,7 @@ exports.findRecentBackendError = findRecentBackendError;
|
|
|
23
23
|
exports.findRecentPageTransition = findRecentPageTransition;
|
|
24
24
|
exports.isWizardExitAction = isWizardExitAction;
|
|
25
25
|
exports.isAdvanceStep = isAdvanceStep;
|
|
26
|
+
exports.shouldCaptureSelectionState = shouldCaptureSelectionState;
|
|
26
27
|
exports.shouldWarnMissingAdvancePattern = shouldWarnMissingAdvancePattern;
|
|
27
28
|
exports.parseCaptureIndex = parseCaptureIndex;
|
|
28
29
|
exports.latestCaptureIndex = latestCaptureIndex;
|
|
@@ -32,9 +33,7 @@ exports.windowHasAdvanceTransition = windowHasAdvanceTransition;
|
|
|
32
33
|
exports.shouldVetoFallbackAdvance = shouldVetoFallbackAdvance;
|
|
33
34
|
exports.isDomOnlyAdvanceVerified = isDomOnlyAdvanceVerified;
|
|
34
35
|
exports.isClickViewSwapVerified = isClickViewSwapVerified;
|
|
35
|
-
exports.isClickStateToggleVerified = isClickStateToggleVerified;
|
|
36
36
|
exports.selectionCountFromSignature = selectionCountFromSignature;
|
|
37
|
-
exports.isSelectionCounterStalled = isSelectionCounterStalled;
|
|
38
37
|
exports.shouldReadbackFillOnActSuccess = shouldReadbackFillOnActSuccess;
|
|
39
38
|
exports.countNgInvalidContainers = countNgInvalidContainers;
|
|
40
39
|
exports.describeAttemptEffectSignals = describeAttemptEffectSignals;
|
|
@@ -526,8 +525,15 @@ const DOM_SNAPSHOT_EXPR = `(() => { const b = document.body; if (!b) return { ht
|
|
|
526
525
|
* `url()` rejects (OOPIF detached by the submit it just fired) can still
|
|
527
526
|
* report the main frame's post-navigation URL instead of throwing the
|
|
528
527
|
* whole attempt out of `executeStepWithHealing`.
|
|
528
|
+
*
|
|
529
|
+
* `captureSelectionState` (default false) additionally builds the per-element
|
|
530
|
+
* `selectionStateByXpath` baseline for the element-scoped click verifier. The
|
|
531
|
+
* caller sets it only for the pre/post pair of a selection/field-answer click
|
|
532
|
+
* step — never for submit/advance steps (so a self-toggling submit/Next button
|
|
533
|
+
* can't false-credit) nor for the mid-probe/retry snapshots — which also keeps
|
|
534
|
+
* this second full-DOM evaluate off the common (non-selection) path.
|
|
529
535
|
*/
|
|
530
|
-
async function snapshotPage(target, signalCounter, page) {
|
|
536
|
+
async function snapshotPage(target, signalCounter, page, captureSelectionState = false) {
|
|
531
537
|
let bodyHtmlLength = 0;
|
|
532
538
|
let visibleTextSignature = "";
|
|
533
539
|
let formValueSignature = "";
|
|
@@ -552,6 +558,25 @@ async function snapshotPage(target, signalCounter, page) {
|
|
|
552
558
|
// Snapshot is observational; on failure, defaults to 0/"" so the verifier
|
|
553
559
|
// sees no delta. Real state-class checks already cover the verified path.
|
|
554
560
|
}
|
|
561
|
+
// Per-element selection baseline for the element-scoped click verifier
|
|
562
|
+
// (see StepSnapshot.selectionStateByXpath). Captured ONLY when the caller
|
|
563
|
+
// asks (`captureSelectionState`) — the pre/post pair of a selection/field-
|
|
564
|
+
// answer click step. Skipping it for submit/advance steps is both the perf
|
|
565
|
+
// guard (this is a second full-DOM evaluate on a hot function) AND the
|
|
566
|
+
// correctness gate: with no baseline, verifyDomEffect's element read-back
|
|
567
|
+
// finds nothing to diff and defers to network/URL, so a submit/advance
|
|
568
|
+
// button's own self-toggle can never false-credit the step. Its own
|
|
569
|
+
// try/catch so a failure here can't lose the coarse signals above; an empty
|
|
570
|
+
// map leaves the verifier deferring to network/URL, never a false credit.
|
|
571
|
+
let selectionStateByXpath = {};
|
|
572
|
+
if (captureSelectionState) {
|
|
573
|
+
try {
|
|
574
|
+
selectionStateByXpath = asSelectionStateMap(await target.evaluate(SELECTION_STATE_MAP_EXPR));
|
|
575
|
+
}
|
|
576
|
+
catch {
|
|
577
|
+
// observational — empty baseline on failure
|
|
578
|
+
}
|
|
579
|
+
}
|
|
555
580
|
// A resolved child FrameTarget's url() reads location.href off the CDP
|
|
556
581
|
// frame session (frame-target.ts's childFrameTarget), which rejects (or
|
|
557
582
|
// trips its watchdog) once the OOPIF detaches — most commonly right after
|
|
@@ -566,6 +591,7 @@ async function snapshotPage(target, signalCounter, page) {
|
|
|
566
591
|
visibleTextSignature,
|
|
567
592
|
formValueSignature,
|
|
568
593
|
selectionStateSignature,
|
|
594
|
+
selectionStateByXpath,
|
|
569
595
|
};
|
|
570
596
|
}
|
|
571
597
|
/**
|
|
@@ -737,6 +763,22 @@ function isAdvanceStep(instruction) {
|
|
|
737
763
|
const haystack = instruction.toLowerCase();
|
|
738
764
|
return ADVANCE_STEP_PHRASES.some((p) => haystack.includes(p));
|
|
739
765
|
}
|
|
766
|
+
/**
|
|
767
|
+
* Whether `snapshotPage` should build the per-element selection baseline
|
|
768
|
+
* (`StepSnapshot.selectionStateByXpath`) for this step — i.e. whether
|
|
769
|
+
* `verifyDomEffect`'s element-scoped click read-back is allowed to credit it.
|
|
770
|
+
* True ONLY for a field-answer/selection step: a submit, a final, or an advance
|
|
771
|
+
* step must be verified by a real network/URL transition, so its own
|
|
772
|
+
* self-toggling button (a submit flipping to a loading/pressed class, a "Next"
|
|
773
|
+
* flipping `aria-pressed`) must never earn an element-scoped credit — matching
|
|
774
|
+
* the `!submit`/`!final`/`!advance` exclusions the former
|
|
775
|
+
* `isClickStateToggleVerified` gate enforced. Pure + exported so the gate the
|
|
776
|
+
* cascade depends on is unit-testable, not buried in `executeStepWithHealing`.
|
|
777
|
+
*/
|
|
778
|
+
function shouldCaptureSelectionState(params) {
|
|
779
|
+
const { step, isFinalStep, submitStep } = params;
|
|
780
|
+
return !(isFinalStep || submitStep || isAdvanceStep(step));
|
|
781
|
+
}
|
|
740
782
|
/**
|
|
741
783
|
* Whether a flow should be WARNed that its DOM-only advance guard is disarmed.
|
|
742
784
|
* `isDomOnlyAdvanceVerified` only vetoes a DOM-only "advance" when
|
|
@@ -1009,38 +1051,6 @@ function isClickViewSwapVerified(params) {
|
|
|
1009
1051
|
return true;
|
|
1010
1052
|
return textChanged && bytesDelta >= VIEW_SWAP_REVEAL_MIN_BYTES;
|
|
1011
1053
|
}
|
|
1012
|
-
/**
|
|
1013
|
-
* Credits a network-free click when it flipped a client-side SELECTION state —
|
|
1014
|
-
* an `aria-pressed`/`aria-checked`/`aria-selected` attribute, a `data-state`,
|
|
1015
|
-
* or a `selected`/`active`/`checked` class on the toggled control (captured by
|
|
1016
|
-
* `StepSnapshot.selectionStateSignature`). React/SPA multi-select wizards toggle
|
|
1017
|
-
* options this way with a trivial or NEGATIVE byte delta and no
|
|
1018
|
-
* network/URL/innerText/input-value change, so `isClickViewSwapVerified` and
|
|
1019
|
-
* the byte-floor phantom check both miss them and score the real selection a
|
|
1020
|
-
* phantom no-op. This is the size-independent counterpart to that gate. Scope
|
|
1021
|
-
* guards: only a plain `click`, never a final/submit step (those keep their
|
|
1022
|
-
* stronger submit-judge gate), and only when zero network fired (a network POST
|
|
1023
|
-
* is authoritative). **Stricter than `isClickViewSwapVerified` on advance
|
|
1024
|
-
* steps:** it excludes ANY advance/"Next" step (`isAdvance`), not just an
|
|
1025
|
-
* advance-WITH-pattern step. The selection signal fires on a validation-render
|
|
1026
|
-
* that toggles a control's own state, and an advance without a configured
|
|
1027
|
-
* `advanceTransitionBodyPattern` has no real-transition veto
|
|
1028
|
-
* (see `shouldWarnMissingAdvancePattern`) — so crediting a bare selection change
|
|
1029
|
-
* would desync the wizard step pointer. A real advance still verifies via
|
|
1030
|
-
* network/URL; a field-toggle answer step (not an advance) keeps this credit.
|
|
1031
|
-
*/
|
|
1032
|
-
function isClickStateToggleVerified(params) {
|
|
1033
|
-
const { resolvedAction, isFinalStep, submitStep, isAdvance, networkDelta, selectionStateChanged, } = params;
|
|
1034
|
-
if (resolvedAction?.method !== "click")
|
|
1035
|
-
return false;
|
|
1036
|
-
if (isFinalStep || submitStep)
|
|
1037
|
-
return false;
|
|
1038
|
-
if (isAdvance)
|
|
1039
|
-
return false;
|
|
1040
|
-
if (networkDelta !== 0)
|
|
1041
|
-
return false;
|
|
1042
|
-
return selectionStateChanged;
|
|
1043
|
-
}
|
|
1044
1054
|
/**
|
|
1045
1055
|
* Reads the "N settings/items selected" running count that multi-select wizard
|
|
1046
1056
|
* steps render as a live heading above the option grid. Returns the integer, or
|
|
@@ -1049,11 +1059,12 @@ function isClickStateToggleVerified(params) {
|
|
|
1049
1059
|
* `"<len>:<first-200-chars-of-innerText>"` — the counter idiom sits at the top
|
|
1050
1060
|
* of these steps, so it is always inside the 200-char window.
|
|
1051
1061
|
*
|
|
1052
|
-
*
|
|
1053
|
-
*
|
|
1054
|
-
*
|
|
1055
|
-
*
|
|
1056
|
-
*
|
|
1062
|
+
* Used by the deep-locator candidate walk to disambiguate WHICH of several
|
|
1063
|
+
* option candidates registered a selection: after clicking a candidate, the
|
|
1064
|
+
* running counter rising confirms that candidate was the real option (a
|
|
1065
|
+
* candidate-selection signal, distinct from the click verifier — which now
|
|
1066
|
+
* reads the resolved element's own committed state directly via
|
|
1067
|
+
* {@link verifyDomEffect}).
|
|
1057
1068
|
*/
|
|
1058
1069
|
function selectionCountFromSignature(visibleTextSignature) {
|
|
1059
1070
|
const colon = visibleTextSignature.indexOf(":");
|
|
@@ -1062,27 +1073,6 @@ function selectionCountFromSignature(visibleTextSignature) {
|
|
|
1062
1073
|
const digits = match?.[1];
|
|
1063
1074
|
return digits === undefined ? null : Number.parseInt(digits, 10);
|
|
1064
1075
|
}
|
|
1065
|
-
/**
|
|
1066
|
-
* Veto for a SELECTION-intent step whose weak DOM signals (view-swap byte delta,
|
|
1067
|
-
* a non-advance `domVerified` reflow) would otherwise credit a click that did
|
|
1068
|
-
* NOT register the selection. When the step's snapshot exposes a running
|
|
1069
|
-
* "N selected" counter (see {@link selectionCountFromSignature}) and that count
|
|
1070
|
-
* did not increase across the click, the option was not actually selected — a
|
|
1071
|
-
* phantom the fingerprint can't catch — so the caller must NOT treat the weak
|
|
1072
|
-
* signals as verification and should keep healing. Returns `true` only when a
|
|
1073
|
-
* counter is present on BOTH snapshots and did not rise; absent a counter it
|
|
1074
|
-
* returns `false` (no veto), so counter-less selection widgets are untouched.
|
|
1075
|
-
*/
|
|
1076
|
-
function isSelectionCounterStalled(params) {
|
|
1077
|
-
const { isSelectionStep, preVisibleTextSignature, postVisibleTextSignature } = params;
|
|
1078
|
-
if (!isSelectionStep)
|
|
1079
|
-
return false;
|
|
1080
|
-
const pre = selectionCountFromSignature(preVisibleTextSignature);
|
|
1081
|
-
const post = selectionCountFromSignature(postVisibleTextSignature);
|
|
1082
|
-
if (pre === null || post === null)
|
|
1083
|
-
return false;
|
|
1084
|
-
return post <= pre;
|
|
1085
|
-
}
|
|
1086
1076
|
/**
|
|
1087
1077
|
* Whether an act-success step warrants a committed-value read-back before the
|
|
1088
1078
|
* weak view-swap/form-value signals are allowed to accept it. A controlled
|
|
@@ -2454,13 +2444,12 @@ async function fillTextDatepickerInput(target, selector, value) {
|
|
|
2454
2444
|
// Month/year-picker variant: click the target month cell (0-based month
|
|
2455
2445
|
// index maps to the __month-N class) — THIS is the commit. The year/month
|
|
2456
2446
|
// dropdowns are set first only so the calendar shows the right year's month
|
|
2457
|
-
// page.
|
|
2458
|
-
//
|
|
2459
|
-
// other wrappers.
|
|
2447
|
+
// page. The bare "select" query covers every wrapper generically (site CSS
|
|
2448
|
+
// classes are never named here).
|
|
2460
2449
|
// Stock react-datepicker renders its month/year dropdowns as clickable divs,
|
|
2461
2450
|
// so on those the select loop is inert and the month-cell click carries it.
|
|
2462
2451
|
if (monthCells.length > 0) {
|
|
2463
|
-
const selects = cal.querySelectorAll("select
|
|
2452
|
+
const selects = cal.querySelectorAll("select");
|
|
2464
2453
|
for (const sel of selects) {
|
|
2465
2454
|
const opts = Array.from(sel.options).map((o) => (o.value || "").trim());
|
|
2466
2455
|
const yearOpt = opts.indexOf(String(year));
|
|
@@ -2660,6 +2649,150 @@ function xpathBodyForEvaluate(selector) {
|
|
|
2660
2649
|
const stripped = selector.startsWith("xpath=") ? selector.slice("xpath=".length) : selector;
|
|
2661
2650
|
return stripped.startsWith("/") || stripped.startsWith("(") ? stripped : null;
|
|
2662
2651
|
}
|
|
2652
|
+
/**
|
|
2653
|
+
* Browser-side expression body that resolves `xpath` and returns that one
|
|
2654
|
+
* element's {@link ElementSelectionFingerprint} (or `null` when the node is
|
|
2655
|
+
* absent). Trust boundary: `xpath` is a Stagehand-resolved selector, and
|
|
2656
|
+
* `JSON.stringify` produces a safe JS string literal, so composing it cannot
|
|
2657
|
+
* inject behavior. The `data-state` disclosure values `open`/`closed` (the
|
|
2658
|
+
* `aria-expanded` equivalent) are blanked so opening a popover isn't mistaken
|
|
2659
|
+
* for a selection.
|
|
2660
|
+
*/
|
|
2661
|
+
function elementSelectionFingerprintExpr(xpath) {
|
|
2662
|
+
return `(() => { const r = document.evaluate(${JSON.stringify(xpath)}, document, null, XPathResult.FIRST_ORDERED_NODE_TYPE, null); const el = r.singleNodeValue; if (!el) return null; const ds = el.getAttribute("data-state") || ""; return { kind: el.getAttribute("kind") || "", cls: el.getAttribute("class") || "", ariaPressed: el.getAttribute("aria-pressed") || "", ariaChecked: el.getAttribute("aria-checked") || "", ariaSelected: el.getAttribute("aria-selected") || "", dataState: (ds === "open" || ds === "closed") ? "" : ds, dataSelected: el.hasAttribute("data-selected") ? "1" : "", dataChecked: el.hasAttribute("data-checked") ? "1" : "", checked: (el.type === "checkbox" || el.type === "radio") ? (el.checked ? "1" : "0") : "", value: typeof el.value === "string" ? el.value.slice(0, 200) : "" }; })()`;
|
|
2663
|
+
}
|
|
2664
|
+
/**
|
|
2665
|
+
* Browser-side expression: build `{ absolutePositionalXpath →
|
|
2666
|
+
* ElementSelectionFingerprint }` for every VISIBLE interactive element, the
|
|
2667
|
+
* pre-action baseline {@link verifyDomEffect} diffs the resolved element
|
|
2668
|
+
* against. Trust boundary: static string literal, no interpolation. The xpath
|
|
2669
|
+
* is generated by `xpathOf`, character-identical to the two other in-page
|
|
2670
|
+
* copies in this file (the ng-invalid probe and the field-label scan) so it
|
|
2671
|
+
* byte-matches Stagehand's resolved `xpath=/html[1]/…` selectors: hardcoded
|
|
2672
|
+
* `/html[1]/body[1]/` prefix, walk `!== document.body`, `nodeName.toLowerCase()`,
|
|
2673
|
+
* `parentElement`. `[role=dialog],[role=tooltip],[aria-live]` subtrees are
|
|
2674
|
+
* skipped — they churn without a selection change. Visibility uses the same
|
|
2675
|
+
* rect + `getComputedStyle` idiom as `deep-locator-scan.ts`'s `IS_VISIBLE_EXPR`
|
|
2676
|
+
* (rather than `offsetParent`), so on-screen `position:fixed` controls — a
|
|
2677
|
+
* sticky Next/Submit bar, whose `offsetParent` is `null` — still enter the map.
|
|
2678
|
+
*/
|
|
2679
|
+
const SELECTION_STATE_MAP_EXPR = `(() => {
|
|
2680
|
+
const b = document.body;
|
|
2681
|
+
if (!b) return {};
|
|
2682
|
+
const xpathOf = (node) => {
|
|
2683
|
+
const parts = [];
|
|
2684
|
+
while (node && node.nodeType === 1 && node !== document.body) {
|
|
2685
|
+
const tag = node.nodeName.toLowerCase();
|
|
2686
|
+
let idx = 1;
|
|
2687
|
+
let sib = node.previousElementSibling;
|
|
2688
|
+
while (sib) {
|
|
2689
|
+
if (sib.nodeName.toLowerCase() === tag) idx++;
|
|
2690
|
+
sib = sib.previousElementSibling;
|
|
2691
|
+
}
|
|
2692
|
+
parts.unshift(tag + "[" + idx + "]");
|
|
2693
|
+
node = node.parentElement;
|
|
2694
|
+
}
|
|
2695
|
+
return "/html[1]/body[1]/" + parts.join("/");
|
|
2696
|
+
};
|
|
2697
|
+
const visible = (el) => {
|
|
2698
|
+
const rect = el.getBoundingClientRect();
|
|
2699
|
+
if (rect.width === 0 && rect.height === 0) return false;
|
|
2700
|
+
const style = getComputedStyle(el);
|
|
2701
|
+
return style.display !== "none" && style.visibility !== "hidden";
|
|
2702
|
+
};
|
|
2703
|
+
const sel = "button,[role=button],a,[tabindex],input,select,textarea,[role=option],[role=tab],[role=switch],[role=checkbox],[role=menuitemcheckbox]";
|
|
2704
|
+
const skip = (el) => el.closest("[role=dialog],[role=tooltip],[aria-live]") !== null;
|
|
2705
|
+
const out = {};
|
|
2706
|
+
for (const el of b.querySelectorAll(sel)) {
|
|
2707
|
+
if (!visible(el) || skip(el)) continue;
|
|
2708
|
+
const ds = el.getAttribute("data-state") || "";
|
|
2709
|
+
out[xpathOf(el)] = {
|
|
2710
|
+
kind: el.getAttribute("kind") || "",
|
|
2711
|
+
cls: el.getAttribute("class") || "",
|
|
2712
|
+
ariaPressed: el.getAttribute("aria-pressed") || "",
|
|
2713
|
+
ariaChecked: el.getAttribute("aria-checked") || "",
|
|
2714
|
+
ariaSelected: el.getAttribute("aria-selected") || "",
|
|
2715
|
+
dataState: (ds === "open" || ds === "closed") ? "" : ds,
|
|
2716
|
+
dataSelected: el.hasAttribute("data-selected") ? "1" : "",
|
|
2717
|
+
dataChecked: el.hasAttribute("data-checked") ? "1" : "",
|
|
2718
|
+
checked: (el.type === "checkbox" || el.type === "radio") ? (el.checked ? "1" : "0") : "",
|
|
2719
|
+
value: typeof el.value === "string" ? el.value.slice(0, 200) : "",
|
|
2720
|
+
};
|
|
2721
|
+
}
|
|
2722
|
+
return out;
|
|
2723
|
+
})()`;
|
|
2724
|
+
/**
|
|
2725
|
+
* Narrows an unknown `evaluate` result into the
|
|
2726
|
+
* {@link StepSnapshot.selectionStateByXpath} map, discarding any malformed
|
|
2727
|
+
* entry. Defaults to `{}` so a snapshot failure leaves the verifier with an
|
|
2728
|
+
* empty baseline (defer-to-network), never a throw.
|
|
2729
|
+
*/
|
|
2730
|
+
function asSelectionStateMap(raw) {
|
|
2731
|
+
if (raw === null || typeof raw !== "object")
|
|
2732
|
+
return {};
|
|
2733
|
+
const out = {};
|
|
2734
|
+
for (const [xpath, value] of Object.entries(raw)) {
|
|
2735
|
+
const fp = asSelectionFingerprint(value);
|
|
2736
|
+
if (fp)
|
|
2737
|
+
out[xpath] = fp;
|
|
2738
|
+
}
|
|
2739
|
+
return out;
|
|
2740
|
+
}
|
|
2741
|
+
/** True when two {@link ElementSelectionFingerprint}s differ in any tracked field. */
|
|
2742
|
+
function selectionFingerprintChanged(pre, post) {
|
|
2743
|
+
return (pre.kind !== post.kind ||
|
|
2744
|
+
pre.cls !== post.cls ||
|
|
2745
|
+
pre.ariaPressed !== post.ariaPressed ||
|
|
2746
|
+
pre.ariaChecked !== post.ariaChecked ||
|
|
2747
|
+
pre.ariaSelected !== post.ariaSelected ||
|
|
2748
|
+
pre.dataState !== post.dataState ||
|
|
2749
|
+
pre.dataSelected !== post.dataSelected ||
|
|
2750
|
+
pre.dataChecked !== post.dataChecked ||
|
|
2751
|
+
pre.checked !== post.checked ||
|
|
2752
|
+
pre.value !== post.value);
|
|
2753
|
+
}
|
|
2754
|
+
/**
|
|
2755
|
+
* Narrows an unknown `evaluate` result to an {@link ElementSelectionFingerprint}.
|
|
2756
|
+
* Every field is emitted as a string by {@link elementSelectionFingerprintExpr},
|
|
2757
|
+
* so a present object with a string `kind` is a sufficient shape check.
|
|
2758
|
+
*/
|
|
2759
|
+
function asSelectionFingerprint(raw) {
|
|
2760
|
+
if (raw === null || typeof raw !== "object")
|
|
2761
|
+
return null;
|
|
2762
|
+
const r = raw;
|
|
2763
|
+
if (typeof r.kind !== "string")
|
|
2764
|
+
return null;
|
|
2765
|
+
return {
|
|
2766
|
+
kind: r.kind,
|
|
2767
|
+
cls: typeof r.cls === "string" ? r.cls : "",
|
|
2768
|
+
ariaPressed: typeof r.ariaPressed === "string" ? r.ariaPressed : "",
|
|
2769
|
+
ariaChecked: typeof r.ariaChecked === "string" ? r.ariaChecked : "",
|
|
2770
|
+
ariaSelected: typeof r.ariaSelected === "string" ? r.ariaSelected : "",
|
|
2771
|
+
dataState: typeof r.dataState === "string" ? r.dataState : "",
|
|
2772
|
+
dataSelected: typeof r.dataSelected === "string" ? r.dataSelected : "",
|
|
2773
|
+
dataChecked: typeof r.dataChecked === "string" ? r.dataChecked : "",
|
|
2774
|
+
checked: typeof r.checked === "string" ? r.checked : "",
|
|
2775
|
+
value: typeof r.value === "string" ? r.value : "",
|
|
2776
|
+
};
|
|
2777
|
+
}
|
|
2778
|
+
/**
|
|
2779
|
+
* Reads one element's {@link ElementSelectionFingerprint} off `target` by
|
|
2780
|
+
* xpath, or `null` when the selector isn't an xpath / the node is absent / the
|
|
2781
|
+
* evaluate throws. Used by the click branch of {@link verifyDomEffect} to
|
|
2782
|
+
* compare the resolved element's committed selection state against the
|
|
2783
|
+
* pre-action baseline captured in {@link StepSnapshot.selectionStateByXpath}.
|
|
2784
|
+
*/
|
|
2785
|
+
async function readElementSelectionFingerprint(target, selector) {
|
|
2786
|
+
const xpath = xpathBodyForEvaluate(selector);
|
|
2787
|
+
if (!xpath)
|
|
2788
|
+
return null;
|
|
2789
|
+
try {
|
|
2790
|
+
return asSelectionFingerprint(await target.evaluate(elementSelectionFingerprintExpr(xpath)));
|
|
2791
|
+
}
|
|
2792
|
+
catch {
|
|
2793
|
+
return null;
|
|
2794
|
+
}
|
|
2795
|
+
}
|
|
2663
2796
|
/** How long the upload primitive waits for a post-setInputFiles network POST. */
|
|
2664
2797
|
const UPLOAD_NETWORK_TIMEOUT_MS = 5_000;
|
|
2665
2798
|
/** Polling interval while waiting for the upload's network signal. */
|
|
@@ -4572,8 +4705,16 @@ async function dispatchJqueryChangeEvent(target, selector) {
|
|
|
4572
4705
|
*
|
|
4573
4706
|
* `target` scopes both the locator/evaluate reads and the jQuery-change
|
|
4574
4707
|
* dispatch to the resolved frame (main or a cross-origin child).
|
|
4708
|
+
*
|
|
4709
|
+
* `preSelectionState` is the pre-action per-element baseline
|
|
4710
|
+
* ({@link StepSnapshot.selectionStateByXpath}) the click branch diffs the
|
|
4711
|
+
* resolved element against — the element-scoped authoritative "did this
|
|
4712
|
+
* selection register" signal for design-system option/toggle buttons that
|
|
4713
|
+
* expose no native `checked` (Base Web `kind`, hashed styletron class, ARIA).
|
|
4714
|
+
* Defaults to `{}` so callers/tests that don't supply it keep the prior
|
|
4715
|
+
* radio/checkbox-and-network behavior.
|
|
4575
4716
|
*/
|
|
4576
|
-
async function verifyDomEffect(target, action) {
|
|
4717
|
+
async function verifyDomEffect(target, action, preSelectionState = {}) {
|
|
4577
4718
|
const selector = action.selector;
|
|
4578
4719
|
const method = action.method;
|
|
4579
4720
|
if (!selector || !method)
|
|
@@ -4717,9 +4858,15 @@ async function verifyDomEffect(target, action) {
|
|
|
4717
4858
|
return true;
|
|
4718
4859
|
case "click": {
|
|
4719
4860
|
// Clicks on radios and checkboxes toggle `:checked` without firing a
|
|
4720
|
-
// network request — same false-fail class as fill.
|
|
4721
|
-
//
|
|
4722
|
-
//
|
|
4861
|
+
// network request — same false-fail class as fill. Every OTHER click
|
|
4862
|
+
// (design-system option/toggle buttons, links, custom controls) is
|
|
4863
|
+
// verified element-scoped: compare the RESOLVED element's own committed
|
|
4864
|
+
// selection state before vs. after the click. A Base Web option flips
|
|
4865
|
+
// its `kind` (`tertiary`→`primary`) + a hashed styletron class with no
|
|
4866
|
+
// network, no URL change, and a trivial/negative byte delta — the
|
|
4867
|
+
// authoritative signal is that the clicked element's own fingerprint
|
|
4868
|
+
// moved, which this reads directly instead of guessing from page-wide
|
|
4869
|
+
// DOM deltas.
|
|
4723
4870
|
const xpath = xpathBody(selector);
|
|
4724
4871
|
if (!xpath)
|
|
4725
4872
|
return false;
|
|
@@ -4740,8 +4887,21 @@ async function verifyDomEffect(target, action) {
|
|
|
4740
4887
|
return false;
|
|
4741
4888
|
}
|
|
4742
4889
|
if (inputType !== "radio" && inputType !== "checkbox") {
|
|
4743
|
-
//
|
|
4744
|
-
|
|
4890
|
+
// Element-scoped selection read-back. The pre-map is keyed by the
|
|
4891
|
+
// same absolute positional xpath Stagehand resolves to, so a lookup
|
|
4892
|
+
// by the resolved element's xpath body yields THAT element's baseline.
|
|
4893
|
+
// Credit the click iff the element's own committed state moved across
|
|
4894
|
+
// it. No baseline (element newly mounted, or an xpath the generator
|
|
4895
|
+
// and Stagehand index differently) → hold no opinion and let the
|
|
4896
|
+
// network/URL signal decide: a miss is never a false credit, and a
|
|
4897
|
+
// selection change on any OTHER element can never credit this step.
|
|
4898
|
+
const preFingerprint = preSelectionState[xpath];
|
|
4899
|
+
if (!preFingerprint)
|
|
4900
|
+
return false;
|
|
4901
|
+
const postFingerprint = await readElementSelectionFingerprint(target, selector);
|
|
4902
|
+
if (!postFingerprint)
|
|
4903
|
+
return false;
|
|
4904
|
+
return selectionFingerprintChanged(preFingerprint, postFingerprint);
|
|
4745
4905
|
}
|
|
4746
4906
|
const isCheckedNow = await locator.isChecked();
|
|
4747
4907
|
if (!isCheckedNow)
|
|
@@ -5281,6 +5441,13 @@ async function executeStepWithHealing(params) {
|
|
|
5281
5441
|
// (verified 2026-06-15 on one measured tenant's telemetry). Site-agnostic: any flow
|
|
5282
5442
|
// whose submit is mid-list can mark its submit step explicitly.
|
|
5283
5443
|
const requireSubmitEndpoint = (isFinalStep || submitStep) && submitEndpointPattern !== null;
|
|
5444
|
+
// Capture the per-element selection baseline (for verifyDomEffect's element-
|
|
5445
|
+
// scoped click read-back) ONLY for selection/field-answer click steps — never
|
|
5446
|
+
// for submit/advance steps, whose own self-toggling buttons must not
|
|
5447
|
+
// false-credit the step, and whose advance/submit verdicts require a real
|
|
5448
|
+
// network/URL transition. Also keeps the extra full-DOM evaluate off the
|
|
5449
|
+
// submit/advance path. Step-level intent (available before the attempt loop).
|
|
5450
|
+
const captureSelectionState = shouldCaptureSelectionState({ step, isFinalStep, submitStep });
|
|
5284
5451
|
const attempts = [];
|
|
5285
5452
|
const triedSelectors = [];
|
|
5286
5453
|
const failureReasons = [];
|
|
@@ -5538,6 +5705,7 @@ async function executeStepWithHealing(params) {
|
|
|
5538
5705
|
visibleTextSignature: "",
|
|
5539
5706
|
selectionStateSignature: "",
|
|
5540
5707
|
formValueSignature: "",
|
|
5708
|
+
selectionStateByXpath: {},
|
|
5541
5709
|
};
|
|
5542
5710
|
attempts.push({
|
|
5543
5711
|
attempt: 0,
|
|
@@ -5719,7 +5887,7 @@ async function executeStepWithHealing(params) {
|
|
|
5719
5887
|
if (attempt > 1) {
|
|
5720
5888
|
await page.waitForTimeout(attempt * ATTEMPT_BACKOFF_MS);
|
|
5721
5889
|
}
|
|
5722
|
-
const pre = await snapshotPage(frameTarget ?? (0, frame_target_1.mainFrameTarget)(page), signalCounter, page);
|
|
5890
|
+
const pre = await snapshotPage(frameTarget ?? (0, frame_target_1.mainFrameTarget)(page), signalCounter, page, captureSelectionState);
|
|
5723
5891
|
// Snapshot the meta-tail length so the final-step pattern gate can scope
|
|
5724
5892
|
// its URL scan to captures added DURING this attempt (not historical
|
|
5725
5893
|
// tail from earlier steps).
|
|
@@ -6576,11 +6744,14 @@ async function executeStepWithHealing(params) {
|
|
|
6576
6744
|
// State-class actions (fill/check/etc.) never move the network counter or URL,
|
|
6577
6745
|
// so the legacy heuristic false-negatived every form fill. Re-read DOM state
|
|
6578
6746
|
// for those; keep the navigation signal authoritative for clicks/links —
|
|
6579
|
-
// but ALSO route clicks through verifyDomEffect, which
|
|
6580
|
-
//
|
|
6581
|
-
//
|
|
6747
|
+
// but ALSO route clicks through verifyDomEffect, which authoritatively
|
|
6748
|
+
// credits a radio/checkbox toggle OR (via the pre/post element fingerprint
|
|
6749
|
+
// baseline `pre.selectionStateByXpath`) a design-system option/toggle whose
|
|
6750
|
+
// own committed state changed; otherwise it returns false and the
|
|
6751
|
+
// network/URL signal decides. Radios/checkboxes are click-but-no-network
|
|
6752
|
+
// just like fills.
|
|
6582
6753
|
const domVerified = resolvedAction !== null && (isStateClass || isClick)
|
|
6583
|
-
? await verifyDomEffect(frameTarget ?? (0, frame_target_1.mainFrameTarget)(page), resolvedAction)
|
|
6754
|
+
? await verifyDomEffect(frameTarget ?? (0, frame_target_1.mainFrameTarget)(page), resolvedAction, pre.selectionStateByXpath)
|
|
6584
6755
|
: false;
|
|
6585
6756
|
// Interior-advance transition gate (opt-in). On SPAs where a page advance
|
|
6586
6757
|
// and a mere field-edit share one endpoint URL (the wizard ATS's `/gq`:
|
|
@@ -6654,20 +6825,15 @@ async function executeStepWithHealing(params) {
|
|
|
6654
6825
|
bytesDelta: post.bodyHtmlLength - pre.bodyHtmlLength,
|
|
6655
6826
|
textChanged: post.visibleTextSignature !== pre.visibleTextSignature,
|
|
6656
6827
|
});
|
|
6657
|
-
//
|
|
6658
|
-
//
|
|
6659
|
-
//
|
|
6660
|
-
//
|
|
6661
|
-
//
|
|
6662
|
-
//
|
|
6663
|
-
|
|
6664
|
-
|
|
6665
|
-
|
|
6666
|
-
submitStep,
|
|
6667
|
-
isAdvance: isAdvanceStep(step),
|
|
6668
|
-
networkDelta: post.networkCount - pre.networkCount,
|
|
6669
|
-
selectionStateChanged: post.selectionStateSignature !== pre.selectionStateSignature,
|
|
6670
|
-
});
|
|
6828
|
+
// A network-free selection/option toggle — including a Base Web `kind`
|
|
6829
|
+
// flip or hashed-class swap that exposes no aria/data-state marker and
|
|
6830
|
+
// moves a trivial or NEGATIVE byte delta — is now credited authoritatively
|
|
6831
|
+
// and element-scoped by `verifyDomEffect` (via the pre/post per-element
|
|
6832
|
+
// fingerprint baseline), flowing through `domVerified`. The former
|
|
6833
|
+
// page-wide `clickStateToggleVerified` guess and its `selectionCounterStalled`
|
|
6834
|
+
// veto (which only existed to suppress that guess's false positives) are
|
|
6835
|
+
// removed: there is nothing to guess or veto when the resolved element's own
|
|
6836
|
+
// committed state is read directly.
|
|
6671
6837
|
// Form-value-diff signal: `visibleTextSignature` never reflects a plain
|
|
6672
6838
|
// <input>/<textarea>/<select>'s `value` property, so a fill with no
|
|
6673
6839
|
// secondary UI side effect (no toggle, no formatted display) had ZERO
|
|
@@ -6731,33 +6897,11 @@ async function executeStepWithHealing(params) {
|
|
|
6731
6897
|
}
|
|
6732
6898
|
}
|
|
6733
6899
|
}
|
|
6734
|
-
// Selection-counter veto. A multi-select option click whose widget exposes
|
|
6735
|
-
// no aria/data-state marker (obfuscated hashed classes) leaves
|
|
6736
|
-
// `clickStateToggleVerified` blind, so a phantom that merely reflowed the
|
|
6737
|
-
// DOM would ride the weak `domVerifiedForStep`/`clickViewSwapVerified`
|
|
6738
|
-
// signals to a FALSE credit — the flow then advances to a "Next" that
|
|
6739
|
-
// no-ops because nothing was actually selected. When the step's running
|
|
6740
|
-
// "N selected" counter did not rise, suppress those weak DOM-delta signals
|
|
6741
|
-
// so the cascade keeps trying a real selection. Strong signals (real
|
|
6742
|
-
// network/URL transition) and the counter-independent state-toggle /
|
|
6743
|
-
// form-value signals — each of which IS the selection registering — are
|
|
6744
|
-
// never vetoed; counter-less widgets are untouched (the helper no-ops).
|
|
6745
|
-
const selectionCounterStalled = isSelectionCounterStalled({
|
|
6746
|
-
isSelectionStep: parseSelectStep(step) !== null,
|
|
6747
|
-
preVisibleTextSignature: pre.visibleTextSignature,
|
|
6748
|
-
postVisibleTextSignature: post.visibleTextSignature,
|
|
6749
|
-
});
|
|
6750
|
-
if (selectionCounterStalled && (domVerifiedForStep || clickViewSwapVerified)) {
|
|
6751
|
-
logger.info(`${formatStepPrefix(stepIndex, totalSteps)} selection step credited only via DOM reflow but the "N selected" counter did not rise — the option did not register; not treating the weak DOM signal as verified`);
|
|
6752
|
-
}
|
|
6753
|
-
const domVerifiedAfterCounter = selectionCounterStalled ? false : domVerifiedForStep;
|
|
6754
|
-
const clickViewSwapAfterCounter = selectionCounterStalled ? false : clickViewSwapVerified;
|
|
6755
6900
|
let verified = networkIsRealAdvance ||
|
|
6756
6901
|
urlChanged ||
|
|
6757
|
-
|
|
6902
|
+
domVerifiedForStep ||
|
|
6758
6903
|
datepickerCommitted ||
|
|
6759
|
-
(!datepickerRejected &&
|
|
6760
|
-
(clickViewSwapAfterCounter || formValueVerified || clickStateToggleVerified));
|
|
6904
|
+
(!datepickerRejected && (clickViewSwapVerified || formValueVerified));
|
|
6761
6905
|
// Final-step submit-verification gate. Replaces the deterministic
|
|
6762
6906
|
// submitEndpointPattern regex with a Haiku 4.5 LLM judgment over
|
|
6763
6907
|
// multi-signal evidence (network captures, page URL/title, DOM
|
|
@@ -6884,9 +7028,7 @@ async function executeStepWithHealing(params) {
|
|
|
6884
7028
|
? "dom"
|
|
6885
7029
|
: formValueVerified
|
|
6886
7030
|
? "form-value"
|
|
6887
|
-
:
|
|
6888
|
-
? "state-toggle"
|
|
6889
|
-
: "dom";
|
|
7031
|
+
: "dom";
|
|
6890
7032
|
}
|
|
6891
7033
|
// N+16 probe: Stagehand's CDP click sometimes lands on the button without
|
|
6892
7034
|
// triggering React's SyntheticEvent layer (or jQuery delegated handlers).
|
|
@@ -6974,13 +7116,26 @@ async function executeStepWithHealing(params) {
|
|
|
6974
7116
|
const retryHtmlDelta = retryPost.bodyHtmlLength - pre.bodyHtmlLength;
|
|
6975
7117
|
const retryTextChanged = retryPost.visibleTextSignature !== pre.visibleTextSignature;
|
|
6976
7118
|
const retryFormValueChanged = retryPost.formValueSignature !== pre.formValueSignature;
|
|
6977
|
-
//
|
|
6978
|
-
//
|
|
6979
|
-
//
|
|
6980
|
-
//
|
|
6981
|
-
//
|
|
7119
|
+
// Element-scoped selection read-back for the n+16 fallback — the same
|
|
7120
|
+
// authoritative signal the primary verifier uses via `verifyDomEffect`,
|
|
7121
|
+
// applied to the element this fallback just re-clicked. Credits only
|
|
7122
|
+
// when the RESOLVED element's own committed state moved across the
|
|
7123
|
+
// fallback click (Base Web `kind`/class, ARIA, native checked), read
|
|
7124
|
+
// against the pre-action baseline. Excludes advance/"Next" steps: an
|
|
7125
|
+
// advance without a configured transition pattern has no real-transition
|
|
7126
|
+
// veto, so crediting a bare selection change would desync the step
|
|
7127
|
+
// pointer. No page-wide signature, no counter veto — nothing to guess
|
|
7128
|
+
// or suppress when the element's own state is read directly.
|
|
6982
7129
|
const retrySelectionStateChanged = !isAdvanceStep(step) &&
|
|
6983
|
-
|
|
7130
|
+
(await (async () => {
|
|
7131
|
+
const preFingerprint = xpath ? pre.selectionStateByXpath[xpath] : undefined;
|
|
7132
|
+
if (!preFingerprint || !resolvedAction?.selector)
|
|
7133
|
+
return false;
|
|
7134
|
+
const postFingerprint = await readElementSelectionFingerprint(frameTarget ?? (0, frame_target_1.mainFrameTarget)(page), resolvedAction.selector);
|
|
7135
|
+
return postFingerprint !== null
|
|
7136
|
+
? selectionFingerprintChanged(preFingerprint, postFingerprint)
|
|
7137
|
+
: false;
|
|
7138
|
+
})());
|
|
6984
7139
|
// RC2: an advance/`kind=click` "Next" that only grew the DOM
|
|
6985
7140
|
// (validation errors rendered) with NO network/URL change is a
|
|
6986
7141
|
// validation-blocked no-op, not a real transition — but the
|
|
@@ -7031,41 +7186,25 @@ async function executeStepWithHealing(params) {
|
|
|
7031
7186
|
logger.info(`${formatStepPrefix(stepIndex, totalSteps)} n+16 fallback advanced but no real transition (non-advancing POST / field toggle); not treating as verified`);
|
|
7032
7187
|
}
|
|
7033
7188
|
// Weak DOM-only signals (an html-byte delta, a visible-text change, a
|
|
7034
|
-
// form-value change
|
|
7189
|
+
// form-value change) are NOT sufficient to
|
|
7035
7190
|
// verify a final/submit step on their own: a validation re-render
|
|
7036
7191
|
// produces exactly these without the submit landing. The primary
|
|
7037
|
-
// verifier is safe here because clickViewSwapVerified/formValueVerified
|
|
7038
|
-
//
|
|
7192
|
+
// verifier is safe here because clickViewSwapVerified/formValueVerified
|
|
7193
|
+
// self-exclude final/submit; the n+16
|
|
7039
7194
|
// fallback ORs the raw deltas, so gate them explicitly. On a
|
|
7040
7195
|
// final/submit step they only count when the submit-endpoint judge will
|
|
7041
7196
|
// actually run below (requireSubmitEndpoint) to corroborate — otherwise
|
|
7042
7197
|
// only a strong signal (network/url) or a verified checkbox state may
|
|
7043
7198
|
// pass. Non-final/submit steps are unaffected.
|
|
7044
7199
|
const weakDomSignalsAllowed = (!isFinalStep && !submitStep) || requireSubmitEndpoint;
|
|
7045
|
-
// Selection-counter veto for the n+16 fallback — the same gate the
|
|
7046
|
-
// primary verifier applies, recomputed against `retryPost`. Without
|
|
7047
|
-
// it this path would re-credit a stalled selection the primary veto
|
|
7048
|
-
// already suppressed: the fallback `el.click()` can reflow the DOM
|
|
7049
|
-
// (html/text/selection-state delta) without registering the option,
|
|
7050
|
-
// and those weak deltas are exactly what the OR below admits for a
|
|
7051
|
-
// non-final selection step. Only the weak-delta disjunct is vetoed;
|
|
7052
|
-
// strong signals (network/url) and a verified checkbox state pass.
|
|
7053
|
-
const retrySelectionCounterStalled = isSelectionCounterStalled({
|
|
7054
|
-
isSelectionStep: parseSelectStep(step) !== null,
|
|
7055
|
-
preVisibleTextSignature: pre.visibleTextSignature,
|
|
7056
|
-
postVisibleTextSignature: retryPost.visibleTextSignature,
|
|
7057
|
-
});
|
|
7058
7200
|
let retryVerified = !clickBlockedByInvalid &&
|
|
7059
7201
|
!fallbackDomOnlyAdvance &&
|
|
7060
7202
|
(retryNetworkFired ||
|
|
7061
7203
|
retryUrlChanged ||
|
|
7062
7204
|
checkboxStateVerified ||
|
|
7205
|
+
retrySelectionStateChanged ||
|
|
7063
7206
|
(weakDomSignalsAllowed &&
|
|
7064
|
-
|
|
7065
|
-
(retryHtmlDelta !== 0 ||
|
|
7066
|
-
retryTextChanged ||
|
|
7067
|
-
retryFormValueChanged ||
|
|
7068
|
-
retrySelectionStateChanged)));
|
|
7207
|
+
(retryHtmlDelta !== 0 || retryTextChanged || retryFormValueChanged)));
|
|
7069
7208
|
// Apply the same submit-endpoint gate the primary verifier uses.
|
|
7070
7209
|
// Without this, the n+16 fallback would still ride past a
|
|
7071
7210
|
// tracking-pixel-only click on the final step. Same Haiku LLM
|
|
@@ -7141,17 +7280,7 @@ async function executeStepWithHealing(params) {
|
|
|
7141
7280
|
logger.info(`n+16 probe: step=${stepIndex + 1}/${totalSteps?.() ?? "?"} attempt=${attempt} el.click() fallback fired=${fired === true} kind=${probeResult.kind ?? "none"} checkboxStateVerified=${checkboxStateVerified} ancestorStillInvalid=${ancestorStillInvalid}; network=${retryNetworkFired} url=${retryUrlChanged} htmlDelta=${retryHtmlDelta} textChanged=${retryTextChanged} formValueChanged=${retryFormValueChanged} selectionStateChanged=${retrySelectionStateChanged} verified=${retryVerified}`);
|
|
7142
7281
|
if (retryVerified) {
|
|
7143
7282
|
if (record.verifiedBy === null) {
|
|
7144
|
-
record.verifiedBy = retryUrlChanged
|
|
7145
|
-
? "url"
|
|
7146
|
-
: retryNetworkFired
|
|
7147
|
-
? "network"
|
|
7148
|
-
: retryHtmlDelta === 0 &&
|
|
7149
|
-
!retryTextChanged &&
|
|
7150
|
-
!retryFormValueChanged &&
|
|
7151
|
-
!checkboxStateVerified &&
|
|
7152
|
-
retrySelectionStateChanged
|
|
7153
|
-
? "state-toggle"
|
|
7154
|
-
: "dom";
|
|
7283
|
+
record.verifiedBy = retryUrlChanged ? "url" : retryNetworkFired ? "network" : "dom";
|
|
7155
7284
|
}
|
|
7156
7285
|
record.post = retryPost;
|
|
7157
7286
|
attempts.push(record);
|
|
@@ -7173,7 +7302,7 @@ async function executeStepWithHealing(params) {
|
|
|
7173
7302
|
attempts.push(record);
|
|
7174
7303
|
if (verified) {
|
|
7175
7304
|
if (attempt > 1) {
|
|
7176
|
-
logger.info(`${formatStepPrefix(stepIndex, totalSteps)} healed on attempt ${attempt} via ${record.technique} (network=${networkFired} url=${urlChanged} dom=${domVerified}
|
|
7305
|
+
logger.info(`${formatStepPrefix(stepIndex, totalSteps)} healed on attempt ${attempt} via ${record.technique} (network=${networkFired} url=${urlChanged} dom=${domVerified} verifiedBy=${record.verifiedBy})`);
|
|
7177
7306
|
}
|
|
7178
7307
|
else {
|
|
7179
7308
|
// Why log first-try wins explicitly: prior to this change, attempt-1
|
|
@@ -7183,7 +7312,7 @@ async function executeStepWithHealing(params) {
|
|
|
7183
7312
|
// collapse" (log showed 2 heals) but telemetry calls.ndjson showed
|
|
7184
7313
|
// 32 successful Stagehand acts. Surfacing attempt-1 wins lets the
|
|
7185
7314
|
// log match telemetry and prevents the same false alarm.
|
|
7186
|
-
logger.info(`${formatStepPrefix(stepIndex, totalSteps)} succeeded on attempt 1 via ${record.technique} (network=${networkFired} url=${urlChanged} dom=${domVerified}
|
|
7315
|
+
logger.info(`${formatStepPrefix(stepIndex, totalSteps)} succeeded on attempt 1 via ${record.technique} (network=${networkFired} url=${urlChanged} dom=${domVerified} verifiedBy=${record.verifiedBy})`);
|
|
7187
7316
|
}
|
|
7188
7317
|
trajectory?.push({ stepIndex, verifiedBy: record.verifiedBy });
|
|
7189
7318
|
return "completed";
|
|
@@ -7198,6 +7327,11 @@ async function executeStepWithHealing(params) {
|
|
|
7198
7327
|
actResultSuccess: record.actResultSuccess,
|
|
7199
7328
|
pre,
|
|
7200
7329
|
post,
|
|
7330
|
+
// Authoritative element-scoped signal: `verifyDomEffect` read the resolved
|
|
7331
|
+
// element's own committed-state delta (Base Web `kind`/class, ARIA, native
|
|
7332
|
+
// checked) into `domVerified`. A registered selection toggle no longer
|
|
7333
|
+
// reads as a phantom just because it moved no network/URL/bytes.
|
|
7334
|
+
elementStateChanged: domVerified,
|
|
7201
7335
|
isSubmitShapedStep: isFinalStep || submitStep,
|
|
7202
7336
|
});
|
|
7203
7337
|
const reason = record.errorMessage
|