fix(github): use alias-specific options when tool_alias has its own config (#8892)

## Summary
- Fixed regression where `tool_alias` with per-alias `asset_pattern` (or
other options) would use the original tool's options instead of the
alias-specific ones
- Changed `get_tool_opts()` to prefer the alias-specific config entry
when it has options set, falling back to the resolved name match for
CLI-created requests without options (preserves the treesize/gitlab fix
from #8084)
- Added e2e test verifying both the original tool and its alias install
with their respective asset patterns

Fixes #8847

## Test plan
- [x] `cargo test` — all 563 unit tests pass
- [x] `mise run test:e2e test_github_tool_alias_asset_pattern` — new e2e
test passes
- [x] All lint hooks pass

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Medium Risk**
> Changes tool option resolution for aliased backends, which can affect
how tools are downloaded/installed (e.g., `asset_pattern`) for some
configurations; scope is localized but impacts installation behavior.
> 
> **Overview**
> Fixes a regression where a `tool_alias` could incorrectly inherit the
original tool’s option set (notably `asset_pattern`) instead of using
the alias-specific config.
> 
> `Config::get_tool_opts()` now considers both the alias short-name
match and the resolved full-name match and selects the entry that
actually has options, preferring the alias-specific entry when present
while preserving the fallback for resolved-name configs.
> 
> Adds an e2e regression test that configures a GitHub tool and its
alias with different `asset_pattern` values and verifies both install
using their respective assets.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
7f9ce83. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

Author: 3 people

e2e/backend/test_github_tool_alias_asset_pattern

@@ -0,0 +1,28 @@
+#!/usr/bin/env bash
+# Test that tool_alias with asset_pattern uses the alias-specific options
+# Regression test for https://github.com/jdx/mise/discussions/8847
+# When both the original tool and an aliased tool are configured with different
+# asset_pattern values, each should use its own pattern.
+
+cat <<'EOF' >mise.toml
+[tool_alias]
+hw2 = "github:jdx/mise-test-fixtures"
+
+[tools."github:jdx/mise-test-fixtures"]
+version = "1.0.0"
+asset_pattern = "hello-world-1.0.0.tar.gz"
+bin_path = "hello-world-1.0.0/bin"
+postinstall = "chmod +x $MISE_TOOL_INSTALL_PATH/hello-world-1.0.0/bin/hello-world"
+
+[tools.hw2]
+version = "1.0.0"
+asset_pattern = "hello-world-2.0.0.tar.gz"
+bin_path = "hello-world-2.0.0/bin"
+postinstall = "chmod +x $MISE_TOOL_INSTALL_PATH/hello-world-2.0.0/bin/hello-world"
+EOF
+
+mise install
+# Both should install successfully with the correct asset
+assert_contains "mise x -- hello-world" "hello world"
+# The aliased tool should have installed using hello-world-2.0.0.tar.gz
+assert "test -d ~/.local/share/mise/installs/hw2/1.0.0/hello-world-2.0.0"

src/config/mod.rs

@@ -351,17 +351,34 @@ impl Config {
         backend_arg: &Arc<BackendArg>,
     ) -> Result<Option<ToolVersionOptions>> {
         let trs = self.get_tool_request_set().await?;
-        // Try matching by resolved full name first for aliased tools.
+        let short_match = trs.iter().find(|tr| tr.0.short == backend_arg.short);
+        // Also try matching by resolved full name for aliased tools.
         // e.g., ba.short="treesize" resolves to full="gitlab:FBibonne/treesize"
         // while the config entry has short="gitlab-f-bibonne-treesize" with api_url set.
-        // We check the resolved name first because the direct short match might find
-        // a CLI-created tool request without options.
         let full = backend_arg.full();
         let resolved_ba = BackendArg::new(full, None);
-        let tool_request = trs
-            .iter()
-            .find(|tr| tr.0.short == resolved_ba.short)
-            .or_else(|| trs.iter().find(|tr| tr.0.short == backend_arg.short));
+        let resolved_match = trs.iter().find(|tr| tr.0.short == resolved_ba.short);
+
+        let has_opts = |tr: &(&Arc<BackendArg>, &Vec<crate::toolset::ToolRequest>, _)| -> bool {
+            tr.1.first()
+                .is_some_and(|req| !req.options().opts.is_empty())
+        };
+        // Prefer whichever match has options set. When both have options,
+        // prefer the short (alias-specific) match since it's more specific.
+        // Fall back to the resolved match for cases where a CLI-created
+        // request without options shadows the config entry (treesize case).
+        let tool_request = match (short_match, resolved_match) {
+            (Some(s), Some(r)) => {
+                if has_opts(&s) {
+                    Some(s)
+                } else {
+                    Some(r)
+                }
+            }
+            (Some(s), None) => Some(s),
+            (None, Some(r)) => Some(r),
+            (None, None) => None,
+        };
         Ok(tool_request.and_then(|tr| tr.1.first().map(|req| req.options())))
     }