refactor: extract validation into separate function following DOT principle

Co-authored-by: yCodeTech <31927084+yCodeTech@users.noreply.github.com>

Author: Copilot

src/extensionData.ts

@@ -95,7 +95,6 @@ export class ExtensionData {
 	 */
 	private setExtensionDiscoveryPaths() {
 		// Get the DEV_USER_EXTENSIONS_PATH env variable if it exists.
-		// This variable is validated and sanitized by addDevEnvVariables() in utils.ts
 		const devUserExtensionsPath = process.env.DEV_USER_EXTENSIONS_PATH;
 
 		// The path to the user extensions.

src/utils.ts

@@ -240,17 +240,10 @@ export function mergeArraysBy<T>(primaryArray: T[], secondaryArray: T[], key: ke
 }
 
 /**
- * Add development environment variables from a local .env file located in the project root.
- * Validates and sanitizes specific environment variables after loading.
+ * Validate and sanitize the DEV_USER_EXTENSIONS_PATH environment variable.
+ * Removes invalid paths from the environment with logged errors.
  */
-export function addDevEnvVariables() {
-	// Try to load the local .env file from the project root.
-	try {
-		process.loadEnvFile(path.join(__dirname, "../../.env"));
-	} catch (error) {
-		// Ignore errors if the .env file doesn't exist
-	}
-
+function validateDevEnvVariables() {
 	// Validate DEV_USER_EXTENSIONS_PATH if it was loaded
 	if (process.env.DEV_USER_EXTENSIONS_PATH) {
 		let devPath = process.env.DEV_USER_EXTENSIONS_PATH;
@@ -287,3 +280,18 @@ export function addDevEnvVariables() {
 		}
 	}
 }
+
+/**
+ * Add development environment variables from a local .env file located in the project root.
+ */
+export function addDevEnvVariables() {
+	// Try to load the local .env file from the project root.
+	try {
+		process.loadEnvFile(path.join(__dirname, "../../.env"));
+	} catch (error) {
+		// Ignore errors if the .env file doesn't exist
+	}
+
+	// Validate the loaded environment variables
+	validateDevEnvVariables();
+}