fix(stubs): lookup for aqua tools stubs fails because of tool options

mise-en-dev · mise-en-dev · commit 0c56957a7970 · 2025-11-02T17:10:59.000+01:00
- tool options introduced with PR #6807 lead to lookup failures in context of aqua and tool stubs
diff --git a/e2e/cli/test_tool_stub_basic b/e2e/cli/test_tool_stub_basic
@@ -84,3 +84,17 @@ assert_contains "bin/gh --version" "gh version"
 assert_contains "bin/gh --version" "gh version" # short-circuit works
 mise uninstall --all
 assert_contains "bin/gh --version" "gh version" # refetches even if it's cached
+
+# Test 12: Aqua backend
+cat >bin/flyctl <<'EOF'
+#!/usr/bin/env -S mise tool-stub
+tool = "flyctl"
+version = "0.3.206"
+bin = "flyctl"
+EOF
+
+chmod +x bin/flyctl
+
+assert_contains "bin/flyctl version" "flyctl v0.3.206"
+mise uninstall --all
+assert_contains "bin/flyctl version" "flyctl v0.3.206"
diff --git a/src/backend/aqua.rs b/src/backend/aqua.rs
@@ -247,7 +247,7 @@ impl Backend for AquaBackend {
 
 impl AquaBackend {
     pub fn from_arg(ba: BackendArg) -> Self {
-        let full = ba.full();
+        let full = ba.full_without_opts();
         let mut id = full.split_once(":").unwrap_or(("", &full)).1;
         if !id.contains("/") {
             id = REGISTRY
diff --git a/src/cli/args/backend_arg.rs b/src/cli/args/backend_arg.rs
@@ -273,6 +273,14 @@ impl BackendArg {
         full
     }
 
+    pub fn full_without_opts(&self) -> String {
+        let full = self.full();
+        if let Some(c) = regex!(r"^(.+)\[(.+)\]$").captures(&full) {
+            return c.get(1).unwrap().as_str().to_string();
+        }
+        full
+    }
+
     pub fn opts(&self) -> ToolVersionOptions {
         self.opts.clone().unwrap_or_else(|| {
             if let Some(c) = regex!(r"^(.+)\[(.+)\]$").captures(&self.full()) {
