feat: detailed error reporting — per-account error messages, screenshot context, UI tooltips

account_manager.py

@@ -1,5 +1,6 @@
 import asyncio
 import logging
+import time
 from collections import deque
 from dataclasses import dataclass, field
 from typing import Dict, Optional
@@ -9,8 +10,6 @@ from deepseek_browser import DeepSeekBrowser
 logger = logging.getLogger(__name__)
 
 
-import time
-
 @dataclass
 class Account:
     email: str
@@ -20,6 +19,7 @@ class Account:
     browser: Optional[DeepSeekBrowser] = field(default=None, repr=False)
     in_use: bool = False
     error_count: int = 0
+    last_error: str = ""
     logged_in: bool = False
     is_muted: bool = False
     muted_until: str = ""
@@ -35,10 +35,7 @@ class AccountManager:
 
     def add_account(self, email: str, password: str, name: str = "", proxy: Optional[str] = None):
         self.accounts[email] = Account(
-            email=email,
-            password=password,
-            name=name,
-            proxy=proxy,
+            email=email, password=password, name=name, proxy=proxy,
         )
 
     async def acquire(self) -> Account:
@@ -48,23 +45,19 @@ class AccountManager:
                     account.in_use = True
                     account.last_used = time.time()
                     return account
-
         return await self._wait_for_account()
 
     async def _wait_for_account(self) -> Account:
         event = asyncio.Event()
         async with self._lock:
             self.queue.append(event)
-
         await event.wait()
-
         async with self._lock:
             for account in self.accounts.values():
                 if not account.in_use and account.error_count < 3 and not account.is_muted:
                     account.in_use = True
                     account.last_used = time.time()
                     return account
-
         raise RuntimeError("No account available")
 
     async def release(self, account: Account):
@@ -75,9 +68,10 @@ class AccountManager:
                 event = self.queue.popleft()
                 event.set()
 
-    async def mark_error(self, account: Account):
+    async def mark_error(self, account: Account, error_msg: str = ""):
         async with self._lock:
             account.error_count += 1
+            account.last_error = error_msg
             account.in_use = False
             if self.queue:
                 event = self.queue.popleft()
@@ -99,21 +93,20 @@ class AccountManager:
             if account.browser is None:
                 await self._enforce_browser_limit()
                 account.browser = DeepSeekBrowser(
-                    email=account.email,
-                    password=account.password,
-                    profile_dir="./profiles",
-                    headless=headless,
-                    humanize=True,
-                    proxy=account.proxy,
+                    email=account.email, password=account.password,
+                    profile_dir="./profiles", headless=headless,
+                    humanize=True, proxy=account.proxy,
                 )
                 await account.browser.start()
                 account.logged_in = True
-                # Check mute status
+                account.error_count = 0
+                account.last_error = ""
                 account.is_muted = account.browser.is_muted()
                 account.muted_until = account.browser.muted_until()
             return account.browser
         except Exception as e:
             logger.error("Error creating browser for %s: %s", account.email, e)
+            await self.mark_error(account, str(e))
             await self.close_browser(account)
             raise
 
@@ -141,22 +134,15 @@ class AccountManager:
         muted = sum(1 for a in self.accounts.values() if a.is_muted)
         accounts_list = [
             {
-                "email": a.email,
-                "name": a.name,
-                "in_use": a.in_use,
-                "logged_in": a.logged_in,
-                "is_muted": a.is_muted,
-                "muted_until": a.muted_until,
-                "error_count": a.error_count,
+                "email": a.email, "name": a.name,
+                "in_use": a.in_use, "logged_in": a.logged_in,
+                "is_muted": a.is_muted, "muted_until": a.muted_until,
+                "error_count": a.error_count, "last_error": a.last_error,
             }
             for a in self.accounts.values()
         ]
         return {
-            "total": total,
-            "in_use": in_use,
-            "available": available,
-            "logged_in": logged_in,
-            "muted": muted,
-            "queue_size": len(self.queue),
-            "accounts": accounts_list,
-        }
+            "total": total, "in_use": in_use, "available": available,
+            "logged_in": logged_in, "muted": muted,
+            "queue_size": len(self.queue), "accounts": accounts_list,
+        }

