dsh-wisp 1.33.1 → 1.33.2

package/README.md

@@ -448,6 +448,34 @@ node tools/audit-log.mjs --confirm <manifest>     # 生成后：并入代理登
 
 ## 变更
 
+### 1.33.2
+
+**修「她在屏幕角落时，子菜单跟着偏」。** 现象是：右下 / 左下时子面板整体**偏下**，
+右上时**压住主菜单**一部分。
+
+根因只有一句：**我用了菜单"请求"的位置，而不是"实际"的位置。**
+
+菜单在视口边缘会被**夹取**（`clamp` 进 `vw()/vh()`）：
+
+- 她在**右下 / 左下**时，菜单会被**往上推** → 实际 `top` 比请求的小 → 子面板却按请求的 `top` 算 → **偏下**；
+- 她在**右上**时，菜单会被**往左推** → 实际 `left` 比请求的小 → 子面板按请求的 `left` 算 → 横向也偏 → **压住菜单**。
+
+而且越靠角落、夹取越多、偏得越厉害 —— 正是观察到的规律。
+
+改法就是读`menuEl.style.left/top`（菜单真实的图层坐标，夹取之后的值），
+横向仍用"菜单宽度 + 面板宽度"、纵向仍用"行相对菜单的偏移"，这两者本来就与坐标系无关。
+
+**顺便把这道防线补成四个角落各测一遍**（`tools/smoke.mjs`）：她先站到每一角，
+再开菜单、悬停分组行、量三个矩形的相对关系，断言
+「面板在菜单**旁边**」「在视口内」「与它所属那一行**对齐**」。
+用户就是这样发现问题的，所以这条检查直接照着他的操作路径写。
+
+实测四个角落：面板始终在菜单旁边（右侧角落翻到左边），`panel.t` 与 `row.t` **完全相等**。
+
+冒烟 57 → **61 项**。
+
+### 1.33.1
+
 ### 1.33.1
 
 **修「子菜单遮住主菜单」。** 上一版的定位是错的，而且错得不显眼 —— 面板确实"在旁边"，

package/lib/client.js

