registry: update aube and pitchfork aqua entries#10285
Conversation
📝 WalkthroughWalkthroughRepository ownership for the aube tool is transferred from ChangesRegistry and Tool Configuration Updates
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Greptile SummaryTransfers ownership of the
Confidence Score: 5/5Safe to merge — changes are purely registry metadata (TOML and YAML config), with no executable code paths affected. The diff is limited to two config files: a one-line backend swap in aube.toml and coordinated updates to the vendored aqua registry YAML. The ownership-transfer alias pattern matches how pitchfork was previously handled, the new <= 1.18.1 version override block mirrors the existing catch-all block (minus attestation), and the signer workflow path update is consistent with the repo rename. No files require special attention. Important Files Changed
Reviews (1): Last reviewed commit: "registry: update aube and pitchfork aqua..." | Re-trigger Greptile |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@registry/aube.toml`:
- Line 1: Backends list uses "aqua:jdx/aube" which will fail because
vendor/aqua-registry/registry.yml has no name: jdx/aube entry (only
endevco/aube); either add a registry alias entry for name: jdx/aube in
vendor/aqua-registry/registry.yml pointing to the existing aube entry, or change
the backends array in the registry/aube.toml to prefer "github:jdx/aube" first
(or remove the aqua entry) so deployments don't attempt the missing aqua lookup;
update the backends array values ("aqua:jdx/aube" / "github:jdx/aube") or add
the name: jdx/aube alias in the registry file accordingly.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: e70e0b19-2ee4-4f94-afcf-a2dd55cac44a
📒 Files selected for processing (2)
registry/aube.tomlvendor/aqua-registry/registry.yml
| @@ -1,3 +1,3 @@ | |||
| backends = ["aqua:endevco/aube", "github:endevco/aube", "cargo:aube"] | |||
| backends = ["aqua:jdx/aube", "github:jdx/aube", "cargo:aube"] | |||
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Verify jdx/aube is accessible in GitHub and check aqua registry entry
# Check GitHub repository exists and has releases
echo "Checking GitHub repository..."
gh repo view jdx/aube --json nameWithOwner,url,isArchived,latestRelease
# Check if aqua registry has jdx/aube entry
echo -e "\nChecking aqua registry for jdx/aube..."
rg -A 20 'name: jdx/aube' vendor/aqua-registry/registry.yml || echo "Entry not found in vendored registry"
# Verify backward compatibility alias for endevco/aube if mentioned
echo -e "\nChecking for endevco/aube alias..."
rg 'endevco/aube' vendor/aqua-registry/registry.yml || echo "No endevco/aube alias found"Repository: jdx/mise
Length of output: 477
Backends update risks aqua-registry lookup failure for jdx/aube.
GitHub jdx/aube exists (latest release v1.18.2), but vendor/aqua-registry/registry.yml has no name: jdx/aube entry; only an endevco/aube alias is present. This means deployments may fail the first backend (aqua:jdx/aube) before falling back to github:jdx/aube (so ensure the GitHub backend works in your install environment and that aqua registry gets updated).
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@registry/aube.toml` at line 1, Backends list uses "aqua:jdx/aube" which will
fail because vendor/aqua-registry/registry.yml has no name: jdx/aube entry (only
endevco/aube); either add a registry alias entry for name: jdx/aube in
vendor/aqua-registry/registry.yml pointing to the existing aube entry, or change
the backends array in the registry/aube.toml to prefer "github:jdx/aube" first
(or remove the aqua entry) so deployments don't attempt the missing aqua lookup;
update the backends array values ("aqua:jdx/aube" / "github:jdx/aube") or add
the name: jdx/aube alias in the registry file accordingly.
Hyperfine Performance
|
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
mise-2026.6.1 x -- echo |
19.4 ± 1.2 | 17.2 | 25.1 | 1.00 |
mise x -- echo |
22.2 ± 2.3 | 18.9 | 47.3 | 1.15 ± 0.14 |
x -- echo measured 15% slower, but hyperfine reported statistical outliers. |
mise env
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
mise-2026.6.1 env |
19.4 ± 1.4 | 16.8 | 25.0 | 1.00 |
mise env |
20.6 ± 1.5 | 18.0 | 27.3 | 1.06 ± 0.11 |
mise hook-env
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
mise-2026.6.1 hook-env |
20.5 ± 1.5 | 18.0 | 25.3 | 1.00 |
mise hook-env |
22.3 ± 1.4 | 19.5 | 27.6 | 1.09 ± 0.10 |
mise ls
| Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
|---|---|---|---|---|
mise-2026.6.1 ls |
16.3 ± 1.1 | 13.8 | 20.4 | 1.00 |
mise ls |
17.8 ± 1.2 | 15.6 | 24.3 | 1.09 ± 0.10 |
xtasks/test/perf
| Command | mise-2026.6.1 | mise | Variance |
|---|---|---|---|
| install (cached) | 142ms | 145ms | -2% |
| ls (cached) | 63ms | 63ms | +0% |
| bin-paths (cached) | 67ms | 71ms | -5% |
| task-ls (cached) | 135ms | 138ms | -2% |
1 participant
Summary
aqua:jdx/aubeandgithub:jdx/aubejdx/aubeandjdx/pitchforkendevco/aubeandendevco/pitchforkaliases for compatibility<= 1.18.1, matching fix(aube): update moved repository owner aquaproj/aqua-registry#55084Popularity
Testing
taplo fmt --check registry/aube.tomlruby -e 'require "yaml"; YAML.load_file("vendor/aqua-registry/registry.yml", aliases: true); puts "vendor yaml ok"'\n-ruby -e 'require "yaml"; data = YAML.load_file("vendor/aqua-registry/registry.yml", aliases: true); pkgs = data.fetch("packages"); %w[aube pitchfork].each { |name| p pkgs.find { |pkg| pkg["repo_owner"] == "jdx" && pkg["repo_name"] == name }.slice("repo_owner", "repo_name", "aliases") }'\n-git diff --checkNote
Low Risk
Registry and vendored install metadata only; no runtime or auth changes, with backward-compatible aliases.
Overview
Moves aube and pitchfork install metadata from
endevcotojdxas the canonical GitHub owner, while keepingendevco/*aqua aliases so old references still resolve.In
registry/aube.toml, aqua/github backends now point atjdx/aube. The vendored aqua registry mirrors upstream: both packages userepo_owner: jdx, aliases swap toendevco/..., and aube gains asemver("<= 1.18.1")override (glibc/musl Linux variants, no GitHub artifact attestations). Newer aube releases use attestation signer workflowjdx/aube/...instead ofendevco/aube/....Reviewed by Cursor Bugbot for commit 4839c04. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by CodeRabbit