Skip to content

registry: add skim - #12239

Merged
jdx merged 3 commits into
jdx:mainfrom
risu729:registry/skim
Aug 21, 2026
Merged

jdx merged 3 commits into
jdx:mainfrom
risu729:registry/skim

Conversation

@risu729

@risu729 risu729 commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • add a skim shorthand with Aqua as the primary backend, followed by GitHub releases and Cargo fallbacks
  • register the sk binary and its version check
  • collapse multiple backend-specific search results into one registry shorthand

Motivation

The Aqua standard registry contains two separate packages for skim: the normal GitHub release package and a Cargo package. Today this produces two Aqua suggestions:

$ mise search skim
Tool                              Description
aqua:skim-rs/skim                 Fuzzy Finder in rust. https://github.com/skim-rs/skim
aqua:crates.io/skim               Fuzzy Finder in rust!. https://github.com/skim-rs/skim

After #12225 translates Aqua registry package types into usable mise backends, the second result becomes cargo:skim, but the search remains duplicated:

aqua:skim-rs/skim
cargo:skim

Adding the skim registry shorthand merges those installation choices under one tool name:

$ mise search skim
skim  Fuzzy Finder in rust. https://github.com/skim-rs/skim

This is the same behavior already provided for overlapping Aqua/Cargo entries such as eza, bat, gitu, and zizmor: search shows the shorthand once, while its registry entry retains the ordered backend choices.

Popularity

  • GitHub: 6,930 stars, 256 forks; latest release v5.6.5 published 2026-08-16
  • crates.io: 1,928,676 total downloads and 159,415 recent downloads
  • Active maintenance: latest repository push 2026-08-17

Validation

  • mise run lint-fix
  • mise run build
  • mise ls-remote aqua:skim-rs/skim
  • mise ls-remote github:skim-rs/skim
  • mise ls-remote cargo:skim
  • target/debug/mise search --match-type equal skim
  • mise x aqua:skim-rs/skim@5.6.5 -- sk --version
  • mise x github:skim-rs/skim@5.6.5 -- sk --version
  • mise x cargo:skim@5.6.5 -- sk --version

AI-assisted — Tool: Codex; model: unavailable/unavailable; version: unavailable.

Summary by CodeRabbit

  • New Features
    • Added the cargo:skim installation backend for the Skim package.
    • Preserved existing binary, description, version-check, and version-ordering details.

@coderabbitai

coderabbitai Bot commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: 36528b87-78ea-482b-ae63-4e787f1df515

📥 Commits

Reviewing files that changed from the base of the PR and between 05a020d and f19c6ab.

📒 Files selected for processing (1)
  • registry/skim.toml

Included review availability: Your plan provides up to 10 included reviews per hour; 5 remain after this review.


📝 Walkthrough

Walkthrough

The change adds the Cargo backend to the skim registry entry. It retains the sk binary, description, version smoke test, and semver ordering.

Changes

Skim registry

Layer / File(s) Summary
Skim package registration
registry/skim.toml
Adds the Cargo backend while retaining Aqua and GitHub metadata, the sk binary, description, version smoke test, and semver ordering.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to f19c6

This localized registry change adds the skim shorthand and backend order without any identified merge-blocking risk; it is merge-ready after normal checks and review.

Poem

A rabbit adds Cargo with a hop,
While sk stays ready at the top.
Aqua and GitHub remain in line,
Semver keeps the versions fine,
And registry carrots never stop.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: adding skim to the registry.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@greptile-apps

greptile-apps Bot commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds a skim registry shorthand backed by Aqua, GitHub releases, and Cargo.

  • Registers the sk executable.
  • Adds version validation using sk --version.
  • Uses semantic-version ordering for supported backends.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
registry/skim.toml Adds registry metadata for resolving and validating the skim fuzzy finder through three ordered backends.

Reviews (3): Last reviewed commit: "registry: add cargo fallback for skim" | Re-trigger Greptile

@risu729
risu729 marked this pull request as ready for review August 21, 2026 05:54
@jdx
jdx merged commit 59cb157 into jdx:main Aug 21, 2026
31 checks passed
@risu729
risu729 deleted the registry/skim branch August 21, 2026 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants