fix: browser tool IPC timeout — add executeJsWithTimeout + isBrowserOpen guard

- browser.ts: add executeJsWithTimeout() wrapping executeJavaScript with 10-15s timeout
  (renderer 冻结时不再无限挂起导致 IPC 25s 超时)
- All browser functions (evaluate/extract/click/type/scroll/screenshot) check isBrowserOpen()
  before execution, return clear error instead of auto-recreating empty browser window
- tool-registry.ts: browser tool IPC timeout increased from 25s to 60s
- Root cause: heavy pages (718+ links) can freeze Chromium renderer, executeJavaScript hangs
  indefinitely; after browser_close, getAgentBrowser() auto-created blank window confusing agent
This commit is contained in:
Metona
2026-04-20 18:10:58 +08:00
parent a8e4f5208c
commit 9999e669d3
2 changed files with 57 additions and 16 deletions
+54 -14
View File
@@ -12,6 +12,11 @@ function sendLog(level: 'info' | 'success' | 'warn' | 'error', message: string,
let agentBrowser: BrowserWindow | null = null; let agentBrowser: BrowserWindow | null = null;
/** 浏览器是否已打开且可用 */
export function isBrowserOpen(): boolean {
return agentBrowser !== null && !agentBrowser.isDestroyed();
}
/** 获取或创建 Agent 浏览器窗口 */ /** 获取或创建 Agent 浏览器窗口 */
function getAgentBrowser(): BrowserWindow { function getAgentBrowser(): BrowserWindow {
if (agentBrowser && !agentBrowser.isDestroyed()) return agentBrowser; if (agentBrowser && !agentBrowser.isDestroyed()) return agentBrowser;
@@ -32,6 +37,23 @@ function getAgentBrowser(): BrowserWindow {
return agentBrowser; return agentBrowser;
} }
/**
* 在 webContents 上执行 JS,带超时保护。
* renderer 冻结时 executeJavaScript 会无限挂起,必须加超时。
*/
function executeJsWithTimeout(webContents: Electron.WebContents, js: string, timeoutMs = 15000): Promise<unknown> {
return new Promise((resolve, reject) => {
const timer = setTimeout(() => {
reject(new Error('浏览器 JS 执行超时(renderer 无响应)'));
}, timeoutMs);
webContents.executeJavaScript(js, true).then(
result => { clearTimeout(timer); resolve(result); },
err => { clearTimeout(timer); reject(err); }
);
});
}
/** 打开 URL */ /** 打开 URL */
export async function browserOpen(url: string): Promise<{ success: boolean; title?: string; url?: string; error?: string }> { export async function browserOpen(url: string): Promise<{ success: boolean; title?: string; url?: string; error?: string }> {
try { try {
@@ -51,7 +73,10 @@ export async function browserOpen(url: string): Promise<{ success: boolean; titl
/** 截图 */ /** 截图 */
export async function browserScreenshot(): Promise<{ success: boolean; png?: string; error?: string }> { export async function browserScreenshot(): Promise<{ success: boolean; png?: string; error?: string }> {
try { try {
const win = getAgentBrowser(); if (!isBrowserOpen()) {
return { success: false, error: '浏览器未打开,请先使用 browser_open 加载页面' };
}
const win = agentBrowser!;
if (win.webContents.getURL() === 'about:blank') { if (win.webContents.getURL() === 'about:blank') {
return { success: false, error: '浏览器未加载任何页面' }; return { success: false, error: '浏览器未加载任何页面' };
} }
@@ -68,12 +93,15 @@ export async function browserScreenshot(): Promise<{ success: boolean; png?: str
/** 执行 JavaScript */ /** 执行 JavaScript */
export async function browserEvaluate(js: string): Promise<{ success: boolean; result?: string; error?: string }> { export async function browserEvaluate(js: string): Promise<{ success: boolean; result?: string; error?: string }> {
try { try {
const win = getAgentBrowser(); if (!isBrowserOpen()) {
return { success: false, error: '浏览器未打开,请先使用 browser_open 加载页面' };
}
const win = agentBrowser!; // isBrowserOpen 已确认非 null
if (win.webContents.getURL() === 'about:blank') { if (win.webContents.getURL() === 'about:blank') {
return { success: false, error: '浏览器未加载任何页面' }; return { success: false, error: '浏览器未加载任何页面,请先使用 browser_open 加载页面' };
} }
sendLog('info', `🌐 browser 执行 JS`, js.slice(0, 100)); sendLog('info', `🌐 browser 执行 JS`, js.slice(0, 100));
const result = await win.webContents.executeJavaScript(js, true); const result = await executeJsWithTimeout(win.webContents, js, 15000);
const resultStr = typeof result === 'string' ? result : JSON.stringify(result, null, 2); const resultStr = typeof result === 'string' ? result : JSON.stringify(result, null, 2);
sendLog('success', `🌐 browser JS 完成`, resultStr?.slice(0, 100)); sendLog('success', `🌐 browser JS 完成`, resultStr?.slice(0, 100));
return { success: true, result: resultStr }; return { success: true, result: resultStr };
@@ -86,12 +114,15 @@ export async function browserEvaluate(js: string): Promise<{ success: boolean; r
/** 提取页面文本和链接 */ /** 提取页面文本和链接 */
export async function browserExtract(): Promise<{ success: boolean; title?: string; text?: string; links?: Array<{text: string; href: string}>; error?: string }> { export async function browserExtract(): Promise<{ success: boolean; title?: string; text?: string; links?: Array<{text: string; href: string}>; error?: string }> {
try { try {
const win = getAgentBrowser(); if (!isBrowserOpen()) {
return { success: false, error: '浏览器未打开,请先使用 browser_open 加载页面' };
}
const win = agentBrowser!;
if (win.webContents.getURL() === 'about:blank') { if (win.webContents.getURL() === 'about:blank') {
return { success: false, error: '浏览器未加载任何页面' }; return { success: false, error: '浏览器未加载任何页面' };
} }
const result = await win.webContents.executeJavaScript(` const result = await executeJsWithTimeout(win.webContents, `
(() => { (() => {
const title = document.title; const title = document.title;
const getText = (el) => { const getText = (el) => {
@@ -112,7 +143,7 @@ export async function browserExtract(): Promise<{ success: boolean; title?: stri
}); });
return { title, text, links: links.slice(0, 50) }; return { title, text, links: links.slice(0, 50) };
})() })()
`, true); `, 15000);
sendLog('success', `🌐 browser 提取完成`, `标题: ${result.title}, 文本: ${result.text?.length || 0}字, 链接: ${result.links?.length || 0}`); sendLog('success', `🌐 browser 提取完成`, `标题: ${result.title}, 文本: ${result.text?.length || 0}字, 链接: ${result.links?.length || 0}`);
return { success: true, ...result }; return { success: true, ...result };
@@ -125,9 +156,12 @@ export async function browserExtract(): Promise<{ success: boolean; title?: stri
/** 点击元素 */ /** 点击元素 */
export async function browserClick(selector: string): Promise<{ success: boolean; error?: string }> { export async function browserClick(selector: string): Promise<{ success: boolean; error?: string }> {
try { try {
const win = getAgentBrowser(); if (!isBrowserOpen()) {
return { success: false, error: '浏览器未打开,请先使用 browser_open 加载页面' };
}
const win = agentBrowser!;
sendLog('info', `🌐 browser 点击`, selector); sendLog('info', `🌐 browser 点击`, selector);
await win.webContents.executeJavaScript(`document.querySelector('${selector.replace(/'/g, "\\'")}').click()`, true); await executeJsWithTimeout(win.webContents, `document.querySelector('${selector.replace(/'/g, "\\'")}').click()`, 10000);
await new Promise(r => setTimeout(r, 500)); await new Promise(r => setTimeout(r, 500));
sendLog('success', `🌐 browser 点击完成`, selector); sendLog('success', `🌐 browser 点击完成`, selector);
return { success: true }; return { success: true };
@@ -140,11 +174,14 @@ export async function browserClick(selector: string): Promise<{ success: boolean
/** 输入文本 */ /** 输入文本 */
export async function browserType(selector: string, text: string): Promise<{ success: boolean; error?: string }> { export async function browserType(selector: string, text: string): Promise<{ success: boolean; error?: string }> {
try { try {
const win = getAgentBrowser(); if (!isBrowserOpen()) {
return { success: false, error: '浏览器未打开,请先使用 browser_open 加载页面' };
}
const win = agentBrowser!;
const escapedSelector = selector.replace(/'/g, "\\'"); const escapedSelector = selector.replace(/'/g, "\\'");
const escapedText = text.replace(/'/g, "\\'").replace(/\n/g, "\\n"); const escapedText = text.replace(/'/g, "\\'").replace(/\n/g, "\\n");
sendLog('info', `🌐 browser 输入`, `${selector}: ${text.slice(0, 50)}`); sendLog('info', `🌐 browser 输入`, `${selector}: ${text.slice(0, 50)}`);
await win.webContents.executeJavaScript(` await executeJsWithTimeout(win.webContents, `
(() => { (() => {
const el = document.querySelector('${escapedSelector}'); const el = document.querySelector('${escapedSelector}');
if (!el) throw new Error('元素未找到: ${escapedSelector}'); if (!el) throw new Error('元素未找到: ${escapedSelector}');
@@ -153,7 +190,7 @@ export async function browserType(selector: string, text: string): Promise<{ suc
el.dispatchEvent(new Event('input', { bubbles: true })); el.dispatchEvent(new Event('input', { bubbles: true }));
el.dispatchEvent(new Event('change', { bubbles: true })); el.dispatchEvent(new Event('change', { bubbles: true }));
})() })()
`, true); `, 10000);
sendLog('success', `🌐 browser 输入完成`, selector); sendLog('success', `🌐 browser 输入完成`, selector);
return { success: true }; return { success: true };
} catch (err) { } catch (err) {
@@ -165,12 +202,15 @@ export async function browserType(selector: string, text: string): Promise<{ suc
/** 滚动页面 */ /** 滚动页面 */
export async function browserScroll(direction: 'down' | 'up' | 'top' | 'bottom'): Promise<{ success: boolean; error?: string }> { export async function browserScroll(direction: 'down' | 'up' | 'top' | 'bottom'): Promise<{ success: boolean; error?: string }> {
try { try {
const win = getAgentBrowser(); if (!isBrowserOpen()) {
return { success: false, error: '浏览器未打开,请先使用 browser_open 加载页面' };
}
const win = agentBrowser!;
const js = direction === 'down' ? 'window.scrollBy(0, 500)' const js = direction === 'down' ? 'window.scrollBy(0, 500)'
: direction === 'up' ? 'window.scrollBy(0, -500)' : direction === 'up' ? 'window.scrollBy(0, -500)'
: direction === 'top' ? 'window.scrollTo(0, 0)' : direction === 'top' ? 'window.scrollTo(0, 0)'
: 'window.scrollTo(0, document.body.scrollHeight)'; : 'window.scrollTo(0, document.body.scrollHeight)';
await win.webContents.executeJavaScript(js, true); await executeJsWithTimeout(win.webContents, js, 10000);
await new Promise(r => setTimeout(r, 300)); await new Promise(r => setTimeout(r, 300));
sendLog('info', `🌐 browser 滚动`, direction); sendLog('info', `🌐 browser 滚动`, direction);
return { success: true }; return { success: true };
+3 -2
View File
@@ -806,11 +806,12 @@ export async function executeTool(toolName: string, args: Record<string, unknown
}; };
} }
// 其他工具:IPC 调用 + 25 秒硬超时 // 其他工具:IPC 调用 + 硬超时(browser 工具 60 秒,其余 25 秒
const timeoutMs = toolName.startsWith('browser_') ? 60000 : 25000;
const result = await Promise.race([ const result = await Promise.race([
bridge.tool.execute(toolName, args), bridge.tool.execute(toolName, args),
new Promise<ToolResult>((_, reject) => new Promise<ToolResult>((_, reject) =>
setTimeout(() => reject(new Error(`工具 ${toolName} IPC 超时(25秒)`)), 25000) setTimeout(() => reject(new Error(`工具 ${toolName} IPC 超时(${Math.round(timeoutMs / 1000)}秒)`)), timeoutMs)
) )
]); ]);
logToolResult(toolName, result.success, result.success ? undefined : result.error); logToolResult(toolName, result.success, result.success ? undefined : result.error);