qwenproxy-cli 1.0.17 → 1.0.19

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "qwenproxy-cli",
-  "version": "1.0.17",
+  "version": "1.0.19",
   "description": "High-performance OpenAI & Anthropic compatible API gateway for Qwen with multi-account rotation, interactive TUI, and resilient tool calling.",
   "main": "src/index.ts",
   "bin": {

package/src/api/server.ts

@@ -721,27 +721,32 @@ export async function startServer(options?: {
 
           // Warm reserve account in background for fast failover without delaying startup
           if (config.playwright.maxActiveContexts > 1 && remainingAccounts.length > 0) {
-            const reserveAccount = remainingAccounts[0];
-            accountsToValidate = remainingAccounts.slice(1);
-            try {
-              const ok = await prepareAccountRuntime(
-                reserveAccount,
-                getAccountCredentials,
-                initPlaywrightForAccount,
-                disableNativeTools,
-                warmQwenChatPool,
-              );
-              if (ok) {
-                ensureAccountInPriority(reserveAccount.id);
-                console.log(
-                  `✅ [Server] Reserve account ready (2/${totalAccounts}): ${maskEmail(reserveAccount.email)}`,
+            let reserveCandidateIdx = 0;
+            for (; reserveCandidateIdx < remainingAccounts.length; reserveCandidateIdx++) {
+              const reserveAccount = remainingAccounts[reserveCandidateIdx];
+              try {
+                const ok = await prepareAccountRuntime(
+                  reserveAccount,
+                  getAccountCredentials,
+                  initPlaywrightForAccount,
+                  disableNativeTools,
+                  warmQwenChatPool,
+                );
+                if (ok) {
+                  ensureAccountInPriority(reserveAccount.id);
+                  console.log(
+                    `✅ [Server] Reserve account ready (2/${totalAccounts}): ${maskEmail(reserveAccount.email)}`,
+                  );
+                  reserveCandidateIdx++;
+                  break;
+                }
+              } catch (err) {
+                console.warn(
+                  `⚠️  [Server] Failed to warm reserve account ${maskEmail(reserveAccount.email)}: ${getErrorMessage(err)}`,
                 );
               }
-            } catch (err) {
-              console.warn(
-                `⚠️  [Server] Failed to warm reserve account ${maskEmail(reserveAccount.email)}: ${getErrorMessage(err)}`,
-              );
             }
+            accountsToValidate = remainingAccounts.slice(reserveCandidateIdx);
           }
 
           let validated = 0;

package/src/index.ts

@@ -14,41 +14,35 @@ if (fs.existsSync(envPath)) {
   dotenv.config({ quiet: true })
 }
 // Prevent benign asynchronous driver/browser teardown exceptions from crashing the server
-process.on('uncaughtException', (error: unknown) => {
-  const msg = error instanceof Error ? error.message : String(error)
-  if (
-    msg.includes('Cannot find parent object') ||
-    msg.includes('Target page, context or browser has been closed') ||
-    msg.includes('Browser has been closed') ||
-    msg.includes('Target closed') ||
-    msg.includes('Target crashed') ||
-    msg.includes('Page crashed') ||
-    msg.includes('Assertion error') ||
-    msg.includes('Connection closed')
-  ) {
-    console.warn(`⚠️  [Playwright] Handled benign driver teardown exception: ${msg}`)
-    return
+process.on('uncaughtException', async (error: unknown) => {
+  const { isPlaywrightAlreadyClosedError } = await import('./services/playwright.ts');
+  if (isPlaywrightAlreadyClosedError(error)) {
+    const msg =
+      error instanceof Error
+        ? error.message
+        : typeof error === 'object' && error !== null && 'message' in error
+          ? String((error as any).message)
+          : String(error);
+    console.warn(`⚠️  [Playwright] Handled benign driver teardown exception: ${msg}`);
+    return;
   }
-  console.error('❌ [Process] Uncaught Exception:', error)
-})
+  console.error('❌ [Process] Uncaught Exception:', error);
+});
 
-process.on('unhandledRejection', (reason: unknown) => {
-  const msg = reason instanceof Error ? reason.message : String(reason)
-  if (
-    msg.includes('Cannot find parent object') ||
-    msg.includes('Target page, context or browser has been closed') ||
-    msg.includes('Browser has been closed') ||
-    msg.includes('Target closed') ||
-    msg.includes('Target crashed') ||
-    msg.includes('Page crashed') ||
-    msg.includes('Assertion error') ||
-    msg.includes('Connection closed')
-  ) {
-    console.warn(`⚠️  [Playwright] Handled benign driver teardown rejection: ${msg}`)
-    return
+process.on('unhandledRejection', async (reason: unknown) => {
+  const { isPlaywrightAlreadyClosedError } = await import('./services/playwright.ts');
+  if (isPlaywrightAlreadyClosedError(reason)) {
+    const msg =
+      reason instanceof Error
+        ? reason.message
+        : typeof reason === 'object' && reason !== null && 'message' in reason
+          ? String((reason as any).message)
+          : String(reason);
+    console.warn(`⚠️  [Playwright] Handled benign driver teardown rejection: ${msg}`);
+    return;
   }
-  console.error('❌ [Process] Unhandled Rejection:', reason)
-})
+  console.error('❌ [Process] Unhandled Rejection:', reason);
+});
 import { startServer } from './api/server.js'
 const isTui = process.argv.includes('--tui') || process.env.QWEN_TUI === 'true'
 

package/src/services/playwright.ts

@@ -279,9 +279,11 @@ export async function saveStorageState(
       `storageState timed out after ${timeoutMs}ms`,
     );
   } catch (error) {
-    console.warn(
-      `[Playwright] Failed to save storage state for ${accountId}: ${getErrorMessage(error)}`,
-    );
+    if (!isPlaywrightAlreadyClosedError(error)) {
+      console.warn(
+        `[Playwright] Failed to save storage state for ${accountId}: ${getErrorMessage(error)}`,
+      );
+    }
   }
 }
 
@@ -302,6 +304,37 @@ async function hasValidAuthCookie(context: BrowserContext, timeoutMs = 3_000): P
   }
 }
 
+/**
+ * Detect whether the page is authenticated.
+ * Probes the authoritative /api/v1/auths/ endpoint from the page context (verified via HAR forensics)
+ * and falls back to inspecting the presence of visible "Log in" / "Sign up" buttons.
+ */
+export async function isPageLoggedIn(page: Page): Promise<boolean> {
+  if (!page) return false;
+  if (typeof page.isClosed === "function" && page.isClosed()) return false;
+  try {
+    const url = typeof page.url === "function" ? page.url() : "";
+    if (url.includes("/auth") || url.includes("/login")) return false;
+    if (typeof page.evaluate !== "function") return true;
+
+    return await page
+      .evaluate(async () => {
+        try {
+          const res = await fetch("/api/v1/auths/", { method: "GET" });
+          return res.status === 200;
+        } catch {
+          const btn = document.querySelector(
+            ".header-right-auth-button, button.header-right-auth-button, a[href*='/auth'], a[href*='/login']",
+          );
+          return !btn || (btn as HTMLElement).offsetWidth === 0;
+        }
+      })
+      .catch(() => false);
+  } catch {
+    return false;
+  }
+}
+
 export async function getOrLaunchSharedBrowser(
   browserType: BrowserType = "chromium",
   headless = true,
@@ -492,7 +525,7 @@ function isAccountServingStream(accountId: string): boolean {
   return true;
 }
 
-function getStealthScript(profile: FingerprintProfile): string {
+export function getStealthScript(profile: FingerprintProfile): string {
   const profileJson = JSON.stringify(profile).replace(/</g, "\\u003c");
   return `
     (function() {
@@ -740,6 +773,14 @@ function getStealthScript(profile: FingerprintProfile): string {
         function makeMime(desc, suffixes, type) {
           return { description: desc, suffixes: suffixes, type: type };
         }
+        function attachPlugin(mime, plugin) {
+          Object.defineProperty(mime, 'enabledPlugin', {
+            value: plugin,
+            enumerable: false,
+            configurable: true,
+            writable: true,
+          });
+        }
         const pdfMime = makeMime('Portable Document Format', 'pdf', 'application/pdf');
         const pdfxMime = makeMime('Portable Document Format', 'pdf', 'text/pdf');
         const pdfPlugin = {
@@ -750,8 +791,8 @@ function getStealthScript(profile: FingerprintProfile): string {
           0: pdfMime,
           1: pdfxMime,
         };
-        pdfMime.enabledPlugin = pdfPlugin;
-        pdfxMime.enabledPlugin = pdfPlugin;
+        attachPlugin(pdfMime, pdfPlugin);
+        attachPlugin(pdfxMime, pdfPlugin);
 
         const chromePdfMime = makeMime('Portable Document Format', 'pdf', 'application/pdf');
         const chromePdfMime2 = makeMime('Portable Document Format', 'pdf', 'text/pdf');
@@ -763,8 +804,8 @@ function getStealthScript(profile: FingerprintProfile): string {
           0: chromePdfMime,
           1: chromePdfMime2,
         };
-        chromePdfMime.enabledPlugin = chromePdfPlugin;
-        chromePdfMime2.enabledPlugin = chromePdfPlugin;
+        attachPlugin(chromePdfMime, chromePdfPlugin);
+        attachPlugin(chromePdfMime2, chromePdfPlugin);
 
         const nativePlugin = {
           name: 'Native Client',
@@ -774,9 +815,8 @@ function getStealthScript(profile: FingerprintProfile): string {
           0: makeMime('Native Client Executable', '', 'application/x-nacl'),
           1: makeMime('Portable Native Client Executable', '', 'application/x-pnacl'),
         };
-        nativePlugin[0].enabledPlugin = nativePlugin;
-        nativePlugin[1].enabledPlugin = nativePlugin;
-
+        attachPlugin(nativePlugin[0], nativePlugin);
+        attachPlugin(nativePlugin[1], nativePlugin);
         const pluginsList = [pdfPlugin, chromePdfPlugin, nativePlugin];
         const mimeList = [pdfMime, pdfxMime, chromePdfMime, chromePdfMime2, nativePlugin[0], nativePlugin[1]];
 
@@ -1358,17 +1398,24 @@ export async function initPlaywrightForAccount(
             waitUntil: "domcontentloaded",
             timeout: config.timeouts.navigation,
           });
-          const url = acctPage.url();
-          if (url.includes("auth") || url.includes("login")) {
+          const loggedIn = await isPageLoggedIn(acctPage);
+          if (!loggedIn) {
             if (account.email && account.password) {
               console.warn(
                 `⚠️  [Playwright] Session expired for ${maskEmail(account.email)}, re-authenticating...`,
               );
-              await loginToQwen(account.id, account.email, account.password);
+              const ok = await loginToQwen(account.id, account.email, account.password);
+              if (!ok) {
+                validationError = new Error(
+                  `Session expired for ${maskEmail(account.email)} and re-authentication failed`,
+                );
+                continue;
+              }
             } else {
-              console.warn(
-                `[Playwright] Session expired for account ${account.id} but no credentials available.`,
+              validationError = new Error(
+                `Session expired for account ${account.id} but no credentials available for re-login (run 'qpx login')`,
               );
+              break;
             }
           }
           validationError = null;
@@ -1389,8 +1436,6 @@ export async function initPlaywrightForAccount(
         );
         throw validationError;
       }
-
-      // Capture headers by navigating and intercepting
       await captureQwenHeaders(account.id);
 
       // Header capture may leave the UI on a generated chat page. Return the
@@ -1411,7 +1456,7 @@ export async function initPlaywrightForAccount(
 
       touchAccountActivity(account.id);
     } catch (error) {
-      await closePlaywrightContextBestEffort(account.id, acctContext);
+      await closePlaywrightContextBestEffort(account.id, acctContext, { skipStorageSave: true });
       cleanupPlaywrightAccountState(account.id);
       throw error;
     }
@@ -1521,23 +1566,20 @@ export async function validateAccountLogin(
         } finally {
           accountPages.delete(account.id);
         }
-      } else if (hasAuthCookie) {
-        // Validate session by navigating to chat page
+      } else {
+        // Validate session by navigating to chat page and checking login state
         try {
           await acctPage.goto(qwenUrl("/"), {
             waitUntil: "domcontentloaded",
             timeout: config.timeouts.navigation,
           });
-          const url = acctPage.url();
-          if (url.includes("auth") || url.includes("login")) {
-            loggedIn = false;
-            if (account.email && account.password) {
-              accountPages.set(account.id, acctPage);
-              try {
-                loggedIn = await loginToQwen(account.id, account.email, account.password);
-              } finally {
-                accountPages.delete(account.id);
-              }
+          loggedIn = await isPageLoggedIn(acctPage);
+          if (!loggedIn && account.email && account.password) {
+            accountPages.set(account.id, acctPage);
+            try {
+              loggedIn = await loginToQwen(account.id, account.email, account.password);
+            } finally {
+              accountPages.delete(account.id);
             }
           }
         } catch {
@@ -1939,59 +1981,64 @@ export async function captureQwenHeaders(
   touchAccountActivity(accountId);
   const cache = getHeaderCache(accountId);
 
-  return new Promise<void>((resolve, reject) => {
-    let settled = false;
-    let routeRegistered = false;
-    let timeout: ReturnType<typeof setTimeout> | undefined;
-    let routeHandler: (route: any, request: any) => Promise<void>;
-    let sawIncompleteHeaders = false;
-    let headersCaptured = false;
-    let retriggerRequested = false;
-    let lastAttemptGraceTimedOut = false;
-    let graceTimeoutCount = 0;
-    let wakeTriggerLoop: (() => void) | undefined;
-    const deadline = Date.now() + timeoutMs;
-    const remainingBudgetMs = () => deadline - Date.now();
-
-    const cleanupRoute = () => {
-      if (!routeRegistered) return;
-      void page
-        .unroute("**/api/v2/chat/completions*", routeHandler)
-        .catch(() => {});
-    };
+  let cleanupRoute = async () => {};
+  try {
+    return await new Promise<void>((resolve, reject) => {
+      let settled = false;
+      let routeRegistered = false;
+      let timeout: ReturnType<typeof setTimeout> | undefined;
+      let routeHandler: (route: any, request: any) => Promise<void>;
+      let sawIncompleteHeaders = false;
+      let headersCaptured = false;
+      let retriggerRequested = false;
+      let lastAttemptGraceTimedOut = false;
+      let graceTimeoutCount = 0;
+      let wakeTriggerLoop: (() => void) | undefined;
+      const deadline = Date.now() + timeoutMs;
+      const remainingBudgetMs = () => deadline - Date.now();
+
+      cleanupRoute = async () => {
+        if (!routeRegistered) return;
+        routeRegistered = false;
+        if (!page.isClosed()) {
+          try {
+            await page.unroute("**/api/v2/chat/completions*", routeHandler);
+          } catch {}
+        }
+      };
 
-    const wakeTrigger = () => {
-      const wake = wakeTriggerLoop;
-      wakeTriggerLoop = undefined;
-      wake?.();
-    };
+      const wakeTrigger = () => {
+        const wake = wakeTriggerLoop;
+        wakeTriggerLoop = undefined;
+        wake?.();
+      };
 
-    const settle = (error?: Error) => {
-      if (settled) return;
-      settled = true;
-      if (timeout) clearTimeout(timeout);
-      cleanupRoute();
-      // A trigger loop parked between attempts has to be released, otherwise it
-      // stays pending forever behind an already-settled capture.
-      wakeTrigger();
-      // When a trigger grace period expired (page fired no completion request),
-      // log the OUTCOME so the operator can see whether the retry loop
-      // recovered or the account is being rotated into cooldown — the bare
-      // per-attempt warning leaves that dangling.
-      if (graceTimeoutCount > 0) {
-        if (headersCaptured) {
-          console.log(
-            `✅ [Playwright] Header capture recovered for ${accountId} after ${graceTimeoutCount} silent send(s)`,
-          );
-        } else {
-          console.warn(
-            `❌ [Playwright] Header capture failed for ${accountId} after ${graceTimeoutCount} silent send(s): ${error?.message ?? "no completion request"}`,
-          );
+      const settle = (error?: Error) => {
+        if (settled) return;
+        settled = true;
+        if (timeout) clearTimeout(timeout);
+        void cleanupRoute();
+        // A trigger loop parked between attempts has to be released, otherwise it
+        // stays pending forever behind an already-settled capture.
+        wakeTrigger();
+        // When a trigger grace period expired (page fired no completion request),
+        // log the OUTCOME so the operator can see whether the retry loop
+        // recovered or the account is being rotated into cooldown — the bare
+        // per-attempt warning leaves that dangling.
+        if (graceTimeoutCount > 0) {
+          if (headersCaptured) {
+            console.log(
+              `✅ [Playwright] Header capture recovered for ${accountId} after ${graceTimeoutCount} silent send(s)`,
+            );
+          } else {
+            console.warn(
+              `❌ [Playwright] Header capture failed for ${accountId} after ${graceTimeoutCount} silent send(s): ${error?.message ?? "no completion request"}`,
+            );
+          }
         }
-      }
-      if (error) reject(error);
-      else resolve();
-    };
+        if (error) reject(error);
+        else resolve();
+      };
 
     const incompleteHeadersError = () =>
       new Error(
@@ -2152,8 +2199,8 @@ export async function captureQwenHeaders(
       // burn every trigger attempt on a textarea that does not exist. Re-login
       // immediately when credentials are available; otherwise fail fast with a
       // clear diagnosis instead of 3 pointless grace timeouts.
-      const currentUrl = page.url();
-      if (currentUrl.includes("/auth") || currentUrl.includes("/login")) {
+      const loggedIn = await isPageLoggedIn(page);
+      if (!loggedIn) {
         const { getAccountCredentials } = await import("../core/accounts.ts");
         const creds = getAccountCredentials(accountId);
         if (creds && creds.email && creds.password) {
@@ -2176,7 +2223,7 @@ export async function captureQwenHeaders(
         } else {
           settle(
             new Error(
-              `Header capture failed for ${accountId}: session expired and no credentials available for re-login`,
+              `Header capture failed for ${accountId}: session expired and no credentials available for re-login (run 'qpx login')`,
             ),
           );
           return;
@@ -2320,11 +2367,12 @@ export async function captureQwenHeaders(
             : new Error(`Header capture route registration failed for ${accountId}`),
         );
       });
-  });
+    });
+  } finally {
+    await cleanupRoute();
+  }
 }
-
 type CookieSnapshot = Awaited<ReturnType<BrowserContext["cookies"]>>;
-
 /**
  * Fetch the account context cookies once. The snapshot feeds every validity
  * check and the cookie string build, avoiding repeated CDP round-trips.
@@ -3022,15 +3070,23 @@ function cleanupPlaywrightAccountState(accountId: string): void {
 async function closePlaywrightContextBestEffort(
   accountId: string,
   context: BrowserContext,
+  options?: { skipStorageSave?: boolean },
 ): Promise<void> {
-  try {
-    if (await hasValidAuthCookie(context)) {
-      await saveStorageState(context, accountId);
-    }
-  } catch {}
+  if (!options?.skipStorageSave) {
+    try {
+      if (await hasValidAuthCookie(context)) {
+        await saveStorageState(context, accountId);
+      }
+    } catch {}
+  }
 
   try {
     const pages = context.pages();
+    for (const page of pages) {
+      if (!page.isClosed()) {
+        await (page as any).unrouteAll?.({ behavior: "ignoreErrors" }).catch(() => {});
+      }
+    }
     await Promise.all(
       pages.map((page) =>
         withTimeout(
@@ -3078,7 +3134,13 @@ async function closePlaywrightForAccountLocked(
  * that must not be logged as keep-alive failures.
  */
 export function isPlaywrightAlreadyClosedError(error: unknown): boolean {
-  const message = error instanceof Error ? error.message : String(error);
+  if (!error) return false;
+  const message =
+    error instanceof Error
+      ? error.message
+      : typeof error === "object" && "message" in error
+        ? String((error as any).message)
+        : String(error);
   return (
     message.includes("Target page, context or browser has been closed") ||
     message.includes("Browser has been closed") ||
@@ -3087,7 +3149,11 @@ export function isPlaywrightAlreadyClosedError(error: unknown): boolean {
     message.includes("Page crashed") ||
     message.includes("Assertion error") ||
     message.includes("Cannot find parent object") ||
-    message.includes("Connection closed")
+    message.includes("Connection closed") ||
+    message.includes("session closed") ||
+    message.includes("Session closed") ||
+    message.includes("Network.setCacheDisabled") ||
+    message.includes("Protocol error")
   );
 }
 

package/src/tools/parser.ts

@@ -962,7 +962,7 @@ function repairCommonMalformedToolJson(content: string): string {
       // a bare-word value, so inserting the quote is safe (true/false/null and
       // numbers are excluded). The trailing `\"` escapes are preserved, so the
       // model's closing quote still terminates the string.
-      /([,{]\s*"[a-zA-Z_][a-zA-Z0-9_]*"\s*:\s*)(?=(?!true|false|null)[A-Za-z_])/g,
+      /([,{]\s*"[a-zA-Z_][a-zA-Z0-9_]*"\s*:\s*)(?=(?!true|false|null|\d|\[|\{|")[^\s])/g,
       '$1"',
     );
   return repairMissingArrayClose(repaired);

package/src/tui/index.ts

@@ -37,6 +37,10 @@ async function main() {
   const app = new TuiApp(initialTab);
 
   process.on("uncaughtException", async (err) => {
+    const { isPlaywrightAlreadyClosedError } = await import("../services/playwright.ts");
+    if (isPlaywrightAlreadyClosedError(err)) {
+      return;
+    }
     try {
       await app.stop();
     } catch {}
@@ -45,6 +49,10 @@ async function main() {
   });
 
   process.on("unhandledRejection", async (err) => {
+    const { isPlaywrightAlreadyClosedError } = await import("../services/playwright.ts");
+    if (isPlaywrightAlreadyClosedError(err)) {
+      return;
+    }
     try {
       await app.stop();
     } catch {}