From 81bcda060ab59361861ab007a59b8b281a307590 Mon Sep 17 00:00:00 2001
From: Miles Johnson <mileswjohnson@gmail.com>
Date: Wed, 22 Nov 2023 00:00:43 -0800
Subject: [PATCH 1/2] Fix.

---
 CHANGELOG.md                        |  1 +
 crates/core/src/version_resolver.rs | 35 ++++++++++++++++-------------
 2 files changed, 20 insertions(+), 16 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 3418e333f..f99b31c3e 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -19,6 +19,7 @@
 #### 🐞 Fixes
 
 - Fixed an issue where `proto list-global` would panic when canonicalizing paths.
+- Fixed multi-version ranges (`||`) not resolving locally installed versions correctly.
 
 ## 0.23.2
 
diff --git a/crates/core/src/version_resolver.rs b/crates/core/src/version_resolver.rs
index 0862698c4..cec91cc8c 100644
--- a/crates/core/src/version_resolver.rs
+++ b/crates/core/src/version_resolver.rs
@@ -61,15 +61,15 @@ impl<'tool> VersionResolver<'tool> {
     }
 }
 
-pub fn match_highest_version<'l, I>(req: &'l VersionReq, versions: I) -> Option<VersionSpec>
-where
-    I: IntoIterator<Item = &'l Version>,
-{
+pub fn match_highest_version<'l>(
+    req: &'l VersionReq,
+    versions: &'l [&'l Version],
+) -> Option<VersionSpec> {
     let mut highest_match: Option<Version> = None;
 
     for version in versions {
         if req.matches(version)
-            && (highest_match.is_none() || highest_match.as_ref().is_some_and(|v| version > v))
+            && (highest_match.is_none() || highest_match.as_ref().is_some_and(|v| *version > v))
         {
             highest_match = Some((*version).clone());
         }
@@ -79,7 +79,7 @@ where
 }
 
 // Filter out aliases because they cannot be matched against
-fn extract_installed_versions(installed: &HashSet<VersionSpec>) -> HashSet<&Version> {
+fn extract_installed_versions(installed: &HashSet<VersionSpec>) -> Vec<&Version> {
     installed
         .iter()
         .filter_map(|item| match item {
@@ -95,10 +95,11 @@ pub fn resolve_version(
     aliases: &BTreeMap<String, UnresolvedVersionSpec>,
     manifest: Option<&ToolManifest>,
 ) -> miette::Result<VersionSpec> {
+    let remote_versions = versions.iter().collect::<Vec<_>>();
     let installed_versions = if let Some(manifest) = manifest {
         extract_installed_versions(&manifest.installed_versions)
     } else {
-        HashSet::new()
+        vec![]
     };
 
     match &candidate {
@@ -123,34 +124,36 @@ pub fn resolve_version(
         UnresolvedVersionSpec::Req(req) => {
             // Check locally installed versions first
             if !installed_versions.is_empty() {
-                if let Some(version) = match_highest_version(req, installed_versions) {
+                if let Some(version) = match_highest_version(req, &installed_versions) {
                     return Ok(version);
                 }
             }
 
             // Otherwise we'll need to download from remote
-            if let Some(version) = match_highest_version(req, versions) {
+            if let Some(version) = match_highest_version(req, &remote_versions) {
                 return Ok(version);
             }
         }
         UnresolvedVersionSpec::ReqAny(reqs) => {
-            for req in reqs {
-                // Check locally installed versions first
-                if !installed_versions.is_empty() {
-                    if let Some(version) = match_highest_version(req, installed_versions.clone()) {
+            // Check locally installed versions first
+            if !installed_versions.is_empty() {
+                for req in reqs {
+                    if let Some(version) = match_highest_version(req, &installed_versions) {
                         return Ok(version);
                     }
                 }
+            }
 
-                // Otherwise we'll need to download from remote
-                if let Some(version) = match_highest_version(req, versions) {
+            // Otherwise we'll need to download from remote
+            for req in reqs {
+                if let Some(version) = match_highest_version(req, &remote_versions) {
                     return Ok(version);
                 }
             }
         }
         UnresolvedVersionSpec::Version(ver) => {
             // Check locally installed versions first
-            if installed_versions.contains(ver) {
+            if installed_versions.contains(&ver) {
                 return Ok(VersionSpec::Version(ver.to_owned()));
             }
 

From f1fa128e25d95b857c427e709820c3c5468e73da Mon Sep 17 00:00:00 2001
From: Miles Johnson <mileswjohnson@gmail.com>
Date: Wed, 22 Nov 2023 00:02:32 -0800
Subject: [PATCH 2/2] Polish.

---
 crates/core/src/version_resolver.rs | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/crates/core/src/version_resolver.rs b/crates/core/src/version_resolver.rs
index cec91cc8c..d1bb19c37 100644
--- a/crates/core/src/version_resolver.rs
+++ b/crates/core/src/version_resolver.rs
@@ -61,10 +61,7 @@ impl<'tool> VersionResolver<'tool> {
     }
 }
 
-pub fn match_highest_version<'l>(
-    req: &'l VersionReq,
-    versions: &'l [&'l Version],
-) -> Option<VersionSpec> {
+pub fn match_highest_version(req: &VersionReq, versions: &[&Version]) -> Option<VersionSpec> {
     let mut highest_match: Option<Version> = None;
 
     for version in versions {
@@ -104,7 +101,7 @@ pub fn resolve_version(
 
     match &candidate {
         UnresolvedVersionSpec::Canary => {
-            return Ok(VersionSpec::Alias("canary".into()));
+            return Ok(VersionSpec::Canary);
         }
         UnresolvedVersionSpec::Alias(alias) => {
             let mut alias_value = None;