dsh-router-core 0.3.14 → 0.3.16

package/src/router/index.ts

@@ -71,6 +71,7 @@ function stripAlias(model: string, alias: string): string {
  *
  * 只对**瞬时**故障等：连接层失败 / 上游不可用（对应 9router 的 502/503/504）。
  * 模型不属于本供应商（no_such_model）绝不能等 —— 它不是故障，等纯属浪费。
+ * 请求本身非法（bad_request）同理：换哪个模型都是同一个 400，等也是白等。
  */
 const TRANSIENT_SETTLE_MS = 2_000
 
@@ -998,7 +999,8 @@ export class Router {
     if (s.accounts !== undefined && s.accounts().length === 0) {
       const r = await s.chatOnce('', req.lv ?? 'auto', req)
       if (!r.ok) {
-        if (r.state === 'no_such_model') return false // 不是我的模型：换供应商，不记账
+        // 模型不属于本供应商、或请求本身被上游拒收：换供应商，不记账
+        if (r.state === 'no_such_model' || r.state === 'bad_request') return false
         if (trace !== undefined) trace.lastState = r.state
         pool.noteFailure('', req.model, r.state, r.message)
         return false
@@ -1028,10 +1030,15 @@ export class Router {
       tried.add(uid)
       const r = await s.chatOnce(uid, req.lv ?? 'auto', req)
       if (!r.ok) {
-        // 模型不属于本供应商：整个供应商都跳过，换号重试没有意义，
-        // 也不能记在账号头上（否则无关账号会被攒够错误冷却掉）
-        if (r.state === 'no_such_model') {
-          if (trace !== undefined) trace.lastError = 'no_such_model'
+        // 模型不属于本供应商、或请求本身被上游拒收：整个供应商都跳过，
+        // 换号重试没有意义，也不能记在账号头上（否则无关账号会被冷掉）。
+        // bad_request（如 tool_call 配对断裂、图片格式不认）对每个号结果相同，
+        // 冷号只会把一次「请求有问题」放大成「这个模型谁都别用」。
+        if (r.state === 'no_such_model' || r.state === 'bad_request') {
+          if (trace !== undefined) {
+            trace.lastError = r.state
+            trace.lastState = r.state
+          }
           return false
         }
         pool.noteFailure(uid, req.model, r.state, r.message)

package/src/router/models-cache.test.ts

@@ -613,7 +613,6 @@ test('组合请求不污染无关供应商的账号（no_such_model 不记账）
   const acc = other.s.pool.decorate([{ uid: 'u1', credits: 0, state: 'ok' }])[0]
   assert.equal(acc?.err_count, 0, '不该累计连续错误')
   assert.equal(acc?.cooling, false, '不该被冷却')
-  assert.equal(acc?.disabled, false, '不该被禁用')
 })
 
 /**

package/src/router/types.ts

@@ -14,7 +14,6 @@ export interface SupplierAccount {
   cooling: boolean
   until?: string
   reason?: string
-  disabled: boolean
   err_count?: number
 }
 

package/src/shared.ts

@@ -28,7 +28,6 @@ export interface RouterAccount {
   cooling: boolean
   until?: string
   reason?: string
-  disabled: boolean
   err_count?: number
   /** 所属供应商 id。 */
   supplier?: string

package/src/suppliers/contract.ts

@@ -31,7 +31,7 @@ export interface SupplierEnv {
    * 不报上来，它就会继续留在池里被轮转选中，每次都白撞一次同一个错误。
    *
    * 核心实现 = `pool.noteFailure(uid, model, state, message)`，按该状态的规则
-   * 冷却/禁用。真实案例：TRAE 免费通道对某账号拒绝某模型时发 HTTP 200 +
+   * 冷却。真实案例：TRAE 免费通道对某账号拒绝某模型时发 HTTP 200 +
    * 流内 `code=4008`，三个号里两个如此，round-robin 下 2/3 请求直接失败
    * （2026-09-08 实测）——有了这条通道，坏号被冷一次就退出轮转。
    *
@@ -46,7 +46,7 @@ export type SupplierFactory = (env: SupplierEnv) => SupplierModule
 /**
  * 账户此刻的状态 —— 插件**解读**上游信号后的语义状态。
  *
- * 插件只报「现在怎么了」，不说「该怎么办」：冷却多久、是否禁用、要不要换号
+ * 插件只报「现在怎么了」，不说「该怎么办」：冷却多久、要不要换号
  * 都是核心的策略，见 `AccountState` 各值的处置表（docs/suppliers.md）。
  *
  * 天花板：目前是固定枚举。若将来某供应商需要更细的语义，再加值而不是放宽成
@@ -67,8 +67,18 @@ export type AccountState =
    * 把它冷却掉（历史上正是这个坑）。
    */
   | 'no_such_model'
+  /**
+   * 请求本身不合法，被上游拒（参数错、图片格式不支持、tool_call 配对断裂…）。
+   *
+   * **不是账号的错**：同一个请求对池里每个号都会失败，冷号只会让好号白白
+   * 失效，把一次「这条请求有问题」放大成「这个模型 30 秒内谁都别用」——
+   * 组合里两条腿同时被冷就是这个机制（2026-09-15 读图 11148 事故）。
+   * 核心据此**不惩罚账号**，直接跳出本供应商换下一个模型/供应商。
+   * 与 `no_such_model` 的区别：那个是「模型不归我」，这个是「请求我不认」。
+   */
+  | 'bad_request'
 
-/** 面板展示的账号此刻状态（插件只报它观察到的部分，冷却/禁用由核心叠加）。 */
+/** 面板展示的账号此刻状态（插件只报它观察到的部分，冷却由核心叠加）。 */
 export interface SupplierAccountNow {
   uid: string
   nickname?: string
@@ -98,7 +108,7 @@ export type ChatOnceResult =
   | { ok: true; stream: ReadableStream<Uint8Array> }
   /** 非流式：核心写 JSON。 */
   | { ok: true; status: number; body: string }
-  /** 失败：核心据此做冷却/禁用/换号。 */
+  /** 失败：核心据此做冷却/换号。 */
   | { ok: false; state: AccountState; message: string }
 
 /** 供应商模块 —— 契约（核心必须，差异化可选）。 */
@@ -120,7 +130,7 @@ export interface SupplierModule {
   readonly apiKeyHint?: string
 
   // ---- 核心（必须） ----
-  /** 报账号「现在状态」。冷却/禁用/错误累计由核心叠加，插件不算这些。 */
+  /** 报账号「现在状态」。冷却/错误累计由核心叠加，插件不算这些。 */
   status(): SupplierStatusNow
   /** 模型来源。`force=true` 时强制刷新来源（核心「获取模型」按钮调用）。 */
   listModels(force?: boolean): Promise<ModelInfo[]> | ModelInfo[]

package/src/suppliers/http-state.test.ts

@@ -0,0 +1,36 @@
+/**
+ * HTTP 状态码 → AccountState 公共映射测试。
+ *
+ * 存在的原因：这条映射曾在内置三家供应商里各写一份，于是「哪个错误算账号的错」
+ * 有三份实现——bad_request 加固差点只补上 codebuddy，三家内置仍是 4xx→unknown
+ * →瞬冷 30s（同型事故）。收敛成一处后，这一份测试同时守住三家。
+ */
+import { test } from 'node:test'
+import assert from 'node:assert/strict'
+import { stateFromHttpStatus } from './http-state.ts'
+
+test('映射：账号侧错误（换号会好的）归各自的冷却状态', () => {
+  assert.equal(stateFromHttpStatus(429), 'rate_limit', '限流 = 换号有用')
+  assert.equal(stateFromHttpStatus(401), 'session_dead', '凭证失效')
+  assert.equal(stateFromHttpStatus(403), 'session_dead', '403 分不清死活与风控，按连接级冷却')
+  assert.equal(stateFromHttpStatus(402), 'quota', '要钱/额度不足')
+  assert.equal(stateFromHttpStatus(404), 'unavailable', '上游没有这个端点/服务下线')
+})
+
+test('映射：请求侧 4xx 归 bad_request —— 同一个请求对每个号都一样失败，不该冷号', () => {
+  for (const status of [400, 405, 409, 413, 422, 499]) {
+    assert.equal(stateFromHttpStatus(status), 'bad_request', `${status} 是请求形态问题，不是账号的错`)
+  }
+})
+
+test('映射：5xx 与异常码仍归 unknown（上游故障侧，瞬冷换号是对的）', () => {
+  for (const status of [500, 502, 503, 504, 599]) {
+    assert.equal(stateFromHttpStatus(status), 'unknown', `${status} 是上游故障`)
+  }
+})
+
+test('映射：绝不返回 ok —— 这个函数只在非 2xx 分支被调用', () => {
+  for (const status of [400, 401, 402, 403, 404, 429, 500]) {
+    assert.notEqual(stateFromHttpStatus(status), 'ok')
+  }
+})

package/src/suppliers/http-state.ts

@@ -0,0 +1,31 @@
+/**
+ * HTTP 状态码 → AccountState 的公共映射（内置三家供应商共用）。
+ *
+ * 为什么收敛成一处：opencode / openrouter / nvidia 本来各写一份逐字相同的
+ * 三元表达式，于是「哪个状态算谁的错」这件事有三份实现，改一处就漏两处
+ * （bad_request 就是这么差点只补了 codebuddy）。
+ *
+ * 关键判据：**这个错误换一个账号会不会好？**
+ *  - 会好（限流/额度/凭证）→ 是账号的错，交给核心冷却/换号
+ *  - 不会好（请求参数、模型不存在）→ 不是账号的错，绝不能冷号
+ *    否则一次坏请求会把整个池冷掉（2026-09-15 codebuddy 读图事故的同型：
+ *    组合两条腿同时被冷，之后连正常文本请求也全灭 503）
+ */
+import type { AccountState } from './contract.ts'
+
+/**
+ * 非 2xx 上游响应 → 语义状态。
+ *
+ * 4xx 里除了「限流/凭证/不存在」这三种明确归属账号的，剩下的都是**请求本身
+ * 有问题**（400 参数错、413 太大、422 语义错…）——同一个请求对池里每个号结果
+ * 都相同，归 `bad_request`（核心不惩罚账号，直接换下一个模型）。
+ * 5xx / 说不清的仍归 `unknown`（上游故障侧，瞬冷换号是对的）。
+ */
+export function stateFromHttpStatus(status: number): AccountState {
+  if (status === 429) return 'rate_limit'
+  if (status === 401 || status === 403) return 'session_dead'
+  if (status === 404) return 'unavailable'
+  if (status === 402) return 'quota'
+  if (status >= 400 && status < 500) return 'bad_request'
+  return 'unknown'
+}

package/src/suppliers/loader.ts

@@ -124,7 +124,7 @@ export function wrapModule(instance: SupplierModule, env: SupplierEnv, source: s
     if (isFn((instance as unknown as Record<string, unknown>)[key])) capabilities.add(key)
   }
 
-  // 账号池由核心持有（选号/冷却/禁用/错误累计），插件不参与。
+  // 账号池由核心持有（选号/冷却/错误累计），插件不参与。
   // 挂在 supplier 上：status() 叠加状态与 Router 的 chat 循环共用同一实例。
   // 冷却键带 supplierId（跨供应商同名模型不串），故构造时注入本供应商 id。
   const pool = new AccountPool(instance.id)

package/src/suppliers/nvidia/plugin.ts

@@ -18,6 +18,7 @@ import { join } from 'node:path'
 import type { ChatRequest, ModelInfo } from '../../router/types.ts'
 import type { AccountState, ChatOnceResult, SupplierStatusNow } from '../contract.ts'
 import type { SupplierEnv, SupplierModule } from '../contract.ts'
+import { stateFromHttpStatus } from '../http-state.ts'
 
 export const id = 'nvidia'
 export const name = 'NVIDIA NIM'
@@ -287,12 +288,7 @@ export default function factory(env: SupplierEnv): SupplierModule {
       if (upstream.status < 200 || upstream.status >= 300) {
         const text = await upstream.text().catch(() => '')
         const msg = `upstream ${upstream.status}: ${text.slice(0, 120)}`
-        const state: AccountState =
-          upstream.status === 429 ? 'rate_limit'
-            : upstream.status === 401 || upstream.status === 403 ? 'session_dead'
-              : upstream.status === 404 ? 'unavailable'
-                : 'unknown'
-        return { ok: false, state, message: msg }
+        return { ok: false, state: stateFromHttpStatus(upstream.status), message: msg }
       }
 
       // 流式：上游已是 OpenAI SSE，原样交回核心写

package/src/suppliers/opencode/plugin.ts

@@ -9,7 +9,8 @@
  * 面板只显示模型卡片（连接池、签到由能力检测自动隐藏）。
  */
 import type { ChatRequest, ModelInfo } from '../../router/types.ts'
-import type { AccountState, ChatOnceResult, SupplierEnv, SupplierModule, SupplierStatusNow } from '../contract.ts'
+import type { ChatOnceResult, SupplierEnv, SupplierModule, SupplierStatusNow } from '../contract.ts'
+import { stateFromHttpStatus } from '../http-state.ts'
 
 
 export const id = 'opencode'
@@ -125,9 +126,7 @@ export default function factory(env: SupplierEnv): SupplierModule {
       if (upstream.status < 200 || upstream.status >= 300) {
         const text = await upstream.text().catch(() => '')
         const msg = `upstream ${upstream.status}: ${text.slice(0, 200)}`
-        const state: AccountState =
-          upstream.status === 429 ? 'rate_limit' : upstream.status === 404 ? 'unavailable' : 'unknown'
-        return { ok: false, state, message: msg }
+        return { ok: false, state: stateFromHttpStatus(upstream.status), message: msg }
       }
 
       // 流式：上游已是 OpenAI SSE，原样交回核心（核心负责写响应）

package/src/suppliers/openrouter/plugin.ts

@@ -12,6 +12,7 @@
 import type { ChatRequest, ModelInfo } from '../../router/types.ts'
 import type { AccountState, ChatOnceResult, SupplierStatusNow } from '../contract.ts'
 import type { SupplierEnv, SupplierModule } from '../contract.ts'
+import { stateFromHttpStatus } from '../http-state.ts'
 
 export const id = 'openrouter'
 export const name = 'OpenRouter'
@@ -202,13 +203,8 @@ export default function factory(env: SupplierEnv): SupplierModule {
       if (upstream.status < 200 || upstream.status >= 300) {
         const text = await upstream.text().catch(() => '')
         const msg = `upstream ${upstream.status}: ${text.slice(0, 120)}`
-        // 401/403 = 这个 key 本身不可用；429 = 限流；其余归为说不清
-        const state: AccountState =
-          upstream.status === 429 ? 'rate_limit'
-            : upstream.status === 401 || upstream.status === 403 ? 'session_dead'
-              : upstream.status === 404 ? 'unavailable'
-                : 'unknown'
-        return { ok: false, state, message: msg }
+        // 401/403 = 这个 key 本身不可用；429 = 限流；其余见 stateFromHttpStatus
+        return { ok: false, state: stateFromHttpStatus(upstream.status), message: msg }
       }
 
       // 流式：上游已是 OpenAI SSE，原样交回核心写

package/src/suppliers/registry.test.ts

@@ -30,7 +30,7 @@ function fakeRes(): { status: number; body: unknown; res: never } {
   return { get status() { return out.status }, get body() { return out.body }, res: res as never }
 }
 
-type FakeAccount = { uid: string; disabled?: boolean; credits?: number }
+type FakeAccount = { uid: string; credits?: number }
 
 /** 造一个只带 checkinNow 的假供应商：status() 返回账号列表，checkinNow 按 uid 应答。
  *  accounts 可传函数（每次 status() 重新求值，用来模拟插件后台异步刷积分）。 */
@@ -50,7 +50,6 @@ function fixture(
         nickname: a.uid,
         credits: a.credits ?? 0,
         cooling: false,
-        disabled: a.disabled ?? false,
         err_count: 0,
       })),
     }),
@@ -109,10 +108,10 @@ test('签到：遍历所有链接，逐个调 checkinNow(uid)', async () => {
   assert.deepEqual(body.results?.map((r) => r.uid), ['a', 'b'])
 })
 
-test('签到：禁用链接照样签（禁用由插件自己判定，核心不筛）', async () => {
+test('签到：插件自标 disabled 的链接照样签（筛不筛由插件自己判定，核心不筛）', async () => {
   const seen: string[] = []
   const { body } = await postCheckin(
-    fixture([{ uid: 'a', disabled: true }, { uid: 'b' }], async (uid) => {
+    fixture([{ uid: 'a' }, { uid: 'b' }], async (uid) => {
       seen.push(uid)
       // 插件（如 traework）自己会对禁用链接返回 status:'disabled'
       return uid === 'a' ? { ok: false, status: 'disabled', message: '链接已禁用' } : { ok: true, status: 'ok' }

package/src/suppliers/registry.ts

@@ -66,7 +66,7 @@ function cap(set: Set<string>, key: string): boolean {
 
 /** 账号状态指纹（积分 + 健康），用于判断刷新是否落地。 */
 function fingerprint(accounts: SupplierStatus['accounts']): string {
-  return accounts.map((a) => `${a.uid}:${a.credits}:${a.cooling}:${a.disabled}`).join('|')
+  return accounts.map((a) => `${a.uid}:${a.credits}:${a.cooling}`).join('|')
 }
 
 /**