magicrew-auth 0.3.23 → 0.3.24

package/AGENTS.md

@@ -67,6 +67,7 @@ curl -s "https://usemagpie.ai/api/latest?channel=stable" | grep -o '"version":"[
 | 14 | **卡片上额度正常，路由页里这个号却是「剩余额度未知」** | 窗口**不能标 `aside`**：magpie 的 `allowanceOf()` 把 `Aside`/`Text` 的窗口整条跳过，一条不剩 ⇒ 该号 `Known=false`（官网文档 issue #3650 明写）。判据：`curl -s http://127.0.0.1:3425/v1/magpie/trace`，看这个 provider 的 `known` 是 `true` 还是**字段缺失** | HOOKS 三 · `src/hooks/usage.mjs` 头 |
 | 15 | 接 ZCode / Claude Code 系客户端，带工具的请求回 `400 {"error":{"message":"模型响应失败","code":4017}}` | **上游的 `additionalProperties` 只认布尔值，任何对象形态都拒** —— 不只是 `{}`，`{type:"string"}` 这种有约束的 schema 一样 400（两个模型逐行一致）。判据：同一请求只把该字段换成 `true` → 200。改写已在 `rewriteRequestBody` 里做（非布尔一律换 true；**这是降级放宽、会丢约束**，取舍见 internals 十七） | internals 十七 · `tests/body-rewrite.mjs` |
 | 16 | 网关路径（agent 从 magpie 的 `/v1` 进来）报 `502 did not stream: token无效`，而 provider 显示已登录 | auth 里没带 org → `accountOf()` 落 `"default"` → 读不到真实账号的 key 槽 → 只能用 `auth.access`，它过了 2 小时就 403。`accountFor()` 已在**恰好一个真实账号**时兜到它；**多账号不猜**（猜错就是拿别人的 key 扣别人的积分） | internals 十四 · 十五 · `tests/account-fallback.mjs` |
+| 17 | 单轮回 `400 ... tools.N.function.parameters/properties/functions/items: invalid JSON Schema` | 上游不接受**空 schema 对象**当 `items`：`items:{}` 必 400，`items:{description:"x"}` 却 200。判据：同一请求只删 `items:{}` → 200。改写已做（删 `items:{}` / `true`，**严格等价**；`false` 不动） | internals 十七 · `tests/body-rewrite.mjs` |
 
 **坑 14 的两个附带判据**（都实测过）：
 

package/CHANGELOG.md

@@ -1,5 +1,53 @@
 # 更新记录
 
+## 0.3.24 — 2026-10-28
+
+**修 `items: {}` 导致的整轮 400**
+
+### 症状
+
+用户的一次普通对话直接失败，界面只显示：
+
+```
+本轮运行失败 400: {"code":null,"message":"超级麦吉: HTTP 400 请求错误:
+{\"error\":{\"code\":\"InvalidParameter\",\"message\":\"tools.40.function.parameters/
+ properties/functions/items: invalid JSON Schema; check the schema keywords
+ and references Request id: ...\",\"param\":\"\",\"type\":\"BadRequest\"}}",
+ "param":null,"type":"invalid_request_error"}
+```
+
+### 根因
+
+某个工具的参数 schema 里，数组的 `items` 是**空对象 `{}`**。上游的 JSON Schema
+校验器只拒这一种空 schema 形态；`items:{description:"x"}`、`items:{type:"string"}`、
+缺省 `items` 都是 200。国内站 `magicrew` 与国际站 `magicrew-intl` 的报错逐行一致。
+
+按 JSON Schema 规范 `items:{}` 是合法的（"任意元素"），客户端没写错；这是上游
+兼容性不足。但插件是唯一能改请求体的地方。
+
+### 修法
+
+`src/hooks/loader.mjs` 新增 `loosenEmptyItems()`，在 `rewriteRequestBody` 里递归扫
+`tools[].function.parameters`：
+
+```
+items: {}      → 删除      // JSON Schema 缺省值就是 {}，严格等价
+items: true    → 删除      // 同上；上游对它回的是另一种 400
+items: false   → 不动      // 「数组必须为空」，删掉就是放开
+items: {...}   → 不动      // 有任何一个键，上游都收
+```
+
+与 `additionalProperties`、`max_tokens`、`stream_options` 共用一次 `JSON.parse`；
+没改到时仍返回原始字符串。
+
+### 验证
+
+- `tests/body-rewrite.mjs`：31 个断言（新增 11 个），`items:{}` / `true` 删除、
+  `{description}` / `{type}` / `false` 不动、`tools[40]` 命中、combined 改写、不误伤。
+- 端到端（2026-10-28）：41 个工具、第 41 个带 `properties.functions.items:{}`，
+  原样请求 → `400 tools.40.../items`；过 `rewriteRequestBody` → `200`。
+  国内站、国际站都跑过。
+
 ## 0.3.23 — 2026-10-28
 
 **修工具 schema 兼容 + 网关路径拿错 key，顺手把「签一把顶一把」测清楚了**

