dsh-router-core 0.4.1 → 0.4.3

package/src/host-base-url.test.ts

@@ -0,0 +1,129 @@
+/**
+ * 网关端点按**宿主实际监听端口**解析 —— issue #9 的闸门。
+ *
+ * ## 这条闸门拦的是什么
+ *
+ * `/v1/*` 是注册在**宿主** webServer 上的路由，端口由启动通道给：`dsh web` 默认
+ * 3080，桌面端官方通道、以及 `port: 0`（OS 分配）都不是。adapter 曾把
+ * `http://localhost:3080/v1` 写死，于是那些宿主上 Router 每一轮对话都是
+ * `dsh-router upstream call failed: fetch failed`，而 `/v1` 明明就在本机跑着。
+ *
+ * 判据刻意用**真服务器 + OS 分配端口**而不是打桩 fetch：打桩只能证明字符串拼对了，
+ * 证明不了「非 3080 端口上请求真的落得到」（这正是当初漏掉这条 bug 的原因）。
+ * `listen(0)` 拿到的端口几乎不可能是 3080，于是「用 3080 也能过」的假修法在这条
+ * 闸门下会当场变红。
+ */
+import test from 'node:test'
+import assert from 'node:assert/strict'
+import { createServer } from 'node:http'
+import type { AddressInfo } from 'node:net'
+import { readdirSync, readFileSync } from 'node:fs'
+import { fileURLToPath } from 'node:url'
+import { RouterAdapter } from './llm/adapter.ts'
+import { loopbackBaseURL } from './host-base-url.ts'
+
+/** 起一个只回一条最小 SSE 的真服务器，回报 OS 分配的端口与收到的路径。 */
+async function serveSse(): Promise<{ port: number; hits: string[]; close: () => Promise<void> }> {
+  const hits: string[] = []
+  const server = createServer((req, res) => {
+    hits.push(req.url ?? '')
+    res.writeHead(200, { 'content-type': 'text/event-stream' })
+    res.end(`data: ${JSON.stringify({ choices: [{ delta: { content: 'ok' } }] })}\n\ndata: [DONE]\n\n`)
+  })
+  await new Promise<void>((resolve) => server.listen(0, '127.0.0.1', resolve))
+  const { port } = server.address() as AddressInfo
+  return { port, hits, close: () => new Promise<void>((resolve) => server.close(() => resolve())) }
+}
+
+/** 跑完一次 adapter 流，返回收到的块类型。 */
+async function drain(adapter: RouterAdapter): Promise<string[]> {
+  const types: string[] = []
+  for await (const c of adapter.stream({ model: 'm', messages: [], signal: AbortSignal.timeout(5000) } as never)) {
+    types.push(c.type)
+  }
+  return types
+}
+
+test('端点：宿主端口不是 3080 时，请求照样落得到（issue #9 的原始现场）', async () => {
+  const up = await serveSse()
+  try {
+    assert.notEqual(up.port, 3080, '这条用例的判据就是「端口不是 3080」')
+    const adapter = new RouterAdapter(loopbackBaseURL(() => up.port), { comboModels: async () => [] })
+    const types = await drain(adapter)
+    assert.deepEqual(up.hits, ['/v1/chat/completions'], `请求应打到宿主 /v1，实际打到：${JSON.stringify(up.hits)}`)
+    assert.ok(types.includes('text-delta'), `应真的收到上游内容，实际块类型：${JSON.stringify(types)}`)
+  } finally {
+    await up.close()
+  }
+})
+
+test('端点：端口每次请求现算（listen 前读不到、宿主重启换端口都不缓存）', async () => {
+  const first = await serveSse()
+  const second = await serveSse()
+  try {
+    let port: number | undefined = first.port
+    let reads = 0
+    const adapter = new RouterAdapter(
+      loopbackBaseURL(() => {
+        reads += 1
+        return port
+      }),
+      { comboModels: async () => [] },
+    )
+    await drain(adapter)
+    port = second.port // 宿主换了端口（重启 / 重新 listen）
+    await drain(adapter)
+    assert.deepEqual(first.hits, ['/v1/chat/completions'], '第一次应打在第一个端口')
+    assert.deepEqual(second.hits, ['/v1/chat/completions'], '端口变了之后应改打新端口（不能缓存第一次的值）')
+    assert.ok(reads >= 2, `端口应每请求现读，实际只读了 ${reads} 次`)
+  } finally {
+    await first.close()
+    await second.close()
+  }
+})
+
+test('端点：端口读不到时报出原因，不回退 3080 硬猜', async () => {
+  const adapter = new RouterAdapter(loopbackBaseURL(() => undefined), { comboModels: async () => [] })
+  await assert.rejects(
+    () => drain(adapter),
+    (err: Error) => {
+      assert.match(err.message, /webServer/, `报错必须带原因，实际：${err.message}`)
+      assert.equal((err as { code?: string }).code, 'TRANSPORT', '应是可重试的传输类错误')
+      return true
+    },
+  )
+})
+
+test('端点：常量形态的 baseURL 仍然可用（测试与显式传入的老调用方）', () => {
+  assert.equal(loopbackBaseURL(() => 19387)(), 'http://127.0.0.1:19387/v1')
+})
+
+test('源码：非测试代码里不得再出现写死的 3080 端点', () => {
+  // 防复发闸门：这次的根因就是一行写死的字符串，只靠「端口现算」的实现测试拦不住
+  // 将来有人在别处再写一遍。判据只看**代码**（先剥掉注释）——注释里可以、也应该
+  // 写明「当年就是这里写死了 3080」。
+  const root = fileURLToPath(new URL('.', import.meta.url))
+  const offenders: string[] = []
+  for (const file of walk(root)) {
+    if (/(?:localhost|127\.0\.0\.1):3080/.test(stripComments(readFileSync(file, 'utf8')))) {
+      offenders.push(file.slice(root.length))
+    }
+  }
+  assert.deepEqual(offenders, [], `这些文件里还有写死的 3080 端点（宿主端口随启动通道变）：${offenders.join(', ')}`)
+})
+
+/** 去掉块注释与行注释，只留可执行代码。 */
+function stripComments(text: string): string {
+  return text.replace(/\/\*[\s\S]*?\*\//g, '').replace(/^\s*\/\/.*$/gm, '')
+}
+
+/** 递归列出 src 下的 .ts 文件（跳过测试自身：测试里出现端口字面量是合法的）。 */
+function walk(dir: string): string[] {
+  const out: string[] = []
+  for (const entry of readdirSync(dir, { withFileTypes: true })) {
+    const path = `${dir}${entry.name}`
+    if (entry.isDirectory()) out.push(...walk(`${path}/`))
+    else if (entry.name.endsWith('.ts') && !entry.name.endsWith('.test.ts')) out.push(path)
+  }
+  return out
+}

package/src/host-base-url.ts

@@ -0,0 +1,43 @@
+/**
+ * 网关端点地址（`http://127.0.0.1:<宿主端口>/v1`）的解析。
+ *
+ * ## 为什么端口不能写死
+ *
+ * dsh-router **自己不开端口**：`/v1/*` 是注册在**宿主** web server 上的路由
+ * （见 `index.ts` 的 `ctx.webServer.register`）。所以端点地址里的端口由启动通道
+ * 决定，而不是插件决定 —— `dsh web` 默认 3080，桌面端官方通道、以及配置
+ * `port: 0`（让 OS 分配）时都**不是** 3080。之前 adapter 把
+ * `http://localhost:3080/v1` 写死在构造参数里，于是非 3080 的宿主上 Router
+ * 的每一轮对话都是 `dsh-router upstream call failed: fetch failed`（issue #9）。
+ *
+ * ## 为什么返回函数而不是字符串
+ *
+ * `webServer.port` 是**监听之后**才有值的 getter（listen 前是 `undefined`），
+ * 且宿主重启/换端口后它会变。所以这里返回函数、**每次请求现算**，不在注册时
+ * 算一次 —— 注册发生在启动早期，那时算只会把「拿不到端口」固化成常量。
+ *
+ * ## 拿不到端口时为什么抛，而不是回退 3080
+ *
+ * 回退等于把同一个 bug 换种写法：端口读不到时宿主的路由也没在服务，回退到 3080
+ * 只会重新得到一句无从查证的 `fetch failed`。这里抛明确的话，让它出现在
+ * `LlmError` 消息里（adapter 在 fetch 同一个 try 里调它），报错自带原因。
+ */
+
+/** 回环地址：宿主即使监听 `0.0.0.0`（LAN 模式），走 127.0.0.1 也通。 */
+const LOOPBACK_HOST = '127.0.0.1'
+
+/**
+ * 造一个「每次调用现读宿主端口」的端点解析器。
+ *
+ * @param readPort 读宿主监听端口（`webServer.port`）。取不到时抛错而不是猜。
+ * @param path 端点路径前缀，默认 `/v1`。
+ */
+export function loopbackBaseURL(readPort: () => number | undefined, path: string = '/v1'): () => string {
+  return () => {
+    const port = readPort()
+    if (port === undefined) {
+      throw new Error('dsh-router: 宿主 webServer 端口未就绪（webServer.port 读不到），无法定位 /v1 端点')
+    }
+    return `http://${LOOPBACK_HOST}:${String(port)}${path}`
+  }
+}

package/src/host-version.test.ts

@@ -1,9 +1,12 @@
 /**
- * 宿主版本探测测试 —— 锁死「最近命中徽章仅在 >= 0.1.7 出现」这条判定。
+ * 宿主版本探测测试 —— 锁死那条**版本地板**（>= 0.2.0）。
  *
- * 起因：`conversation.composer.dock` 座位在 0.1.5 也在，但渲染在 InputBar 之后
- * 的独立块（位置不对）；只有 0.1.7 才放进底部操作行。座位名相同没法靠契约区分，
- * 只能按宿主版本号判定——本测试就是这条判据的唯一守卫。
+ * 这条判据现在只有一个用途：**在老宿主上出声**。兼容分叉（三处 `>= 0.1.7`）随
+ * 0.2.0 单面编写一起删掉了，所以本测试守的是「低于地板会被判为不支持 → apply 时
+ * 打警告」这条报警链，而不是某个功能开关。
+ *
+ * 地板抬到 0.2.0 的那一刻，0.1.7 从「支持」翻成「不支持」——这条断言就是那次
+ * 翻面的记录：谁想再加回 0.1.x 兼容，本测试会先红。
  */
 import test from 'node:test'
 import assert from 'node:assert/strict'
@@ -11,7 +14,7 @@ import { mkdirSync, mkdtempSync, writeFileSync } from 'node:fs'
 import { tmpdir } from 'node:os'
 import { join } from 'node:path'
 import { pathToFileURL } from 'node:url'
-import { detectHostVersion, isHost017Plus, supportsLastHitDock } from './host-version.ts'
+import { detectHostVersion, isHostSupported } from './host-version.ts'
 
 /** 造一个「装了指定版本 dsh」的临时 profile，返回它的 file: URL。 */
 function profileWithDsh(version: string | null): string {
@@ -25,37 +28,27 @@ function profileWithDsh(version: string | null): string {
 }
 
 test('读得到版本：detectHostVersion 返回 profile 里装的 dsh 版本', () => {
-  assert.equal(detectHostVersion(profileWithDsh('0.1.5-rc.3')), '0.1.5-rc.3')
-  assert.equal(detectHostVersion(profileWithDsh('0.1.7-alpha.2')), '0.1.7-alpha.2')
+  assert.equal(detectHostVersion(profileWithDsh('0.1.7-rc.2')), '0.1.7-rc.2')
+  assert.equal(detectHostVersion(profileWithDsh('0.2.0-rc.1')), '0.2.0-rc.1')
 })
 
-test('支持位：>= 0.1.7 才为真（预发布标识忽略，按前导数字比）', () => {
-  assert.equal(supportsLastHitDock(profileWithDsh('0.1.7-alpha.2')), true)
-  assert.equal(supportsLastHitDock(profileWithDsh('0.1.7')), true)
-  assert.equal(supportsLastHitDock(profileWithDsh('0.2.0')), true)
-  assert.equal(supportsLastHitDock(profileWithDsh('1.0.0')), true)
-  // 低于 0.1.7 → 不挂（0.1.5 位置不对，宁可不显示）
-  assert.equal(supportsLastHitDock(profileWithDsh('0.1.5-rc.3')), false)
-  assert.equal(supportsLastHitDock(profileWithDsh('0.1.6-alpha.2')), false)
-  assert.equal(supportsLastHitDock(profileWithDsh('0.1.4')), false)
+test('地板：>= 0.2.0 才算支持（预发布标识忽略，按前导数字比）', () => {
+  for (const v of ['0.2.0-rc.1', '0.2.0', '0.2.1', '1.0.0']) {
+    assert.equal(isHostSupported(profileWithDsh(v)), true, v)
+  }
 })
 
-test('isHost017Plus 与徽章支持位同源（三处兼容分叉共用一个判据）', () => {
-  for (const v of ['0.1.7-alpha.2', '0.1.7', '0.2.0', '1.0.0']) {
-    assert.equal(isHost017Plus(profileWithDsh(v)), true, v)
-    assert.equal(supportsLastHitDock(profileWithDsh(v)), true, v)
-  }
-  for (const v of ['0.1.5-rc.3', '0.1.6-alpha.2', '0.1.4']) {
-    assert.equal(isHost017Plus(profileWithDsh(v)), false, v)
-    assert.equal(supportsLastHitDock(profileWithDsh(v)), false, v)
+test('地板之下（0.1.5/0.1.6/0.1.7）判为不支持 —— 会打警告而不是静默走错路径', () => {
+  for (const v of ['0.1.5-rc.3', '0.1.6-alpha.2', '0.1.7-rc.2', '0.1.4']) {
+    assert.equal(isHostSupported(profileWithDsh(v)), false, v)
   }
 })
 
-test('读不到版本 → 一律当不支持（不抛、宁可不显示）', () => {
+test('读不到版本 → 判为不支持（无法确认也值得说一声），且不抛', () => {
   assert.equal(detectHostVersion(profileWithDsh(null)), undefined)
-  assert.equal(supportsLastHitDock(profileWithDsh(null)), false)
+  assert.equal(isHostSupported(profileWithDsh(null)), false)
   // 垃圾版本号
-  assert.equal(supportsLastHitDock(profileWithDsh('not-a-version')), false)
+  assert.equal(isHostSupported(profileWithDsh('not-a-version')), false)
   // baseUrl 缺失（冷启动/单测）→ 回落 ~/.dsh/profiles/web，不抛
-  assert.doesNotThrow(() => supportsLastHitDock(undefined))
+  assert.doesNotThrow(() => isHostSupported(undefined))
 })

package/src/host-version.ts

@@ -1,37 +1,46 @@
 /**
- * 宿主 DSH 版本探测 —— **0.1.5 / 0.1.7 兼容分叉的唯一判据来源**。
+ * 宿主 DSH 版本探测 —— 现在只剩两件事：**报版本**（日志 + `/router/api/health`）与
+ * **守版本地板**。
  *
- * 原先三处兼容都靠「能力检测」（看方法/属性在不在），语义上是同一个问题
- * ——「这是 0.1.7 还是更老」——却各检各的、还容易误判（插件自己 node_modules
- * 里的依赖版本与宿主实际加载的版本并不一致：实测插件解析到 schemastery 3.18.4，
- * profile 里却是 3.18.3、全局是 3.18.2）。所以统一改成**读宿主版本号**判定：
+ * ## 为什么不再有兼容分叉
  *
- *   1. 「最近命中」徽章：composer.dock 在 0.1.5 渲染位置不对 → >= 0.1.7 才挂。
- *   2. 设置命名空间：0.1.5 用 `settings.installSection` 注册；0.1.7 改由导出的
- *      Config 驱动（无此方法）→ 按版本走对应注册路径。
- *   3. adapter 的 tool-result 消息形态：0.1.5/0.1.6 = user 里的 'tool-result'
- *      块；0.1.7+ = 独立 role:'tool' 消息。
+ * 原来这里有三处 `>= 0.1.7` 的分叉（设置卡片注册路径、「最近命中」徽章座位、adapter
+ * 的工具结果消息形态），彼此独立判断、还容易误判。本插件现已按 **0.2.0 单面**编写，
+ * 三处的旧分支全部删除：
+ *
+ *   1. 设置卡片：0.1.5 走 `settings.installSection`，0.1.7+ 走导出的 volatile Config。
+ *      0.2.0 只有后者 → installSection 分支删除。
+ *   2. 「最近命中」徽章：0.1.5 渲染在 InputBar 之后的独立块（位置不对），0.1.7+ 才在
+ *      composer 底部操作行。0.2.0 起无条件挂载，不再问支持位。
+ *   3. adapter 工具结果形态：旧的「user 内嵌 tool-result 块」0.1.7 起退役；0.2.0 的
+ *      v3→v4 会话迁移还主动拒绝该包装，所以它到不了 adapter → 分支删除。
+ *
+ * ## 为什么仍要读版本
+ *
+ * 因为**跑在老宿主上必须出声**。删掉分叉不等于「老宿主也能跑」：0.1.x 上本插件会
+ * 静默走错路径（卡片不出、徽章错位、工具结果丢失），而用户看到的现象与「插件坏了」
+ * 无法区分。所以这里只保留一个判据 `isHostSupported`（>= 0.2.0），仅用于 apply 时
+ * 打一条醒目的警告 —— 不是兼容，是可诊断。
  *
  * 读法：宿主把 profile 目录挂在 `ctx.baseUrl`（见 data-dir.ts）。本插件跑在宿主
  * 进程内，`require.resolve('@deepseek-ai/dsh/package.json', { paths: [profile] })`
- * 会沿 profile 的 node_modules 向上解析到全局安装根（实测命中 bun 全局目录）。
- * 拿不到就回落直接拼路径，再不行就返回 undefined（按「不支持/老版本」处理，不抛）。
+ * 会沿 profile 的 node_modules 向上解析到全局安装根。拿不到就回落直接拼路径，再不行
+ * 返回 undefined（按「无法确认」处理，不抛）。
  *
- * 注意：**只能在 `apply(ctx)` 里用**（那时才有 baseUrl）。模块顶层（如 Config 导出）
- * 解析到的是插件自己的依赖，不能代表宿主——那处保留能力检测，见 index.ts 注释。
+ * 注意：**只能在 `apply(ctx)` 里用**（那时才有 baseUrl）。模块顶层解析到的是插件自己
+ * 的依赖，不能代表宿主。
  */
 import { readFileSync } from 'node:fs'
 import { createRequire } from 'node:module'
 import { join } from 'node:path'
 import { profileDirOf } from './data-dir.ts'
 
-/** 各兼容分叉的版本门槛。 */
-/** composer.dock 徽章 / 独立 role:'tool' 消息形态：0.1.7 起。 */
-const V0_1_7: readonly number[] = [0, 1, 7]
+/** 版本地板：低于此版本的宿主本插件不再兼容（0.1.5/0.1.6/0.1.7 全部落此下）。 */
+const V0_2_0: readonly number[] = [0, 2, 0]
 
 /**
  * 解析并比较 semver（只取前导数字段，预发布标识忽略）。
- * `0.1.7-alpha.2` → [0,1,7]，`0.1.5-rc.3` → [0,1,5]。
+ * `0.2.0-rc.1` → [0,2,0]，`0.1.7-alpha.2` → [0,1,7]。
  * 返回负数/0/正数，语义同 `<`/`==`/`>`。
  */
 function compareVersion(a: readonly number[], b: readonly number[]): number {
@@ -43,7 +52,7 @@ function compareVersion(a: readonly number[], b: readonly number[]): number {
   return 0
 }
 
-/** `0.1.7-alpha.2` → [0,1,7]；解析失败返回 null。 */
+/** `0.2.0-rc.1` → [0,2,0]；解析失败返回 null。 */
 function parseVersion(raw: string): number[] | null {
   const m = /^(\d+)(?:\.(\d+))?(?:\.(\d+))?/.exec(raw.trim())
   if (m === null) return null
@@ -53,11 +62,11 @@ function parseVersion(raw: string): number[] | null {
 /**
  * 探测 profile 里安装的 DSH 宿主版本。
  * @param baseUrl 宿主注入的 profile 锚点（`ctx.baseUrl`）
- * @returns 形如 `0.1.7-alpha.2`；读不到返回 undefined（调用方按「不支持」处理）
+ * @returns 形如 `0.2.0-rc.1`；读不到返回 undefined
  */
 export function detectHostVersion(baseUrl?: string): string | undefined {
   const profile = profileDirOf(baseUrl)
-  // 先走模块解析（能沿 node_modules 向上找到全局安装根），再回落直接拼路径。
+  // 先走模块解析（能沿 node_modules 向上找到全局安装根），再回落直接路径。
   const candidates: string[] = []
   try {
     candidates.push(createRequire(join(profile, 'noop.js')).resolve('@deepseek-ai/dsh/package.json'))
@@ -76,26 +85,17 @@ export function detectHostVersion(baseUrl?: string): string | undefined {
   return undefined
 }
 
-/** 宿主版本是否 >= 门槛。读不到版本 → false（按老版本处理）。 */
-function atLeast(baseUrl: string | undefined, min: readonly number[]): boolean {
+/**
+ * 宿主是否在本插件支持的地板之上（>= 0.2.0）。
+ *
+ * **只用于报警，不用于分流** —— 低于地板时插件照常加载，但要在日志里说清「这台宿主
+ * 不再兼容、会出现什么症状」，把静默的错变成可查的错。读不到版本 → false（当作
+ * 「无法确认」，同样值得说一句）。
+ */
+export function isHostSupported(baseUrl?: string): boolean {
   const v = detectHostVersion(baseUrl)
   if (v === undefined) return false
   const parsed = parseVersion(v)
   if (parsed === null) return false
-  return compareVersion(parsed, min) >= 0
-}
-
-/**
- * 宿主是否 >= 0.1.7。三处兼容分叉共用这一个判据：
- *   - 「最近命中」徽章（composer.dock 位置正确）
- *   - 设置命名空间（Config 驱动而非 installSection）
- *   - adapter 的独立 role:'tool' 消息形态
- */
-export function isHost017Plus(baseUrl?: string): boolean {
-  return atLeast(baseUrl, V0_1_7)
-}
-
-/** 「最近命中」徽章是否可用（= 宿主 >= 0.1.7）。 */
-export function supportsLastHitDock(baseUrl?: string): boolean {
-  return isHost017Plus(baseUrl)
+  return compareVersion(parsed, V0_2_0) >= 0
 }