dsh-router-core 0.4.2 → 0.4.4

package/src/shared.ts

@@ -164,6 +164,40 @@ export interface RouterExtItem {
    * 「包含的组件」里占一行了，两处都列就是同一个东西显示两遍（与内置供应商同理）。
    */
   source?: 'builtin'
+  /**
+   * 扩展自带的子开关（如准则的每一条）。**由扩展自报**，核心只搬运。
+   *
+   * ⚠️ 与 `enabled`（总开关）不同：子开关**不改变扩展是否被加载**，只改它
+   *   自己的行为。总开关关时这些子开关一并失效（面板上禁用，不隐藏）。
+   */
+  controls?: ExtControlItem[]
+}
+
+/** `/router/api/ext` 里的一条子开关（与 `router.ext` 契约的 `ExtControl` 对应）。 */
+export interface ExtControlItem {
+  /** 稳定 id（持久化键；改名会丢用户已保存的选择）。 */
+  id: string
+  title: string
+  /** 当前是否开启（扩展自算：默认值 + 用户已存选择）。 */
+  on: boolean
+  /** 一句话说明（面板副行，可选）。 */
+  detail?: string
+  /** 原文（面板点开即可读到；不给出就是在盲切，见 `ExtControl.body`）。 */
+  body?: string
+  /** 文本可编辑（缺省 = 只读；见 `ExtControl.editable`）。 */
+  editable?: boolean
+  /**
+   * 自建条目（可删）还是内置（不可删、只有开关与还原）。
+   *
+   * ⚠️ **必填，不给可选**（2026-09-30 实测踩过）：曾声明成 `custom?: boolean`，
+   *   而服务端只在 `custom === true` 时才带这个字段 ⇒ 内置条目拿到的是
+   *   `undefined`，面板判 `c.custom === true` 为假，**恰好是对的** ——
+   *   但那条正确性建立在"缺省恰好等于 false"上，改任何一边的默认值都会翻转。
+   *   ⇒ 让服务端**总是**给显式布尔，两边不必猜。
+   */
+  custom: boolean
+  /** 文本被改过（决定「还原」是否可点）。必填，理由同 `custom`。 */
+  overridden: boolean
 }
 
 /** `/router/api/ext` response（扩展插件列表 + 开关）。 */

package/src/suppliers/registry.test.ts

@@ -211,3 +211,59 @@ test('刷新：status() 抛错 → 500 带错误信息', async () => {
   assert.equal(body.ok, false)
   assert.match(body.error ?? '', /status boom/)
 })
+
+/**
+ * 插件在 `listModels` 里抛错时，核心必须回 **JSON 500**。
+ *
+ * 为什么值得单锁：webServer 的兜底是 `res.writeHead(400); res.end()` —— **空
+ * body**。面板 `response.json()` 于是抛 "Unexpected end of JSON input"，真实原因
+ * （上游 500 / token 失效…）全丢。插件契约要求「出错就抛」(docs/suppliers.md)，
+ * 所以这层必须接住。traework 修掉 `catch {}` 之后这条才第一次真被走到。
+ */
+test('models 路由：插件 listModels 抛错 → 500 + JSON body（不是空 400）', async () => {
+  for (const path of ['models', 'models/fetch']) {
+    const supplier = {
+      id: 'fake',
+      name: 'fake',
+      status: () => ({ id: 'fake', name: 'fake', accounts: [] }),
+    }
+    const loaded = { supplier, capabilities: new Set(), source: 'builtin' }
+    // router.modelsOf 复刻核心语义：无旧值 → 把插件的错抛出来
+    const router = {
+      modelsOf: async (): Promise<never> => { throw new Error('models api returned empty list') },
+      invalidateModels: () => {},
+    }
+    const routes = supplierRoutes('/router/api', loaded as never, { get: () => ({ custom: [], disabled: [], poolOrder: [], credits: {} }) } as never, router as never)
+    const route = routes.find((r) => r.path === `/router/api/suppliers/fake/${path}`)
+    assert.ok(route, `${path} 路由应注册`)
+    const sink = fakeRes()
+    await route!.handler({} as never, sink.res)
+    const body = sink.body as { ok: boolean; error?: string }
+    assert.equal(sink.status, 500, `${path}: 抛错该回 500`)
+    assert.equal(body.ok, false, `${path}: 必须是 JSON body，不能是空 body`)
+    assert.match(String(body.error), /empty list/, `${path}: 真实原因不能丢`)
+  }
+})
+
+test('models 路由：body 不是合法 JSON → 400（不是 500，也不是空 body）', async () => {
+  const supplier = { id: 'fake', name: 'fake', status: () => ({ id: 'fake', name: 'fake', accounts: [] }) }
+  const loaded = { supplier, capabilities: new Set(), source: 'builtin' }
+  const router = { modelsOf: async () => [], invalidateModels: () => {} }
+  const routes = supplierRoutes('/router/api', loaded as never, { get: () => ({ custom: [], disabled: [], poolOrder: [], credits: {} }) } as never, router as never)
+  const route = routes.find((r) => r.path === '/router/api/suppliers/fake/models/toggle')
+  assert.ok(route)
+  // 造一个会吐非法 JSON 的 req（readBody 读 data/end）
+  const req = {
+    on(event: string, cb: (chunk?: Buffer) => void) {
+      if (event === 'data') cb(Buffer.from('{ not json'))
+      if (event === 'end') cb()
+      return req
+    },
+  }
+  const sink = fakeRes()
+  await route!.handler(req as never, sink.res)
+  const body = sink.body as { ok: boolean; error?: string }
+  assert.equal(sink.status, 400, '客户端传坏 body 是 400，不是核心 500')
+  assert.equal(body.ok, false)
+  assert.ok(String(body.error).length > 0, '400 也要带原因，不能是空 body')
+})

