python-delphi-lsp 2.3.2__tar.gz → 2.3.3__tar.gz

{python_delphi_lsp-2.3.2/python_delphi_lsp.egg-info → python_delphi_lsp-2.3.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-delphi-lsp
-Version: 2.3.2
+Version: 2.3.3
 Summary: Python Delphi/Object Pascal parser, semantic indexer, and language server.
 Author: Dark Light
 License-Expression: MPL-2.0
@@ -42,7 +42,7 @@ Dynamic: license-file
 
 `python-delphi-lsp` parses Delphi/Object Pascal, builds semantic and project
 indexes, serves LSP, and provides bounded codebase navigation for agents.
-Version 2.3.2 is authored by Dark Light and supports Windows, macOS, and Linux.
+Version 2.3.3 is authored by Dark Light and supports Windows, macOS, and Linux.
 
 ## Install and quick start
 

{python_delphi_lsp-2.3.2 → python_delphi_lsp-2.3.3}/README.md

@@ -2,7 +2,7 @@
 
 `python-delphi-lsp` parses Delphi/Object Pascal, builds semantic and project
 indexes, serves LSP, and provides bounded codebase navigation for agents.
-Version 2.3.2 is authored by Dark Light and supports Windows, macOS, and Linux.
+Version 2.3.3 is authored by Dark Light and supports Windows, macOS, and Linux.
 
 ## Install and quick start
 

python_delphi_lsp-2.3.3/delphi_lsp/_version.py

@@ -0,0 +1 @@
+__version__ = "2.3.3"

{python_delphi_lsp-2.3.2 → python_delphi_lsp-2.3.3}/delphi_lsp/agent_cache.py

@@ -679,6 +679,7 @@ class _CacheService:
         self.last_budget = BudgetResult(0, 0.0, 0.0, False, False, False)
         self.prewarm_seconds = 0.0
         self.shutdown = threading.Event()
+        self.prewarm_thread: threading.Thread | None = None
 
     def _measure_retained_bytes(self) -> int:
         process_rss = current_process_rss_bytes()
@@ -709,15 +710,38 @@ class _CacheService:
         self.prewarm_seconds = time.monotonic() - started
         self.stats.parallel_fallbacks = self.context.parallel_stats.fallbacks
 
+    def start_prewarm(self) -> None:
+        def run() -> None:
+            try:
+                with self.lock:
+                    self.prewarm()
+            except Exception:
+                self.cache_state = "failed"
+
+        self.prewarm_thread = threading.Thread(
+            target=run,
+            name="delphi-cache-prewarm",
+            daemon=True,
+        )
+        self.prewarm_thread.start()
+
     def request(self, request: dict[str, object]) -> CacheClientResponse:
+        action = request.get("action")
+        if action == "status":
+            warning = "" if request.get("_startup_probe") is True else self._consume_warning()
+            return CacheClientResponse(self.status(), warning)
+        if action == "stop":
+            self.shutdown.set()
+            return CacheClientResponse({"stopping": True})
+        if self.cache_state == "warming":
+            self.last_activity = time.monotonic()
+            raise CacheClientError(
+                "cache_warming",
+                "Cache is still warming. Retry after cache status reports ready.",
+            )
+        if self.cache_state == "failed":
+            raise CacheClientError("cache_failed", "Cache warm-up failed.")
         with self.lock:
-            action = request.get("action")
-            if action == "status":
-                warning = "" if request.get("_startup_probe") is True else self._consume_warning()
-                return CacheClientResponse(self.status(), warning)
-            if action == "stop":
-                self.shutdown.set()
-                return CacheClientResponse({"stopping": True})
             self.last_activity = time.monotonic()
             before = self.last_revision
             was_warm = self.context.navigation_cache_is_warm
@@ -865,7 +889,7 @@ def run_cache_daemon(
     watcher: threading.Thread | None = None
     try:
         service = _CacheService(metadata)
-        service.prewarm()
+        _write_metadata(metadata)
         watcher = threading.Thread(
             target=_watch_workspace,
             args=(service,),
@@ -873,8 +897,14 @@ def run_cache_daemon(
             daemon=True,
         )
         watcher.start()
-        _write_metadata(metadata)
-        while not service.shutdown.is_set() and time.monotonic() - service.last_activity < idle_timeout:
+        service.start_prewarm()
+        while (
+            not service.shutdown.is_set()
+            and (
+                service.cache_state == "warming"
+                or time.monotonic() - service.last_activity < idle_timeout
+            )
+        ):
             try:
                 connection, _ = listener.accept()
             except socket.timeout:
@@ -1011,7 +1041,13 @@ def query_cache(root: str | Path, request: dict[str, object]) -> CacheClientResp
     if not _pid_alive(metadata.pid):
         _remove_metadata_if_owned(metadata)
         raise CacheClientError("cache_not_running", "Cache daemon is not running.")
-    return _client_exchange(metadata, request)
+    while True:
+        try:
+            return _client_exchange(metadata, request)
+        except CacheClientError as error:
+            if error.code != "cache_warming":
+                raise
+            time.sleep(0.05)
 
 
 def cache_status(root: str | Path) -> dict[str, object]:

{python_delphi_lsp-2.3.2 → python_delphi_lsp-2.3.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "python-delphi-lsp"
-version = "2.3.2"
+version = "2.3.3"
 description = "Python Delphi/Object Pascal parser, semantic indexer, and language server."
 readme = "README.md"
 requires-python = ">=3.10"

{python_delphi_lsp-2.3.2 → python_delphi_lsp-2.3.3/python_delphi_lsp.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-delphi-lsp
-Version: 2.3.2
+Version: 2.3.3
 Summary: Python Delphi/Object Pascal parser, semantic indexer, and language server.
 Author: Dark Light
 License-Expression: MPL-2.0
@@ -42,7 +42,7 @@ Dynamic: license-file
 
 `python-delphi-lsp` parses Delphi/Object Pascal, builds semantic and project
 indexes, serves LSP, and provides bounded codebase navigation for agents.
-Version 2.3.2 is authored by Dark Light and supports Windows, macOS, and Linux.
+Version 2.3.3 is authored by Dark Light and supports Windows, macOS, and Linux.
 
 ## Install and quick start
 

{python_delphi_lsp-2.3.2 → python_delphi_lsp-2.3.3}/tests/test_agent_cache.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 from collections.abc import Iterator, Mapping
+import contextlib
 from dataclasses import dataclass
 import json
 import os
@@ -149,6 +150,103 @@ def test_cache_daemon_lifecycle_reuses_one_authenticated_process(tmp_path: Path)
         stop_cache(tmp_path)
 
 
+def test_cache_daemon_reports_ready_before_slow_prewarm_finishes(
+    tmp_path: Path,
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    from delphi_lsp import agent_cache
+
+    write_source(tmp_path / "Demo.dpr", "program Demo; begin end.")
+    prewarm_started = threading.Event()
+    allow_prewarm = threading.Event()
+    real_prewarm = agent_cache._CacheService.prewarm
+
+    def slow_prewarm(service: agent_cache._CacheService) -> None:
+        prewarm_started.set()
+        assert allow_prewarm.wait(timeout=5)
+        real_prewarm(service)
+
+    monkeypatch.setattr(agent_cache._CacheService, "prewarm", slow_prewarm)
+    daemon = threading.Thread(
+        target=agent_cache.run_cache_daemon,
+        args=(tmp_path,),
+        kwargs={"idle_timeout": 10},
+        daemon=True,
+    )
+    daemon.start()
+
+    try:
+        assert prewarm_started.wait(timeout=2)
+        deadline = time.monotonic() + 2
+        metadata = agent_cache._read_metadata(tmp_path)
+        while metadata is None and time.monotonic() < deadline:
+            time.sleep(0.01)
+            metadata = agent_cache._read_metadata(tmp_path)
+
+        assert metadata is not None
+        status = agent_cache._client_exchange(
+            metadata,
+            {"action": "status", "_startup_probe": True},
+        ).payload
+        assert status["cache_state"] == "warming"
+    finally:
+        allow_prewarm.set()
+        deadline = time.monotonic() + 2
+        metadata = agent_cache._read_metadata(tmp_path)
+        while metadata is None and time.monotonic() < deadline:
+            time.sleep(0.01)
+            metadata = agent_cache._read_metadata(tmp_path)
+        if metadata is not None:
+            with contextlib.suppress(agent_cache.CacheClientError):
+                agent_cache._client_exchange(metadata, {"action": "stop"})
+        daemon.join(timeout=3)
+
+
+def test_query_cache_retries_while_daemon_is_warming(
+    tmp_path: Path,
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    from delphi_lsp import agent_cache
+
+    metadata = agent_cache.CacheMetadata(
+        2,
+        str(tmp_path.resolve()),
+        os.getpid(),
+        1,
+        "x" * 32,
+        "test",
+        "",
+        512 * 1024**2,
+        0,
+        10,
+        time.time(),
+    )
+    attempts = 0
+
+    def exchange(
+        _metadata: agent_cache.CacheMetadata,
+        _request: dict[str, object],
+    ) -> agent_cache.CacheClientResponse:
+        nonlocal attempts
+        attempts += 1
+        if attempts == 1:
+            raise agent_cache.CacheClientError(
+                "cache_warming",
+                "Cache is still warming.",
+            )
+        return agent_cache.CacheClientResponse({"schema": 2})
+
+    monkeypatch.setattr(agent_cache, "_read_metadata", lambda _root: metadata)
+    monkeypatch.setattr(agent_cache, "_pid_alive", lambda _pid: True)
+    monkeypatch.setattr(agent_cache, "_client_exchange", exchange)
+    monkeypatch.setattr(agent_cache.time, "sleep", lambda _seconds: None)
+
+    response = agent_cache.query_cache(tmp_path, {"action": "open"})
+
+    assert response.payload == {"schema": 2}
+    assert attempts == 2
+
+
 def test_cache_daemon_compacts_handles_bad_clients_and_idles(tmp_path: Path) -> None:
     from delphi_lsp.agent_cache import cache_metadata_path, cache_status, query_cache, start_cache, stop_cache
 

{python_delphi_lsp-2.3.2 → python_delphi_lsp-2.3.3}/tests/test_package_metadata.py

@@ -28,12 +28,12 @@ def test_lsp_console_script_dependencies_are_installed_by_default() -> None:
     assert '"lsprotocol>=2023.0.1"' in project
 
 
-def test_release_metadata_declares_2_3_2_sole_namespace_author_and_windows_support() -> None:
+def test_release_metadata_declares_2_3_3_sole_namespace_author_and_windows_support() -> None:
     pyproject = (ROOT / 'pyproject.toml').read_text(encoding='utf-8')
     project = _section('project', pyproject)
     scripts = _section('project.scripts', pyproject)
 
-    assert 'version = "2.3.2"' in project
+    assert 'version = "2.3.3"' in project
     assert '"Operating System :: OS Independent"' in project
     assert '"Operating System :: Microsoft :: Windows"' in project
     assert '"Operating System :: MacOS"' in project
@@ -83,7 +83,7 @@ def test_readme_documents_v2_release_plugin_protocol_discovery_and_vllm_proof()
     assert 'Auto-discovery reads `.dpr`, `.dpk`, `.dproj`, `.cfg`, and `.dof` files' in readme
     assert 'no file-size threshold' in readme
     assert 'final answer' in readme
-    assert 'Version 2.3.2' in readme
+    assert 'Version 2.3.3' in readme
     assert 'view --layer metrics' in readme
     assert '`metrics`' in readme
     assert 'total_loc_with_includes' in readme

python_delphi_lsp-2.3.2/delphi_lsp/_version.py

@@ -1 +0,0 @@
-__version__ = "2.3.2"