answer42 0.4.98__tar.gz → 0.4.99__tar.gz

{answer42-0.4.98 → answer42-0.4.99}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: answer42
-Version: 0.4.98
+Version: 0.4.99
 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>
@@ -164,9 +164,9 @@ answer42 install-skills --dry-run --agent all
 
 ## Linux display prerequisite
 
-На Linux `start_session` **всегда** запускает два изолированных Xvfb-дисплея: один для test manager и второй для test client. Это намеренно не использует существующий `DISPLAY`, даже если Answer42 запущен в живой X11-сессии: автоматизация и её скриншоты не должны захватывать пользовательский рабочий стол.
+На Linux `start_session` использует живую X11-сессию, если процесс Answer42 унаследовал непустой `DISPLAY`: test manager и test client запускаются на этом дисплее, поэтому их окна и скриншоты видны на desktop.
 
-Поэтому пакет `xvfb` обязателен и для desktop-, и для headless-установок:
+Если `DISPLAY` не задан (headless service, SSH без X11 forwarding и т. п.), Answer42 поднимает два собственных изолированных Xvfb-дисплея: один для test manager и второй для test client. В этом режиме пакет `xvfb` обязателен:
 
 ```bash
 # Debian / Ubuntu
@@ -176,7 +176,7 @@ sudo apt install xvfb
 sudo dnf install xorg-x11-server-Xvfb
 ```
 
-Если Xvfb отсутствует, `start_session` завершается до создания ресурсов с понятной подсказкой по установке.
+Если не задан ни доступный `DISPLAY`, ни Xvfb, `start_session` завершается до создания ресурсов с понятной подсказкой по установке. Чтобы desktop-сервис видел живой X11, его launcher/service должен передать корректный `DISPLAY` и права доступа к X server (обычно через `XAUTHORITY`).
 
 ## Скриншоты: stdio и StreamableHTTP
 

{answer42-0.4.98 → answer42-0.4.99}/README.md

@@ -130,9 +130,9 @@ answer42 install-skills --dry-run --agent all
 
 ## Linux display prerequisite
 
-На Linux `start_session` **всегда** запускает два изолированных Xvfb-дисплея: один для test manager и второй для test client. Это намеренно не использует существующий `DISPLAY`, даже если Answer42 запущен в живой X11-сессии: автоматизация и её скриншоты не должны захватывать пользовательский рабочий стол.
+На Linux `start_session` использует живую X11-сессию, если процесс Answer42 унаследовал непустой `DISPLAY`: test manager и test client запускаются на этом дисплее, поэтому их окна и скриншоты видны на desktop.
 
-Поэтому пакет `xvfb` обязателен и для desktop-, и для headless-установок:
+Если `DISPLAY` не задан (headless service, SSH без X11 forwarding и т. п.), Answer42 поднимает два собственных изолированных Xvfb-дисплея: один для test manager и второй для test client. В этом режиме пакет `xvfb` обязателен:
 
 ```bash
 # Debian / Ubuntu
@@ -142,7 +142,7 @@ sudo apt install xvfb
 sudo dnf install xorg-x11-server-Xvfb
 ```
 
-Если Xvfb отсутствует, `start_session` завершается до создания ресурсов с понятной подсказкой по установке.
+Если не задан ни доступный `DISPLAY`, ни Xvfb, `start_session` завершается до создания ресурсов с понятной подсказкой по установке. Чтобы desktop-сервис видел живой X11, его launcher/service должен передать корректный `DISPLAY` и права доступа к X server (обычно через `XAUTHORITY`).
 
 ## Скриншоты: stdio и StreamableHTTP
 

{answer42-0.4.98 → answer42-0.4.99}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "answer42"
-version = "0.4.98"
+version = "0.4.99"
 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"

{answer42-0.4.98 → answer42-0.4.99}/src/mcp_1c/runtime.py

@@ -118,26 +118,53 @@ def find_free_port(start: int = DEFAULT_WS_PORT, end: int | None = None) -> int:
     raise RuntimeError(f"No free port found in range {start}..{end}")
 
 
+def inherited_x11_display() -> str:
+    """Return the caller's live X11 display, if one was intentionally inherited."""
+    if os_support.IS_WINDOWS:
+        return ""
+    return os.getenv("DISPLAY", "").strip()
+
+
 def require_xvfb() -> str:
-    """Return the Xvfb executable or raise an actionable Linux prerequisite error.
+    """Return the Xvfb executable or raise an actionable headless prerequisite error.
 
