@azumag/opencode-rate-limit-fallback 1.0.18 → 1.0.19

package/dist/index.js

@@ -316,35 +316,28 @@ export const RateLimitFallback = async ({ client, directory }) => {
                 parts: parts,
                 model: { providerID: nextModel.providerID, modelID: nextModel.modelID },
             };
-            // CRITICAL PATH: promptAsync BEFORE abort.
+            // abort → promptAsync: works in TUI, fails in headless (server disposes)
+            // promptAsync → abort: fails in TUI (abort kills the new prompt too)
             //
-            // In headless mode (opencode run), abort → promptAsync fails because:
-            //   1. abort triggers server dispose sequence
-            //   2. promptAsync is accepted and message created
-            //   3. Server starts processing (busy) but dispose interrupts it (idle)
-            //   4. server.instance.disposed — all within ~6ms
-            //
-            // By sending promptAsync FIRST, the server knows there is pending work
-            // before abort triggers the dispose check. When abort cancels the retry
-            // loop and the session goes idle, the server should process the queued
-            // prompt instead of disposing.
+            // Use abort → promptAsync as the primary path (TUI confirmed working).
+            // Headless mode (opencode run) is a known limitation — the server
+            // dispose sequence interrupts the new prompt before it can complete.
             //
             // promptAsync: HTTP POST /session/{id}/prompt_async → 204 (SDK sdk.gen.js).
-            // prompt (sync): blocks until generation completes — do NOT use.
             const t0 = Date.now();
-            await client.session.promptAsync({
-                path: { id: sessionID },
-                body: promptBody,
-            });
-            logToFile(`promptAsync completed for session ${sessionID} (${Date.now() - t0}ms) with model ${nextModel.providerID}/${nextModel.modelID}`);
-            const t1 = Date.now();
             try {
                 await client.session.abort({ path: { id: sessionID } });
-                logToFile(`abort succeeded for session ${sessionID} (${Date.now() - t1}ms, total ${Date.now() - t0}ms)`);
+                logToFile(`abort succeeded for session ${sessionID} (${Date.now() - t0}ms)`);
             }
             catch (abortErr) {
-                logToFile(`abort failed (${Date.now() - t1}ms): ${abortErr}`);
+                logToFile(`abort failed (${Date.now() - t0}ms): ${abortErr}`);
             }
+            const t1 = Date.now();
+            await client.session.promptAsync({
+                path: { id: sessionID },
+                body: promptBody,
+            });
+            logToFile(`promptAsync completed for session ${sessionID} (${Date.now() - t1}ms, total ${Date.now() - t0}ms) with model ${nextModel.providerID}/${nextModel.modelID}`);
             // Toast is best-effort notification. The toast() function (line ~185) has
             // built-in fallback: showToast failure → app.log. After promptAsync the
             // server may already be disposing, so both showToast and app.log could fail.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@azumag/opencode-rate-limit-fallback",
-  "version": "1.0.18",
+  "version": "1.0.19",
   "description": "OpenCode plugin that automatically switches to fallback models when rate limited",
   "type": "module",
   "main": "./dist/index.js",