Notify swiftly is unlinked when performing use, install and update

Author: plemarquand

Sources/Swiftly/Install.swift

@@ -86,6 +86,7 @@ struct Install: SwiftlyCommand {
         defer {
             versionUpdateReminder()
         }
+        try await valitateLinked(ctx)
 
         var config = try await Config.load(ctx)
         let toolchainVersion = try await Self.determineToolchainVersion(ctx, version: self.version, config: &config)

Sources/Swiftly/Link.swift

@@ -44,10 +44,14 @@ struct Link: SwiftlyCommand {
 
         if pathChanged {
             await ctx.print("""
-            Linked swiftly to \(toolchainVersion.name).
+            Linked swiftly to Swift \(toolchainVersion.name).
 
             \(Messages.refreshShell)
             """)
+        } else {
+            await ctx.print("""
+            Swiftly is already linked to Swift \(toolchainVersion.name).
+            """)
         }
     }
 }

Sources/Swiftly/Unlink.swift

@@ -32,25 +32,14 @@ struct Unlink: SwiftlyCommand {
         }
 
         var pathChanged = false
-        if let proxyTo = try? await Swiftly.currentPlatform.findSwiftlyBin(ctx) {
-            let swiftlyBinDir = Swiftly.currentPlatform.swiftlyBinDir(ctx)
-            let swiftlyBinDirContents = (try? await fs.ls(atPath: swiftlyBinDir)) ?? [String]()
-
-            var existingProxies = [String]()
-            for bin in swiftlyBinDirContents {
-                let linkTarget = try? await fs.readlink(atPath: swiftlyBinDir / bin)
-                if linkTarget == proxyTo {
-                    existingProxies.append(bin)
-                }
-            }
+        let existingProxies = try await symlinkedProxies(ctx)
 
-            for p in existingProxies {
-                let proxy = Swiftly.currentPlatform.swiftlyBinDir(ctx) / p
+        for p in existingProxies {
+            let proxy = Swiftly.currentPlatform.swiftlyBinDir(ctx) / p
 
-                if try await fs.exists(atPath: proxy) {
-                    try await fs.remove(atPath: proxy)
-                    pathChanged = true
-                }
+            if try await fs.exists(atPath: proxy) {
+                try await fs.remove(atPath: proxy)
+                pathChanged = true
             }
         }
 
@@ -60,3 +49,27 @@ struct Unlink: SwiftlyCommand {
         }
     }
 }
+
+func symlinkedProxies(_ ctx: SwiftlyCoreContext) async throws -> [String] {
+    if let proxyTo = try? await Swiftly.currentPlatform.findSwiftlyBin(ctx) {
+        let swiftlyBinDir = Swiftly.currentPlatform.swiftlyBinDir(ctx)
+        let swiftlyBinDirContents = (try? await fs.ls(atPath: swiftlyBinDir)) ?? [String]()
+        var proxies = [String]()
+        for bin in swiftlyBinDirContents {
+            let linkTarget = try? await fs.readlink(atPath: swiftlyBinDir / bin)
+            if linkTarget == proxyTo {
+                proxies.append(bin)
+            }
+        }
+        return proxies
+    }
+    return []
+}
+
+extension SwiftlyCommand {
+    func valitateLinked(_ ctx: SwiftlyCoreContext) async throws {
+        if try await symlinkedProxies(ctx).isEmpty {
+            await ctx.print(Messages.currentlyUnlinked)
+        }
+    }
+}

Sources/Swiftly/Update.swift

@@ -87,9 +87,9 @@ struct Update: SwiftlyCommand {
         defer {
             versionUpdateReminder()
         }
+        try await valitateLinked(ctx)
 
         var config = try await Config.load(ctx)
-
         guard let parameters = try await self.resolveUpdateParameters(ctx, &config) else {
             if let toolchain = self.toolchain {
                 await ctx.print("No installed toolchain matched \"\(toolchain)\"")

Sources/Swiftly/Use.swift

@@ -67,6 +67,8 @@ struct Use: SwiftlyCommand {
 
         var config = try await Config.load(ctx)
 
+        try await valitateLinked(ctx)
+
         // This is the bare use command where we print the selected toolchain version (or the path to it)
         guard let toolchain = self.toolchain else {
             let (selectedVersion, result) = try await selectToolchain(ctx, config: &config, globalDefault: self.globalDefault)
@@ -113,6 +115,7 @@ struct Use: SwiftlyCommand {
         }
 
         try await Self.execute(ctx, toolchain, globalDefault: self.globalDefault, assumeYes: self.root.assumeYes, &config)
+
     }
 
     /// Use a toolchain. This method can modify and save the input config and also create/modify a `.swift-version` file.

Sources/SwiftlyCore/Messages.swift

@@ -10,10 +10,21 @@ public enum Messages {
     """
 
     public static let unlinkSuccess = """
-    Swiftly is now unlinked and will not manage the active toolchain until the following command is run:
+    Swiftly is now unlinked and will not manage the active toolchain until the following
+    command is run:
 
         $ swiftly link
 
+
+    """
+
+    public static let currentlyUnlinked = """
+    Swiftly is currently unlinked and will not manage the active toolchain. You can run
+    the following command to link swiftly to the active toolchain:
+
+        $ swiftly link
+
+
     """
 
     public static func postInstall(_ command: String) -> String {