Issue #130 Fix json-transforms compilation visibility and warnings

Co-authored-by: simbo1905 <simbo1905@60hertz.com>

Author: cursoragent

json-transforms/src/main/java/json/java21/transforms/JsonTransformException.java

@@ -2,6 +2,8 @@
 
 /// Exception thrown for invalid transform syntax or runtime failures applying a transform.
 public final class JsonTransformException extends RuntimeException {
+    private static final long serialVersionUID = 1L;
+
     public JsonTransformException(String message) {
         super(message);
     }

json-transforms/src/main/java/json/java21/transforms/TransformAst.java

@@ -18,7 +18,7 @@ record ObjectTransform(
             List<MergeOp> merges,
             List<RenameOp> renames
     ) implements TransformAst {
-        ObjectTransform {
+        public ObjectTransform {
             Objects.requireNonNull(nonVerbMembers, "nonVerbMembers must not be null");
             Objects.requireNonNull(childObjects, "childObjects must not be null");
             Objects.requireNonNull(removes, "removes must not be null");
@@ -36,15 +36,15 @@ record ObjectTransform(
 
     sealed interface RemoveOp extends TransformAst permits RemoveOp.ByName, RemoveOp.RemoveThis, RemoveOp.ByPath {
         record ByName(String name) implements RemoveOp {
-            ByName {
+            public ByName {
                 Objects.requireNonNull(name, "name must not be null");
             }
         }
 
         record RemoveThis() implements RemoveOp {}
 
         record ByPath(String rawPath, JsonPath path) implements RemoveOp {
-            ByPath {
+            public ByPath {
                 Objects.requireNonNull(rawPath, "rawPath must not be null");
                 Objects.requireNonNull(path, "path must not be null");
             }
@@ -53,13 +53,13 @@ record ByPath(String rawPath, JsonPath path) implements RemoveOp {
 
     sealed interface ReplaceOp extends TransformAst permits ReplaceOp.ReplaceThis, ReplaceOp.ByPath {
         record ReplaceThis(JsonValue value) implements ReplaceOp {
-            ReplaceThis {
+            public ReplaceThis {
                 Objects.requireNonNull(value, "value must not be null");
             }
         }
 
         record ByPath(String rawPath, JsonPath path, JsonValue value) implements ReplaceOp {
-            ByPath {
+            public ByPath {
                 Objects.requireNonNull(rawPath, "rawPath must not be null");
                 Objects.requireNonNull(path, "path must not be null");
                 Objects.requireNonNull(value, "value must not be null");
@@ -70,13 +70,13 @@ record ByPath(String rawPath, JsonPath path, JsonValue value) implements Replace
     sealed interface MergeOp extends TransformAst permits MergeOp.MergeThis, MergeOp.ByPath {
 
         record MergeThis(Value value) implements MergeOp {
-            MergeThis {
+            public MergeThis {
                 Objects.requireNonNull(value, "value must not be null");
             }
         }
 
         record ByPath(String rawPath, JsonPath path, Value value) implements MergeOp {
-            ByPath {
+            public ByPath {
                 Objects.requireNonNull(rawPath, "rawPath must not be null");
                 Objects.requireNonNull(path, "path must not be null");
                 Objects.requireNonNull(value, "value must not be null");
@@ -85,13 +85,13 @@ record ByPath(String rawPath, JsonPath path, Value value) implements MergeOp {
 
         sealed interface Value permits Value.Raw, Value.TransformObjectValue {
             record Raw(JsonValue value) implements Value {
-                Raw {
+                public Raw {
                     Objects.requireNonNull(value, "value must not be null");
                 }
             }
 
             record TransformObjectValue(JsonObject rawObject, ObjectTransform compiled) implements Value {
-                TransformObjectValue {
+                public TransformObjectValue {
                     Objects.requireNonNull(rawObject, "rawObject must not be null");
                     Objects.requireNonNull(compiled, "compiled must not be null");
                 }
@@ -101,14 +101,14 @@ record TransformObjectValue(JsonObject rawObject, ObjectTransform compiled) impl
 
     sealed interface RenameOp extends TransformAst permits RenameOp.Mapping, RenameOp.ByPath {
         record Mapping(Map<String, String> renames) implements RenameOp {
-            Mapping {
+            public Mapping {
                 Objects.requireNonNull(renames, "renames must not be null");
                 renames = Map.copyOf(renames);
             }
         }
 
         record ByPath(String rawPath, JsonPath path, String newName) implements RenameOp {
-            ByPath {
+            public ByPath {
                 Objects.requireNonNull(rawPath, "rawPath must not be null");
                 Objects.requireNonNull(path, "path must not be null");
                 Objects.requireNonNull(newName, "newName must not be null");

json-transforms/src/main/java/json/java21/transforms/TransformCompiler.java

@@ -12,8 +12,9 @@
 
 import static json.java21.transforms.TransformSyntax.*;
 
-sealed interface TransformCompiler permits TransformCompiler.Nothing {
-    enum Nothing implements TransformCompiler { INSTANCE }
+final class TransformCompiler {
+
+    private TransformCompiler() {}
 
     static TransformAst.ObjectTransform compileObject(JsonObject transformObject) {
         Objects.requireNonNull(transformObject, "transformObject must not be null");

json-transforms/src/main/java/json/java21/transforms/TransformPatch.java

@@ -8,8 +8,9 @@
 import java.util.List;
 import java.util.Objects;
 
-sealed interface TransformPatch permits TransformPatch.Nothing {
-    enum Nothing implements TransformPatch { INSTANCE }
+final class TransformPatch {
+
+    private TransformPatch() {}
 
     static JsonValue removeAt(JsonValue root, List<JsonPathLocationStep> location) {
         Objects.requireNonNull(root, "root must not be null");

json-transforms/src/main/java/json/java21/transforms/TransformRunner.java

@@ -9,10 +9,11 @@
 
 import static json.java21.transforms.TransformPatch.*;
 
-sealed interface TransformRunner permits TransformRunner.Nothing {
-    enum Nothing implements TransformRunner { INSTANCE }
+final class TransformRunner {
 
-    Logger LOG = Logger.getLogger(TransformRunner.class.getName());
+    private static final Logger LOG = Logger.getLogger(TransformRunner.class.getName());
+
+    private TransformRunner() {}
 
     static JsonValue applyAtDocumentRoot(JsonObject source, TransformAst.ObjectTransform transform) {
         Objects.requireNonNull(source, "source must not be null");

json-transforms/src/main/java/json/java21/transforms/TransformSyntax.java

@@ -2,18 +2,19 @@
 
 import java.util.Objects;
 
-sealed interface TransformSyntax permits TransformSyntax.Nothing {
-    enum Nothing implements TransformSyntax { INSTANCE }
+final class TransformSyntax {
 
-    String SYNTAX_PREFIX = "@jdt.";
+    private TransformSyntax() {}
 
-    String VERB_REMOVE = "@jdt.remove";
-    String VERB_REPLACE = "@jdt.replace";
-    String VERB_MERGE = "@jdt.merge";
-    String VERB_RENAME = "@jdt.rename";
+    static final String SYNTAX_PREFIX = "@jdt.";
 
-    String ATTR_PATH = "@jdt.path";
-    String ATTR_VALUE = "@jdt.value";
+    static final String VERB_REMOVE = "@jdt.remove";
+    static final String VERB_REPLACE = "@jdt.replace";
+    static final String VERB_MERGE = "@jdt.merge";
+    static final String VERB_RENAME = "@jdt.rename";
+
+    static final String ATTR_PATH = "@jdt.path";
+    static final String ATTR_VALUE = "@jdt.value";
 
     static boolean isSyntaxKey(String key) {
         return key != null && key.startsWith(SYNTAX_PREFIX);