answer42 0.3.5__tar.gz → 0.4.2__tar.gz

{answer42-0.3.5 → answer42-0.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: answer42
-Version: 0.3.5
+Version: 0.4.2
 Summary: Answer42 — The Answer to Life, Universe, and 1C — UI Driver. MCP-powered 1C:Enterprise UI automation: click, fill, navigate, test, and introspect managed forms through the test-client API
 Author: Marvin (AI Assistant), 42Clouds, and contributors
 Author-email: "Kosolapov Stanislav (proDOOMman)" <prodoomman@gmail.com>
@@ -16,7 +16,7 @@ Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
 Classifier: Topic :: Software Development :: Testing
 Requires-Python: >=3.11
-Requires-Dist: mcp>=1.9.0
+Requires-Dist: mcp<3,>=2.0.0
 Requires-Dist: pydantic>=2.0
 Requires-Dist: websockets>=12.0
 Provides-Extra: dev
@@ -126,6 +126,31 @@ answer42 install-skills --dry-run --agent all
 4. зарегистрируйте MCP-сервер в агентском клиенте;
 5. проверьте установку через `credentials_check`, затем smoke-сессию `start_session` → `active_window` → `stop_session`.
 
+## Транспорты: stdio и StreamableHTTP
+
+По умолчанию Answer42 использует stdio MCP transport (запускается MCP-хостом как subprocess). Для удалённого deployment доступен **StreamableHTTP** режим с MCP auth:
+
+```bash
+answer42 --http --http-host 0.0.0.0 --http-port 8080 \
+  --http-token "static-secret-token" \
+  --http-account-id "tenant-42"
+```
+
+Переменные окружения:
+
+```bash
+export ONEC_MCP_HTTP=1
+export ONEC_MCP_HTTP_HOST=0.0.0.0
+export ONEC_MCP_HTTP_PORT=8080
+export ONEC_MCP_HTTP_TOKEN="static-secret-token"
+export ONEC_MCP_ACCOUNT_ID="tenant-42"
+answer42
+```
+
+- Каждый MCP-запрос должен содержать заголовок `Authorization: Bearer <token>`.
+- `account_id` определяет namespace для credentials. Если не задан, вычисляется как стабильный хеш токена (`token-<16hex>`).
+- Режим по умолчанию **stateless** (`--http-stateless`/`ONEC_MCP_HTTP_STATELESS` default `1`); для resumable sessions передай `--http-stateless`/`0`.
+
 ## Безопасное хранение логинов и паролей
 
 Чтобы креды были доступны MCP-серверу, но не попадали в чат и аргументы tool-call, храните их в локальном файле за пределами репозитория и передайте путь в окружение процесса Answer42:
@@ -140,28 +165,39 @@ export ONEC_MCP_CREDENTIALS_FILE=/secure/path/credentials.json
 ~/.answer42-credentials.json
 ```
 
-Формат файла:
+Формат файла v2 (аккаунт-bound):
 
 ```json
 {
-  "entries": [
-    {
-      "url": "https://example.invalid/infobase",
-      "username": "<USERNAME>",
-      "password": "<PASSWORD>"
-    },
-    {
-      "url": "https://*.example.invalid/*",
-      "username": "<WILDCARD_USERNAME>",
-      "password": "<WILDCARD_PASSWORD>"
+  "version": 2,
+  "accounts": {
+    "tenant-42": {
+      "entries": [
+        {
+          "url": "https://example.invalid/infobase",
+          "username": "<USERNAME>",
+          "password": "<PASSWORD>",
+          "title": "dev-example",
+          "aliases": ["dev42"]
+        },
+        {
+          "url": "https://*.example.invalid/*",
+          "username": "<WILDCARD_USERNAME>",
+          "password": "<WILDCARD_PASSWORD>",
+          "title": "wildcard-example",
+          "aliases": ["we"]
+        }
+      ]
     }
-  ]
+  }
 }
 ```
 
+Устаревший v1 формат (flat `entries`) автоматически загружается в account `legacy` и мигрирует в v2 при следующем сохранении. Рекомендуется явно задать `account_id` (через HTTP auth claim или `ONEC_MCP_ACCOUNT_ID` в stdio) и перенести записи в соответствующий account.
+
 Рекомендуемые права на файл: `0600`.
 
-Правила матчинга: сначала точное совпадение `url`, затем wildcard (`*`, `?`) через `fnmatch`; первое совпадение побеждает. Пароли не логируются и не возвращаются наружу.
+Правила матчинга внутри аккаунта: сначала точное совпадение `url`, затем wildcard (`*`, `?`) через `fnmatch`; первое совпадение побеждает. Пароли не логируются и не возвращаются наружу.
 
 Для сохранения или обновления записи можно использовать MCP-tool:
 
@@ -185,7 +221,7 @@ credentials_remove(url="<TARGET_INFOBASE_URL>")
 credentials_check(base_url="<TARGET_INFOBASE_URL>")
 ```
 
-Ответы этих tools содержат только факт наличия/изменения/удаления записи и URL; логины и пароли не возвращаются. Tool `credentials_list()` оставлен в коде для локальной диагностики, но в OpenClaw-конфигурации его рекомендуется скрывать через `toolFilter.exclude`, чтобы агент не мог получить список URL-шаблонов.
+Ответы этих tools содержат только факт наличия/изменения/удаления записи и URL; логины и пароли не возвращаются. `credentials_list()` дополнительно показывает статус проверки (`verified` / `verification_status`): новые или изменённые записи сохраняются как `unverified`, после успешного `start_session` с этой учёткой помечаются как `verified`, а `unverified` запись удаляется при ошибке авторизации. Tool `credentials_list()` оставлен в коде для локальной диагностики, но в OpenClaw-конфигурации его рекомендуется скрывать через `toolFilter.exclude`, чтобы агент не мог получить список URL-шаблонов.
 
 Остановка сессии:
 

{answer42-0.3.5 → answer42-0.4.2}/README.md

@@ -92,6 +92,31 @@ answer42 install-skills --dry-run --agent all
 4. зарегистрируйте MCP-сервер в агентском клиенте;
 5. проверьте установку через `credentials_check`, затем smoke-сессию `start_session` → `active_window` → `stop_session`.
 
+## Транспорты: stdio и StreamableHTTP
+
+По умолчанию Answer42 использует stdio MCP transport (запускается MCP-хостом как subprocess). Для удалённого deployment доступен **StreamableHTTP** режим с MCP auth:
+
+```bash
+answer42 --http --http-host 0.0.0.0 --http-port 8080 \
+  --http-token "static-secret-token" \
+  --http-account-id "tenant-42"
+```
+
+Переменные окружения:
+
+```bash
+export ONEC_MCP_HTTP=1
+export ONEC_MCP_HTTP_HOST=0.0.0.0
+export ONEC_MCP_HTTP_PORT=8080
+export ONEC_MCP_HTTP_TOKEN="static-secret-token"
+export ONEC_MCP_ACCOUNT_ID="tenant-42"
+answer42
+```
+
+- Каждый MCP-запрос должен содержать заголовок `Authorization: Bearer <token>`.
+- `account_id` определяет namespace для credentials. Если не задан, вычисляется как стабильный хеш токена (`token-<16hex>`).
+- Режим по умолчанию **stateless** (`--http-stateless`/`ONEC_MCP_HTTP_STATELESS` default `1`); для resumable sessions передай `--http-stateless`/`0`.
+
 ## Безопасное хранение логинов и паролей
 
 Чтобы креды были доступны MCP-серверу, но не попадали в чат и аргументы tool-call, храните их в локальном файле за пределами репозитория и передайте путь в окружение процесса Answer42:
@@ -106,28 +131,39 @@ export ONEC_MCP_CREDENTIALS_FILE=/secure/path/credentials.json
 ~/.answer42-credentials.json
 ```
 
-Формат файла:
+Формат файла v2 (аккаунт-bound):
 
 ```json
 {
-  "entries": [
-    {
-      "url": "https://example.invalid/infobase",
-      "username": "<USERNAME>",
-      "password": "<PASSWORD>"
-    },
-    {
-      "url": "https://*.example.invalid/*",
-      "username": "<WILDCARD_USERNAME>",
-      "password": "<WILDCARD_PASSWORD>"
+  "version": 2,
+  "accounts": {
+    "tenant-42": {
+      "entries": [
+        {
+          "url": "https://example.invalid/infobase",
+          "username": "<USERNAME>",
+          "password": "<PASSWORD>",
+          "title": "dev-example",
+          "aliases": ["dev42"]
+        },
+        {
+          "url": "https://*.example.invalid/*",
+          "username": "<WILDCARD_USERNAME>",
+          "password": "<WILDCARD_PASSWORD>",
+          "title": "wildcard-example",
+          "aliases": ["we"]
+        }
+      ]
     }
-  ]
+  }
 }
 ```
 
+Устаревший v1 формат (flat `entries`) автоматически загружается в account `legacy` и мигрирует в v2 при следующем сохранении. Рекомендуется явно задать `account_id` (через HTTP auth claim или `ONEC_MCP_ACCOUNT_ID` в stdio) и перенести записи в соответствующий account.
+
 Рекомендуемые права на файл: `0600`.
 
-Правила матчинга: сначала точное совпадение `url`, затем wildcard (`*`, `?`) через `fnmatch`; первое совпадение побеждает. Пароли не логируются и не возвращаются наружу.
+Правила матчинга внутри аккаунта: сначала точное совпадение `url`, затем wildcard (`*`, `?`) через `fnmatch`; первое совпадение побеждает. Пароли не логируются и не возвращаются наружу.
 
 Для сохранения или обновления записи можно использовать MCP-tool:
 
@@ -151,7 +187,7 @@ credentials_remove(url="<TARGET_INFOBASE_URL>")
 credentials_check(base_url="<TARGET_INFOBASE_URL>")
 ```
 
-Ответы этих tools содержат только факт наличия/изменения/удаления записи и URL; логины и пароли не возвращаются. Tool `credentials_list()` оставлен в коде для локальной диагностики, но в OpenClaw-конфигурации его рекомендуется скрывать через `toolFilter.exclude`, чтобы агент не мог получить список URL-шаблонов.
+Ответы этих tools содержат только факт наличия/изменения/удаления записи и URL; логины и пароли не возвращаются. `credentials_list()` дополнительно показывает статус проверки (`verified` / `verification_status`): новые или изменённые записи сохраняются как `unverified`, после успешного `start_session` с этой учёткой помечаются как `verified`, а `unverified` запись удаляется при ошибке авторизации. Tool `credentials_list()` оставлен в коде для локальной диагностики, но в OpenClaw-конфигурации его рекомендуется скрывать через `toolFilter.exclude`, чтобы агент не мог получить список URL-шаблонов.
 
 Остановка сессии:
 

answer42-0.4.2/credentials.example.json

@@ -0,0 +1,23 @@
+{
+  "version": 2,
+  "accounts": {
+    "default": {
+      "entries": [
+        {
+          "url": "https://example.invalid/infobase",
+          "username": "<USERNAME>",
+          "password": "<PASSWORD>",
+          "verified": false
+        },
+        {
+          "url": "https://*.example.invalid/*",
+          "username": "<WILDCARD_USERNAME>",
+          "password": "<WILDCARD_PASSWORD>",
+          "verified": true,
+          "title": "wildcard-example",
+          "aliases": ["we"]
+        }
+      ]
+    }
+  }
+}

{answer42-0.3.5 → answer42-0.4.2}/pyproject.toml

@@ -1,10 +1,10 @@
 [project]
 name = "answer42"
-version = "0.3.5"
+version = "0.4.2"
 description = "Answer42 — The Answer to Life, Universe, and 1C — UI Driver. MCP-powered 1C:Enterprise UI automation: click, fill, navigate, test, and introspect managed forms through the test-client API"
 readme = "README.md"
 requires-python = ">=3.11"
-dependencies = ["mcp>=1.9.0", "websockets>=12.0", "pydantic>=2.0"]
+dependencies = ["mcp>=2.0.0,<3", "websockets>=12.0", "pydantic>=2.0"]
 keywords = ["mcp", "1c", "1c-enterprise", "ui-automation", "test-client"]
 classifiers = [
     "Development Status :: 3 - Alpha",
@@ -37,6 +37,18 @@ answer42 = "mcp_1c.server:main"
 [tool.ruff]
 line-length = 100
 src = ["src", "tests"]
+lint.ignore = [
+    "BLE001",
+    "DTZ003",
+    "DTZ005",
+    "DTZ007",
+    "DTZ901",
+    "LOG015",
+    "PLW1510",
+    "S110",
+    "S112",
+    "TRY004",
+]
 
 [build-system]
 requires = ["hatchling"]

answer42-0.4.2/scripts/e2e_http_smoke.py

@@ -0,0 +1,229 @@
+#!/usr/bin/env python3
+# ruff: noqa: E402
+"""Autonomous smoke E2E test for Answer42 StreamableHTTP transport.
+
+Starts Answer42 in HTTP mode with a static Bearer token, connects via
+mcp.client.streamable_http_client, and runs a minimal smoke flow:
+- credentials_save / credentials_check scoped to the HTTP account
+- start_session
+- active_window
+- stop_session
+
+Requires the same environment variables as stdio smoke for the 1C target:
+  ONEC_MCP_BASE_URL, ONEC_MCP_USERNAME, ONEC_MCP_PASSWORD, ONEC_MCP_VERSION (optional)
+Override the HTTP token/account with:
+  ONEC_MCP_HTTP_TOKEN (default: generated random token)
+  ONEC_MCP_HTTP_ACCOUNT_ID (default: "http-smoke-<random>")
+"""
+
+from __future__ import annotations
+
+import asyncio
+import json
+import os
+import secrets
+import socket
+import subprocess
+import sys
+import tempfile
+import time
+from pathlib import Path
+
+ROOT = Path(__file__).resolve().parents[1]
+sys.path.insert(0, str(ROOT / "src"))
+
+import httpx2
+from mcp import ClientSession
+from mcp.client.streamable_http import streamable_http_client
+from mcp.client._memory import InMemoryTransport
+from mcp.client.client import Client
+
+
+def _parse_tool_result(result):
+    sc = result.structured_content
+    if isinstance(sc, dict) and "result" in sc:
+        return sc["result"]
+    if sc not in (None, {}):
+        return sc
+    texts = [getattr(item, "text", "") for item in result.content if getattr(item, "type", "") == "text"]
+    if len(texts) == 1:
+        try:
+            return json.loads(texts[0])
+        except json.JSONDecodeError:
+            return texts[0]
+    parsed = []
+    for text in texts:
+        try:
+            parsed.append(json.loads(text))
+        except json.JSONDecodeError:
+            parsed.append(text)
+    return parsed
+
+
+async def call_ok(session: ClientSession, name: str, args: dict | None = None):
+    result = await session.call_tool(name, args or {})
+    if result.is_error:
+        raise AssertionError(f"{name} unexpectedly failed: {result.content}")
+    value = _parse_tool_result(result)
+    print(f"OK {name}", flush=True)
+    return value
+
+
+async def main() -> int:
+    base_url = os.environ.get("ONEC_MCP_BASE_URL", "")
+    username = os.environ.get("ONEC_MCP_USERNAME", "")
+    password = os.environ.get("ONEC_MCP_PASSWORD", "")
+    if not base_url or not username or not password:
+        print("Set ONEC_MCP_BASE_URL, ONEC_MCP_USERNAME and ONEC_MCP_PASSWORD", flush=True)
+        return 2
+
+    token = os.environ.get("ONEC_MCP_HTTP_TOKEN") or f"smoke-{secrets.token_urlsafe(24)}"
+    account_id = os.environ.get("ONEC_MCP_HTTP_ACCOUNT_ID") or f"http-smoke-{secrets.token_hex(6)}"
+    host = os.environ.get("ONEC_MCP_HTTP_HOST", "127.0.0.1")
+    port_env = os.environ.get("ONEC_MCP_HTTP_PORT")
+    if port_env:
+        port = int(port_env)
+    else:
+        # Pick a random free port to avoid collisions with OpenClaw gateway/other MCP servers.
+        with socket.socket(socket.AF_INET, socket.SOCK_STREAM) as s:
+            s.bind((host, 0))
+            port = s.getsockname()[1]
+    http_url = f"http://{host}:{port}/mcp"
+
+    # Use a temp credentials file so the smoke does not mutate the user's store.
+    creds_file = Path(tempfile.gettempdir()) / f"answer42-http-smoke-{secrets.token_hex(6)}.json"
+    env = {
+        **os.environ,
+        "ONEC_MCP_CREDENTIALS_FILE": str(creds_file),
+        "ONEC_MCP_HTTP": "1",
+        "ONEC_MCP_HTTP_HOST": host,
+        "ONEC_MCP_HTTP_PORT": str(port),
+        "ONEC_MCP_HTTP_TOKEN": token,
+        "ONEC_MCP_ACCOUNT_ID": account_id,
+        "ONEC_MCP_LOG_LEVEL": "INFO",
+    }
+
+    print(f"=== Answer42 StreamableHTTP smoke ===", flush=True)
+    print(f"endpoint={http_url} account_id={account_id}", flush=True)
+
+    server_cmd = [
+        sys.executable,
+        "-m",
+        "mcp_1c.server",
+    ]
+    proc = subprocess.Popen(
+        server_cmd,
+        cwd=str(ROOT),
+        env=env,
+        stdout=subprocess.PIPE,
+        stderr=subprocess.STDOUT,
+        text=True,
+        bufsize=1,
+    )
+    print(f"server pid={proc.pid}", flush=True)
+
+    # Wait for the HTTP endpoint to become ready.
+    deadline = time.monotonic() + 30.0
+    ready = False
+    http_client = httpx2.AsyncClient(
+        headers={"Authorization": f"Bearer {token}"},
+        timeout=httpx2.Timeout(10.0, read=120.0),
+        follow_redirects=True,
+    )
+    last_err = ""
+    while time.monotonic() < deadline:
+        try:
+            resp = await http_client.head(http_url)
+            if resp.status_code in (200, 202, 404, 405):
+                ready = True
+                break
+        except Exception as exc:
+            last_err = str(exc)
+            await asyncio.sleep(0.2)
+    if not ready:
+        proc.terminate()
+        try:
+            proc.wait(timeout=5)
+        except subprocess.TimeoutExpired:
+            proc.kill()
+        print(f"Server did not become ready: {last_err}", flush=True)
+        if proc.stdout:
+            print(proc.stdout.read(), flush=True)
+        return 1
+
+    try:
+        async with streamable_http_client(http_url, http_client=http_client, terminate_on_close=False) as (
+            read_stream,
+            write_stream,
+        ):
+            async with ClientSession(read_stream, write_stream) as session:
+                init_result = await session.initialize()
+                print("OK initialize", flush=True)
+                if init_result:
+                    server_info = getattr(init_result, "serverInfo", None)
+                    if server_info:
+                        print(f"serverInfo.name={getattr(server_info, 'name', None)} version={getattr(server_info, 'version', None)}", flush=True)
+
+                tools_result = await session.list_tools()
+                tool_names = [t.name for t in tools_result.tools]
+                print(f"OK list_tools ({len(tools_result.tools)} tools): {tool_names}", flush=True)
+
+                await call_ok(
+                    session,
+                    "credentials_save",
+                    {
+                        "url": base_url,
+                        "username": username,
+                        "password": password,
+                        "title": "http-smoke-creds",
+                    },
+                )
+
+                check = await call_ok(session, "credentials_check", {"base_url": base_url})
+                assert check.get("found") is True, f"credentials not found: {check}"
+                assert check.get("account_id") == account_id, f"wrong account_id: {check}"
+                print(f"OK credentials_check account_id={check.get('account_id')}", flush=True)
+
+                session_id = "http-smoke-session"
+                start_args = {
+                    "session_id": session_id,
+                    "base_url": base_url,
+                    "idle_timeout_minutes": 10,
+                }
+                # Let the server auto-resolve credentials by not passing username/password.
+                start = await call_ok(session, "start_session", start_args)
+                assert start.get("session_id") == session_id, f"unexpected session start result: {start}"
+
+                await call_ok(session, "active_window", {"session_id": session_id})
+                await call_ok(session, "stop_session", {"session_id": session_id, "clean_data": True})
+
+        # Verify the credentials file was written in v2 account-bound format.
+        data = json.loads(creds_file.read_text(encoding="utf-8"))
+        assert data.get("version") == 2, f"expected v2 credentials file: {data}"
+        assert account_id in data.get("accounts", {}), f"account {account_id} missing: {data}"
+        print("OK credentials file v2 with correct account", flush=True)
+
+        print("=== Answer42 StreamableHTTP smoke PASSED ===", flush=True)
+        return 0
+    except Exception as exc:
+        import traceback
+        traceback.print_exc()
+        print(f"FAIL: {exc}", flush=True)
+        return 1
+    finally:
+        creds_file.unlink(missing_ok=True)
+        if proc.stdout:
+            remaining = proc.stdout.read()
+            if remaining:
+                print(f"--- server stdout ---\n{remaining}\n--- end server stdout ---", flush=True)
+        proc.terminate()
+        try:
+            proc.wait(timeout=10)
+        except subprocess.TimeoutExpired:
+            proc.kill()
+            proc.wait(timeout=5)
+        await http_client.aclose()
+
+
+if __name__ == "__main__":
+    raise SystemExit(asyncio.run(main()))

{answer42-0.3.5 → answer42-0.4.2}/scripts/e2e_stable.py

@@ -92,7 +92,7 @@ def _safe_suffix(value: str) -> str:
 
 
 def _parse_tool_result(result: Any):
-    sc = result.structuredContent
+    sc = result.structured_content
     if isinstance(sc, dict) and "result" in sc:
         return sc["result"]
     if sc not in (None, {}):
@@ -114,7 +114,7 @@ def _parse_tool_result(result: Any):
 
 async def call_ok(session: ClientSession, name: str, args: dict | None = None):
     result = await session.call_tool(name, args or {})
-    if result.isError:
+    if result.is_error:
         raise AssertionError(f"{name} unexpectedly failed: {result.content}")
     value = _parse_tool_result(result)
     print(f"OK {name}", flush=True)
@@ -125,7 +125,7 @@ async def call_ui_ok(session: ClientSession, name: str, args: dict | None = None
     args = args or {}
     read_timeout = timedelta(seconds=timeout) if timeout else None
     result = await session.call_tool(name, args, read_timeout_seconds=read_timeout)
-    if result.isError:
+    if result.is_error:
         value = "\n".join(getattr(item, "text", "") for item in result.content)
         await _record_manual_step(session, args.get("session_id", "e2e-mcp"), name, args, {"error": value})
         raise AssertionError(f"{name} unexpectedly failed: {result.content}")
@@ -139,7 +139,7 @@ async def call_ui_ok(session: ClientSession, name: str, args: dict | None = None
 async def call_ui_optional_error(session: ClientSession, name: str, args: dict | None = None, allowed_contains: str = ""):
     args = args or {}
     result = await session.call_tool(name, args)
-    if not result.isError:
+    if not result.is_error:
         value = _parse_tool_result(result)
         print(f"OK {name}", flush=True)
         return value
@@ -154,7 +154,7 @@ async def call_ui_optional_error(session: ClientSession, name: str, args: dict |
 async def call_ui_expected_error(session: ClientSession, name: str, args: dict | None = None, contains: str = ""):
     args = args or {}
     result = await session.call_tool(name, args)
-    if not result.isError:
+    if not result.is_error:
         value = _parse_tool_result(result)
         await _record_manual_step(session, args.get("session_id", "e2e-mcp"), name, args, value)
         raise AssertionError(f"{name} unexpectedly succeeded: {value!r}")
@@ -181,7 +181,7 @@ async def _record_manual_step(session: ClientSession, sid: str, name: str, args:
         f"Response: {_summarize_for_note(value, 1200)}"
     )
     rec = await session.call_tool("recording_capture", {"action": name, "note": note, "session_id": sid})
-    if rec.isError:
+    if rec.is_error:
         text = "\n".join(getattr(item, "text", "") for item in rec.content)
         raise AssertionError(f"recording_capture failed after {name}: {text}")
 
@@ -283,7 +283,7 @@ async def run_smoke(session: ClientSession, sid: str, suffix: str, screenshot_di
     for tool in ("goto_start_page", "goto_next_window", "goto_previous_window"):
         res = await session.call_tool(tool, {"session_id": sid})
         print(f"OK_OR_EXPECTED_ERROR {tool}", flush=True)
-        if not res.isError:
+        if not res.is_error:
             _parse_tool_result(res)
 
     await call_ui_ok(session, "wait_window", {"title": windows.get("active_title", ""), "timeout": 3, "session_id": sid})
@@ -323,7 +323,7 @@ async def run_smoke(session: ClientSession, sid: str, suffix: str, screenshot_di
     await call_ui_ok(session, "click_button", {"name": "ФормаЗаписатьИЗакрыть", "session_id": sid})
     await call_ui_ok(session, "table_goto_row", {"name": "Список", "text": "Тестовый smoke", "session_id": sid})
     await call_ui_ok(session, "click_button", {"name": "ФормаИзменить", "session_id": sid})
-    await call_ui_ok(session, "ui_tree", {"session_id": sid, "max_depth": 3, "include_rag_types": True})
+    await call_ui_ok(session, "ui_tree", {"session_id": sid, "include_rag_types": True})
     await call_ui_ok(session, "command_bar", {"session_id": sid})
     await call_ui_ok(session, "current_element", {"session_id": sid})
     await call_ui_ok(session, "click_button", {"name": "ФормаЗаписатьИЗакрыть", "session_id": sid})
@@ -510,7 +510,7 @@ async def run_coverage(session: ClientSession, sid: str, suffix: str, screenshot
     for tool, args, expected in negative_calls:
         res = await session.call_tool(tool, args)
         if expected:
-            if not res.isError:
+            if not res.is_error:
                 value = _parse_tool_result(res)
                 if not (isinstance(value, dict) and value.get("found") is False):
                     raise AssertionError(f"{tool} unexpectedly succeeded: {value!r}")