@magpie-community/opencode-qoder-auth 0.2.18 → 0.2.20

package/README.md

@@ -247,6 +247,39 @@ International campaign discovery was checked with a real account on
 omitting either header hid them. This verification only queried the
 activity; a live claim was not performed.
 
+## Credits available, but a request is refused
+
+A positive credit balance does not guarantee a queue slot or access to
+every model. Use the model ids listed for that particular account and
+region. Do not infer another account's permission from its balance.
+
+Qoder code **10605**, including `RESOURCE_LIMIT`, `serviceAvailable` or
+`isQueued` details, is a resource/queue answer. Keep its suggested retry
+delay; repeatedly signing in does not free that queue. Since **0.2.18**
+([#185](https://github.com/magpie-community/plugins/pull/185)), a 10605 led
+by a BOM or wrapped in an outer message is read as busy too; see
+[#167](https://github.com/magpie-community/plugins/issues/167) for the broader report.
+That parser fix does not remove the provider's limit or prove that every
+live 401 had the same cause. A timeout likewise does not establish an
+empty wallet or an expired login.
+
+For a bounded offline reproduction, run `packages/qoder/busy.test.mjs`
+from #185 in an isolated temporary HOME. It compares plain queue JSON,
+BOM-prefixed JSON and known message wrappers, keeping explicit login and
+credit failures distinct. Extracting a JSON body from a translated log
+cannot recover the original wire bytes; synthetic wrappers demonstrate
+parser behavior, not which wrapper a historical request actually used.
+After selecting a local fix with magpie's `plugin use`, confirm which spec
+serves `qoder` in `plugin list`; a patched checkout alone does not change
+the running provider.
+
+
+For native check-in, an existing `CLAIMED` result is not a new credit grant.
+The host's recheck/cache behavior is tracked in
+[magpie#2696](https://github.com/yetone/magpie/issues/2696). Verify a new
+claim against the vendor record; a successful activity query alone is not
+proof that credits arrived.
+
 ## Not here
 
 - Qoder's usage and quota display.

package/index.mjs

@@ -1015,7 +1015,7 @@ async function* sseLines(body) {
 async function* events(body) {
   const a = new Splitter()
   let index = -1 // the tool call being given
-  let native = -1 // the native tool call being assembled
+  const native = new Map() // upstream index -> call and its output index
   // one piece of the split text as an event: a call, text, or a call that
   // can't be read, which ends the turn as a 502
   const piece = (f) =>
@@ -1055,16 +1055,29 @@ async function* events(body) {
     // tool calls served the OpenAI way keep Qoder's id: a result answers it
     for (const tc of delta.tool_calls ?? []) {
       const i = Number(tc.index ?? 0)
-      if (i !== native) {
-        native = i
+      const name = typeof tc.function?.name === "string" ? tc.function.name.trim() : ""
+      const args = tc.function?.arguments ?? ""
+      if (!tc.id && !name && !args) continue
+      let call = native.get(i)
+      if (!call) {
+        call = { id: "", args: "" }
+        native.set(i, call)
+      }
+      if (tc.id) call.id = tc.id
+      if (call.index === undefined) {
+        // An id or arguments can arrive before the name. Never emit an
+        // unnamed tool: clients persist it and other providers reject it.
+        call.args += args
+        if (!name) continue
         for (const f of a.flush()) {
           const e = piece(f)
           yield e
           if (e.error) return
         }
         a.sawTool = true
-        yield { tool: { index: ++index, id: tc.id || "call_" + hexID(), name: tc.function?.name ?? "", args: tc.function?.arguments ?? "" } }
-      } else if (tc.function?.arguments) yield { args: { index, text: tc.function.arguments } }
+        call.index = ++index
+        yield { tool: { index: call.index, id: call.id || "call_" + hexID(), name, args: call.args } }
+      } else if (args) yield { args: { index: call.index, text: args } }
     }
     fr = chunk?.choices?.[0]?.finish_reason || fr
     // the usage comes in a chunk of its own after the finish, with no
@@ -1085,6 +1098,10 @@ async function* events(body) {
     yield e
     if (e.error) return
   }
+  if ([...native.values()].some((call) => call.index === undefined)) {
+    yield { error: { status: 502, message: "the model's native tool call has no name; it is not sent on as a tool" } }
+    return
+  }
   yield { stop: a.sawTool ? "tool_calls" : fr === "length" ? "length" : "stop", usage }
 }
 
@@ -1111,7 +1128,7 @@ async function answer(res, chat) {
       if (e.text) msg.content += e.text
       if (e.reasoning) reasoning += e.reasoning
       if (e.tool) calls.push({ id: e.tool.id, type: "function", function: { name: e.tool.name, arguments: e.tool.args } })
-      if (e.args) calls[calls.length - 1].function.arguments += e.args.text
+      if (e.args) calls[e.args.index].function.arguments += e.args.text
       if (e.stop) [stop, usage] = [e.stop, e.usage]
     }
     if (reasoning) msg.reasoning_content = reasoning
@@ -1708,9 +1725,8 @@ const makePlugin = (site) => async ({ client }) => {
     try {
       m = (await models(cred)).find((x) => x.key === chat.model) ?? (await models(cred, true)).find((x) => x.key === chat.model)
     } catch (e) {
-      // the built-in answered any failure to read the list 400
-      // (QoderModelOf), a refused one included, marking nothing
-      return errorResponse({ status: 400, message: e.message })
+      if (e instanceof SignInGone || e?.expired) return signedInError(e)
+      return errorResponse({ status: 502, message: e?.message ?? String(e) })
     }
     if (!m) return errorResponse({ status: 400, message: `unknown or disabled model "${chat.model}"` })
     const CHAT_URL = siteOf(site, cred).api + CHAT_PATH

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@magpie-community/opencode-qoder-auth",
-  "version": "0.2.18",
+  "version": "0.2.20",
   "description": "Qoder subscriptions in OpenCode and magpie: Qoder (qoder.com) and Qoder CN (qoder.cn)",
   "type": "module",
   "main": "./index.mjs",