package/docs/internals.md

@@ -1055,7 +1055,12 @@ magicrew 既不返回 `maxInputTokens`（`flows/models` 的 `configs` 只有 tem
 
 ---
 
-## 十七、上游不认 `additionalProperties: {}` —— 工具 schema 要在请求里兜住
+## 十七、上游不认空 schema —— `additionalProperties: {}` 与 `items: {}`
+
+> 两处是同一类兼容性问题的两个面：客户端按 JSON Schema 规范写出**空对象**，
+> 上游的校验器/适配层拒收。都只能在请求改写里兜。
+
+### 一、`additionalProperties: {}` → `400 code 4017`
 
 **症状**：接 ZCode / Claude Code 系客户端时，一次带 54 个工具定义的请求必定失败：
 
@@ -1111,19 +1116,61 @@ POST /v1/chat/completions
 保留原值就是整个请求失败。两害相权，让工具调用能跑（可能多出未声明的字段）
 比让 agent 完全不可用要好；方向也安全 —— 是放宽不是收紧，不会让原本合法的调用被拒。
 
-它与 `max_tokens`、`stream_options` 三处改写共用**一次** `JSON.parse`；
+它与 `max_tokens`、`stream_options`、`items: {}` 四处改写共用**一次** `JSON.parse`；
 没改到时返回原始字符串（省一次序列化 —— 54 工具的请求体可达上百 KB）。
 
-边界由 `tests/body-rewrite.mjs` 钉住（20 个断言）：非布尔一律换、布尔一律不动、
+边界由 `tests/body-rewrite.mjs` 钉住（31 个断言）：非布尔一律换、布尔一律不动、
 其余字段一字不动、没改到就不重新序列化。
 
+### 二、`items: {}` → `400 invalid JSON Schema`
+
+**症状**（2026-10-28，用户截图原文）：
+
+```
+HTTP 400
+{"code":null,"message":"超级麦吉: HTTP 400 请求错误: {\"error\":{\"code\":\"InvalidParameter\",
+ \"message\":\"tools.40.function.parameters/properties/functions/items: invalid JSON Schema;
+ check the schema keywords and references Request id: ...\",\"param\":\"\",\"type\":\"BadRequest\"}}",
+ "param":null,"type":"invalid_request_error"}
+```
+
+与 `additionalProperties` 那条不同：这次上游**给了完整路径**（`tools.40.../items`），
+而且国内站与 `magicrew-intl` 的报错逐行一致。
+
+**最小复现**：41 个工具，第 41 个的参数是
+
+```json
+{"type":"object","properties":{"functions":{"type":"array","items":{}}},"required":["functions"]}
+```
+
+| `items` | 结果 |
+|---|---|
+| 缺省 | 200 |
+| `{}` | **400 invalid JSON Schema**（路径 `.../properties/functions/items`） |
+| `{description:"x"}` | 200 —— 空对象才炸，不是"缺 type" |
+| `{type:"string"}` | 200 |
+| `true` | 400 `模型响应失败`（另一条 400） |
+| `false` | 400 `模型响应失败` |
+
+**根因**：客户端用 `z.array(z.unknown())` / 动态函数列表生成 schema 时，数组元素序列化成
+`{}`。JSON Schema 里 `items` 的缺省值本来就是 `{}` —— "任意元素"，所以
+`items: {}` 删掉是**严格等价**，不是降级放宽；`items: true` 同理。
+`items: false`（"数组必须为空"）语义不同，不能删，原样留着。
+
+**修法**：`loosenEmptyItems()` 递归删 `items: {}` 与 `items: true`，与
+`loosenAdditionalProperties()` 在同一个 `rewriteRequestBody` 里各跑一遍。
+
+**端到端验证**（2026-10-28）：同一份 41 工具请求
+`raw → 400 tools.40.../items`；过 `rewriteRequestBody` 后 `→ 200`。
+国内站、国际站都验过。
+
 ### 为什么算插件的事，不算客户端的事
 
 按规范 `{}` 与 `true` 等价，客户端没写错；**上游的兼容性不足**。
 但插件是唯一能改请求体的地方（见 `src/hooks/loader.mjs` 顶部），
 在这里兜住比让每个用户去改 ZCode 的源码现实。
 
