feat(spawn-jdk): add AddExports, AddOpens, AddReads, and PackageName option types

TODO.md

@@ -2,8 +2,6 @@
 # NOW
 ------------------------------------------------------------------------------------------------------------------------
 
-* Refactor/Replace Docker okhttp client with Java HTTP Client
-
 * Introduce Debugging support
 
 * TODO: Create issues for the remaining JDK features/options (eg: JMX, Garbage Collection, Logging, Flight Recorder, Early Access, Preview etc)
@@ -17,15 +15,6 @@
 
 * Introduce Composition Tests
 
-* FIX! Introduce AddReads option to support representing "--add-reads" (and inheritance)
-* FIX! Introduce AddOpens option to support representing "--add-opens" (and inheritance)
-
-eg: to mimic this stuff that's added automatically!
-
-//            JDKOption.of("--add-reads"), JDKOption.of("build.spawn.platform.local.jdk=ALL-UNNAMED"),
-//            JDKOption.of("--add-opens"),
-//            JDKOption.of("build.spawn.platform.local.jdk/build.spawn.platform.local.jdk=ALL-UNNAMED"),
-
 ------------------------------------------------------------------------------------------------------------------------
 # LATER
 ------------------------------------------------------------------------------------------------------------------------

spawn-jdk/src/main/java/build/spawn/jdk/JDKApplication.java

@@ -30,7 +30,10 @@
 import build.spawn.application.Platform;
 import build.spawn.application.option.ApplicationSubscriber;
 import build.spawn.application.option.LaunchIdentity;
+import build.spawn.jdk.option.AddExports;
 import build.spawn.jdk.option.AddModules;
+import build.spawn.jdk.option.AddOpens;
+import build.spawn.jdk.option.AddReads;
 import build.spawn.jdk.option.ClassPath;
 import build.spawn.jdk.option.Headless;
 import build.spawn.jdk.option.Jar;
@@ -112,6 +115,15 @@ public void onPreparing(final Platform platform,
 
                     PatchModule.detect()
                         .forEach(options::add);
+
+                    AddExports.detect()
+                        .forEach(options::add);
+
+                    AddOpens.detect()
+                        .forEach(options::add);
+
+                    AddReads.detect()
+                        .forEach(options::add);
                 }
             }
 

spawn-jdk/src/main/java/build/spawn/jdk/option/AddExports.java

@@ -0,0 +1,179 @@
+package build.spawn.jdk.option;
+
+/*-
+ * #%L
+ * Spawn JDK
+ * %%
+ * Copyright (C) 2026 Workday, Inc.
+ * %%
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * #L%
+ */
+
+import build.base.configuration.CollectedOption;
+import build.base.configuration.ConfigurationBuilder;
+import build.base.option.JDKVersion;
+import build.base.table.Cell;
+import build.base.table.Table;
+import build.base.table.Tabular;
+import build.base.table.option.CellSeparator;
+import build.base.table.option.TableName;
+import build.spawn.application.Platform;
+
+import java.lang.management.ManagementFactory;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Objects;
+import java.util.Optional;
+import java.util.function.Supplier;
+import java.util.stream.Stream;
+
+/**
+ * A {@link JDKOption} to specify a {@code --add-exports=source-module/package=target-module}.
+ *
+ * @author reed.vonredwitz
+ * @since May-2026
+ */
+public class AddExports
+    implements JDKOption, CollectedOption<List>, Tabular {
+
+    /**
+     * The source {@link ModuleName} that owns the package.
+     */
+    private final ModuleName sourceModule;
+
+    /**
+     * The {@link PackageName} to export.
+     */
+    private final PackageName packageName;
+
+    /**
+     * The target {@link ModuleName} to export to.
+     */
+    private final ModuleName targetModule;
+
+    /**
+     * Constructs an {@link AddExports}.
+     *
+     * @param sourceModule the source {@link ModuleName}
+     * @param packageName  the {@link PackageName}
+     * @param targetModule the target {@link ModuleName}
+     */
+    private AddExports(final ModuleName sourceModule,
+                       final PackageName packageName,
+                       final ModuleName targetModule) {
+        this.sourceModule = Objects.requireNonNull(sourceModule, "The source ModuleName must not be null");
+        this.packageName = Objects.requireNonNull(packageName, "The PackageName must not be null");
+        this.targetModule = Objects.requireNonNull(targetModule, "The target ModuleName must not be null");
+    }
+
+    @Override
+    public boolean isSupported(final JDKVersion jdkVersion,
+                               final ConfigurationBuilder options) {
+
+        return jdkVersion.isModular();
+    }
+
+    @Override
+    public Stream<String> resolve(final Platform platform,
+                                  final ConfigurationBuilder options) {
+
+        final var arguments = new ArrayList<String>(2);
+
+        arguments.add("--add-exports");
+        arguments.add(this.sourceModule.get() + "/" + this.packageName.get() + "=" + this.targetModule.get());
+
+        return arguments.stream();
+    }
+
+    @Override
+    public Optional<Supplier<Table>> getTableSupplier() {
+        return Optional.of(() -> {
+            final Table table = Table.create();
+            table.options().add(TableName.of("Add Exports"));
+            table.options().add(CellSeparator.of("="));
+            return table;
+        });
+    }
+
+    @Override
+    public void tabularize(final Table table) {
+        table.addRow(
+            Cell.of(this.sourceModule.get() + "/" + this.packageName.get()),
+            Cell.of(this.targetModule.get()));
+    }
+
+    @Override
+    public boolean equals(final Object object) {
+        if (this == object) {
+            return true;
+        }
+        if (!(object instanceof final AddExports that)) {
+            return false;
+        }
+        return Objects.equals(this.sourceModule, that.sourceModule)
+            && Objects.equals(this.packageName, that.packageName)
+            && Objects.equals(this.targetModule, that.targetModule);
+    }
+
+    @Override
+    public int hashCode() {
+        return Objects.hash(this.sourceModule, this.packageName, this.targetModule);
+    }
+
+    /**
+     * Creates an {@link AddExports}.
+     *
+     * @param sourceModule the source {@link ModuleName}
+     * @param packageName  the {@link PackageName}
+     * @param targetModule the target {@link ModuleName}
+     * @return a new {@link AddExports}
+     */
+    public static AddExports of(final ModuleName sourceModule,
+                                final PackageName packageName,
+                                final ModuleName targetModule) {
+        return new AddExports(sourceModule, packageName, targetModule);
+    }
+
+    /**
+     * Creates an {@link AddExports}.
+     *
+     * @param sourceModule the source module name
+     * @param packageName  the package name
+     * @param targetModule the target module name
+     * @return a new {@link AddExports}
+     */
+    public static AddExports of(final String sourceModule,
+                                final String packageName,
+                                final String targetModule) {
+        return new AddExports(ModuleName.of(sourceModule), PackageName.of(packageName), ModuleName.of(targetModule));
+    }
+
+    /**
+     * Detects the {@link Stream} of {@link AddExports} directives for this Virtual Machine.
+     *
+     * @return the {@link Stream} of {@link AddExports}
+     */
+    public static Stream<AddExports> detect() {
+        return ManagementFactory.getRuntimeMXBean()
+            .getInputArguments()
+            .stream()
+            .filter(argument -> argument.startsWith("--add-exports="))
+            .map(argument -> argument.substring("--add-exports=".length()))
+            .map(value -> value.split("=", 2))
+            .map(parts -> {
+                final var moduleAndPackage = parts[0].split("/", 2);
+                return AddExports.of(moduleAndPackage[0], moduleAndPackage[1], parts[1]);
+            });
+    }
+}