dsh-crossterm 0.8.5 → 0.8.7

package/CHANGELOG.md

@@ -1,5 +1,39 @@
 # dsh-crossterm 变更记录
 
+## 0.8.7 — 2026-10-19
+
+### 修 0.8.6 的回归：云端设备列表整个消失
+
+- **回归原因**：0.8.6 在 `/crossterm/api/devices` 路由里把 `const layout = …` 写在内部
+  `try {}` 块里，块外的 `const ownerDevice = (layout && …)` 直接抛
+  `ReferenceError: layout is not defined`，于是接口返回
+  `{"ok":false,"configured":true,"devices":[],"message":"layout is not defined"}`，
+  卡片「云端设备」一行整块消失。这类错误 0.8.6 的探针测不到 —— 它只测纯函数，没测路由。
+- **修法**：`let layout = null` 提到 `try` 外；更重要的是把「老总线镜像 + 新布局心跳」的合并逻辑
+  提成导出的纯函数 `mergeDevices({ old, fresh, selfId, ownerDevice })`（同一 nodeId 新布局覆盖老镜像、
+  `source:'layout'`、过滤本机、补角色、按心跳 `at` 倒序），路由只做 IO 和拼装。
+- **排序口径**：之前先把 `lastSeen`（ISO 字符串）和 `at`（毫秒）混在一起比，单位不一致导致顺序错乱；
+  现在 `annotateDevices()` 统一把 `at → lastSeen → updatedAt` 归一成毫秒，`mergeDevices()` 只按 `at` 倒序。
+- 探针 `probe-devices-086.mjs` 扩到 **20 项全 PASS**：新增 `mergeDevices` 九项
+  （排除本机、同 nodeId 合并、新布局字段覆盖、来源标记、补进新设备、补角色、心跳倒序、空输入、null 容忍）。
+- **要不要跟着动手：不用**（补丁只改云端设备列表的拼装与排序，装上重启即生效）。
+
+## 0.8.6 — 2026-10-19
+
+### 云端设备列表：只列别人，带权限和状态
+
+- **不显示本机**：`/crossterm/api/devices` 与卡片「云端设备」行都过滤掉本机 ——
+  本机已经在「当前终端」一栏，列在设备里等于把自己当远端。
+- **每台标角色**：`annotateDevices(list, { selfId, ownerDevice })`（`lib/web.js` 导出，纯函数可测）；
+  角色取云端 `_account.json` 的 `ownerDevice`：是它 → **管理员**，其余 → **成员**。
+- **离线时间口径修正**：以前只读 `updatedAt`，新布局心跳写的是 `at`，离线设备全显示「未知」；
+  现在按 `updatedAt → at → lastSeen` 依次取，显示「x 分钟前」。
+- **`term_list` 云端行带角色**：`lib/cloud/match.js` 的 `deviceRow()` 增 `role`；
+  `Term.cloudDevices()` 过滤本机，跨设备匹配更干净。
+- 探针 `probe-devices-086.mjs` **11 PASS**（排除本机、剔除无 nodeId、管理员/成员、字段保留、
+  at 三种来源、已有 role 不覆盖、空列表）。
+- **要不要跟着动手：不用**（纯展示改动，装上即可）。
+
 ## 0.8.5 — 2026-10-19
 
 ### 负责人留痕 + 运行日志可查（日志轮转）

package/README.md

@@ -28,33 +28,34 @@ A 终端看不到 B 终端里有哪个会话、在干什么，也没法把一条
 > `locatePkgJson()` 对子路径行返回 undefined），而插件页顶部标题与这一行共用同一份 `locale/zh.json` 的 meta，
 > 所以它单独做总开关，标题带「（总开关）」；卡片头用 `shortTitle` 显示短名「跨端互通」。
 
