magpie-marvis 0.2.3 → 0.2.5

package/CHANGELOG.md

@@ -3,7 +3,24 @@
 > 每个版本一节，**一条一行**，说用户能感知的变化。
 > 格式与规矩见 [UPDATING.md](./UPDATING.md)。
 
-## 0.2.3 — 2026-11-06
+## 0.2.5 — 2026-11-06
+
+**修复**
+
+- **`state` 格式与客户端不一致**：app 是 `state=${Math.random()*100}`（**小数**，如 `31.1177412`），
+  我写成了 `Math.floor(Math.random()*1e9)`（整数）。抓包确认服务端收到的是 `845583851` 这种
+- **`loginType` 映射错了**：渲染层 `yct={QC:"1",WX:"2",WXAPP:"2"}`，
+  `Ual-Access-Login-Type` 应该是 `1`/`2`，我写的是 `6`/`5`
+
+**修复（`-109` 的最终根因）**
+
+- **`marvis_oauth` 会被请求两次**：QQ 的 `self_redirect` + 页面自身重载，
+  让同一个 code 打两遍 —— 第一次成功返回 token，第二次因为 code 已消耗而回
+  `-109 RC_PARAMS_INVALID`。我之前读的是**页面最终状态**，拿到的正是第二次。
+  客户端主进程用 `[oauth-dedup]`（"reuse first request"）处理这件事；
+  插件改成监听 `Network.responseReceived`，**只认第一个响应体**，抓到立刻返回
+- **`authorize()` 阻塞 1.4 秒**：里面调了 `findBrowser()`（15 次注册表查询），
+  违反 HOOKS 四·陷阱 1。挪进 `callback()`
 
 **修复**
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "magpie-marvis",
-  "version": "0.2.3",
+  "version": "0.2.5",
   "description": "magpie / OpenCode provider plugin for Tencent Marvis: its DeepSeek V4 Pro, Kimi K2.5, Claude Opus/Sonnet 4.6, GPT-5.4 and Hunyuan models through the Marvis model gateway — no API key to paste, sign in with QQ or WeChat.",
   "type": "module",
   "main": "./index.mjs",

package/src/hooks/methods.mjs