-    Answer42 deliberately runs the test manager and test client on two isolated
-    X displays.  It must therefore have Xvfb even when the MCP process already
-    has access to a live desktop ``DISPLAY``.
+    Xvfb is needed only when Answer42 did not inherit a live X11 ``DISPLAY``.
     """
     if os_support.IS_WINDOWS:
         raise RuntimeError("Xvfb is not available on Windows")
     xvfb_bin = shutil.which("Xvfb") or "/usr/bin/Xvfb"
     if not Path(xvfb_bin).is_file():
         raise RuntimeError(
-            "Xvfb is required to start an Answer42 session on Linux, including on a host with an active X11 desktop. "
-            "Answer42 starts isolated displays for the test manager and test client. "
+            "Xvfb is required to start an Answer42 session on Linux when DISPLAY is not set. "
+            "Set DISPLAY to an accessible live X11 session, or install Xvfb for headless operation. "
             "Install the 'xvfb' package (Debian/Ubuntu: sudo apt install xvfb; Fedora/RHEL: sudo dnf install xorg-x11-server-Xvfb) "
             "and retry start_session."
         )
     return xvfb_bin
 
 
+def session_displays(display_offset: int) -> tuple[str, str, bool]:
+    """Choose inherited desktop display or two Answer42-owned Xvfb displays.
+
+    Returns ``(manager_display, client_display, starts_xvfb)``. A live X11
+    display is deliberately shared by manager and client: it is the caller's
+    explicit request to run visibly on that desktop. Otherwise the historical
+    isolated two-Xvfb setup is retained for headless operation.
+    """
+    desktop_display = inherited_x11_display()
+    if desktop_display:
+        return desktop_display, desktop_display, False
+    if os_support.IS_WINDOWS:
+        return "", "", False
+    require_xvfb()
+    manager_display = find_free_display(DEFAULT_DISPLAY_START + display_offset)
+    # The first X socket does not exist until start_xvfb() is called, so derive
+    # the second search position from the selected number instead of asking the
+    # same free-display scan twice.
+    client_display = find_free_display(int(manager_display.removeprefix(":")) + 1)
+    return manager_display, client_display, True
+
+
 def start_xvfb(display: str, resolution: str = DEFAULT_DISPLAY_RES) -> subprocess.Popen:
     """Start Xvfb for the given display and return the Popen handle."""
     xvfb_bin = require_xvfb()

{answer42-0.4.98 → answer42-0.4.99}/src/mcp_1c/server.py

@@ -5215,18 +5215,18 @@ async def start_session(
     if sid in _SESSIONS:
         raise ValueError(f"Session {sid!r} already exists.")
 
-    # A desktop DISPLAY is intentionally not reused: two isolated displays
-    # prevent an interactive user desktop from becoming part of automation or
-    # screenshots.  Check the prerequisite before allocating any session state
-    # so a missing package produces a clear error rather than FileNotFoundError.
-    if not os_support.IS_WINDOWS:
+    # A caller that inherited DISPLAY explicitly requests visible automation
+    # on that live X11 session. In a headless process, retain two private Xvfb
+    # displays and validate the prerequisite before allocating session state.
+    inherited_display = runtime.inherited_x11_display()
+    if not os_support.IS_WINDOWS and not inherited_display:
         runtime.require_xvfb()
 
     data_dir = os_support.default_data_dir() / "sessions" / sid
     with _port_allocation_lock():
         port_offset = _stable_session_port_offset(sid)
         display_offset = port_offset // _PORT_BAND_SIZE
-        display = "" if os_support.IS_WINDOWS else runtime.find_free_display(runtime.DEFAULT_DISPLAY_START + display_offset)
+        display, client_display, starts_xvfb = runtime.session_displays(display_offset)
         ws_port = _find_session_port(runtime.DEFAULT_WS_PORT, port_offset, sub_offset=0)
         ibsrv_port = _find_session_port(runtime.DEFAULT_IBSRV_HTTP_PORT, port_offset, sub_offset=0)
         ibsrv_direct_regport = _find_session_port(runtime.DEFAULT_IBSRV_DIRECT_PORT, port_offset, sub_offset=0)
@@ -5256,13 +5256,10 @@ async def start_session(
             bound_snapshot = _snapshot_for_base_url(base_url)
             if bound_snapshot:
                 sess.rag_snapshot = bound_snapshot
-        if display:
+        sess.client_display = client_display
+        if starts_xvfb:
             sess.xvfb_process = runtime.start_xvfb(display)
             sess.xvfb_pid = sess.xvfb_process.pid
-        # Separate display for the test client so screenshots capture only the client window
-        client_display = "" if os_support.IS_WINDOWS else runtime.find_free_display(runtime.DEFAULT_DISPLAY_START + display_offset + 1)
-        sess.client_display = client_display
-        if client_display:
             sess.client_xvfb_process = runtime.start_xvfb(client_display)
             sess.client_xvfb_pid = sess.client_xvfb_process.pid
         await sess.bridge.start()