Skip to content

Commit d8bc14a

Browse files
committed
Update Path Validation
1 parent 32dde34 commit d8bc14a

3 files changed

Lines changed: 9 additions & 6 deletions

File tree

src/main/java/liquidjava/mcp/tools/context/ContextRequest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
public record ContextRequest(String path, String file, Integer line, Integer column) {
99
public ContextRequest {
10-
Path verificationPath = PathUtils.requireExisting(path);
10+
Path verificationPath = PathUtils.requireExistingSource(path);
1111
path = verificationPath.toString();
1212

1313
if (file != null) {

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

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,11 @@
11
package liquidjava.mcp.tools.verification;
22

3-
import java.nio.file.Files;
4-
import java.nio.file.Path;
53
import java.util.Map;
64
import liquidjava.mcp.utils.PathUtils;
75

86
public record VerifyRequest(String path, boolean debug) {
97
public VerifyRequest {
10-
Path source = PathUtils.requireExisting(path);
11-
if (!Files.isDirectory(source) && !PathUtils.isJavaSourceFile(source))
12-
throw new IllegalArgumentException("path must be a Java source file or directory");
8+
PathUtils.requireExistingSource(path);
139
}
1410

1511
public static VerifyRequest fromArguments(Map<String, Object> arguments) {

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

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,13 @@ public static Path requireExistingJavaFile(String value, String name) {
3838
return source;
3939
}
4040

41+
public static Path requireExistingSource(String value) {
42+
Path source = requireExisting(value);
43+
if (!Files.isDirectory(source) && !isJavaSourceFile(source))
44+
throw new IllegalArgumentException("path must be a Java source file or directory");
45+
return source;
46+
}
47+
4148
public static boolean isJavaSourceFile(Path path) {
4249
return path != null && Files.isRegularFile(path) && path.getFileName().toString().endsWith(".java");
4350
}

0 commit comments

Comments
 (0)