Some cleanup and Javadocs

src/main/java/net/neoforged/moddevgradle/internal/ArtifactManifestEntry.java

@@ -7,6 +7,11 @@
 import java.io.Serializable;
 import java.util.Objects;
 
+/**
+ * Represents an entry in the Artifact manifest given to NFRT to prevent re-downloading of
+ * dependencies that Gradle has already downloaded to its cache.
+ * It uses the same format used in NeoForm/NeoForge configuration data to identify an artifact (Maven format).
+ */
 final class ArtifactManifestEntry implements Serializable {
     @Input
     private final String artifactId;

src/main/java/net/neoforged/moddevgradle/internal/CreateArtifactManifestTask.java

@@ -12,6 +12,10 @@
 import java.io.IOException;
 import java.util.Properties;
 
+/**
+ * This task creates a properties file for NFRT that maps artifact ids found in
+ * NeoForge/NeoForm configuration data to existing files on disk.
+ */
 abstract class CreateArtifactManifestTask extends DefaultTask {
     @Input
     abstract SetProperty<ArtifactManifestEntry> getNeoForgeModDevArtifacts();

src/main/java/net/neoforged/moddevgradle/internal/CreateMinecraftArtifactsTask.java

@@ -12,6 +12,10 @@
 import java.util.ArrayList;
 import java.util.Collections;
 
+/**
+ * The primary task for creating the Minecraft artifacts that mods will be compiled against,
+ * using the NFRT CLI.
+ */
 @DisableCachingByDefault(because = "Implements its own caching")
 abstract class CreateMinecraftArtifactsTask extends NeoFormRuntimeEngineTask {
     @Inject

src/main/java/net/neoforged/moddevgradle/internal/DistributionDisambiguation.java

@@ -3,6 +3,10 @@
 import org.gradle.api.attributes.AttributeDisambiguationRule;
 import org.gradle.api.attributes.MultipleCandidatesDetails;
 
+/**
+ * We generally will use "client" dependencies when we have to decide between client and server,
+ * since client libraries will usually be a superset of the server libraries.
+ */
 public abstract class DistributionDisambiguation implements AttributeDisambiguationRule<String> {
     @Override
     public void execute(MultipleCandidatesDetails<String> details) {

src/main/java/net/neoforged/moddevgradle/internal/DownloadAssetsTask.java

@@ -7,6 +7,10 @@
 import javax.inject.Inject;
 import java.util.List;
 
+/**
+ * Use the NFRT CLI to download the asset index and assets for the Minecraft version used by the
+ * underlying NeoForge/NeoForm configuration.
+ */
 abstract class DownloadAssetsTask extends NeoFormRuntimeEngineTask {
     @Inject
     public DownloadAssetsTask() {

src/main/java/net/neoforged/moddevgradle/internal/ModDevPlugin.java

@@ -68,6 +68,9 @@
 import java.util.function.Consumer;
 import java.util.stream.Collectors;
 
+/**
+ * The main plugin class.
+ */
 public class ModDevPlugin implements Plugin<Project> {
     private static final Attribute<String> ATTRIBUTE_DISTRIBUTION = Attribute.of("net.neoforged.distribution", String.class);
     private static final Attribute<String> ATTRIBUTE_OPERATING_SYSTEM = Attribute.of("net.neoforged.operatingsystem", String.class);

src/main/java/net/neoforged/moddevgradle/internal/NeoFormRuntimeTask.java

@@ -20,7 +20,7 @@
 import java.util.List;
 
 /**
- * Base task implementation for running NFRT.
+ * Base task implementation for running the NFRT CLI, regardless of which sub-command is used.
  */
 abstract public class NeoFormRuntimeTask extends DefaultTask {
 

src/main/java/net/neoforged/moddevgradle/internal/OperatingSystemDisambiguation.java

@@ -5,6 +5,9 @@
 import org.gradle.api.attributes.AttributeDisambiguationRule;
 import org.gradle.api.attributes.MultipleCandidatesDetails;
 
+/**
+ * This disambiguation rule will select native dependencies based on the operating system Gradle is currently running on.
+ */
 public abstract class OperatingSystemDisambiguation implements AttributeDisambiguationRule<String> {
     @Override
     public void execute(MultipleCandidatesDetails<String> details) {

src/main/java/net/neoforged/moddevgradle/internal/PrepareRun.java

@@ -8,6 +8,9 @@
 
 import javax.inject.Inject;
 
+/**
+ * Prepare all files needed to run a particular flavor of the game.
+ */
 abstract class PrepareRun extends PrepareRunOrTest {
     @Input
     public abstract Property<String> getRunType();