fix(github): clean up static_helpers.rs and fix archive bin= option

[jdx]

Summary

This PR combines functional fixes with code cleanup for src/backend/static_helpers.rs:

Functional fixes:

• Add bin= option support for archive extraction (.tar.gz, .zip)
• Fix rename_exe to search bin_path subdirectories when specified
• Handle non-executable files in ZIP archives (chmod before rename)

Refactoring:

• Extract lookup_with_fallback() helper (replaces 9 duplicate patterns)
• Extract should_skip_file() with SKIP_EXTENSIONS/SKIP_FILE_NAMES constants
• Use file::ls() and file::rename() instead of std::fs equivalents
• Add lazy regex compilation with LazyLock for VERSION_PATTERN
• Add documentation to key functions

Registry updates:

• Add GitHub backend for 6 tools: bosh-backup-and-restore, clusterawsadm, helm-diff, k3kcli, mdbook-linkcheck, opsgenie-lamp
• Fix test commands for clusterawsadm and k3kcli to use correct binary names after bin= rename

This supersedes #7348 which had only the functional fixes.

Test plan

• cargo build compiles successfully
• All 8 unit tests in backend::static_helpers pass
• All 6 updated tools pass mise test-tool:
  • bosh-backup-and-restore ✓
  • clusterawsadm ✓
  • helm-diff ✓
  • k3kcli ✓
  • mdbook-linkcheck ✓
  • opsgenie-lamp ✓

🤖 Generated with Claude Code

---

Note

Adds archive bin= support with smarter (bin_path-aware) renaming and ZIP exec handling, refactors option lookups/utilities, and updates registry backends for several tools.

• Backend (static helpers):
  • Archive install: Support bin= for archives; search bin_path subdirs for bin/rename_exe; handle non-executable files in ZIPs by chmod+rename.
  • Refactors: Add lookup_with_fallback(); introduce should_skip_file() with skip lists; switch to file::ls/file::rename; add LazyLock-compiled VERSION_PATTERN; small helpers cleanup (e.g., extension reattachment).
  • Verification: Use platform fallback for checksum/size lookups.
• Registry:
  • Add GitHub backends for bosh-backup-and-restore, clusterawsadm, helm-diff, k3kcli, mdbook-linkcheck, opsgenie-lamp.
  • Switch glab to gitlab: backend; adjust tests/names where needed (e.g., clusterawsadm, k3kcli).

^{Written by Cursor Bugbot for commit 72e37dd. This will update automatically on new commits. Configure here.}

[Copilot]

Pull request overview

This PR refactors src/backend/static_helpers.rs while adding functional fixes for archive extraction. The changes consolidate duplicate code patterns, improve maintainability, and extend support for the bin= option to work with archives.

Key changes:

• Added bin= option support for archive extraction, enabling binaries to be renamed during extraction
• Fixed rename_exe to search within bin_path subdirectories when specified
• Extracted common lookup pattern into lookup_with_fallback() helper function to eliminate code duplication

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The unwrap() call on the regex compilation could panic if the regex pattern is invalid. Consider using expect() with a descriptive message to make debugging easier if this pattern ever needs to be modified: LazyLock::new(|| regex::Regex::new(r\"[-_]v?\\d+(\\.\\d+)*(-[a-zA-Z0-9]+(\\.\\d+)?)?$\").expect(\"VERSION_PATTERN regex must be valid\"))

Suggested change

	LazyLock::new(|| regex::Regex::new(r"[-_]v?\d+(\.\d+)*(-[a-zA-Z0-9]+(\.\d+)?)?$").unwrap());
	LazyLock::new(|| {
	regex::Regex::new(r"[-_]v?\d+(\.\d+)*(-[a-zA-Z0-9]+(\.\d+)?)?$")
	.expect("VERSION_PATTERN regex must be valid")
	});

[Copilot]

The comparison *file_name == *new_name is comparing borrowed strings. This can be simplified to file_name == new_name without the explicit dereferences, which is more idiomatic Rust.

Suggested change

	if file_name.contains(tool_name) || *file_name == *new_name {
	if file_name.contains(tool_name) || file_name == new_name {

[github-actions]

Hyperfine Performance

mise x -- echo

Command	Mean [ms]	Min [ms]	Max [ms]	Relative
mise-2025.12.10 x -- echo	19.9 ± 0.4	19.2	22.2	1.00
mise x -- echo	20.7 ± 0.3	19.7	22.8	1.04 ± 0.03

mise env

Command	Mean [ms]	Min [ms]	Max [ms]	Relative
mise-2025.12.10 env	19.7 ± 0.5	18.9	23.6	1.00
mise env	20.2 ± 0.5	18.8	21.3	1.03 ± 0.04

mise hook-env

Command	Mean [ms]	Min [ms]	Max [ms]	Relative
mise-2025.12.10 hook-env	19.7 ± 0.4	18.9	20.8	1.00
mise hook-env	20.2 ± 0.7	18.8	23.2	1.02 ± 0.04

mise ls

Command	Mean [ms]	Min [ms]	Max [ms]	Relative
mise-2025.12.10 ls	16.7 ± 0.6	16.2	23.7	1.00
mise ls	16.9 ± 0.2	16.3	18.0	1.01 ± 0.04

xtasks/test/perf

Command	mise-2025.12.10	mise	Variance
install (cached)	108ms	108ms	+0%
ls (cached)	66ms	66ms	+0%
bin-paths (cached)	73ms	73ms	+0%
task-ls (cached)	298ms	289ms	+3%

[jdx]

bugbot run