6ede33ccdb
Build and Push Docker Images / create_manifest (web, surfsense-web, , cpu) (push) Has been cancelled
Build and Push Docker Images / finalize_release (push) Has been cancelled
Obsidian Plugin Lint / lint (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_web, cpu, ./surfsense_web/Dockerfile, web, surfsense-web, ubuntu-24.04-arm, linux/arm64, arm64, , runner, false, cpu) (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_web, cpu, ./surfsense_web/Dockerfile, web, surfsense-web, ubuntu-latest, linux/amd64, amd64, , runner, false, cpu) (push) Has been cancelled
Build and Push Docker Images / compute_version (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_backend, cpu, ./surfsense_backend/Dockerfile, backend, surfsense-backend, ubuntu-24.04-arm, linux/arm64, arm64, , production, false, cpu) (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_backend, cpu, ./surfsense_backend/Dockerfile, backend, surfsense-backend, ubuntu-latest, linux/amd64, amd64, , production, false, cpu) (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_backend, cu126, ./surfsense_backend/Dockerfile, backend, surfsense-backend, ubuntu-24.04-arm, linux/arm64, arm64, -cuda126, production, true, cuda126) (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_backend, cu126, ./surfsense_backend/Dockerfile, backend, surfsense-backend, ubuntu-latest, linux/amd64, amd64, -cuda126, production, true, cuda126) (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_backend, cu128, ./surfsense_backend/Dockerfile, backend, surfsense-backend, ubuntu-24.04-arm, linux/arm64, arm64, -cuda, production, true, cuda) (push) Has been cancelled
Build and Push Docker Images / build (./surfsense_backend, cu128, ./surfsense_backend/Dockerfile, backend, surfsense-backend, ubuntu-latest, linux/amd64, amd64, -cuda, production, true, cuda) (push) Has been cancelled
Build and Push Docker Images / verify_digests (push) Has been cancelled
Build and Push Docker Images / create_manifest (backend, surfsense-backend, , cpu) (push) Has been cancelled
Build and Push Docker Images / create_manifest (backend, surfsense-backend, -cuda, cuda) (push) Has been cancelled
Build and Push Docker Images / create_manifest (backend, surfsense-backend, -cuda126, cuda126) (push) Has been cancelled
139 lines
4.6 KiB
Python
139 lines
4.6 KiB
Python
"""The agent door (05): generate one LangChain tool per registry verb."""
|
|
|
|
from types import SimpleNamespace
|
|
from unittest.mock import AsyncMock
|
|
|
|
import pytest
|
|
from pydantic import BaseModel, Field
|
|
|
|
from app.capabilities.core.types import BillingUnit, Capability
|
|
from app.services.web_crawl_credit_service import InsufficientCreditsError
|
|
|
|
pytestmark = pytest.mark.asyncio
|
|
|
|
|
|
class _EchoInput(BaseModel):
|
|
text: str = Field(description="The text to echo back.")
|
|
|
|
|
|
class _EchoOutput(BaseModel):
|
|
echoed: str
|
|
|
|
@property
|
|
def billable_units(self) -> int:
|
|
return 1
|
|
|
|
|
|
def _capability(
|
|
*, name: str, output: _EchoOutput, unit=BillingUnit.WEB_CRAWL
|
|
) -> Capability:
|
|
async def _executor(payload: _EchoInput) -> _EchoOutput:
|
|
_executor.seen = payload
|
|
return output
|
|
|
|
cap = Capability(
|
|
name=name,
|
|
description=f"{name} does a thing.",
|
|
input_schema=_EchoInput,
|
|
output_schema=_EchoOutput,
|
|
executor=_executor,
|
|
billing_unit=unit,
|
|
)
|
|
cap.executor.seen = None # type: ignore[attr-defined]
|
|
return cap
|
|
|
|
|
|
class _FakeSessionCtx:
|
|
async def __aenter__(self):
|
|
return SimpleNamespace()
|
|
|
|
async def __aexit__(self, *exc):
|
|
return False
|
|
|
|
|
|
@pytest.fixture
|
|
def isolate(monkeypatch):
|
|
"""Stub the billing session + charge/gate so tools never hit the DB."""
|
|
from app.capabilities.core.access import agent as mod
|
|
|
|
monkeypatch.setattr(mod, "async_session_maker", lambda: _FakeSessionCtx())
|
|
charge = AsyncMock()
|
|
gate = AsyncMock()
|
|
monkeypatch.setattr(mod, "charge_capability", charge)
|
|
monkeypatch.setattr(mod, "gate_capability", gate)
|
|
return SimpleNamespace(module=mod, charge=charge, gate=gate)
|
|
|
|
|
|
def _verb_tool(tools, name: str):
|
|
"""Pick one capability tool out of the list (readers are appended after)."""
|
|
return next(t for t in tools if t.name == name)
|
|
|
|
|
|
async def test_registry_becomes_one_tool_per_verb_plus_readers(isolate):
|
|
caps = [
|
|
_capability(name="web.scrape", output=_EchoOutput(echoed="a")),
|
|
_capability(name="web.discover", output=_EchoOutput(echoed="b"), unit=None),
|
|
]
|
|
|
|
tools = isolate.module.build_capability_tools(workspace_id=7, capabilities=caps)
|
|
|
|
by_name = {t.name: t for t in tools}
|
|
# One tool per verb, plus the two shared run-reader tools.
|
|
assert set(by_name) == {"web_scrape", "web_discover", "read_run", "search_run"}
|
|
assert by_name["web_scrape"].description == "web.scrape does a thing."
|
|
assert by_name["web_scrape"].args_schema is _EchoInput
|
|
|
|
|
|
async def test_input_field_docs_reach_the_model(isolate):
|
|
"""Per-field descriptions must surface in the tool's args schema (LLM context)."""
|
|
cap = _capability(name="web.scrape", output=_EchoOutput(echoed="a"))
|
|
tools = isolate.module.build_capability_tools(workspace_id=7, capabilities=[cap])
|
|
tool = _verb_tool(tools, "web_scrape")
|
|
|
|
assert tool.args["text"]["description"] == "The text to echo back."
|
|
|
|
|
|
async def test_tool_runs_executor_and_returns_serialized_output(isolate):
|
|
cap = _capability(name="web.scrape", output=_EchoOutput(echoed="hi there"))
|
|
tools = isolate.module.build_capability_tools(workspace_id=7, capabilities=[cap])
|
|
tool = _verb_tool(tools, "web_scrape")
|
|
|
|
result = await tool.ainvoke({"text": "ping"})
|
|
|
|
# Fake session makes record_run fail -> no run_id key, plain serialized output.
|
|
assert result == {"echoed": "hi there"}
|
|
assert cap.executor.seen.text == "ping"
|
|
|
|
|
|
async def test_tool_charges_owner(isolate):
|
|
output = _EchoOutput(echoed="hi")
|
|
cap = _capability(name="web.scrape", output=output)
|
|
tools = isolate.module.build_capability_tools(workspace_id=7, capabilities=[cap])
|
|
tool = _verb_tool(tools, "web_scrape")
|
|
|
|
await tool.ainvoke({"text": "ping"})
|
|
|
|
isolate.charge.assert_awaited_once()
|
|
(charged_output, unit, ctx), _ = isolate.charge.call_args
|
|
assert charged_output is output
|
|
assert unit is BillingUnit.WEB_CRAWL
|
|
assert ctx.workspace_id == 7
|
|
|
|
|
|
async def test_over_budget_returns_friendly_message(isolate):
|
|
cap = _capability(name="web.scrape", output=_EchoOutput(echoed="hi"))
|
|
isolate.gate.side_effect = InsufficientCreditsError(
|
|
message="This run would exceed your available credit.",
|
|
balance_micros=0,
|
|
required_micros=1_000_000,
|
|
)
|
|
tools = isolate.module.build_capability_tools(workspace_id=7, capabilities=[cap])
|
|
tool = _verb_tool(tools, "web_scrape")
|
|
|
|
result = await tool.ainvoke({"text": "ping"})
|
|
|
|
assert isinstance(result, str)
|
|
assert "credit" in result.lower()
|
|
assert cap.executor.seen is None
|
|
isolate.charge.assert_not_awaited()
|