arrayview 0.33.0__tar.gz → 0.33.1__tar.gz

{arrayview-0.33.0 → arrayview-0.33.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: arrayview
-Version: 0.33.0
+Version: 0.33.1
 Summary: Fast multi-dimensional array viewer
 Project-URL: Home, https://github.com/oscarvanderheide/arrayview
 Project-URL: Source, https://github.com/oscarvanderheide/arrayview

{arrayview-0.33.0 → arrayview-0.33.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "arrayview"
-version = "0.33.0"
+version = "0.33.1"
 description = "Fast multi-dimensional array viewer"
 readme = { file = "README.md", content-type = "text/markdown" }
 requires-python = ">=3.12"

{arrayview-0.33.0 → arrayview-0.33.1}/src/arrayview/_instance_registry.py

@@ -9,8 +9,6 @@ from __future__ import annotations
 from contextlib import contextmanager
 from dataclasses import asdict, dataclass
 import ctypes
-import getpass
-import hashlib
 import json
 import os
 from pathlib import Path
@@ -70,9 +68,9 @@ def runtime_directory() -> Path:
     base = os.environ.get("XDG_RUNTIME_DIR")
     if base:
         return Path(base) / "arrayview"
-    user = getpass.getuser().encode("utf-8", "replace")
-    suffix = hashlib.sha256(user).hexdigest()[:12]
-    return Path(tempfile.gettempdir()) / f"arrayview-{suffix}"
+    if hasattr(os, "getuid"):
+        return Path(tempfile.gettempdir()) / f"arrayview-{os.getuid()}"
+    return Path(tempfile.gettempdir()) / f"arrayview-{os.getpid()}"
 
 
 def process_start_identity(pid: int) -> str | None:

{arrayview-0.33.0 → arrayview-0.33.1}/src/arrayview/_launch_plan.py

@@ -132,6 +132,7 @@ class ServerSnapshot:
     server_process_start: str | None = None
     server_capabilities: tuple[str, ...] = ()
     server_protocol_version: str | None = None
+    server_uid: int | None = None
 
 
 @dataclass(frozen=True)
@@ -635,6 +636,7 @@ def _server_snapshot(port: int) -> ServerSnapshot:
         port_busy=_port_busy(port),
         arrayview_server_alive=payload is not None,
         server_pid=_int_or_none(payload.get("pid")) if payload else None,
+        server_uid=_int_or_none(payload.get("uid")) if payload else None,
         server_hostname=_str_or_none(payload.get("hostname")) if payload else None,
         server_instance_id=(
             _str_or_none(payload.get("instance_id")) if payload else None
@@ -658,8 +660,14 @@ def _server_snapshot(port: int) -> ServerSnapshot:
 
 
 def _server_reusable(server: ServerSnapshot) -> bool:
+    same_user = (
+        server.server_uid == os.geteuid()
+        if hasattr(os, "geteuid")
+        else True
+    )
     return (
         server.arrayview_server_alive
+        and same_user
         and server.server_instance_id is not None
         and server.server_protocol_version == "1"
         and "identity-fenced-load" in server.server_capabilities

{arrayview-0.33.0 → arrayview-0.33.1}/src/arrayview/_launcher.py

@@ -111,8 +111,13 @@ def _register_server_runtime(port: int, owner_mode: str):
     from arrayview._instance_registry import InstanceRecord, InstanceRegistry
 
     registry = InstanceRegistry()
+    registry._prepare()
     log_directory = registry.directory / "logs"
-    log_directory.mkdir(parents=True, exist_ok=True)
+    log_directory.mkdir(parents=True, exist_ok=True, mode=0o700)
+    try:
+        os.chmod(log_directory, 0o700)
+    except OSError:
+        pass
     record = InstanceRecord.create(
         port=port,
         protocol_version=_server_mod().SERVER_PROTOCOL_VERSION,

{arrayview-0.33.0 → arrayview-0.33.1}/src/arrayview/_server.py

@@ -228,6 +228,7 @@ def ping():
         "ok": True,
         "service": "arrayview",
         "pid": os.getpid(),
+        "uid": os.geteuid() if hasattr(os, "geteuid") else None,
         "hostname": socket.gethostname(),
         "protocol_version": runtime.protocol_version,
         "package_version": _av_version,

{arrayview-0.33.0 → arrayview-0.33.1}/src/arrayview/_vscode_extension.py

@@ -54,7 +54,7 @@ _VSCODE_EXT_INSTALLED = False  # cached so we only check once per process
 _VSCODE_EXT_FRESH_INSTALL = False  # True if we just installed it this session
 _VSCODE_EXT_RELOAD_REQUIRED = False  # installed files are newer than the live host
 _VSCODE_EXT_INSTALL_FAILED = False  # automatic install could not complete safely
-_VSCODE_EXT_VERSION = "0.14.70"  # current bundled extension version
+_VSCODE_EXT_VERSION = "0.14.71"  # current bundled extension version
 _VSCODE_CONFIGURED_PORTS: set[int] = set()
 
 def _bundled_vscode_vsix_version(vsix_path: str) -> str | None:

{arrayview-0.33.0 → arrayview-0.33.1}/tests/test_api.py

@@ -6580,6 +6580,7 @@ def _isolate_view_planner(
             if port in alive_ports
             else (),
             "1" if port in alive_ports else None,
+            server_uid=os.geteuid() if port in alive_ports else None,
         ),
     )
     if alive_ports:

{arrayview-0.33.0 → arrayview-0.33.1}/tests/test_cli.py

@@ -60,6 +60,7 @@ def _mock_launch_server_snapshot(monkeypatch, *, alive: bool, busy: bool | None
             None,
             ("identity-fenced-load", "identity-fenced-mutations") if alive else (),
             "1" if alive else None,
+            server_uid=os.geteuid() if alive else None,
         ),
     )
     if alive:

{arrayview-0.33.0 → arrayview-0.33.1}/tests/test_instance_registry.py

@@ -164,15 +164,17 @@ def test_runtime_directory_override(monkeypatch, tmp_path):
     assert runtime_directory() == tmp_path / "custom"
 
 
-def test_runtime_directory_is_per_user_without_override(monkeypatch, tmp_path):
+def test_runtime_directory_is_per_uid_without_override(monkeypatch, tmp_path):
     monkeypatch.delenv("ARRAYVIEW_RUNTIME_DIR", raising=False)
     monkeypatch.delenv("XDG_RUNTIME_DIR", raising=False)
     monkeypatch.setattr("arrayview._instance_registry.tempfile.gettempdir", lambda: str(tmp_path))
+    current_uid = os.getuid()
     first = runtime_directory()
-    monkeypatch.setattr("arrayview._instance_registry.getpass.getuser", lambda: "different-user")
+    monkeypatch.setattr("arrayview._instance_registry.os.getuid", lambda: current_uid + 1)
     second = runtime_directory()
     assert first.parent == second.parent == tmp_path
     assert first != second
+    assert first.name == f"arrayview-{current_uid}"
 
 
 def test_import_does_not_load_heavy_dependencies():

{arrayview-0.33.0 → arrayview-0.33.1}/tests/test_launch_execution_ownership.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import asyncio
+import os
 from threading import Thread as RealThread
 
 import numpy as np
@@ -167,6 +168,7 @@ def test_http_load_plan_fails_closed_when_selected_server_changes(
             "planned-process-start",
             ("identity-fenced-load", "identity-fenced-mutations"),
             "1",
+            server_uid=os.geteuid(),
         ),
     )
     monkeypatch.setattr(
@@ -222,6 +224,7 @@ def test_display_error_after_http_registration_is_not_relabelled(monkeypatch):
             "planned-process-start",
             ("identity-fenced-load", "identity-fenced-mutations"),
             "1",
+            server_uid=os.geteuid(),
         ),
     )
     monkeypatch.setattr(

{arrayview-0.33.0 → arrayview-0.33.1}/tests/test_launch_plan.py

@@ -1,4 +1,5 @@
 import json
+import os
 import socket
 import sys
 from http.server import BaseHTTPRequestHandler, HTTPServer
@@ -42,6 +43,7 @@ class _PingHandler(BaseHTTPRequestHandler):
                 "ok": True,
                 "service": "arrayview",
                 "pid": 4321,
+                "uid": os.geteuid() if hasattr(os, "geteuid") else None,
                 "hostname": "test-host",
                 "instance_id": "test-instance",
                 "process_start": "test-process-start",
@@ -223,6 +225,9 @@ def test_snapshot_reports_arrayview_server_ping(monkeypatch, clear_launch_env, p
     assert snapshot.server.port_busy is True
     assert snapshot.server.arrayview_server_alive is True
     assert snapshot.server.server_pid == 4321
+    assert snapshot.server.server_uid == (
+        os.geteuid() if hasattr(os, "geteuid") else None
+    )
     assert snapshot.server.server_hostname == "test-host"
     assert snapshot.server.server_instance_id == "test-instance"
     assert snapshot.server.server_process_start == "test-process-start"
@@ -581,6 +586,7 @@ def test_remote_python_reuses_healthy_cli_default_server():
             "remote-process-start",
             ("identity-fenced-load", "identity-fenced-mutations"),
             "1",
+            server_uid=os.geteuid(),
         ),
     )
     plan = plan_launch(LaunchIntent(Invocation.PYTHON, 8123), facts)
@@ -656,6 +662,7 @@ def test_existing_server_is_reused_through_http():
             "process-start",
             ("identity-fenced-load", "identity-fenced-mutations"),
             "1",
+            server_uid=os.geteuid(),
         )
     )
     plan = plan_launch(LaunchIntent(Invocation.CLI, 8123), facts)
@@ -664,6 +671,37 @@ def test_existing_server_is_reused_through_http():
     assert plan.registration.value == "http_load"
 
 
+@pytest.mark.parametrize(
+    "server_uid",
+    [None, os.geteuid() + 1],
+    ids=["missing-uid", "different-uid"],
+)
+def test_unowned_existing_server_uses_new_port(server_uid):
+    from arrayview._launch_plan import Invocation, LaunchIntent, ServerSnapshot, plan_launch
+
+    facts = _facts(
+        server=ServerSnapshot(
+            8123,
+            True,
+            True,
+            99,
+            "host",
+            "instance",
+            "process-start",
+            ("identity-fenced-load", "identity-fenced-mutations"),
+            "1",
+            server_uid=server_uid,
+        )
+    )
+
+    plan = plan_launch(LaunchIntent(Invocation.PYTHON, 8123), facts)
+
+    assert plan.effective_port == 8124
+    assert plan.server_owner.value == "in_process"
+    assert plan.registration.value == "in_process_session"
+    assert "incompatible_server_new_port" in plan.reasons
+
+
 def test_julia_always_spawns_daemon():
     from arrayview._launch_plan import Environment, Invocation, LaunchIntent, plan_launch
 

{arrayview-0.33.0 → arrayview-0.33.1}/tests/test_server_identity_status.py

@@ -29,6 +29,7 @@ def test_ping_preserves_compatibility_and_reports_stable_runtime(monkeypatch):
     assert first["ok"] is True
     assert first["service"] == "arrayview"
     assert first["pid"] == os.getpid()
+    assert first["uid"] == (os.geteuid() if hasattr(os, "geteuid") else None)
     assert first["viewer_sockets"] == 2
     assert first["shell_sockets"] == 1
     assert first["instance_id"] == "test-instance"