magicrew-auth 0.3.24 → 0.3.25

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # 更新记录
 
+## 0.3.25 — 2026-10-28
+
+**0.3.24 的 schema 扫描范围收窄：只走 schema 位置**
+
+0.3.24 的 `loosenEmptyItems()` 与 `loosenAdditionalProperties()` 递归了**所有**对象。
+那会把 `properties` 里恰好叫 `items` / `additionalProperties` 的工具参数当成 schema
+关键字：`properties.items = {}` 的参数会被删掉，`properties.additionalProperties =
+{type:"string"}` 会被改成 `true`；`enum` / `default` 值里的同名键也会被误改。
+
+两者现在共用 `walkSchema()`，只进入真正的 schema 位置：
+
+- `properties` / `patternProperties` / `$defs` / `definitions` / `dependentSchemas` 的**值**
+- `allOf` / `anyOf` / `oneOf` / `prefixItems`
+- `items` / `additionalProperties` / `unevaluatedProperties` / `propertyNames` /
+  `contains` / `not` / `if` / `then` / `else` / `contentSchema` / `dependencies`
+- 不扫键名，不扫 `enum` / `default` / `const` / `examples`（它们不是 schema）
+
+0.3.24 的 `items:{}` 等价替换规则不变，本版只修扫描范围。
+
+### 验证
+
+- `tests/body-rewrite.mjs`：36 个断言（新增 5 个同名属性 / `enum` 值反例）。
+- 端到端复跑：41 个工具、`properties.functions.items:{}` → 200（国内 / 国际）。
+
 ## 0.3.24 — 2026-10-28
 
 **修 `items: {}` 导致的整轮 400**

package/docs/internals.md

@@ -1119,7 +1119,7 @@ POST /v1/chat/completions
 它与 `max_tokens`、`stream_options`、`items: {}` 四处改写共用**一次** `JSON.parse`；
 没改到时返回原始字符串（省一次序列化 —— 54 工具的请求体可达上百 KB）。
 
-边界由 `tests/body-rewrite.mjs` 钉住（31 个断言）：非布尔一律换、布尔一律不动、
+边界由 `tests/body-rewrite.mjs` 钉住（36 个断言）：非布尔一律换、布尔一律不动、
 其余字段一字不动、没改到就不重新序列化。
 
 ### 二、`items: {}` → `400 invalid JSON Schema`
@@ -1157,9 +1157,14 @@ HTTP 400
 `items: {}` 删掉是**严格等价**，不是降级放宽；`items: true` 同理。
 `items: false`（"数组必须为空"）语义不同，不能删，原样留着。
 
-**修法**：`loosenEmptyItems()` 递归删 `items: {}` 与 `items: true`，与
+**修法**：`loosenEmptyItems()` 删 `items: {}` 与 `items: true`，与
 `loosenAdditionalProperties()` 在同一个 `rewriteRequestBody` 里各跑一遍。
 
+两者共用 `walkSchema()`，**只进入真正的 schema 位置**（`properties` 等映射的**值**、
+`allOf` / `anyOf` / `oneOf`、`items` / `additionalProperties` / `not` / `if` 等），
+不扫键名，也不扫 `enum` / `default` / `const`。否则一个恰好叫 `items` 或
+`additionalProperties` 的工具参数会被当成关键字误删/误改（0.3.24 的教训，0.3.25 修）。
+
 **端到端验证**（2026-10-28）：同一份 41 工具请求
 `raw → 400 tools.40.../items`；过 `rewriteRequestBody` 后 `→ 200`。
 国内站、国际站都验过。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "magicrew-auth",
-  "version": "0.3.24",
+  "version": "0.3.25",
   "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

@@ -42,30 +42,82 @@ import { ensureKey, accountOf } from "../keys.mjs"
  * 一次真实请求里 54 个工具中的 `ExitPlanMode` 与 `CreateWorkflow` 就这么写。
  * 客户端没写错（按规范 `{}` 与 `true` 等价），是上游兼容性不足，插件这边兜住。
  *
+ * 只走 schema 位置（见 `walkSchema`）：一个恰好叫 `additionalProperties` 的工具参数
+ * 不会被误改。
+ *
  * @returns {boolean} 有没有改到
  */
