Use List.copyOf

Author: mpkorstanje

codegen/templates/java.java.erb

@@ -5,11 +5,9 @@ package io.cucumber.messages.types;
 import org.jspecify.annotations.Nullable;
 
 import java.util.List;
-import java.util.ArrayList;
 import java.util.Objects;
 import java.util.Optional;
 
-import static java.util.Collections.unmodifiableList;
 import static java.util.Objects.requireNonNull;
 
 /**
@@ -35,7 +33,7 @@ public final class <%= class_name(key) %> {
         <%- schema['properties'].each_with_index do |(property_name_2, _property_2), index| -%>
           <%- if property_name_2 == property_name -%>
             <%- if property['items'] -%>
-            unmodifiableList(new ArrayList<>(requireNonNull(<%= property_name %>, "<%= class_name(key) %>.<%= property_name %> cannot be null")))<%= index < schema['properties'].length - 1 ? ',' : '' %>
+            List.copyOf(requireNonNull(<%= property_name %>, "<%= class_name(key) %>.<%= property_name %> cannot be null"))<%= index < schema['properties'].length - 1 ? ',' : '' %>
             <%- else -%>
             requireNonNull(<%= property_name %>, "<%= class_name(key) %>.<%= property_name %> cannot be null")<%= index < schema['properties'].length - 1 ? ',' : '' %>
             <%- end -%>
@@ -60,13 +58,13 @@ public final class <%= class_name(key) %> {
       -%>
       <%- if required -%>
        <%- if property['items'] -%>
-        this.<%= property_name %> = unmodifiableList(new ArrayList<>(requireNonNull(<%= property_name %>, "<%= class_name(key) %>.<%= property_name %> cannot be null")));
+        this.<%= property_name %> = List.copyOf(requireNonNull(<%= property_name %>, "<%= class_name(key) %>.<%= property_name %> cannot be null"));
        <%- else -%>
         this.<%= property_name %> = requireNonNull(<%= property_name %>, "<%= class_name(key) %>.<%= property_name %> cannot be null");
        <%- end -%>
       <%- else -%>
        <%- if property['items'] -%>
-        this.<%= property_name %> = <%= property_name %> == null ? null : unmodifiableList(new ArrayList<>(<%= property_name %>));
+        this.<%= property_name %> = <%= property_name %> == null ? null : List.copyOf(<%= property_name %>);
        <%- else -%>
         this.<%= property_name %> = <%= property_name %>;
        <%- end -%>

java/src/generated/java/io/cucumber/messages/types/Attachment.java

@@ -3,11 +3,9 @@
 import org.jspecify.annotations.Nullable;
 
 import java.util.List;
-import java.util.ArrayList;
 import java.util.Objects;
 import java.util.Optional;
 
-import static java.util.Collections.unmodifiableList;
 import static java.util.Objects.requireNonNull;
 
 /**

java/src/generated/java/io/cucumber/messages/types/Background.java

@@ -3,11 +3,9 @@
 import org.jspecify.annotations.Nullable;
 
 import java.util.List;
-import java.util.ArrayList;
 import java.util.Objects;
 import java.util.Optional;
 
-import static java.util.Collections.unmodifiableList;
 import static java.util.Objects.requireNonNull;
 
 /**
@@ -35,7 +33,7 @@ public Background(
         this.keyword = requireNonNull(keyword, "Background.keyword cannot be null");
         this.name = requireNonNull(name, "Background.name cannot be null");
         this.description = requireNonNull(description, "Background.description cannot be null");
-        this.steps = unmodifiableList(new ArrayList<>(requireNonNull(steps, "Background.steps cannot be null")));
+        this.steps = List.copyOf(requireNonNull(steps, "Background.steps cannot be null"));
         this.id = requireNonNull(id, "Background.id cannot be null");
     }
 

java/src/generated/java/io/cucumber/messages/types/Ci.java

@@ -3,11 +3,9 @@
 import org.jspecify.annotations.Nullable;
 
 import java.util.List;
-import java.util.ArrayList;
 import java.util.Objects;
 import java.util.Optional;
 
-import static java.util.Collections.unmodifiableList;
 import static java.util.Objects.requireNonNull;
 
 /**

java/src/generated/java/io/cucumber/messages/types/Comment.java

@@ -3,11 +3,9 @@
 import org.jspecify.annotations.Nullable;
 
 import java.util.List;
-import java.util.ArrayList;
 import java.util.Objects;
 import java.util.Optional;
 
-import static java.util.Collections.unmodifiableList;
 import static java.util.Objects.requireNonNull;
 
 /**

java/src/generated/java/io/cucumber/messages/types/DataTable.java

@@ -3,11 +3,9 @@
 import org.jspecify.annotations.Nullable;
 
 import java.util.List;
-import java.util.ArrayList;
 import java.util.Objects;
 import java.util.Optional;
 
-import static java.util.Collections.unmodifiableList;
 import static java.util.Objects.requireNonNull;
 
 /**
@@ -24,7 +22,7 @@ public DataTable(
         List<TableRow> rows
     ) {
         this.location = requireNonNull(location, "DataTable.location cannot be null");
-        this.rows = unmodifiableList(new ArrayList<>(requireNonNull(rows, "DataTable.rows cannot be null")));
+        this.rows = List.copyOf(requireNonNull(rows, "DataTable.rows cannot be null"));
     }
 
     public Location getLocation() {

java/src/generated/java/io/cucumber/messages/types/DocString.java

@@ -3,11 +3,9 @@
 import org.jspecify.annotations.Nullable;
 
 import java.util.List;
-import java.util.ArrayList;
 import java.util.Objects;
 import java.util.Optional;
 
-import static java.util.Collections.unmodifiableList;
 import static java.util.Objects.requireNonNull;
 
 /**

java/src/generated/java/io/cucumber/messages/types/Duration.java

@@ -3,11 +3,9 @@
 import org.jspecify.annotations.Nullable;
 
 import java.util.List;
-import java.util.ArrayList;
 import java.util.Objects;
 import java.util.Optional;
 
-import static java.util.Collections.unmodifiableList;
 import static java.util.Objects.requireNonNull;
 
 /**

java/src/generated/java/io/cucumber/messages/types/Envelope.java

@@ -3,11 +3,9 @@
 import org.jspecify.annotations.Nullable;
 
 import java.util.List;
-import java.util.ArrayList;
 import java.util.Objects;
 import java.util.Optional;
 
-import static java.util.Collections.unmodifiableList;
 import static java.util.Objects.requireNonNull;
 
 /**

java/src/generated/java/io/cucumber/messages/types/Examples.java

@@ -3,11 +3,9 @@
 import org.jspecify.annotations.Nullable;
 
 import java.util.List;
-import java.util.ArrayList;
 import java.util.Objects;
 import java.util.Optional;
 
-import static java.util.Collections.unmodifiableList;
 import static java.util.Objects.requireNonNull;
 
 /**
@@ -36,12 +34,12 @@ public Examples(
         String id
     ) {
         this.location = requireNonNull(location, "Examples.location cannot be null");
-        this.tags = unmodifiableList(new ArrayList<>(requireNonNull(tags, "Examples.tags cannot be null")));
+        this.tags = List.copyOf(requireNonNull(tags, "Examples.tags cannot be null"));
         this.keyword = requireNonNull(keyword, "Examples.keyword cannot be null");
         this.name = requireNonNull(name, "Examples.name cannot be null");
         this.description = requireNonNull(description, "Examples.description cannot be null");
         this.tableHeader = tableHeader;
-        this.tableBody = unmodifiableList(new ArrayList<>(requireNonNull(tableBody, "Examples.tableBody cannot be null")));
+        this.tableBody = List.copyOf(requireNonNull(tableBody, "Examples.tableBody cannot be null"));
         this.id = requireNonNull(id, "Examples.id cannot be null");
     }