Small refactors

Author: FastestMolasses

CodeEdit/Features/LSP/Registry/PackageManagers/CargoPackageManager.swift

@@ -35,8 +35,6 @@ final class CargoPackageManager: PackageManagerProtocol {
         }
 
         let packagePath = installationDirectory.appending(path: source.entryName)
-        print("Installing \(source.entryName)@\(source.version) in \(packagePath.path)")
-
         try await initialize(in: packagePath)
 
         do {
@@ -63,9 +61,7 @@ final class CargoPackageManager: PackageManagerProtocol {
             }
 
             _ = try await executeInDirectory(in: packagePath.path, cargoArgs)
-            print("Successfully installed \(source.entryName)@\(source.version)")
         } catch {
-            print("Installation failed: \(error)")
             throw error
         }
     }
@@ -82,7 +78,6 @@ final class CargoPackageManager: PackageManagerProtocol {
             }
             return output.starts(with: "cargo")
         } catch {
-            print("Cargo version check failed: \(error)")
             return false
         }
     }

CodeEdit/Features/LSP/Registry/PackageManagers/GithubPackageManager.swift

@@ -59,7 +59,6 @@ final class GithubPackageManager: PackageManagerProtocol {
             }
             return output.contains("git version")
         } catch {
-            print("Git version check failed: \(error)")
             return false
         }
     }
@@ -73,7 +72,7 @@ final class GithubPackageManager: PackageManagerProtocol {
         if !FileManager.default.fileExists(atPath: packagePath.path) {
             throw RegistryManagerError.downloadFailed(
                 url: url,
-                error: NSError(domain: "Coould not download package", code: -1)
+                error: NSError(domain: "Could not download package", code: -1)
             )
         }
 
@@ -89,7 +88,6 @@ final class GithubPackageManager: PackageManagerProtocol {
             let repoPath = installPath.appending(path: source.pkgName, directoryHint: .isDirectory)
             _ = try await executeInDirectory(in: repoPath.path, [command])
         } catch {
-            print("Failed to build from source: \(error)")
             throw PackageManagerError.installationFailed("Source build failed.")
         }
     }

CodeEdit/Features/LSP/Registry/PackageManagers/GolangPackageManager.swift

@@ -43,8 +43,6 @@ final class GolangPackageManager: PackageManagerProtocol {
         }
 
         let packagePath = installationDirectory.appending(path: source.entryName)
-        print("Installing Go package \(source.entryName)@\(source.version) in \(packagePath.path)")
-
         try await initialize(in: packagePath)
 
         do {
@@ -78,10 +76,7 @@ final class GolangPackageManager: PackageManagerProtocol {
                 let execPath = packagePath.appending(path: "bin").appending(path: binaryName).path
                 _ = try await runCommand("chmod +x \"\(execPath)\"")
             }
-
-            print("Successfully installed \(source.entryName)@\(source.version)")
         } catch {
-            print("Installation failed: \(error)")
             try? cleanupFailedInstallation(packagePath: packagePath)
             throw PackageManagerError.installationFailed(error.localizedDescription)
         }
@@ -108,7 +103,6 @@ final class GolangPackageManager: PackageManagerProtocol {
             }
             return output.range(of: versionPattern, options: .regularExpression) != nil
         } catch {
-            print("Go version check failed: \(error)")
             return false
         }
     }

CodeEdit/Features/LSP/Registry/PackageManagers/NPMPackageManager.swift

@@ -56,8 +56,6 @@ final class NPMPackageManager: PackageManagerProtocol {
         }
 
         let packagePath = installationDirectory.appending(path: source.entryName)
-        print("Installing \(source.entryName)@\(source.version) in \(packagePath.path)")
-
         try await initialize(in: packagePath)
 
         do {
@@ -73,10 +71,7 @@ final class NPMPackageManager: PackageManagerProtocol {
 
             _ = try await executeInDirectory(in: packagePath.path, installArgs)
             try verifyInstallation(folderName: source.entryName, package: source.pkgName, version: source.version)
-
-            print("Successfully installed \(source.entryName)@\(source.version)")
         } catch {
-            print("Installation failed: \(error)")
             let nodeModulesPath = packagePath.appending(path: "node_modules").path
             try? FileManager.default.removeItem(atPath: nodeModulesPath)
             throw error

CodeEdit/Features/LSP/Registry/PackageManagers/PipPackageManager.swift

@@ -43,8 +43,6 @@ final class PipPackageManager: PackageManagerProtocol {
         }
 
         let packagePath = installationDirectory.appending(path: source.entryName)
-        print("Installing \(source.entryName)@\(source.version) in \(packagePath.path)")
-
         try await initialize(in: packagePath)
 
         do {
@@ -58,7 +56,7 @@ final class PipPackageManager: PackageManagerProtocol {
             }
 
             let extras = source.options["extra"]
-            if let extras = extras {
+            if let extras {
                 if let lastIndex = installArgs.indices.last {
                     installArgs[lastIndex] += "[\(extras)]"
                 }
@@ -67,10 +65,7 @@ final class PipPackageManager: PackageManagerProtocol {
             _ = try await executeInDirectory(in: packagePath.path, installArgs)
             try await updateRequirements(packagePath: packagePath)
             try await verifyInstallation(packagePath: packagePath, package: source.pkgName)
-
-            print("Successfully installed \(source.entryName)@\(source.version)")
         } catch {
-            print("Installation failed: \(error)")
             throw error
         }
     }