-export function loosenAdditionalProperties(node) {
-  if (!node || typeof node !== "object") return false
-  if (Array.isArray(node)) {
-    let any = false
-    for (const n of node) if (loosenAdditionalProperties(n)) any = true
-    return any
-  }
-  let changed = false
+// JSON Schema 里"值是 schema"的关键字。只走这些位置，**不扫 properties 的键名**，
+// 也不扫 enum / default / const / examples —— 那些不是 schema。
+// 旧写法递归所有对象，会把一个叫 `additionalProperties` 的工具参数（在
+// properties 的键上）当成关键字改掉，也会把 enum 值里的 `items` 键删掉。
+const SCHEMA_MAPS = new Set([
+  "properties",
+  "patternProperties",
+  "$defs",
+  "definitions",
+  "dependentSchemas",
+])
+const SCHEMA_ONE = new Set([
+  "additionalProperties",
+  "unevaluatedProperties",
+  "propertyNames",
+  "contains",
+  "not",
+  "if",
+  "then",
+  "else",
+  "contentSchema",
+  "items",
+  "additionalItems",
+  "unevaluatedItems",
+])
+const SCHEMA_LIST = new Set(["allOf", "anyOf", "oneOf", "prefixItems"])
+
+/**
+ * 在 schema 对象上原地跑 `visit`，并只递归进真正的 schema 位置。
+ * @returns {boolean} `visit` 或任一子 schema 改到过
+ */
+function walkSchema(node, visit) {
+  if (!node || typeof node !== "object" || Array.isArray(node)) return false
+  let changed = visit(node)
   for (const [k, v] of Object.entries(node)) {
-    if (k === "additionalProperties") {
-      // 布尔值上游认，原样留着（false 是"不许额外属性"，语义相反，绝对不能动）
-      if (typeof v !== "boolean") {
-        node[k] = true
-        changed = true
-      }
+    if (!v || typeof v !== "object") continue
+    if (SCHEMA_MAPS.has(k) && !Array.isArray(v)) {
+      for (const child of Object.values(v)) if (walkSchema(child, visit)) changed = true
+      continue
+    }
+    if (SCHEMA_LIST.has(k) && Array.isArray(v)) {
+      for (const child of v) if (walkSchema(child, visit)) changed = true
       continue
     }
-    if (v && typeof v === "object" && loosenAdditionalProperties(v)) changed = true
+    if (SCHEMA_ONE.has(k)) {
+      if (Array.isArray(v)) {
+        for (const child of v) if (walkSchema(child, visit)) changed = true
+      } else if (walkSchema(v, visit)) changed = true
+      continue
+    }
+    if (k === "dependencies" && !Array.isArray(v)) {
+      for (const child of Object.values(v)) {
+        if (child && typeof child === "object" && !Array.isArray(child) && walkSchema(child, visit)) {
+          changed = true
+        }
+      }
+    }
   }
   return changed
 }
 
+export function loosenAdditionalProperties(node) {
+  return walkSchema(node, (n) => {
+    // 布尔值上游认，原样留着（false 是"不许额外属性"，语义相反，绝对不能动）
+    if (!Object.hasOwn(n, "additionalProperties")) return false
+    if (typeof n.additionalProperties === "boolean") return false
+    n.additionalProperties = true
+    return true
+  })
+}
+
 /**
  * 上游的 JSON Schema 校验器不接受 `items: {}`，整轮请求回 HTTP 400：
  *
@@ -93,29 +145,21 @@ export function loosenAdditionalProperties(node) {
  * 数组元素会序列化成 `{}`。一个真实请求里第 41 个工具的参数是
  * `properties.functions.items: {}`，于是整轮失败。
  *
+ * 只走 schema 位置：`properties` 里一个恰好叫 `items` 的工具参数（哪怕 schema 就是
+ * `{}`）不会被误删；`enum` / `default` 值里的 `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
+  return walkSchema(node, (n) => {
+    if (!Object.hasOwn(n, "items")) return false
+    const v = n.items
+    const emptyObject =
+      v && typeof v === "object" && !Array.isArray(v) && Object.keys(v).length === 0
+    if (!emptyObject && v !== true) return false
+    delete n.items
+    return true
+  })
 }
 
 /**