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

{java_functional_lsp-0.3.2 → 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.3.2"
+  "version": "0.4.1"
 }

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: java-functional-lsp
-Version: 0.3.2
+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.
@@ -146,6 +174,7 @@ Create `.java-functional-lsp.json` in your project root to customize rules:
 
 ```json
 {
+  "excludes": ["**/generated/**", "**/vendor/**"],
   "rules": {
     "null-literal-arg": "warning",
     "throw-statement": "info",
@@ -155,8 +184,13 @@ Create `.java-functional-lsp.json` in your project root to customize rules:
 }
 ```
 
-Severity levels: `error`, `warning`, `info`, `hint`, `off`.
-All rules default to `warning` when not configured.
+**Options:**
+- `excludes` — glob patterns for files/directories to skip entirely (supports `**` for multi-segment wildcards)
+- `rules` — per-rule severity: `error`, `warning` (default), `info`, `hint`, `off`
+
+**Spring-aware behavior:**
+- `throw-statement` and `catch-rethrow` are automatically suppressed inside `@Bean` methods
+- `mutable-dto` suggests `@ConstructorBinding` instead of `@Value` when the class has `@ConfigurationProperties`
 
 ## How it works
 

{java_functional_lsp-0.3.2 → 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.
@@ -118,6 +146,7 @@ Create `.java-functional-lsp.json` in your project root to customize rules:
 
 ```json
 {
+  "excludes": ["**/generated/**", "**/vendor/**"],
   "rules": {
     "null-literal-arg": "warning",
     "throw-statement": "info",
@@ -127,8 +156,13 @@ Create `.java-functional-lsp.json` in your project root to customize rules:
 }
 ```
 
-Severity levels: `error`, `warning`, `info`, `hint`, `off`.
-All rules default to `warning` when not configured.
+**Options:**
+- `excludes` — glob patterns for files/directories to skip entirely (supports `**` for multi-segment wildcards)
+- `rules` — per-rule severity: `error`, `warning` (default), `info`, `hint`, `off`
+
+**Spring-aware behavior:**
+- `throw-statement` and `catch-rethrow` are automatically suppressed inside `@Bean` methods
+- `mutable-dto` suggests `@ConstructorBinding` instead of `@Value` when the class has `@ConfigurationProperties`
 
 ## How it works
 

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

@@ -45,6 +45,7 @@ Create `.java-functional-lsp.json` in your project root:
 
 ```json
 {
+  "excludes": ["**/generated/**", "**/vendor/**"],
   "rules": {
     "imperative-loop": "hint",
     "mutable-variable": "info",
@@ -53,7 +54,10 @@ Create `.java-functional-lsp.json` in your project root:
 }
 ```
 
-Severity levels: `error`, `warning` (default), `info`, `hint`, `off`.
+- `excludes` — glob patterns to skip files/directories entirely
+- `rules` — per-rule severity: `error`, `warning` (default), `info`, `hint`, `off`
+- `throw-statement`/`catch-rethrow` auto-suppressed in `@Bean` methods
+- `mutable-dto` suggests `@ConstructorBinding` for `@ConfigurationProperties` classes
 
 ## On-Demand Linting
 
@@ -70,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.3.2 → 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.3.2",
+  "version": "0.4.1",
   "publisher": "aviadshiber",
   "license": "MIT",
   "engines": {

{java_functional_lsp-0.3.2 → 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.3.2 → java_functional_lsp-0.4.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "java-functional-lsp"
-version = "0.3.2"
+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.3.2 → 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.3.2 → 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.3.2"
+__version__ = "0.4.1"

{java_functional_lsp-0.3.2 → java_functional_lsp-0.4.1}/src/java_functional_lsp/analyzers/base.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import fnmatch
 from collections.abc import Generator
 from dataclasses import dataclass
 from enum import IntEnum
@@ -147,3 +148,26 @@ def severity_from_config(config: dict[str, Any], rule_id: str, default: Severity
         "info": Severity.INFO,
         "hint": Severity.HINT,
     }.get(level, default)
+
+
+def is_excluded(path_str: str, patterns: list[str]) -> bool:
+    """Return True if path matches any exclude glob pattern.
+
+    Patterns support ** for multi-segment wildcards (e.g. **/generated/**).
+    Uses fnmatch which handles ** correctly across Python 3.10+.
+    """
+    normalized = path_str.replace("\\", "/")
+    return any(fnmatch.fnmatch(normalized, pattern) for pattern in patterns)
+
+
+def has_sibling_annotation(modifiers_node: Node, annotation_name: bytes) -> bool:
+    """Check if a modifiers node contains an annotation with the given name.
+
+    Checks both marker_annotation (@Foo) and annotation (@Foo(...)) forms.
+    """
+    for child in modifiers_node.named_children:
+        if child.type in ("marker_annotation", "annotation"):
+            name_node = child.child_by_field_name("name")
+            if name_node is not None and name_node.text == annotation_name:
+                return True
+    return False

{java_functional_lsp-0.3.2 → java_functional_lsp-0.4.1}/src/java_functional_lsp/analyzers/exception_checker.py

@@ -4,7 +4,7 @@ from __future__ import annotations
 
 from typing import Any
 
-from .base import Diagnostic, find_nodes, severity_from_config
+from .base import Diagnostic, find_nodes, has_sibling_annotation, severity_from_config
 
 _MESSAGES = {
     "throw-statement": ("Avoid throwing exceptions. Use Either.left(error) or Try.of(() -> ...).toEither()."),
@@ -14,6 +14,19 @@ _MESSAGES = {
 }
 
 
+def _is_in_bean_method(node: Any) -> bool:
+    """Check if node is inside a method annotated with @Bean."""
+    parent = node.parent
+    while parent:
+        if parent.type == "method_declaration":
+            modifiers = next((c for c in parent.children if c.type == "modifiers"), None)
+            if modifiers and has_sibling_annotation(modifiers, b"Bean"):
+                return True
+            return False
+        parent = parent.parent
+    return False
+
+
 class ExceptionChecker:
     """Detects throw statements and catch-rethrow anti-patterns."""
 
@@ -24,6 +37,8 @@ class ExceptionChecker:
         severity = severity_from_config(config, "throw-statement")
         if severity is not None:
             for node in find_nodes(tree.root_node, "throw_statement"):
+                if _is_in_bean_method(node):
+                    continue
                 diagnostics.append(
                     Diagnostic(
                         line=node.start_point[0],
@@ -40,10 +55,11 @@ class ExceptionChecker:
         severity = severity_from_config(config, "catch-rethrow")
         if severity is not None:
             for node in find_nodes(tree.root_node, "catch_clause"):
+                if _is_in_bean_method(node):
+                    continue
                 body = node.child_by_field_name("body")
                 if body is None:
                     continue
-                # Check if the block has exactly one named statement and it's a throw
                 statements = [c for c in body.named_children if c.type not in ("line_comment", "block_comment")]
                 if len(statements) == 1 and statements[0].type == "throw_statement":
                     diagnostics.append(

{java_functional_lsp-0.3.2 → java_functional_lsp-0.4.1}/src/java_functional_lsp/analyzers/mutation_checker.py

@@ -4,7 +4,7 @@ from __future__ import annotations
 
 from typing import Any
 
-from .base import Diagnostic, find_nodes, find_nodes_multi, has_ancestor, severity_from_config
+from .base import Diagnostic, find_nodes, find_nodes_multi, has_ancestor, has_sibling_annotation, severity_from_config
 
 _MESSAGES = {
     "mutable-variable": "Avoid reassigning variables. Use final + functional transforms (map, flatMap, fold).",
@@ -45,8 +45,15 @@ class MutationChecker:
             if ann_text in (b"Data", b"Setter"):
                 # Verify it's on a class declaration
                 if node.parent and node.parent.type == "modifiers":
-                    grandparent = node.parent.parent
+                    modifiers = node.parent
+                    grandparent = modifiers.parent
                     if grandparent and grandparent.type == "class_declaration":
+                        if has_sibling_annotation(modifiers, b"ConfigurationProperties"):
+                            message = (
+                                "Use @ConstructorBinding instead of @Data/@Setter for @ConfigurationProperties classes."
+                            )
+                        else:
+                            message = _MESSAGES["mutable-dto"]
                         diagnostics.append(
                             Diagnostic(
                                 line=name_node.start_point[0],
@@ -55,7 +62,7 @@ class MutationChecker:
                                 end_col=name_node.end_point[1],
                                 severity=severity,
                                 code="mutable-dto",
-                                message=_MESSAGES["mutable-dto"],
+                                message=message,
                             )
                         )
 

{java_functional_lsp-0.3.2 → java_functional_lsp-0.4.1}/src/java_functional_lsp/cli.py

@@ -7,7 +7,7 @@ import sys
 from pathlib import Path
 from typing import Any
 
-from .analyzers.base import Analyzer, Diagnostic, Severity, get_parser
+from .analyzers.base import Analyzer, Diagnostic, Severity, get_parser, is_excluded
 from .analyzers.exception_checker import ExceptionChecker
 from .analyzers.mutation_checker import MutationChecker
 from .analyzers.null_checker import NullChecker
@@ -107,7 +107,10 @@ def main() -> None:
     config = load_config(files[0])
 
     total_diags = 0
+    excludes: list[str] = config.get("excludes", [])
     for path in files:
+        if excludes and is_excluded(path.as_posix(), excludes):
+            continue
         diags = check_file(path, config)
         for d in diags:
             print(format_diagnostic(path, d))

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

@@ -6,17 +6,19 @@ 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
+from .analyzers.base import Analyzer, Severity, get_parser, is_excluded
 from .analyzers.base import Diagnostic as LintDiagnostic
 from .analyzers.exception_checker import ExceptionChecker
 from .analyzers.mutation_checker import MutationChecker
@@ -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]:
@@ -96,6 +104,13 @@ def _to_lsp_diagnostic(diag: LintDiagnostic) -> lsp.Diagnostic:
 
 def _analyze_document(source_text: str, uri: str = "") -> list[lsp.Diagnostic]:
     """Run all custom analyzers on the given source text. Uses incremental parsing when possible."""
+    # Check excludes before parsing
+    if uri:
+        excludes: list[str] = server._config.get("excludes", [])
+        if excludes:
+            path_str = to_fs_path(uri) or uri
+            if is_excluded(path_str, excludes):
+                return []
     source_bytes = source_text.encode("utf-8")
     old_tree = server._trees.get(uri) if uri else None
     tree = server._parser.parse(source_bytes, old_tree) if old_tree else server._parser.parse(source_bytes)
@@ -177,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
 
@@ -216,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) ---

{java_functional_lsp-0.3.2 → java_functional_lsp-0.4.1}/tests/test_base.py

@@ -7,6 +7,8 @@ from java_functional_lsp.analyzers.base import (
     find_nodes_multi,
     get_parser,
     has_ancestor,
+    has_sibling_annotation,
+    is_excluded,
 )
 
 
@@ -120,3 +122,36 @@ class TestCollectNodesByType:
         buckets = collect_nodes_by_type(tree.root_node, {"null_literal", "throw_statement"})
         assert len(buckets["null_literal"]) == 0
         assert len(buckets["throw_statement"]) == 0
+
+
+class TestIsExcluded:
+    def test_matches_double_star_pattern(self):
+        assert is_excluded("/home/user/project/nlu-trs-client-shaded/Foo.java", ["**/nlu-trs-client-shaded/**"])
+
+    def test_no_match(self):
+        assert not is_excluded("/home/user/project/src/Foo.java", ["**/nlu-trs-client-shaded/**"])
+
+    def test_empty_patterns(self):
+        assert not is_excluded("/any/path/Foo.java", [])
+
+    def test_multiple_patterns(self):
+        assert is_excluded("/project/generated/Model.java", ["**/shaded/**", "**/generated/**"])
+
+    def test_windows_backslashes_normalized(self):
+        assert is_excluded("C:\\project\\generated\\Model.java", ["**/generated/**"])
+
+
+class TestHasSiblingAnnotation:
+    def test_finds_sibling_annotation(self):
+        tree = _parse("@ConfigurationProperties @Setter class Props { String name; }")
+        for node in find_nodes(tree.root_node, "marker_annotation"):
+            name = node.child_by_field_name("name")
+            if name and name.text == b"Setter" and node.parent:
+                assert has_sibling_annotation(node.parent, b"ConfigurationProperties")
+
+    def test_no_sibling(self):
+        tree = _parse("@Setter class Props { String name; }")
+        for node in find_nodes(tree.root_node, "marker_annotation"):
+            name = node.child_by_field_name("name")
+            if name and name.text == b"Setter" and node.parent:
+                assert not has_sibling_annotation(node.parent, b"ConfigurationProperties")

{java_functional_lsp-0.3.2 → java_functional_lsp-0.4.1}/tests/test_exception_checker.py

@@ -60,3 +60,45 @@ class TestCatchRethrow:
         """
         diags = parse_and_analyze(ExceptionChecker(), source)
         assert not any(d.code == "catch-rethrow" for d in diags)
+
+
+class TestBeanSuppression:
+    def test_ignores_throw_in_bean_method(self) -> None:
+        source = b"""
+        class Config {
+            @Bean
+            DataSource dataSource() {
+                if (url == null) {
+                    throw new IllegalStateException("url required");
+                }
+                return new DataSource(url);
+            }
+        }
+        """
+        diags = parse_and_analyze(ExceptionChecker(), source)
+        assert not any(d.code == "throw-statement" for d in diags)
+
+    def test_flags_throw_in_regular_method(self) -> None:
+        source = b"""
+        class Service {
+            void process() {
+                throw new RuntimeException("error");
+            }
+        }
+        """
+        diags = parse_and_analyze(ExceptionChecker(), source)
+        assert any(d.code == "throw-statement" for d in diags)
+
+    def test_ignores_catch_rethrow_in_bean_method(self) -> None:
+        source = b"""
+        class Config {
+            @Bean
+            DataSource dataSource() {
+                try { return connect(); }
+                catch (Exception e) { throw new RuntimeException(e); }
+            }
+        }
+        """
+        diags = parse_and_analyze(ExceptionChecker(), source)
+        assert not any(d.code == "catch-rethrow" for d in diags)
+        assert not any(d.code == "throw-statement" for d in diags)

{java_functional_lsp-0.3.2 → java_functional_lsp-0.4.1}/tests/test_mutation_checker.py

@@ -58,6 +58,20 @@ class TestMutableDto:
         diags = parse_and_analyze(MutationChecker(), source)
         assert not any(d.code == "mutable-dto" for d in diags)
 
+    def test_config_properties_suggests_constructor_binding(self) -> None:
+        source = b"@ConfigurationProperties @Setter class Props { String name; }"
+        diags = parse_and_analyze(MutationChecker(), source)
+        dto_diags = [d for d in diags if d.code == "mutable-dto"]
+        assert len(dto_diags) == 1
+        assert "@ConstructorBinding" in dto_diags[0].message
+
+    def test_regular_setter_suggests_value(self) -> None:
+        source = b"@Setter class Foo { String name; }"
+        diags = parse_and_analyze(MutationChecker(), source)
+        dto_diags = [d for d in diags if d.code == "mutable-dto"]
+        assert len(dto_diags) == 1
+        assert "@Value" in dto_diags[0].message
+
 
 class TestImperativeOptionUnwrap:
     def test_detects_is_defined_get(self) -> None: