dsh-inner-browser-use 0.34.98 → 0.34.99

package/client.js

@@ -73,7 +73,7 @@ window.__ModuleLoader__.load({
       '.dshib-side-win[data-on="1"]{background:var(--dsw-alias-bg-subtle);color:var(--dsw-alias-label-primary)}',
       '.dshib-side-win > span{overflow:hidden;text-overflow:ellipsis;white-space:nowrap}',
       '.dshib-side-stage{flex:1 1 auto;position:relative;min-width:0;min-height:0;display:flex;align-items:center;justify-content:center;overflow:hidden;background:var(--dsw-alias-bg-base);outline:none;cursor:default}',
-      '.dshib-side-frame{display:block;user-select:none;-webkit-user-drag:none;cursor:default}',
+      '.dshib-side-frame{position:absolute;inset:0;display:block;width:100%;height:100%;object-fit:contain;user-select:none;-webkit-user-drag:none;cursor:default}',
       '.dshib-side-ime{position:absolute;width:1px;height:1px;padding:0;margin:0;border:0;resize:none;overflow:hidden;opacity:0;pointer-events:none}',
       /* The black arrow is the agent's pointer, not the person's cursor: both are
          on screen at once, and moving the mouse never takes the arrow over. */
@@ -645,9 +645,27 @@ window.__ModuleLoader__.load({
           .catch(() => {})
       }
 
-      const scale = size.w > 0 && size.h > 0 ? Math.min(size.w / view.w, size.h / view.h) : 0
-      const padX = (size.w - view.w * scale) / 2
-      const padY = (size.h - view.h * scale) / 2
+      /* Where the page actually got drawn inside the column: the frame fills
+         the stage and the picture sits inside it, so the answer comes from the
+         element that is on screen rather than from the last size remembered.
+         A stale measure can then never shrink the picture to a thumbnail. */
+      const frameEl = React.useRef(null)
+      const picture = () => {
+        const at = frameEl.current
+        const areaEl = stage.current
+        if (at === null || at === undefined || areaEl === null || areaEl === undefined) return null
+        if (!(view.w > 0) || !(view.h > 0)) return null
+        const a = at.getBoundingClientRect()
+        const b = areaEl.getBoundingClientRect()
+        if (!(a.width > 1) || !(a.height > 1)) return null
+        const scale = Math.min(a.width / view.w, a.height / view.h)
+        if (!(scale > 0)) return null
+        return {
+          scale,
+          x: a.left - b.left + (a.width - view.w * scale) / 2,
+          y: a.top - b.top + (a.height - view.h * scale) / 2,
+        }
+      }
 
       /* The column tells the page how big it is, a beat after the resize stops:
          a page that relayouts on every pointer step would never settle. */
@@ -729,10 +747,12 @@ window.__ModuleLoader__.load({
       }
       const toView = (event) => {
         const el = stage.current
-        if (el === null || scale <= 0) return null
+        if (el === null) return null
+        const spot = picture()
+        if (spot === null) return null
         const box = el.getBoundingClientRect()
-        const x = (event.clientX - box.left - padX) / scale
-        const y = (event.clientY - box.top - padY) / scale
+        const x = (event.clientX - box.left - spot.x) / spot.scale
+        const y = (event.clientY - box.top - spot.y) / spot.scale
         if (x < 0 || y < 0 || x > view.w || y > view.h) return null
         return { x: Math.round(x), y: Math.round(y) }
       }
@@ -828,6 +848,7 @@ window.__ModuleLoader__.load({
           h('div', { className: 'dshib-side-note' }, L('side.foreign')))
       }
       const pointer = person === true || hand === true ? null : ptr
+      const spot = pointer === null ? null : (picture() ?? { scale: 1, x: 0, y: 0 })
       const noteText = phase === 'failed' ? note || L('side.failed') : phase === 'live' ? L('side.empty') : L('side.connecting')
       return h('div', { className: 'dshib-side' },
         h('div', { className: 'dshib-side-bar' },
@@ -894,7 +915,7 @@ window.__ModuleLoader__.load({
             tabIndex: 0,
             spellCheck: false,
             'aria-label': L('side.type'),
-            style: pointer === null ? undefined : { left: `${padX + pointer.x * scale}px`, top: `${padY + pointer.y * scale}px` },
+            style: pointer === null || spot === null ? undefined : { left: `${spot.x + pointer.x * spot.scale}px`, top: `${spot.y + pointer.y * spot.scale}px` },
             onCompositionStart: () => {
               composing.current = true
             },
@@ -913,9 +934,10 @@ window.__ModuleLoader__.load({
           }),
           frame === null ? null : h('img', {
             className: 'dshib-side-frame',
+            ref: frameEl,
             src: `data:image/jpeg;base64,${frame.jpeg}`,
-            width: Math.max(1, Math.round(view.w * scale)),
-            height: Math.max(1, Math.round(view.h * scale)),
+            width: Math.max(1, Math.round(view.w)),
+            height: Math.max(1, Math.round(view.h)),
             alt: '', // ui-copy-ignore
             draggable: false,
           }),
@@ -923,7 +945,7 @@ window.__ModuleLoader__.load({
             className: 'dshib-side-ptr',
             src: SIDE_POINTER, // ui-copy-ignore
             alt: '', // ui-copy-ignore
-            style: { left: `${padX + pointer.x * scale}px`, top: `${padY + pointer.y * scale}px` },
+            style: spot === null ? undefined : { left: `${spot.x + pointer.x * spot.scale}px`, top: `${spot.y + pointer.y * spot.scale}px` },
           })))
     }
   },

package/index.js

@@ -20,7 +20,7 @@
  * learns no port and no secret. // ui-copy-ignore
  */
 
-import { appendFileSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs'
+import { appendFileSync, mkdirSync, readFileSync, statSync, writeFileSync } from 'node:fs'
 import { homedir } from 'node:os'
 import { join } from 'node:path'
 import { Sidecar } from './sidecar.js?v=1'
@@ -50,6 +50,27 @@ function writeMirror(value) {
   }
 }
 
+/** The note stops growing: past this size the oldest half goes, so a long-lived run never fills the disk. */
+const NOTES_MAX = 256 * 1024
+const NOTES_KEEP = 128 * 1024
+
+/**
+ * Add one line to the mirror's own note, keeping the file bounded.
+ * @param {string} line - what happened, one line.
+ */
+export function note(line) {
+  const file = join(STATE_DIR, 'asks.log')
+  try {
+    mkdirSync(STATE_DIR, { recursive: true })
+    appendFileSync(file, `${new Date().toISOString()} ${line}\n`)
+    if (statSync(file).size <= NOTES_MAX) return
+    const whole = readFileSync(file, 'utf8')
+    writeFileSync(file, whole.slice(whole.length - NOTES_KEEP).replace(/^[^\n]*\n/, ''))
+  } catch {
+    /* a note that cannot be written must not take the mirror down */
+  }
+}
+
 /** Route prefix owned by this plugin. Never under `/api`, which is the Host's own channel. */ // ui-copy-ignore
 export const ROUTE = '/dsh-inner-browser-use'
 
@@ -336,7 +357,7 @@ export function apply(ctx) {
         if (due !== true) return
         last = Date.now()
         bridge.want(owner, Date.now(), 'keepalive')
-        appendFileSync(join(STATE_DIR, 'asks.log'), `${new Date().toISOString()} asked ${owner.slice(-8)} to draw the mirror again\n`)
+        note(`asked ${owner.slice(-8)} to draw the mirror again`)
       } catch {}
     }, 4000)
     timer.unref?.()
@@ -511,7 +532,7 @@ export function apply(ctx) {
         if (Date.now() - before > 2000) {
           lastAsk.set(chat, Date.now())
           try {
-            appendFileSync(join(STATE_DIR, 'asks.log'), `${new Date().toISOString()} ${chat.slice(-8)} alive=${String(said?.alive === true)} viewers=${String(said?.viewers ?? 0)}\n`)
+            note(`${chat.slice(-8)} alive=${String(said?.alive === true)} viewers=${String(said?.viewers ?? 0)}`)
           } catch {}
         }
       }
@@ -715,7 +736,17 @@ export function apply(ctx) {
       const body = await readBody(req).then((raw) => JSON.parse(raw || '{}')).catch(() => ({}))
       const who = acting(body)
       if (who === '') return
-      json(200, { ok: true, forgot: bridge.forget(who) })
+      const forgot = bridge.forget(who)
+      /* Nobody is watching a picture the person just closed: the browser itself
+         is put away a minute later rather than a quarter of an hour, unless an
+         agent asks for it again in the meantime. */
+      try {
+        if (sidecar.watched() === 0) {
+          sidecar.touch(REST_MS)
+          note('said ' + who.slice(-8) + ' closed the mirror - browser rests in a minute')
+        }
+      } catch { /* a browser that never started needs no rest timer */ }
+      json(200, { ok: true, forgot })
       return
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-inner-browser-use",
-  "version": "0.34.98",
+  "version": "0.34.99",
   "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

@@ -95,6 +95,12 @@ export function nextQuality(current, base64) {
 
 /** Idle lifetime, so a forgotten sidecar does not sit on a login forever. */
 const IDLE_MS = 15 * 60_000
+/* A pane the person closed is a stronger signal than a pane nobody touched: the
+   browser is put away a minute later unless someone asks for it again. A whole
+   Chrome with its renderers is a few hundred megabytes, and leaving it up for a
+   quarter of an hour after a deliberate close is exactly the cost they did not
+   want. */
+const REST_MS = 60_000
 /* One browser serves every chat, but a chat that is not on screen need not keep
    a full renderer alive for ever: past this many chats the oldest unwatched tab
    is closed, and its address brings it back on the next visit. */
@@ -2836,7 +2842,18 @@ export class Sidecar {
     }
   }
 
-  touch() {
+  /** @returns {number} how many panes are watching a picture right now. */
+  watched() {
+    let n = this.page?.frames?.size ?? 0
+    for (const page of this.pages?.values() ?? []) n += page.frames?.size ?? 0
+    return n
+  }
+
+  /**
+   * Starts the idle clock again.
+   * @param {number} [ms] - how long a browser nobody watches may stay up.
+   */
+  touch(ms = IDLE_MS) {
     if (this.idle !== null) clearTimeout(this.idle)
     this.idle = setTimeout(() => {
       this.idle = null
@@ -2849,7 +2866,7 @@ export class Sidecar {
         return
       }
       void this.shutdown()
-    }, IDLE_MS)
+    }, ms)
     this.idle.unref?.()
   }