@mzzsfy/dsh-tunnel 0.1.2 → 0.1.4

package/README.md

@@ -29,15 +29,19 @@
 - 持久化 `~/.dsh/dsh-tunnel/tunnels.json`(随 `DSH_HOME`), 重启自动恢复; 目标拒连 502, 响应头等待超时(默认 10s, 头到即解除, SSE 不受影响)504
 - REST: `/api/tunnel/tunnels`(GET/POST/DELETE `/api/tunnel/tunnels/:name`) + `/api/tunnel/status` + `/api/tunnel/ui-settings`(POST: aiTools / sidebarTab), 与 GUI/ai 同一入口
 
+## client 依赖
+
+看板通过 `@mzzsfy/dsh-common-tools/client` 订阅数据变更。安装时同时启用 `dsh-session-manager`，由其唯一代挂条目装载公共 client，并排在 tunnel 前物化。公共模块缺失时装载层会报告具体依赖错误；宿主 React 模块缺失时控制台报告原因并禁用面板。
+
 ## 验证
 
-- `node --test "test/*.test.mjs"`: 37 场景 + 8 api 用例 + 11 ai 开关注册面 + 11 守卫断言 + client 端 8 用例(id 守卫 + 总线生命周期;mock webServer + 真实 http/net 端到端)
+- `npm test`(经 _mocks.common-tools.mjs 预载公共依赖桩): 37 场景 + 8 api 用例 + 11 ai 开关注册面 + 11 守卫断言 + client 端 8 用例(id 守卫 + 总线生命周期;mock webServer + 真实 http/net 端到端)
 - `node scripts/compat/tunnel-fullchain.mjs`: 隔离 dsh 实例全链路(真实激活/恢复/分发面七项探针)
 
 ## 设计文档
 
-`docs/调研-路径穿透插件.md`(「方案设计」两节为本包实施规格, 含已知限制); `docs/设计-隧道GUI.md`(GUI/设置/侧边栏注入实施规格)。
+`docs/调研-路径穿透插件.md`(「方案设计」两节为本包实施规格, 含已知限制); `docs/设计-隧道GUI.md`(GUI/设置/侧边栏注入实施规格)。两份均为仓库本地维护文档,不入库,npm 安装读者不可见。
 
 ## dsh 版本兼容
 