@@ -51,7 +51,7 @@ window.__ModuleLoader__.load({
     var exports = module.exports
     Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' })
 
-    const VERSION = '1.33.1'
+    const VERSION = '1.33.2'
 
 /* 每一版一句话。**必须写当前版本** —— 预检里有一条断言盯着它，
    忘了更新就会红，不会静默过期。 */
@@ -75,6 +75,7 @@ const WHATS_NEW = {
   '1.32.0': '分组改成鼠标悬停就展开、移开就收起（点一下则是钉住）。',
   '1.33.0': '子菜单改成在旁边弹出（真正的菜单栏形状），→/← 进出自如。',
   '1.33.1': '修子菜单位置算错、会压住主菜单的问题。',
+  '1.33.2': '修她在屏幕四角时子菜单跟着偏的问题。',
 }
 
 /* 更新这件事**这个插件自己做不到**，如实写在代码里，免得以后有人以为能：
@@ -2286,8 +2287,12 @@ body[data-ds-dark-theme] .wisp-layer{--wisp-say-fg:#eaf2f8;--wisp-say-bg:rgba(20
           const mr = menuEl && menuEl.getBoundingClientRect ? menuEl.getBoundingClientRect() : null
           if (mr && mr.width > 0) menuW = mr.width
         } catch (e) { /* 用常量兜底 */ }
-        const menuLeft = menuAtX
-        const menuTop = menuAtY
+        /* 菜单的**实际**位置，不是"请求"的位置（menuAtX/menuAtY）。
+           菜单会被夹进视口：她站在右下/左下时菜单被往上推，站在右上时被往左推。
+           用请求坐标算出面板位置，就会跟着偏 —— 而且越靠角落偏得越多
+           （这正是"她在右下角时子菜单更偏下、右上角时压住菜单"的原因）。 */
+        const menuLeft = menuEl ? (Number.parseFloat(menuEl.style.left) || menuAtX) : menuAtX
+        const menuTop = menuEl ? (Number.parseFloat(menuEl.style.top) || menuAtY) : menuAtY
         let x = menuLeft + menuW + 2
         flyoutSide = 'right'
         if (x + SUBMENU_W > vw() - 8) {

package/lib/client.template.js

@@ -51,7 +51,7 @@ window.__ModuleLoader__.load({
     var exports = module.exports
     Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' })
 
-    const VERSION = '1.33.1'
+    const VERSION = '1.33.2'
 
 /* 每一版一句话。**必须写当前版本** —— 预检里有一条断言盯着它，
    忘了更新就会红，不会静默过期。 */
@@ -75,6 +75,7 @@ const WHATS_NEW = {
   '1.32.0': '分组改成鼠标悬停就展开、移开就收起（点一下则是钉住）。',
   '1.33.0': '子菜单改成在旁边弹出（真正的菜单栏形状），→/← 进出自如。',
   '1.33.1': '修子菜单位置算错、会压住主菜单的问题。',
+  '1.33.2': '修她在屏幕四角时子菜单跟着偏的问题。',
 }
 
 /* 更新这件事**这个插件自己做不到**，如实写在代码里，免得以后有人以为能：
@@ -2235,8 +2236,12 @@ body[data-ds-dark-theme] .wisp-layer{--wisp-say-fg:#eaf2f8;--wisp-say-bg:rgba(20
           const mr = menuEl && menuEl.getBoundingClientRect ? menuEl.getBoundingClientRect() : null
           if (mr && mr.width > 0) menuW = mr.width
         } catch (e) { /* 用常量兜底 */ }
-        const menuLeft = menuAtX
-        const menuTop = menuAtY
+        /* 菜单的**实际**位置，不是"请求"的位置（menuAtX/menuAtY）。
+           菜单会被夹进视口：她站在右下/左下时菜单被往上推，站在右上时被往左推。
+           用请求坐标算出面板位置，就会跟着偏 —— 而且越靠角落偏得越多
+           （这正是"她在右下角时子菜单更偏下、右上角时压住菜单"的原因）。 */
+        const menuLeft = menuEl ? (Number.parseFloat(menuEl.style.left) || menuAtX) : menuAtX
+        const menuTop = menuEl ? (Number.parseFloat(menuEl.style.top) || menuAtY) : menuAtY
         let x = menuLeft + menuW + 2
         flyoutSide = 'right'
         if (x + SUBMENU_W > vw() - 8) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-wisp",
-  "version": "1.33.1",
+  "version": "1.33.2",
   "description": "DeepSeek娘 — an unofficial floating desktop companion for the DeepSeek Harness UI. Zero dependencies; sprites inlined as data URIs; schedules on the Client timer service instead of the trapped browser timer globals.",
   "type": "module",
   "main": "lib/index.js",

package/tools/smoke.mjs

@@ -581,61 +581,64 @@ check(skinSwap.layersDuringFade === 1, 'the cross-fade layer is gone once it set
 // 每个页面都要装下这份包与解码后的精灵图（单张就是 naturalWidth*naturalHeight*4 字节），
 // 多开一个页面 = 多一份；之前正是这样把浏览器推崩的，表现为随机的
 // "Target page, context or browser has been closed"，极难定位。
-/* 级联子菜单的几何检查：必须在**页面还活着**的时候跑（它要量真实布局），
-   也必须在键盘段之后（它会开关菜单、动焦点）。 */
-/* 级联子菜单的几何检查放在**最后**：它会开关菜单、动焦点，放在中间会扰动后面的键盘段。 */
-/* 级联子菜单的**几何**：它在旁边，不在上面。
-   这一条替身测不到（假 DOM 没有布局），而"子菜单遮住主菜单"那个 bug 正好出在这里 ——
-   混用了视口坐标（getBoundingClientRect）与图层坐标（菜单的 left/top）。 */
-const flyoutGeo = await page.evaluate(async () => {
-  const body = document.querySelector('.wisp-body')
-  const box = document.querySelector('.wisp-root')
-  if (!body || !box) return { noBody: true }
-  const r0 = box.getBoundingClientRect()
-  body.dispatchEvent(new MouseEvent('contextmenu', {
-    bubbles: true, cancelable: true, clientX: r0.left + r0.width / 2, clientY: r0.top + r0.height / 2,
-  }))
-  const menu = document.querySelector('.wisp-menu')
-  if (!menu) return { noMenu: true }
-  const row = [...menu.querySelectorAll('.wisp-menu-item')].find((el) => el.dataset && el.dataset.group)
-  if (!row) return { noRow: true }
-  row.dispatchEvent(new PointerEvent('pointerenter', { pointerType: 'mouse' }))
-  await new Promise((resolve) => setTimeout(resolve, 350))
-  const panel = document.querySelector('.wisp-submenu')
-  if (!panel) return { noPanel: true }
-  const mr = menu.getBoundingClientRect()
-  const pr = panel.getBoundingClientRect()
-  const rr = row.getBoundingClientRect()
-  const layer = menu.parentNode
-  const lr = layer.getBoundingClientRect ? layer.getBoundingClientRect() : { left: -999, top: -999, width: 0, height: 0 }
-  return {
-    menu: { l: Math.round(mr.left), r: Math.round(mr.right), w: Math.round(mr.width), styleL: menu.style.left, styleT: menu.style.top },
-    panel: { l: Math.round(pr.left), r: Math.round(pr.right), t: Math.round(pr.top), w: Math.round(pr.width), styleL: panel.style.left, styleT: panel.style.top },
-    row: { t: Math.round(rr.top), h: Math.round(rr.height) },
-    layer: { l: Math.round(lr.left), t: Math.round(lr.top), w: Math.round(lr.width), h: Math.round(lr.height), pos: getComputedStyle(layer).position },
-    view: { w: window.innerWidth, h: window.innerHeight },
-    rows: panel.querySelectorAll('.wisp-menu-item').length,
-  }
-})
-check(flyoutGeo.panel !== undefined && flyoutGeo.rows > 0,
-  'hovering a group row pops a submenu panel', JSON.stringify(flyoutGeo))
-check(flyoutGeo.panel !== undefined
-  && (flyoutGeo.panel.l >= flyoutGeo.menu.r - 6 || flyoutGeo.panel.r <= flyoutGeo.menu.l + 6),
-  'the submenu sits BESIDE the menu, not on top of it — window coords and layer coords must not be mixed',
-  flyoutGeo.panel ? `menu=${flyoutGeo.menu.l}..${flyoutGeo.menu.r} panel=${flyoutGeo.panel.l}..${flyoutGeo.panel.r}` : 'n/a')
-check(flyoutGeo.panel !== undefined
-  && flyoutGeo.panel.l >= 0 && flyoutGeo.panel.r <= flyoutGeo.view.w
-  && flyoutGeo.panel.t >= 0,
-  'and it fits on screen', flyoutGeo.panel ? JSON.stringify(flyoutGeo.panel) : 'n/a')
-check(flyoutGeo.panel !== undefined && Math.abs(flyoutGeo.panel.t - flyoutGeo.row.t) < 40,
-  'and lines up with the row it belongs to',
-  flyoutGeo.panel ? `row.t=${flyoutGeo.row.t} panel.t=${flyoutGeo.panel.t}` : 'n/a')
-await page.evaluate(() => {
-  /* 菜单的键盘监听挂在 **window** 上，不是 document —— 发错目标菜单就不会关，
-     后面的键盘段会在"一个还没关掉的菜单"上重开，焦点状态全乱（就是这么挂掉三条的）。 */
-  window.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape' }))
-  window.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape' }))
-})
+/* 级联子菜单的几何：**四个角落各测一次**。
+   她在角落里时菜单会被夹进视口（右下/左下往上推、右上往左推），
+   子面板必须跟着菜单的**实际**位置走 —— 用"请求坐标"算就会越靠角落偏得越多，
+   严重时压住主菜单。用户就是这样发现该 bug 的，所以这里直接把它编成回归检查。
+   假 DOM 没有布局，这一整类问题只有真浏览器能测。 */
+const cornerGeo = []
+for (const which of ['br', 'bl', 'tr', 'tl']) {
+  const geo = await page.evaluate(async (corner) => {
+    const wisp = window.__wisp
+    const box = document.querySelector('.wisp-root')
+    const bw = parseFloat(box.style.width) || 560
+    const bh = parseFloat(box.style.height) || 840
+    const vw = window.innerWidth
+    const vh = window.innerHeight
+    const right = (corner === 'br' || corner === 'tr') ? 26 : Math.max(0, vw - 26 - bw)
+    const bottom = (corner === 'br' || corner === 'bl') ? 46 : Math.max(0, vh - 46 - bh)
+    wisp.configure({ right, bottom })
+    wisp.resetPosition()
+    await new Promise((resolve) => setTimeout(resolve, 120))
+    /* 主菜单的入口是 .wisp-body 上的 Enter —— contextmenu 只挂在"躲起来"之后的迷你标签上。 */
+    const body = document.querySelector('.wisp-body')
+    body.dispatchEvent(new KeyboardEvent('keydown', { key: 'Enter', bubbles: true, cancelable: true }))
+    const menu = document.querySelector('.wisp-menu')
+    if (!menu) return { which: corner, noMenu: true }
+    const row = [...menu.querySelectorAll('.wisp-menu-item')].find((el) => el.dataset && el.dataset.group)
+    if (!row) return { which: corner, noRow: true }
+    row.dispatchEvent(new PointerEvent('pointerenter', { pointerType: 'mouse' }))
+    await new Promise((resolve) => setTimeout(resolve, 320))
+    const panel = document.querySelector('.wisp-submenu')
+    if (!panel) return { which: corner, noPanel: true }
+    const mr = menu.getBoundingClientRect()
+    const pr = panel.getBoundingClientRect()
+    const rr = row.getBoundingClientRect()
+    window.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape' }))
+    window.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape' }))
+    return {
+      which: corner,
+      menu: { l: Math.round(mr.left), r: Math.round(mr.right), t: Math.round(mr.top) },
+      panel: { l: Math.round(pr.left), r: Math.round(pr.right), t: Math.round(pr.top) },
+      row: { t: Math.round(rr.top) },
+      vw, vh,
+    }
+  }, which)
+  cornerGeo.push(geo)
+}
+const withPanel = cornerGeo.filter((g) => g && g.panel)
+check(withPanel.length === 4, 'the submenu opens from every corner',
+  cornerGeo.map((g) => g.which + (g.panel ? '' : '✗')).join(' '))
+check(withPanel.every((g) => g.panel.r <= g.menu.l + 6 || g.panel.l >= g.menu.r - 6),
+  'BESIDE the menu in every corner — never on top of it',
+  withPanel.map((g) => `${g.which} menu=${g.menu.l}..${g.menu.r} panel=${g.panel.l}..${g.panel.r}`).join(' | '))
+check(withPanel.every((g) => g.panel.l >= 0 && g.panel.r <= g.vw && g.panel.t >= 0 && g.panel.t < g.vh),
+  'and inside the viewport in every corner',
+  withPanel.map((g) => `${g.which} l=${g.panel.l} r=${g.panel.r} t=${g.panel.t}`).join(' | '))
+check(withPanel.every((g) => Math.abs(g.panel.t - g.row.t) < 40),
+  'and lined up with its own row in every corner',
+  withPanel.map((g) => `${g.which} row.t=${g.row.t} panel.t=${g.panel.t}`).join(' | '))
+
 
 await browser.close()