java-functional-lsp 0.9.0__tar.gz → 0.9.1__tar.gz

{java_functional_lsp-0.9.0 → java_functional_lsp-0.9.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: java-functional-lsp
-Version: 0.9.0
+Version: 0.9.1
 Summary: Java LSP server enforcing functional programming best practices — null safety, immutability, no exceptions
 Project-URL: Homepage, https://github.com/aviadshiber/java-functional-lsp
 Project-URL: Repository, https://github.com/aviadshiber/java-functional-lsp

{java_functional_lsp-0.9.0 → java_functional_lsp-0.9.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "java-functional-lsp"
-version = "0.9.0"
+version = "0.9.1"
 description = "Java LSP server enforcing functional programming best practices — null safety, immutability, no exceptions"
 readme = "README.md"
 license = { text = "MIT" }

{java_functional_lsp-0.9.0 → java_functional_lsp-0.9.1}/src/java_functional_lsp/__init__.py

@@ -1,3 +1,3 @@
 """java-functional-lsp: A Java LSP server enforcing functional programming best practices."""
 
-__version__ = "0.9.0"
+__version__ = "0.9.1"

{java_functional_lsp-0.9.0 → java_functional_lsp-0.9.1}/src/java_functional_lsp/proxy.py

@@ -777,6 +777,11 @@ class JdtlsProxy:
         Uses a ``deque(maxlen=200)`` so oldest entries are dropped in O(1)
         when the queue overflows during long jdtls startup.
         """
+        if len(self._queued_notifications) >= _MAX_QUEUED_NOTIFICATIONS:
+            logger.warning(
+                "jdtls: notification queue full (%d cap) — oldest entry dropped; some didOpen events may be missed",
+                _MAX_QUEUED_NOTIFICATIONS,
+            )
         self._queued_notifications.append((method, params))
 
     async def flush_queued_notifications(self) -> None:
@@ -832,7 +837,7 @@ class JdtlsProxy:
         Calls ``modules.mark_added()`` before any ``await`` to prevent
         duplicate add calls from concurrent coroutines.
         """
-        if not self._available:
+        if not self._available or self._workspace_expanded:
             return None
         module_uri = _resolve_module_uri(file_uri)
         if module_uri is None or self.modules.was_added(module_uri):
@@ -873,11 +878,14 @@ class JdtlsProxy:
             return
         self.modules.mark_added(root_uri)
 
-        # Remove initial module folder to avoid double-indexing.
+        # Remove ALL individually-added modules to prevent double-indexing with
+        # the full root.  This covers both the initial scoped module and any
+        # modules added by add_module_if_new during the ensure_started window.
         removed: list[dict[str, str]] = []
-        if self._initial_module_uri and self._initial_module_uri != root_uri:
-            ini_path = to_fs_path(self._initial_module_uri) or self._initial_module_uri
-            removed.append({"uri": self._initial_module_uri, "name": Path(ini_path).name})
+        for uri in list(self.modules._states):
+            if uri != root_uri:
+                p = to_fs_path(uri) or uri
+                removed.append({"uri": uri, "name": Path(p).name})
 
         logger.info("jdtls: expanding to full workspace %s", _redact_path(root_path))
         await self.send_notification(

{java_functional_lsp-0.9.0 → java_functional_lsp-0.9.1}/src/java_functional_lsp/server.py

@@ -98,6 +98,8 @@ class JavaFunctionalLspServer(LanguageServer):
                 _fire_and_forget(_apply_module_diff(self._proxy, module_uri))
         try:
             _analyze_and_publish(uri)
+        except FileNotFoundError:
+            pass  # Virtual jdtls file (e.g., decompiled source in jdtls-data); skip silently.
         except Exception as e:
             logger.error("Error re-publishing diagnostics for %s: %s", uri, e)
 
@@ -602,14 +604,21 @@ async def _lazy_start_jdtls(file_uri: str) -> None:
     """Background task: start jdtls scoped to the module containing *file_uri*.
 
     Runs in the background so ``on_did_open`` returns immediately with custom
-    diagnostics. After jdtls initializes, registers capabilities and flushes
-    queued notifications. Workspace expansion is NOT done eagerly — modules
-    are loaded on-demand via ``add_module_if_new()`` as files are opened.
+    diagnostics.  After jdtls initializes, the workspace is immediately expanded
+    to the full IDE workspace root (inside-out) so jdtls discovers all sibling
+    modules before any queued ``didOpen`` notifications are flushed.  This means
+    ``add_module_if_new`` is a no-op for all subsequent file opens — jdtls
+    already covers the whole project.
     """
     try:
         started = await server._proxy.ensure_started(server._init_params, file_uri, config=server._config)
         if started:
             logger.info("jdtls proxy active — full Java language support enabled")
+            # Expand to full workspace BEFORE flushing queued notifications so
+            # all files are processed in the full-workspace context (inside-out).
+            # expand_full_workspace is a no-op for standalone projects where
+            # the workspace root equals the initial module.
+            await server._proxy.expand_full_workspace()
             if server._skip_jdtls_registration:
                 logger.info("Skipping dynamic capability registration (no-register mode)")
             else:

{java_functional_lsp-0.9.0 → java_functional_lsp-0.9.1}/tests/test_proxy.py

@@ -1051,7 +1051,7 @@ class TestLazyStart:
         assert proxy._workspace_expanded is True
 
     async def test_expand_full_workspace_removes_initial_module(self) -> None:
-        """When _initial_module_uri differs from root, it should be in the removed list."""
+        """All modules in _states (not just _initial_module_uri) are removed on expansion."""
         from unittest.mock import AsyncMock
 
         from java_functional_lsp.proxy import JdtlsProxy
@@ -1060,13 +1060,17 @@ class TestLazyStart:
         proxy._available = True
         proxy._original_root_uri = "file:///workspace/monorepo"
         proxy._initial_module_uri = "file:///workspace/monorepo/module-a"
+        # Simulate that module-a was mark_added during start() and module-b via add_module_if_new
+        proxy.modules.mark_added("file:///workspace/monorepo/module-a")
+        proxy.modules.mark_added("file:///workspace/monorepo/module-b")
         proxy.send_notification = AsyncMock()  # type: ignore[assignment]
         await proxy.expand_full_workspace()
         proxy.send_notification.assert_called_once()  # type: ignore[attr-defined]
         call_args = proxy.send_notification.call_args[0]  # type: ignore[attr-defined]
         event = call_args[1]["event"]
-        assert len(event["removed"]) == 1
-        assert event["removed"][0]["uri"] == "file:///workspace/monorepo/module-a"
+        removed_uris = {e["uri"] for e in event["removed"]}
+        assert "file:///workspace/monorepo/module-a" in removed_uris
+        assert "file:///workspace/monorepo/module-b" in removed_uris
         assert event["added"][0]["uri"] == "file:///workspace/monorepo"
 
     async def test_expand_full_workspace_noop_when_not_available(self) -> None: