dsh-router-core 0.4.3 → 0.4.4

package/lib/ext-prompt/index.js

@@ -29,13 +29,13 @@ const PROMPT_CATEGORIES = [
 		id: "closure",
 		title: "全局收口",
 		defaultOn: true,
-		body: "收口(与跑测试同级)：①反查过期：改动让哪些注释/文档/结论/代码引用点(凡以名字对齐的调用方、开关名、注册键)不符事实？当场改或删；②反查重复：新旧成两份就并成一处；③反查形状：该合的合、该归档的归位；④前三问都空时处置一件存量(删/并/归档)。做完那块区域该比动手前更整或更少。"
+		body: "收口(只在任务完成时做，与跑测试同级)：①反查过期：改动让哪些注释/文档/结论/代码引用点(凡以名字对齐的调用方、开关名、注册键)不符事实？当场改或删；②反查重复：新旧成两份就并成一处；③反查形状：该合的合、该归档的归位。做完那块区域该比动手前更整或更少。"
 	},
 	{
 		id: "hardening",
 		title: "固化",
 		defaultOn: true,
-		body: "固化：每次造成用户可见损失的失败，当天固化为可自动重放的检查——会变红(注入违规必须失败)、可重放(一条命令)、归属清晰(失败信息指向违规处)；面向他人的检查只警告不阻断。不固化的教训等于决定再犯一次。"
+		body: "固化：只对数据丢失/安全这类真损失补检查——会变红(注入违规必须失败)、可重放(一条命令)、归属清晰(失败信息指向违规处)；普通 bug 修好即止，不额外造闸门。"
 	},
 	{
 		id: "verdifiable",

package/lib/index.js

@@ -4009,6 +4009,33 @@ function writeJson$1(res, status, body) {
 	});
 	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) {
+	return async (req, res) => {
+		try {
+			await handler(req, res);
+		} catch (err) {
+			if (res.headersSent) {
+				res.end();
+				return;
+			}
+			writeJson$1(res, err instanceof SyntaxError ? 400 : 500, {
+				ok: false,
+				error: err.message
+			});
+		}
+	};
+}
 async function readBody$1(req, limit = 65536) {
 	const chunks = [];
 	let size = 0;
@@ -4067,14 +4094,14 @@ function supplierRoutes(base, loaded, store, router) {
 	routes.push({
 		kind: "exact",
 		path: `${p}/models`,
-		handler: async (_req, res) => {
+		handler: guard(async (_req, res) => {
 			const models = await router.modelsOf(s.id);
 			writeJson$1(res, 200, {
 				ok: true,
 				alias: s.getAlias(),
 				models
 			});
-		}
+		})
 	});
 	routes.push({
 		kind: "exact",
@@ -4202,7 +4229,7 @@ function supplierRoutes(base, loaded, store, router) {
 	routes.push({
 		kind: "exact",
 		path: `${p}/models/toggle`,
-		handler: async (req, res) => {
+		handler: guard(async (req, res) => {
 			const body = JSON.parse(await readBody$1(req));
 			store.get(s.id);
 			const models = await router.modelsOf(s.id);
@@ -4216,12 +4243,12 @@ function supplierRoutes(base, loaded, store, router) {
 			store.setModelEnabled(s.id, body.id, !!body.enabled);
 			router.invalidateModels(s.id);
 			writeJson$1(res, 200, { ok: true });
-		}
+		})
 	});
 	routes.push({
 		kind: "exact",
 		path: `${p}/models/add`,
-		handler: async (req, res) => {
+		handler: guard(async (req, res) => {
 			const id = (JSON.parse(await readBody$1(req)).id ?? "").trim();
 			if (id === "") {
 				writeJson$1(res, 400, {
@@ -4240,7 +4267,7 @@ function supplierRoutes(base, loaded, store, router) {
 			store.addCustomModel(s.id, id);
 			router.invalidateModels(s.id);
 			writeJson$1(res, 200, { ok: true });
-		}
+		})
 	});
 	routes.push({
 		kind: "exact",
@@ -4263,13 +4290,13 @@ function supplierRoutes(base, loaded, store, router) {
 	routes.push({
 		kind: "exact",
 		path: `${p}/models/bulk`,
-		handler: async (req, res) => {
+		handler: guard(async (req, res) => {
 			const body = JSON.parse(await readBody$1(req));
 			const models = await router.modelsOf(s.id);
 			store.setAllModelsEnabled(s.id, !!body.enabled, models.map((mm) => mm.id));
 			router.invalidateModels(s.id);
 			writeJson$1(res, 200, { ok: true });
-		}
+		})
 	});
 	routes.push({
 		kind: "exact",
@@ -4297,12 +4324,12 @@ function supplierRoutes(base, loaded, store, router) {
 	routes.push({
 		kind: "exact",
 		path: `${p}/models/fetch`,
-		handler: async (_req, res) => {
+		handler: guard(async (_req, res) => {
 			writeJson$1(res, 200, {
 				ok: true,
 				models: await router.modelsOf(s.id, true)
 			});
-		}
+		})
 	});
 	routes.push({
 		kind: "exact",

package/lib/types/ext-prompt/content.d.ts

@@ -1,18 +1,25 @@
 /**
- * 提示词内容层 —— 准则 v5 的**逐字**副本，按原分隔拆成分类。
+ * 提示词内容层 —— 准则 v5 的副本，按原分隔拆成分类。
  *
  * ════════════════════════════════════════════════════════════════════
- * 为什么一字不改
+ * 为什么原则上不改措辞
  * ════════════════════════════════════════════════════════════════════
  * 这些字是**用户自己写的准则**，不是本插件的文案。任何"顺手润色"都是擅自
  * 改写用户的规则 —— 改出来的版本和用户心里的版本不一致，而用户看不见差异，
- * 于是准则开始按**另一份**文本执行。所以这里是逐字复制，**不做任何措辞优化**。
+ * 于是准则开始按**另一份**文本执行。所以改动只在**用户点名**时发生，不做
+ * 未被要求的措辞优化。
  *
  * 拆分只为「可开关」，不为了重排：顺序与原文出现顺序一致，`order` 也按此递增。
  *
  * ⚠️ **本文件是唯一事实源**。准则原本存放在 Mnemon 记忆系统的 `USER.md`；
  * 那份已经在插件验证生效后删除（否则新旧两份注入，每轮白烧一次 token 且
  * 永远有两份可能漂移的规则）。**改动准则 = 只改这里。**
+ *
+ * ⚠️ 2026-10-08 用户点名的两处改动（治「提示词导致思考过久」）：
+ *   ① 收口：加"只在任务完成时做"，并**删掉第 ④ 条**「前三问都空时处置一件存量」
+ *      —— 它没有终止条件，每次收口都必然能再找出一件存量，等于无限循环。
+ *   ② 固化：由"每次用户可见损失都必须固化"降级为"只对数据丢失/安全这类真损失
+ *      补检查"，并删掉「不固化的教训等于决定再犯一次」这句无界义务。
  */
 /** 一个可独立开关的提示词分类。 */
 export interface PromptCategory {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-router-core",
-  "version": "0.4.3",
+  "version": "0.4.4",
   "description": "DSH plugin: a simplified 9router (AI routing gateway). Adds a 路由系统 (Routing System) sidebar entry beside 记忆系统; the panel shows suppliers, account pool, combos, and endpoint/API keys.",
   "type": "module",
   "repository": {

package/src/ext-prompt/content.ts

@@ -1,18 +1,25 @@
 /**
- * 提示词内容层 —— 准则 v5 的**逐字**副本，按原分隔拆成分类。
+ * 提示词内容层 —— 准则 v5 的副本，按原分隔拆成分类。
  *
  * ════════════════════════════════════════════════════════════════════
- * 为什么一字不改
+ * 为什么原则上不改措辞
  * ════════════════════════════════════════════════════════════════════
  * 这些字是**用户自己写的准则**，不是本插件的文案。任何"顺手润色"都是擅自
  * 改写用户的规则 —— 改出来的版本和用户心里的版本不一致，而用户看不见差异，
- * 于是准则开始按**另一份**文本执行。所以这里是逐字复制，**不做任何措辞优化**。
+ * 于是准则开始按**另一份**文本执行。所以改动只在**用户点名**时发生，不做
+ * 未被要求的措辞优化。
  *
  * 拆分只为「可开关」，不为了重排：顺序与原文出现顺序一致，`order` 也按此递增。
  *
  * ⚠️ **本文件是唯一事实源**。准则原本存放在 Mnemon 记忆系统的 `USER.md`；
  * 那份已经在插件验证生效后删除（否则新旧两份注入，每轮白烧一次 token 且
  * 永远有两份可能漂移的规则）。**改动准则 = 只改这里。**
+ *
+ * ⚠️ 2026-10-08 用户点名的两处改动（治「提示词导致思考过久」）：
+ *   ① 收口：加"只在任务完成时做"，并**删掉第 ④ 条**「前三问都空时处置一件存量」
+ *      —— 它没有终止条件，每次收口都必然能再找出一件存量，等于无限循环。
+ *   ② 固化：由"每次用户可见损失都必须固化"降级为"只对数据丢失/安全这类真损失
+ *      补检查"，并删掉「不固化的教训等于决定再犯一次」这句无界义务。
  */
 
 /** 一个可独立开关的提示词分类。 */
@@ -69,8 +76,8 @@ export const PROMPT_CATEGORIES: readonly PromptCategory[] = [
     title: '全局收口',
     defaultOn: true,
     body:
-      '收口(与跑测试同级)：①反查过期：改动让哪些注释/文档/结论/代码引用点(凡以名字对齐的调用方、开关名、注册键)不符事实？当场改或删；' +
-      '②反查重复：新旧成两份就并成一处；③反查形状：该合的合、该归档的归位；④前三问都空时处置一件存量(删/并/归档)。' +
+      '收口(只在任务完成时做，与跑测试同级)：①反查过期：改动让哪些注释/文档/结论/代码引用点(凡以名字对齐的调用方、开关名、注册键)不符事实？当场改或删；' +
+      '②反查重复：新旧成两份就并成一处；③反查形状：该合的合、该归档的归位。' +
       '做完那块区域该比动手前更整或更少。',
   },
   {
@@ -78,8 +85,8 @@ export const PROMPT_CATEGORIES: readonly PromptCategory[] = [
     title: '固化',
     defaultOn: true,
     body:
-      '固化：每次造成用户可见损失的失败，当天固化为可自动重放的检查——会变红(注入违规必须失败)、可重放(一条命令)、' +
-      '归属清晰(失败信息指向违规处)；面向他人的检查只警告不阻断。不固化的教训等于决定再犯一次。',
+      '固化：只对数据丢失/安全这类真损失补检查——会变红(注入违规必须失败)、可重放(一条命令)、' +
+      '归属清晰(失败信息指向违规处)；普通 bug 修好即止，不额外造闸门。',
   },
   {
     id: 'verdifiable',

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 路径，账号池回退/冷却自动生效） ----