-按仓库动态窗口验证(scripts/compat/window.mjs),不预置任何固定版本槽:主测 = 渠道 dist-tag 最新 rc,完全适配口径;基线 = npm latest,不崩溃底线口径;主测与基线同版时收敛单槽。兼容面以窗口实测为准,不逐版本承诺。逐轮实测结论见 `docs/兼容性测试/逐包测试进度.md` 与本包 `兼容性测试.md`。
+按仓库动态窗口验证(scripts/compat/window.mjs),不预置任何固定版本槽:主测 = 渠道 dist-tag 最新 rc,完全适配口径;基线 = npm latest,不崩溃底线口径;主测与基线同版时收敛单槽。兼容面以窗口实测为准,不逐版本承诺。逐轮实测结论见 `docs/兼容性测试/逐包测试进度.md`(本地维护,不入库)与本包 `兼容性测试.md`。

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@mzzsfy/dsh-tunnel",
   "description": "路径穿透隧道:在 dsh 宿主进程内把 /p/<name> 反向代理到本地 TCP 端口(HTTP strip 转发 + 声明式 WS upgrade + 3xx Location 单头改写),支持 <name>.* 子域名全路径透传(任意 WS 路径);GUI 看板手动管理隧道,设置页独立配置节与侧边栏注入开关;向 ai 注册 tunnel_open/tunnel_list/tunnel_close 工具且「允许 ai 使用」可开关(关闭即注销);代理逻辑全在插件进程内,不引入 nginx/caddy 等外部反代",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "type": "module",
   "main": "src/index.js",
   "exports": {

package/src/client.js

@@ -169,15 +169,14 @@ if (typeof window !== 'undefined' && window.__ModuleLoader__) {
   function factory(require) {
     let React = null
     let ReactDOMClient = null
-    let tools = null
     try {
       React = require('react')
       ReactDOMClient = require('react-dom/client')
-      // 公共工具库:bus 订阅是数据刷新唯一通道,缺失即整面板禁用(无轮询回退)
-      tools = require('@mzzsfy/dsh-common-tools/client')
-    } catch {
+    } catch (error) {
+      console.error('[dsh-tunnel] 宿主 React 模块加载失败', error)
       return noopFactory()
     }
+    const tools = require('@mzzsfy/dsh-common-tools/client')
     const { useState, useEffect, useRef, useCallback } = React
     const h = (type, props, ...children) => React.createElement(type, props ?? null, ...children)
 

package/src/index.js

@@ -90,10 +90,12 @@ export const SETTINGS_SCHEMA = z.object({
 
 const normalizeWsPath = (value) => value.replace(/\/+$/, '')
 
-// 校验作用于归一前的原始值; '/' 为合法输入
+// 校验作用于归一前的原始值; '/' 为合法输入。空串 = 根路径归一产物(F26:
+// '/' 去尾斜杠得 '',恢复校验须接受,否则根 WS 隧道重启丢)
 function normalizeWsPaths(raw) {
   if (raw === undefined) return []
-  if (!Array.isArray(raw) || raw.some((item) => typeof item !== 'string' || !WS_PATH_CHARS_RE.test(item))) return undefined
+  const legal = (item) => typeof item === 'string' && (item === '' || WS_PATH_CHARS_RE.test(item))
+  if (!Array.isArray(raw) || raw.some((item) => !legal(item))) return undefined
   return [...new Set(raw.map(normalizeWsPath))]
 }
 
@@ -201,6 +203,11 @@ function forwardHttp(req, res, record, connectTimeoutMs) {
     }
     res.writeHead(proxyRes.statusCode, headers)
     proxyRes.pipe(res)
+    // 上游响应中断即终止下游(F25,与 res close 反向联动成对):
+    // aborted 覆盖请求体阶段中断,error 覆盖响应期套接字错误;
+    // 两态都是"响应不再完整",挂着的下游连接应立刻回收
+    proxyRes.on('aborted', () => res.destroy())
+    proxyRes.on('error', () => res.destroy())
   })
   proxyReq.on('error', (error) => {
     if (res.headersSent) return res.destroy()
@@ -385,6 +392,11 @@ export function apply(ctx, config) {
       if (!VALID_ENTRIES.has(entry)) {
         return { ok: false, error: `entry 不合法: 须为 ${ENTRY_PATH} 或 ${ENTRY_SUBDOMAIN}` }
       }
+      // 自代理拒绝(F27):target 指向本监听端口时,子域名全路径透传+Host 原样
+      // 会把上游请求再导回自身,形成无限自代理环
+      if (args.targetPort === webServer.port) {
+        return { ok: false, error: `targetPort 不得为宿主监听端口 ${webServer.port}(自代理环)` }
+      }
       // 子域名全路径全协议透传, wsPaths 无意义
       const wsPaths = entry === ENTRY_SUBDOMAIN ? [] : normalizeWsPaths(args.wsPaths)
       if (wsPaths === undefined) {
@@ -521,6 +533,11 @@ export function apply(ctx, config) {
       console.warn(`${WARN_TAG}隧道表存在非法行, 已跳过: ${JSON.stringify(row)}`)
       continue
     }
+    // 自代理拒绝同 open(F27):恢复面不放行指向宿主监听端口的环
+    if (row.targetPort === webServer.port) {
+      console.warn(`${WARN_TAG}隧道 ${row.name} 的 targetPort 为宿主监听端口 ${webServer.port}(自代理环), 已跳过`)
+      continue
+    }
     if (tunnels.has(row.name)) {
       console.warn(`${WARN_TAG}隧道表存在重复名 ${row.name}, 仅恢复首行`)
       continue

package/test/client-dependency.test.mjs

@@ -0,0 +1,29 @@
+import test from 'node:test'
+import assert from 'node:assert/strict'
+import { readFileSync } from 'node:fs'
+import vm from 'node:vm'
+
+const source = readFileSync(new URL('../src/client.js', import.meta.url), 'utf8')
+function materialize(missing) {
+  let definition
+  const diagnostics = []
+  const window = { __ModuleLoader__: { load(value) { definition = value } } }
+  vm.runInNewContext(source, { window, console: { error: (...args) => diagnostics.push(args) } })
+  const require = name => {
+    if (name === missing) throw new Error(`module missing: ${name}`)
+    return {}
+  }
+  return { definition, require, diagnostics }
+}
+
+test('Given 缺公共client When 物化隧道模块 Then 向装载层抛出具体依赖错误', () => {
+  const f = materialize('@mzzsfy/dsh-common-tools/client')
+  assert.throws(() => f.definition.factory(f.require), /module missing: @mzzsfy\/dsh-common-tools\/client/)
+})
+for (const missing of ['react', 'react-dom/client']) test(`Given 缺${missing} When 物化 Then 禁用并报告宿主依赖原因`, () => {
+  const f = materialize(missing)
+  const module = f.definition.factory(f.require)
+  assert.deepEqual(Array.from(module.inject), [])
+  assert.equal(f.diagnostics.length, 1)
+  assert.match(String(f.diagnostics[0]), new RegExp(missing))
+})