compiler warnings

Author: simbo1905

json-java21-jsonpath/src/main/java/json/java21/jsonpath/JsonPathAst.java

@@ -4,18 +4,18 @@
 import java.util.Objects;
 
 /// AST representation for JsonPath expressions.
-/// Based on the JSONPath specification from https://goessner.net/articles/JsonPath/
+/// Based on the JSONPath specification from [...](https://goessner.net/articles/JsonPath/)
 ///
 /// A JsonPath expression is a sequence of path segments starting from root ($).
 /// Each segment can be:
-/// - PropertyAccess: access a named property (e.g., .store or ['store'])
-/// - ArrayIndex: access array element by index (e.g., [0] or [-1])
-/// - ArraySlice: slice array with start:end:step (e.g., [0:2] or [::2])
-/// - Wildcard: match all children (e.g., .* or [*])
+/// - PropertyAccess: access a named property (e.g., .store or \['store'\])
+/// - ArrayIndex: access array element by index (e.g., \[0\] or \[-1\])
+/// - ArraySlice: slice array with start:end:step (e.g., \[0:2\] or \[::2\])
+/// - Wildcard: match all children (e.g., .* or \[*\])
 /// - RecursiveDescent: search all descendants (e.g., ..author)
-/// - Filter: filter by predicate (e.g., [?(@.isbn)] or [?(@.price<10)])
-/// - Union: multiple indices or names (e.g., [0,1] or ['a','b'])
-/// - ScriptExpression: computed index (e.g., [(@.length-1)])
+/// - Filter: filter by predicate (e.g., \[?(@.isbn)\] or \[?(@.price<10)\])
+/// - Union: multiple indices or names (e.g., \[0,1\] or \['a','b'\])
+/// - ScriptExpression: computed index (e.g., \[(@.length-1)\])
 sealed interface JsonPathAst {
 
     /// Root element ($) - the starting point of all JsonPath expressions
@@ -80,7 +80,7 @@ record Union(List<Segment> selectors) implements Segment {
         }
     }
 
-    /// Script expression for computed index: [(@.length-1)]
+    /// Script expression for computed index: \[(@.length-1)\]
     record ScriptExpression(String script) implements Segment {
         public ScriptExpression {
             Objects.requireNonNull(script, "script must not be null");
@@ -116,7 +116,7 @@ record ComparisonFilter(
         }
     }
 
-    /// Logical combination of filters: &&, ||, !
+    /// Logical combination of filters: "&&", "||", "!"
     record LogicalFilter(
             FilterExpression left,
             LogicalOp op,

json-java21-jsonpath/src/main/java/json/java21/jsonpath/JsonPathParseException.java

@@ -1,9 +1,12 @@
 package json.java21.jsonpath;
 
+import java.io.Serial;
+
 /// Exception thrown when a JsonPath expression cannot be parsed.
 /// This is a runtime exception as JsonPath parsing failures are typically programming errors.
 public class JsonPathParseException extends RuntimeException {
 
+    @Serial
     private static final long serialVersionUID = 1L;
 
     private final int position;

json-java21-jsonpath/src/main/java/json/java21/jsonpath/JsonPathParser.java

@@ -1,7 +1,6 @@
 package json.java21.jsonpath;
 
 import java.util.ArrayList;
-import java.util.List;
 import java.util.Objects;
 import java.util.logging.Logger;
 

json-java21-jsonpath/src/test/java/json/java21/jsonpath/JsonPathAstTest.java

@@ -95,7 +95,7 @@ void testFilter() {
         LOG.info(() -> "TEST: testFilter");
         final var filter = new JsonPathAst.Filter(
             new JsonPathAst.ExistsFilter(
-                new JsonPathAst.PropertyPath(List.of("isbn"))
+                new JsonPathAst.PropertyPath(List.of("ISBN"))
             )
         );
         assertThat(filter.expression()).isInstanceOf(JsonPathAst.ExistsFilter.class);

json-java21-jsonpath/src/test/java/json/java21/jsonpath/JsonPathFilterEvaluationTest.java

@@ -157,7 +157,7 @@ void testComplexNestedLogic() {
     }
 
     // Helper to extract integer field for assertions
-    private int asInt(JsonValue v, String key) {
+    private int asInt(JsonValue v, @SuppressWarnings("SameParameterValue") String key) {
         if (v instanceof jdk.sandbox.java.util.json.JsonObject obj) {
             return (int) obj.members().get(key).toLong();
         }