-**本版更新（0.8.5）**
-
-- **负责人写清楚、换人及时更正**：新增 `MAINTAINERS.md`（任期表 + 换人步骤 + 机器可读注释）。
-  当前负责人 `session-b6aa099b…`（TS-01），助手 `session-ccf34a5a…`（同端多会话2），
-  上一任是 `session-6ef6da30…`（2026-10-18 01:09 因会话故障交卸）。
-  负责人一变，插件启动时自动往 `<busDir>\logs\ownership.log` 记一行。
-- **运行日志存档、可轮转**：`logs\<nodeId>.jsonl` 超过 5 MB 自动压成
-  `logs\archive\<nodeId>-<时间戳>.jsonl.gz`，只留最近 20 份；不再担心日志把盘写满。
-- **方便调阅**：卡片底部新增「运行日志」按钮（看最近 120 行，当前文件不够时接上前一个归档）；
-  接口是 `GET /crossterm/api/logs?lines=120`（需接口令牌），同时返回归档清单与最近 10 条负责人变更。
-- 卡片新增「维护者」一栏，直接显示当前负责人（含上一任与交卸时间）。
-
-**上一版（0.8.4）**
-
-- **修统计口径**：0.8.3 卡片一度显示 `已归档 0/24 · 活跃中 38/10`，现在按「状态里有字节」算已备，
-  恢复成 `已归档 24/24 · 活跃中 10/10`。
-- 备份行为本身没变：归档会话仍然只传一次，活跃会话仍按水位增量。
-- **活跃会话照旧增量**：从 `_session.json` 的水位往后传，不会重传整份。
-
-**上一版（0.8.2）**
-
-- 死信按 20s / 1m / 5m / 15m / 1h 退避，8 次后进 `dead/`；收件人写设备名会回退「最近活跃会话」；
+**本版更新（0.8.7）**
+
+- **修 0.8.6 的回归**：0.8.6 装上后卡片「云端设备」整块消失，接口报
+  `layout is not defined`（路由里把 `layout` 声明在 `try` 块内、块外又用了它）。
+  0.8.7 已修；本来就还没升 0.8.6 的设备可以直接装 0.8.7。
+- **合并逻辑变成可测的纯函数**：`mergeDevices({ old, fresh, selfId, ownerDevice })` 负责
+  「老总线镜像 + 新布局心跳」的合并、排除本机、补管理员/成员角色、按心跳倒序，
+  路由不再自己拼装，配套 20 项单测。
+- **离线时间排序口径统一**：`annotateDevices()` 把 `at → lastSeen → updatedAt` 归一成毫秒，
+  列表按它倒序，不会再把 ISO 字符串和毫秒时间戳混着比。
+- 纯展示与拼装修复，装上重启即可，不用改配置。
+
+**上一版（0.8.6）**
+
+- **云端设备只列「别人」**：卡片里那行不再显示本机（自己已经在「当前终端」一栏），
+  只列同一账号下的其他设备。
+- **每台带权限和状态**：逐台显示 `名称 · 管理员/成员 · 在线/离线（x 分钟前）· v版本号`；
+  管理员就是云端 `_account.json` 里 `ownerDevice` 那台，其余都是成员。
+- **离线时间不再显示「未知」**：以前只认 `updatedAt`，新布局心跳写的是 `at`，
+  离线设备一律「未知」；现在按 `updatedAt → at → lastSeen` 依次取。
+- `term_list` 的云端行也带角色。
+
+**更早（0.8.5 / 0.8.2 / 0.8.0）**
+
+- 0.8.5：负责人留痕 `MAINTAINERS.md`、运行日志可轮转（5 MB 压 `logs/archive/` 留 20 份）、
+  卡片「维护者」行与「运行日志」按钮、`GET /crossterm/api/logs`。
+- 0.8.2：死信按 20s/1m/5m/15m/1h 退避，8 次后进 `dead/`；收件人写设备名会回退「最近活跃会话」；
   同一个信封不会投两次；`/crossterm/api/*` 要求本机回环 `Host`。
-
-**更早（0.8.1 / 0.8.0）**
-
-- 0.8.1：旧布局缓存缺签名密钥时会重算并补写 `relaySecret`。
 - 0.8.0 安全加固：附件名净化 + 越界校验、写接口要 `x-crossterm-token`、信封 HMAC 签名、入站限额、按 id 去重。
 
 ## 一、安装与卸载

package/client/client.js