package/src/suppliers/registry.ts

@@ -44,6 +44,36 @@ function writeJson(res: ServerResponse, status: number, body: unknown): void {
   res.end(payload)
 }
 
+/**
+ * 包住会调插件的 handler：插件抛错时回 **JSON 500**，别让异常裸奔到 webServer。
+ *
+ * 为什么必须有这一层：webServer 的兜底是 `res.writeHead(400); res.end()`——**空
+ * body**。面板拿到空 body，`response.json()` 抛 "Unexpected end of JSON input"，
+ * 真实原因（上游 500 / token 失效…）全丢，用户只看到一句解析错误。
+ *
+ * 这条路径是插件契约「出错就抛」(docs/suppliers.md「模型统一策略」) 的下游：插件
+ * 老实抛了，核心就得老实接住并说清楚。以前 `listModels` 静默吞错，所以从没暴露过
+ * 这个缺口（traework 2026-10-10 「获取模型只有几个」修完才浮现）。
+ */
+function guard(handler: (req: IncomingMessage, res: ServerResponse) => Promise<void>) {
+  return async (req: IncomingMessage, res: ServerResponse): Promise<void> => {
+    try {
+      await handler(req, res)
+    } catch (err) {
+      // headers 已发出就没法再改状态码了（writeJson 是原子的，这里基本不会发生）
+      if (res.headersSent) {
+        res.end()
+        return
+      }
+      // body 不是合法 JSON 是客户端的事（400），不该报成核心 500。这些 handler 里
+      // 唯一的 SyntaxError 来源就是 `JSON.parse(await readBody(req))`。
+      // （改 guard 之前这种情况由 webServer 兜底成**空** 400，改后保住状态码又带上原因。）
+      const status = err instanceof SyntaxError ? 400 : 500
+      writeJson(res, status, { ok: false, error: (err as Error).message })
+    }
+  }
+}
+
 async function readBody(req: IncomingMessage, limit = 64 << 10): Promise<string> {
   const chunks: Buffer[] = []
   let size = 0
@@ -109,10 +139,10 @@ export function supplierRoutes(base: string, loaded: LoadedSupplier, store: Supp
   routes.push({
     kind: 'exact',
     path: `${p}/models`,
-    handler: async (_req, res) => {
+    handler: guard(async (_req, res) => {
       const models = await router.modelsOf(s.id)
       writeJson(res, 200, { ok: true, alias: s.getAlias(), models })
-    },
+    }),
   })
 
   // ---- 通用: 供应商开关 ----
@@ -235,7 +265,7 @@ export function supplierRoutes(base: string, loaded: LoadedSupplier, store: Supp
   routes.push({
     kind: 'exact',
     path: `${p}/models/toggle`,
-    handler: async (req, res) => {
+    handler: guard(async (req, res) => {
       const body = JSON.parse(await readBody(req)) as { id?: string; enabled?: boolean }
       const cfg = store.get(s.id)
       const models = await router.modelsOf(s.id)
@@ -246,14 +276,14 @@ export function supplierRoutes(base: string, loaded: LoadedSupplier, store: Supp
       store.setModelEnabled(s.id, body.id, !!body.enabled)
       router.invalidateModels(s.id)
       writeJson(res, 200, { ok: true })
-    },
+    }),
   })
 
   // ---- 通用: models/add（自定义模型） ----
   routes.push({
     kind: 'exact',
     path: `${p}/models/add`,
-    handler: async (req, res) => {
+    handler: guard(async (req, res) => {
       const body = JSON.parse(await readBody(req)) as { id?: string }
       const id = (body.id ?? '').trim()
       if (id === '') {
@@ -268,7 +298,7 @@ export function supplierRoutes(base: string, loaded: LoadedSupplier, store: Supp
       store.addCustomModel(s.id, id)
       router.invalidateModels(s.id)
       writeJson(res, 200, { ok: true })
-    },
+    }),
   })
 
   // ---- 通用: models/remove（自定义模型） ----
@@ -292,13 +322,13 @@ export function supplierRoutes(base: string, loaded: LoadedSupplier, store: Supp
   routes.push({
     kind: 'exact',
     path: `${p}/models/bulk`,
-    handler: async (req, res) => {
+    handler: guard(async (req, res) => {
       const body = JSON.parse(await readBody(req)) as { enabled?: boolean }
       const models = await router.modelsOf(s.id)
       store.setAllModelsEnabled(s.id, !!body.enabled, models.map((mm) => mm.id))
       router.invalidateModels(s.id)
       writeJson(res, 200, { ok: true })
-    },
+    }),
   })
 
   // ---- 通用: pool/order ----
@@ -324,10 +354,10 @@ export function supplierRoutes(base: string, loaded: LoadedSupplier, store: Supp
   routes.push({
     kind: 'exact',
     path: `${p}/models/fetch`,
-    handler: async (_req, res) => {
+    handler: guard(async (_req, res) => {
       const models = await router.modelsOf(s.id, true)
       writeJson(res, 200, { ok: true, models })
-    },
+    }),
   })
 
   // ---- 通用: 测试模型（核心统一走 chatOnce 路径，账号池回退/冷却自动生效） ----