dsh-plugin-desktop-notify 0.6.4 → 0.6.5

package/README.md

@@ -16,7 +16,7 @@
 - **等待你操作**——工具审批、提问、执行计划确认
 - **任务失败**——这一轮出错时提示
 - **说清楚怎么结束的**——正常完成、被拦截、输出到上限、被中止，各自一句话，不把中断说成「已完成」
-- **需要你的排在前面**——运行中、待你审批/回答、出错、未读完成的会话，自动排到**手动置顶的下面、其余会话的上面**，处理完自动放回原位
+- **还在进行的排在前面**——正在跑、等你审批/回答、刚出错 的会话排到**手动置顶的下面、其余会话的上面**，处理完自动放回原位；完成未读只进小红点，不搬动列表
 - **小红点**——应用图标上显示「需要你的会话数」，你看过之后才清零
 - **点击通知**——切到对应会话，而不是只把应用叫到前面
 
@@ -46,7 +46,7 @@ dsh-plugin-desktop-notify
 | `notifyViewed` | `false` | 正在看的会话也通知（默认跳过） |
 | `badge` | `true` | 显示小红点 |
 | `excludeSubagents` | `true` | 忽略子智能体会话 |
-| `orderAttention` | `true` | 把需要你处理的会话排到手动置顶下面第一条（不改置顶状态；关掉后完全不碰排序） |
+| `orderAttention` | `true` | 把仍在进行中的会话排到手动置顶下面（不改置顶状态；关掉后完全不碰排序） |
 | `titleComplete` / `titlePending` / `titleError` | 空 | 自定义标题，空 = 用内置文案 |
 | `maxBodyChars` | `140` | 正文长度上限 |
 
@@ -112,7 +112,8 @@ urllib.request.urlopen(request, timeout=3).read()
 
 - 重启或刷新窗口时，已经存在的「未读 / 待审批」只恢复小红点，不再补弹横幅
 - 这个排序是插件接管「手动排序」实现的：插件只写「需要你」那几条的顺序，其余会话仍然按最近更新往后排，
-  所以视觉上和「最近更新」一样，只是多了一条需要你的带状区域。代价是侧栏的排序方式会显示成「手动」，
+  所以视觉上和「最近更新」一样，只是多了一条仍在进行中的带状区域。切换窗口焦点不会触发重排。
+  代价是侧栏的排序方式会显示成「手动」，
   而且你把会话拖动排序后，下一次有会话需要你时会被重排覆盖。不想要就 `orderAttention: false`
 
 - **Windows**：点通知会切到对应会话，但窗口不一定到前台——`AppUserModelID` 和主进程 focus 通道属于桌面端，渲染层插件做不到

package/lib/client.js

@@ -30,7 +30,7 @@ window.__ModuleLoader__.load({
   factory: (require) => {
     const NS = "dsh-plugin-desktop-notify"
     /** Reported by capabilities(); keep in step with package.json (cross-machine support). */
-    const VERSION = "0.6.4"
+    const VERSION = "0.6.5"
     const PREFERENCES_KEY = `${NS}:preferences`
 
     /** Defaults; override per field with the localStorage JSON described in README. */
@@ -374,6 +374,9 @@ window.__ModuleLoader__.load({
         const snapshot = workspaceSnapshot()
         const list = ctx.sessions.list.getSnapshot()
         if (snapshot === undefined || list?.phase !== "ready") return
+        // The workspace registry hydrates separately; writing before it is ready
+        // would file every session under "" and then rewrite them a moment later.
+        if (snapshot.phase !== "ready" || snapshot.state === "loading") return
         const archived = new Set(Array.isArray(snapshot.archivedSessionIds) ? snapshot.archivedSessionIds : [])
         const live = tier.filter((sessionId) => !archived.has(sessionId))
         const active = preferences.orderAttention && live.length > 0
@@ -393,7 +396,10 @@ window.__ModuleLoader__.load({
           const members = new Set(memberIds)
           const order = active ? live.filter((sessionId) => members.has(sessionId)) : []
           const signature = order.join("\n")
+          // Never touch a group this plugin has not seated before: writing an
+          // empty order there would wipe the user's own drag arrangement.
           if (lastOrderByKey.get(key) === signature) continue
+          if (signature === "" && !lastOrderByKey.has(key)) continue
           try {
             view.setSessionOrder(key, order, {})
             lastOrderByKey.set(key, signature)
@@ -716,10 +722,14 @@ window.__ModuleLoader__.load({
             })
           }
 
-          // The tier follows the same facts the banner and badge use, plus a
-          // running turn: anything that would make you look belongs on top.
+          // Only what is still in flight is seated: a running turn, a question
+          // waiting on you, or a failure. An unread completion stays in the
+          // badge instead — promoting it would move the list every time you
+          // read a session, and it is exactly what made the band churn.
+          // Focus is deliberately not part of this: a running turn keeps its
+          // seat while you tab away, so the band never reshuffles on blur.
           const current = attention.get(sessionId) ?? {}
-          if (!looking && (running || current.completed === true || current.pendingKind !== undefined || current.error !== undefined)) {
+          if (running || pendingKind !== undefined || current.error !== undefined) {
             wanted.add(sessionId)
           }
 
@@ -823,6 +833,21 @@ window.__ModuleLoader__.load({
         }
       }, "dsh-plugin-desktop-notify: badge release")
 
+      /* Give the sidebar back the order this plugin took when it unloads. */
+      ctx.effect(() => () => {
+        const view = ctx.uiWorkspace?.view
+        if (typeof view?.setSessionOrder !== "function") return
+        for (const [key, signature] of lastOrderByKey) {
+          if (signature === "") continue
+          try {
+            view.setSessionOrder(key, [], {})
+          } catch {
+            /* a torn-down view keeps whatever it had */
+          }
+        }
+        lastOrderByKey.clear()
+      }, "dsh-plugin-desktop-notify: order release")
+
       /*
        * Diagnostic surface for DevTools (⌘⌥I):
        *   __DSH_DESKTOP_NOTIFY__.capabilities()      — what this renderer supports

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-plugin-desktop-notify",
-  "version": "0.6.4",
+  "version": "0.6.5",
   "type": "module",
   "description": "Desktop notifications and a Dock/taskbar badge for DeepSeek Harness: a finished turn, a pending approval or question, and a failure, with click-through back to the session",
   "license": "MIT",