dsh-inner-browser-use 0.35.0 → 0.35.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-inner-browser-use",
-  "version": "0.35.0",
+  "version": "0.35.1",
   "description": "An agent-owned Chromium sidecar mirrored into the DeepSeek Harness right column: the agent drives it, the person watches and can take the mouse, and logins survive restarts.",
   "type": "module",
   "main": "index.js",

package/sidecar.js

@@ -826,18 +826,17 @@ export class SidecarPage {
        probe is only for the case where the click never made one, so it gets a
        single short look instead of the old half-second of polling. */
     if (target === null) {
-      /* Look, wait for the announcement, look again: a new tab is announced
-         before the browser lists it, so one look is not enough. */
-      const deadline = Date.now() + POPUP_GRACE_MS
-      while (target === null && Date.now() < deadline) {
-        const found = await this.findPopup(foreign)
-        if (found !== null) {
-          target = found
-          break
+      /* One look at the tab list answers the common case — a click that opened
+         nothing. When it says nothing, the announcement is the cheaper door:
+         a tab exists a moment before the browser lists it, so the wait ends on
+         the event instead of running out the window. */
+      target = await this.findPopup(foreign)
+      if (target === null) {
+        await this.waitPopup(Math.min(8, POPUP_GRACE_MS))
+        if (this.opened !== null && this.opened !== undefined && Date.now() - Number(this.opened.at ?? 0) < 2000) {
+          target = { targetId: String(this.opened.targetId), url: String(this.opened.url) }
         }
-        const left = deadline - Date.now()
-        if (left <= 0) break
-        await this.waitPopup(Math.min(8, left))
+        this.opened = null
       }
     }
     if (target === null) {

package/tools.js

@@ -310,7 +310,17 @@ export async function outcome(page, before) {
   const [wasTitle, wasUrl, wasText, wasNodes] = before.split('|')
   const [nowTitle, nowUrl, nowText, nowNodes] = after.split('|')
   if (after === before) return ' · no visible change'
-  if (nowUrl !== wasUrl) return ` · navigated to ${String(nowUrl).slice(0, 300)}`
+  if (nowUrl !== wasUrl) {
+    /* The brief on the same line already names the address this step went to.
+       What it cannot say is whether the page moved within itself, so a jump to
+       another spot on the same address is said as the spot alone. */
+    const was = String(wasUrl ?? '')
+    const now = String(nowUrl ?? '')
+    const base = (value) => (value.includes('#') ? value.slice(0, value.indexOf('#')) : value)
+    const spot = now.includes('#') ? now.slice(now.indexOf('#')) : ''
+    if (spot !== '' && base(was) === base(now)) return ` · ${spot.slice(0, 120)}`
+    return ' · new page'
+  }
   const words = Math.abs(Number(nowText ?? 0) - Number(wasText ?? 0))
   const nodes = Math.abs(Number(nowNodes ?? 0) - Number(wasNodes ?? 0))
   if (nowTitle !== wasTitle) return ' · page moved on'
@@ -352,9 +362,27 @@ export function withoutSeenAddress(page, url, text) {
   return body.slice(first.length + 1)
 }
 
+/** The focus line repeats the control the call named: say the name once. */
+export function dropSameFocus(line, args) {
+  const squeezed = (value) => String(value ?? '').toLowerCase().replace(/\s+/g, '')
+  const named = squeezed(args?.text ?? args?.name ?? args?.ref ?? '')
+  if (named === '') return line
+  const at = line.indexOf(' · focus ')
+  if (at < 0) return line
+  const head = line.slice(0, at)
+  const parts = line.slice(at + ' · focus '.length).split(' · ')
+  const said = String(parts[0] ?? '')
+  /* A field carries a value the words do not: that part is news, so it stays. */
+  if (said.includes(' = ')) return line
+  const label = squeezed(said)
+  if (label !== named && !label.includes(named) && !named.includes(label)) return line
+  const rest = parts.slice(1)
+  return rest.length === 0 ? head : `${head} · ${rest.join(' · ')}`
+}
+
 async function acted(page, args, look, label) {
   if (args?.read === true) return await look()
-  return `ok ${label} · ${oncePerPage(page, await brief(page))}`
+  return `ok ${label} · ${dropSameFocus(oncePerPage(page, await brief(page)), args)}`
 }
 
 /**