@@ -60,7 +60,9 @@ function oauthLogin(loginType, label, instructions) {
     label,
     async authorize() {
       const guid = deviceGuid()
-      const browser = findBrowser()
+      // ⚠ 不要在这里 `findBrowser()` —— 它要查 15 次注册表（实测 1.4 秒），
+      //   而 `authorize()` 必须**立刻**返回（HOOKS 四·陷阱 1）。
+      //   浏览器查找挪进 `callback`，那时才真正需要它。
       const url = authorizeUrl(loginType, guid)
 
       return {
@@ -70,6 +72,7 @@ function oauthLogin(loginType, label, instructions) {
         method: "auto",
         instructions,
         async callback(_url, _input, signal) {
+          const browser = findBrowser()
           if (!browser) {
             throw new Error(
               "没有找到可用的 Chromium 内核浏览器（Chrome / Edge / 360 / QQ 浏览器都行）。" +

package/src/marvis/browser.mjs

@@ -315,52 +315,58 @@ export async function openIsolatedLogin({ browser, url, match, extraHeaders, tim
       // 导航到授权页（钩子已装好：UA / client hints / navigator.webdriver）
       await send(6, "Page.navigate", { url }).catch(() => {})
 
-      // ⚠ 不拦截、不重放、不替换页面 —— 让浏览器**自己**走完换取请求。
-      //   为什么：`marvis_oauth` 的换取依赖 (a) 浏览器 TLS 指纹（Node 的 undici 会被 WAF 400）、
-      //   (b) 同源 Cookie、(c) Referer。这三样只有「浏览器自己发」才齐全。
-      //   之前用 `Fetch.fulfillRequest` 换成静态页再在页面里 fetch，会因为执行上下文
-      //   随导航销毁而拿不到结果 —— 表现就是卡在「正在完成授权…」。
+      // ⚠⚠ 核心：`marvis_oauth` 会被**请求两次** —— QQ 的 `self_redirect` + 页面自身
+      //   的重载会让同一个 code 打两遍。第一次成功并返回 token，第二次因为 code
+      //   已被消耗而回 `-109 RC_PARAMS_INVALID`。
+      //   客户端主进程用 `[oauth-dedup]` 处理这件事（"reuse first request"）；
+      //   我们这边等价的做法是：**只认第一个响应体**，绝不去读页面最终状态。
       //
-      //   服务端换完 code 会**直接返回 token JSON**，所以只要等页面跳到回调地址、
-      //   再把 body 读出来就行。
+      //   所以：监听 `Network.responseReceived` → 命中 `marvis_oauth` →
+      //   立刻 `Network.getResponseBody` 把正文抓下来，然后马上返回。
       let nextId = 10
+      /** @type {string | null} */
+      let firstRequestId = null
+      /** @type {string} */
       let hitUrl = ""
-      let body = ""
 
-      while (Date.now() < deadline) {
-        if (signal?.aborted) throw new Error("登录已取消")
-
-        let href = ""
+      ws.addEventListener("message", (event) => {
+        let msg
         try {
-          const r = await send(nextId++, "Runtime.evaluate", {
-            expression: "location.href",
-            returnByValue: true,
-          })
-          href = String(r?.result?.value ?? "")
+          msg = JSON.parse(String(event.data))
         } catch {
-          /* 页面正在导航 */
+          return
+        }
+        if (msg.method === "Network.responseReceived" && !firstRequestId) {
+          const { requestId, response } = msg.params ?? {}
+          const u = String(response?.url ?? "")
+          // 只认 2xx/4xx 的**文档**响应；图片/脚本不算
+          const type = String(response?.type ?? "")
+          if (u && match.test(u) && (type === "Document" || type === "Fetch" || type === "XHR")) {
+            firstRequestId = requestId
+            hitUrl = u
+          }
         }
+      })
+
+      while (Date.now() < deadline) {
+        if (signal?.aborted) throw new Error("登录已取消")
 
-        if (href && match.test(href)) {
-          hitUrl = href
-          // 页面可能还在渲染，多读几次直到拿到非空正文
-          for (let i = 0; i < 20; i++) {
+        if (firstRequestId) {
+          // 多试几次：responseReceived 之后 body 可能还没落盘
+          for (let i = 0; i < 15; i++) {
             try {
-              const r = await send(nextId++, "Runtime.evaluate", {
-                expression: "document.body ? document.body.innerText : ''",
-                returnByValue: true,
-              })
-              body = String(r?.result?.value ?? "")
-              if (body.trim()) break
+              const r = await send(nextId++, "Network.getResponseBody", { requestId: firstRequestId })
+              const raw = String(r?.body ?? "")
+              const body = r?.base64Encoded ? Buffer.from(raw, "base64").toString("utf8") : raw
+              if (body.trim()) return { url: hitUrl, body, browser: browser.name }
             } catch {
-              /* 还在导航 */
+              /* body 还没就绪 */
             }
-            await new Promise((r) => setTimeout(r, 400))
+            await new Promise((r) => setTimeout(r, 300))
           }
-          return { url: hitUrl, body, browser: browser.name }
         }
 
-        await new Promise((r) => setTimeout(r, 600))
+        await new Promise((r) => setTimeout(r, 500))
       }
 
     } finally {

package/src/marvis/desktop.mjs

@@ -226,7 +226,7 @@ export function parseLoginState(text) {
       openid: m[1],
       accessToken: m[2],
       refreshToken: m[3] || undefined,
-      loginType: /^WX$/i.test(m[4] ?? "") ? "5" : "6",
+      loginType: /^WX/i.test(m[4] ?? "") ? "2" : "1",
       guid: memGuid ? memGuid[1].toLowerCase() : guid,
       expiresAt: expire ? Number(expire[1]) * 1000 : Date.now() + 30 * 24 * 3600_000,
     }

package/src/marvis/login.mjs

@@ -56,7 +56,9 @@ export function authorizeUrl(loginType, guid) {
     client_id: QQ_APP_ID,
     // ⚠ 必须带 ?login_type=QC，见 REDIRECT 上方说明
     redirect_uri: `${REDIRECT}?login_type=${loginType}`,
-    state: String(Math.floor(Math.random() * 1e9)),
+    // ⚠ 必须与 Marvis 渲染层逐字一致：`state=${Math.random()*100}`
+    //   是**小数**（如 31.1177412），不是整数。服务端会拿它做校验，格式不符会被判参数无效。
+    state: String(Math.random() * 100),
   })
   return `https://graph.qq.com/oauth2.0/authorize?${p}`
 }
@@ -228,7 +230,9 @@ function normalize(info, accessToken, openid) {
     refreshToken: info?.refresh_token ? String(info.refresh_token) : undefined,
     nickname: info?.nick_name ? safeDecode(info.nick_name) : undefined,
     avatar: info?.head_img_url ?? undefined,
-    loginType: "6",
+    // ⚠ `Ual-Access-Login-Type` 的真实取值：QC→"1"、WX/WXAPP→"2"
+    //   （渲染层 `yct={QC:"1",WX:"2",WXAPP:"2"}`）。不是 6。
+    loginType: info?.login_type === "WX" || info?.login_type === "WXAPP" ? "2" : "1",
     expiresAt: expire > 1e9 ? expire * 1000 : Date.now() + 30 * 24 * 3600_000,
   }
 }