Code Refactoring

Author: rcosta358

src/main/java/liquidjava/mcp/runtime/AnalysisKey.java

@@ -30,9 +30,7 @@ public static AnalysisKey read(String input, boolean debug) throws IOException {
     private static List<Path> sourceFiles(Path path) throws IOException {
         if (!Files.isDirectory(path)) return List.of(path);
         try (Stream<Path> paths = Files.walk(path, FileVisitOption.FOLLOW_LINKS)) {
-            return paths.filter(Files::isRegularFile)
-                .filter(file -> file.toString().endsWith(".java"))
-                .sorted().toList();
+            return paths.filter(PathUtils::isJavaSourceFile).sorted().toList();
         } catch (UncheckedIOException e) {
             throw e.getCause();
         }

src/main/java/liquidjava/mcp/runtime/LiquidJavaRunner.java

@@ -27,7 +27,7 @@ public final class LiquidJavaRunner {
         1, Thread.ofPlatform().daemon().name("liquidjava-runner").factory());
     static {
         EXECUTOR.setRemoveOnCancelPolicy(true);
-    };
+    }
     private static CachedAnalysis cachedAnalysis;
     private record CachedAnalysis(AnalysisKey key, String output) {}
 

src/main/java/liquidjava/mcp/tools/verification/VerifyRequest.java

@@ -8,7 +8,7 @@
 public record VerifyRequest(String path, boolean debug) {
     public VerifyRequest {
         Path source = PathUtils.requireExisting(path);
-        if (!Files.isDirectory(source) && (!Files.isRegularFile(source) || !source.getFileName().toString().endsWith(".java")))
+        if (!Files.isDirectory(source) && !PathUtils.isJavaSourceFile(source))
             throw new IllegalArgumentException("path must be a Java source file or directory");
     }
 

src/main/java/liquidjava/mcp/utils/PathUtils.java

@@ -33,11 +33,15 @@ public static Path requireExistingJavaFile(String value, String name) {
         } catch (IOException e) {
             throw new IllegalArgumentException(name + " must be an existing Java source file", e);
         }
-        if (!Files.isRegularFile(source) || !source.getFileName().toString().endsWith(".java"))
+        if (!isJavaSourceFile(source))
             throw new IllegalArgumentException(name + " must be an existing Java source file");
         return source;
     }
 
+    public static boolean isJavaSourceFile(Path path) {
+        return path != null && Files.isRegularFile(path) && path.getFileName().toString().endsWith(".java");
+    }
+
     public static Path canonicalPath(String path) throws IOException {
         return Path.of(path).toRealPath();
     }