kaiyuan/chatgpt-on-wechat
1
0
Fork 0
mirror of https://github.com/zhayujie/chatgpt-on-wechat.git synced 2026-07-19 21:07:28 +08:00
Code Issues Packages Projects Releases Wiki Activity

refactor(tools): remove dead playwright-missing branches

Browse Source
This commit is contained in:
zhayujie
2026-07-16 14:56:23 +08:00
parent 356b02ac79
commit 7f8f690497
2 changed files with 6 additions and 33 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
agent/tools/__init__.py
View File

@@ -90,20 +90,14 @@ FileSave = _optional_tools.get('FileSave')
Terminal = _optional_tools.get('Terminal') Terminal = _optional_tools.get('Terminal')
# BrowserTool (requires playwright) # BrowserTool: playwright is soft-imported inside browser_service, so this
# import always succeeds even without playwright. Readiness (playwright pkg /
# system Chrome / downloaded Chromium) is checked at call time in BrowserTool.
def _import_browser_tool(): def _import_browser_tool():
from common.log import logger from common.log import logger
try: try:
from agent.tools.browser.browser_tool import BrowserTool from agent.tools.browser.browser_tool import BrowserTool
return BrowserTool return BrowserTool
except ImportError as e:
logger.info(
f"[Tools] BrowserTool not loaded - missing dependency: {e}\n"
f" To enable browser tool, run:\n"
f" pip install playwright\n"
f" playwright install chromium"
)
return None
except Exception as e: except Exception as e:
logger.error(f"[Tools] BrowserTool failed to load: {e}") logger.error(f"[Tools] BrowserTool failed to load: {e}")
return None return None

27
agent/tools/tool_manager.py
View File

@@ -152,14 +152,7 @@ class ToolManager:
except ImportError as e: except ImportError as e:
# Handle missing dependencies with helpful messages # Handle missing dependencies with helpful messages
error_msg = str(e) error_msg = str(e)
if "playwright" in error_msg: if "markdownify" in error_msg:
logger.warning(
f"[ToolManager] Browser tool not loaded - missing dependencies.\n"
f" To enable browser tool, run:\n"
f" pip install playwright\n"
f" playwright install chromium"
)
elif "markdownify" in error_msg:
logger.warning( logger.warning(
f"[ToolManager] {cls.__name__} not loaded - missing markdownify.\n" f"[ToolManager] {cls.__name__} not loaded - missing markdownify.\n"
f" Install with: pip install markdownify" f" Install with: pip install markdownify"
@@ -222,14 +215,7 @@ class ToolManager:
except ImportError as e: except ImportError as e:
# Handle missing dependencies with helpful messages # Handle missing dependencies with helpful messages
error_msg = str(e) error_msg = str(e)
if "playwright" in error_msg: if "markdownify" in error_msg:
logger.warning(
f"[ToolManager] Browser tool not loaded - missing dependencies.\n"
f" To enable browser tool, run:\n"
f" pip install playwright\n"
f" playwright install chromium"
)
elif "markdownify" in error_msg:
logger.warning( logger.warning(
f"[ToolManager] {cls.__name__} not loaded - missing markdownify.\n" f"[ToolManager] {cls.__name__} not loaded - missing markdownify.\n"
f" Install with: pip install markdownify" f" Install with: pip install markdownify"
@@ -261,14 +247,7 @@ class ToolManager:
# If there are missing tools, record warnings # If there are missing tools, record warnings
if missing_tools: if missing_tools:
for tool_name in missing_tools: for tool_name in missing_tools:
if tool_name == "browser": if tool_name == "google_search":
logger.warning(
f"[ToolManager] Browser tool is configured but not loaded.\n"
f" To enable browser tool, run:\n"
f" pip install playwright\n"
f" playwright install chromium"
)
elif tool_name == "google_search":
logger.warning( logger.warning(
f"[ToolManager] Google Search tool is configured but may need API key.\n" f"[ToolManager] Google Search tool is configured but may need API key.\n"
f" Get API key from: https://serper.dev\n" f" Get API key from: https://serper.dev\n"