@coclaw/openclaw-coclaw 0.11.2 → 0.11.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@coclaw/openclaw-coclaw",
-  "version": "0.11.2",
+  "version": "0.11.4",
   "type": "module",
   "license": "Apache-2.0",
   "description": "OpenClaw CoClaw channel plugin for remote chat",

package/src/cli-registrar.js

@@ -41,7 +41,7 @@ async function restartGatewayProcess(spawnFn) {
 }
 /* c8 ignore stop */
 
-// bind/unbind/enroll 的 RPC 超时（覆盖默认 10s）
+// bind/unbind/enroll 的 RPC 超时（覆盖 openclaw gateway call 默认 10s）
 // 卡点是 gateway ↔ server 的网络通信，bind 最多两次（先解绑再绑定）
 const RPC_TIMEOUT_MS = 30_000;
 

package/src/common/gateway-notify.js

@@ -27,6 +27,7 @@ export function escapeJsonForCmd(json) {
  * 2. 监听 stdout，解析 JSON 输出判断 RPC 成功/失败
  * 3. 检测到完整 JSON 后启动 KILL_DELAY_MS grace period 等待自然退出
  * 4. 总超时默认 NOTIFY_TIMEOUT_MS（注册 CLI 路径覆盖为 30s）
+ *    同时通过 `--timeout` 传递给子进程，确保内外层超时一致
  * 5. 无论成功失败，最终都主动 kill 子进程
  *
  * grace period 设计：openclaw 进程因 WS handle 滞留可能 10s+ 才退出，
@@ -55,6 +56,10 @@ export function callGatewayMethod(method, spawnFn, opts) {
 		try {
 			const isWin = opts?.isWin ?? IS_WIN;
 			const args = ['gateway', 'call', method, '--json'];
+			// 将超时传递给 openclaw gateway call（默认 10s），避免内外层超时不一致
+			if (opts?.timeoutMs) {
+				args.push('--timeout', String(opts.timeoutMs));
+			}
 			if (opts?.params) {
 				const json = JSON.stringify(opts.params);
 				// Windows 需 shell 解析 .cmd → 必须转义 JSON；非 Windows 不经 shell，直传

package/src/realtime-bridge.js

@@ -941,11 +941,9 @@ export class RealtimeBridge {
 				.then((v) => { this.__pluginVersion = v; })
 				.catch(() => { this.__pluginVersion = 'unknown'; }),
 		]);
-		// 竞态保护：若 preload 期间 stop() 已执行，不再赋值，立即释放 cleanup
+		// 竞态保护：若 preload 期间 stop() 已执行，不再赋值，直接返回。
+		// 不调 cleanup()——与 stop() 策略一致，native threads 保持活跃供后续复用。
 		if (!this.started) {
-			if (preloadResult.cleanup) {
-				try { preloadResult.cleanup(); } catch {}
-			}
 			return;
 		}
 		this.__ndcPreloadResult = preloadResult;
@@ -979,15 +977,12 @@ export class RealtimeBridge {
 			this.webrtcPeer = null;
 			this.__webrtcPeerReady = null;
 		}
-		// ndc cleanup：node-datachannel 的 native threads 必须通过 cleanup() 释放，
-		// 否则会阻止进程退出（issue #366）。
-		// start() 已 await preload 完成并缓存 cleanup 引用，此处直接使用。
-		// 注意：若进程被 SIGKILL 强杀，此处不会执行，OS 会回收资源。
-		// TODO: 若 OpenClaw 未来提供 graceful shutdown 钩子，应在钩子中也调用 cleanup。
-		if (this.__ndcCleanup) {
-			try { this.__ndcCleanup(); }
-			catch (err) { remoteLog(`ndc.cleanup-failed error=${err?.message}`); }
-		}
+		// 不在 stop() 中调用 ndc.cleanup()：
+		// cleanup() 是同步 native 调用，需 join native threads，耗时 10s+，
+		// 会阻塞事件循环导致 RPC handler 超时。
+		// gateway 是长驻进程，native threads 保持活跃即可；
+		// 下次 start() 重新 import（ESM 缓存命中）可直接复用。
+		// 进程退出时 OS 会回收所有资源。
 		this.__ndcCleanup = null;
 		this.__ndcPreloadResult = null;
 		if (this.__fileHandler) {

package/src/webrtc/ndc-preloader.js

@@ -60,6 +60,33 @@ export function defaultResolvePaths(platformKey, pluginRoot) {
 	return { src, dest, destDir };
 }
 
+/**
+ * ndc polyfill 的 RTCPeerConnection 将 iceServers 的 username:credential 直接拼入 URL，
+ * 但 TURN REST API 的 username 格式为 `timestamp:identity`（含冒号），
+ * 导致 libdatachannel 的 URL parser 截断 username。
+ * 此 wrapper 在传入 polyfill 前对 username/credential 做 percent-encoding 规避该问题。
+ */
+function wrapNdcCredentials(NativeRTC) {
+	return class extends NativeRTC {
+		constructor(config = {}) {
+			if (config?.iceServers) {
+				config = {
+					...config,
+					iceServers: config.iceServers.map(s => {
+						if (!s.username && !s.credential) return s;
+						return {
+							...s,
+							username: s.username ? encodeURIComponent(s.username) : s.username,
+							credential: s.credential ? encodeURIComponent(s.credential) : s.credential,
+						};
+					}),
+				};
+			}
+			super(config);
+		}
+	};
+}
+
 /**
  * 预加载 WebRTC 实现：优先 node-datachannel，失败回退 werift，全部失败返回 null。
  *
@@ -160,7 +187,7 @@ export async function preloadNdc(deps = {}) {
 		// 当前由 RealtimeBridge.stop() 负责调用。若 gateway 被 SIGKILL 强杀则无法执行，
 		// 但 OS 会回收所有资源。若 OpenClaw 提供了优雅终止钩子，应在钩子中也调用 cleanup。
 		log(`ndc.loaded platform=${platformKey}`);
-		return { PeerConnection: RTCPeerConnection, cleanup, impl: 'ndc' };
+		return { PeerConnection: wrapNdcCredentials(RTCPeerConnection), cleanup, impl: 'ndc' };
 	} catch (err) {
 		// resolvePaths 或其他未预期异常的兜底
 		log(`ndc.fallback reason=unexpected error=${err.message}`);