IJPL-797 intellij.platform.runtime.product review internal API

Constructor is replaced by a factory method in RawRuntimeModuleDescriptor.

GitOrigin-RevId: fda5def0c258cb94b1d960da69439115163b528f

platform/runtime/product/api-dump.txt

@@ -28,22 +28,18 @@
 *:com.intellij.platform.runtime.product.serialization.IncludedProductModulesResolver
 - a:readProductModules(com.intellij.platform.runtime.repository.RuntimeModuleId):java.io.InputStream
 *f:com.intellij.platform.runtime.product.serialization.ProductModulesSerialization
-- <init>():V
 - s:loadProductModules(java.io.InputStream,java.lang.String,com.intellij.platform.runtime.product.ProductMode,com.intellij.platform.runtime.repository.RuntimeModuleRepository):com.intellij.platform.runtime.product.ProductModules
 - s:loadProductModules(java.nio.file.Path,com.intellij.platform.runtime.product.ProductMode,com.intellij.platform.runtime.repository.RuntimeModuleRepository):com.intellij.platform.runtime.product.ProductModules
 - s:readProductModulesAndMergeIncluded(java.io.InputStream,java.lang.String,com.intellij.platform.runtime.product.serialization.IncludedProductModulesResolver):com.intellij.platform.runtime.product.serialization.RawProductModules
 *f:com.intellij.platform.runtime.product.serialization.RawIncludedFromData
-- <init>(com.intellij.platform.runtime.repository.RuntimeModuleId,java.util.Set):V
 - f:getFromModule():com.intellij.platform.runtime.repository.RuntimeModuleId
 - f:getWithoutModules():java.util.Set
 *f:com.intellij.platform.runtime.product.serialization.RawIncludedRuntimeModule
-- <init>(com.intellij.platform.runtime.repository.RuntimeModuleId,com.intellij.platform.runtime.product.ModuleImportance):V
 - getImportance():com.intellij.platform.runtime.product.ModuleImportance
 - getModuleId():com.intellij.platform.runtime.repository.RuntimeModuleId
 - resolve(com.intellij.platform.runtime.repository.RuntimeModuleRepository):com.intellij.platform.runtime.product.IncludedRuntimeModule
 - toString():java.lang.String
 *f:com.intellij.platform.runtime.product.serialization.RawProductModules
-- <init>(java.util.List,java.util.List,java.util.List):V
 - f:getBundledPluginMainModules():java.util.List
 - f:getIncludedFrom():java.util.List
 - f:getMainGroupModules():java.util.List

platform/runtime/product/src/serialization/ProductModulesSerialization.java

@@ -22,6 +22,8 @@ import java.nio.file.Path;
 import java.util.*;
 
 public final class ProductModulesSerialization {
+  private ProductModulesSerialization() { }
+
   public static @NotNull ProductModules loadProductModules(@NotNull Path xmlFile, @NotNull ProductMode currentMode,
                                                            @NotNull RuntimeModuleRepository repository) {
     try {

platform/runtime/product/src/serialization/RawIncludedRuntimeModule.java

@@ -5,6 +5,7 @@ import com.intellij.platform.runtime.product.IncludedRuntimeModule;
 import com.intellij.platform.runtime.product.ModuleImportance;
 import com.intellij.platform.runtime.repository.*;
 import com.intellij.platform.runtime.product.impl.IncludedRuntimeModuleImpl;
+import org.jetbrains.annotations.ApiStatus;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
@@ -12,6 +13,7 @@ public final class RawIncludedRuntimeModule {
   private final RuntimeModuleId myModuleId;
   private final ModuleImportance myImportance;
 
+  @ApiStatus.Internal
   public RawIncludedRuntimeModule(@NotNull RuntimeModuleId moduleId, @NotNull ModuleImportance importance) {
     myModuleId = moduleId;
     myImportance = importance;

platform/runtime/product/src/serialization/RawProductModules.kt

@@ -5,13 +5,13 @@ import com.intellij.platform.runtime.repository.RuntimeModuleId
 import java.io.IOException
 import java.io.InputStream
 
-class RawProductModules(
+class RawProductModules internal constructor(
   val mainGroupModules: List<RawIncludedRuntimeModule>,
   val bundledPluginMainModules: List<RuntimeModuleId>,
   val includedFrom: List<RawIncludedFromData>,
 )
 
-class RawIncludedFromData(
+class RawIncludedFromData internal constructor(
   val fromModule: RuntimeModuleId,
   val withoutModules: Set<RuntimeModuleId>,
 )