-### 同一处的第三个改写，以及它的教训
+### 同一处这些改写的两个教训
 
 同一次排查里还确认了两件事，都值得记住：
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "magicrew-auth",
-  "version": "0.3.23",
+  "version": "0.3.24",
   "description": "超级麦吉 / MagiCrew for magpie and OpenCode — one package, two providers: letsmagic.cn (国内) and magicrew.ai (international). Password or web-JWT sign-in, the account's own live model list, points balance, newcomer rewards, invite codes.",
   "type": "module",
   "main": "./index.mjs",

package/src/hooks/loader.mjs

@@ -67,9 +67,61 @@ export function loosenAdditionalProperties(node) {
 }
 
 /**
- * 改写发往上游的请求体。**一次 parse，三处改写都在这**。
+ * 上游的 JSON Schema 校验器不接受 `items: {}`，整轮请求回 HTTP 400：
  *
- * 三处都是实测出来的，不是猜的：
+ *   {"error":{"code":null,"message":"超级麦吉: HTTP 400 请求错误: {\"error\":
+ *    {\"code\":\"InvalidParameter\",\"message\":\"tools.40.function.parameters/
+ *    properties/functions/items: invalid JSON Schema; check the schema
+ *    keywords and references Request id: ...\",\"param\":\"\",\"type\":
+ *    \"BadRequest\"}}","param":null,"type":"invalid_request_error"}}
+ *
+ * 报错只给路径，不说是哪个客户端。实测（2026-10-28，同一把 key、同一份请求、
+ * 只改 tools[0] 的这一处；国内站 `magicrew` 与国际站 `magicrew-intl` 逐行一致）：
+ *
+ *   items: 缺省                → 200
+ *   items: {}                  → 400 invalid JSON Schema
+ *   items: {description:"x"}   → 200   // 空对象才炸，不是"缺 type"
+ *   items: {type:"string"}     → 200
+ *   items: true                → 400 模型响应失败（另一种 400）
+ *   items: false               → 400 模型响应失败
+ *
+ * 为什么可以删：JSON Schema 里 `items` 的缺省值就是 `{}` —— "任意元素"。
+ * 所以 `items: {}` 删掉是**严格等价**（`true` 同理），不是降级放宽。
+ * `items: false` 表示"数组必须为空"，语义不同，不能删，原样留着。
+ *
+ * 命中来源：客户端用 `z.array(z.unknown())` / 动态函数列表生成工具 schema 时，
+ * 数组元素会序列化成 `{}`。一个真实请求里第 41 个工具的参数是
+ * `properties.functions.items: {}`，于是整轮失败。
+ *
+ * @returns {boolean} 有没有改到
+ */
+export function loosenEmptyItems(node) {
+  if (!node || typeof node !== "object") return false
+  if (Array.isArray(node)) {
+    let any = false
+    for (const n of node) if (loosenEmptyItems(n)) any = true
+    return any
+  }
+  let changed = false
+  for (const [k, v] of Object.entries(node)) {
+    if (k === "items") {
+      const emptyObject =
+        v && typeof v === "object" && !Array.isArray(v) && Object.keys(v).length === 0
+      if (emptyObject || v === true) {
+        delete node[k]
+        changed = true
+        continue
+      }
+    }
+    if (v && typeof v === "object" && loosenEmptyItems(v)) changed = true
+  }
+  return changed
+}
+
+/**
+ * 改写发往上游的请求体。**一次 parse，四处改写都在这**。
+ *
+ * 四处都是实测出来的，不是猜的：
  *
  * ① max_tokens 抬到 512
  *    实测 glm-5.3-flash：64 → finish_reason:"length" 且 content:""
@@ -83,6 +135,8 @@ export function loosenAdditionalProperties(node) {
  *
  * ③ tools 里的 additionalProperties: {} → true（见 looser 的注释）
  *
+ * ④ tools 里的 items: {} → 删除（见 loosenEmptyItems 的注释）
+ *
  * 没改到时返回原始字符串（省一次序列化 —— 54 个工具的请求体不小）。
  *
  * @param {unknown} raw
@@ -116,7 +170,9 @@ export function rewriteRequestBody(raw) {
   if (Array.isArray(j.tools)) {
     for (const t of j.tools) {
       const p = t?.function?.parameters
-      if (p && typeof p === "object" && loosenAdditionalProperties(p)) changed = true
+      if (!p || typeof p !== "object") continue
+      if (loosenAdditionalProperties(p)) changed = true
+      if (loosenEmptyItems(p)) changed = true
     }
   }
 
@@ -163,7 +219,7 @@ export function makeLoader(c) {
         let body = init?.body
         if (body === undefined && req) body = await req.text()
 
-        // 三处请求体改写，全在 rewriteRequestBody 里（那里有实测出处）
+        // 四处请求体改写，全在 rewriteRequestBody 里（那里有实测出处）
         body = rewriteRequestBody(body)
 
         return fetch(url, {