feat: update proxy binaries on self-update

Signed-off-by: Christopher Arndt <[email protected]>

Author: SpotlightKid

src/choosenim.nim

@@ -168,6 +168,10 @@ proc updateSelf(params: CliParams) =
   display("Info:", "Updated choosenim to version " & $version,
           Success, HighPriority)
 
+  # Any Nim installation currently activated by choosenim?
+  if $getCurrentVersion(params) != "":
+    discard installProxies(params)
+
 proc update(params: CliParams) =
   if params.commands.len != 2:
     raise newException(ChooseNimError,

src/choosenimpkg/switcher.nim

@@ -232,6 +232,31 @@ proc writeProxy(bin: string, params: CliParams) =
     display("Hint:", "Ensure that '$1' is before '$2' in the PATH env var." %
             [params.getBinDir(), fromPATH.splitFile.dir], Warning, HighPriority)
 
+proc installProxies*(params: CliParams): bool =
+  ## Install or update proxy executables.
+  ##
+  ## Returns `true` when any proxies were newly installed or updated,
+  ## `false` otherwise.
+  var proxiesToInstall = @proxies
+
+  # Handle MingW proxies.
+  when defined(windows):
+    if not isDefaultCCInPath(params):
+      let mingwBin = getMingwBin(params)
+      if not fileExists(mingwBin / "gcc".addFileExt(ExeExt)):
+        let msg = "No 'gcc' binary found in '$1'." % mingwBin
+        raise newException(ChooseNimError, msg)
+
+      proxiesToInstall.add(mingwProxies)
+
+  if not params.areProxiesInstalled(proxiesToInstall):
+    # Create the proxy executables.
+    for proxy in proxiesToInstall:
+      writeProxy(proxy, params)
+    return true
+
+  return false
+
 proc switchToPath(filepath: string, params: CliParams): bool =
   ## Switches to the specified file path that should point to the root of
   ## the Nim repo.
@@ -254,30 +279,17 @@ proc switchToPath(filepath: string, params: CliParams): bool =
                      "running `choosenim \"#v0.16.0\"` or similar.",
             Warning, HighPriority)
 
-  var proxiesToInstall = @proxies
-  # Handle MingW proxies.
-  when defined(windows):
-    if not isDefaultCCInPath(params):
-      let mingwBin = getMingwBin(params)
-      if not fileExists(mingwBin / "gcc".addFileExt(ExeExt)):
-        let msg = "No 'gcc' binary found in '$1'." % mingwBin
-        raise newException(ChooseNimError, msg)
-
-      proxiesToInstall.add(mingwProxies)
-
   # Return early if this version is already selected.
-  let selectedPath = params.getSelectedPath()
-  let proxiesInstalled = params.areProxiesInstalled(proxiesToInstall)
-  if selectedPath == filepath and proxiesInstalled:
+  let
+    selectedPath = params.getSelectedPath()
+    proxiesInstalled = installProxies(params)
+
+  if selectedPath == filepath and not proxiesInstalled:
     return false
   else:
     # Write selected path to "current file".
     writeFile(params.getCurrentFile(), filepath)
 
-  # Create the proxy executables.
-  for proxy in proxiesToInstall:
-    writeProxy(proxy, params)
-
   when defined(windows):
     if not isNimbleBinInPath(params):
       display("Hint:", "Use 'choosenim <version/channel> --firstInstall' to add\n" &