java-functional-lsp 0.4.0__tar.gz → 0.4.1__tar.gz

{java_functional_lsp-0.4.0 → java_functional_lsp-0.4.1}/.claude-plugin/plugin.json

@@ -1,5 +1,5 @@
 {
   "name": "java-functional-lsp",
   "description": "Java LSP with functional programming rules enforcement — null safety, immutability, no exceptions, Spring best practices. Wraps jdtls for full Java language support.",
-  "version": "0.4.0"
+  "version": "0.4.1"
 }

{java_functional_lsp-0.4.0 → java_functional_lsp-0.4.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: java-functional-lsp
-Version: 0.4.0
+Version: 0.4.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
@@ -109,7 +109,18 @@ See [editors/intellij/README.md](editors/intellij/README.md) for detailed instru
 
 ### Claude Code
 
-Install as a plugin directly from GitHub:
+**Step 1: Enable LSP support** (required, one-time):
+
+Add to `~/.claude/settings.json`:
+```json
+{
+  "env": {
+    "ENABLE_LSP_TOOL": "1"
+  }
+}
+```
+
+**Step 2: Install the plugin:**
 
 ```bash
 claude plugin add https://github.com/aviadshiber/java-functional-lsp.git
@@ -130,6 +141,23 @@ Or manually add to your Claude Code config:
 }
 ```
 
+**Step 3: Nudge Claude to use diagnostics** (recommended):
+
+Add to your project's `CLAUDE.md`:
+```markdown
+After writing or editing Java code, check LSP diagnostics before moving on.
+Fix any violations immediately — do not explain, just apply the fix.
+```
+
+**Troubleshooting:**
+
+| Issue | Fix |
+|-------|-----|
+| No diagnostics appear | Ensure `ENABLE_LSP_TOOL=1` is set, restart Claude Code |
+| "java-functional-lsp not found" | Run `brew install aviadshiber/tap/java-functional-lsp` |
+| Plugin not active | Run `claude plugin list` to verify, then `/reload-plugins` |
+| Diagnostics slow on first open | Normal — tree-sitter parses on first load, then incremental |
+
 ### Other Editors
 
 Any LSP client that supports stdio transport can use this server. Point it to the `java-functional-lsp` command for `java` files.

{java_functional_lsp-0.4.0 → java_functional_lsp-0.4.1}/README.md

@@ -81,7 +81,18 @@ See [editors/intellij/README.md](editors/intellij/README.md) for detailed instru
 
 ### Claude Code
 
-Install as a plugin directly from GitHub:
+**Step 1: Enable LSP support** (required, one-time):
+
+Add to `~/.claude/settings.json`:
+```json
+{
+  "env": {
+    "ENABLE_LSP_TOOL": "1"
+  }
+}
+```
+
+**Step 2: Install the plugin:**
 
 ```bash
 claude plugin add https://github.com/aviadshiber/java-functional-lsp.git
@@ -102,6 +113,23 @@ Or manually add to your Claude Code config:
 }
 ```
 
+**Step 3: Nudge Claude to use diagnostics** (recommended):
+
+Add to your project's `CLAUDE.md`:
+```markdown
+After writing or editing Java code, check LSP diagnostics before moving on.
+Fix any violations immediately — do not explain, just apply the fix.
+```
+
+**Troubleshooting:**
+
+| Issue | Fix |
+|-------|-----|
+| No diagnostics appear | Ensure `ENABLE_LSP_TOOL=1` is set, restart Claude Code |
+| "java-functional-lsp not found" | Run `brew install aviadshiber/tap/java-functional-lsp` |
+| Plugin not active | Run `claude plugin list` to verify, then `/reload-plugins` |
+| Diagnostics slow on first open | Normal — tree-sitter parses on first load, then incremental |
+
 ### Other Editors
 
 Any LSP client that supports stdio transport can use this server. Point it to the `java-functional-lsp` command for `java` files.

{java_functional_lsp-0.4.0 → java_functional_lsp-0.4.1}/SKILL.md

@@ -74,8 +74,23 @@ To release a new version:
 5. CI automatically publishes to PyPI and builds the VS Code extension `.vsix`
 6. Run `python3 scripts/generate-formula.py <version>` and update the Homebrew tap
 
+## Enabling LSP in Claude Code
+
+LSP support requires `ENABLE_LSP_TOOL=1` in `~/.claude/settings.json`:
+```json
+{ "env": { "ENABLE_LSP_TOOL": "1" } }
+```
+
+To nudge Claude to act on diagnostics, add to your project's `CLAUDE.md`:
+```
+After writing or editing Java code, check LSP diagnostics before moving on.
+Fix any violations immediately — do not explain, just apply the fix.
+```
+
 ## Troubleshooting
 
+- **No diagnostics in Claude Code**: Ensure `ENABLE_LSP_TOOL=1` is set, restart Claude Code
 - **"java-functional-lsp not found"**: Run `brew install aviadshiber/tap/java-functional-lsp`
-- **No completions/hover**: Install jdtls: `brew install jdtls`
-- **Too many warnings**: Create `.java-functional-lsp.json` to tune severity or disable noisy rules
+- **No completions/hover**: Install jdtls: `brew install jdtls` (requires JDK 21+)
+- **Too many warnings**: Create `.java-functional-lsp.json` with `excludes` or per-rule severity
+- **Plugin not active**: Run `claude plugin list` to verify, then `/reload-plugins`

{java_functional_lsp-0.4.0 → java_functional_lsp-0.4.1}/editors/vscode/package.json

@@ -2,7 +2,7 @@
   "name": "java-functional-lsp",
   "displayName": "Java Functional LSP",
   "description": "Java LSP server enforcing functional programming best practices — null safety, immutability, no exceptions",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "publisher": "aviadshiber",
   "license": "MIT",
   "engines": {

{java_functional_lsp-0.4.0 → java_functional_lsp-0.4.1}/hooks/java_linter_reminder.py

@@ -21,11 +21,9 @@ def main() -> None:
         "hookSpecificOutput": {
             "hookEventName": "PostToolUse",
             "additionalContext": (
-                "CHECK for <new-diagnostics> from java-functional-lsp above. "
-                "If any appear, you MUST acknowledge them and suggest fixes: "
-                "null → Option/Either, throw → Either.left()/Try.of(), "
-                "mutable → final + functional transforms, loops → .map()/.filter()/.flatMap(), "
-                "@Data → @Value, @Autowired → constructor injection, @Component → @Configuration+@Bean."
+                "If <new-diagnostics> from java-functional-lsp appear above, "
+                "fix each violation immediately. Do not explain or list them — "
+                "just apply the fix in your next Edit."
             ),
         }
     }

{java_functional_lsp-0.4.0 → java_functional_lsp-0.4.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "java-functional-lsp"
-version = "0.4.0"
+version = "0.4.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.4.0 → java_functional_lsp-0.4.1}/scripts/generate-formula.py

@@ -10,31 +10,31 @@ import sys
 import urllib.request
 
 
-def get_pypi_sha256(package: str, version: str) -> str:
-    """Get the sdist sha256 for a package from PyPI."""
+def get_pypi_sdist_info(package: str, version: str) -> tuple[str, str]:
+    """Get the sdist URL and sha256 for a package from PyPI."""
     url = f"https://pypi.org/pypi/{package}/{version}/json"
     with urllib.request.urlopen(url) as resp:
         data = json.loads(resp.read())
 
     for file_info in data.get("urls", []):
         if file_info["filename"].endswith(".tar.gz"):
-            return file_info["digests"]["sha256"]
+            return file_info["url"], file_info["digests"]["sha256"]
 
     for file_info in data.get("urls", []):
         if file_info["packagetype"] == "sdist":
-            return file_info["digests"]["sha256"]
+            return file_info["url"], file_info["digests"]["sha256"]
 
     raise ValueError(f"No sdist found for {package}=={version}")
 
 
 def generate_formula(version: str) -> str:
     """Generate the Homebrew formula."""
-    sha256 = get_pypi_sha256("java-functional-lsp", version)
+    sdist_url, sha256 = get_pypi_sdist_info("java-functional-lsp", version)
 
     return f'''class JavaFunctionalLsp < Formula
   desc "Java LSP server enforcing functional programming best practices"
   homepage "https://github.com/aviadshiber/java-functional-lsp"
-  url "https://files.pythonhosted.org/packages/source/j/java-functional-lsp/java_functional_lsp-{version}.tar.gz"
+  url "{sdist_url}"
   sha256 "{sha256}"
   license "MIT"
 

{java_functional_lsp-0.4.0 → java_functional_lsp-0.4.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.4.0"
+__version__ = "0.4.1"

{java_functional_lsp-0.4.0 → java_functional_lsp-0.4.1}/src/java_functional_lsp/server.py

@@ -6,15 +6,17 @@ Proxies to jdtls for full Java language features (completions, hover, go-to-def)
 
 from __future__ import annotations
 
+import asyncio
 import json
 import logging
+import sys
 from pathlib import Path
 from typing import Any
-from urllib.parse import unquote, urlparse
 
 import cattrs
 from lsprotocol import types as lsp
 from pygls.lsp.server import LanguageServer
+from pygls.uris import to_fs_path
 
 from .analyzers.base import Analyzer, Severity, get_parser, is_excluded
 from .analyzers.base import Diagnostic as LintDiagnostic
@@ -59,11 +61,17 @@ class JavaFunctionalLspServer(LanguageServer):
 
 server = JavaFunctionalLspServer()
 
+# Debounce state for didChange events (only affects human typing in IDEs, not agents)
+_pending: dict[str, asyncio.Task[None]] = {}
+_DEBOUNCE_SECONDS = 0.15
 
-def _uri_to_path(uri: str) -> str:
-    """Convert a file:// URI to a filesystem path."""
-    parsed = urlparse(uri)
-    return unquote(parsed.path)
+
+def _handle_exception(exc_type: type[BaseException], exc_value: BaseException, exc_tb: Any) -> None:
+    """Log uncaught exceptions for crash debugging."""
+    logger.error("Uncaught exception", exc_info=(exc_type, exc_value, exc_tb))
+
+
+sys.excepthook = _handle_exception
 
 
 def _load_config(workspace_root: str | None) -> dict[str, Any]:
@@ -100,7 +108,7 @@ def _analyze_document(source_text: str, uri: str = "") -> list[lsp.Diagnostic]:
     if uri:
         excludes: list[str] = server._config.get("excludes", [])
         if excludes:
-            path_str = _uri_to_path(uri)
+            path_str = to_fs_path(uri) or uri
             if is_excluded(path_str, excludes):
                 return []
     source_bytes = source_text.encode("utf-8")
@@ -184,7 +192,7 @@ def on_initialize(params: lsp.InitializeParams) -> lsp.InitializeResult:
 
     root = None
     if params.root_uri:
-        root = _uri_to_path(params.root_uri)
+        root = to_fs_path(params.root_uri)
     elif params.root_path:
         root = params.root_path
 
@@ -223,28 +231,50 @@ async def on_initialized(params: lsp.InitializedParams) -> None:
 # --- Document sync (forward to jdtls + run custom analyzers) ---
 
 
+async def _deferred_validate(uri: str) -> None:
+    """Debounced validation — waits before analyzing to batch rapid edits."""
+    await asyncio.sleep(_DEBOUNCE_SECONDS)
+    await asyncio.to_thread(_publish_diagnostics, uri)
+
+
 @server.feature(lsp.TEXT_DOCUMENT_DID_OPEN)
 async def on_did_open(params: lsp.DidOpenTextDocumentParams) -> None:
-    """Forward to jdtls and analyze."""
+    """Forward to jdtls and analyze immediately."""
     if server._proxy.is_available:
         await server._proxy.send_notification("textDocument/didOpen", _serialize_params(params))
-    _publish_diagnostics(params.text_document.uri)
+    await asyncio.to_thread(_publish_diagnostics, params.text_document.uri)
 
 
 @server.feature(lsp.TEXT_DOCUMENT_DID_CHANGE)
 async def on_did_change(params: lsp.DidChangeTextDocumentParams) -> None:
-    """Forward to jdtls and re-analyze."""
+    """Forward to jdtls and schedule debounced re-analysis."""
+    uri = params.text_document.uri
     if server._proxy.is_available:
         await server._proxy.send_notification("textDocument/didChange", _serialize_params(params))
-    _publish_diagnostics(params.text_document.uri)
+    # Cancel pending validation, schedule new one (150ms debounce for IDE typing)
+    if uri in _pending:
+        _pending[uri].cancel()
+    _pending[uri] = asyncio.create_task(_deferred_validate(uri))
 
 
 @server.feature(lsp.TEXT_DOCUMENT_DID_SAVE)
 async def on_did_save(params: lsp.DidSaveTextDocumentParams) -> None:
-    """Forward to jdtls and re-analyze."""
+    """Forward to jdtls and re-analyze immediately (no debounce on save)."""
     if server._proxy.is_available:
         await server._proxy.send_notification("textDocument/didSave", _serialize_params(params))
-    _publish_diagnostics(params.text_document.uri)
+    await asyncio.to_thread(_publish_diagnostics, params.text_document.uri)
+
+
+@server.feature(lsp.TEXT_DOCUMENT_DID_CLOSE)
+async def on_did_close(params: lsp.DidCloseTextDocumentParams) -> None:
+    """Clean up cached state and forward to jdtls."""
+    uri = params.text_document.uri
+    server._trees.pop(uri, None)
+    if uri in _pending:
+        _pending[uri].cancel()
+        del _pending[uri]
+    if server._proxy.is_available:
+        await server._proxy.send_notification("textDocument/didClose", _serialize_params(params))
 
 
 # --- Forwarded features (jdtls passthrough) ---