@coclaw/openclaw-coclaw 0.26.5 → 0.26.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@coclaw/openclaw-coclaw",
-  "version": "0.26.5",
+  "version": "0.26.6",
   "type": "module",
   "license": "Apache-2.0",
   "description": "OpenClaw plugin for remote chat over WebRTC. Run `openclaw coclaw enroll` after install.",

package/src/auto-upgrade/state.js

@@ -7,6 +7,11 @@
  * 2. OPENCLAW_STATE_DIR 环境变量（worker 子进程，由 spawner 传入）
  * 3. ~/.openclaw（兜底默认值）
  *
+ * 注：第 3 档（homedir）实际够不着——gateway 进程内 runtime 必在（full mode 才启
+ * scheduler）、worker 进程 spawner 总会传 OPENCLAW_STATE_DIR，故不存在“worker 写错盘”。
+ * 曾被对抗式 review 当 bug 捞出，核实为不可达防御，记此免再捞。
+ * 同理“runtime 非空但 .state 缺失”的部分注入形态也不可达，勿为对齐 claw-paths.js 改成抛错。
+ *
  * 锁策略：state 文件的 read-modify-write 统一走 stateMutex；纯读不加锁。
  * 锁只能护住同进程内并发（gateway 与 worker 跨进程仍各写各的），但 worker /
  * scheduler 各自内部是串行流，进程内互斥已足够。
@@ -24,8 +29,13 @@ const STATE_FILENAME = 'upgrade-state.json';
 const LOG_FILENAME = 'upgrade-log.jsonl';
 const LOG_MAX_LINES = 200;
 const LOG_KEEP_LINES = 100;
-// lastUpgrade.error 截断上限：远端上报行不宜过长；jsonl 保留完整 error
-const ERROR_MAX_CHARS = 500;
+// lastUpgrade.error 截断上限：远端上报行不宜过长；jsonl 保留完整 error。
+// 取 1600 是为 ≥ worker.js formatCmdFailure 复合串的理论最大长度（≈1547：
+// prefix + message/stdout/stderr 三段各尾截 500 + 标签与 " | " 分隔符开销），
+// 保证 worker 格式化的失败串绝不被 state.js 二次全局截断而切掉中段真因；
+// 这里不解析 worker 的分段格式（保持两者解耦），只保留一道防御性硬上限——
+// state.js 的 error 还来自 'boom'/'timeout'/verify 等其它短串，需防意外超长串无界落盘。
+const ERROR_MAX_CHARS = 1600;
 
 const stateMutex = createMutex();
 const logMutex = createMutex();

package/src/auto-upgrade/updater.js

@@ -582,8 +582,12 @@ export class AutoUpgradeScheduler {
 				);
 			}
 
-			// 来源验明 npm 后才上报 available——否则永不升级的装置每小时刷一条
-			remoteLog(`upgrade.available from=${result.currentVersion} to=${result.latestVersion}`);
+			// 来源验明 npm 后才上报 available——否则永不升级的装置每小时刷一条；
+			// available 同为稳定态（直到升级落地或新版发布），与兄弟信号同模式按 (原因, toVersion) 去重防刷屏
+			this.__gateSignalOnce(
+				`available|${result.latestVersion}`,
+				`upgrade.available from=${result.currentVersion} to=${result.latestVersion}`,
+			);
 			this.__logger.info?.(`[auto-upgrade] Update available: ${result.currentVersion} → ${result.latestVersion}`);
 
 			// installPath 取自权威记录（新鲜）；缺失时回退自推包根，

package/src/session-manager/manager.js

@@ -32,7 +32,6 @@ async function safeReaddir(dir) {
 	try { return await fsp.readdir(dir); }
 	catch (err) {
 		if (err.code === 'ENOENT' || err.code === 'ENOTDIR') return [];
-		/* c8 ignore next 2 -- 非 ENOENT/ENOTDIR 的 fs 错误按原样上抛 */
 		throw err;
 	}
 }
@@ -45,7 +44,6 @@ async function safeAccess(filePath) {
 	try { await fsp.access(filePath); return true; }
 	catch (err) {
 		if (err.code === 'ENOENT' || err.code === 'ENOTDIR') return false;
-		/* c8 ignore next 2 -- 非 ENOENT/ENOTDIR 的 fs 错误按原样上抛 */
 		throw err;
 	}
 }
@@ -150,12 +148,10 @@ export function createSessionManager(options = {}) {
 			const full = nodePath.join(dir, file);
 			let stat;
 			try { stat = await fsp.stat(full); }
-			/* c8 ignore start -- readdir→stat race window：文件被并发删除时跳过 */
 			catch (err) {
 				if (err.code === 'ENOENT') continue;
 				throw err;
 			}
-			/* c8 ignore stop */
 			const row = {
 				sessionId: parsed.sessionId,
 				sessionKey: sessionKeyById.get(parsed.sessionId) ?? null,
@@ -224,12 +220,10 @@ export function createSessionManager(options = {}) {
 			const full = nodePath.join(dir, name);
 			let stat;
 			try { stat = await fsp.stat(full); }
-			/* c8 ignore start -- readdir→stat race window */
 			catch (err) {
 				if (err.code === 'ENOENT') continue;
 				throw err;
 			}
-			/* c8 ignore stop */
 			candidates.push({
 				path: full,
 				archiveStamp,
@@ -240,7 +234,6 @@ export function createSessionManager(options = {}) {
 			if (a.archiveStamp !== b.archiveStamp) {
 				return b.archiveStamp.localeCompare(a.archiveStamp);
 			}
-			/* c8 ignore next -- 同 sessionId 同 archiveStamp 的归档实测 0 case */
 			return b.updatedAt - a.updatedAt;
 		});
 		if (candidates.length > 0) {

package/src/utils/file-backed-queue.js

@@ -486,6 +486,7 @@ class FileBackedQueue {
 			/* c8 ignore next 2 -- rm with force rarely fails */
 			this.logger?.warn?.('fbq.handleFsError rm error', rmErr);
 		}
+		// 故意不 __dispatchSpillEnd：webrtc 只 destroy 不 clear + fsBroken 粘性不再 spill，spillActive 卡 true 不可达；即便发生也仅 monitor 观测标志失真，不丢消息
 		this.spilled = false;
 		this.writtenBytes = 0;
 		this.readOffset = 0;

package/src/webrtc/webrtc-peer.js

@@ -729,13 +729,13 @@ export class WebRtcPeer {
 		// 连接状态变更（校验 pc 归属，防止旧 PC 异步回调删除新 session）
 		pc.onconnectionstatechange = () => {
 			const state = pc.connectionState;
-			this.__remoteLog(`rtc.state conn=${connId} ${state}`);
-			this.logger.info?.(`${this.__rtcTag} [${connId}] connectionState: ${state}`);
-
 			// 校验 pc 归属：旧 PC 的异步回调可能在新 session 已建立后触发
 			const cur = this.__sessions.get(connId);
 			if (!cur || cur.pc !== pc) return;
 
+			this.__remoteLog(`rtc.state conn=${connId} ${state}`);
+			this.logger.info?.(`${this.__rtcTag} [${connId}] connectionState: ${state}`);
+
 			// 离开 failed 状态时清理 TTL timer（ICE restart 恢复、自然关闭等）
 			if (state !== 'failed' && cur.__failedTimer) {
 				clearTimeout(cur.__failedTimer);
@@ -829,7 +829,7 @@ export class WebRtcPeer {
 				// 后变 fire-and-forget（WebRTC 实现的 ondatachannel 是 sync 回调，不能 await）
 				session.rpcChannel = channel;
 				this.__setupDataChannel(connId, channel).catch((err) => {
-					/* c8 ignore next 2 -- setup 内部已经 try/catch 所有 await；此处仅防御性兜底 */
+					/* c8 ignore next 2 -- async 装配段（FBQ 构造+init）无内部 try/catch，此 catch 是其抛错的真实吞没点（非"防御兜底"）；生产不可达（仅 server 违反 connId 契约触发）暂未修，详见 TODO「__setupDataChannel 装配抛错 → rpcQueue 半残静默丢消息」 */
 					this.logger.warn?.(`${this.__rtcTag} [${connId}] __setupDataChannel error: ${err?.message}`);
 				});
 			} else if (channel.label.startsWith('file:')) {
@@ -879,6 +879,8 @@ export class WebRtcPeer {
 			// 注入 __onRequest 或 enqueue 到新 rpcQueue。session 已删除时（rpcChannel===null 或
 			// sess undefined）也按 stale 处理，避免向已清空的 session 注入消息。
 			const sess = this.__sessions.get(connId);
+			// 身份守卫故意放在派发处（而非 dc.onmessage 入口）：reassembler 每 dc 独立、陈旧 chunk 派发前
+			// 无害，放此处可挡住所有通向派发的路径（含未来 reassembler 定时刷新等非 onmessage 旁路）。
 			if (!sess || sess.rpcChannel !== dc) {
 				return;
 			}
@@ -924,6 +926,9 @@ export class WebRtcPeer {
 		}, { logger: this.logger });
 
 		dc.onopen = () => {
+			// 陈旧通道早返回：旧 dc 的迟到 onopen 不应对新 session 误报 transport
+			const sess = this.__sessions.get(connId);
+			if (!sess || sess.rpcChannel !== dc) return;
 			this.__remoteLog(`dc.open conn=${connId} label=${dc.label}`);
 			this.logger.info?.(`${this.__rtcTag} [${connId}] DataChannel "${dc.label}" opened`);
 			// rpc DC 建立后，把本端 transport 信息单播给 UI。
@@ -963,6 +968,9 @@ export class WebRtcPeer {
 			}
 		};
 		dc.onerror = (err) => {
+			// 陈旧通道早返回：旧 dc 的迟到 onerror 不打进新 session 上下文
+			const sess = this.__sessions.get(connId);
+			if (!sess || sess.rpcChannel !== dc) return;
 			this.__remoteLog(`dc.error conn=${connId} label=${dc.label}`);
 			/* c8 ignore next -- ?./?? fallback */
 			this.logger.warn?.(`${this.__rtcTag} [${connId}] DataChannel "${dc.label}" error: ${String(err?.message ?? err)}`);