deepseek_browser.py

@@ -32,11 +32,21 @@ class DeepSeekBrowser:
     def _safe_email(self):
         return self.email.replace("@", "_at_").replace("+", "_plus_")
 
-    async def _save_screenshot(self, tag):
+    async def _save_screenshot(self, tag, error_msg=""):
         try:
             SCREENSHOT_DIR.mkdir(parents=True, exist_ok=True)
             path = SCREENSHOT_DIR / f"{tag}_{self._safe_email()}.png"
             await self.page.screenshot(path=str(path))
+            # Write companion error context file
+            if error_msg:
+                txt_path = SCREENSHOT_DIR / f"{tag}_{self._safe_email()}.txt"
+                txt_path.write_text(
+                    f"Account: {self.email}\n"
+                    f"Tag: {tag}\n"
+                    f"Time: {time.strftime('%Y-%m-%d %H:%M:%S')}\n"
+                    f"Error: {error_msg}\n",
+                    encoding="utf-8"
+                )
             logger.error("Screenshot saved to %s", path)
         except Exception as e:
             logger.debug("Screenshot failed: %s", e)
@@ -165,7 +175,7 @@ class DeepSeekBrowser:
             await email_input.fill(self.email)
             await asyncio.sleep(0.1)
         except Exception as e:
-            await self._save_screenshot("login_fail_email")
+            await self._save_screenshot("login_fail_email", str(e))
             logger.error("Email input error: %s", e)
             raise
 
@@ -175,7 +185,7 @@ class DeepSeekBrowser:
             await pwd.fill(self.password)
             await asyncio.sleep(0.1)
         except Exception as e:
-            await self._save_screenshot("login_fail_password")
+            await self._save_screenshot("login_fail_password", str(e))
             logger.error("Password input error: %s", e)
             raise
 
@@ -184,7 +194,7 @@ class DeepSeekBrowser:
             await btn.click()
             await asyncio.sleep(1.5)
         except Exception as e:
-            await self._save_screenshot("login_fail_button")
+            await self._save_screenshot("login_fail_button", str(e))
             logger.error("Login button error: %s", e)
             raise
 
@@ -193,9 +203,9 @@ class DeepSeekBrowser:
             self._logged_in = True
             self._ready = True
             logger.info("Login successful for %s", self.email)
-        except Exception:
-            await self._save_screenshot("login_fail_final")
-            raise Exception("Login failed — screenshot at /static/screenshots/")
+        except Exception as e:
+            await self._save_screenshot("login_fail_final", str(e))
+            raise Exception("Login failed")
 
     async def new_chat(self):
         try:

main.py

@@ -308,7 +308,7 @@ async def chat_completions(
         }
 
     except Exception as e:
-        await manager.mark_error(account)
+        await manager.mark_error(account, str(e))
         logger.error("Chat completion error for model=%s: %s", request.model, e)
         raise HTTPException(status_code=503, detail=str(e))
 
@@ -550,7 +550,7 @@ async def admin_chat(request: Request, admin_key: str = Header(...)):
             "usage": {"prompt_tokens": prompt_tokens, "completion_tokens": completion_tokens, "total_tokens": prompt_tokens + completion_tokens},
         }
     except Exception as e:
-        await manager.mark_error(account)
+        await manager.mark_error(account, str(e))
         logger.error("Admin chat error: %s", e)
         raise HTTPException(status_code=503, detail=str(e))
 
@@ -666,17 +666,23 @@ async def list_screenshots(admin_key: str = Header(...)):
     if not SCREENSHOT_DIR.exists():
         return {"screenshots": []}
     files = sorted(SCREENSHOT_DIR.glob("*.png"), key=lambda p: p.stat().st_mtime, reverse=True)
-    return {
-        "screenshots": [
-            {
-                "name": f.name,
-                "url": f"/static/screenshots/{f.name}",
-                "size_kb": round(f.stat().st_size / 1024, 1),
-                "time": time.strftime("%m-%d %H:%M", time.localtime(f.stat().st_mtime)),
-            }
-            for f in files[:50]
-        ]
-    }
+    results = []
+    for f in files[:50]:
+        txt_path = f.with_suffix(".txt")
+        error_text = ""
+        if txt_path.exists():
+            try:
+                error_text = txt_path.read_text(encoding="utf-8").strip()
+            except Exception:
+                pass
+        results.append({
+            "name": f.name,
+            "url": f"/static/screenshots/{f.name}",
+            "size_kb": round(f.stat().st_size / 1024, 1),
+            "time": time.strftime("%m-%d %H:%M", time.localtime(f.stat().st_mtime)),
+            "error": error_text,
+        })
+    return {"screenshots": results}
 
 
 @app.post("/admin/logs/level")

static/index.html

@@ -591,7 +591,7 @@ async function loadAccounts(){
         <td><span class="badge ${a.logged_in?'badge-on':'badge-off'}">${a.logged_in?'在线':'离线'}</span></td>
         <td><span class="badge ${a.in_use?'badge-on':'badge-idle'}">${a.in_use?'使用中':'空闲'}</span></td>
         <td class="hide-xs">${a.is_muted?`<span class="badge badge-warn" title="${a.muted_until||''}">禁言</span>`:'<span class="badge badge-idle">正常</span>'}</td>
-        <td class="hide-xs">${a.error_count>0?`<span class="badge badge-off">${a.error_count}</span>`:'—'}</td>
+        <td class="hide-xs">${a.error_count>0?`<span class="badge badge-off" title="${(a.last_error||'').replace(/"/g,'&quot;').replace(/'/g,"&#39;")}">${a.error_count}</span>`:'—'}</td>
         <td><button class="btn btn-sm" onclick="doLoginAccount('${a.email}')">${a.logged_in?'重连':'登录'}</button></td>
       </tr>`;
     }
@@ -716,13 +716,15 @@ async function loadScreenshots(){
   try{
     const d=await api('/admin/screenshots',{headers:{'admin-key':getAdminKey()}});
     const el=document.getElementById('ssList');
-    if(!d.screenshots||!d.screenshots.length){el.innerHTML='<span>暂无截图</span>';return}
-    el.innerHTML=d.screenshots.map(s=>
-      `<a href="${s.url}" target="_blank" style="display:inline-flex;align-items:center;gap:4px;padding:4px 10px;background:var(--surface-solid);border:1px solid var(--border);border-radius:6px;color:var(--text);text-decoration:none;font-size:11px" title="${s.name}&#10;${s.time} · ${s.size_kb}KB">
-        🖼️ ${s.name.replace('login_fail_','').replace('_at_gmail.com','').substring(0,25)}
-        <span style="color:var(--text-dim);font-size:10px">${s.time}</span>
-      </a>`
-    ).join('');
+    if(!d.screenshots||!d.screenshots.length){el.innerHTML='<span style="color:var(--text-muted)">暂无截图</span>';return}
+    el.innerHTML=d.screenshots.map(s=>{
+      const label=s.name.replace('login_fail_','').replace('_at_gmail.com','');
+      const errPart=s.error?`<div style="font-size:10px;color:var(--red);margin-top:3px;max-width:300px;overflow:hidden;text-overflow:ellipsis;white-space:nowrap">${s.error.replace(/</g,'&lt;').split('\\n')[0].substring(0,80)}</div>`:'';
+      return `<a href="${s.url}" target="_blank" style="display:inline-block;padding:6px 10px;background:var(--surface-solid);border:1px solid var(--border);border-radius:6px;color:var(--text);text-decoration:none;font-size:11px;margin:3px" title="${s.error||s.name}">
+        🖼️ ${label.substring(0,20)} <span style="color:var(--text-dim);font-size:10px">${s.time} · ${s.size_kb}KB</span>
+        ${errPart}
+      </a>`;
+    }).join('');
   }catch(e){}
 }
 </script>