Pup - the AI's own browser

@@ -5181,7 +5181,9 @@ async function launchSession(sessionId, url, { freshProfile = false, profile, st     } catch (e) {       console.log(`[creds] applyCredentialsToPage on launchSession failed: ${e.message}`);     }-    await page.goto(url, { waitUntil: 'domcontentloaded', timeout: 30000 });+    // v1.9.105: keep the main-frame HTTP status — the strongest broken-render signal there is.+    const _resp = await page.goto(url, { waitUntil: 'domcontentloaded', timeout: 30000 });+    try { page.__pupHttpStatus = _resp && typeof _resp.status === 'function' ? _resp.status() : null; } catch (e) {}     // v1.8.68: wait for the page to SETTLE (load event, capped at 2.5s) while the window is     // still OFF-SCREEN, so the first paint's window-raise is invisible. Skip the wait if the     // page is already complete.@@ -5743,6 +5745,47 @@ const PUP_ISSUES_REF = 'adom/adom-desktop-puppeteer-bridge'; // says nothing about whether the page painted, whether the CSS loaded, or whether a stack trace is // filling the viewport. pup is the one component that knows a window was just pointed at content, // so it is the right place to insist on verification — in the response the AI actually reads.+// v1.9.105 (John: "you didn't even attempt to test their content and as you can see its 404") —+// PROGRAMMATIC RENDER CHECK. Telling the AI "go screenshot it" proved too weak: the launcher-mockup+// thread opened a .adom.cloud app that served "404 page not found", pup answered ok:true and said+// NOTHING, and the AI reported success. pup HAS the page, so it can just LOOK. This probes the loaded+// document and returns a verdict, making a broken render a FACT in the response instead of a suggestion+// the caller may skip. One evaluate, no screenshot, and it never throws into the caller's path.+async function probeRender(page, httpStatus) {+  const out = { httpStatus: (httpStatus === undefined ? null : httpStatus), ok: true, signals: [] };+  try {+    const probe = await Promise.race([+      page.evaluate(() => {+        const t = (document.body ? (document.body.innerText || '') : '').trim();+        return {+          title: document.title || '',+          bodyChars: t.length,+          head: t.slice(0, 400),+          elements: document.body ? document.body.querySelectorAll('*').length : 0,+          styleSheets: document.styleSheets ? document.styleSheets.length : 0,+          brokenImgs: document.images+            ? Array.from(document.images).filter(i => i.complete && i.naturalWidth === 0).length : 0,+        };+      }),+      new Promise(r => setTimeout(() => r(null), 4000)),+    ]);+    if (!probe) { out.signals.push('probe_timed_out'); return out; }+    out.title = probe.title; out.bodyChars = probe.bodyChars; out.elements = probe.elements;+    const head = String(probe.head || '');+    const low = head.toLowerCase();+    if (typeof out.httpStatus === 'number' && out.httpStatus >= 400) out.signals.push('http_' + out.httpStatus);+    if (/\b404\b|page not found|not found\b/.test(low)) out.signals.push('page_says_not_found');+    if (/cannot (get|post) \//i.test(head)) out.signals.push('dev_server_cannot_get');+    if (/internal server error|bad gateway|service unavailable/.test(low)) out.signals.push('server_error_text');+    if (/traceback \(most recent call last\)|unhandled (exception|rejection)|referenceerror|typeerror:|syntaxerror:/.test(low)) out.signals.push('stack_trace_visible');+    if (probe.bodyChars < 12 && probe.elements < 8) out.signals.push('effectively_blank');+    if (probe.bodyChars > 40 && probe.styleSheets === 0) out.signals.push('no_stylesheets_loaded');+    if (probe.brokenImgs > 0) out.signals.push('broken_images_' + probe.brokenImgs);+    out.ok = out.signals.length === 0;+    if (!out.ok) out.excerpt = head.slice(0, 180);+  } catch (e) { out.signals.push('probe_failed'); }+  return out;+} const RENDER_VERBS = new Set(['browser_open_window', 'browser_open_tab', 'browser_navigate', 'browser_reload']); // A URL that is almost certainly the caller's OWN work-in-progress (a dev server, a local file, an // Adom-hosted app) earns the strongest wording — that is exactly the case John described.@@ -5757,9 +5800,22 @@ function looksLikeOwnContent(u) {         || /^(\d{1,3}\.){3}\d{1,3}$/.test(h);   } catch (e) { return false; } }-function renderVerifyHint(sessionId, url) {+function renderVerifyHint(sessionId, url, rc) {   const sid = sessionId || '<sessionId>';   const own = looksLikeOwnContent(url);+  // A DETECTED failure outranks advice: say what is wrong in the first words the caller reads.+  if (rc && rc.ok === false && (rc.signals || []).length) {+    const sig = rc.signals.join(', ');+    const st = (typeof rc.httpStatus === 'number') ? ' (HTTP ' + rc.httpStatus + ')' : '';+    const ex = rc.excerpt ? ' The page says: "' + String(rc.excerpt).replace(/\s+/g, ' ').slice(0, 140) + '".' : '';+    return 'THIS PAGE DID NOT RENDER CORRECTLY — pup checked it, so do NOT report success. Detected: '+      + sig + st + '.' + ex+      + ' See it yourself: `adom-desktop browser_screenshot {"sessionId":"' + sid + '"}`.'+      + ' Then FIX THE CAUSE: a 404 or "Cannot GET" usually means the dev server is not running, is on a'+      + ' different port, or the path is wrong; unstyled HTML means the CSS 404d; a stack trace means the'+      + ' app threw. After fixing, `browser_reload` and re-check until this response comes back clean.'+      + ' Do not tell the user it works while this warning is present.';+  }   const lead = own     ? 'THIS LOOKS LIKE CONTENT YOU JUST BUILT OR EDITED — SCREENSHOT IT AND LOOK BEFORE YOU CLAIM IT WORKS.'     : 'VERIFY WHAT YOU JUST PUT ON SCREEN — do not assume it rendered.';