Hundao
aaa5d661c3
* fix(tools): move playwright back to main dependencies playwright was moved to the browser extra inc7e85aa9as part of the GCU refactor to use a browser extension. But web_scrape_tool still imports playwright at module level and requires it unconditionally, so CI's Test Tools job breaks with ModuleNotFoundError. web_scrape_tool has no fallback without playwright — it's a hard dependency, not optional. Put it back in main deps. Fixes CI failure on Test Tools (ubuntu-latest). * chore: remove dead test_highlights.py script tools/test_highlights.py is orphaned from the GCU refactor inc7e85aa9: - imports highlight_coordinate and highlight_element from gcu.browser.highlight, but highlight.py was deleted in that refactor - calls BrowserSession.start(), open_tab(), get_active_page(), stop() — none of these methods exist on the current BrowserSession class The script can't run at all, and it's tripping ruff's I001 import-order check (fail on Lint CI after cache invalidation). * test: fix browser/refs tests broken by GCU refactor Tests were still testing the old Playwright-based API afterc7e85aa9moved GCU to an extension-bridge architecture. test_refs.py (6 tests): Refs system now produces CSS selectors like [role="button"][aria-label="Submit"]:nth-of-type(1) for the bridge's DOM matcher, instead of Playwright's role=button[name="Submit"] >> nth=0. Updated expected values to match. Renamed test_escapes_quotes_in_name to test_quoted_name_passes_through and added a comment noting that inner quotes aren't currently escaped (follow-up concern). test_browser_tools_comprehensive.py (4 tests): - test_screenshot_full_page: browser_screenshot passes selector=None when no selector is provided; update assertion. - test_file_upload: browser_upload validates file paths exist on disk. Create real tmp files and mock the CDP calls it makes. - test_evaluate_with_bare_return: renamed to test_evaluate_passes_script_through_to_bridge. IIFE wrapping lives in bridge.evaluate, not in the browser_evaluate tool — mocking the bridge bypasses the wrapping logic, so the tool just passes the script through. - test_evaluate_complex_script: browser_evaluate returns bridge's raw result (no 'ok' wrapper); check for 'result' key instead. test_browser_advanced_tools.py (deleted): The whole file patched get_session and page.wait_for_function (the old Playwright-based API). The bug it guarded against (user text interpolated into a JS source string) is architecturally impossible in the new bridge-based tools, which send text via structured RPC. Coverage for browser_wait exists in test_browser_tools_comprehensive.py. * test(core): fix event_loop tests broken by hive-v1 refactor Several framework tests were left failing or hanging after the hive-v1 refactor landed. This un-breaks CI without touching production code. - Worker auto-escalation: 8 tests were hanging because EventLoopNode with event_bus treats non-queen/non-subagent nodes as workers and auto-escalates to queen, then blocks on _await_user_input forever (no queen in standalone tests). Opt out via is_subagent_mode=True. - MockConversationStore: added clear() to match the production store (storage/conversation_store.py), which event_loop_node.py:425 calls. - Executor output semantics: result.output now only contains terminal- node outputs; two handoff tests now read intermediate outputs from result.session_state["data_buffer"]. - Restore filter: test_restore_from_checkpoint needs set_current_phase so restore()'s phase_id filter matches. - Removed two _build_context tests whose target method no longer exists (replaced by standalone build_node_context()). Remaining execution_id coverage is adequate in TestExecutionId + integration tests. * style: ruff format + drop em dash in comment * test(core): fix remaining framework tests broken by hive-v1 refactor Rounds out the fix started in the previous commit. Full framework suite now passes (1589 passed, 0 failed). - conftest.py: force-bind framework.runner submodules (mcp_registry, mcp_client, mcp_connection_manager) as attributes on the parent package. Without this, pytest monkeypatch.setattr with dotted-string paths fails because the attribute walker can't resolve the submodule even though __init__.py imports from it. Affects ~25 MCP tests. - test_queen_memory: _execute_tool() grew a required caller kwarg for worker type-restrictions. Pass caller="queen" so path-traversal checks run without caller restrictions interfering. - test_session_manager_worker_handoff: _subscribe_worker_digest was removed in the refactor, dropped the dead monkeypatches. - test_skill_context_protection: NodeConversation now reads _run_id in add_tool_result(), so the __new__-based test helper has to initialise it. - test_node_conversation: restore() now filters parts by run_id for crash recovery. Renamed the stale test and flipped the assertion to match the new filtering semantics. - test_tool_registry: CONTEXT_PARAMS was updated (workspace_id out, profile in). Switched the test's example stripped params. * docs: drop circular PR reference in test_refs comment Addresses CodeRabbit nitpick. The comment referenced the PR that was adding the comment, which becomes a self-reference after merge.
132 lines
2.8 KiB
TOML
132 lines
2.8 KiB
TOML
[project]
|
|
name = "tools"
|
|
version = "0.1.0"
|
|
description = "Tools library for the Aden agent framework"
|
|
readme = "README.md"
|
|
requires-python = ">=3.11"
|
|
license = { text = "Apache-2.0" }
|
|
authors = [{ name = "Aden", email = "team@aden.ai" }]
|
|
keywords = ["ai", "agents", "tools", "llm"]
|
|
classifiers = [
|
|
"Development Status :: 3 - Alpha",
|
|
"Intended Audience :: Developers",
|
|
"License :: OSI Approved :: Apache Software License",
|
|
"Programming Language :: Python :: 3",
|
|
"Programming Language :: Python :: 3.11",
|
|
"Programming Language :: Python :: 3.12",
|
|
]
|
|
|
|
dependencies = [
|
|
"pydantic>=2.0.0",
|
|
"httpx>=0.27.0",
|
|
"beautifulsoup4>=4.12.0",
|
|
"pypdf>=4.0.0",
|
|
"pandas>=2.0.0",
|
|
"jsonpath-ng>=1.6.0",
|
|
"fastmcp>=2.0.0",
|
|
"diff-match-patch>=20230430",
|
|
"python-dotenv>=1.0.0",
|
|
"playwright>=1.40.0",
|
|
"playwright-stealth>=2.0.0",
|
|
"litellm==1.81.7", # pinned: supply chain attack in >=1.82.7 (adenhq/hive#6783)
|
|
"dnspython>=2.4.0",
|
|
"resend>=2.0.0",
|
|
"asana>=3.2.0",
|
|
"google-analytics-data>=0.18.0",
|
|
"framework",
|
|
"stripe>=14.3.0",
|
|
"arxiv>=2.1.0",
|
|
"requests>=2.31.0",
|
|
"websockets>=12.0",
|
|
"psycopg2-binary>=2.9.0",
|
|
]
|
|
|
|
[project.optional-dependencies]
|
|
dev = [
|
|
"pytest>=7.0.0",
|
|
"pytest-asyncio>=0.21.0",
|
|
]
|
|
sandbox = [
|
|
"RestrictedPython>=7.0",
|
|
]
|
|
browser = [
|
|
"pillow>=10.0.0",
|
|
]
|
|
ocr = [
|
|
"pytesseract>=0.3.10",
|
|
"pillow>=10.0.0",
|
|
]
|
|
excel = [
|
|
"openpyxl>=3.1.0",
|
|
]
|
|
sql = [
|
|
"duckdb>=1.0.0",
|
|
]
|
|
bigquery = [
|
|
"google-cloud-bigquery>=3.0.0",
|
|
]
|
|
databricks = [
|
|
"databricks-sdk>=0.30.0",
|
|
"databricks-mcp>=0.1.0",
|
|
]
|
|
all = [
|
|
"RestrictedPython>=7.0",
|
|
"pytesseract>=0.3.10",
|
|
"pillow>=10.0.0",
|
|
"duckdb>=1.0.0",
|
|
"openpyxl>=3.1.0",
|
|
"google-cloud-bigquery>=3.0.0",
|
|
"databricks-sdk>=0.30.0",
|
|
"databricks-mcp>=0.1.0",
|
|
]
|
|
|
|
[tool.uv.sources]
|
|
framework = { workspace = true }
|
|
|
|
[build-system]
|
|
requires = ["hatchling"]
|
|
build-backend = "hatchling.build"
|
|
|
|
[tool.hatch.build.targets.wheel]
|
|
packages = ["src/aden_tools"]
|
|
|
|
[tool.ruff]
|
|
target-version = "py311"
|
|
line-length = 100
|
|
|
|
lint.select = [
|
|
"B", # bugbear errors
|
|
"C4", # flake8-comprehensions errors
|
|
"E", # pycodestyle errors
|
|
"F", # pyflakes errors
|
|
"I", # import sorting
|
|
"Q", # flake8-quotes errors
|
|
"UP", # py-upgrade
|
|
"W", # pycodestyle warnings
|
|
]
|
|
|
|
lint.isort.combine-as-imports = true
|
|
lint.isort.known-first-party = ["aden_tools"]
|
|
lint.isort.section-order = [
|
|
"future",
|
|
"standard-library",
|
|
"third-party",
|
|
"first-party",
|
|
"local-folder",
|
|
]
|
|
|
|
[tool.pytest.ini_options]
|
|
testpaths = ["tests"]
|
|
asyncio_mode = "auto"
|
|
addopts = "-m 'not live'"
|
|
markers = [
|
|
"live: Tests that call real external APIs (require credentials, never run in CI)",
|
|
]
|
|
|
|
[dependency-groups]
|
|
dev = [
|
|
"ty>=0.0.13",
|
|
"ruff>=0.14.14",
|
|
"duckdb>=1.4.4",
|
|
]
|