Add missing period terminators to line comments

Author: ctrueden

src/main/java/org/apposed/appose/GroovyWorker.java

@@ -183,9 +183,9 @@ public static void main(String... args) {
 					Binding binding = new Binding();
 					GroovyShell shell = new GroovyShell(binding);
 					shell.evaluate(initCode);
-					// Store all variables from the init script for use in tasks
+					// Store all variables from the init script for use in tasks.
 					initVars.putAll(binding.getVariables());
-					// Clean up the temp file
+					// Clean up the temp file.
 					initFile.delete();
 				}
 				catch (Exception e) {

src/main/java/org/apposed/appose/builder/MambaBuilder.java

@@ -70,7 +70,7 @@ public String name() {
 	public Environment build() throws IOException {
 		File envDir = envDir();
 
-		// Check for incompatible existing environments
+		// Check for incompatible existing environments.
 		if (new File(envDir, ".pixi").isDirectory()) {
 			throw new IOException("Cannot use MambaBuilder: environment already managed by Pixi at " + envDir);
 		}
@@ -81,16 +81,16 @@ public Environment build() throws IOException {
 		// Is this envDir an already-existing conda directory?
 		boolean isCondaDir = new File(envDir, "conda-meta").isDirectory();
 		if (isCondaDir) {
-			// Environment already exists, just wrap it
+			// Environment already exists, just wrap it.
 			return createEnvironment(envDir);
 		}
 
-		// Building a new environment - config content is required
+		// Building a new environment - config content is required.
 		if (sourceContent == null) {
 			throw new IllegalStateException("No source specified for MambaBuilder. Use .file() or .content()");
 		}
 
-		// Infer scheme if not explicitly set
+		// Infer scheme if not explicitly set.
 		if (scheme == null) scheme = Schemes.fromContent(sourceContent).name();
 
 		if (!"environment.yml".equals(scheme)) {
@@ -110,7 +110,7 @@ public Environment build() throws IOException {
 		mamba.setEnvVars(envVars);
 		mamba.setFlags(flags);
 
-		// Check for unsupported features
+		// Check for unsupported features.
 		if (!channels.isEmpty()) {
 			throw new UnsupportedOperationException(
 				"MambaBuilder does not yet support programmatic channel configuration. " +
@@ -120,15 +120,15 @@ public Environment build() throws IOException {
 		try {
 			mamba.install();
 
-			// Two-step build: create empty env, write config, then update
-			// Step 1: Create empty environment
+			// Two-step build: create empty env, write config, then update.
+			// Step 1: Create empty environment.
 			mamba.create(envDir);
 
-			// Step 2: Write environment.yml to envDir
+			// Step 2: Write environment.yml to envDir.
 			File envYaml = new File(envDir, "environment.yml");
 			Files.write(envYaml.toPath(), sourceContent.getBytes(StandardCharsets.UTF_8));
 
-			// Step 3: Update environment from yml
+			// Step 3: Update environment from yml.
 			mamba.update(envDir, envYaml);
 
 			return createEnvironment(envDir);
@@ -141,14 +141,14 @@ public Environment build() throws IOException {
 	public Environment wrap(File envDir) throws IOException {
 		FilePaths.ensureDirectory(envDir);
 
-		// Look for environment.yml configuration file
+		// Look for environment.yml configuration file.
 		File envYaml = new File(envDir, "environment.yml");
 		if (envYaml.exists() && envYaml.isFile()) {
-			// Read the content so rebuild() will work even after directory is deleted
+			// Read the content so rebuild() will work even after directory is deleted.
 			sourceContent = new String(Files.readAllBytes(envYaml.toPath()), StandardCharsets.UTF_8);
 		}
 
-		// Set the base directory and build (which will detect existing env)
+		// Set the base directory and build (which will detect existing env).
 		base(envDir);
 		return build();
 	}

src/main/java/org/apposed/appose/builder/MambaBuilderFactory.java

@@ -40,6 +40,7 @@
  * @author Curtis Rueden
  */
 public class MambaBuilderFactory implements BuilderFactory {
+
 	@Override
 	public Builder<?> createBuilder() {
 		return new MambaBuilder();
@@ -67,7 +68,7 @@ public boolean supportsScheme(String scheme) {
 
 	@Override
 	public boolean supportsSource(String source) {
-		// Support YAML environment files
+		// Support YAML environment files.
 		return source.endsWith(".yml") || source.endsWith(".yaml");
 	}
 
@@ -78,7 +79,7 @@ public double priority() {
 
 	@Override
 	public boolean canWrap(java.io.File envDir) {
-		// Check for conda/mamba environment marker
+		// Check for conda/mamba environment marker.
 		return new java.io.File(envDir, "conda-meta").isDirectory();
 	}
 }

src/main/java/org/apposed/appose/builder/PixiBuilder.java

@@ -98,7 +98,7 @@ public String name() {
 	public Environment build() throws IOException {
 		File envDir = envDir();
 
-		// Check for incompatible existing environments
+		// Check for incompatible existing environments.
 		if (new File(envDir, "conda-meta").exists() && !new File(envDir, ".pixi").exists()) {
 			throw new IOException("Cannot use PixiBuilder: environment already managed by Mamba/Conda at " + envDir);
 		}
@@ -122,53 +122,53 @@ public Environment build() throws IOException {
 		try {
 			pixi.install();
 
-			// Check if this is already a pixi project
+			// Check if this is already a pixi project.
 			boolean isPixiDir = new File(envDir, "pixi.toml").isFile() ||
 			                    new File(envDir, "pyproject.toml").isFile() ||
 			                    new File(envDir, ".pixi").isDirectory();
 
 			if (isPixiDir && sourceContent == null && condaPackages.isEmpty() && pypiPackages.isEmpty()) {
-				// Environment already exists, just use it
+				// Environment already exists, just use it.
 				return createEnvironment(pixi, envDir);
 			}
 
-			// Handle source-based build (file or content)
+			// Handle source-based build (file or content).
 			if (sourceContent != null) {
 				if (isPixiDir) {
-					// Already initialized, just use it
+					// Already initialized, just use it.
 					return createEnvironment(pixi, envDir);
 				}
 
-				// Infer scheme if not explicitly set
+				// Infer scheme if not explicitly set.
 				if (scheme == null) scheme = Schemes.fromContent(sourceContent).name();
 
 				if (!envDir.exists() && !envDir.mkdirs()) {
 					throw new IOException("Failed to create environment directory: " + envDir);
 				}
 
 				if ("pixi.toml".equals(scheme)) {
-					// Write pixi.toml to envDir
+					// Write pixi.toml to envDir.
 					File pixiTomlFile = new File(envDir, "pixi.toml");
 					Files.write(pixiTomlFile.toPath(), sourceContent.getBytes(StandardCharsets.UTF_8));
 				} else if ("pyproject.toml".equals(scheme)) {
-					// Write pyproject.toml to envDir (Pixi natively supports it)
+					// Write pyproject.toml to envDir (Pixi natively supports it).
 					File pyprojectTomlFile = new File(envDir, "pyproject.toml");
 					Files.write(pyprojectTomlFile.toPath(), sourceContent.getBytes(StandardCharsets.UTF_8));
 				} else if ("environment.yml".equals(scheme)) {
-					// Write environment.yml and import
+					// Write environment.yml and import.
 					File environmentYamlFile = new File(envDir, "environment.yml");
 					Files.write(environmentYamlFile.toPath(), sourceContent.getBytes(StandardCharsets.UTF_8));
 					pixi.exec("init", "--import", environmentYamlFile.getAbsolutePath(), envDir.getAbsolutePath());
 				}
 
-				// Add any programmatic channels to augment source file
+				// Add any programmatic channels to augment source file.
 				if (!channels.isEmpty()) {
 					pixi.addChannels(envDir, channels.toArray(new String[0]));
 				}
 			} else {
-				// Programmatic package building
+				// Programmatic package building.
 				if (isPixiDir) {
-					// Already initialized, just use it
+					// Already initialized, just use it.
 					return createEnvironment(pixi, envDir);
 				}
 
@@ -178,7 +178,7 @@ public Environment build() throws IOException {
 
 				pixi.init(envDir);
 
-				// Fail fast for vacuous environments
+				// Fail fast for vacuous environments.
 				if (condaPackages.isEmpty() && pypiPackages.isEmpty()) {
 					throw new IllegalStateException(
 						"Cannot build empty environment programmatically. " +
@@ -226,23 +226,23 @@ public Environment build() throws IOException {
 	public Environment wrap(File envDir) throws IOException {
 		FilePaths.ensureDirectory(envDir);
 
-		// Look for pixi.toml configuration file first
+		// Look for pixi.toml configuration file first.
 		File pixiToml = new File(envDir, "pixi.toml");
 		if (pixiToml.exists() && pixiToml.isFile()) {
-			// Read the content so rebuild() will work even after directory is deleted
+			// Read the content so rebuild() will work even after directory is deleted.
 			sourceContent = new String(Files.readAllBytes(pixiToml.toPath()), StandardCharsets.UTF_8);
 			scheme = "pixi.toml";
 		} else {
-			// Check for pyproject.toml
+			// Check for pyproject.toml.
 			File pyprojectToml = new File(envDir, "pyproject.toml");
 			if (pyprojectToml.exists() && pyprojectToml.isFile()) {
-				// Read the content so rebuild() will work even after directory is deleted
+				// Read the content so rebuild() will work even after directory is deleted.
 				sourceContent = new String(Files.readAllBytes(pyprojectToml.toPath()), StandardCharsets.UTF_8);
 				scheme = "pyproject.toml";
 			}
 		}
 
-		// Set the base directory and build (which will detect existing env)
+		// Set the base directory and build (which will detect existing env).
 		base(envDir);
 		return build();
 	}
@@ -262,7 +262,7 @@ public PixiBuilder channels(String... channels) {
 
 	private Environment createEnvironment(Pixi pixi, File envDir) {
 		String base = envDir.getAbsolutePath();
-		// Check which manifest file exists (pyproject.toml takes precedence)
+		// Check which manifest file exists (pyproject.toml takes precedence).
 		File manifestFile = new File(envDir, "pyproject.toml");
 		if (!manifestFile.exists()) manifestFile = new File(envDir, "pixi.toml");
 		List<String> launchArgs = Arrays.asList(

src/main/java/org/apposed/appose/builder/PixiBuilderFactory.java

@@ -75,7 +75,7 @@ public boolean supportsScheme(String scheme) {
 
 	@Override
 	public boolean supportsSource(String source) {
-		// Support pixi.toml and YAML environment files
+		// Support pixi.toml and YAML environment files.
 		return source.endsWith("pixi.toml") ||
 		       source.endsWith(".yml") ||
 		       source.endsWith(".yaml");
@@ -88,7 +88,7 @@ public double priority() {
 
 	@Override
 	public boolean canWrap(java.io.File envDir) {
-		// Check for pixi environment markers
+		// Check for pixi environment markers.
 		return new java.io.File(envDir, ".pixi").isDirectory() ||
 		       new java.io.File(envDir, "pixi.toml").isFile();
 	}

src/main/java/org/apposed/appose/builder/SimpleBuilder.java

@@ -107,7 +107,7 @@ public SimpleBuilder inheritRunningJava() {
 		if (javaHome != null) {
 			File javaHomeBin = new File(javaHome, "bin");
 			if (javaHomeBin.isDirectory()) {
-				// Prepend to beginning of list for highest priority
+				// Prepend to beginning of list for highest priority.
 				customBinPaths.add(0, javaHomeBin.getAbsolutePath());
 			}
 			envVars.put("JAVA_HOME", javaHome);
@@ -127,7 +127,7 @@ public Environment build() throws IOException {
 		File base = envDir();
 		if (base == null) base = new File(".");
 
-		// Create directory if it doesn't exist
+		// Create directory if it doesn't exist.
 		if (!base.exists() && !base.mkdirs()) {
 			throw new IOException("Failed to create base directory: " + base);
 		}
@@ -136,13 +136,13 @@ public Environment build() throws IOException {
 		List<String> launchArgs = new ArrayList<>();
 		List<String> binPaths = new ArrayList<>();
 
-		// Add bin directory from the environment itself (highest priority)
+		// Add bin directory from the environment itself (highest priority).
 		File binDir = new File(base, "bin");
 		if (binDir.isDirectory()) {
 			binPaths.add(binDir.getAbsolutePath());
 		}
 
-		// Add custom binary paths configured via builder methods
+		// Add custom binary paths configured via builder methods.
 		binPaths.addAll(customBinPaths);
 
 		Map<String, String> environmentVars = new HashMap<>(envVars);

src/main/java/org/apposed/appose/builder/Uv.java

@@ -146,46 +146,46 @@ protected void decompress(final File archive) throws IOException, InterruptedExc
 		if (!uvBinDir.exists() && !uvBinDir.mkdirs())
 			throw new IOException("Failed to create UV bin directory: " + uvBinDir);
 
-		// Extract archive
+		// Extract archive.
 		Downloads.unpack(archive, uvBinDir);
 
 		String uvBinaryName = Platforms.isWindows() ? "uv.exe" : "uv";
 		File uvDest = new File(command);
 
-		// Check if uv binary is directly in bin dir (Windows ZIP case)
+		// Check if uv binary is directly in bin dir (Windows ZIP case).
 		File uvDirectly = new File(uvBinDir, uvBinaryName);
 		if (uvDirectly.exists()) {
-			// Windows case: binaries are directly in uvBinDir
-			// Just ensure uv.exe is in the right place (uvCommand)
+			// Windows case: binaries are directly in uvBinDir.
+			// Just ensure uv.exe is in the right place (uvCommand).
 			if (!uvDirectly.equals(uvDest) && !uvDirectly.renameTo(uvDest)) {
 				throw new IOException("Failed to move uv binary from " + uvDirectly + " to " + uvDest);
 			}
-			// uvw.exe and uvx.exe are already in the right place (uvBinDir)
+			// uvw.exe and uvx.exe are already in the right place (uvBinDir).
 		} else {
-			// Linux/macOS case: binaries are in uv-<platform>/ subdirectory
+			// Linux/macOS case: binaries are in uv-<platform>/ subdirectory.
 			File[] platformDirs = uvBinDir.listFiles(f -> f.isDirectory() && f.getName().startsWith("uv-"));
 			if (platformDirs == null || platformDirs.length == 0) {
 				throw new IOException("Expected uv binary or uv-<platform> directory not found in: " + uvBinDir);
 			}
 
 			File platformDir = platformDirs[0];
 
-			// Move all binaries from platform subdirectory to bin directory
+			// Move all binaries from platform subdirectory to bin directory.
 			File[] binaries = platformDir.listFiles();
 			if (binaries != null) {
 				for (File binary : binaries) {
 					File dest = new File(uvBinDir, binary.getName());
 					if (!binary.renameTo(dest)) {
 						throw new IOException("Failed to move " + binary.getName() + " from " + binary + " to " + dest);
 					}
-					// Set executable permission
+					// Set executable permission.
 					if (!dest.canExecute()) {
 						dest.setExecutable(true);
 					}
 				}
 			}
 
-			// Clean up the now-empty platform directory
+			// Clean up the now-empty platform directory.
 			if (!platformDir.delete()) {
 				throw new IOException("Failed to delete platform directory: " + platformDir);
 			}