dsh-mobile-plugin 0.1.82 → 0.1.84

package/lib/sidecar-facade-worker.cjs

@@ -160,6 +160,41 @@ function snapshotRead(info) {
   }
   return body
 }
+/**
+ * 「这份配置里哪几项决定隧道要重建」。
+ *
+ * 只有决定**连到哪台、以谁的身份连**的字段在这里。配对码、名字、局域网候选都
+ * 不在此列——它们能在现有连接上重报一次注册信息就生效（见 reannounceRegistration）。
+ *
+ * 为什么要把这条显式写出来：以前 `configure()` 是无条件 `stopServices()` 把隧道和
+ * 局域网代理一起拆了重建，于是每次配置下发（哪怕一个字段没变）都是一次全断。真机
+ * 现场就是「切个网络，接下来几秒点什么都失败」，宿主日志里是一串「隧道关闭 + 局域网
+ * 代理开始监听」，跑几小时后内存堆到几个 G 把宿主撑爆。
+ */
+function tunnelSignature(c) {
+  return JSON.stringify([Boolean(c.enabled), c.relayUrl || '', c.instanceId || '', c.upstreamPort || 0, c.registerSecret || ''])
+}
+/** 「这份配置里哪几项决定局域网代理要重建」。 */
+function lanSignature(c) {
+  return JSON.stringify([Boolean(c.enabled), Boolean(c.lanEnabled), c.lanPort || 0, c.upstreamPort || 0, c.instanceId || '', c.pairingCode || '', c.lanSecret || ''])
+}
+/** 「这份配置里哪几项决定网络监视器要不要重排」。 */
+function monitorSignature(c) {
+  return JSON.stringify([Boolean(c.enabled), c.relayUrl || '', Boolean(c.lanEnabled)])
+}
+/** 当前应该上报给中转的局域网候选。 */
+function currentLanUrls() { return config.lanEnabled ? lanUrls(lanUrlPort()) : [] }
+/**
+ * 局域网候选变了：只重报注册信息，**不重建隧道**。
+ *
+ * 重建一次要把手机所有在途请求连同局域网监听一起拆掉，而这条路径（地址变化、实际
+ * 端口与首选端口不一致）本来就只是「名片上的地址要更新」。隧道还没连上时不用额外
+ * 做事：下一次 startTunnel 会按当前状态现算候选。
+ */
+function reannounceRegistration() {
+  if (!tunnel) return
+  tunnel.updateRegistration({ lanUrls: currentLanUrls() })
+}
 function stopNetworkMonitor() {
   if (networkTimer) { clearInterval(networkTimer); networkTimer = null }
 }
@@ -178,8 +213,7 @@ function startNetworkMonitor() {
     const current = lanUrls(lanUrlPort())
     if (JSON.stringify(current) === JSON.stringify(previous)) return
     previous = current
-    // lanUrls is sent during register; reconnect only after a real change.
-    startTunnel()
+    reannounceRegistration()
   }, NETWORK_POLL_MS)
 }
 function stopLan() {
@@ -196,7 +230,7 @@ function startTunnel() {
     relayUrl: core.tunnelWsUrl(config.relayUrl), instanceId: config.instanceId, pairingCode: config.pairingCode,
     name: config.name, upstream: upstream(),
     registerSecret: config.registerSecret,
-    lanUrls: config.lanEnabled ? lanUrls(lanUrlPort()) : [],
+    lanUrls: currentLanUrls(),
     readSnapshot: snapshotRead,
     // 实例身份守卫：只有还活着的那个隧道实例才许改状态。onClose / onPairRequest /
     // start().catch 都有这道闸，只有 onRegistered 漏了——而它恰恰是最容易迟到的一个
@@ -251,9 +285,9 @@ function startLan() {
     if (bound > 0) lanBoundPort = bound
     publishState()
     // 实际端口和首选端口不同时，中转侧那份 lanUrls（注册时随 metadata 上报的）
-    // 带的就是首选端口，手机从公网拿到的局域网候选全是死地址。重连一次隧道，让
-    // 中转拿到真正的端口。
-    if (changed && config.enabled && config.relayUrl) startTunnel()
+    // 带的就是首选端口，手机从公网拿到的局域网候选全是死地址。重报一次注册信息就
+    // 够了——重建隧道会把手机在途请求全掐掉，代价远大于这条名片更新。
+    if (changed) reannounceRegistration()
   }).catch(error => { if (lan === current) { lanState = 'error'; lanError = error instanceof Error ? error.message : String(error); publishState() } })
 }
 async function configure(next) {
@@ -265,12 +299,34 @@ async function configure(next) {
   if (typeof next.upstreamPort !== 'number' || !Number.isInteger(next.upstreamPort) || next.upstreamPort < 1 || next.upstreamPort > 65535) throw new Error('invalid upstreamPort')
   if (next.lanPort !== undefined && (!Number.isInteger(next.lanPort) || next.lanPort < 1024 || next.lanPort > 65535)) throw new Error('invalid lanPort')
   if (next.registerSecret !== undefined && (typeof next.registerSecret !== 'string' || next.registerSecret.length > 512)) throw new Error('invalid registerSecret')
-  await stopServices()
+  // 只重建真正变了的那一半：
+  //   - 决定隧道「连到哪台、以谁的身份连」的字段变了才重建隧道；
+  //   - 决定局域网监听的字段变了才重建局域网代理；
+  //   - 其余（配对码 / 名字 / 局域网候选）在现有连接上重报一次注册信息即可。
+  // 无条件全拆全建就是「每隔几分钟断一次 + 内存一直涨」的根因，别改回去。
+  const previous = config
+  const tunnelNeedsRebuild = tunnelSignature(previous) !== tunnelSignature(next)
+  const lanNeedsRebuild = lanSignature(previous) !== lanSignature(next)
+  const monitorNeedsRestart = monitorSignature(previous) !== monitorSignature(next)
   config = { ...next }
-  if (config.enabled) {
+  if (tunnelNeedsRebuild) {
+    stopTunnel()
     startTunnel()
+  }
+  if (lanNeedsRebuild) {
+    stopLan()
+    startLan()
+  }
+  if (!tunnelNeedsRebuild && tunnel) {
+    const patch = {}
+    if (previous.pairingCode !== config.pairingCode) patch.pairingCode = config.pairingCode
+    if (previous.name !== config.name) patch.name = config.name
+    if (lanNeedsRebuild) patch.lanUrls = currentLanUrls()
+    if (Object.keys(patch).length) tunnel.updateRegistration(patch)
+  }
+  if (monitorNeedsRestart) {
+    stopNetworkMonitor()
     startNetworkMonitor()
-    if (config.lanEnabled) startLan()
   }
   return status()
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-mobile-plugin",
-  "version": "0.1.82",
+  "version": "0.1.84",
   "description": "Mobile API (/api/mobile/v1) and relay client for the DSH mobile app — desktop profile build (0.2.0-rc.2 compatible, peers resolved by the host)",
   "license": "MIT",
   "type": "module",