@@ -175,13 +175,15 @@ window.__ModuleLoader__.load({
 			if (!(status && status.cloud && status.cloud.configured)) return "未连接云盘";
 			if (!devices || devices.loading) return "读取中…";
 			if (!devices.ok) return devices.message || "读取失败";
-			const list = devices.devices || [];
+			const list = (devices.devices || []).filter((d) => d && (!devices.me || d.nodeId !== devices.me));
 			if (!list.length) return "云端还没有其他设备";
 			return list
 				.map((d) => {
-					const label = (d.name || d.nodeId || "未知设备") + (d.nodeId === devices.me ? "（本机）" : "");
-					const state = d.online ? "在线" : agoText(d.updatedAt);
-					return label + " · " + state + (d.version ? " · v" + d.version : "");
+					const label = d.name || d.nodeId || "未知设备";
+					const role = d.role || "成员";
+					const at = d.updatedAt || d.at || (d.lastSeen ? Date.parse(d.lastSeen) : 0);
+					const state = d.online ? "在线" : agoText(at);
+					return label + " · " + role + " · " + state + (d.version ? " · v" + d.version : "");
 				})
 				.join("；");
 		}

package/lib/bus.js

@@ -23,7 +23,7 @@ import path from 'node:path'
 import crypto from 'node:crypto'
 import { rotateIfNeeded } from './logs.js'
 
-export const VERSION = '0.8.5'
+export const VERSION = '0.8.7'
 
 /** DSH 的家目录：与宿主其它状态（sessions / profiles）同一个根，多终端天然共享 */
 export function dshHome() {

package/lib/cloud/match.js

@@ -34,6 +34,7 @@ export function deviceRow(d) {
     hostname: d.hostname || '',
     workspace: '',
     pid: d.pid,
+    role: d.role || '',
     ageMs: Number(d.at) ? Math.max(0, Date.now() - Number(d.at)) : 0,
     online: d.online !== false,
     sessions: [],

package/lib/index.js

@@ -348,12 +348,13 @@ class CrossTerm {
 
   // ── 云端设备（跨设备投递，0.7.3 起）─────────────────────────────────
 
-  /** 云端设备列表：没连云端、没布局、或还读不到时统一回空数组 */
+  /** 云端设备列表：没连云端、没布局、或还读不到时统一回空数组（0.8.6 起不含本机） */
   async cloudDevices() {
     if (!this.home) return []
     try {
       const r = await this.home.devices()
-      return r && r.ok && Array.isArray(r.devices) ? r.devices : []
+      const list = r && r.ok && Array.isArray(r.devices) ? r.devices : []
+      return list.filter((d) => d && d.nodeId && d.nodeId !== this.nodeId)
     } catch (e) {
       this.warn(`cloudDevices: ${(e && e.message) || String(e)}`)
       return []

package/lib/web.js

@@ -8,7 +8,7 @@ import { fileURLToPath } from 'node:url'
 import { spawn } from 'node:child_process'
 import { VERSION, readConfig, writeConfig } from './bus.js'
 import { createPan } from './cloud/pan.js'
-import { listCloudDevices, localDevice } from './cloud/devices.js'
+import { listCloudDevices } from './cloud/devices.js'
 import { listPresences } from './cloud/presence.js'
 import { runSetup } from './cloud/setup.js'
 import { listLogs, tailLogs, readMaintainersDoc, listOwnership } from './logs.js'
@@ -158,6 +158,40 @@ function cloudProvider(cfg) {
   return { id: 'none', name: '未连接', configured: false }
 }
 
+/**
+ * 云端设备列表的展示口径（0.8.6 起）：
+ * - 不显示本机（自己已经在「当前终端」那一行）；
+ * - 每台标上账号里的角色：`_account.json` 的 `ownerDevice` 是管理员，其余是成员；
+ * - 保底字段：`role` / `online` / `at`（离线时卡片用它算「x 分钟前」）。
+ */
+export function annotateDevices(list, { selfId = '', ownerDevice = '' } = {}) {
+  return (Array.isArray(list) ? list : [])
+    .filter((d) => d && d.nodeId && d.nodeId !== selfId)
+    .map((d) => ({
+      ...d,
+      role: d.role || (d.nodeId === ownerDevice ? '管理员' : '成员'),
+      at: Number(d.at || 0) || (d.lastSeen ? Date.parse(d.lastSeen) || 0 : 0) || (d.updatedAt ? Number(d.updatedAt) || 0 : 0),
+    }))
+}
+
+/**
+ * 合并两路设备来源（老总线镜像 `listCloudDevices` + 新布局心跳 `listPresences`），
+ * 同一个 nodeId 以新布局为准，最后过滤本机、补角色、按心跳时间倒序。
+ * 0.8.7：把这段纯逻辑从路由里提出来 —— 0.8.6 曾在路由里把 `layout` 写成块级变量，
+ * 导致 `layout is not defined`、设备列表整个返回 ok:false，单测能挡住这类回归。
+ */
+export function mergeDevices({ old = [], fresh = [], selfId = '', ownerDevice = '' } = {}) {
+  const byId = new Map()
+  for (const d of Array.isArray(old) ? old : []) if (d && d.nodeId) byId.set(d.nodeId, d)
+  for (const d of Array.isArray(fresh) ? fresh : []) {
+    if (d && d.nodeId) byId.set(d.nodeId, { ...byId.get(d.nodeId), ...d, source: 'layout' })
+  }
+  return annotateDevices([...byId.values()], { selfId, ownerDevice }).sort(
+    // annotateDevices 已把 at 归一成毫秒时间戳（含 lastSeen / updatedAt 兜底），这里只按它倒序
+    (a, b) => Number(b.at || 0) - Number(a.at || 0),
+  )
+}
+
 /** 语义化版本比较：只比 主.次.补（忽略预发布尾巴），a>b 返回正数 */
 function compareVersion(a, b) {
   const num = (v) => String(v || '').split('-')[0].split('.').map((x) => parseInt(x, 10) || 0)
@@ -614,21 +648,21 @@ export function registerWeb(ctx, term, log) {
                 })
                 const result = await listCloudDevices(pan, cfg.cloudRoot)
                 let fresh = { devices: [], files: 0 }
+                let layout = null
                 try {
-                  const layout = term.home ? await term.home.getLayout(false) : null
+                  layout = term.home ? await term.home.getLayout(false) : null
                   if (layout) fresh = await listPresences({ pan, layout, log: (m) => log('devices: ' + m) })
                 } catch (error) {
                   log('devices: 新布局读取失败 ' + String((error && error.message) || error))
                 }
-                const byId = new Map()
-                for (const d of result.devices || []) byId.set(d.nodeId, d)
-                for (const d of fresh.devices || []) byId.set(d.nodeId, { ...byId.get(d.nodeId), ...d, source: 'layout' })
-                result.devices = [...byId.values()].sort((a, b) => Number(b.lastSeen ? Date.parse(b.lastSeen) : b.at || 0) - Number(a.lastSeen ? Date.parse(a.lastSeen) : a.at || 0))
+                // 0.8.7：合并/过滤/角色/排序都走纯函数，路由里不再直接碰 layout 的块级作用域
+                const ownerDevice = (layout && layout.accountInfo && layout.accountInfo.ownerDevice) || (layout && layout.deviceId) || ''
+                const selfId = info.nodeId || ''
+                result.devices = mergeDevices({ old: result.devices, fresh: fresh.devices, selfId, ownerDevice })
                 delete result.tmpFiles
                 result.files = (result.files || 0) + (fresh.files || 0)
-                const me = await localDevice(path.join(cfg.busDir || '', 'presence'), info.nodeId || '')
-                if (me && !result.devices.some((d) => d.nodeId === me.nodeId)) result.devices.unshift(me)
-                json(res, 200, { ...result, configured: true, me: info.nodeId || '' })
+                result.ownerDevice = ownerDevice
+                json(res, 200, { ...result, configured: true, me: selfId })
               } catch (error) {
                 json(res, 200, {
                   ok: false,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-crossterm",
-  "version": "0.8.5",
+  "version": "0.8.7",
   "description": "跨端互通：同一 DSH 账号下的会话、终端、设备之间互通 —— 特征码链、内联邮件、云端中转站与自动备份",
   "type": "module",
   "main": "./lib/index.js",